Merge "Fix modifiers order to comply with Java coding guidelines" into stable/boron

Merge "Fix checkstyle - api.openflow.md.queue" into stable/boron

Merge "Fix minor issues regarding checkstyle" into stable/boron

Merge "Fix checkstyle warnings." into stable/boron

Fix checkstyle - api.openflow.md.queue

Change-Id: I1c91dcee7b74ac1dba17e5c04651009c312124c3
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
(cherry picked from commit dc8e9554a3ca8e00667cdcc582c567d6106f24fa)

Fix checkstyle warnings.

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

Change-Id: Ida0bf4149aaa70657a22fb527708d61b0d7d43b3
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
(cherry picked from commit f5cea4f9c7c5e44c1ea776204eb21497dc2bad2e)

Fix checkstyle warnings

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

Change-Id: I98490f3a4054bcfdb1339ac490d432ece0539097
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
(cherry picked from commit b1723fcc1e90149c95159e553a1084a046cf25c3)

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>
(cherry picked from commit 5eea4b8cabfacacfbecade86f0fc2f6863b0b0b5)

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>
(cherry picked from commit b260fb55a235bf4ce5b538515b9a4537ba0c3a6d)

Merge "Bug 8217: Set error information into direct statistics RPC result." into stable/boron

Merge "Fix table miss flow push" into stable/boron

Merge "Fix DeviceFlowRegistry performance regression" into stable/boron

Merge "Revert "Revert "Fix statistics race condition on big flows""" into stable/boron

Bug 8217: Set error information into direct statistics RPC result.

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

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>
(cherry picked from commit 6dbbd412a88dafdb7d7acd824516850b473b145f)

Bumping versions by x.y.(z+1) for next dev cycle

Change-Id: I24d2af439829cb1b037478d7799a2aa08ed01f05
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

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: Id6d83bf746f05cc7e91f804e6b9c8fc84b39f501
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Revert "Revert "Fix statistics race condition on big flows""

This reverts commit 1537fd31483e3c9f5ec62e882d54acf9e508e556.

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

Fix table miss flow push

- Use ClusteredDataTreeChangeListener

Resolves: Bug 7770

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

Revert "Fix statistics race condition on big flows"

This reverts commit 501d4d64c806ad39e90b97def853fa043dda5f30.

Change-Id: I9e7c43ac81ec79c12db8ed2203464664d1a6910a
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>

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

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

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>
(cherry picked from commit 113fc0c316f814452a535a66f0da085076d282b5)

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>
(cherry picked from commit 44a28ad62e641c0b4b0e7b81cc634316836e921f)

Merge "Bug 6110: Fixed bugs in statistics manager due to race condition." into stable/boron

Merge "Bug 7910 - Flow with ethernet mask (ff:ff:ff:ff:ff:ff), get stored under alien-id in operational data store" into stable/boron

Merge "Fix comparison between port numbers in match" into stable/boron

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>

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>
(cherry picked from commit 0ac8877c3539d82e55ffe49dc0a2c8bcb0c82b19)

Bug 7763 - Openflow plugin deletes switch from topology
while changing mastership from one controller to another

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

Merge "Bug 7736 - Forwarding Rules application cluster singleton id should not use the same cluster singleton id as the openflow switch singleton connection handler" into stable/boron

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>

Revert "Bug 7500 - TransactionChainManager: fix synchronization issues and error handling when mdsal throws an error."

This reverts commit 4683e5748682a4739139fd11d426bb193385f346.

Change-Id: Ibe06731e2d17895d642864ce4f6a9838b33bc0c7
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Bug 7501 - Ensure delete old statistics and create new ones are executed sequentially to ensure stats are updated properly." into stable/boron

Bug 7764 - Do no throw warning on explicit task cancellation

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

Bug 7501 - Ensure delete old statistics and create new ones are executed sequentially to ensure stats are updated properly.

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

Bug 7500 - TransactionChainManager: fix synchronization issues and error handling when mdsal throws an error.

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

