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>

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

Change-Id: I3845c7d2a9147125b185d3abcc3fc8dd63ba5da8
Depends-On: I15f89cbd43310b0a8e4b7ac78595eeaaa82a65ee
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Merge "BUG-6890:"

Fix direct statistics

- Initialize statistics gathering

Resolves: bug 7007

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

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:

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>

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>

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

Revert "Revert "Move the Openflow connections blueprint to OpenflowJava project""

This reverts commit e84b8d9e2c0e38a950db4295a61b24dd77538e01.

Change-Id: I1c79ddcb0041d4599e82fa5f6ed227d530dfab97
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Merge "Remove RoleManager and RoleContext"

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>

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: 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>

Revert "Move the Openflow connections blueprint to OpenflowJava project"

This reverts commit f44ebeb5ca3e25ff4c863f374ed8b9b33466e991.

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

Merge "Optimize LLDP packet check"

Merge "BUG-6890: Enabling statistics rpc config through openflowplugin.cfg"

BUG-6890: Enabling statistics rpc config through
openflowplugin.cfg

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

Move the Openflow connections blueprint to OpenflowJava project

This patch is related to https://git.opendaylight.org/gerrit/#/c/46518/
Moving the Openflow connections (defaultSwitchConnProvider and legacySwitchConnProvider)
blueprint to OpenflowJava project as its config data model "openflow-switch-connection-provider-impl"
exist in OpenflowJava project

Change-Id: Ic6db83e98d57cf17ef5883d64755118214815731
Signed-off-by: Mohamed El-Serngawy <melserngawy@inocybe.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>

Bug 6745 SimplifiedOperationalListener optimation

- optimize processNodeModification method
- update test

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

Merge "Bug 6745 Do not skip first data for reconciliation"

Bug 6745 Do not skip first data for reconciliation

 - we can accept first modification since all statistics
   are intentionally collected in one step on startup
 - added test
 - updated comments

Change-Id: I94590fb268accb0237260a09d184c2cecd1fb324
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Fix unregistering of cluster services

When unregistering cluster singleton services, check if we are not
already in unregistering phase, and if yes, skip this process. Also,
chain stopClusterServices futures properly, so Singleton will know, when
to finish unregistering.

See also: bug 6672

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

Create SemaphoreKeeper inside decorators

Change-Id: I29fb2da53d32b331459a4492ca15a4081ea79195
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Merge "Bug 6745 Improve compression queue locking and handle InterruptedException"

Merge "Revert "Bug 6745 Do not ignore syncup return value""

Bug 6745 Improve compression queue locking and handle InterruptedException

 - compression queue could be locked per device and not for every change
   since there is only one entry per device in it
 - IE handled in one place

Change-Id: Ic28b448d619c663757f391e7a443b28c8f6a871d
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Add finals and move thread name constant to provider

 - add onDTC annotations
 - code cleanup

Change-Id: I585d08e67138805672d19293892358205650c998
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Revert "Bug 6745 Do not ignore syncup return value"

This reverts commit df561a83c33b7af9818d8d0d0e8e6b00f7a37a93.

Change-Id: I9133264ba297bf4293a50e26e1d2e1051df1f093
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Merge "Bug 6745 Set compression semaphore to fair"

Merge "Bug 6745 Do not ignore syncup return value"

Merge "Bug 6745 Remove thread renaming and unnecessary logging"

Merge "Bug 6745 Fix replacing in compression queue"

Bug 6745 Set compression semaphore to fair

 - prevent changing the order of incoming changes

Change-Id: If7df69334b2ff8217986cd933e25463cb6e90171
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6745 Do not ignore syncup return value

Change-Id: Ide811a0db0d684dc3b37ea3f6707d779162cb87f
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6745 Remove thread renaming and unnecessary logging

 - change name format for thread pool
 - remove thread renaming and useless logging
 - remove duplicit time logging
 - update return values

Change-Id: I78abca1870d278c162a7f717ff800dd37c1d4234
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6745 Fix replacing in compression queue

 When there were more changes in a row on startup it could happend
 that the first syncup entry (config-operational) was not updated
 but replaced with following config-config changes. Therefore we need
 to update syncup entry in compression queue everytime when new config
 diff is comming (not only if config-config entry resides there).

