Fix broken hashCode & equals in FlowEntity related classes

This in turn then fixes TestIMdsalApiManager assertFlowsInAnyOrder bug.

Change includes new TestIMdsalApiManagerTest non-regression test.

Also includes missing toString() - not directly related, but useful.

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

Merge "Clean-up of unwanted ERROR logs"

Merge "Remove deprecated yang models and rpcs"

Merge "Adding API to support bucket addition/deletion"

InstructionInfo implementations made valid beans (XtendBeanGenerator)

This is required to unblock the (now) failing *AclServiceImplTest/s in
c/53002.  The IllegalStateException seen there were due to the
InstructionInfo implementations not having been valid beans; this caused
XtendBeanGenerator to be unable to get their fields values.

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

Merge "Spec for Counter Framework bundle"

Remove deprecated yang models and rpcs

interface-manager has some yang models which have been
deprecated from Beryllium time-frame, but still not removed
from the yangs.

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

Clean-up of unwanted ERROR logs

Some of the ERROR logs are coming in the standard call flow, and is
expected to fail, hence making them debugs

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

Fix exception & null handling in AbstractAlivenessProtocolHandler

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

Adding API to support bucket addition/deletion

Introduced new API in mdsal using which we can add or remove a backet from
the group.
With this application need not construct the entire group if it knows the
bucketId.
Have tested and made sure that the groups are getting updated properly
when this API is used.

Change-Id: I5b07da503bb33d39adf7bc4d7cdc53122ce36f03
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Merge "Modify ACL table numbers for remote acl indirection"

Remove unnecessary casts

Remove unnecessary casts in the call to newReleaseId.

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

Merge "Handling RACE conditions in bind/unbind service"

Improve exception message in NWUtil.isIpv4Address(String)

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

Handling RACE conditions in bind/unbind service

There are race conditions seen while binding/unbinding services,
due to the configlistener and statelistener being executed parallelly.
Avoiding this by adding a new service_binding_entity, and processing
service binding only on the entity owner.

Removing batching of interface-child-info as it is causing issues in
clustered tests

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

Merge "Spec to support load balancing and high availability of multiple VxLAN tunnels"

Modify ACL table numbers for remote acl indirection

Change table numbers of ingress ACL, add another table
to each ACL service and add a metadata match for the
ACL service, in preparation for remote_acl_indirection
changes.

This patch must be merged combined with
https://git.opendaylight.org/gerrit/#/c/52215/
due to changes in ACL tables naming.

Change-Id: I597f705c68f898a2efa03b85182ab28618ffb0f7
Signed-off-by: Alon Kochba <alonko@hpe.com>

Merge "Improve scale of AsyncXListenerBase"

Merge "Fix (some) unlogged Thread death uncaught exceptions, now logged"

Fix (some) unlogged Thread death uncaught exceptions, now logged

see also email list thread "Re: [netvirt-dev] [genius-dev] Exceptions
that don't even reach the log"

This doesn't actually fix any such exceptions, but it makes them get
properly logged, to the logging subsystem, instead of only appear on the
console.

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

Merge "Add missing <name> fields for pom.xml files"

AlivenessMonitor final clean-up, remove unneeded dep & service setters

and use constructor for initializing more fields (some already are),
instead of start() method, which prevents use of final fields.

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

Fix issues related to checkstyle compliance in module mdsalutil-impl

Fix issues related to checkstyle compliance in module
mdsalutil-impl:
- Copyright sections
- Line lengths
- Order imports
- Formatting
- Declaration of constants: name, modifiers
- Rename variables
- Exception catching
- Move declaration of variables closer to their use
- Remove unused variables
- Enforcement in pom.xml

There is more cleanup to be done, apart from checkstyle, I'll rise a
separate commit.

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

Merge "Bind/Unbind Service should work irrespective of Port Status"

Improve scale of AsyncXListenerBase

Convert single static thread pool for all listeners to thread pool per
listener.
Deprecate the base classes.

Change-Id: I0d86b2c017ed443db3093dc81f124677e1cf0449
Signed-off-by: Guy Sela <guy.sela@hpe.com>

Bind/Unbind Service should work irrespective of Port Status

- Removing port status check during bind-service
- Clean up of bind/unbind code path

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

Fix missing alivenessmonitor-impl-protocols in features.xml

Somehow forgot to uncomment this in c/52020
(a202894405787214fd9271902af55525b1324f09) - and nobody noticed?!  (Not
in code review, and no failing CSIT - our processes are broken...)

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