Fix PacketInV10TranslatorTest

Use object instead hardcoded string

Change-Id: I6408a927e0c6c186e78884a9851f1dbea56456aa
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
(cherry picked from commit ad3188d4af8ced3235b90f997e4799926f3a6a56)

Merge "Bug 7349 - Flow ID not updated in operational after removing and adding a flow with same match" into stable/boron

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

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>
(cherry picked from commit 0d5442f3684c32b084a95ee38589c6fee576554f)

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>

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>
(cherry picked from commit 80cee5d71d306b8806d9316d12dceb5c1d1d63e6)

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>

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

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>
(cherry picked from commit 49b53da651c4159841f5e9701df22cbe7e5984e7)

Merge "Bug7485 Make statistics poller parameters configurable." into stable/boron

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>
(cherry picked from commit 5b61abe19fce14040eddb8c70bd3a53d1c987a1e)

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: Ic2d694351ee968a6f360e7a218aa17e69c5e9035
Signed-off-by: Sai Marapareddy <sai.marapareddy@gmail.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>
(cherry picked from commit 1bd4c2b474f6301f9e9282cfc728e714a73fc21e)

Bug 7481 - Flows with nicira actions get corrupted after cluster failure

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

Merge "Bug 7335 - Flow update rejected by switch generates faulty flow entry in operational DS" into stable/boron

BUG:6997 supporting OXM_OF_MPLS_LABEL in nicira extensiona

        Description : currently under nicira-extension, MPLS support is not
        present in nicira Move action. Mainly in ECMP use-case we need to use
        mpls_label under nicira_move_action type. The mpls_label will be using as
        example ( move:NXM_NX_REG1[0..19]->OXM_OF_MPLS_LABEL[] )

Change-Id: Ia497b963f30bf9ffcd09430db8ed85661a64859f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
(cherry picked from commit d24260189065b6620af4f6829b91313c0a681c6d)

Bug 7415 Reducing the severity of the log message

Whenever there is a change in configuration of l2switch,
bundle refresh happens. When bundles refresh, openflowplugin
pushes the same flow. Changing the severity of of the exception
to warning since it is expected to have a flow with same id.

Change-Id: I69874d14b83053edc97b6d299c8fdb05d7ccdd8c
Signed-off-by: Sai MarapaReddy <sai.marapareddy@gmail.com>
Author: Sai MarapaReddy <sai.marapareddy@gmail.com>

Merge "Add LOG.isDebugEnabled to add performance." into stable/boron

Merge "Improve class with lambdas. Change wrong parameters and variables." into stable/boron

Merge "Split long lines (>120)" into stable/boron

Merge "Remove unused imports, repair checkstyle first sentence." into stable/boron

Bumping versions by 0.0.1 for next dev cycle

Change-Id: I923c53fef2d3ec12757cea39285decaf46bac906
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

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>

Add LOG.isDebugEnabled to add performance.

- one more lambda expression
- test checkstyle corrections

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

Improve class with lambdas.
Change wrong parameters and variables.

- checkstyle issues

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

Split long lines (>120)

- checkstyle warning

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

Remove unused imports, repair checkstyle first sentence.

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

Bug 7335 - Flow update rejected by switch generates
faulty flow entry in operational DS

When flow update request comes to plugin, it cleans up
the flow hashmap for the updated flow, but it does not
clean up the flow with the assumption that in the next
statistics cycle, this stale flow in operational data
store will be overwritten by the new modified flow,
but as reported in the issue, if installation of modified
flow fails, it will leave the stale flow in the data store.

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

Merge "Revert the functional behavior change introduced through" into stable/boron

Revert the functional behavior change introduced through

https://git.opendaylight.org/gerrit/#/c/47138/3

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

Merge "Bug 6820 - Implement SalExperimenterMpMessageService" into stable/boron

Merge "BUG-6890:Flow-Removed Notification configuration" into stable/boron

Bug 6820 - Implement SalExperimenterMpMessageService

