fix broken build

due to today's I4f16674ba65f499944e4582fb900d90e873d14e2 (INFRAUTILS-51)

by using the new TestSystemReadyMonitor from infrautils ready
introduced in I545195b1b976f5951d2dc8e9599e7ab33bc407e9

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

avoid any extra memory allocation in Version.fromString()

as this is called a lot, it's (apparently, Java Mission Control on a
scale lab test shows) worth it to avoid the RegExp Matcher and x3
intermediate String object allocations, which we can safe here by
implementing this a little smarter (lower level; feels like 6502!).

JIRA: OVSDB-469
Change-Id: I642e0bfd698e594d3c263f0a335b92f80fbde8c4
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Incorrect IPv6 address for controller

getControllersFromOvsdbNode() method to uses ':' as
delimiter for protocol, IP and Port, which is broken
by IPv6 addresses which use : as delimiters for address
octets.

Fix is to handle IPv6 addresses differently.

JIRA: OVSDB-466
Change-Id: Ifc7070d0d3ba5e60a124dc612cdfc66cafa1527b
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

fix broken build by adding new SystemReadyMonitor getFailureCause() impl

impact of I2420c5440366daee626da9b0ea62bbe322bc1cde

JIRA: INFRAUTILS-48
Change-Id: Ifa7255497361a17ab52784616b908efdde6f2dcd
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: Iad90d1107f4b407dfd947761d96e97311496b759
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

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

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

Cleanup API assumptions

Migrate Identifier.Item use and allow DataObjectModification
to return a collection of implementations.

JIRA: MDSAL-342
Change-Id: I65e398ca6d6c1aabbd5a12eb75601cafb551d22e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump to odlparent 3.1.3

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

Move OVSDB documentation from docs

Move OVSDB docs hosted in docs project
to OVSDB/docs.

Change-Id: Iac705f47a8b8a68662bd880b101bca32d4d719b1
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Use union's stringValue() method

getValue() clashes which user-derived names, migrate to stringValue(),
which also has a better type.

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

Do not mock IpAddress

There really is no need, we can use the generated implementation.

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

Do not mock Ipv4Address

There really is no need, we can use the generated implementation.

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

Do not mock enumeration

This reworks mocking to eliminate open-mocked implementation, but
rather use a Guava implementation.

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

Pull in minimal set of models

Rather than pulling in all mdsal models, require on the models
we specifically need.

JIRA: MDSAL-363
Change-Id: I63e2f4a01a82e361dd2847fed6796461692ba903
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not mock InetAddress

Rather than mocking it, create a known-good constant and rely on it.

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

Remove an unneded explicit cast

Implicit cast does better here, as inference works out nullness
mapping.

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

Move unchecked warnings

Tagging specific statements is better here, as there are only two. It makes
the method better protected and the code is more readable
('here I need to do a cast, which is deemed unsafe').

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

Expose methods for testing

Using @VisibleForTesting to expose a method from private to package-private,
hence we do not need to use WhiteBox reflection and can do direct calls.

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

Do not squash PortNumber instances in test

There is no need to squash all PortNumbers to one instance, hence
reduce the use of mockito.

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

Eliminate unneeded mocking of static methods

We really do not need to override behavior of contructors and
static methods, do not do that.

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

Do not mock generated classes

These is no point in mocking classes which deliver well-defined
functionality. Just use them without mocking their contents.

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

Fix some eclipse warnings

Deprecation and unneeded raw types, this patch fixes them.

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

Remove powermock from library/impl

There is no need for powermock features in these tests, remove
the dependency.

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

Add missing southbound.cfg to etc

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

OVSDB-439 Stale connection check

if old client echo succeeds
   do not notify new client as it has to wait
else
   if all old clients got disconnected notify the new client

if new client gets disconnected remove it from pending clients

JIRA: OVSDB-439
JIRA: OVSDB-462
Change-Id: Id3cd37559825d7f0ef58331bfbd4757eae41d253
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

OVSDB-462 Bridge randomly missing in operds

After registering for eos check if eos state is set
if eos state is set go ahead and register for callbacks which evenutually
puts the bridge in operational datastore.

Added error log statement for transaction chain failures

Addressed connection flap race conditions.
If the bridge disconnects from one odl and connects to other odl.
Before disowning ownership, delete the bridge from oper store.
The second odl after getting the ownership will put it back in oper store.

Change-Id: I0fafa1c1d3a73523bc102e2d720fdcf65dba1e3f
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Update odl-jackson-2.8 version

This version should reference odlparent version, which unfortunately
is not properly exposed. Fix the declaration here while odlparent
fixes its artifacts.pom.

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

