Fix checkstyle warnings.

- org.opendaylight.openflowplugin.api.openflow.md.core

Change-Id: Ida0bf4149aaa70657a22fb527708d61b0d7d43b3
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Fix checkstyle warnings

- org.opendaylight.openflowplugin.api.openflow.md.core

Change-Id: I98490f3a4054bcfdb1339ac490d432ece0539097
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Merge "Bug 7915 - Zero flows populated in all switches when connected to Leader Node"

Bug 7349 - Flow ID not updated in operational after removing and adding a flow with same match

Change-Id: Ie370cf017bd8642c52f1469cc0b01b10fa38842d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 66c19f4bbb00399056dce40db7d3312375558bc1)

Bug 7915 - Zero flows populated in all switches when connected to Leader Node

Change-Id: I8e8df5d6cb51c114b23f8690d3d6f01fef37f81b
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "Add arbitrary mask for nxm-reg"

Merge "Fix statistics race condition on big flows"

Merge "Fix checkstyle warnings"

Merge "Fix checkstyle warnings"

Merge "Fix checkstyle warnings"

Merge "Remove drop-test feature from He design"

Bug 6110: Fixed bugs in statistics manager due to race condition.

  * Stats notification listener needs to wait for the XID to be cached.
  * Enqueue DS operation after all notifications are received.

Change-Id: I42ac315a65be1a1f02152fbd9ea9510bee586eb3
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Bug 7916: MatchConvertorImpl detailed IllegalStateException instead NPE"

Fix checkstyle warnings

- org.opendaylight.openflowplugin.api.openflow.lifecycle

Change-Id: Iaa95b9c49441adc943a4c510e1583ac92146ee63
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Remove drop-test feature from He design

Change-Id: I3c9ffc1aa0d105601ceaa427f2bc043324d8e296
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Fix checkstyle warnings

- org.opendylight.openflowplugin.api.openflow.device

Change-Id: I775fd56bee4a1d80b2c2ace37f32836c993c5b97
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Add arbitrary mask for nxm-reg

According to specification, nxm-reg should be fully maskable
using arbitrary mask.

Resolves: bug 7897

Change-Id: I912017e6bd33813ac65ed5799c8bfa7ebf217889
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Fix checkstyle warnings

- org.opendaylight.openflowplugin.api.openflow.connection

Change-Id: Ib061f1fd2dbb6da1cd24e23e1846d79e41026f3e
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Fix statistics race condition on big flows

Fix race condition that happens when we receive statistics faster
than flow is deleted from device, but we already sent request for
deletion and removed this flow from device registy.

Resolves: bug 6917

Change-Id: I95510fbc5d507eaffc34d58b3b7743e60911b541
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Fix checkstyle warnings

- org.opendaylight.openflowplugin.api

Change-Id: I7b24e696af69505163bbd515ffa7db3b32ca497d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Merge "Remove old drop-test module and files."

Merge "Implement core API's methods."

Merge "Define new API for context chain and context chain holder."

Merge "Add module and feature for ONF extensions"

Bug 7910 - Flow with ethernet mask (ff:ff:ff:ff:ff:ff),
get stored under alien-id in operational data store

Custom comparator for ethernet mask address is modified
to take care of this issue.

Change-Id: I0c5c1dbd61dd1a3fb83d3b16695553649c4ecf8f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "Add configuration knob for Single Layer"

Merge "Turn Single Layer Serialization on by default"

Implement core API's methods.

Change-Id: I3005c82e2cdea0cf719c5f500041ad3ad9ce4480
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Define new API for context chain and context chain holder.

Change-Id: I1ad94bab7b5ff3d98dee939420b2f28d3eb36261
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Revert "Use Karaf 4 for the Karaf distribution"

This reverts commit c14a28f3a6f7f095d597fb7354cbc317b3c3100a.

Change-Id: I6f9c29023380ff7942ebdb3e903d6cfbc2071f6d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Bug 7916: MatchConvertorImpl detailed IllegalStateException instead NPE

This won't fix bug 7916, but perhaps it can help understand the root
cause of it, for a future follow-up actual fix.

Change-Id: I3a006bdc38fec1f2b194a6d9d637c920ddd6b742
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Add configuration knob for Single Layer

Add configuration knob to openflowplugin.cfg that will enable/disable
use-single-layer-serialization.

Change-Id: I9473a80495060aaa252757ba4d0b1450d252b08e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Turn Single Layer Serialization on by default

Change leaf user-single-layer-serialization to "true" to turn
Single Layer Serialization on by default.
More informations about this change can be found
here: http://ow.ly/NNJu309qSRv.

