Merge "OPNFLWPLUG-929 : Remove deprecated guava library"

Merge "OPNFLWPLUG-929 : Remove deprecated guava library"

Merge "OFPGC_ADD_OR_MOD support in openflowplugin"

Eliminate explicit yangtools dependencies and pom cleanup

Changed the openflowplugin-parent pom to derive from bindng-parent so
yangtools version and dependencies come from mdsal. Also removed
unnecessary, duplicated configurations, eg yang-maven-plugin, in derived
poms thatcare inherited from the parents.

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

Bump yangtools to 2.0.1

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

Merge remote-tracking branch 'liblldp/master'

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

Move liblldp into libraries/liblldp

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

Bump to yangtools-2.0.0 and odlparent-3.0.2

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

Merge "Fix odlparent 3 Checkstyle issues"

Bump to odlparent 3.0.2

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

Bump yangtools to 2.0.0

This bumps odlparent to 3.0.1 and yangtools to 2.0.0. Also fixes
references to features, so that the build reactor is consistent.

Compilation fixes are addressed in a follow-up patch.

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

OPNFLWPLUG-929 : Remove deprecated guava library

use overloaded transformAsync api

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

OPNFLWPLUG-929 : Remove deprecated guava library

use overloaded transformAsync api

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

OFPGC_ADD_OR_MOD support in openflowplugin

Addition of new command OFPGC_ADD_OR_MOD for OFPT_GROUP_MOD message
that adds a new group that does not exist (like ADD) or modifies
an existing groups like (MODIFY).

Added a configurable property group_add_mod_enabled which when set to
"true", the addition/updation of a group would send OFPGC_ADD_OR_MOD
message to the switches. When set to false, the addition/updation of the
group has the same existing behavior,ie, OFPGC_ADD for addition and
OFPGC_UPDATE for the updation of the groups.

This patch fixes the following bugs: OPNFLWPLUG-788, OPNFLWPLUG-877, OPNFLWPLUG-667

Gerrit link for the specs: https://git.opendaylight.org/gerrit/#/c/65465/

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

Merge "OPNFLWPLUG-929 : Remove deprecated guava library in openflowplugin-impl"

Merge "OPNFLWPLUG-929 : Remove deprecated guava library in applications"

Merge "OPNFLWPLUG-929 : Remove deprecated guava library"

OPNFLWPLUG-929 : Remove deprecated guava library in openflowplugin-impl

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

OPNFLWPLUG-929 : Remove deprecated guava library

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

OPNFLWPLUG-929 : Remove deprecated guava library in applications

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

OPNFLWPLUG-953: Property change listener for lldp discovery config properties

Runtime update for properties in openflowplugin.cfg is failing
as there is no listener added for newly added toplogy lldp
discovery properties

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

Fix odlparent 3 Checkstyle issues

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

Merge "Rename addDeleteOperationTotTxChain => addDeleteOperationToTxChain"

Run time check and update of OFP service status

Change-Id: I8b41cd32990e7c4ea14cabbf862008fe77c7cc91
Signed-off-by: gobinath <gobinath@ericsson.com>
(cherry picked from commit 042a4f1a62e14cd78fab243b450f77bc6e0bc86e)
Signed-off-by: gobinath <gobinath@ericsson.com>

Merge "System ready integration for OFP"

System ready integration for OFP

The openflowplugin will start switch connections only on the system
ready event triggered by the infrautils.

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

Rename addDeleteOperationTotTxChain => addDeleteOperationToTxChain

to drop the 't' which looks like an obvious typo in this method

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

Merge "Minor: Remove 3 un-used private methods found by FindBugs"

Merge "Make HandshakeManagerImpl more thread-safe, as recommended by FindBugs"

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>

Minor: Remove 3 un-used private methods found by FindBugs

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

Make HandshakeManagerImpl more thread-safe, as recommended by FindBugs

Change-Id: I7b60abf3dfca6cad9936acdb1ce03f77f1a97768
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Change-Id: I7b60abf3dfca6cad9936acdb1ce03f77f1a97768
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>

Allow PacketException to store the cause

This avoids callers’ losing the original exception.

Change-Id: I99f11dd6866670229f8e3d2aa6556f540369de19
Signed-off-by: Stephen Kitt <skitt@redhat.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>

Bump odlparent 2.0.4 to 2.0.5

Also moves config/rpc-context.yang to include revision, making
the tests compatible with both 1.2.0 and 2.0.0 version of yangtools.

Change-Id: Ic7eb7d8fd27f5f3e5ae0f0aea9ab642a9b1b6fac
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

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>

Remove explicit default super-constructor calls

The default constructor is called by default (hence its name), no need
to call it explicitly.

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

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>