Bump odlparent to 3.1.2

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

NETVIRT-1241 : Preprovisioning of l2gw conf. fails

Removing use of ThreadLocals to avoid Np in this usecase when switch
gets connected after provisioning l2gw and l2gwconnection.

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

Make UpdateNotification context a String

UpdateNotification contexts are always strings, use the appropriate
type.

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

Don't use hostIP for controller-ip

If unable to get controller IP from manager entries of
OvsdbNode, don't assume hostIp to be controller IP.

JIRA: OVSDB-463
Change-Id: Ibfb34eb9d8b95fd676d9a5c6be458da02d5485d8
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Use odl-jackson-2.8

We should use the odl-jackson-2.8 feature to fulfill our dependencies
on Jackson.

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

defer OVSDB port 6640 opening until system is ready

JIRA: OVSDB-458
Change-Id: Ic51e0bf3144489b0dc0297191d01c16bdc0e26a5
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

OVSDB-458 DiagStatus support for OVSDB

Change-Id: I5a3d46d612d478746d25f1e7c0ef4095f995c839
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Make OvsdbConnectionService instance-only

This patch ensures that the volatile data managed by
OvsdbConnectionService is stored per instance, and it removes the
static initialiser and all non-BluePrint instantiations. Other library
users can still create their own instances if necessary, but the
library code will no longer create multiple instances by default,
ensuring that BluePrint-mediated configuration changes are applied.

Change-Id: I41ca0c679d68e7dfeba3b5fd503194180358a9a2
JIRA: OVSDB-465
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Add org.osgi.core dependency to hwvtepsouthbound-impl

This was picked up transitively from config-api but that
is going away.

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

Make OVSDB Listener IP configurable

JIRA: OVSDB-464
Change-Id: I11a852f497eab655e17d4a2c230f319282424283
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Add specs folder and design spec template

JIRA: OVSDB-461
Change-Id: I0b6e34ea32c9e5c41df889ed66d680b0df1039f7
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>

Remove unused references to BindingAwareBroker

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

Adjust to AbstractMdsalTestBase changes

getSession() is being removed so don't use it.

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

TSC-101: Fixup Augmentable and Identifiable methods change

This is a fixup of the change in binding codegen, adjusting:
- getKey() -> key()
- setKey() -> withKey()
- getAugmentation() -> augmentation()

Change-Id: I5158dd9f133470150c80bf5cc20766c58fdbbf4b
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Add Documentation for OVSDB

lfdocs-conf is a new project that allows us to centrally manage
project theme configuration without having to duplicate it
everywhere or have the "docs" project become a large submodule
project.

We will push to ReadTheDocs separately and then create a new path
for this doc under docs.opendaylight.org/projects/integration-packaging

For further details refer to:
http://docs.releng.linuxfoundation.org/en/latest/project-documentation.html

JIRA: DOCS-69
Change-Id: Ibd03538877a5078aae5c6b87dce8be5a6c0dae7e
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Detect stale conns when ping times out

When an ovsdb client connects StalePassiveConnectionService
checks to see if there are any previous connections to that
same client. If there are, they are pinged to determine
whether or not they are still actually connected. The
callback that handles the ping results does not fire
properly in the event of a timeout and the onFailure method
does not call
OvsdbConnectionService.notifyListenerForPassiveConnection
for the new connection. This results in the new connection
not being being reported "up the stack."

Note: easiest way to reproduce this is to "unplug" (destroy
in libvirt) a VM running the OVS and then start it.

Issue: NETVIRT-1178
Change-Id: Ie3494c46719954f862d8edb6e8921a752dcd30ea
Signed-off-by: Josh <jhershbe@redhat.com>

OVSDB-457 Populate mac_in_use field for bridge ports

1. Added yang for mac_in_use and mac fields.
2. Added method for updating Port's operational with mac_in_use
   and mac [if present].

Change-Id: Id0624b273a2e81910277f3041b2398e9fe8f0019
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Remove usage of AsyncDataChangeEvent

AsynDataChangeEvent is being removed in controller - it's used
internally in southbound-impl so replace the usage with an internal
DataChangeEvent interface.

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

Bump to odlparent 3.1.0 and yangtools 2.0.3

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

Revert "Add Karaf build profile"

This reverts commit f2d14686bf2ad4d8086974b2709676fde167b634.

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

Convert to mdsal EntityOwnershipService APIs

The controller APIs are deprecated and being removed.

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

Add Karaf build profile