Merge "Add support for IPv6 VXLAN tunnels"

Merge "Enhancing interface-manager logging"

Enhancing interface-manager logging

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

Added set_field action for meta data.

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

Merge "Minor: Clean up fcapsmanager/countermanager POM"

Spec for Counter Framework bundle

Change-Id: I923908bc901c2aa931294b8748118948b187123f
Signed-off-by: Viji J <viji.j@ericsson.com>

Add missing <name> fields for pom.xml files

This is used by autorelease scripts to automatically parse which project
is failing a build and report to the mailing list automatically. We need
names in the format:

    ODL :: <groupId> :: <artifactId>

This patch formats in the same format as found in the startup archetypes
patch found here: https://git.opendaylight.org/gerrit/52522

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

Merge "TestIMdsalApiManager assertFlowsInAnyOrder LOG.warn"

Interfacemanager blueprint migration

Change-Id: I4b3f7ea54ef290546a4e7e031a0f3b5b3023c7aa
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Migrate mdsalutil-impl module to blueprint

Second approach (first one was [1]) to migrate the module
mdsalutil-impl to blueprint after the patch [2] that breaks some cyclic
dependencies.

[1] https://git.opendaylight.org/gerrit/#/c/48317/
[2] https://git.opendaylight.org/gerrit/#/c/52020/

I couldn't manage to get it pass the SFT locally, but I want you to
review it so that I can get a clue on how to progress it. I tried to
make as few as possible changes (it is not checkstyle compliant, etc.)
to keep it focused.

Change-Id: Ib83ce98bde9d113f6be02ae72e0ea55d17740f3c
Signed-off-by: David <david.suarez.fuentes@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Adding job retries for DJC bind/unbind service jobs"

Spec to support load balancing and high availability of multiple VxLAN tunnels

Change-Id: Id608d4007a7be8419380314f27c40f8f8ff06ecb
Signed-off-by: Tali <tali.ben-meir@hpe.com>

TestIMdsalApiManager assertFlowsInAnyOrder LOG.warn

based on the discussion in
https://git.opendaylight.org/gerrit/#/c/50467/

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

Adding job retries for DJC bind/unbind service jobs

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

Minor: Clean up fcapsmanager/countermanager POM

We should not be using this old outdated version of maven-bundle-plugin
here, but the one that odlparent provides.

Ran into this while testing Bug 7182 (but it's not directly related to
that), where I hit this problem in the Eclipse IDE:

Manifest org.opendaylight.genius:countermanager:bundle:0.2.0-SNAPSHOT :
Input file does not exist: target/classes/LICENSE

org.apache.maven.plugin.MojoExecutionException: Error(s) found in
manifest configuration

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

Bug 7182 related: Remove M2E lifecycle mapping

These should never be in individual projects anymore now, we handle this
centrally, either in odlparent, or
https://github.com/vorburger/opendaylight-eclipse-setup, or by
appropriate lifecycle-mapping-metadata.xml in a Maven plugin (that's
what Bug 7182 does for the yang-maven-plugin; this removes what makes
that not work for these projects).

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

Make 'bindService' transaction-embeddable

Change-Id: I3608e43991a9b00a021f3f49f9c4413567d17acc
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Remove try { } finally { LOG.error } introduced in previous commit

In 29ede9a79082ff38b5de705d9b1e8e779edf4f23, David turned a try {} catch
(Exception e) { LOG.error(e) } into a finally { LOG.error }, which to me
seems wrong - as-is now, it would ALWAYS LOG.error, wouldn't it?  So
best to just remove this finally.

This looks like a bigger change than it is because indention changed.
(I've not touched anything in that big block, just un-indent'ed it.)

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

Add API methods to check if service is bound on iface

Change-Id: I21ed6d2e5871be3c91a5f874f384f21045698196
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Add support for IPv6 VXLAN tunnels

Description: IPv6 tunnels are supported since OVS 2.6

Change-Id: I10dbecc145c9ec8ff76115a34c9824827fd22c28
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Use def-tz-enabled and of-tunnel for TEP addition

1) Using def-tz-enabled config flag while adding TEP under default-transport-zone.
2) Added support to configure of-tunnel as TEP parameter from southbound and
use of-tunnel flag while adding TEP under transport-zone or tepsNotHostedInTransportZone.

