Merge changes from topic 'BUG-4117'

* changes:
  BUG-4117: add impl for FlowCapableNode Connector notif.
  BUG-4117: add impl for FlowCapableNode notification

Merge "BUG-4117: preparation for FlowCapableNode notification impl."

Merge "Bug 5914 - Flow statistics don't show up on the same flow id, if flow uses IPv6 match with subnet mask"

Bug 5692 - Direct statistics RPC

- Added new RPCs for direct gathering of statistics
  with optional flag of storing the statistics to DS/operational

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

Bug 5692 - Statistics ON/OFF

- Removed DeviceState.getRole
- Statistics are properly shut down when device changes to state other than Master
- Statistics are properly started only when device becomes Master
- LifecycleConductor now have access to StatisticsManager

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

Bug 5914 - Flow statistics don't show up on the same flow id,
if flow uses IPv6 match with subnet mask

Patch2: Added unit test

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

Merge "Bug 5591 - Changed DeviceContext to DeviceState in translate methods of MessageTranslator classes - Added util static method createWithOook to AggregateFlowsInTableService   and to OpendaylightFlowStatisticsServiceImpl (calls DeviceContext.oook() before   creating instance of the service)"

Bug 5590 - writeToTransaction+submitTransaction
- New interface TxFacade, DeviceContext now implements this interface
- Changed constructor of SalTableServiceImpl to accept TxFacade and NodeId
- Changed MultipartRequestOnTheFlyCallback to not need the DeviceContext and accept TxFacade
- Updated some methods in StatisticGatheringUtils to not depend on DeviceContext (because of change above)
- Updated usages of these classes according to changes.
- Updated tests

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

Bug 5591
- Changed DeviceContext to DeviceState in translate methods of MessageTranslator classes
- Added util static method createWithOook to AggregateFlowsInTableService
  and to OpendaylightFlowStatisticsServiceImpl (calls DeviceContext.oook() before
  creating instance of the service)

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

Merge "BUG-5573: Altering the HandshakePool to have a LinkedBlockingQueue instead of an ArrayBlockingQueue with size 20. The main difference between ArrayBlockingQueue and LinkedBlockingQueue is clear from constructor one has underlying data structure Array and other linkedList."

Bug 5543 - JUnit tests update

Change-Id: I7bfcec5ad87213870d3043a3ea00c098e21391b9
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Merge "BUG 5680 - TxManager init submit fix"

BUG-5573: Altering the HandshakePool to have a LinkedBlockingQueue instead of an
ArrayBlockingQueue with size 20. The main difference between
ArrayBlockingQueue and LinkedBlockingQueue is clear from constructor one
has underlying data structure Array and other linkedList.

ArrayBlockingQueue uses single-lock double condition algorithm and
LinkedBlockingQueue is variant of the "two lock queue" algorithm and it
has 2 locks 2 conditions ( takeLock , putLock)

In scenarios that involve adding and removing elemenst LinkedBlockingQueue
performs better than ArrayBlockingQueue.

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

Merge "Bug 5592 - NodeId+DatapathId in Services (remove dependency on DeviceContext) - Changed AbstractService.getVersion and AbstractService.getDatapathId to return data from DeviceState - Added AbstractService.getNodeId what returns nodeId from DeviceState - Updated all ocurences of getting above 3 values in subclasses to use these methods - Updated tests (in most of them, DeviceState was not mocked)"

Merge "BUG-4148: Improving the logging levels, adding more information wherever required"

Merge "BUG-5841: Enhances bulk-o-matic to stress test Data Store and Openflowplugin RPC Added asciidoc for the same"

BUG-5841: Enhances bulk-o-matic to stress test Data Store and Openflowplugin RPC
Added asciidoc for the same

- Takes JSON input from REST to provision flows
- Different options to push to DS or Openflowplugin
- Results are printed to karaf.log

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

