Adjust Checkstyle DTD URIs

The DTDs have moved from checkstyle.sourceforge.net to checkstyle.org,
using secure URLs.

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

Correct yangtools version

This bump seems to have been forgotten: neon should work on
version 2.1.8.

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

Port arputil to mdsal 3

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

Switch to JDT annotations for Nullable and NonNull

This reduces our dependency on JSR-305 which causes a number of issues
(e.g. with javax.annotation resolution in OSGi bundles).

Annotations on local variables are removed because they trigger a
Javac bug which crashes ASM and then FindBugs, SpotBugs and JaCoCo.
SpotBugs mis-detects null returns in packages marked as “non-null by
default”, so its NP_NONNULL_RETURN_VIOLATION check is disabled on
affected methods.

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

Remove upgrade-in-progress flag

JIRA: GENIUS-190

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

NETVIRT-1262: Tep's are not part of transport zone when a transport zone
is removed and added from NB

Problem
-------
TEPs do not add back to transport-zone which is removed and then added
again from northbound.

Problem solution
----------------
When tz delete event arrives for a particular Transport zone,
teps of that will be moved to tepsInNotHostedTransportZone OperDs.
And, when same name tz gets re-added from NBI, then these teps
will go back to re-added tz.

Change-Id: I482528a1e695ee439ca731c63138639dff2dce3f
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

GENIUS-249: Add CLI to print bridge from given DPN-ID

Change-Id: I50bd87711487d3071286b3d39878c5095a206952
Signed-off-by: souvikdas95 <souvikdas95@yahoo.co.in>

Genius-130: Test plan for OF-Tunnels

OF Tunnels support for ITM direct tunnels

Change-Id: I28698713338773eb245f06a0c9a18b6554c1ec6a
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>

Fix build with Java 11

This adds required dependecies on javax.annotation-api, fixing the
build with JDK11.

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

Use scope=provided for karaf shell

The is filtered from features, but its dependencies are not,
fix that by having shell at scope=provided.

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

Drop Karaf console dependency from genius-api

We shouldn’t be passing the command session around in the API layer;
use exceptions for errors throughout, and rework showTeps to return
the information to be displayed instead of displaying it directly.

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

GENIUS-248: ITM-Vtem Auto tunnel feature- Tunnels not coming up after deleting external id's in OVS

Issue:
When exterenal_id is deleted from OVS, then TEPs are supposed to move in def-TZ.
When TEP's movement to default-transport-zone from TZA happens, tunnels do not come up.
Main issue is during movement, TEP is added followed by TEP deletion
which makes TEP not available in tep-show-state. This happens because
TEP is keyed on dpn-id in DpnTepsInfo and tz-membership do not help in synchronization
issues.

Fix:
Introduced DataTreeEventCallbackRegistrar on DpnTepsInfo to check if TEP is deleted and
when it is removed and event is notified, then only perform TEP add in TZ movement case.

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

Bump versions by x.y.(z+1)

Change-Id: I5363fef9ef2795a360283a066117bddef3caf937
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>

GENIUS: Migrate fcaps from XML to annotation based BP

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

Use odl-apache-sshd

... instead of pulling in the sshd bundle.

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

GENIUS-185: Removing usage of databroker tx

- replaces most uses of databroker tx
- few worker classes have been refactored into their respective listeneres
- some classes ommitted in /itm-impl/..../cli/
    would lead to other comprehensive changes which are out of scope

Change-Id: I8d2218636471923b7fb9e61dd2b42b6d80207bcf
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>

Creating OF Tunnel parent child in memory mapping.

Change-Id: Id14d216b7bd744bd1d6ce28bd7baede9777a24ae
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Add a wrapping DataBroker test wiring ...

which wraps an MD-SAL DataBroker with a Controller DataBroker for test
purposes. (This will be removed once the migration is complete in
Genius and NetVirt.)

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

GENIUS-268: Disabling BFD by default.

Change-Id: I172779668282fce77f58946b8a303483b4d8ca11
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Add mdsal 3 DataBroker test modules

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

genius/interfacemanager annotation based bp

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

Adding OF Tunnel detail to config DS.

Change-Id: I4ad233308e64d8e2dcaba6eee7d8a4fc1398d961
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Port fcapsapplication to mdsal 3

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

GENIUS-130: Adding OFTunnel enablement configuration
flag in ITM and removing from IFM.

Change-Id: I5d2c7c5e8a80251d2fc824963075d5a0623abfe4
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