Change-Id: Ibfbdea25957b8259a6b5070fb0598172d44108bc
Signed-off-by: yunyunhan <han.yunyun@zte.com.cn>
(cherry picked from commit 2dec517931a2fb352b5391197f2baa305460ad91)

Bug 7209 - Null Pointer Exception in LearnCodecUtil when add learn flow for ipv6

The exception caused by using ByteBuf::readShort
(valid range: -32,768 32,767) for ipv6 etherType '0x86dd'
The fix is to use ByteBuf::readUnsignedShort

Change-Id: I238535565a66692b5d4dbe39b8cb7809867f3cde
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

BUG-6890:Flow-Removed Notification configuration

Changing the flag to enable/disable flow removed
notification.

Currently the flow-removed notification is controlled by the flag
notification-flow-removed-off which is on by default(value set to false).
However this logic being double-negative causes some confusion to the
end-user. Thus modifying the flag to enable-flow-removed-notification
and setting it to true by default.

The associated logic involved at oyther places have been modified as
well as a part of this commit.

Change-Id: I1452de6e3e72030608c89168f19328d9c6aa8f1c
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

lower log level when stats come before flow is written to deviceflowregistry

Change-Id: Ib24b9e97c86b2d1a24610204b61fcf9237871f0e
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Merge "Bug 7058 - [Helium Plugin]Stats collection issue when controller disconnect the device" into stable/boron

Implement SalExperimenterMpMessageService

Add missing implementation for experimenter multipart message
RPC service and register it as routed RPC implementation

Resolves: bug 6820

Change-Id: I5646f93dd6375b729e995a9e60e6d688f1508559
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit cf84afd05137e68dcda33f88ae30312b649d79a9)

Improve cleanup after device disconnected event

- Ensure that all contexts are removed from managers at once
  at end of disconnect chain right before unregistration from
  cluster singleton service
- Prevent closing some services multiple times during device
  disconnect chain
- Prevent some NPEs in DeviceInitializationUtils what was caused
  by connecting and disconnecting device with no time in between
- Unify all openflow managers to implement OFPManager interface
- Ensure that when we will fail to set device to SLAVE we will wait
  until device is removed from operational DS and then continue

Resolves: bug 6672
See also: bug 6710, bug 5271

Actual work done by Tomas Slusny. Colin Dixon merely fixed a merge
conflict.

Change-Id: I3845c7d2a9147125b185d3abcc3fc8dd63ba5da8
Depends-On: I15f89cbd43310b0a8e4b7ac78595eeaaa82a65ee
Co-author: Tomas Slusny <tomas.slusny@pantheon.sk>
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
Signed-off-by: Colin Dixon <colin@colindixon.com>
(cherry picked from commit 4a89e389589676a2b6975096ba721cd4c354e1bd)

Merge "Bug 7011 - Race condition in statistics collection related transaction chain handling" into stable/boron

Merge "Optimize port number lookups" into stable/boron

Merge "BUG-6890: Statistics-polling configuration" into stable/boron

Bumping versions by 0.0.1 for next dev cycle

Change-Id: I85a05ba2c9ea0ab3971e7a9930b3f26aa5cc41cb
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Bug 7058 - [Helium Plugin]Stats collection issue when controller disconnect the device

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

BUG-6890: Statistics-polling configuration

Changing the current flag to enable/disable statistics
polling.

Currently the statistics polling is controlled by the flag
is-statistics-polling-off which is off by default(value set to false).
However this logic being double-negative causes some confusion to the
end-user. Thus modifying the flag to is-statistics-polling-on and setting
it to true by default.

The associated logic involved at oyther places have been modified as
well as a part of this commit.

Change-Id: Ia9a027433611912c11ff0f3166aa6f68efdc6913
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Optimize port number lookups

- Remove unnecessary OpenflowPortsUtil#init() and
  OpenflowPortsUtil#close() methods and add static constructors
  instead, so BiMaps will be initialized only when really needed,
  and we do not need to call OpenflowPortsUtil#init() method
  everywhere
