Move ObjectMapper to JsonRpcEndpoint

JSON mapping is an implementation detail of JsonRpcEndpoint, hence
move it to that class, allowing us to shed a field and statically
bind to it.

Change-Id: Ie0a314b3aa6c830d9cc5936eb4ec948823ba351c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 6ea8a455e8f3e1a2a099be256f97257a82dde409)

Improve schemas population

Fetching the schema is an asynchronous process, which means we could
end up with competing requests overwriting the schema. Make sure to
re-check presence when the resolution process completes.

Change-Id: I0a7076b7a476394d4815f3c8f544be7a9ffaddd8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 0faae677b33efe1dd5bec783f8e57c5c0a10cdc4)

Remove use of deprecated Guava methods

Pass down MoreExecutors.directExecutor() to future-proof the code.

Change-Id: Iaf847f2dbcce63bac685d5a22a35fc9b8085d06d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 2630eeeadf852d4327d9990f59e8c87d13c5e30e)

Turn JsonRpcEndpoint into a proper OvsdbRPC implementation

The Netty wiring and invocation handling is quite arcane here, relying
mostly on reflection to perform dispatch.

There is only a single direct user JsonRpcEndpoint, which expects
a OvsdbRPC to be exposed from it. Furthermore it is obvious that
JsonRpcEndpoint has a 1:1 correspondence with JsonRpcServiceBinderHandler
and that the invocation context used to pass request processing relies
on Netty Channel instance -- to which the JsonRpcEndpoint is already
bound.

This integrates JsonRpcServiceBinderHandler into JsonRpcEndpoint,
thus making the fusion obvious, eliminating the need to pass Object
context between the two.

Furthermore we eliminate JsonRpcEndpoint.getClient() and opt to make
JsonRpcEndpoint implement OvsdbRPC, providing a very clear connection
between the two and completely eliminating use of reflection in the
process.

Change-Id: I1cfcf784330fdf6e30a55fcaff6f5bd716d987e3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit fc3bf4c16d4bd71512ca991fcfc95d804c73c05e)

Reuse ObjectMapper across all connections

ObjectMapper is documented to be safe for sharing, make sure we
reuse a single instance across all connections.

Change-Id: Ia3ebbe1eb4772efaa1f68c0b0d4ee721431325b0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 4d598cffc88333c551be88e45cef27f932b800bf)

Use a constant ObjectMapper in UpdateNotificationDeser

Rather than instantiating a fresh mapper for each request,
share a single pre-configured instance.

Change-Id: I231165f82dc950ae9e3adbfb816da123139cc878
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 4402e56b96f9d64e23efec322c42e6d2d7d5f3bb)

Use proper constant in JsonUtils

We are not using the Mapper here at all, just the writer. Make sure
we retain only the writer and make it a proper constant (to allow
JIT to do constant propagation).

Change-Id: I7df2a6234511c9dbf51509c495a59e642173fffe
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 7b32c396b165fbecd707afb0d8753885816eaeed)

Do not reconfigure ObjectMapper in FutureTransformUtils

This is a shared ObjectMapper, which will be reconfigured on first
access -- we can do that upfront and save some cycles from the fast
path.

Change-Id: Ibdf27ff194350b96ee6aadeaa9725d4cfdc6e9db
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit d2410c0a3da59dfa11f03a74bc01f252db4894a5)

Bump odlparent/yangtools/mdsal to 4.0.14/2.1.14/3.0.13

Adopts upstream upgrades and fixes.

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

Bump to odlparent-4.0.13/yangtools-2.1.13/mdsal-3.0.7

This picks up latest upgrades and fixes.

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

Fix checkstyle

This fixes up checkstyle violations.

Change-Id: I7ee4391b58eef76954157deb11b611a3837d0960
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 27d66a4602fca44386e6ab733b3d7748f9120b11)

Do not use Foo.toString() when logging

This was spotted in a JFR run, where JsonRpcEndpoint would consume
a lot of CPU formatting a logging object. Eliminate calls to toString()
in logging arguments, as the framework will call these when really
needed.

Change-Id: Id077c285074c1fd8012539ab7c144ddc4b286376
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 10119331189066c009db4f0000a39ae586cc164c)

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

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

Cleanup OvsdbDataTreeChangeListener

This patch restructures the logic a bit, so that it does not invoke
getData{After,Before}() multiple times, fixing Eclipse warnings and
improving performance.

Furthermore augmentation handling is refactored so that the two
cases (node/bridge) are handled in an if cascade, preserving type
safety.

Finally updateConnections() is refactored to use a switch statement
instead of multiple if conditions, which shows that delete of the
entire node are not handled -- a FIXME is placed there.

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

OVSDB-486 Fix Null Pointer Exception in OVSDB

Fix for Null Pointer Exception

Change-Id: I93227ab66df4529a9b8c127c4b41096335424cc3
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>
(cherry picked from commit cfab9e1aefdd0c7bdb09770309f8ad05c4c2c6ce)

Bump mdsal to 3.0.10

This brings in latest fixes/improvements.

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

