Fix codestyle

- explicit type

See also:Bug-8607

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

Fix codestyle

- unnecessary boxing removed

See also:Bug-8607

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

Fix possible NPE on ContextChainHolderImpl

- deviceConnected method

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

Do not mark device as connecting when closing it

Do not put incoming device connection to connectingDevices map when it
is still in closing state, to prevent having stale entry in
connectingDevices map.

Resolves: bug 9088

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

Redesign statistics context and manager

- To simplify statistics context and statistics manager code,
have better control over statistics polling and to remove
referencing of statistics manager from statistics context,
remove current method of statistics scheduling and replace
it with Guava scheduled service that will handle starting,
periodic gathering and stopping of statistics.
- Simplify chain of steps that continues with statistics initialization
with reconciliation framework enabled

See also: bug 8913

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

Merge "BUG8607 Fix checkstyle issues"

BUG8607 Fix checkstyle issues

- Formatting
- JavaDoc
- Line lengths
- Name of variables
- Exception catching
- Remove redundant modifiers
- Add default case to switch
- Move variables closer to their

- Enforcement in pom.xml

Change-Id: If17fcdc00e8d22818a5501027a6bb7db3f2dcba3
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Enforce checkstyle

Enforce checkstyle by changing severity to error.

Change-Id: I4a6ae191bfec6df72e17f1c8ca50e92a9fd68ba5
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Merge "Remove CSS artifact remnants"

Merge "Fix checkstyle warnings for impl/protocol test package"

Merge "Fix checkstyle warnings for impl/karaf, lifecycle, common, mastership"

Merge "Fix checkstyle warnings for impl/device package"

Merge "Fix checkstyle warnings for impl/datastore package"

Merge "Fix checkstyle warnings for impl/protocol package"

Remove CSS artifact remnants

The project has already been converted to blueprint (except for test-extension)
so removed he remaining CSS remnants (eg use of config-parent, JMXGenerator).

Change-Id: I38c3e16ece82e2f5deada17a3faafa180e11a210
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle warnings for impl/datastore package

See also: bug 8607

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

Fix checkstyle warnings for impl/device package

See also: bug 8607

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

Fix checkstyle warnings for impl/karaf, lifecycle, common, mastership

See also: bug 8607

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

Fix checkstyle warnings for impl/protocol test package

See also: bug 8607

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

Merge "Fix checkstyle warnings for impl/role package"

Fix checkstyle warnings for impl/protocol package

See also: bug 8607

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

Remove unsupported statistics warning

Remove warning for unsupported statistics type when deleting all groups,
flows and meters since it is unwanted.

See also: bug 8607

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

Fix checkstyle warnings for impl/role package

See also: bug 8607

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

Solves issue with two connections from one device.

See also:Bug 8723

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

Fix checkstyle warnings for rpc package

See also: bug 8607

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

Fix checkstyle warnings for services package

See also: bug 8607

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

Fix checkstyle warnings for statistics package

See also: bug 8607

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

Fix checkstyle warnings for translator and registry package

See also: bug 8607

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

Fix checkstyle warnings for util package

See also: bug 8607

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

Merge "Remove deprecated bootstrap channel options"

Remove deprecated bootstrap channel options

- LOW and HIGH WATERMARK are deprecated

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

Add RoleManager and RoleContext

- Move sending of MASTER and SLAVE role from DeviceContext
  and ContextChainHolder to new RoleContext and RoleManager
- Use hashed wheel timer for setting SLAVE role on device in
  case MASTER role was not propagated instead of ItemScheduler
- Remove retries from SalRoleService

See also: bug 8913

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

Guard lifecycle of contexts

- Add GuardedContext wrapper for OFPContext that will guard each
phase of context lifecycle (new instance, instantiateServiceInstance,
closeServiceInstance, close)

Resolves: bug 8913

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

Merge "Update ONF tests"

Update ONF tests

- previously ingored tests fix

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

Fix checkstyle impl/common and impl/configuration

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

Fix chectyle warning.

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

Merge "Fix checkstyle"

Merge "Remove deprecated CheckedFuture."

Merge "Move extension implementation from API to impl"

Fix checkstyle

- on openflowplugin.api
- set default checkstyle to error instead of warning to prevent new checstyle violations

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

Remove deprecated CheckedFuture.

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

Bug 8902 - Changes in FRM for Reconciliation Framework

Change-Id: Id063eb8be61396113d0b10fabb43f0d64d02f6a8
Signed-off-by: gobinath <gobinath@ericsson.com>

Merge "Bump reconciliation"

Fix version reference

This component has not been adjusted to reflect version bump,
fix that.

Change-Id: I9702ef9dbc9b56faf9ef43d6ad7565da1bbaf0f9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump reconciliation

Change-Id: Ic35a2543fbb3864134bd11860ef7d889f47882e2
Signed-off-by: Gobinath <gobinath@ericsson.com>

Merge "Bug 8902 - Reconciliation Framework changes"

Move extension implementation from API to impl

