Merge changes I400fdd74,I18d70f22,Icc83aef3,I3a67dc87,I05b3e988

* changes:
  Make two classes _static_ inner classes, as recommended by FindBugs
  Suppress FindBugs null analysis warning in FlowDescriptorDtoTest
  Suppress FindBugs concurrency warning which is wrong
  Fix FindBugs null analysis problems due to wrong @Nullable
  Fix FindBugs level "Scariest" problem in SalAsyncConfigServiceImplTest

Merge "FindBugs enforcement in module openflowplugin-api/"

Merge changes Ic5f4324c,I7611359f

* changes:
  FindBugs enforcement in module common/
  Checkstyle enforcement (and required fixes) in module common/

Fix possible transaction leak in StatisticsGatheringUtils

see https://jira.opendaylight.org/projects/OPNFLWPLUG/issues/OPNFLWPLUG-961;
5x transactions opened here which were not closed

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

Make two classes _static_ inner classes, as recommended by FindBugs

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

Suppress FindBugs null analysis warning in FlowDescriptorDtoTest

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

Suppress FindBugs concurrency warning which is wrong

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

Fix FindBugs null analysis problems due to wrong @Nullable

In each of these 3 cases, the RpcResult is very clearly de facto not
@Nullable (because it is de-referenced in the method, so if it was null,
that would cause a NPE) - so just remove the @Nullable.

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

Fix FindBugs level "Scariest" problem in SalAsyncConfigServiceImplTest

FB says: "VERY confusing to have methods
org.opendaylight.openflowplugin.impl.services.sal.SalAsyncConfigServiceImplTest.setUp()
and org.opendaylight.openflowplugin.impl.services.ServiceMocking.setup()
The referenced methods have names that differ only by capitalization.
This is very confusing because if the capitalization were identical then
one of the methods would override the other."

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

FindBugs enforcement in module openflowplugin-api/

No serious FindBugs violation needed to be corrected (1 very minor)

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

FindBugs enforcement in module common/

No FindBugs violation needed to be corrected in TransactionChainManager

Includes .gitignore of .fbExcludeFilterFile created by FB Eclipse plugin

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

Checkstyle enforcement (and required fixes) in module common/

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

Merge "Clean up in module reconciliation-framework"

Spec: Reconciliation framework

Change-Id: I80c8a6ce4b073f59d69ecd01558ce024adce0748
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Clean up in module reconciliation-framework

- Remove some deprecated
- Make some variables final
- Other minor clean up

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

Merge "Revert "Bug 8902 : Spec for Reconciliation Framework""

Remove the rst file and add later with proper formatting

Change-Id: Ib92caeb6c7e657b51e76dd169c5569e2bb45bbe7
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Revert "Bug 8902 : Spec for Reconciliation Framework"

Patch #60283 is preventing patch #65556 from being verified

This reverts commit f1e058a660cf98d1411ba763c6b0989e2439fabc.

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

Merge "Add Docs for Openflowplugin"

Merge "Bug 8902 : Spec for Reconciliation Framework"

Merge "DiagStatus integration for openflowplugin"

Add Docs for Openflowplugin

This patch just initializes sphinx for Openflowplugin

Change-Id: Ie7d0e2004d48e7ddacaf32ab18d0f94f5bc40c27
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Bug 8902 : Spec for Reconciliation Framework

This spec provides the documentation for Reconciliation framework
implementation and usage

Change-Id: I10c312d9890effb317a695450d381ee4d6725b60
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Drop Karaf shell specifiers

We no longer need to support Karaf 3, and the karaf.shell
"[3.0.0,4.1)" declarations prevent us from upgrading to Karaf 4.1
(which is coming with odlparent 3).

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

DiagStatus integration for openflowplugin

Openflowplugin should report their initial status once
all switch connection providers are initialized.
Subsequent patches will query the dynamic status of port
6653/6633 to advertise the live status.
Further enhancements can be done if openflowplugin can query
some of the major service status before opening up the
southbound interfaces.

Change-Id: Id57f52d3ad02d59f7469762b1e1f0fb468b785ce
Signed-off-by: Faseela K <faseela.k@ericsson.com>

OPNFLWPLUG-930 Inconsistent flow IDs between flows
 in config and operational data stores

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

Merge "Fix issues related to checkstyle enforcement on openflow-impl module"

Merge "Remove deprecated"

Do not use fix custom version in ofp feature

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

Remove deprecated

- CheckedFuture
- Futures.addCallback
- Some other changes regarding checkstyle, sonar and annotations

Change-Id: I6ab7b10a6170bee6e78895b21d40ee1a047d961a
Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>

Fix issues related to checkstyle enforcement on openflow-impl module

This is the last commit. It enforces the checkstyle level to error.

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

Fix checkstyle issues in openflowplugin-impl module

Change-Id: I8d6aa0b35078350bf897c5bf2b9a6cc11d0a4bf2
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Merge "Cli to display all the connected DPNs"

Cli to display all the connected DPNs

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

Merge "Fix log message"

Fix log message

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

Disable the test that's causing merge failure

It's bit of a weird situation. Locally this unit test is passing consistently
but it's failing in merge job. Looks like merge job for the following patch
https://git.opendaylight.org/gerrit/#/c/62822/

failed and it didn't publish the artifacts and looks like that's causing
the merge job failure. This patch is an attempt to get a green merge job
and once all the latest artifacts are publish, this test will be re-enabled.

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