Merge "BUG-5839: Removing groups and meters on being stale-marked"

Merge "BUG-4148: Improving the logging in flow/group/meter to provide more contextual information as to the success/failure of the operation"

Merge "Add custom compare for ArpMatch objects"

Bug 5592 - NodeId+DatapathId in Services (remove dependency on DeviceContext)
- Changed AbstractService.getVersion and AbstractService.getDatapathId to return data from DeviceState
- Added AbstractService.getNodeId what returns nodeId from DeviceState
- Updated all ocurences of getting above 3 values in subclasses to use these methods
- Updated tests (in most of them, DeviceState was not mocked)

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

Add custom compare for ArpMatch objects

The problem here was that layer3MatchEquals was falling through to
the default compare where the generated equals() method is called
which was comparing the mac addresses case sensitive. This caused
an IT test we're working on in netvirt to fail because certain ARP
flows were not making their way into the operational store. This
only occurs on the beryllium branch but not on master as this
method has a fully different implementation there.

Change-Id: I3aa6e79b48741f656ae036b16743e0e097623593
Signed-off-by: Josh <jhershbe@redhat.com>

BUG 5680 - TxManager init submit fix

Initial submit could fail with unexpected reason and we have to catch it, because we could have distributed DS without corect Node path and sub-path initialization. Added initCommit property.
TODO: think about alternative solution (try to reinit whole Node)

Change-Id: I6a12fb1a9542312408d76a774dd7c0ff66328e47
Signed-off-by: andrej.leitner <anleitne@cisco.com>

Bug 5595 - RpcContext nodeId+xidSequencer (remove dependency on DeviceContext)
- Changed RpcContextImpl to not depend on DeviceContext.
- Moved isStatisticsRpcEnabled to DeviceManager and DeviceContext
- Updated RpcContextManagerImpl to use new RpcContextImpl constructor
- Updated tests

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

Bug-5523 Lifecycle conductor changes

- changed lifecycle conductor from static thread safe singleton
to singleton created in OpenFlowPluginImpl as are managers defined

Change-Id: I1bd21b2e3a842ce3152882a746a2b9efd3f2cb83
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.sk>

Bug-5523 CloseConnection changes

- dependency of deviceContext removed from starting
- lifecycle conductor thread safe now
- role manager was not removing context from map
- test update

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

Bug-5523 RoleManager changes

- all functionality moved to role manager
- role context hold only information about status
- life cycle conductor defined as listener to role change
- able to register listener to start/stop master services
- timer for initialization device disabled
- added thrown exception when not able to write to transaction
- added junit tests

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

Merge "Remove nexusproxy property"

BUG-5839: Removing groups and meters on being stale-marked

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

BUG-4148: Improving the logging levels, adding more information wherever required

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

BUG-4148: Improving the logging in flow/group/meter to provide more contextual
information as to the success/failure of the operation

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

WIP: BUG-5464: scaling issue

    - pull up table-features to flow-cabable-node
    - fix adjanced logic
    - remove augmentation of flow-capable-node (redundant table-features)
    - fixed unit test

Change-Id: I798f87420330cfd71045eac3de61291a10abdafa
Signed-off-by: Michal Rehak <mirehak@cisco.com>
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Merge "Derive distributuon/karaf pom from karaf-parent"

Merge "Log at ERROR level when add/remove flow fail"

Log at ERROR level when add/remove flow fail

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

Merge "Log at ERROR level when request context failed to close"

Log at ERROR level when request context failed to close

And also log the whole exception as the message can be null

 RequestContextUtil               | 190 - org.opendaylight.openflowplugin.impl - 0.3.0.SNAPSHOT | Request context wasn't closed. Exception message: null

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

Log at ERROR level when the queue isn't set...

... as this results in failing the request.
This helps debug, and I believe should be turned
at error level

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

Derive distributuon/karaf pom from karaf-parent

