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>

Minor changes

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

Support write-only typed cancellable transactions

When support for cancelling non-written write-only transactions was
added, typed write-only transactions got left out; this patch adds
support for them.

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

add @CheckReturnValue to Managed/New/TransactionFactory/Runner/Impl

to 3 internal private & protected methods, and more consistently to
public; where we had it on some but not all @Override methods.. the
ManagedTransactionFactory and ManagedNewTransactionRunner interface
methods also have it.

Perhaps impl don't need to repeat it (inheritance of annotations is a a
bit of a mess AFAIK), but it can't hurt.  Otherwise let's remove it from
all impl, for consistency (but keep it on those 3 internal methods).

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

Migrate genius user-guide to project repo

Issue: DOCS-69
Change-Id: Ie81e3ca990a6a14bcfc65ea354fd11713773efcf
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

remove apparently un-used public method in IdManager impl

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

make ManagedTransactionFactoryImpl's internal methods protected

instead of package local, just on the off chance that this currently
entirely package local class will be made public when moving to mdsal.

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

Refer to java.util Future instead of Netty’s

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

minor: move JavaDoc to public RetryingManagedNewTransactionRunner

from package-local RetryingManagedNewTransactionRunnerImpl

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

Refactor ManagedTransactionFactory to reduce dupes

This reworks the factory and runner to have a single method in charge
of each type of behaviour.

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

Document that the commit result is nullable

The result passed to ManagedNewTransactionRunnerImpl::commit or
ManagedTransactionFactoryImpl::commit can be null, document that by
adding @Nullable.

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

make never written to WriteTransaction cancel themselves

follow up to Ic7175574097702a45290c7500fcad2c37475a228, which did this
for ReadWriteTransaction operations in ManagedNewTransactionRunnerImpl;
this applies the same idea also to WriteTransaction, for consistency.

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

reduce duplication in ManagedTransactionFactory impls (2/2)

in ManagedNewTransactionRunnerImpl and ManagedTransactionFactoryImpl

incl. minor required NPE fix in TepCommandHelperTest

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

Add support for retrying when reads fail

In some circumstances (e.g. cluster bring-up, or just after a node
fails), reads should be retried. This patch implements read-failure
handling in retrying transactions, for read-only and read-write
transactions.

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

remove UpgradeState and UpgradeStateImpl (not UpgradeStateListener)

now that all of its users in netvirt have been switched to serviceutils.

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

deprecate UpgradeState and UpgradeStateImpl (not UpgradeStateListener)

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

base UpgradeStateImpl on (cached) UpgradeConfig instead of Config

This gives a consistent view of the genius and serviceutils upgrade
state to overall code possibly using a mix of both.  Before this change,
they could potentially see an inconsistent view of the
isUpgradeInProgress flag since they're obtaining it from different
sources.

In reality we only set this during installation, and it should normally
get in sync quickly, so IRL it's very unlikely to be a problem, but it's
still good to do this right.

based on review feedback in I08601afb77ca9faaa29e2839aefcdfbcbbb66f5e

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

factor out UpgradeStateImpl from UpgradeStateListener

This is a pure technical refactoring and it should still behave in
exactly the same way as before; during GENIUS-207 it just occurred to me
that there really are 2 separate concerns here, and this makes that much
clearer.

Also added some JavaDoc explaining what both of these actually do.

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

minor edit to ManagedTransactionFactory's JavaDoc (1/2)

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

use un-clustered UpgradeStateListener with cache

doing this with a clustered DTCL only leads to
OptimisticLockFailedException (OLFE) on nodes, but there's really no
need for this to be clustered; it's sufficient for 1 node (leader) to
ract to changes and update the flag.

We also replace the isUpgradeInProgress with a DataObjectCache, because
that (only) does need to use a clustered DTCL to work correctly.

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

reduce duplication in ManagedTransactionFactory impls (1/2)

in ManagedNewTransactionRunnerImpl and ManagedTransactionFactoryImpl

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

introduce WriteTrackingTransaction

this is used in the next commit to reduce code duplication

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

minor: add missing @Override in ManagedNewTransactionRunnerImpl

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

Add support for transaction chains in managed txes

The nice thing about transaction chains is that they allow multiple
transactions to be processed in sequence and see the results of
previous transactions, without needing to explicitly wait (which is a
common pattern in NetVirt).

This is a first stab at support for transaction chains, in particular
it doesn’t allow overriding the listener. It does however introduce
support for read-only transactions, which become particularly useful
with transaction chains. Asynchronous support is coming in another
patch.

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

Split out a managed transaction factory

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

Add dp-id to ARP notifications