TSC-181
Marking the fields set to be removed as depercated.
Taking care of Hema's comments.
Gerrit link for the corresponding spec change:-
https://git.opendaylight.org/gerrit/77605

Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>
Change-Id: I611cfbe52a38c84f4dd3221c1df9e48e1c06d857

Bump mdsal to 3.0.6

This aligns with odlparent-4.0.9 and yangtools-2.1.8.

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

Bump to odlparent 4.0.9

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

Huge INFO log in TerminationPointStateListener

This log is coming in csit runs for each bfd status
update on tunnel creation.

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

RD allocation using IDManager

JIRA: GENIUS-262

RD is currently allocated by opensatck in netvirt.
For COE, RD has to be auto generated.
These changes is to introduce the logic in GENIUS using IDmanager

Change-Id: I8afadbc330b41f8dc7b71252de396f8500bfd5d1
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

Bump mdsal to 3.0.4

This realigns odlparent/yangtools/mdsal and brings in the latest
improvements.

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

Bump to odlparent 4.0.8

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

Port alivenessmonitor to mdsal 3

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

GENIUS-130: Adding OFTunnel enablement configuration.

Change-Id: I0458f4d1f1af760570bdb22e96af8fd94ab2e39a
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

GENIUS:arputil from XML to annotation based BP

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

GENIUS: idmanager from xml to annotation based bp

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

Bump odlparent to 4.0.7

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

Fix odl-genius-fcaps-framework

This feature is using stuff from infrautils, make sure we depend
on the correct upstream feature.

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

Use odl-apache-commons-net

odlparent provides a wrapper feature, use it to package commons-net.

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

Revert "skip tests using xtend until odlparent bump"

This reverts commit c30f9b64ecde99cb4078bd8fd3a7eb1ad752d3a0. With
odlparent-4.0.5, the tests should work again as expected.

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

GENIUS:alivenessmonitor from XML to annotation based blueprint

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

Bump mdsal to 3.0.3

This brings in latest fixes and alignes with odlparent-4.0.5.

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

Bump to odlparent 4.0.5

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

skip tests using xtend until odlparent bump

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

Drop nullToEmpty

Use the nonnull getter variants where appropriate.

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

Move IdManager based VNI Carving to a centralized module

JIRA: GENIUS-250

Netvirt currently has a vni carving logic which is
tightly coupled with NAT. There are more netvirt
services which are in need of this utility now, and
hence we need to move this to a centralized place.
Adding a new VniUtils in genius, to which later
netvirt services can be migrated to.

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

GENIUS-246:Removing updation of cache.

When tunnels are not found in cache today, ITM does a dip into DS
and updates the cache accordingly. This cause stale entries
getting added into the cache.
This issue was observed when a tep delete command is executed:
1. ITM initiates delete process.
2. ITM removes the tunnel from the InternalTunnel cache.
3. Before the DS is cleaned up, we try to fetch tunnels from cache
   to process clearance of Alarm if any for that tunnel.
   As the cache does not have that specific tunnel; ITM
   repopulates the cache by doing a dip into the DS.
4. Thereby a stale tunnel gets added to the cache.

The fix is to dip into DS if the tunnel is not present in the
cache but not update the cache.

Change-Id: I1fa7dded96f7a15cba0e63cf9318af5c75e2b41d
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Stale interface cache entry

interface-config and interface-state DS entries
are front ended by a ConcurrentHashMap for easy
subsequent access. These maps are populated
by a clustered DTCN, but on top of that there
is an additional logic for populating the maps
when an external entity tries to access the same
from DS. This logic is causing stale data to be
written to the cache in some cases.

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

Ensure that maps accesses are protected

All methods which directly access the flow, group and bucket maps
should be synchronized.

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

Remove genius/resourcemanager

resourcemanager is not used by any applications for
the last few releases. Let us remove it from the
main genius feature, and add it back only when
someone really needs it.

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

Use cached invariant objects

This caches operation result futures and make use of them, so we
do not need to allocate them.

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

reduce 'Total taken ##time' log level in ResourceBatchingManager from INFO to DEBUG

JIRA: GENIUS-255
Change-Id: I5d2c5d9e257474a4d5d5abb415ea5792812f2c5e
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Adding few log statements for Cache updation.

Change-Id: Ie7eab2422bf2b797bff83d0addd2634a4f95b88f
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Fix an xtend warning

This is != vs !== confusion, xtend is not java.

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

