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)

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>

Create SemaphoreKeeper inside decorators

Change-Id: I29fb2da53d32b331459a4492ca15a4081ea79195
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>
(cherry picked from commit 191d8581d6e34829141ee7c44c6e05e98379b86d)

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>

Merge "Add finals and move thread name constant to provider" into stable/boron

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>

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>

Merge "Bug 6745 Set compression semaphore to fair" into stable/boron

Merge "Bug 6745 Do not ignore syncup return value" into stable/boron

Merge "Bug 6745 Remove thread renaming and unnecessary logging" into stable/boron

Merge "Bug 6745 Fix replacing in compression queue" into stable/boron

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

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 "Fix translation to *packet.received.MatchBuilder" into stable/boron

Merge "BUG-6118: making the OFentityListener aware of the InJeopardy() flag" into stable/boron

Merge "Bug 6633 : NXM_OF_IN_PORT support in openflowplugin" into stable/boron

Merge "ClusterSingletonService cleaning FRM/FRS" into stable/boron

Merge "Bug 6749: Set the openflow connection config at xml file" into stable/boron

Merge "Update comments and imports after DataChangeListener changes" into stable/boron

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>

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:0->NXM_OF_IN_PORT[]).

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

Update comments and imports after DataChangeListener changes

Change-Id: Ic8174cc2627deb12d48f6d41444f2e9b4dc60406
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>
(cherry picked from commit 049ffd00ea4e24fe3fb3d4f6abeaf51390ddd316)

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>

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>
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

Merge "Bug 6665 Clean code" into stable/boron

Merge "SONAR TD - StatisticsContextImpl, StatisticsManagerImpl" into stable/boron

Merge "Update comments around flat-batch service" into stable/boron

Merge "Update old links in code to deprecated DataChangeListener" into stable/boron

Merge "Convert OF samples to use DTCL instead of DCL" into stable/boron

Merge "Convert openflowplugin-it to use DTCL instead of DCL" into stable/boron

Bug 6665 Clean code

Change-Id: Ia87a66c37f906a3351f3a982209f6778f7c8a311
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.com>

ClusterSingletonService cleaning FRM/FRS

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

Merge "Bug 6665 - Fix switches scalability" into stable/boron

SONAR TD - StatisticsContextImpl, StatisticsManagerImpl

- Reduced cyclomatic complexity of
  StatisticsContextImpl.chooseStat
- Reduced cyclomatic complexity of
  StatisticsManagerImpl.pollStatistics
- Fixed naming of instance fields to match
  '^[a-z][a-zA-Z0-9]*$' expression

Change-Id: Ieb9d06cda534b01240ba2a952c927667ece5565b
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
(cherry picked from commit 5c853ea0a399706a0a475638f9c542945a993bd9)

Merge "Bug 6542 FRS - prevent concurrent reconciliation node config add" into stable/boron

Update comments around flat-batch service

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

Convert openflowplugin-it to use DTCL instead of DCL

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

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>

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>

Bumping versions by 0.0.1 for next dev cycle

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

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

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

BUG-6595: Adding a retry for the issue when the
mastership of the switch hasnot been established while
the port status is processed

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

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

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

Revert "BUG6595: Adding a retry for getting a transactionsafely"

This reverts commit b26b58ec652a1e7d8eb77f571b39604bf5e3cc86.

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

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>

BUG6595: Adding a retry for getting a transactionsafely

since the RCA for the issue pointed to the fact that the
mastership of the switch hadnot been established while
the port status is processed.

Change-Id: If243a051b8417b35160144213dd268f1c2bf4225
Signed-off-by: Shuva Kar <shuva.jyoti.kar@ericsson.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>

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

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>

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>

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>

Bug 5596 - restart devices management improvement

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

Bug 6357 - FRM integration with singleton

 - using ClusterSingletonServiceProvider instead if EOS
 - junit tests update

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

Bug 5596 Initial commit failed

- added closing connection after unsuccessful initial submit
- added enum with possible connection state after handshake

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

DeviceFlowRegistry - prevent NPE

- Added more nonNull filters to DeviceFlowRegistry.fillFromDatastore
- Added nonNull filters when printing debug info of added flows
  in LifecycleServiceImpl
- Added more JUnit tests to test fail cases of DeviceFlowRegistry.fill
- More detailed log output for DeviceFlowRegistry

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

Bug 5596 Changes when closing device

- speed up device closing process
- prevent to wait for closing device context
- log improvements
- Added nonNull filters when filling DeviceFlowRegistry
- Updated unit tests to test for above case

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

Bug 5596 Cleaning lifecycle conductor

- removed lifecycle conductor

Change-Id: I7143726358c7d2145e2109e269e8d50e2c865a1f
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Cleaning part 4

Change-Id: I967cab5cb220bfbd91ff32552c0d9e8e157ab0fd
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Cleaning part 3

- Role Context
- Role Manager
- Rpc Context
- Rpc Manager