Merge "BUG-9218: use packaged inventory model"

Master: cherry-pick changes for BUG-9223

    Cherry-pick changes for BUG-9223 from stable/boron branch for remove hardcoded value of lldp interval
    Removed debug log from LLDPLinkAger.java

Change-Id: I95361679b6f12824f45e0ae31972d81485e23fc7
Signed-off-by: Jalpa Modasiya <jmodasiy@luminanetworks.com>

BUG-9218: use packaged inventory model

Rather than packing it here, use controller's dedicated feature
to pull it in.

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

This patch implements ct-mark support in nicira extensions.

Specifically:
- support for ct-mark MATCH, including mask.
- support for ct-mark ACTION (set-field), but not including mask

This patch supports implementing ACL spec:
https://git.opendaylight.org/gerrit/#/c/60580

Change-Id: Ib828da39b4470e7172fe7028e8d3123fb184c742
Signed-off-by: Bertrand <bertrand.low@hcl.com>

Fix issues related to checkstyle enforcement

This is the first of a series of two or three commits, since the module
is quite big. The latest commit will set the severity to "error".

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

Bump odlparent 2.0.4 to 2.0.5

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

Merge changes

Merge "Bump openflowplugin to use yangtools 1.2.0"

Bump openflowplugin to use yangtools 1.2.0

Jira: releng-485

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

Merge "Remove Itemlifecycle"

Remove Itemlifecycle

ItemLifecycle was created for one time use case. Devices without
statistics possibility at all
and this workaround was made to store flows, groups, meters to the DS
as a reaction on services (flow, group, meters) when statistics
gathering could not be used.

Resolves: Bug 9010

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

Sort bucket actions

Sort bucket actions by order when serializing them in
GroupMessageSerializer.

Resolves: bug 9216

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

Merge "Fix issues related to checkstyle enforcement"

Fix issues related to checkstyle enforcement

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

Merge "Gather statistics in separate thread"

Merge "Async config message"

Gather statistics in separate thread

Gather statistics in separate thread to free netty thread from
processing statistics when they arrive.

See also: bug 8787

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

Async config message

- Add support for async config message
  (Supported from version 1.3)

Resolves: bug 4082

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

Checkstyle enforcement.

This commit [1] left enforcement on warning, this one rise to error.

[1] https://git.opendaylight.org/gerrit/#/c/63234/

Change-Id: I80c896444e065b9f2aea5640775195deceb3ca24
Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>

Merge "Fix checkstyle warnings for impl/connection package and OpenFlowPluginProviderImpl"

Merge "Drop the odlparent.netty property"

Merge "Fix issues related to checkstyle enforcement"

Merge "Fix transaction manager closing."

Merge "Fix issues related to checkstyle enforcement"

Fix transaction manager closing.

- the event onTransactionFailed we properly closing chain and creating
  a new chain
- changed writeOnlyTransaction to readWriteTransaction
- moved transaction manager to the common module
- topology manager using transaction manager instead of creating a
  transaction chain for itself
- added control of unfinished transaction on close

See also: Bug-9038

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

Fix issues related to checkstyle enforcement

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

Merge "Terminate SLAVE task before sending role change"

Merge "Fix NPE, version 1.0 returns null on auxiliary id."

Fix issues related to checkstyle enforcement

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

Fix issues related to checkstyle enforcement

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

Fix NPE, version 1.0 returns null on auxiliary id.

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

Terminate SLAVE task before sending role change

- Terminate SLAVE task in role context when role change is requested and
not after role change is finished.
- Do not send notification about role failed to set to mastership
service in case future was cancelled

See also: bug 9089

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

Fix issues related to checkstyle enforcement

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

Fix checkstyle warnings for impl/connection package and OpenFlowPluginProviderImpl

See also: bug 8607

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

Move used code from openflowplugin to impl

Move used code that was used only in openflowplugin-impl from
openflowplugin (old Helium design)

See also: bug 8653

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

Merge "Fix codestyle"

Merge "Fix codestyle"

Merge "Fix codestyle"

Fix codestyle

- Removed deprecated getLOGValue from deviceInfo

See also:Bug-8607

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

Fix codestyle

- Anonymous type replaced by lambda

See also:Bug-8607

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

Fix codestyle

- Removed unused import

See also:Bug-8607

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

Merge "Remove dependency of impl on onf-extensions"

Merge "Add missing bundle converters"

Merge "Fix codestyle"

Merge "Fix codestyle"

Merge "Cleanup"

Merge "Move match converter to new converter system"

Fix codestyle

- Optional.isPresent replacesd to functional expression

See also:Bug-8607

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

Fix codestyle

- Arrays.asList() memory save

See also:Bug-8607

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

Cleanup

- Remove dead code
- Join if statements
- Private methods called from inner classes should be moved to those
classes

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

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>

Move match converter to new converter system

In preparation of removal of Helium design, remove MatchReactor and move
MatchConvertor to new converter manager based system.

See also: bug 8653

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

Remove dependency of impl on onf-extensions

As preparation for removal of helium design:

- Remove dependency of openflowplugin-impl on onf-extensions and move
SalBundleService to openflowplugin-impl, as it is registered and used
only there
- Move onf extension yang models to extension-api

See also: bug 8653

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

Add missing bundle converters

Add missing bundle converters about what was device context throwing
warnings.

Change-Id: Ic3d89c386c32f06d80afe1221f2d30fd919d087d
Signed-off-by: Tomas Slusny <tomas.slusny@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"