Change-Id: I9d73fb348f5e1ace1b138052bcd2a5614946efa9
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Write SwitchFeatures to operational datastore

Ensures that SwitchFeatures are written to operational
datastore during device initialization in Lithium design.

Fixes: bug 6764
Change-Id: Ib32f15b73febe4d9115884d418bfe87b8e716635
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Remove reference to sal-inmemory-datastore config system file

As it was removed from the controller

Change-Id: Ied428beee2f46cfd000afcf9ca56e0f25c8c5594
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Remove excessive (trace) logging in FRS

 - removed not so necessary logging + some updates
 - added some finals
 - simplify statements

Change-Id: I0db57a38a45efd6bb8334a9f05cdba991bff556d
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Merge "BUG-6118: making the OFentityListener aware of the InJeopardy() flag"

Merge "ClusterSingletonService cleaning FRM/FRS"

Merge "Create DeviceMasterShipManager before forwarders"

Fix translation to *packet.received.MatchBuilder

Fix incorrect cast of *flow.MatchBuilder to
*packet.received.MatchBuilder in Helium plugin, what
was causing PacketInTranslator to be broken.

Fixes: bug 6456
Change-Id: I9e42402a3599dd2e0a91381019c4ca86e8294aa2
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Create DeviceMasterShipManager before forwarders

This prevents NPE when forwarder receives DataTreeChange
event faster thatn DeviceMasterShipManager is created.

Fixes: bug 6459
Change-Id: Ifb9d4520b58d9f5fc265934968386641c60772ef
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Merge "Bug 6749: Set the openflow connection config at xml file"

Bug 6749: Set the openflow connection config at xml file

Set the openflow connections (default and legacy) inital configuration
to xml file to be able to change it after starting the distribution

The openflow TLS configuration was missing in the blueprint,
add it with default values

Change-Id: Iad34bd3d7e237da52fe3193c4ad6d6ff2141358c
Signed-off-by: melserngawy <melserngawy@inocybe.com>

Update comments and imports after DataChangeListener changes

Change-Id: Ic8174cc2627deb12d48f6d41444f2e9b4dc60406
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Merge "Update comments around flat-batch service"

Merge "Convert openflowplugin-it to use DTCL instead of DCL"

Merge "Update old links in code to deprecated DataChangeListener"

Merge "Convert OF samples to use DTCL instead of DCL"

Fix connection closing before initialization

Check if connection is fully initialized when closing
objects created during initialization phase to prevent
NPEs, what can cause incorrect close of connection
and later program unstability.

Also, increase timeout to propagate role to 10 seconds
in case of slower machine.

Resolves: bug 5271
See also: bug 6672

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

Merge "Bug 6665 Clean code"

Merge "Bug 6665 - Fix switches scalability"

Convert openflowplugin-it to use DTCL instead of DCL

Change-Id: I8a3f3b690bef5d29dcd9b51e243bd83ad5c501c6
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Update old links in code to deprecated DataChangeListener

 - update old links to DataChangeListener in comments, logs,
   names of variables since DataTreeChangeListener is already used
 - optimize imports and cleanup

Change-Id: I20f3359f5f7c4bbd3af844bc45506a5575c98285
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Convert OF samples to use DTCL instead of DCL

Change-Id: I53eebe8672996ed4351240a8c649e6c2815e8a0c
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Merge "Bug 6633 : NXM_OF_IN_PORT support in openflowplugin"

ClusterSingletonService cleaning FRM/FRS

Change-Id: I7c9a31ccab5b7b79b318cc7b59742aa5652e41f8
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Update comments around flat-batch service

Change-Id: I62f3ccbf38fd6f5aeb830e55c5f60923c6f8e8da
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6665 Clean code

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

Bug 6665 - Fix switches scalability

- Stop gathering services if going to be slave or disconected
- Duplicate code removed in role context
- Lifecycle service LOG info cleaning
- Propagate close even if we are in close state
- NPE in statistics manager
- Test correction

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

Bug 6633 : NXM_OF_IN_PORT support in openflowplugin

