InterfaceManagerConfigurationTest using TestableQueues awaitEmpty

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

Clean up MDSALManager.removeGroup()

Removing a group only needs a DPN and group identifier; deprecate the
other forms, add a simple variant of removeGroup() and remove
duplicates.

This patch also moves the test package so that we avoid needing to
make methods public.

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

Cleanup config-parent usage

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

Migrating IDManager test cases to use GuiceRules

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

Removed unnecessary/unused CSS config yang remnants

Removed unnecessary/unused config yang imports and service type
identities.

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

Remove redundant dependencies in pom

These dependencies are already declared in parent pom.

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

Minor cleanup in pom

- Remove redundant dependencies already defined in parent pom.

Change-Id: Id4f05a454a17847995a5d0d2596038e3a537711b
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>

Remove redundant dependencies in pom

These dependencies are already declared in parent pom.

Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>
Change-Id: Id532fca6e8c97a9f54c55f9252eeec7b49a3402b

Remove unused parameters

This patch removes unused parameters (except where they’re obviously
WIP). Where necessary, deprecated variants of affected methods are
provided so downstreams don’t break straight away.

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

Optional clean-up

More whack-a-mole:
* method reference in IdManager;
* Optional-creation-instead-of-null-check in
  OvsInterfaceConfigAddHelper.

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

Initial bundle setup for SRM

This adds following:

1. Initial api, impl and shell bundles for ServiceRecoveryManager (SRM)
2. Adds them to genius features
3. Skeletal code for CLI and RPCs

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

Clean up collections of Futures

This patch replaces lists of Futures with singleton lists where
possible, and cleans up some transaction initialisations so we don’t
end up creating a transaction and leaving it dangling following an
early return.

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

Minor: LockManager "Failed to get lock" incl. number of retries

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

bug 7380: Add getIfaceInfoFromConfigDataStore

Add getInterfaceInfoFromConfigDataStore to
TestInterfaceManager

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

Remove deprecated CheckedFuture

Remove deprecated CheckedFuture.

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

Remove un-used MDSALUtil buildAction()

This has come up in https://git.opendaylight.org/gerrit/#/c/62921/

I've done a grep like search using this:

findx.sh "buildAction(" | grep -v "buildAction()" | grep ","

and found that this method appears to be un-used in genius and netvirt.

It's very weird anyway - why is it hard-coded to a PopVlanAction, but
called buildAction() like a general utility?  This probably shouldn't be
in this API.

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

Fix decl for ItmInternalTunnelAddWorker.updateOperationalDatastore

Function name stated write was to operational when it
is actually written to config. Plus, three out of four
params were not used in the function.

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

Make some classes final and not instantiables

Make some classes final and not instantiables.

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

Avoid nullable Optional references

Optional variables must never be null.

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

Remove deleteInterfaceChildEntry

This method is deprecated, obsolete and no longer used anywhere
(including NetVirt), let’s remove it.

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

Store original iface correctly in ISL worker

Presumably originalIface should store the originalIface parameter, not
the updatedIface.

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

idmanager now waits for Daexim's import-on-boot

This requires https://git.opendaylight.org/gerrit/#/c/55035/ and
https://git.opendaylight.org/gerrit/#/c/56686/ and
https://git.opendaylight.org/gerrit/#/c/61486/ to go into Daexim,
first.

read also JavaDoc on https://git.opendaylight.org/gerrit/#/c/61480/ (BUT
this change does NOT require that c/61480; that's just a generalization
of this approach)

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

ITM JMX MBean un-registeration

This is required for https://git.opendaylight.org/gerrit/#/c/62656/
(because otherwise there are ERROR logs in the test for JMX name dupes.)

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

Add support for mpls-gre tunnels

OVS 2.8 supports mpls-gre as type=gr with option packet_type=legacy_l3.
With this change it will be possible to use MPLS-gre tunnels for DCGW
connectivity on switches that support it.

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

Add LogCaptureRule to some component tests, to fail if any errors logged

which is useful as is already to see test failure causes more clearly,
but will provide even more useful full information together with
https://git.opendaylight.org/gerrit/#/c/62654/

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

Minor: ItmProvider Eclipse auto-save code clean-up (mechanical)

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

Bug 9099 - Suspected WriteOnlyTransaction leak in interfacemanager

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

Service Binding on Tunnels - Basic Framework

Adding the basic constants and utilities and the base structure
of code flow for service binding on interface-type

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

Remove unused references to DataChangeListener

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

LockManager unlock should not pretend its successful when it was not

or was this intentionally done as-is, for some reason I'm missing?

It's seems strange to "hide" error during unlock (currently it logs an
INFO not even an ERROR, and returns sucess, when if there was a
problem).

If so, then it should be documented with an in-line comment...

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

Remove TestableDataChangeListener

This class is deprecated and implements DataChangeListener so
remove it, although I don't know if there are any downstream users.

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

LockManagerTest without (unused) TestableDataTreeChangeListenerModule

It didn't do any harm, but it's not required or used (because this test
nowhere waits for a data store listener), so simpler.

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

Cleanup