Bump to mdsal-3.0.2

This bumps mdsal to 3.0.2.

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

Add missing optionOfTunnel

As done in I5c34b8d147dff82e8e56bc82fa472919e425f368, this adds a
missing optionOfTunnel, which is necessary for the mdsal 3.0.2 bump.

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

Reduce liblldp usage

We can use Byte.SIZE instead of NetUtils.NUM_BITS_IN_A_BYTE.

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

Fix mtu type in router-advertisement-packet

This patch also defines few constants used while
framing an RA.

Change-Id: Ic67a92522438c6d60dfa90198196bb0e7af13f5f
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Do not use createMissingParents

Locks does not have any elements which would be created, so do not
force this operation to be needlessly slow.

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

Allocate InstanceIdentifier only when needed

LockData, which we are passing around anyway, has enough information
to build an InstanceIdentifier. Lower the number of arguments being
passed around by moving InstanceIdentifier creation to the point
where it is actually needed.

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

GENIUS-212: TEP IP Update does not work in ITM auto-tunnel

Issue: During local_ip update, old-tep is being set.

Fix: New tep ip is getting passed during tep addition after old tep is
removed.

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

Double registration of listener

InterfaceConfigListener is registering
its listener twice, resulting in all events
being triggered twice.

JIRA: CONTROLLER-1874

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

Fix for Id allocation failure

This issue is caused due to gerrit
https://git.opendaylight.org/gerrit/#/c/75824/, where
"ReadFailedException" exception is not handled, which is preventing
further code execution. Hence resulting in id allocation failure.

Fix is to handle the exception, so that code excecution doesn't stop
if read fails.

Change-Id: I9b16f40310205550c866785ff1197a9ad1eaaa89
Signed-off-by: Ankit Jain <ankit.j.jain@ericsson.com>

fix wrong JavaDoc of SingleTransactionDataBroker syncRead()

prompted by review of I9b16f40310205550c866785ff1197a9ad1eaaa89 where
this was not read and done wrong, but if the doc is misleading, that
doesn't help... ;-)

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

GENIUS-243: Timing issues in tunnel mesh creation

- takes care of timing issues during mesh creation in a scaled setup
- introduction of a new immutable to synchronize between event processing
  of dpnTepsState & dpnTepsInfo

Change-Id: Iceed44605c84a73398d020a387f52e210e008bd1
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>

Additional DEBUG logs for bridge-interface deletion

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

Fix comment in TunnelInventoryStateListener

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

Do not leak implementation CompletableFutures

Rather than LockListener performing a lookup and modification of
CompletableFuture, expose a package-private method to remove the
lock -- thus properly encapsulating the operation.

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

Add JvmGlobalLocks

As explained in NETVIRT-1510, the String.intern()-based locking is
a thing of pure horror.

Provide a migration class, where the semantics are explained, so
that we can catch all places performing this funkiness. This will
bind the locking domain, which we can then split into finer domains,
properly expressing our locking.

JIRA: NETVIRT-1510
Change-Id: I82b1ad1c150ad191055ca3049525d99af0612e8e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make nullToEmpty return immutable empty lists

This matches the mdsal behaviour.

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

optimize creation of JobCoordinator key String in EntityOwnershipUtils

because (Generic)Entity.toString() appears to have a noticeable cost,
and it's easy to avoid, so let's do that locally here (without having to
change JobCoordinator), it cannot hurt and may help.

JIRA: GENIUS-237
Change-Id: I0263ab250b4adce75adefd279f23aec7e1feafbe
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

switch from FindBugs to SpotBugs

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

Unnecessarly log removal in auto-tunnels

Removed un-necessary trace log message being invoked so frequently.
It will help in lesser log messages in karaf.log

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

GENIUS-240 IdPoolListener equals() on DataObject seems to cause huge object allocation overhead

Cause
=====
ReleasedIdsHolder contains the list of ids have been released and these
ids are not used until the AvailableIdsHolder exhausts. Hence size of
ReleasedIdsHolder object keeps on increasing for each id release and
results in huge object allocation when two objects are compared.

Fix
====
ReleasedIdsHolder has an attribute called "availableIdCount" to keep track of
number of released ids. Instead of comparing entire ReleasedIdsHolder
object, we can only compare "availableIdCount" attribute to check if object is
updated or not.

Change-Id: I4222cfe8df4aaed7508c982a3d4f645b10497ecb
Signed-off-by: Ankit Jain <ankit.j.jain@ericsson.com>