Change-Id: I00846d0776a64c160e8fff122b30cfc620eb79b0
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

BUG-6859 - Binding generator v1 refactoring

Fix compilation failure after binding generator move.

Change-Id: I93a34904054aff4f9f0766ea35a25731adb60e3a
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Add module and feature for ONF extensions

Change-Id: I5d983bfa5f01b98efd68d8094e519fa50905dce7
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Update .gitingore file

- netbeans local configuration file.

Change-Id: I5b97eb0f926e6a3b65781c934954319daa3877b1
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Use Karaf 4 for the Karaf distribution

Change-Id: I6d8ce21866b4ffd5c25590076ad1042dba4e7346
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Add Karaf 4 features.

Change-Id: I31781439d7b5cba76f6e6a52109b13b2546ee924
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Replace odl-dlux-core with odl-dluxapps-topology"

Merge "Convert Functions to private methods"

Merge "Fix some FRM checkstyle violations"

Merge "Fix comparison of match extensions"

Convert Functions to private methods

Using method handles allows us to eliminate explicit dependency
on Function and simplify declaration.

Change-Id: I8317fd3f37c569612b8d865a39871eb09dd30c81
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix some FRM checkstyle violations

- whitespace around generic arguments
- unneeded public modifiers
- line length

Change-Id: I3febf13cf3f01e043f2406b20b57f9e37a6103b4
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix connection closing on switch IDLE state

When device is disconnected, and connection is not equal primary
connection context, do not shut down all services and just remove
this connection context.

Resolves: bug 7689

Change-Id: I3c6149a44f266b423fdc4f6396022ad4c650e872
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Bug 7736 - Forwarding Rules application cluster singleton id should not use the same cluster singleton id as the openflow switch singleton connection handler

Change-Id: Ie82b2d7d23444927375ed5b63f5e765f3277ed00
Signed-off-by: Jon Castro <castro.jon@gmail.com>

Fix comparison of match extensions

Add comparison between match extension lists to MatchComparatorFactory
to treat matches with different extensions as unique.

Change-Id: Ia43322d5e3ca3ee62e8313fd1286a6a9c68f7423
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Merge changes from topic 'ofj-models-to-ofp-models'

* changes:
  Add single layer deserialization support
  Fix errors in serializers and deserializers
  Add multipart reply deserializers unit tests
  Update AbstractService to support isComplete
  Add multipart reply deserializers
  Add PortMessageDeserializer
  Add MeterMessageDeserializer
  Add GroupMessageDeserializer
  Add FlowMessageDeserializer
  Add Instruction deserializers
  Add InstructionDeserializerInjector
  Add Action deserializers
  Add ActionDeserializerInjector
  Add Match entry deserializers
  Add MatchDeserializerInjector
  Add MessageDeserializerInjector
  Add PortMessageService
  Add GroupMessageService
  Add MeterMessageService
  Add FlowMessageService
  Add AbstractMessageService
  Add configuration parameter for single layer

Add single layer deserialization support

  - Update MultiMsgCollector to support multiple reply types
  - Update DeviceContext to support mulyiple reply types
  - Update AbstractService and MultipartService* implementations
    to support multiple reply types
  - Update GroupDirectStatisticsService
  - Update FlowDirectStatisticsService
  - Update MeterDirectStatisticsService
  - Update PortDirectStatisticsService
  - Update QueueDirectStatisticsService
  - Update SalExperimenterMpMessageService
  - Update SalTableService
  - Change giant StatisticsGatheringUtils to separate statistics
    writers and add support for writing both deserialization paths
  - Refactor structure of services because of a lot of new classes
    added, it became mess
  - Create new initialization chain for device that supports both
    deserialization paths

Resolves: bug 7141

Change-Id: I21b9b894d89db220ccf20ebb031fcb9e039b184a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Fix errors in serializers and deserializers

Fixed issues with deserializing missing multipart table features,
flows and unsigned longs.
Fixed issues when serializing Ipv4 and Ipv6 source and destination
match entries.
Added additional safety checks to match entry deserializers.

See also: bug 7139, bug 7141

Change-Id: Ie93a486aa847fdb39f01d24768f259cc4661fcad
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Bug 7811 - Conntrack Action deserialization - IndexOutOfBoundsException

1)Added check for the presence of NAT Action during deserialization
2)Added UT to check the presence of NAT Action.

Change-Id: I87ec08f81b1f6372690e5e8d0dbb7ae1ab3e7c74
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Remove old drop-test module and files.

- for drop test using drop-test-karaf module

Change-Id: I3fc3f490bff2d451aa3a0f2773be32e0715f24ca
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Add multipart reply deserializers unit tests