This picks up the recent changes to the karaf distro from a common
place.

Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Merge "Bug 5589 - Deprecate PortNumberCache"

Merge changes from topic 'bug-5574'

* changes:
  BUG-5574: added flat-bulk service registration
  BUG-5574: flat batch proposal
  BUG-5574: bulk meters proposal
  BUG-5574: introduces sal-groups-batch service impl
  BUG-5574: sal-flows-batch proposal

BUG-5574: added flat-bulk service registration

    - added registration/unregistation for flat-batch routed service

Change-Id: I95d005aeb12d1b817c643ffc11a6f5343361d5a4
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit b5ffe8525d85dcf92d38a1eab96e4f3d2836aad0)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5574: flat batch proposal

    - yang model
    - implementation proposal
    - flat batch util
    - unit tests

Change-Id: I4406eaa19024348e63e2cc6d07039949ba8aa941
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 60ddc33574168296cf1b59f59a1cbb998cecfc36)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5574: bulk meters proposal

    - added bulk API implementation for groups
    - with parametrized barrier-after support
    - MeterUtil
    - unit tests

Change-Id: I9146894ae0b1f630706b8aea6b85a13c8ce3364b
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 26fa4ba6f87c23e159b74526f6fb849f0ceb4783)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

Bug 5589 - Deprecate PortNumberCache

- New class - Port to NodeConnectorRef converter
- New test - Conversion between port and NodeConnectorRef
- Methods in PortNumberCache are now deprecated
- Replaced use of above methods in PacketReceivedTranslator with new methods
- Modified tests for PacketReceivedTranslator to work with new methods

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

Bug 5747 Introduced snapshot gathering marks

Added timestamp of statistics collection start and finish
and successful collection flag.

Change-Id: If00fbb7532f292d69602de7857db73fabc5fd49f
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

BUG-5574: introduces sal-groups-batch service impl

 - added bulk API implementation for groups
 - with parametrized barrier-after support
 - added tests

Change-Id: If3e3d70d4ceb8eeb4a9902e2ea7f4bad26d9c934
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 134bd7a822a5a3d76ec82f3bd7ad6abf1a933a60)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

BUG-5574: sal-flows-batch proposal

    - added common model for attaching barrier
    - added bulk API implementation for flows
      - with parametrized barrier-after support
    - added tests

Change-Id: I4e941d3654dc142e1d1d64a2258e629016e95a0e
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 79e8e834f0cbbccfbff99e51e4ddd854dd812d19)
Signed-off-by: andrej.leitner <anleitne@cisco.com>

Revert "Fix "Bug 5589 - Deprecate PortNumberCache""

This reverts commit 0cd77f8ef3c8045fba03dde20044a08ec44e29f5.
Signed-off-by: Michal Rehak <mirehak@cisco.com>
Change-Id: I852bf5dc979ef8a581b3d8326413451cd21cc83a

Remove nexusproxy property

Change-Id: Ie6cc211e3e4a5b39ef6a09c23cb737eab6c57a96
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Merge "Bug 5592 - DeviceContext -> DeviceState in *ToNotificationTransformer classes"

Merge "Fix "Bug 5589 - Deprecate PortNumberCache" - New class that can convert DeviceState to NodeConnectorRef and NodeConnectorRef to port number. - 1 test that will verify if conversion between DeviceState and NodeConnectorRef is valid. - Made methods in PortNumberCache deprecated and replaced use of them in PacketReceivedTranslator with new methods. - Modified tests for PacketReceivedTranslator to work with new methods."

Fix "Bug 5589 - Deprecate PortNumberCache"
- New class that can convert DeviceState to NodeConnectorRef and NodeConnectorRef to port number.
- 1 test that will verify if conversion between DeviceState and NodeConnectorRef is valid.
- Made methods in PortNumberCache deprecated and replaced use of them in PacketReceivedTranslator with new methods.
- Modified tests for PacketReceivedTranslator to work with new methods.

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