Change-Id: If9a144950a86abfef51fb3cb70c2a22e52e7fa37
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Cleaning part 2

- Statistics Context
- Statistics Manager

Change-Id: I29fc3c358207039184811e09d5c98368656a9ee0
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Cleaning part 1

- Device Context
- Device Manager
- Device State

Change-Id: I85a1269f1acc17a3e09cbe1d92fb993a9be5a4da
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Added cluster provider

Change-Id: I1954b28bfaaf026aebb171fb5c51ef26a6df585c
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Change of start up services

Change-Id: I6bae36251f4d01e6d851f8f09a976e790665ef20
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5596 Created lifecycle service

- created general API for cluster services
- changed OFPContext and OFPManager API

Change-Id: I6506a1dcc328e98aca21446d46b8dd09445a4b61
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 6133 - and more improvements

- LOGs improvements
- RoleContext closing speed up
- Race condition for protocol v 1.0
- OVS remains in disconected state bug 6133

Change-Id: I220c9ab84357c58c4b6887bec76fe274fec7a03f
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>
(cherry picked from commit 66149ccf887da6a65b9a7287b8a43ab9e1b49663)

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>

Merge "Bug 6366 - of-switch-config-pusher - DTCL instead of DTL" into stable/boron

Merge "Sonar - technical debt of FRS app" into stable/boron

Adding learn action - Bug 6445

Change-Id: Iab58e54230003dc8a5ea48bbc7a4d173c305cf8f
Signed-off-by: Slava <slava.radune@hpe.com>
(cherry picked from commit 122bf80)

Merge "Bug 6374 topology-manager - DTCL instead of DTL" into stable/boron

Merge "SONAR TD - FlatBatchUtil, DeviceInitializationUtils" into stable/boron

Merge "SONAR TD - Remove unused fields, fix naming" into stable/boron

Merge "SONAR TD - Group actions redundancy" into stable/boron

Merge "SONAR TD - Sal*ServiceImpl" into stable/boron

Merge "SONAR TD - AbstractService" into stable/boron

Merge "SONAR TD - *ToNotificationTransformer private constr." into stable/boron

Merge "Technical debt MeterUtil, GroupUtil, FlowUtil" into stable/boron

Merge "Drop Felix Gogo" into stable/boron

Merge "Bug 6372 table-miss-enforcer - DTCL instead of DTL" into stable/boron

Removed duplicate declaration in pom.xml.

The pom.xml in the model directory had duplicate declaration of
ietf-yang-types-20130715.

Change-Id: Ib9164412a8e564f98fc92892678bd86196b77734
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>
(cherry picked from commit 294d813a27bc402b462a9565683aa90e1f2a4200)

Merge "Bug 6278: Switch to use odlparent's karaf-parent" into stable/boron

Sonar - technical debt of FRS app

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

SONAR TD - Remove unused fields, fix naming

- Added private constructor to EventsTimeCounter
- Removed unused fields
- Fixed naming of constants

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

SONAR TD - FlatBatchUtil, DeviceInitializationUtils

- Removed unused fields
- Reduced cyclomatic complexity of code
- Added private constructor to DeviceInitializationUtils

Change-Id: I280c9534ad87a6d6826e2a209d07edb1ff25c624
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
(cherry picked from commit 81ec040affdf77f72ae1060436e7010386f410ed)

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

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

SONAR TD - Group actions redundancy

- Moved logic that converts list of action types to group actions bitmap
  to single method in GroupUtil

Change-Id: Ie8fc71945c4a5412c5adf88cb66d26f2d9a9a229
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
(cherry picked from commit 44da2880eee5acbc7da23f82c4807b0337efed66)

SONAR TD - Sal*ServiceImpl

- Changes some anonymous classes to named inner classes
- Shortened big lambda in SalFlatBatchService

Change-Id: I04d9f73ba92928144b2de1d2a86b8c4feac0ad22
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
(cherry picked from commit 5c2b5676715352cc935daa146c660e6b79c7bd03)

SONAR TD - AbstractService

- Remove WAIT_TIME, PRIMARY_CONNECTION unused fields
- Changed buildRequest to throw ServiceException
  instead of generic Exception

Change-Id: If5620887623c2119065a4120e91189120441bbda
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
(cherry picked from commit c53071b939c82b032d8ec6468c531f4abd79fa4c)

SONAR TD - *ToNotificationTransformer private constr.

- Added private constructors to ToNotificationTransformer classes

Change-Id: If64db7a5922ffdc613685c56d077ada7d1c38c7f
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
(cherry picked from commit 70a6663897b6d25e7748a148b74f6d12bb51a0b3)

Technical debt MeterUtil, GroupUtil, FlowUtil

- Reduce anonymous classes number of lines
- Remove unused "LOG" field

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

Drop Felix Gogo

This is obsolete and appears to be unused.

Change-Id: I755c976567bde2747ec332b0834cc58fdc2ee22d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
(cherry picked from commit 96eb290ba1b9f05606f83541453a5d7d07898158)