Project local Karaf distros are handy for devs to test their work
however is unneeded by autorelease builds and should not be released
as part of the Simultanious Release. Add a profile that is active by
default so that default behaviour is unchanged however allows the
autorelease project to disable building this module.

Issue: RELENG-86
Change-Id: I1197b8f705c6eb77d21ac358afd53a696391b14f
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove obsolete Maven Site configuration

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

Ensure bundles are present in a single feature

The following bundles are currently included in multiple features:
* southbound-api
* utils.config
* utils.mdsal-utils
* utils.southbound-utils

This patch ensures that the relevant feature is used wherever
appropriate, so that each of these bundles is only included in a
single feature.

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

Check for null certManager

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

OVSDB-449 Fixes loading keystores dynamically per connection

Currently when OVSDB manager is started with SSL, the SSL Context is
only fetched once, which inherently means the keystores are only read
that one time.  This patch changes the behavior so that the SSL Context
is fetched everytime a new connection is opened, which in turn will
update the keystore data.

Change-Id: Iaafbd34afcb5f4708b48eb3d64eca14ef0a107e8
Signed-off-by: Tim Rozet <trozet@redhat.com>

Enable findbugs enforcement in parent pom

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

Fix findbugs violations in southbound-impl

- Method invokes inefficient Number constructor; use static valueOf instead
- Inefficient use of keySet iterator instead of entrySet iterator
- Reliance on default encoding
- Method ignores exceptional return value
- Should be a static inner class
- Private method is never called
- Method invokes inefficient Number constructor; use static valueOf instead
- Return value of method without side effect is ignored
- Result of integer multiplication cast to long
- Dead store to local variable
- Write to static field from instance method
- Parameter must be non-null but is marked as nullable
- Redundant nullcheck of value known to be non-null

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

Fix odl-ovsdb-utils

This feature should depend on odl-mdsal-broker-local to pull
in the dependencies.

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

Fix findbugs violations in hwvtepsouthbound-impl

- Method call passes null for non-null parameter
- Nullcheck of value previously dereferenced
- Method ignores exceptional return value
- Inefficient use of keySet iterator instead of entrySet iterator
- Should be a static inner class
- Private method is never called
- Unread field
- Result of integer multiplication cast to long
- Dead store to local variable
- Method uses the same code for two switch clauses
- Write to static field from instance method
- Parameter must be non-null but is marked as nullable
- Redundant nullcheck of value known to be non-null
- Field not initialized in constructor but dereferenced without null check
- Useless object created

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

Fix findbugs violations in utils

- Naked notify
- An apparent infinite recursive loop
- Field isn't final but should be
- Boxing/unboxing to parse a primitive
- Method concatenates strings using + in a loop
- Method ignores exceptional return value
- Reliance on default encoding
- Should be a static inner clas
- Could be refactored into a static inner class
- Result of integer multiplication cast to long

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

Fix findbugs violations in library

- Possible null pointer dereference
- Equals method should not assume anything about the type of its argument
- Method ignores exceptional return value
- Equals checks for incompatible operand
- Unusual equals method
- Class is Serializable, but doesn't define serialVersionUID
- Consider using Locale parameterized version of invoked method
- Field isn't final but should be
- Incorrect lazy initialization of static field
- Inefficient use of keySet iterator instead of entrySet iterator
- Should be a static inner class
- Boxing/unboxing to parse a primitive
- Unread field
- Return value of method without side effect is ignored
- Write to static field from instance method
- Load of known null value

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

Fix checkstyle violations in ITs

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

Enable checkstyle enforcement in the parent pom

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

Fix checkstyle violations in hwvtepsouthbound-impl

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

distinguish between controller and device logs

Change-Id: I5fe0a7015f100e20747ab01b514b8190b527faaf
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

OVSDB-453: Use findModules instead of findModule by specific revision

findModules returns a list sorted by latest revision so take the first
element.

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

Fix logging configuration in IT classes

https://git.opendaylight.org/gerrit/#/c/68992/ removed the
logConfiguraton method AbstractConfigTestBase so fix
resulting compilation error..

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

Fix checkstyle violations in utils

NotifyingDataChangeListener is fixed to be thread-safe, with
ConcurrentHashMap-backed sets and volatile fields where necessary.

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

Fix checkstyle violations in schema

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

Add binding-parent pom

This will contain common configurations for checkstyle and
findbugs.

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

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

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

Added device transaction log cli

Added a cli to display device transaction history

Change-Id: I00d8f5bd287342691c7b69b0b75f79c2cdcee1e8
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 8671 same client notified multiple times

only leader node will initialize topology
each node listens for topology creation/add event
once it receives the add event opens the ovsdb port to clients
Now we are absolutely sure that topology node can be created for the
new incoming connections.