- Add private constructor to this utility class
- Fix log
- Move private method to the inner class

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

Fix failing LockManagerTest (NoClassDefFoundError)

The LockManagerTest is currently failing on clean master with:
java.lang.NoClassDefFoundError:
org/opendaylight/controller/md/sal/binding/impl/BindingToNormalizedNodeCodec

The cause of this problem is that genius.testutils has a dependency to
sal-binding-broker-impl's attached <type>test-jar, only - but not the
main artifact (where BindingToNormalizedNodeCodec is).

It's easy to fix like this - but I don't really understand, and don't
have time to further investigate, how this could have suddenly broken...
presumably someone changed a dependency in some POM in controller or
somewhere which had this as a side effect.  Be that as it may - it works
again now with this change.

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

Cleanup

- Private constructor for utility class
- Replace .size() by .empty()
- Change method names to comply with Java style

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

Remove start() method

- Remove start() method to avoid the use of a
useless PostConstruct method, following the recommendations in [1].

- Replace info by debug
- Replace magic number by constant

[1] https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

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

Remove DataStoreJobCoordinator enqueueJob(AbstractDataStoreJob job)

because when we moved genius.DJC to infrautils.JC we had concluded that
this method does not need to be exposes publicly, and (at the time) was
only used in 1 place (netvirt vpnmanager), which was adjusted in
https://git.opendaylight.org/gerrit/#/c/60264/.

Removing this method now makes it a bit easier to in the future either
try making genius.DJC delegate to infrautils.JC to avoid having this
code twice in the code base (which would require both of their API's to
be identical, o proposing to remove this method) or directly removing
it.

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

Bug 9034: LockManager cancel() ReadWriteTransaction if it's not submit()

Based on Robert's analysis, this change will not fix Bug 9034, because
that's due to a leaked TransactionChain (not the ReadWriteTransaction
being fixed here) - but this could well introduce ANOTHER memory leak
in mdsal, so we should get this in nevertheless ASAP.

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

Genius CSIT Failure : Missing Mandatory Node Error

Due to a recent commit in yangtools, Genius CSIT has broken
as the code was missing to populate one of the mandatory nodes in
interface-state

https://git.opendaylight.org/gerrit/#/c/48672/

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

Use INFO for logging null DPID

OVS does not assign a DPID until later in the bridge reporting cycle so it isn't an error if the DPID is not set.

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

Bug 8938 - Resource-batch manager enhancement

Adding capabilities to return Futures in Resource batch manager calls.
This fetures helps the caller to know whether the submitted tx job has
been successful or not.
And upon success or failure , the caller can take more corrective
action.

Change-Id: Iec5a08721979cfa15a34617fb1ccea0065853c84
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>

Move start() method into the constructor

Move start() method into the constructor to avoid the use of a
PostConstruct method, following the recommendations in [1].

[1] https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

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

Enable checkstyle at common-parent

Remove the checkstyle plugin per pom and enable it at the common level

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