GENIUS-213: vxlan tep tunnels are not being created

Problem
-------
EntityOwner was not coming active due to some syncronization issues, which
was resulting in skipping of Default-Transport-Zone creation.

Solution
--------
To resolve synchronization issue , now we are listening for
proper event which is received only when entity owner is elected.

Change-Id: I78a69ebbf71bc193ed0219f2a8fdc485acd3ca23
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

GENIUS-236: Excessive toString in OvsdbNodeListener.getOvsdbTepInfo()

Removed un-necessary trace log message being invoked so frequently.
It will help in lesser log messages in karaf.log

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

IMdsalApiManager clean-up

This patch removes a number of methods which are no longer used in
Genius and NetVirt (once I11c363097e0a667cba4e7ca52e5691d28ea9800c is
merged). It also adds a simpler constructor, allowing dependencies to
switch to that before it gets removed in a subsequent patch.

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

Handle nullable lists

... and a few other potential NPEs (with some drive-by clean-up, which
I can split out if necessary).

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

rootNode.getDataAfter() return null instead of emptylist

After Neon MRI version bump, on a node removal, rootNode.getDataAfter()
is returning null instead of empty list, and hence requires a change
in service-binding logic of GENIUS

JIRA: TSC-179
Change-Id: I098fed50647268b70469b7097b3979d2424f8a0f
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Add vtep optionOfTunnel values

Vteps now has an optionOfTunnel field which is initialised to null by
default but should be false in our tests.

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

Compare lists in TestIMdsalApiManager

assertFlows() ends up comparing a list to a set, which works with
Xtends’s text representations but fails with equals(). This patch
stores the contents of the set in a list before comparing.

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

remove TestableQueues from itm-impl/src/test, use infrautils'

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

remove un-used imports in ITM tests Xtend

right click > Source > Organize Xtend imports

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

Genius Neon MRI changes

Mockito & PowerMock bumps are deferred to "later".

JIRA: GENIUS-210
Change-Id: Ia693f0f19a8304ef8754253fc2956638876c529d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use plain @Nullable

The @NullableDecl reference added in
https://git.opendaylight.org/gerrit/75824 breaks when upgrading Guava;
use @Nullable instead.

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

Missing nicira's NxmOfFieldType's

+ Added missing nicira's NxmOfFieldType's.
+ Added couple of constants in MetaDataUtil which will be used by
  Netvirt.

Change-Id: I491fb106a0352169d526378225b1070c87fed8a7
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Potential fix for id duplication

We have seen the cases where same id was allocated to two different
keys.

Below are the scenarios which may result in allocating duplicate ids.

i) When an Id is released, as part of released id two DJC jobs are
scheduled. First job, to update the released id into the released id
holder of child pool, gets executed successfully. But the second job,
to delete the id from parent pool, somehow gets failed. This will end
up having same id exist at two places, and may eventually result in
allocating same id for two different keys.

Here fix is to return the future object to DJC so that job is retried,
if failed.

ii) It was observed that when a child pool is present in the Idmanager
datastore but not in local cache, Idmanager first creates the child pool
then stores the created child pool in the local cache, and then
allocates id from
the local cache. Since the same child pool is already present in the
datastore,
a pool update event gets triggered, which overrides the local cache. We
have
seen that this scenario is resulting in generating duplicate ids.

When Idmanager datastore is deleted, as part of this id pool delete
events are triggered, which clear the local pool cache. Since Idmanager
doesn't act on pool add event, local cache doesn't populate when same
Idmanager datastore is pushed back through RESTCONF. Hence this can
result in id pool
present is datastore but not in local cache.

If child pool is not present in the local cache, before creating the
child pool, it should be checked if the child is present in the
datastore or
not.

Change-Id: Ia08349bf227fdbfbdb1d6e9009a0eed35cc27b18
Signed-off-by: Ankit Jain <ankit.j.jain@ericsson.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Update docs header to Neon on master

This version name gets rendered into the main header of Genius' docs.

Now that these docs have moved from the docs repo to project-owned
repos, projects need to update this version themselves.

Should be Neon on master and Fluorine on stable/fluorine.

Change-Id: Ibab0c65217b320c3d1d3d72e69a85010c2c1f037
Jira: DOCS-112
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

GRE Config removal doesn't remove tunnel from Oper DS