Currently, some parts of openflowjava extension implementation is
located in API and not in implementation project as it should be, so
move this code from api to implementation.

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

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

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

Merge "Bump odlparent 2.0.2 to 2.0.4"

Bump odlparent 2.0.2 to 2.0.4

Change-Id: I168975753b4ea7728eb361f7ef7803fbb970f64b
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Merge "Update unit tests pt2"

Merge "Update unit tests"

Remove unused code from He design

See also: bug 8656

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

Update unit tests pt2

- Add unit tests for single layer sal services
- Add some missing unit tests for device context
- Add unit test for mastership service delegate
- Fix sal table service for single layer

See also: bug 8607

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

Update unit tests

Add unit tests for:
- Arp target transport address serializer
- Packet in message deserializer
- All serializers for multipart request messages
- All serializers for table feature properties and match fields in table
features
- Configuration service and OpenFlowProviderConfigImpl
- Device initializers (OF10 and OF13)
- Single-layer direct statistics
- Request context and input utils

Remove custom comparators for sorting of actions and instructions and
use OrderComparator.build() instead. Also, add missing sorting of table
feature instructions and actions properties.

See also: bug 8607

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

Bug 8902 - Reconciliation Framework changes

Changes related to the Reconciliation Framework are done in this patch.
The APIs needed to be implemented by the services registering to the
Reconciliation framework are present in the interface
ReconciliationNotificationListener.The APIs to register the service to
Reconciliation framework and the APIs to start the reconciliation /
cancellation of the ongoing reconciliation of a node are present in
ReconciliationManager.

Change-Id: I8bd1845db8b86f4a4314977e05c6b699592f5658
Signed-off-by: gobinath <gobinath@ericsson.com>

Merge "Bug 8873 - Bundle based reconciliation to enable bundling of messages"

Bug 8873 - Bundle based reconciliation to enable bundling of messages

Change-Id: Ida09badca9a16619fec6e6f293042956ce8ec4e4
Signed-off-by: eeiillu <suja.t@ericsson.com>

Merge "Bug 8882 - With conntrack SNAT communication with PNF fails "

Change return type of events

- remove callback from onDevicePrepared
- added ListenableFuture as return type for events
- added many tests
- small fixes found on test run

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

Reconciliation framework binding point improvements

- added tests for mastership service manager
- removed factory managed instantiated direct thought blueprint service
- reconciliation framework returns registration, can be closed directly
- hide explicit unregister method from manager to prevent confuse the consumer
- exception thrown after second reconciliation framework try to register
- reconciliation framework doesn't need register mastership service for disconnect
- added new event for device disconnect into reconciliation framework events

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

Merge "Remove logback related stuff"

Merge "Lower logging level of FRM reconciliation process"

Lower logging level of FRM reconciliation process

Change most of logging to match logging of OpenFlowPlugin sal services
(add and deletion and failures are logged on debug levels) to remove
unnecessary clutter from logs.

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

Remove missing inPort in PacketIn error

Make new PacketIn message handler behave like old one, and that is that
when there was no inPort, we just set null ingress for packet in
notification.

See also: bug 8772

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

Bug 8882 - With conntrack SNAT communication with PNF fails

Added support for load metadata action.

Change-Id: Ia1b0454a4aeccd9a04cecb59b50f516264bddb3e
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Remove logback related stuff

as discussed on https://lists.opendaylight.org/pipermail/odlparent-dev/2017-July/001262.html

openflowplugin-it appears to be completely broken anyway, and is not in build

Change-Id: Ic52d5d38754f2692415788d364969d42eadff1e0
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "Remove config knob for reconciliation framework"

Remove config knob for reconciliation framework

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

Close CSS registration in separate thread

Instead of sending initialization of CSS in separate thread,
do closing of registration in separate thread and only in case
we are closing the registration during initialization, to prevent
CSS freezes. Also, when closing CSS service, add checks for connection
state to prevent closing context when connection is already RIP.

See also: bug 8805

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

Exctract onDevicePrepared to interface

- In the previous implementation the client was able to register
  many of reconciliation frameworks as a mastership service.
  This may lead to broken code where each registration of
  mastership service could implement own onDevicePrepared.
- Issue with configuration parameter isUsingReconciliationFramework
  solved by putting this status into statistics context constructor.
- Removed depricated deviceInfo.getLOGvalue from statistics.

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

Extract mastership blueprint service

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

Mastershipchange service implementation.

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

Mastership service provider API

- create new API for mastership listener
- rename MastershipChangeListener to ContextChainMastershipWatcher

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

Fix context chain initialization and SLAVE change

- Merge ContextChain and LifecycleService
- Let ClusterSingletonService handle stopping of mastership services in
closeServiceInstance method
- Store cluster singleton initialization future and process it
synchronously in case we are closing
- Cancel device initialization future if we got timeout
- Handle initialization chain order in ContextChain instead of storing
each next phase in contexts
- Remove SLAVE task from scheduler in case we acquired SLAVE role
already
- Prevent adding exactly same auxiliary connection twice to list of
auxiliary connections in ContextChain
- When SLAVE role is obtained from cluster singleton service, notify
MastershipChangeListener about it
- Log exceptions from handshake as warning instead of error (as only
problem is that device gets disconnected and reconnected, what is not
critical)
- In case context chain is still closing and we receive new connection,
drop the connection and let context chain finish closing