- Cache inverse port BiMaps in OpenflowPortsUtil to greatly reduce
  lookup time in these BiMaps in some methods
- Try to use more short version instead of deprecated
  OpenflowVersion to reduce deprecation and to reduce
  memory use (less conversions from OpenflowVersion to short)
- Use proper MAX port constant when finding max port number
- Simplify some conditions in OpenflowPortsUtil
- Use proper OF_URI_PREFIX contant in InventoryDataServiceUtil

Resolves: bug 7046

Change-Id: I1ea033963fb48416d004398e2ee72909231aaea6
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
(cherry picked from commit 7acab16585770656cf65a826a4bbd9eee31a8822)

Bug 7011 - Race condition in statistics collection related transaction chain handling

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

Fix direct statistics

- Initialize statistics gathering

Resolves: bug 7007

Change-Id: I0970b3c690f0771dbd1637405e3064a984b1ef4c
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
(cherry picked from commit 02eedd8c320e6b9ff7bfafc7cda61e1c963c46bd)

Merge "BUG-6890: Edit to cfg file reflecting that statistics collection is turned on by default" into stable/boron

Merge "BUG-6890:" into stable/boron

Merge "Remove RoleManager and RoleContext" into stable/boron

Fix flow matching function

- If flow is not present in DeviceFlowRegistry,
  don't write to operational DS.
- Update SalFlowService "update" method
- Update Unit tests

Resolves: bug 6917

Change-Id: If07e94be2d2e8564a2d39d88aec0ea5ea6ac7d19
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
(cherry picked from commit 91cacc74cf40c5fbe27ca912cdf78a27f1e40dd6)

Remove RoleManager and RoleContext

- no more functionality for role manager (after Singleton changes)
- all remain functionality from role-mng and ctx were moved to device-mgn and ctx

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

BUG-6890: Edit to cfg file reflecting that statistics
collection is turned on by default

Change-Id: I8a97cb11b1b98189a04daed50961b4a912113749
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-6890:

Fix to provide default values for configurable parameters by
handling the NumberFormatException in case the configurable
parameters are either null or absent

Change-Id: Idb37565167778a900732df2dfef428847b0efd3b
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "Optimize LLDP packet check" into stable/boron

Bug: 6930 Notiifcation-suppliers was broken

Review contains changes required for making notification-supplier as per
Blueprint Injection method
- Removed Config Subsystem parts - Yang, Config XML, Modules/ModuleFactory
- Added Blueprint XML for notification-supplier
- Removed Config Dependency from features Pom
- Removed references to Config XML

Change-Id: Ifbdcb21140903a668ad0b8dd6e2df666b7e3e37f
Signed-off-by: GauravBhagwani <gaurav.bhagwani@ericsson.com>

BUG-6890: Updated the cfg file with detailed description
of usage

Change-Id: Ibe772a29107296cb96cf4aa48c1d3cb2dee4f3b2
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-6890: Enabling statistics collection through a config
parameter in openflowplugin.cfg

Change-Id: If7dbedd50d7f23a075c53279654e36d33b212867
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-6890: Enabling echo timout configurable through
config file

Change-Id: I3ff729b418809e8342a491ac6ab2a102f5a1ffe5
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

BUG-6890: Enabling barrier configurability through
cfg file

Change-Id: Iaada857a439c7774c6e31c27cc7be1b26c68ad1b
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Optimize LLDP packet check

Deserialize only LLDP packets.

See also: bug 1462

Change-Id: Ibdd2f3eb0dfe784fd235e51a591da64971c2afd3
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
(cherry picked from commit 10922cdfdcbc4c14456cd6a387468e11bb9b802e)

Bug 6745 SimplifiedOperationalListener optimation

- optimize processNodeModification method
- update test

Change-Id: I941a84bff4322c2b2b64f8404791ec6d00a9d200
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
(cherry picked from commit 40b60339e5e8aeb07e782f8c34b5c2ef50da56ab)