Add unit tests

See also: bug 7140

Change-Id: Ice9189919efca286c06ea1cced0265dd7e43429d
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Update AbstractService to support isComplete

Overload AbstractService#handleServiceCall to also have
parameter with isComplete function.

See also: bug 7141

Change-Id: I51ee1032ac7b443cf8b442ef66eca42380e938b0
Signed-off-by: deathbeam <tomas.slusny@pantheon.tech>

Add multipart reply deserializers

 - Add yang models for multipart reply
 - Add MultipartDeserializerInjector that will inject multipart reply
   deserializers to OpenflowJava
 - Add these mutlipart reply deserializers:
    * Flow aggregate stats
    * Flow stats
    * Flow table stats
    * Group desc
    * Group features
    * Group stats
    * Meter config
    * Meter features
    * Meter stats
    * Port stats
    * Queue stats
    * Table features
    * Experimenter

See also: bug 7140

Change-Id: I0844c54435b3b029982e7cd6e6b9c2cb01138c67
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add PortMessageDeserializer

Add PortMessageDeserializer Inject PortMessageDeserializer into OFJ.
This deserializer will deserialize raw bytes into PortMessage.

See also: bug 7140

Change-Id: Ia3fd168d424138e81eb9f24ede232d954503a40c
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MeterMessageDeserializer

Inject MeterMessageDeserializer into OFJ. This deserializer will
deserialize raw bytes into MeterMessage.

See also: bug 7140

Change-Id: Ia66378d4ad01a6b273308f6d6e25e86f639f90a5
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add GroupMessageDeserializer

Inject GroupMessageDeserializer into OFJ.
This deserializer will deserialize raw bytes into GroupMessage.

See also: bug 7140

Change-Id: I07e0ec7601155eb4138b598c9c4f39dfd1e31d03
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add FlowMessageDeserializer

Inject FlowMessageDeserializer into OFJ.
This deserializer will deserialize raw bytes into FlowMessage.

Change-Id: Ic098a80a6d0b97fbf8ab3761d15a09de3d46defd
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Instruction deserializers

Inject instruction deserializers into DeserializerRegistry.
These deserializers will deserialize these instructions:

 - GoToTable
 - WriteMetadata
 - WriteActions
 - ApplyActions
 - ClearActions
 - Meter

See also: bug 7140

Change-Id: Ie38961458157cac46a4bdeba18c0e6c9020d3671
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add InstructionDeserializerInjector

Add injector where all instruction deserializers will be
registered and call it from general DeserializerInjector.
Create AbstractInstructionDeserializer and
AbstractActionInstructionDeserializer that will define
structure for all instruction deserializers.

See also: bug 7140

Change-Id: Icf76321ed0e51c004d003be585f40cfe4710b1b3
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Action deserializers

Inject action deserializers into DeserializerRegistry.
These deserializers will deserialize these actions:

 - CopyTtlIn
 - CopyTtlOut
 - DecMplsTtl
 - DecNwTtl
 - Group
 - Output
 - PopMpls
 - PopPbb
 - PopVlan
 - PushMpls
 - PushPbb
 - PushVlan
 - SetField
 - SetMplsTtl
 - SetNwTtl
 - SetQueue

See also: bug 7140

Change-Id: I0f36092ca99de0ca51719c3c2099ebe1d904a72f
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add ActionDeserializerInjector

Add injector where all action deserializers will be registered and call it
from general DeserializerInjector.
Add AbstractActionDeserializer that will define structure for all action
deserializers.
Add ActionUtil#readAction that will try to deserialize action or use
extensions.
Add MessageCodeExperimenterKey for registering action deserializers with
experimenters.

See also: bug 7140

Change-Id: I1a1eaec5c9aa128f2c2d5e2a31b37086f2a56734
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add Match entry deserializers

Inject match entry deserializers into MatchDeserializer.
These deserializers will deserialize these match entries:

 - ArpOp
 - ArpSha
 - ArpSpa
 - ArpTha
 - ArpTpa
 - InPort
 - InPhyPort
 - EthSrc
 - EthDst
 - EthType
 - Metadata
 - VlanPcp
 - VlanVid
 - IpDscp
 - IpEcn
 - IpProto
 - TcpSrc
 - TcpDst
 - UdpSrc
 - UdpDst
 - SctpSrc
 - SctpDst
 - Icmpv4Code
 - Icmpv4Type
 - Icmpv6Code
 - Icmpv6Type
 - Ipv4Source
 - Ipv4Destination
 - Ipv6Source
 - Ipv6Destination
 - Ipv6ExtHeader
 - Ipv6Flabel
 - Ipv6NdSll
 - Ipv6NdTll
 - Ipv6NdTarget
 - MplsLabel
 - MplsBos
 - MplsTc
 - PbbIsid
 - TunnelId
 - TcpFlags

