Disconnection improvements.

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

Add mastership listener to Initialization callbacks.

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

Implement connection core layer basic behavior

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

Fix DeviceFlowRegistry performance regression

- Remove use of marks and modify DeviceFlowRegistry immediately
- Synchronize only access to DeviceFlowRegistry BiMap and not
  entire class
- Replace custom match comparison with match normalization (should be a
lot faster on big amount of flows)
- When logging to console, compute FlowRegistryKey hashes only when needed

See also: bug 6917

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

Merge "Add missing karaf 4 features to artifacts"

Merge "BUG-7622: Karaf 4 Migration"

Add missing karaf 4 features to artifacts

See also: BUG-7622

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

Merge "Fix table miss flow push"

BUG-7622: Karaf 4 Migration

Add missing karaf 4 features to artifacts.

Change-Id: I907afa3a31b44412ae3ffa6ba3cca9ab81695b07
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Merge "Replace project-specific checkstyle by odlparent's"

Merge "Fix minor issues regarding checkstyle"

Merge "Fix modifiers order to comply with Java coding guidelines"

Merge "Fix checkstyle .api.openflow.protocol"

Merge "Fix checkstyle api.openflow.md.util"

Merge "Fix checkstyle - api.openflow.md.queue"

Merge "Fix checkstyle warnings."

Merge "Fix checkstyle warnings"

Merge "BUG-7901: fix unsynchronized transaction access"

BUG-7901: fix unsynchronized transaction access

The evidence in the issue indicates that we have a race condition
where a batch of modifications is being submitted (triggered by
a write/delete and exceeded batch count) and we are observing a
concurrent modification. After auditing the CDS side of this, while
we can improve safety a bit, the scenario has to involve multi-threaded
access by the client.

Looking at TransactionChainManager, it seems to be heavily synchronized,
but there is a hole in the paths which are using getTransactionSafely().

While the access to the TransactionChain, it returns the transaction
(which really is an alias to the wTx field). That throws of static
analysis, as we end up the object pointed to by wTx without holding
the lock -- hence concurrent threads will end up touching the transaction
without any guards, leading to a violation of Transaction contract (which
requires external synchronization).

This patch reworks getTransactionSafely() so it only ensures the field
is initialized as appropriate and makes its callers access the wTx field
with the proper lock held.

Bug: 7500
Bug: 7901
Bug: 8060
Change-Id: I058feb55f91a40ccda6d69ee87cd1a20f792afba
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix masked NXM reg length

Change-Id: I0ed5fe6a2db52521b13d0943d8b25911d5ac5fe9
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Fix checkstyle .api.openflow.protocol

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

Fix checkstyle api.openflow.md.util

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

Fix table miss flow push

- Use ClusteredDataTreeChangeListener

Resolves: Bug 7770

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

Fix checkstyle - api.openflow.md.queue

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

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>

Replace project-specific checkstyle by odlparent's

This change replaces the project-specific checkstyle rules by the common
ones defined in odlparent's pom.xml as other projects have already done
(Netvirt, Genius, AAA, etc.). The actual enforcement of the checkstyle
rules
are applied on a per-module-basis in their own pom.xml by adding the
following section to the maven-checkstyle-plugin:

<configuration>
 <propertyExpansion>checkstyle.violationSeverity=error</ropertyExpansion>
</configuration>

I have commented the checkstyle enforcement of some current modules to
do this enforcement process in a progressive way by solving the
checkstyle issues module by module. I'll rise separate commits for
those modules soon.

Change-Id: I19bf36ddcd1f02b5cca388562386294b40603566
Signed-off-by: David <david.suarez.fuentes@ericsson.com>

Fix modifiers order to comply with Java coding guidelines

Minor change: replace "final static" with "static final".

Change-Id: If7df3dd6f6999dde5a4bc1b495f28a598b6a9da1
Signed-off-by: David <david.suarez.fuentes@ericsson.com>

Fix minor issues regarding checkstyle

- Make BulkOMaticUtils a final class since its constructor is private.
- Parenthesis not needed

Change-Id: I2e0a58dfe204aec89ab49cb3fbd870229932c317
Signed-off-by: David <david.suarez.fuentes@ericsson.com>

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>