Description : OVS split-horizon logic will drop pkts by default if value
of in_port and out_port is same. But, if we have two vlan-trunk member
VM for which trunk interface is same, the connectivity between these two
VM cannot be established as OVS drops such packets. So, it is required to
override the value of in_port so that split_horizon check is bypassed.
Nicra NXM_OF_IN_PORT support is required in order to achieve this. The
in_port value will be overrriden using load(ex:
load:0x1->NXM_OF_IN_PORT[]).

Change-Id: Ica0a1b2da27fc189b8906462a4420473215a22af
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

BUG-6595: Need more information in logging

Enabling port information in the logging to
understand better which port status addition
is failing

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

Merge "Bug 6554 Fix rejecting connections"

Merge "Bug 6176 - Decrease logging level in Sal-F/G/M-Service and use synchronized BiMap"

Bug 6554 Fix rejecting connections

- two way device context close removed
- txManager lambda improvements
- txManager logs improvements
- device context lazy initialization
- role context close improvement

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

BUG-6118: making the OFentityListener aware of the InJeopardy() flag

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

Bug 6542 FRS - prevent concurrent reconciliation node config add

 - unregister for reconciliation if there was an attempt but no
   config present for node
 - when node config is added essentially reconciliation is done
 - junit updates

Change-Id: Ic8f759e1638285ef7e82a35a0c9d330a2b0340e7
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6176 - Decrease logging level in Sal-F/G/M-Service and use synchronized BiMap

 - decrease logging level to prevent overloading of netty
   thread with logging errors (seen in troughput test)
 - despite the fact that unsuccesful RPCresult brings errors
   from device there is no reason for throwing error since
   it is not error of karaf ot OFP itself (therefore debug)
 - change BiMap to synchronized because of ConcurrentModificationException
   seen in tests

Change-Id: Ia37212647a9c119ecd51d6685ea8fffb8ee94a39
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6524 FlowDescriptor instead of SwitchFlowId in FRS

Change-Id: Ibb02c310be27f02c7543e157d2f5e7ca7cc430a8
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6513 Remove FD from registry immediately

 - remove FlowDescriptor from DeviceFlowRegistry immeadiately
   instead of marking it as to-be-removed
 - updated tests

Change-Id: I269a510fa67b6d04181039515aaefda90b53b827
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Merge "Bug 6465 Controller goes into slave mode"

Bug 6465 Controller goes into slave mode

- Changed lifecycle service in order to
create lazy initilization of context in cluster
- Removed clustering initialization per context
- Changed reconnect strategy if not in disconnecting state
- Some performance improvements
- Removed OFManager general API

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

Merge "BUG-6458:"

BUG-6458:

The issue occurred since during update the older
flowregistry key wasnot getting removed when the
newer one was getting stored. This was because the
match/priority/cookie all forms part of the flow-registry
key.

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

BUG-5464,5954: making table features configurable for the Li plugin.
DEFAULT will be OFF, can be TURNED ON.

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

Merge "Bug 6366 - of-switch-config-pusher - DTCL instead of DTL"

Merge "Bug 5596 Singleton API test"

Merge "Adding learn action"

Bug 5596 Singleton API test

- added missing test for lifecycle service
- added missing test for role context

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

Merge "Bug 6374 topology-manager - DTCL instead of DTL"

Merge "Removed duplicate declaration in pom.xml."

Merge "Bug 6278: Switch to use odlparent's karaf-parent"

Merge "SONAR TD - Remove unused fields, fix naming"

Merge "SONAR TD - StatisticsContextImpl, StatisticsManagerImpl"

Merge "SONAR TD - FlatBatchUtil, DeviceInitializationUtils"

Merge "Removed unused logger and fixed javadoc links"

Merge "Bug 6380 lldp-speaker - DTCL instead of DTL"

Merge "Sonar - technical debt of FRS app"

Bug 6278: Switch to use odlparent's karaf-parent

Change-Id: If65dbe7d41cee7f14f56479634627d107df50d6f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "SONAR TD - Group actions redundancy"

Merge "SONAR TD - Sal*ServiceImpl"

Merge "SONAR TD - AbstractService"

Merge "SONAR TD - *ToNotificationTransformer private constr."

Merge "Technical debt MeterUtil, GroupUtil, FlowUtil"