when a new node joins/leaves the cluster , the leader node
will reinit the topology which is a no op event

Change-Id: I45c319195f593d8bd99b19f07f5568492cf9077f
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 6578 added mdsal read retry

Change-Id: I5dba7362bf41ea84baa35dc29a6a4a6a69ba78fa
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

make reconciliation the first transaction

jira bug ovsdb-441

When the south bound device disconnects and connects back reconciliation
task will try to reconcile to the device.
At the same time if the application pushes some more config and that gets
executed,
then the reconciliation task if executed later will end up corrupting the
device state.
Ensure that reconciliation transaction is the first towards the device

Change-Id: I5b5df1e8cdc3d96e7aa518a5012759d39afe048f
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

OVSDB-438: missing operational node

Handle the following scenarios
client connects to only one odl controller via ha proxy

1) client disconnects and connects back to same node after some delay
2) client disconnects and connects back to same node immediately
3) client disconnects and connects to another node after some delay
4) client disconnects and connects to another node immediately
5) client disconnects and never connects back

When client disconnects all the odl controllers are trying to cleanup the oper
node.
When client connects the owner odl controller is trying to create the oper node.

When the processing of one odl controller which is trying to cleanup the opernode is
delayed , then we end up client node missing in oper topology.

Fix

Listen for oper node removal and if some client is still connected then
recreate it.

Change-Id: Ib88e8385f199c2105c51ce49e01e4318c1ae1463
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Use packaging=bundle for utils.config

This prevents wrapping when the jar is moved to OSGi world.

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

export utils.{config, mdsal-utils, southbound-utils} as feature

Those bundles are used by genius and neturon northbound.
They should be consumable as feature, not bundle.

For detailed discussion, please refer to the followings
https://lists.opendaylight.org/pipermail/release/2018-January/013604.html
https://lists.opendaylight.org/pipermail/release/2017-September/012356.html
NEUTRON-156

Change-Id: I845989115b28bfc9685ed7fdb43e604e16701fe1
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix IidCodec broken by new yangtools

There are multiple versions of network-topology loaded
and yangtools can't tell which one to use. It has ramifications
beyond just OVSDB, but this fixes functionality issues in
OVSDB.

Root issue is two different files with different versions
bundled together. It may not have been issue earlier but is now.

Change-Id: Ibc7f8387c5b84cb6e1c0e8a69454acb661a4acc4
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Adjust for odlparent 3 / yangtools 2

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

Adjust for odlparent 3 Checkstyle settings

This fixes the enforcing modules, library-impl and southbound-impl,
along with a few other violations in other modules.

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

OVSDB-445 Port creation not reflected in OperDS

When a port is deleted and added in same ovs-vsctl command
it doesn't show up in OperDS. This is because
OvsdbOperationalCommandAggregator always processes removes
after Updates. Since delete is being done for same port right
after update, it ends up not adding it to datastore.

Fix is to check if same port is present in updates or not
before deleting it.

Note: This may impact other such configuration too, though
not sure of any valid use cases for bridges etc.

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

OVSDB-440: Fixes detecting SSL mode with OVSDB nodes

The OF controller in netvirt is set by detecting the mode used in the
OVSDB manager connection.  However, this detection was absent for ssl
and was only working with tcp/ptcp.

Fixes: OVSDB-440
Change-Id: I66907a25e7ce5df14072218d33e0a5995081311a
Signed-off-by: Tim Rozet <trozet@redhat.com>

OVSDB-435: fix transaction leak in BridgeOperationState

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

Fix transaction leak in OvsdbConnectionManager

see https://jira.opendaylight.org/browse/NETVIRT-985

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

Fix transaction leak in BridgeConfigReconciliationTask

see https://jira.opendaylight.org/browse/OVSDB-425

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

Fix possible transaction leak in BridgeOperationalState

see https://jira.opendaylight.org/browse/NETVIRT-985

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

fix intermittently failing unit test

Change-Id: I1445d255f10ca05ed2f1f4d0bf1f563f82164904
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

fix unit tests which fail intermittently

using doReturn when pattern instead of when thenReturn pattern.
mockito does not work properly in multi threaded tests when we use
when thenReturn pattern.

Change-Id: I3671d2201b15d2d2061199249c269ca7d5eeb88f
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

OVSDB-432: Add method to restart OVSDB server and limit retry times and timeout

This patch makes following changes:
1. Add restart method to restart OVSDB server. User wiil be able
   to restart OVSDB server with different sets of SSL protocol
   and cipher suites.