Do not deploy the karaf artifact

The karaf artifact is only used for testing. It does not need to be
released to the world.

Change-Id: Ieca324babbf2cf8ac710808fdf40134981ed38e5
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug-5534:RoleService does not take care of rpc errors

Change-Id: Id65e597f5952b689dec176c29ae9e78284123ce2
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Bug 5592
- DeviceContext -> DeviceState in *ToNotificationTransformer classes

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

BUG 5523 RoleCtx changes

 *add onDeviceContextLevelDown method
  * simplify RoleContext close process
  * add timer for clean (fix cluster sync. problems)
 *simplify ownershipChange process
 *RoleContext with one semaphor only
 *add new method for prevent misunderstanding process in RoleContext
 *capsulate Semafors locks in RoleContext
 *add new state markers (isMainCandidateRegister, isTxCandidateRegistered,
  getClusterRole)

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

BUG 5523 Add onDeviceContextLevelDown for Statistics and Rpc Managers

 *change Termination order in OFpluginProvider
 *add onDeviceContextLevelDown methods
 *add pair NodeId with Context for every manager
  *only NodeId is keyProtection with not finished unregistration
   for every shutting down order

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

BUG 5523 DeviceManager handler for connection and disconnection too

 *Init and close DeviceCtx suite has contract with ConnectionCtx
  so we'd like to have only one way for Clean DeviceCtx suite :
onDeviceDisconnected event for primary connectionContext.
 *add timer for shuttinDown TxManager because Cluster
 *clean TxManager close txChain handling

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

BUG 5523 - DeviceContext additional functionality

 *add shutdonwConnection for use in OFP instead deviceCtx.close
 *add DeviceContext states object
 *add method for get actualState
 *TransactionChainManager
   *remove read DS by shuttingDown process
   *remove close tx possibility
   *Guard submitIsEnabled property (only active TxChainFactory
is real marker for set property to true)

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

BUG 5523 - API changes

Fixing closing process for DeviceContext

* rename existing Interfaces
* add method for registration TerminationHandler
* implement TerminationOrder
* fix tests

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

Bug-5486:Device disconnected handers on disconnection get closed in no particular order

HashSets dont gaurantee any retrieval order. Further we dont need hashing as we just iterate through it.

Some log changes.

Change-Id: Ia31ce404e44e78bd0a3483e969da06119afe75f0
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Merge "Bug-5551:Audit of Connection Events"

Bug-5562 - Ipv4 L3-ArbitraryMask patch
Current code expects the Arbitrary Mask to be normalized from switch.
Example:- 3.3.3.3/255.0.255.0 to 3.0.3.0/255.0.255.0
Not all switches normalizes arbitrary bit mask in the flows when they are injected.
Example:- 3.3.3.3/255.0.255.0 to 3.3.3.3/255.0.255.0
This patch ensures that the operational data store is in congruent with the flows
that are shown on the switch. Patch is built on checking the mask & comparing the
normalized stores & stats flows which ensures Ipv4 L3-ArbitraryMask works independant
of how the switches normalizes the ip addresses when flows are pushed.

Conflicts:
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/IpConversionUtil.java
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/match/MatchConvertorImpl.java

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

Bug-5551:Audit of Connection Events

Change-Id: Ida3e632e80add4491e29e6c014a94a19c7ed9909
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Merge "BUG-5489 Group Delete not working with ovs2.5"

Merge "Fix Mockito Generics"

Bug 5521 - flows reconciliation

Fix missing flows by initialization/reconciliation

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

Fix Mockito Generics

 *fix Generic for mockito matcher
 *fix testGroupModConvertSortedBuckets

Change-Id: I8148c94cf572440a1cb9be7a0d962c8501cb3f42
Signed-off-by: andrej.leitner <anleitne@cisco.com>

Merge "Correction classes names in loggers."

Merge "Post "Clustering optimization" updates"