Replace *Index(" by *Index('

Replace lastIndex("") and indexOf("") by their character counterparts.
This improve performance.

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

Fix initialization logs in service-binding helpers

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

BUG 9000 - Fix ITM config file not installed

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

BUG 9001 - Fix race condition on OpenFlow based tunnels

This change ensures for openflow based tunnels that the child interface
entry is created before both the parent and child interface states are
added. Otherwise the entry might be added after the inventory listener
is triggered so that child interface state is never added.

This is achieved creating the child interface entry within the ongoing
job, which key is changed yo the parent interface name in case of
openflow tunnels, so that it is synchronized with the interface state
operations.

Additionally, fix a problem when adding the child interface state if
it is not the first tunnel interface on the node and the parent
interface already exists for the openflow based tunnel.
InterfaceManagerCommonUtils#addStateEntry would fail to do so, using
OvsInterfaceStateAddHelper instead.

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

Fix namespace violations in lockmanager-impl

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

Fixed YANG key'ed list order probs in InterfaceManagerConfigurationTest

YANG keyed lists are backed by maps, and re-created whenever they're
de-serialised (which happens e.g. when getAugmentation() is called,
apparently). This patch rebuilds TerminationPoints before comparing
them, sorting a couple of keyed lists.

See
https://lists.opendaylight.org/pipermail/genius-dev/2017-July/001713.html

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

Ignore newTunnelInterface tests in interafcemanager

As it is not working consistent, and hitting timing issues in master

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

Merge "Logging enchancements for genius"

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

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

Logging enchancements for genius

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

Merge "Log enhancements for better debuggability"

Merge "Bump odlparent 2.0.2 to 2.0.4"

Merge "Spec for ServiceRecovery Framework"

Bump odlparent 2.0.2 to 2.0.4

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

Merge "Bug 8882 - With conntrack SNAT communication with PNF fails "

Log enhancements for better debuggability

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

Merge "Add private constructor to this utility class"

Merge "Cleanup"

Merge "Changing the default value of Interfacemanager statistics polling interval to 15 Minutes"

Merge "BUG 8876 - Fix to remove ELAN interface flows when external network deleted"

Bug 8882 - With conntrack SNAT communication with PNF fails

Make the load metadata  start and end bit configurable.

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

Merge "Add GPE option to ITM config"

Bug 7599 -hwvtep ucast mac consumption improvement

Caching ITM External Tunnels so that it would be faster to get from cache
then to read from Data store.

Change-Id: Iac67eb65ac546cf1c197cedd5e039b06d54ab2af
Signed-off-by: Akash Kumar Sahu <a.k.sahu@ericsson.com>

Merge "Bug 8859 : Table 220 programmed with wrong service-index"

Merge "IdManager: Use lock only when required"

Changing the default value of Interfacemanager statistics polling interval to 15 Minutes

Change-Id: Ia7dbc21971e60fd788956876b8457cbdc989a88a
Signed-off-by: Gowri R <gowri.r@ericsson.com>

Merge "Make DJC to be more concurrent and fixing signal issue b/w enqueue jobs"

Bug 8859 : Table 220 programmed with wrong service-index

Fix synchronization issues with service binding transaction submissions

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

BUG 8876 - Fix to remove ELAN interface flows when external network deleted

Add check for null when getting Interface object from Config DS.
This changeset is related to the patch:
https://git.opendaylight.org/gerrit/#/c/61131/

Change-Id: I57bb96081afcc114346dfea08521ab0df7f9b8e8
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Bug 8882 - With conntrack SNAT communication with PNF fails

Added support for load metadata action.

depends on https://git.opendaylight.org/gerrit/#/c/60869

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

Make DJC to be more concurrent and fixing signal issue b/w enqueue jobs

and job queue handler

Change-Id: I7bfbd15f7a8fa1b80555a0a99689a536a4896c8e
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

IdManager: Use lock only when required

It was observed that there were more than one thread waiting to acquire
lock for the uniqueKey. This can be avoided since there is a local map
which tracks the idValue allocated for every request until the idValue
is persisted in the DS.

During testing, there were some conflictModificationExceptions thrown.
This was because the arrayList for delayedIdEntries were not set as
CopyOnArrayList at certain places.

Also have done some refactoring for allocateIdFromLocalPool()

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

Fix Service-binding flowRef

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

Cleanup

- Add private constructor to utility class
- Remove useless code
- Replace size() == 0 by is empty.

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

Add private constructor to this utility class

Add private constructor to this utility class.

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

Fix IfmClusterUtils logs

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

Merge "Enqueued Job not getting cleaned up in some cases"

Merge "Remove derivation from controller config-parent"

Spec for ServiceRecovery Framework

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

Merge "Avoid .transform() NPEs"

Merge "hwvtep transaction batching separation across shards"

Merge "Bug 8476 Add support for logical switch replication mode"

Merge "Change method names to comply with naming conventions"

Merge "Guard some TRACE logs in DJC"

Merge "Add a private constructor to avoid instantiation"

Merge "Remove un-used TestDataStoreJobCoordinator"

Merge "Remove un-used newWriteOnlyTransaction() in removeTerminationEndPoint()"

Add GPE option to ITM config

Add a GPE option on ITM config as a straight forward way to enable the
VXLAN GPE extension system wide.

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

Bug 8476 Add support for logical switch replication mode

With OVS 2.6, the Logical Switch table, in the hardware_vtep schema, has
a new field, replication_mode. This bug will be used to add support to
netvirt, ovsdb, and genius for setting the replication_mode of logical
switches. The default will be for ODL to set the replication_mode
to "source_node" when creating logical switches.

Refer to [1] for more details on replication_mode.

[1] http://openvswitch.org/support/dist-docs/vtep.5.html

This patch depends on ovsdb patch:
https://git.opendaylight.org/gerrit/#/c/57209/

Change-Id: I538634a791f73d60b5a9c7cc479f4c42847889bd
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Capture flow addition to debug log

debug log any flows that get added to config DS
trace log flow deletion

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

Change method names to comply with naming conventions

Change method names to comply with naming conventions.

And minor cleanup.

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

Remove derivation from controller config-parent

Genius does not use the config subsystem so does nto need to
derive from controller config-parent.

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

Add a private constructor to avoid instantiation

Add a private constructor to avoid instantiation of this utility class.

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

Avoid .transform() NPEs

Guava’s Optional::transform doesn’t support null values returned by
the mapping function — it throws a NPE if that happens. java.util’s
Optional::map does support null values and returns an empty Optional.
Combined with the current YANG-based code’s ability to return null,
this means it’s better to use java.util’s method; this patch switches
all YANG-based Optional transforms to java.util.

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

Enqueued Job not getting cleaned up in some cases

Jobs were not getting cleaned up from JobQueue in case of Throwables.
This results in some residual jobs remaining in the JobQueue forever,
and unnecessarily hogging CPU.

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

Merge "Port Number Change on OVS restart not handled by IFM"

hwvtep transaction batching separation across shards

Hwvtep helpers were doing a single transaction for operations
spread across shards, which is not recommended, and hence refactoring
the same. Also, in case of continuous HWVTEP channel flap, there are
some issues observed with tunnels not getting programmed on TOR.
Fixing this by avoiding read on topology/oper DS to derive global nodeId.

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