When GRE config is removed, GRE external tunnel is not cleaned up on the
switch.
Fix is to cache the internal tunnel in InterfaceManager so that it knows
the internal tunnels to be ignored and only process the external tunnels.
This cache is built across the cluster

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

fix 2 minor Checkstyle problems

these appear in my Eclipse, but for some reason don't fail the build

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

Revert "Revert "Rename ether-types to monitor-protocol-type""

This reverts commit 018ddc36ef8744fb2210f9f404aa42c1a91ed7f5.

Reverting the reverted patch to pass the build.

The dependent patch [1] is already merged in Netvirt, hence this patch
which is revert of [0] is required to pass the build.

[0]. Genius: https://git.opendaylight.org/gerrit/#/c/76271/
[1]. Netvirt: https://git.opendaylight.org/gerrit/#/c/76272/

Change-Id: I54589eb7f9babe76c8b82060e10746a84f26696b
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Revert "Rename ether-types to monitor-protocol-type"

This reverts commit 3c5f6750cf080ce1d9311b03caef3c0a8302010e.

Change-Id: I0c401e21f0b54d82a39e24700892955122ba7ff5
Signed-off-by: Sam Hague <shague@redhat.com>

Drop Xtend Equinox Common overrides

These are no longer necessary and make it harder than it should be to
upgrade Xtend in odlparent.

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

Replace yang notifications in ipv6-nd-util.yang

+ In ipv6-nd-util.yang, instead of "notification na-received",
  neighbor-advertise-packet has been augmented with packet-metadata.
+ Netvirt patch [0] handles replacing yang notifications with direct API call
  for reliability.

This patch will break Netvirt build, hence the corresponding patch in
Netvirt [0] needs to be merged along with this one.

JIRA: NETVIRT-1434

[0] https://git.opendaylight.org/gerrit/#/c/76222/

Change-Id: I9f879f924fa34b5de1b181c9fef8f015ad1bab18
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Rename ether-types to monitor-protocol-type

+ Renamed ether-types to monitor-protocol-type in
  aliveness-monitor.yang.
+ Addressed review comments mentioned in [0].

This patch will break Netvirt build, hence the corresponding patch in
Netvirt [1] needs to be merged along with this one.

[0] https://git.opendaylight.org/gerrit/#/c/73027/
[1] https://git.opendaylight.org/gerrit/#/c/76272/

Change-Id: I15c6eb50b5e82ff8b9fa66086697fad408bf2ec1
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

very minor change to TestIMdsalApiManager

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

GENIUS-201: TepCommandHelper should avoid heavy DS reads in bean initialization

Changes done:

Separate worker queued to DJC on EOS owner to create the default transport zone.
Register listener in the bundle bring up.
 - If vteps are discovered for default TZ,  check if default-TZ is already created.
   - If present, add it
   - If not present ,check if the flag is enabled, then add it to the “tep Not Hosted”.
   - Handle default TZ creation as any other TZ creation and move the teps from “Not hosted” to appropriate TZ

Since default-TZ will be created during start of ItmProvider class
through JobCoordinator which runs in separate thread, this change
impacts on current test cases, so Updated ItmTepAutoConfigTest component
test cases and one change in ItmManagerRpcServiceTest.java

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

Mismatch b/w table 21 and table 220 reg values

Mismatch b/w table 21 and table 220 reg values after tep-delete/tep-add

Issue:
------
During external tep-delete/tep-add, ITM was treating it as
internal tunnel due to incorrect if check. Due to this during
tep-delete, table 21 flows are not deleted and hence it remains as
stale. Even tep-add doesn't program table 21 flows correctly as it is
treated as internal tunnel. Hence traffic fails between DC-GW and VM's.

Solution:
---------
In case of external tunnel, destination-device object will be of IP
address format where-as for internal tunnel, it would contain DPN-ID.
Corrected the if check accordingly.

JIRA: NETVIRT-1429

Change-Id: I267ecd239ddfc9f39a9a5497f9cb27b78ac7e454
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Add more MdsalApiManager test methods, key them

This adds implementations of some of the new MdsalApiManager methods
to TestIMdsalApiManager, allowing them to be used in downstream
tests.

It also reworks the storage to be keyed, using the same distinguishers
as the real datastore, avoiding errors related to duplicate handling.

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

Add NxMatch toString() implementations

These help greatly when debugging.

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

Cherry-picking the preventive fix from the ITM side for the
duplicate ITM tunnels issue.

Change-Id: Iee4c2bbdebca0f3f8ae843b367d9471aa43bd140
Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>