Change-Id: Ic584d4610f2b2230542e449d3fae3f4affcc2756
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MatchDeserializerInjector

Add injector where all match and match entry deserializers will be
registered and call it from general DeserializerInjector.
Add MatchEntryDeserializer and MatchDeserializer to be able to actually
create match entry deserializers and deserialize match body.
Add new method to MatchExtensionHelper that can inject extension
augmentations one by one.

See also: bug 7140

Change-Id: Id15419539792cb4a21a5862be2dd423be34ab117
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MessageDeserializerInjector

Add injector where all message deserializers will be registered and call
it from general DeserializerInjector
Add unit tests for MessageDeserializerInjector

See also: bug 7140

Change-Id: I28e2795148597a1c15cb52dec46728a4324a4d0a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add PortMessageService

Add PortMessageService that will convert Port to PortMessage and send
this meesage to serialization.
Update SalPortServiceImpl to use PortMessageService if supported and
enabled.

See also: bug 7139

Change-Id: I73a452770a745c5c7a7debe95cbcf7e04afebb89
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add GroupMessageService

Add GroupMessageService that will convert Group to GroupMessage and
send this meesage to serialization.
Update SalGroupServiceImpl to use GroupMessageService if supported
and enabled.

See also: bug 7139

Change-Id: I07c2f8949f4b89d0cb921d82dfaed3159ca7da8e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add MeterMessageService

Add MeterMessageService that will convert Meter to MeterMessage
and send this meesage to serialization.
Update SalMeterServiceImpl to use MeterMessageService if supported and enabled.

See also: bug 7139

Change-Id: I158ddc58deb62394c50bc2e6d195abaa3f283532
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Add FlowMessageService

See also: bug 7139

Change-Id: Idb4f5b40ce8f033deee209eec1596df2b8ff7d10
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Add AbstractMessageService

Add AbstractMessageService that will define structure for
all other message services.

Modify AbstractService#handleServiceCall not to be final.

See also: bug 7139

Change-Id: I0bcfe8c0a5ba8ad572427240b29d7522ce0c5cf5
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Add configuration parameter for single layer

 - Add use-single-layer-serialization to openflow-provider-config.yang
 - Pass this new configuration parameter to DeviceContext

See also: bug 7139

Change-Id: I00557cd1c0c637fcf83c42e47808a5bf96a57a1b
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Remove redundand case in nicira-action.yang

    - created probably as unattended copy/paste
    - equal cases are forbidden by yang spec and next version
      of yang parser will check this rule

See: BUG-7424

Change-Id: I83c551dc5dd613163c3f8a47a443081d71f3c328
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Fix failing test.

Yangtools fixed their issue and merged it.

Change-Id: I47c3397145a53c2d2db72093b153a40da5cee571
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Merge "Bug 7764 - Do no throw warning on explicit task cancellation"

Bug 7764 - Do no throw warning on explicit task cancellation

Change-Id: I9477823ffd1b4e9b28484ae8cde166a2d03f5a9f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit d813c7f656b4dd6f460b66a2fa303a448f6a4e07)

NAT Action added in Nicira Extension

Nat action is added as a port of conntrack action, it is an optional
action field.

Change-Id: Iec97d9d859e0e2060abc3a31ceb929c1b1e43b4a
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Bug 7769 - Master branch is broken because of the test failures.

Change-Id: I0f3d787011d388947b10f4fe94ed20e4a270293c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "Fix comparison between port numbers in match"

Fix PacketInV10TranslatorTest

Use object instead hardcoded string

Change-Id: I6408a927e0c6c186e78884a9851f1dbea56456aa
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Merge "Replace mockito-all by mockito-core (see Bug 7662)"

Bug 7499 - ensure statistics scheduler does not die and keep trying while the controller keeps the ownership of the device

Change-Id: Ia5923f200b6c7a8888d4e3c96fdf8358603d4ac9
Signed-off-by: Jon Castro <castro.jon@gmail.com>

Merge "BUG-7608: use blueprint action-provider/action-service"

Bug 7453 - FlowRemoved doesn't have Removed Reason Information

On flow being removed from switch on idle timeout, onFlowRemoved() api was
invoked and while conversion reason was not being converted and added, so
the application using API onFlowRemoved() was not getting the reason for
flow being removed from the switch.
This change will provide the reason invoking onFlowRemoved() API