Bump mdsal to 3.0.9

This picks up latest fixes and improvements.

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

Bump odlparent to 4.0.11

This adopts latest upgrades.

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

Update docs/ovsdb-user-guide.rst and README

odl-ovsdb-openstack has been moved to netvirt and changed as
odl-netvirt-openstack, so update it to reflect current status.

JIRA: OVSDB-480

Change-Id: Ied1268898d0adc63894250b9733a974140bc31c4
Signed-off-by: Yi Yang <yangyi01@inspur.com>

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

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

Bump mdsal to 3.0.8

This brings in latest fixes and enhancements.

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

Bump odlparent to 4.0.10

This brings in latest updates.

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

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

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

Remove useless tests

These tests are not asserting anything at all, remove them to reduce
static mocking proliferation for no value whatsoever.

Change-Id: I53b21da97d7b9c7cf6e64543e6fe73f2c08267e9
JIRA: OVSDB-476
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 4a6fd54eedf2f7275d43ac4da351b5c3abf8302d)

Update .gitreview to stable/neon

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

Remove duplicate artifact declarations

This fixes a few maven warnings about artifacts being declared
multiple times.

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

utils: migrate to the mdsal DataBroker

This duplicates controller-based classes temporarily, so that we can
migrate hwvtepsouthbound and southbound separately.

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

library: migrate to the mdsal DataBroker

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

Add missing annotation dependencies for Java 11

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

use annotations instead of XML for Blueprint

JIRA: OVSDB-474
Change-Id: Ie40dacf4b4a745d77d83bdc7e1b95fecad4fd276
Signed-off-by: Harshini <hm@luminanetworks.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bump to mdsal 3.0.6

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

Bump mdsal to 3.0.5

This aligns with odlparent-4.0.9 and yangtools-2.1.8.

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

Bump to odlparent 4.0.9

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

Bump mdsal to 3.0.4

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

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

Bump odlparent to 4.0.8

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

Fix controller for ptcp manager

If a ptcp manager is set on OVS, we configure
controller through conneciton info. There are two
issues with this code today:

1. Doesn't work well for IPv6.
2. Doesn't check if connection is to same port
   as configured in ptcp manager.

So we end up with an improper controller configured
basde on ptcp even if we already configured correct
ones from manager entries.

JIRA: OVSDB-475

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

Migrate NumberUtils.isNumber()

This method is deprecated, use its replacement, isCreatable().

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

Remove use StringUtils

Guava provides equivalent functionality, use that instead of
commons-lang3.

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

Bump mdsal to 3.0.3

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

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

Bump to odlparent 4.0.5

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

Use targeted node type for bridge name search

The IdentifiableItem we're looking for when checking for a bridge name
is Node; this changes isBridgeOnOvsdbNode() to look for that
directly. If also changes toString() to a match-specific method to
avoid the temptation to use it for other purposes.

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

reduce object allocation in SouthboundUtils.isBridgeOnOvsdbNode()

By avoiding any unnecessary toString() usage, but unfortunately not
being able to complete avoid it, without further/bigger/riskier changes.

This should help to reduce GC churn.

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

Bump to mdsal-3.0.2

This bumps the dependency to 3.0.2.

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

Fixes overwrite when adding termination point

A race condition was observed where OVSDB had already added the VTEP TP
to br-int, and following this Netvirt ELAN Manager added the patch port
(br-ex-patch) to br-int. Upon ELAN manager adding this using
SouthboundUtils addTerminationPoint, the previous VTEP was deleted. This
change modifies the addTerminationPoint to use a merge in MDSAL rather
than a put.

JIRA: OVSDB-472

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

Log the address and port on bind errors

This ensures the address and port are available in the logs when we
fail to bind.

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

Fix ovsdb compilation

southbound-impl relied on getting its dependencies transitively:
- sal-binding-api via aaa
- mdsal-dom-api via sal-core-api

Fix this up by specifying correct dependencies.

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

Inject the certificate manager in the constructor

This simplifies wiring in other contexts (e.g. with Blueprint
annotations or Guice).

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

Upgrade to Neon base platform

This reverts the PowerMock and Mockito upgrades for the time being.

Change-Id: I46d935fed3ef44dede188ffd25295586010cc8a2
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update docs header to Neon on master

This version name gets rendered into the main header of OVSDB's 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: I82a4bffc06a894bbc643c496c0471f3fe413a280
Jira: DOCS-112
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Close old OVSDB connection when ping fails

When rebooting a compute node, the ovsdb hostconfig is missing
from oper d/s.

The change resolves the issue, by closing the connection if
the ping/echo fails. The ping/echo is triggered when a new
connection comes in.

What was happening before this change, is that the new
connection from the rebooted compute node would be rejected
and closed, because the code saw an existing connection, and
had not yet "closed" the old connection, which would fail later,
about 3 seconds later.

Tested this with queens/oxygen by manually rebooting compute node
(sudo shutdown -r now) a number of times.

Many more details in the JIRA.

JIRA: OVSDB-444

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

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>