See also: bug 8805

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

Merge "Remove He-design inventory manager"

Merge changes from topics 'bug/8836', 'bug/8772'

* changes:
  Fix match extensions deserialization
  Add missing augmentations for extensions
  Bail out earlier on alien message

Fix match extensions deserialization

Fix transformation of match extensions for different match types during
deserialization so they will not be lost during this process.

Resolves: bug 8836

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

Merge "Upgrade to odlparent 2.0.2"

Add missing augmentations for extensions

- Add apply actions augmentations for nodes->node->table->flow
  ->instructions to action convertors for nicira extensions
- Add new packet in message augmentations to match convertors for nicira
  extensions

See also: bug 8772

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

Bail out earlier on alien message

Add boolean return to onAlienMessage method that shoudl return true if
alien message was successfully processed. In this case, do not try to
process received OfHeader message as it was processed in alien message
listener to prevent errors when alien message is received when dependant
project is for example gathering statistics.

See also: bug 8772

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

Remove He-design inventory manager

See also: bug 8656

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

Merge "Remove He-design statistics manager"

Upgrade to odlparent 2.0.2

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

Merge "BUG-8607: Fix issues in checkstyle enforcement for module openflowplugin-common"

Remove He-design statistics manager

See also:Bug8656

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

BUG-8607: Fix issues in checkstyle enforcement for module
openflowplugin-common

- Copyright section
- Line lengths
- JavaDoc
- Formatting
- Enforcement in pom.xml

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

Remove config subsystem from OpenFlowJava

- Remove config subsystem from OpenFlowJava, because it was replaced
  by blueprint and it is already removed from OpenFlowPlugin

See also: bug 8747

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

Integrate OpenFlowJava into OpenFlowPlugin build

- Integrate OpenFlowJava artifacts building into OpenFlowPlugin maven
build
- Merge openflowjava-artifacts with openflowplugin-artifacts
- Remove odl-openflowjava-all feature, because it was unused and
contained only odl-openflowjava-protocol
- Change org.opendaylight.openflowjava groupId to
org.opendaylight.openflowplugin.openflowjava
- Move sample-client to OpenFlowPlugin samples and fix missing license
headers in sample-client

Resolves: bug 8747

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

Merge OpenFlowJava repository into OpenFlowPlugin

Merge OpenFlowJava into OpenFlowPlugin repository to decrease the
logistics related to the OpenFlow southbound development.

Merged OpenFlowJava HEAD is
https://git.opendaylight.org/gerrit/gitweb?p=openflowjava.git;a=commit;h=76c83901c7a265e0d00c537d34f3f093c636129c

Old OpenFlowJava repo will serve just for archival purposes.

Detailed informations about this change and mailing lists thread:
https://lists.opendaylight.org/pipermail/openflowplugin-dev/2017-June/007276.html

Resolves: bug 8747

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

Merge "ONF Bundles sample application"

Add single-layer deserializer for PacketIn msg

Add single-layer deserializer for PacketIn message received from device
to OpenFlowPlugin to prevent deserialization errors when deserializing
PacketIn with single-layer-serialization enabled.

Resolves: bug 8772

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

Merge "BUG-8607: Fix issues in checkstyle enforcement for module bulk-o-matic"

Solves compilation error

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

Fix NPE in ConnectionContextImpl

- When we receive port status message, in some cases nodeId can be null,
  so when logging, get nodeId safely and in case it is null, just log
  "null"
- When we are closing outbound queue, log value of nodeId or null in
case nodeId is null

Resolves: bug 8762

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

Fix statistics polling on parameter

Fix value of statistics polling parameter to not be always
false and to be actually loaded from new configuration
service.

See also: bug 8705

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

BUG-8607: Fix issues in checkstyle enforcement for module bulk-o-matic

- Copyright section
- Change variables and methods names
- Remove redundant modifiers
- Order imports
- Line lengths
- Declare some static constants properly
- Enforcement in pom.xml

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

Encapsulate OpenFlowPlugin configuration

- Stop sending configuration to OpenFlow managers in long constructors
  and send just aggregated configuration in one class
- Add ability to register configuration listener to configuration
service
- Separate configuration service from OpenFlowPluginProviderImpl
- Fix FRM configuration breaking OpenFlowPlugin configuration by trying
  to load configuration from openflowplugin.cfg directly, what is only
  accesible to main OpenFlowPlugin project by creating configuration
  service and passing it to FRM
- Add missing 'disable-reconciliation' flag to FRM yang config, and use
  default values from YANG file for FRM configuration

Resolves: bug 8705

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

Merge "Report (TCP) port number for switches"