Change-Id: Id8b957b5b2cc581215fb84b8ca84d1d9a04e5619
Signed-off-by: eeiillu <suja.t@ericsson.com>

BUG-7608: use blueprint action-provider/action-service

The contract for RPC services containing Routed RPCs (RFC7950
actions) with respect to routing is different from normal RPCs.

In order to properly express dependencies, it needs a concept
of a promise to register instantiations (action-provider)
and the dependency on that promise (action-service). Use these
concepts instead of realying on rpc-service which happens to
work due to sal-remoterpc-connector (incorrecty) providing
the equivalent of action-provider for all Routed RPCs defined
in the local SchemaContext.

Change-Id: Iaad5b5b0b4384a2e7d6302bd7de5bfa00c24d89b
Signed-off-by: Robert Varga <rovarga@cisco.com>

Replace FindBugs :jsr305 by full :annotation (Bug 7663)

Change-Id: I05b3a0fd993c246617808d33d743d04976e0286c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Add isComplete callback to commitEntry

Implement commitEntry with isComplete callback
from https://git.opendaylight.org/gerrit/#/c/51064/ to
OpenflowPlugin OutboundQueueProviderImpl.

See also: bug 7140

Change-Id: I7c4224e029f175a54b46246e1d409356f2447b16
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Fix comparison between port numbers in match

Properly extract port number from NodeConnectorId when comparing
port numbers, as NodeConnectorId is basically string, and sometimes
when going back from device, it do not contains datapath id
or prefix (for example in new serialization/deserialization path).

See also: bug 7139

Change-Id: I89999b71dba2e834ea1f37f42b5cddf1082bdd12
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Replace mockito-all by mockito-core (see Bug 7662)

Change-Id: Iceeb2f1082a6ed3be04e99e17b2e6c3c3c4ebb31
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix Direct statistics RPC - actions part

Handle actions for direct statistics.

See also: bug 7232

Change-Id: Ib29f9a760bf123bac1745ea2fe8bc7e6bd3de73c
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

RPC opendaylight-direct-statistics:get-flow-statistics not taking nicira
extension match

This is just a draft review, still testing in progress

Added new MatchPath RPCFLOWSSTATISTICS_FLOWANDSTATISTICSMAPLIST_MATCH.

This will differentiate between the augment model between notification and
rpc.

Change-Id: If80ff86522ba406577b15c4a3d8a30ee364001db
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "Bug7485 Make statistics poller parameters configurable."

BUG-5222: do not pull in odl-mdsal-xsql

xsql is deprecated, buggy and never was meant for production
use. Do not pull it in.

Change-Id: Id534d17150a940ffad7f7c757146bca46fa1a923
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug7485 Make statistics poller parameters configurable.

Currently the the time intervals used for polling the statistics
are harcoded. (basicTimerDelay, maximumTimerDelay).
This patch helps makes parameters more configurable by defining
current default values in yang. Using BluePrint extensions user
can now configure the parameters without changing the source code.

Change-Id: Id06cb1e569b0d3c653c142bf5f5af5a24fd6e3ea
Signed-off-by: Sai Marapareddy <sai.marapareddy@gmail.com>

Merge "BUG 7502: Switch throws an error message for the action type"

Replace odl-dlux-core with odl-dluxapps-topology

Due DLUX split, we removed odl-dlux-topology feature from odl-dlux-core.
Because your project have dependency on odl-dlux-core, i assume, you used
Topology app.

More info:
https://wiki.opendaylight.org/view/Weather#Rename_some_odl-dlux-.2A_features_to_odl-dluxapps-.2A

Change-Id: I037a9297f1cad0b9ee961c6e12876e9f463e061a
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Merge "Remove use of triemap"

Remove use of triemap

Triemap is not used anywhere, do not declare it as
a dependency.

Change-Id: I296374e2be486d6e77a74d709ce2bf021cab9088
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 7071: adding support for fin-timeout

Add support for serialization/deseralization and encoding/decoding of fin-timeout action.
This action currently is created using the learn action, so we nead only deseralization/decoding way (when flow statistics is sent to odl),
but this patch suppors also the other way of insalling flows from odl, in case it will be needed in the future.

Change-Id: I061100fab61e8d3398fddde98e76493a5afef468
Signed-off-by: Slava Radune <slava.radune@hpe.com>

BUG 7502: Switch throws an error message for the action type

       (move:NXM_NX_REG1[0..19]->OXM_OF_MPLS_LABEL[] )
       error message: OFPAT_BAD_MATCH due to wrong OXMClassCode.
       code-review contains the fix to address correct OXMclasscode for the matchtype.

Change-Id: I113e76111b4ca84e2a7c1f0f443af7330f33328f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>