BUG-5489 Group Delete not working with ovs2.5

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

Suppressed INFO log messages.

According to the Logging Best Practices in the OpenDaylight wiki,
messages at INFO level should be for operators and administrators.
However, some log messages for developers inappropriately used the INFO
level. This patch changed the log level of the messages from INFO to
TRACE.

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

Merge "Bug:3026 - Echo response timeout needs be exported to configuration"

Correction classes names in loggers.

- this are probably copy paste errors

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

Post "Clustering optimization" updates

3/2/16

- small updates after clustering optimizations
- grammar policy for method names, class name and logs
- clean ups

3/3/16

- deviceState class, set SLAVE after init
- deviceStateTest updates

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

Merge "Quick fix RPCs and DevicCtx.close"

Bug:3026 - Echo response timeout needs be exported to configuration

Takes care of Bug-4677 : Li: Knob to configure Barrier duration

Also incorporated Jozef bacigal's hot fix in stable

Change-Id: I42697dbbb93cc40c1547b92e9665851e7f5f0b98
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>
(cherry picked from commit e22904723888cfe73fff63e280e4c4c5af51b910)

Bug-5119 - Adding support for L3 address arbitraty bit mask with Unittests coverage & indentation & addressed latest comments &  rebased

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

Quick fix RPCs and DevicCtx.close

Change-Id: I1cd92d91e1025ae3aee00418b86912f2e86580b4
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 5404 - He plugin: Meter stats not reported the first time switch connects to controller

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

Do not retain DeviceState

TransactionChainManager only needs the instance identifier (or Node Id,
available from there), do not retain the entire state.

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

Do not leak internal state

The state transition enum should not be visible from outside, hide it.

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

Do not lose thread on close

We need to make sure we shutdown the spypool on close.

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

Use IetInetUtil conversion

Instead of home-grown covnersion, use the utilities provided.

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

Fix a guaranteed NPE

If we fail to decode a packet, we will end up causing a NPE in
statistics. Use the packetIn class for that instead.

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

Do not override java source/target

odlparent already provides correct definitions (which are Java 8), so
use those.

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

Merge "Bug 4957 Cleaning code and fixing issues from manual tests"

Merge "Bug 4957: OF Role processing changes"

Merge "Bug-4957 Cluster Role change fix"

Merge "Bug-4957: Make async operational DS Read"

Merge "Bug-4957 Fix SalRoleService blocking call"

Merge "Bug-4957: Double candidate, close processing"

Merge "Bug-4957: Double candidate, introduced roleCtx states"

Merge "Bug-4957: Double candidate, clean up"

Merge "Bug-4957 Double candidate Onwership initial commit"

Merge "Bug 4957 Fix Cluster Role change for Statistics Polling"

Merge "Bug 4957 Fix Cluster Role change for RPC registration"

Bug 4957 Cleaning code and fixing issues from manual tests

 *Initialization Device info collecting
  -Device Info collecting is async function and it could finish
   with unactive connection (bug stat collecting always return true)
   so we need to check actual state before we finish MASTER
   roleChange.
  -We don't wish to commit any data and we are able to close
   NEW Transaction. So we try to do it.

 *SalRoleServiceImpl changes
  -add result FutureCallback for release semafor guard
  -add check for cancel for every loop
  -fix return failed future for InterruptedException

 *Change close method for all Managers
  -Fix close method in RoleManager we not suspend TxEntity
  -Use consumingIterator for cleaning manager's contexts
  -Use concumingIterator for close in RpcContext

 *Additional clean
  -clean RoleContext for TxLockOwned
  -fix not propagated Excepton from DeviceContext#onDeviceContextLevelUp
  -clean RoleManager
  -add check for Connection state for statistics
  -fix tests

Change-Id: I04d19e4c0a0e87e4158cd2ca25da32fe15cfef22
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Merge "Pre-compute ipv4 masks"