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

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

Use ThreadFactoryProvider from (now in) infrautils, rm genius.infra

as well as LoggingThreadUncaughtExceptionHandler (same)

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

Bug 8111: Conflicting modification for path

Removed the exception object from being logged. Will run CSIT and check.

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

Merge "Fix getInterfaceInfo API for logical tunnels"

Minor: IdManagerTest Java 8 lambda

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

Merge "Enhancing junits of interface-manager phase5"

Merge "LockManagerTest v1"

Fix getInterfaceInfo API for logical tunnels

getInterfaceInfo was unnecessarily setting a group-id for all
interfaces where actually it is not, and also for logical tunnels,
the groupid returned was not correct.

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

LockManagerTest v1

BTW: LockManager's retry logic appears to be completely broken!

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

Enhancing junits of interface-manager phase5

Adding tests for southbound port and switch updates

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

Merge "To prevent erronus comparison of enums, fixes persistence of mbean alarms on configuring teps."

To prevent erronus comparison of enums,
fixes persistence of mbean alarms on configuring teps.

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

Multiple VxLAN tunnels - initial support

- ITM - global configuration option to guard the feature
- ITM - creation of multiple tunnels between pair of DPNs
- Maintain interface-child-info model for tunnel hierarchy
- Logical tunnel group creation
- Prepare OF select group for egress service
- CLI tep:show-state to show the logical tunnel interface correctly

Change-Id: I3eca1a7995922feafa4135bdaff83b44a6f1ed0a
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

Merge "Removing references to neutron port from logs"

Bug 8070: Update the description of the command tep:deletedatastore

Change-Id: I696660c1f4a2684e301c34fbd38614e5e42eccb6
Signed-off-by: Rong Xu <xu.rong4@zte.com.cn>

Merge "ArpUtil test framework"

Merge "Minor Clean-up post Blueprint migration"

Removing references to neutron port from logs

As this is a common API which can get called from InterfaceConfigListener
add() / update() method, was seeing neutron port references even for other
usecases in the error logs.

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

Merge "Fix MatchIcmpv4 & MatchIcmpv6's problems with XtendBeanGenerator (2/2)"

Minor Clean-up post Blueprint migration

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

ArpUtil test framework

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

Enhancing interface-manager junits - phase3

Cleanup unused methods in interface-manager

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

ParentRef update for tunnels should be skipped

Tunnel interfaces today don't have a parentinterface. Updating it wrongly
tiggers unnecessary delete+add for tunnels. This adds unnecessary
overhead and should not be done till we have use cases that require these
and handle them accordingly.

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

Merge "Workaround for Bug 7451"

Merge "Enhancing junits of InterfaceManager - phase2"

Enhancing junits of InterfaceManager - phase2

Adding junits for interface updates, and vlan member interfaces

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

Fix MatchIcmpv4 & MatchIcmpv6's problems with XtendBeanGenerator (2/2)