Change-Id: I894bef7500bd72836f2020ac320e40ea7329c470
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Merge "Remove an unnecessary augmentation"

Fix some problems introduced after checkstyle enforcement

ResourceBatchingManager violates some checkstyle rules. It should fix
the compilation problems.

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

Merge "Bug 7599 hwvtep ucast mac consumption performance"

Merge "BUG-7758: Fix creation of FLAT and VLAN networks with the same parent"

Remove @Deprecated from interface-manager APIs

Initially GENIUS was planning to move away from API models
to promote the usage of RPCs. However, given the various benefits of
using APIs, removing the Deprecated tags.

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

Intro. alivenessmonitor-impl-protocols to break cyclic dependency

This change makes alivenessmonitor-impl not be dependent on
interfacemanager-api anymore, by refactoring the protocol handlers
(which are somewhat orthogonal) into a separate bundle, thus breaking a
cyclic dependency which caused issues in the BP migration (c/49175).

The real problem actually wasn't the compile time dependency of
alivenessmonitor-impl to interfacemanager-api, but the (implicit)
blueprint DI dependency to interfacemanager-impl (which itself requires
alivenessmonitor); but this solves this, because now
(only) interfacemanager-impl can depend on alivenessmonitor, and the 3rd
new
bundle alivenessmonitor-impl-protocols depends on both.

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

Design spec for Service Binding On Tunnels

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

Interfacemanager-api : Checkstyle Enforcement

Change-Id: I89250d986237ee75bb87ce99c4066c9f5e87d602
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>

Merge "Genius support for Trunks"

Merge "Nat Action added in ActionNxConntrack"

minor AlivenessMonitorUtil clean-up incl. removal of 2 unused methods

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

Merge "MdSalUtilTest no longer extends @deprecated AbstractDataBrokerTest"

Merge "Fix issues related to checkstyle compliance"

Merge "alivenessmonitor-impl reduce dependency to interfacemanager-api"

Merge "Change ITMManagerRpcservice Test frm Mock to Guice"

Merge "Fix issues related to checkstyle compliance"

Nat Action added in ActionNxConntrack

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

alivenessmonitor-impl AlivenessProtocolHandlerLLDP dead code removed

I obviously have no idea what this code really does, but something looks
very wrong in this handlePacketIn() method .. all of that code does
NOTHING, literally (dead code).  --  Can this entire class be removed?

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

Change ITMManagerRpcservice Test frm Mock to Guice

ITM RPC service Test cases are re-written in Guice framework.

Change-Id: I8a2f8a5bee35c9e5edc74292eeb1f1d75cc5056c
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
fcapsapplication-impl:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Exception catching
- Change name of variables
- Checkstyle enforcement activated

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

Merge "ITM Auto-config: Change other_config to ExternalId"

alivenessmonitor-impl reduce dependency to interfacemanager-api

All code removed here is completely dead code, this should be low risk.

BTW: Now only AlivenessProtocolHandlerARP still needs interfacemanager
API.. if that could be removed as well, then we could resolve the cyclic
dependency that is causing us pain... could it refactored?

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

interfacemanager-shell : Checkstyle Enforcement

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

Fix issues related to checkstyle compliance

Fix issues related to checkstyle compliance in module
fcapsappjmx:
- Copyright sections
- Line lengths
- Order imports
- Declaration of constants: name, modifiers
- Formatting
- Checkstyle enforcement activated

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

Merge "Revert I466b2489107172ad7f391ba6b50c4dce9f08343d"

Remove an unnecessary augmentation

The Ovsdb node was being augmented twice - in ItmUtils, and in
OvsdbNodeListener.

With this small change, it is only augmented once in OvsdbNodeListener,
then passed as a parameter into the ItmUtils.

Change-Id: I933fc794c25573371262df746309f215d9706ba5
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

ITM Auto-config: Change other_config to ExternalId

Change TEP fields configuration from other_config to external_ids.
Parse TEP params from external_ids list of ovsdb node, and then use it
for add/update/delete TEP in ITM Config DS.

Change-Id: I29f2a7aabb12ac97b896956f931e6d8fba89df83
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Tunnels in DOWN state in scaled scenario

In a scaled environment, some of the tunnels remain in DOWN
state after ITM full mesh creation. This is due to a race condition
when openflow port add and bfd port status messages are processed
out of order. Making the code more robust against such races.

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

Merge "Prepare Karaf 4 migration: move old features"