2. Limit retry times for certain SSL handshake status. In current
   code, it retry infinite times for some SSL handshake status
   and cause issue. This patch will fix it.
3. Add API in OvsdbClient interface to provide custom timeout value for
   monitor and cancel monitor transactions. This avoids
   blocking thread due to unsuccessful transaction.
4. Fix typo in Function.java

Change-Id: I4f07e2f32889099e65a06a3afc4a263714dd5946
Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>

Remove deploy-site.xml

With the javadoc jobs now available to generate javadoc it is time to
remove the maven-sites and related files.

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

bug 8712 vlan bindings update fix

when global node and physical switch node are updated in same transaction
from application, vlan bindings are missing.

Make vlan bindings participate in dependency workflow.

Do not create/delete port from controller.
Only update the port from controller.
port creation happens from device always.

Added port reconciliation when port is added later.

Change-Id: Ic86a3545428e186f3aeb26c8a36a783f774ecf53
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 7599 avoid unnecessary mdsal reads

avoid the un necessary reads.
skip monitoring version column

Change-Id: I82e50b973a93c57eb65dd34c6dd624c8e47e2344
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 8029 handle expired in transit entries

when ref count goes to zero , locator is kept in transit state.

if the device still has references to it from other sources like tunnels
the locator will not be deleted from device.

The transactions which wait for this locator will expire.

If the intransit state expires , query the device itself to get its
status and proceed with the transactions which depends on this locator.

added a timer job which triggers every 30sec and checks for expired
intransit keys and clears them.

Change-Id: I0022da4a3a2e748629bc031246f6e6c1509d27a4
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 8029 added ref counts for physical locators.

remote mcast and ucast macs refer to physical locators

whenever a new mac is added or updated, update their refcount for that
tep.

When the ref count of the tep goes to zero , the device may delete that
tep after the current transaction.
In that case mark the locator as in transit.

The next transaction which depends on this locator will not use this
locator uuid and will wait for the locator in transit state to be
resolved in dependency queue.

Eventually controller will get locator delete update command from the
device and proceed the transactions which are waiting for this locator
state to be changed from in transit state.

Added onSuccess and onFailure methods to transaction command.
Clear the locator refs for the succeeded delete mcast macs immediately
upon the success of the device transaction.

Clearing the locator ref count immediately upon success of delete mcast
is needed so that the next transaction will not use the possibly stale locator.

Clear the intransit status of the failed commands upon failure.

Change-Id: I84d64f7deb20f801f14e524f88431807afac3a99
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Bump odlparent 2.0.4 to 2.0.5

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

Remove explicit default super-constructor calls

The default constructor is called by default (hence its name), no need
to call it explicitly

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

BUG 9166 - Fix Netvirt L2GW Illegal state exception

Fix exception and modify logic for comparing OVSDB
Hardware_vtep database version retrived from device
to min version specified.

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

Migrate SchemaService references

SchemaService is deprecated in favor of DOMSchemaService, migrate
references.

Change-Id: I8ba2ff0b4f676fc47e3069a9bdb79078c17fe61f
Signed-off-by: Robert Varga <nite@hq.sk>

Bug 8874 - Tunnel_ips of hardware_vtep is cleared when Open vSwitch process restarted in Open vSwitch HWVTEP Emulator

Issue:
  Tunnel_ips of hardware_vtep is cleared when Open vSwitch process
  restarted in Open vSwitch HWVTEP Emulator

Cause of this issue:
  After Open vSwitch process restarted, ODL attempts to acquire tunnel_ips
  from CONFIGURATION MD-SAL and set it to Physical Switch.
  As tunnel_ips is not set from ODL but is set by vtep-ctl command, it
  does not exist in CONFIGURATION MD-SAL.
  This caused an empty value is set to tunnel_ips for Physical Switch.

Fix:
  Let ODL to acquire tunnel_ips from OPERATIONAL MD-SAL after
  VTEP Emulator restarted.

Change-Id: I439ba08c813ede02f82592ca0051b19140f40db2
Signed-off-by: Ryousuke Masuhara <r-masuhara@zn.nes.jp.nec.com>

Refactor compareDbVersionToMinVersion

compareDbVersionToMinVersion uses ovsdb.library to
compare versions. This has added unnecessary
dependency in downstream projects. Replace usage of
ovsdb.lbrary.Version with String matching so downstream
projects don't need to add library, which is OVSDB internal
feature, as a dependency.

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

BUG 9073 Fix memory leak - close TransactionChain

Part of larger effort related to bug 9060 and
bug 9034 to fix memory leaks.

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