This second change can now just remove those 'artificial' Builder
classes, because following an upgrade of the upstream XtendBeanGenerator
version used (in mdsal-binding-test-utils's dependencies), it now
"just works" directly, as one would naturally expect.

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

Workaround for Bug 7451

Leftovers in dispatcher table when unbind and ietf-state delete occurs simultaneously

The purpose of this patch is simply to lower the severity of this bug by making new
dispatcher flows override stale entries. This workaround cannot replace a real solution
since it would not survive OVS reboot scenarios

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

BUG 8098: Add APIs to expose Ovsdb caches

IFM maintains a few caches for TerminationPoints and OVsdbBridges which
can be useful for other modules. This adds support to expose those to for
others to use so they can avoid having to read Datastore.

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

Merge "Add util to get NXM register mask for elan-tag/vpn-id"

Merge "Cleanup of irrelevant comments from genius"

Merge "Preserve Ingress Lport Tag in metadata till Egress"

Cleanup of irrelevant comments from genius

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

Merge "Fix MatchIcmpv4 & MatchIcmpv6's problems with XtendBeanGenerator (1/2)"

Merge "NxMatchInfo redesign: clean up"

Merge "Adding genius-design-doc in rst format"

Fix MatchIcmpv4 & MatchIcmpv6's problems with XtendBeanGenerator (1/2)

This fixes this by adding missing getters.  A future change will
completely remove the respective Builder/s, as a new upstream
XtendBeanGenerator release will directly handle this case.

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

Merge "Bug 8023 CSIT failures with ELAN drop rule"

Merge "Bug 8048 potential fix for ID duplication issue"

Merge "Fix missing node IDs for interface in IFM provider"

Merge "Enhancing junits of interface-manager - phase1"

Preserve Ingress Lport Tag in metadata till Egress

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

Add util to get NXM register mask for elan-tag/vpn-id

Add reg3 and reg7 to NxMatchRegister keys

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

Fix missing node IDs for interface in IFM provider

Prevented some configuration interfaces to processed correctly as no
parent refs can be found for them.

Change-Id: I5b751cda479a0aacbe0086f310cf05e17b53da06
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Bug 7623: Genius' Karaf 4 features

see https://wiki.opendaylight.org/view/Karaf_4_migration

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

Enhancing junits of interface-manager - phase1

Adding junits for InterfaceManagerRpcService

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

NxMatchInfo redesign: clean up

This requires downstream users to have switched to the new classes.
The patch has been verified against netvirt using the corresponding
patch.

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

Adding genius-design-doc in rst format

Change-Id: Ia65dca3d3edb2fa129e9a237c890aeb4435a2a2c
Signed-off-by: Vivek Srivastava <vivek.v.srivastava@ericsson.com>

Fix issues related to checkstyle enforcement for module
interfacemanager-impl

- Dates
- Line lengths
- Formatting
- Exception catching
- Order imports
- Change variable names
- JavaDoc
- Move variables closer to their declaration
- Shorten type declarations
- Remove redundant modifiers
- Group overloaded methods
- Checkstyle enforcement in pom.xml

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

Bug 8023 CSIT failures with ELAN drop rule

Switched the ITM Tunnel List batching to individual transaction submit as
batching latencies causes ELAN to program drop rules. Will revert this
when ELAN starts to listen on ITM's tunnels_state DS

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

Merge "lport-tag not released for vlan member interface deletion"

Merge "ITM Tep Auto Config Unit test using Guice."

Merge "Add optional mask argument to NxMatchRegister"

Merge "Add getChildInterfaces to IInterfaceManager"

Merge "Cache optimizations for setting parent-refs"

Bug 8048 potential fix for ID duplication issue

* Added required info logs to debug issue
* Added appropriate synchronizers in lock manager so that id can't not
released more than once

Change-Id: I5822af7dc6ce040bc0a2983877e13e8868222f23
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Add optional mask argument to NxMatchRegister

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

Merge "mdsalutil-api datastoreutils thread pools etc. shutdown on bundle stop"

Merge "Adding dest IP/MAC in ArpResponseReceived"

ITM Tep Auto Config Unit test using Guice.

Added cases for:
- Change for def-tz-enabled and def-tz-tunnel-type config paramters
- Default-TZ creation and deletion based on def-tz-enabled flag
- TEP addition and deletion into/from default-TZ
- TZ creation and TEP addition into TZ
- TZ deletion and TEP deletion from TZ
- TEP addition and deletion into/from default-TZ via southbound
- TEP addition and deletion into/from northbound configured TZ via southbound
- TEP addition and deletion into/from NotHostedList via southbound
- Move TEP from one TZ to other TZ by changing tzname parameter in
  ExternalsIds list in Ovsdb node from southbound
- Update TEP for its bridge by changing dpn-br-name parameter in
  ExternalsIds list in Ovsdb node from southbound

Fixed errors due to new change of (1) dpn-br-name to br-name and
(2) 'weight' param inside vteps in itm.yang

Fixed checkstyle issues.

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

Cache optimizations for setting parent-refs

https://git.opendaylight.org/gerrit/49907 introduced a few DS reads which
are not very optimal. This adds couple caches to eliminate/optimize those
reads.

This also removes CacheInterfaceConfigListener and CacheInerfaceStateListener.
There are already clustered listeners for same path.
So we move cache updates to these listeners than using separate listeners.

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

Merge "Make InterfaceStateListener Singleton"

Merge "Ignore null tunnel id in getEgressActionInfosForInterface for VxLAN tunnels"

Ignore null tunnel id in getEgressActionInfosForInterface for VxLAN tunnels

When packets arrive from the ingress service pipeline to the logical
group they already contain VNI from L2/L3 services. When redispatching
the packet to the selected tunnel VNI gets overridden by this method

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

Add groupExists API to IMdsalApiManager

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

BUG 8080: IPv6 CSIT is failing

ITM checkstyle patch removed someof the Ipv6 code causing regression. This
patch fixes it and also adds some more IPv6 friendly code that was
initially missed.

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

Make InterfaceStateListener Singleton

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

Enable FindBugs for alivenessmonitor-impl-protocols

* Remove unused private methods.
* Use Optional instead of returning null.
* Add a type parameter to AlivenessProtocolHandler and its implementations.
* Add missing null-checks.
* Remove unnecessary variable initialisations.
* Specify the charset in AlivenessProtocolHandlerLLDP instead of relying on the system default.
* AlivenessMonitor.FutureCallbackImpl doesn't need any state from its parent, make it static.
* AlivenessProtocolHandler.handlePacketIn() can return null, mark it @Nullable.

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

DataStoreJobCoordinator @deprecated use infrautils instead

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

Add getChildInterfaces to IInterfaceManager

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

Added consts for counter service

Change-Id: Ifa12f3353c4fe4b8917bc36c99e37a3f823c0ce5
Signed-off-by: Guy Regev <guy.regev@hpe.com>

Merge "Changes for policy-based path selection"

Merge "AbstractTestableListener timeout increased from 3s to 30s"

Fixing InterfacestateListener after BP migration

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

AbstractTestableListener timeout increased from 3s to 30s

including new "in progress" LOG

dito for TestableJobCoordinatorEventsWaiter (recently added)

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

Fix bug in MatchInPort equals() method

Calling super() is wrong in this particular case, see MatchInPortTest
(which fails without this fix, but passes with it).

BTW: We need to just completelystop hand writing equals() and hashCode()
methods, anywhere... I'll put in a separate new change some time, to
replace all of this with immutables.org gen. code.

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

lport-tag not released for vlan member interface deletion

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

ITM checkstyle enforcement

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

Merge "interface-manager - Minor logging enhancement"

Merge "Revert "Spec for Counter Framework bundle""

interface-manager - Minor logging enhancement

Adding bfd-state in the update log information

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

Revert "Spec for Counter Framework bundle"

This reverts commit 5b27499fd1f7ca7635b50955d194b2d37104b1a7.

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

Fix issues related to checkstyle enforcement for module
interfacemanager-impl

- Dates
- Line lengths
- Formatting
- Exception catching
- Order imports
- Change variable names
- JavaDoc
- Move variables closer to their declaration
- Shorten type declarations
- Remove redundant modifiers

Note: more patches needed, checkstyle not enforced yet.

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

Revert "ITM Tep Auto Config Unit test using Guice."

This reverts commit 381310f948d04ca1c1b830b366c81dada39bcb55.

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

Merge "Fix overlapping bits in metadata between lport tag mask and elan subnet route mask."

Merge "Change externalId tep param dpn-br-name to br-name"

Merge "ITM Tep Auto Config Unit test using Guice."

Minor changes for Netvirt Egress Classifier tables

- renamed EGRESS_SFC_CLASSIFIER_ACL_TABLE to EGRESS_SFC_CLASSIFIER_NEXTHOP_TABLE
- Added EGRESS_SFC_CLASSIFIER_EGRESS_TABLE

Change-Id: Ia2b6ca3824e9196ad8954c078f7ce5f0baa71737
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

mdsalutil-api datastoreutils thread pools etc. shutdown on bundle stop

using an OSGi Activator, which isn't how I would do it in an ideal
world, but for the current design this is probably the best we can do;
the clean solution is as proposed in
https://git.opendaylight.org/gerrit/#/c/51431/14/jobcoordinator/jobcoordinator-impl/src/main/java/org/opendaylight/infrautils/jobcoordinator/internal/JobCoordinatorImpl.java

originally built in the hope that it would solve a problem with the
SingleFeatureTest (SFT) under Karaf 4, which it didn't - but this still
seems like a good idea, for possible future support of clean and fast
shut down.

probably will solve Bug 6564 in NetvirtIT (that wasn't the original
motivation, but as a "side effect" what's going on there should not
happen anymore)

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

Merge "TestIMdsalApiManager with naturally sorted flows"

Removing Deprecated Listener Base Classes

Refer the weather item for more details:
https://wiki.opendaylight.org/view/Weather#Genius_AbstractDataChangeListener_clean-up

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

Changes for policy-based path selection

* Constants definition
* Add classifier methods to MetadataUtil

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

Changes for New Netvirt SFC Classifier

- Adjusted service priorities
- Added SFC Classifier service, ingress and egress
- Added SFC Classifier table offsets
- More detail in Netvirt blueprint:
      https://git.opendaylight.org/gerrit/#/c/52063

Change-Id: I6d738979aaf4358bb429b538c8ee935e36fccd5a
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Fix issues related to checkstyle enforcement for module
interfacemanager-impl

- Dates
- Line lengths
- Formatting
- Exception catching
- Order imports
- Change variable names
- JavaDoc

Note: more patches needed, checkstyle not enforced yet.

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

Support For Egress Service Binding On tunnels

- Adding provision for binding egress services on tunnels
- GetEgressActionsForInterface will return new set of actions for tunnel
  interface
- Whenever a tunnel interface is created, a default service will be bound
  on the interface to take packets out on the tunnel port

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

Merge "runOnlyInLeaderNode() - out of order event processing"

Merge "Ensure unique ids are allocate across cluster"

Merge "Cleanup: avoid immediate Optional.get()"

Fix overlapping bits in metadata between lport tag mask and elan subnet route mask.

When creating a VrfEntry with SubnetRoute augmentation, VrfEntryListener installs a
flow with vpn-id and elan-tag, but this overrides part of the lport-tag bits.
It looks like vpn-id is allocated in the range of 70,000 to 100,000,
but METADATA_MASK_VRFID has 31 bits (00000000FFFFFFFE).

This change shortens the VRFID to 23 bits which is enough for the vpn-id,
and shifts the elan subnet route mask which solves the overlapping.

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

ITM Tep Auto Config Unit test using Guice.

Added cases for:
- Change for def-tz-enabled and def-tz-tunnel-type config paramters
- Default-TZ creation and deletion based on def-tz-enabled flag
- TEP addition and deletion into/from default-TZ
- TZ creation and TEP addition into TZ
- TZ deletion and TEP deletion from TZ
- TEP addition and deletion into/from default-TZ via southbound
- TEP addition and deletion into/from northbound configured TZ via southbound
- TEP addition and deletion into/from NotHostedList via southbound
- Move TEP from one TZ to other TZ by changing tzname parameter in
  ExternalsIds list in Ovsdb node from southbound
- Update TEP for its bridge by changing dpn-br-name parameter in
  ExternalsIds list in Ovsdb node from southbound

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