Revert I466b2489107172ad7f391ba6b50c4dce9f08343d

Reverting the temporary work around as part of
Bug 7538 InterfaceManagerTestModule temporary local work-around

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

Merge "Remove specific version for maven-bundle-plugin"

Prepare Karaf 4 migration: move old features

This sets the stage for the Karaf 4 migration (which needs dluxapps,
ovsdb and openflowplugin first, on top of patches which are already in
progress for yangtools, mdsal, controller, infrautils and netconf).

Bug: 7623
Change-Id: Icf20f546c6a65870a07b546c3fb945c1fca303b4
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "Checkstyle enforcement - idmanager-shell"

Remove specific version for maven-bundle-plugin

Remove specific version for maven-bundle-plugin to avoid problems like:
https://bugs.opendaylight.org/show_bug.cgi?id=7537

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

Genius support for Trunks

Trunk Port API adds certain use cases which are currently not supported in
InterfaceManager. These include but not restricted to:

1. Interface membership will not be known at creation but come as an
upate later. There is no means to know at creation if it will become a
trunk member later or not.
2. Trunk member interfaces will not have any termination point, so
parentRefs will never be updated from southbound event. Trunk members will
add parentRefs through an interface update.
3. Trunk member interface can go from member to trunk if it is removed
from a trunk as subport.
4. Trunk member interface's parentRefs can be removed if it is removd from
a trunk as subport.

This means IFM now needs to support use cases where if2Vlan and parentRef
augmentations of an interface can change dynamically.

Related gerrits:
https://git.opendaylight.org/gerrit/#/q/topic:vlan-aware-vms

Change-Id: Ie240c1307547cce17463e2ff16a223aa72807d4c
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

mdsalutil-api clean up Checkstyle now enforced

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

MdSalUtilTest no longer extends @deprecated AbstractDataBrokerTest

use new AbstractConcurrentDataBrokerTest instead - and fix MDSALManager

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

Merge "Fix NPE in ITM endpoint list usage"

Fix NPE in ITM endpoint list usage

This NPE is popping up in CSIT , while deleting all
transport zones and interfaces via REST API.

Change-Id: Iea4c162519a0dbfe9bb6f926e6c9895fffdab02e
Signed-off-by: Alon Kochba <alonko@hpe.com>

Bug 6952: Support for deleting TEPs in multiple TZ

Resolved merge conflict
Removed commented lines

Change-Id: Ib25a95242cbc96af0e0087f524630dd42ea4f669
Signed-off-by: Hema Gopalkrishnan <hema.gopalkrishnan@ericsson.com>

Merge "fix for id duplication for different id keys"

Merge "Blueprint XML Config support in ITM."

Checkstyle enforcement - idmanager-shell

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

getInterfaceInfoFromOperationalDS Optimisation

vxlan:show CLI was getting stuck in one of the cluster nodes
in a scaled scenario of full mesh creation due to a datastore
reads in the call flow

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

fix for id duplication for different id keys

* handling id pool listener appropriately otherwise it overwrites local pool
which lead to allocation of same id to different key

Change-Id: Ibc075846a7daa07e681ce2b34dfaabf084e7d287
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "Optimizing southbound Tunnel Events"

Merge "Enforce checkstyle in arputil-impl module"

Add missing JavaDoc to ResourceBatchingManager

copy/pasted, with minor adjustments for clarity, from a review comment
by Vivekanandan Narasimhan in
https://git.opendaylight.org/gerrit/#/c/49491/

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

To fix grep not working for tep:show &
tep:show-state on karaf console

Change-Id: I59469c0c1da10d55c2bd5d1a9f87e3021c8a46cf
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>

Optimizing southbound Tunnel Events

When a tunnel is created on OVS, there are a series of events triggered
from plugins(OVSDB and OpenFlow) based on which interface-manager does a
sequence of operations. The code is optimised to batch all these
transactions so that there will be faster programming of Table 0 flows,
and interface-state operational DS population in a scaled scenario.

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

Bug 7538 InterfaceManagerTestModule temporary local work-around

until https://git.opendaylight.org/gerrit/#/c/51486/ finally goes in
upstream in controller ... then this will have to be reverted again.

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

BUG-7758: Fix creation of FLAT and VLAN networks with the same parent

Change-Id: Ia0e08c3ef22de39769c6f81e978060501107a0c2
Signed-off-by: Alon Kochba <alonko@hpe.com>