In order to fix the below referenced JIRA natmanager
must configure flows for a FIP assigned to a neutron
port that is not added to br-int. Normally FIP flows
are configured on the switch which hosts the assignee
port, once it is bound to the switch. However, since
this is a *virtual* ip it is not attached to any switch,
just advertised by GARP by the VM that handles the VIP.
The fix in netvirt will rely on intercepting the GARP to
determine which switch to configure the flows on. This
patch adds the dp-id (extracted from the node's node-id)
to the ARP notification for this purpose.

JIRA: NETVIRT-1402

Change-Id: Ica98567de8d5623b677008db8532e3077232676d
Signed-off-by: Josh <jhershbe@redhat.com>

fix minor typo in DataObjectCache constructor argument name

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

Stale entries in AlivenessMonitor Datastore

JIRA: GENIUS-206
Change-Id: I593bb432bde76db3db8cd0fe8fd8bfc27ad87b9b
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>

forbid use of java.util.concurrent.Executors via Checkstyle

Because we want to enforce use of
org.opendaylight.infrautils.utils.concurrent.Executors
instead of java.util's, because that enforces using a specific prefix
for new thread names and always logs uncaught exceptions.

Includes fixing up 5 remaining usages.

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

git use infrautils' instead of JUL Executors

because I'd like to start enforcing non-use of direct JUL Executors.

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

Convert interfacemanager to datastore-constrained

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

Make read-only transactions cancel themselves

When a read-write transaction is only used to read from the datastore,
there's no point in committing it; this patch changes managed
read-write transactions so they are cancelled rather than submitted
when appropriate.

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

Return Set instead of Iterable in test

Method signature will be changed to Set to define equality, prepare
for that.

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

Drop odlparent FindBugs override

Now that we're using odlparent 3.1.3, we no longer need to pull in a
newer version of its FindBugs configuration.

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

use managed transaction in AlivenessMonitor processReceivedMonitorKey()

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

Remove references to mdsal-model-artifacts

This artifact is going away, migrate to mdsal-artifacts.

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

make MDSALManager's 3 inner classes private instead of package local

and make some private helper methods static because they can be

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

remove not needed version number in alivenessmonitor POM

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

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

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

API to expose tunnelstate cache

Providing an API to get tunnel state from cache
to avoid DS read.

Change-Id: I961ce3bcac9d89d998814846633220414cdc605d
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>

GENIUS-189: Enforce use of managed transactions
- uses the wrapper provided in c/74759/

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

Bump yangtools to 2.0.10

This fixes an issue with actions, hence we need it in Fluorine.

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

Fix for ITM reporting wrong tunnel status

When the tunnel is down and the controller undergoes a reboot, the tunnel
BFD states come as UP and DOWN back to back and this results in the DOWN
state not getting reflected correctly in ITM.
Fix is to cache the tunnel status that comes as DOWN in "update" and
process the add DTCN and then process the update tunnel state DTCN
Fixed checkstyle issues
Fixed issues due to rebase
Moved the unprocessed cache to a separate class
Addressed review comments

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

Upgrade support for ItmDirectTunnels

When the controller gets upgraded from non-itmdirecttunnels version to
itmdirecttunnels version, then some datastores need to be cleaned up.
This patch cleans up two such datastores.
Using TunnelLstenerCreator for conditional instantiation of config and
Internal listener for ITM Direct Tunnels

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

netvirt 3node CSIT - Unable to initialize bean tepCommandHelper

In a 3 node setup, TepCommandHelper bean initialization fails, as all
3 nodes are writing to the same DS, resulting in ConflicitngModificationException.
The bigger question here, whether we really want to do such a heavy method
configureTunnelType() in the bundle bringup thread. It is having several datastore
reads and writes.

JIRA: GENIUS-192

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

Bug: Genius-167

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

GENIUS-195: Transport Zone default-transport-zone has no subnets

This patch is for fix of JIRA bug GENIUS-195
Non-relevant error log message will be removed occuring in karaf logs
which are not error in nature.

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

Enforce use of (only) managed transactions in fcapsmanager

JIRA: GENIUS-182

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

Enforce use of (only) managed transactions in fcapsapplication

JIRA: GENIUS-182

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

Enforce use of (only) managed transactions in idmanager

The real work of changing the code was previously done
by Stephen, so this change just enforces future non-regression.

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

Enforce use of (only) managed transactions in arputil

JIRA: GENIUS-181

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

Tweaks to avoid unncessary .toString

- includes removing redundant if check

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

Renaming workers in itm listeners

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

Remove openflowplugin app-config-pusher dependency

This feature is not used by GENIUS.

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

TepCommandHelper clean-up

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