Drop Karaf shell specifiers

We no longer need to support Karaf 3, and the karaf.shell
"[3.0.0,4.1)" declarations prevent us from upgrading to Karaf 4.1
(which is coming with odlparent 3).

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

Fix compiler and FindBugs warnings in mdsalutil-api

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

Fix broken build from FindBugs RV_RETURN_VALUE_IGNORED violation in RPC

Impact of https://jira.opendaylight.org/browse/MDSAL-294

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

Log cleanup in InterfaceStateListener

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

Remove usage of runOnlyInOwnerNode for InterfaceConfigListener

runOnlyInLeaderNode queues a DJC job to check for entity owner,
that too with a constant string. If too many interfaces are getting
configured at the same time, this will potentially slow down DJC.

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

Stub syncInstallGroup in TestIMdsalApiManager

To resolve random Elan junit failures

Caused by:
org.opendaylight.yangtools.testutils.mockito.UnstubbedMethodException:
syncInstallGroup(java.math.BigInteger dpId,
org.opendaylight.yang.gen.v1.
urn.opendaylight.group.types.rev131018.groups.Group group, long
delayTime) is not implemented in testIMdsalApiManager

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

Remove unnecessary log in FlowBasedServicesInterfaceStateListener

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

Fix ResourceBatchingManager NPE seen in ElanServiceTest

to avoid this problem which occasionally fails ElanServiceTest:

org.opendaylight.genius.datastoreutils.testutils.infra.AutoCloseableRuntimeException:
Failed to close() class
org.opendaylight.genius.interfacemanager.renderer.ovs.utilities.BatchingUtils
(...)
Caused by: java.lang.NullPointerException
10:59:34  at
org.opendaylight.genius.utils.batching.ResourceBatchingManager.deregisterBatchableResource(ResourceBatchingManager.java:215)

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

DiagStatus integration for IFM and ITM

Adding the implementation for registering the
service and reporting the initial status for
interface-manager and ITM.

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

InterfacemgrProvider ListenableFutures.addErrorLogging()

This is better than the TODO, but worse than a real full async solution.

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

Refactor BatchingUtils/SouthboundUtils to non-static singletons

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

Refactor InterfaceMetaUtils to non-static

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

Follow-up to https://git.opendaylight.org/gerrit/#/c/64492/

Addresses Michael's comments. The *Helper classes are now
singletons and inject all dependent services. In addition,
AlivenessMonitorUtils was made a singleton as well to be
consistent.

Since OvsInterfaceTopologyStateRemoveHelper has only one user
and its lone method is not large, I removed the class and
in-lined the code in InterfaceTopologyStateListener rather than
making it a singleton.

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

GENIUS-95: clarify the NONE documentation

The onFailure warning only applies when NONE is used for failure
logging.

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

FutureRpcResults: prevent duplicate builder calls

This patch makes FutureRpcResultBuilder prevent callers from using
onSuccess(), onFailure() and withRpcErrorMessage() twice.

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

Remove Serializable from instructions, actions, MatchInfo

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

Fix warnings/cleanup in alivenessmonitor

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

IdManager RPCs switched to using FutureRpcResults

This change preserves the logging of idmanager failures exactly as-is.

This first change does intentionally not yet change idmanager's behavior
and should have no impacts to users; it just simplifies the code.  (But
this means that it sort of "abuses" the FutureRpcResults, because it
does not implement real async operations returning real futures, but is
forced to be synchronous.  The idea is to raise a separate follow-up
change after this has gone in, which will make all operations async; and
there defer the sync get to the callers (or not) - after checking that
all of them really do @CheckReturnValue.)

FutureRpcResults now also logs every RPC entry point.  This is at at
level TRACE by default, but can be customized; just like success and
failure log levels can.

FutureRpcResults is also extended with a syntactic sugar short-cut
directly accepting the YANG Builder for Output often found in such RPCs.

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

Fix netvirt CSIT failure

Due to accidental removal of Interface Config Listener registration

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

GENIUS-95: Fix interfacemanager RPC error log

see https://jira.opendaylight.org/browse/GENIUS-95 for more background

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

Remove ClusteringUtils and EntityOwnerUtils

All users have been migrated to EntityOwnershipUtils so these
classes can be removed to eliminate the last references to the
controller EOS APIs.

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

Fix the listeners registration process

It is a bad practice to pass the 'this' out of a constructor, specially
in multi-threading environments, since the object is not yet fully
constructed. To fix that, we have added a register() method that will be
invoked by BluePrint just after the constructor; both the register() and
close() methods need to be declared as 'public' and not 'final',
otherwise BluePrint will not be able to call them.

The shutdown() method has been removed in favour of a simple close().

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

Refactor InterfaceManagerCommonUtils to non-static

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

Cleanup in idmanager-impl

- Change public Map fields, allocatedIdMap and releaseIdLatchMap, in
IdUtils to private and added methods to access them.

- In ReleasedIdHolder, delayedEntries is expected to be thread-safe and
is initialized to a CopyOnWriteArrayList however it can be set externally
via setDelayedEntries with no knowledge if the passed List is thread-safe.
It turns out the 3 callers do pass in a CopyOnWriteArrayList however it
shouldn't rely on callers doing the right thing so I changed it to create
a new CopyOnWriteArrayList from the past List to guarantee it. This also
ensures delayedEntries is never null.

Change-Id: I792012ecb218609fb40e1f9efb78dc9d077d2d05
Signed-off-by: Tom Pantelis <tompantelis@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: I7761c1efaf216cacf87d909f9a38ec2b265fda66
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

FutureRpcResults with StackTraces.getCallersCallerMethodName()

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

Handle logging symmetrically in FutureRpcResults

This patch treats logging and result (or error) processing
independently, in a symmetrical fashion for both success and error
cases.

* LogLevel handles the logging façade in front of Logger.
* A new NONE LogLevel allows logging (by the default handler) to be
  disabled.
* The success log level is configurable (including NONE).
* The default consumers do nothing (logging is handled separately).
* The onSuccess/onFailure handlers follow the same pattern, and the
  onFailure handler is re-used for external error handling.

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

Refactor FlowBasedIngress/Egress classes to non-static

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

Refactor AlivenessMonitorUtils to non-static

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

Convert to JobCoordinator in interfacemanager

Also cleaned up some statics in the process.

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

Convert to JobCoordinator in idmanager-impl

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

INFRAUTILS-18: TestableJobCoordinatorEventsWaiter 2 min. instead 30s

and include details of JobCoordinator state when stuck

Requires Iac27dbdd707a599fcb912bb68425e6cbe969d0e3 c/64806 in infrautils
for the new JobCoordinator toString() implementations.

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

Fix the listeners closing process

In some cases, dataChangeListenerRegistration might be null, provoking
an NPE. This fixes that problem.

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

testutils InterfaceHelper with correct @NonNull read

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

SRM: Add implementation code

This adds the following:
1. RPC Implementation in impl and required util class
2. CLI for SRM as mentioned in spec.
3. Debug CLI to clear out SRM operDS.

These cover Work Items 3 and 4 of SRM Spec.

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

Make @Deprecated DataStoreJobCoordinator delegate to infrautils'

In an ideal world, we would instead now switch all uses of the
@Deprecated DataStoreJobCoordinator everywhere in genius and netvirt to
the new infrautils' JobCoordinator @OsgiService instead of having to do
this, but I would propose to do it in 2 steps and get this in first
(now), and then starting migrating, and then once everyone is off
DataStoreJobCoordinator and there are no usages left, delete it.

At least like this we would have only one version of this code, instead
of maintaining 2 split incarnations in genius and infrautils.

Also, and importantly, this change includes new infrastructure making it
easy to use JobCoordinator in the netvirt aclservice and genius
interfacemanager component tests, as used in IdManagerTest, see also
TestableJobCoordinatorEventsWaiterTest.

This change must be merged into genius at the same time as
I0f2c5258315f40f9db88806fe0a33619f0f7bb77 will go into netvirt.

This change requires I5e423212e8f720fb4b20837f12ad59de6f2a66d4 in
infrautils.

This change includes the fix for bug GENIUS-93.

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

JavaDoc for AbstractAsyncDataTreeChangeListener

Change-Id: Ie8acafc15f280c9a7e08c13a6dcbcb5d2ee0bdf5
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Remove IdManager from ItmManagerRpcService & ItmUtils as unused there

NB however that it *IS* used in some other ITM classes, incl. e.g.
ItmProvider, TransportZoneListener, ItmTepRemoveWorker, ItmTepAddWorker.

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

Add component-based removeFlow()

... which will be used in NetVirt to simplify flow removal methods.

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

Interface manager: split makeTunnelIngressFlow

Having separate add/remove methods allows the removal path to be
simplified, and avoids having an obscure “addOrRemove” parameter.

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

Aliveness monitor: manage transactions

Use the transaction manager to handle all write transactions.

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

Enable bound services update

An update for bound services was ignored by the Bound Services Listener.
It has to be allowed for ACL Service in order fix breakage in
communication between two tenant networks using default security groups
(stateful mode).

Change-Id: Ib05a8fcfac8e74fa5a925edd0db49b7254bbed6a
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fix broken netvirt build following TestInterfaceManager change

Several tests in netvirt (in both aclservice and elanmanager) actually
use TestInterfaceManager without a DataBroker... so the netvirt build
broke following merge of I873999721e135c3cc968ef0d1aa239d794d12cb3.

My first attempt in I31ab80c583c2f1da73edc5b42cd7419311897bae was to
"just" fix those tests, but after 1h into that I'm giving up (and will
let Riyazahmed Talikoti finish it up), and instead put together this
workaround, which basically lets one still use TestInterfaceManager
without a DataBroker, like before, for now.

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

RetryingManagedNewTransactionRunner for OptimisticLockFailedException

TODO in xN future follow-up changes: Start fixing (many)
OptimisticLockFailedException that appear in logs, by using this... ;)

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

DataBrokerFailures failSubmits with int howManyTimes

This is particularly useful e.g. to test OptimisticLockFailedException,
and will be used in I8fee2b50b562dc2c0f416d59082ad4bded0cdf58.

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

TestStubs for Elan-Evpn Junit

Teststubs are written  for InterfaceManager and ITM
Right now these stubs contains testcode mainly required by Elan. These can
be expanded as we write more testcases.

Added addInterfaceInfo and addTunnelInterfaceInfo methods.

Elan Junit Testcases are written and raised as seperate review in netvirt
https://git.opendaylight.org/gerrit/#/c/62894/

Change-Id: I873999721e135c3cc968ef0d1aa239d794d12cb3
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>

Use JobCoordinator in TestableJobCoordinatorEventsWaiter

Added support for JobCoordinator in TestableJobCoordinatorEventsWaiter.
Support for DataStoreJobCoordinator was retained for now for
compatibility.

The other alternative is to move TestableJobCoordinatorEventsWaiter et al
to infrautils, see https://git.opendaylight.org/gerrit/#/c/64468/, but
it got -1. Maybe we can revisit that later.

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

ManagedNewTransactionRunnerImplTest with new negative DB failure tests

These 2 new tests will be customized in
I8fee2b50b562dc2c0f416d59082ad4bded0cdf58 to illustrate how the
RetryingManagedNewTransactionRunner, contrary to the
ManagedNewTransactionRunnerImpl, handles the
OptimisticLockFailedException.

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

Convert interfacemanager-impl to use mdsal EOS APIs

The controller EOS APIs are deprecated.

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

Add access to the ExecutorService

Add accessor to the ExecutorService so that the derived listeners can
shutdown it.

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

Remove duplicate (identical) maven-jar-plugin to fix Maven WARNING

[WARNING] Some problems were encountered while building the effective
model for org.opendaylight.genius:mdsalutil-api:bundle:0.4.0-SNAPSHOT
[WARNING] 'build.plugins.plugin.(groupId:artifactId)' must be unique but
found duplicate declaration of plugin
org.apache.maven.plugins:maven-jar-plugin @ line 164, column 15
[WARNING]
[WARNING] It is highly recommended to fix these problems because they
threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support
building such malformed projects.

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

Convert fcapsapplication-impl to use mdsal EOS APIs

The controller EOS APIs are deprecated.

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

Interface Manager: use ManagedNewTransactionRunner

This is a first stab at migrating WriteTransactions to
ManagedNewTransactionRunner (including one ReadWriteTransaction). It
illustrates a variety of different approaches.

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

Add new EntityOwnershipUtils

Added EntityOwnershipUtils that uses the mdsal EOS APIs instead of
the deprecated controller APIs. Deprecated the prior Utils classes -
they will be removed once all users are migrated.

Modified DataStoreJobCoordinator to implement the new JobCoordinator
interface to avoid propagating the further use of a deprecated API
in EntityOwnershipUtils.

EntityOwnerUtils#registerEntityCandidateForOwnerShip has the side
effect of registering a global EntityOwnershipListener that maintains
local ownership state for all entities in a static cache. This is
unnecessary as the state is maintained by he EOS and can be queried.
It also stores change events in a static (unsynchronized) EVENTS_HISTORY
list which grows unbounded. This is only used in one place to print the
history via a CLI command. Both these side effects are problematic
and I don't think need/should be propagated forward. In addition,
it ignores the registration instance that is returned from the EOS
which should be closed by the caller - and the same with the
optional listener that is passed in..

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

Replace deprecated Futures.addCallback by the newer version

The method addCallback(ListenableFuture<V> future,
FutureCallback<? super V> callback) is deprecated and will be replaced
by a new its new version addCallback(ListenableFuture<V> future,
FutureCallback<? super V> callback, Executor executor) in April 2018.

Some other minor changes to pass the new version of checkstyle.

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

This patch implements the genius mdsal interface for
supporting conntrack ct_mark match (with mask) and
action (without mask).

Change-Id: Iedb16c3751f2f6b4500d921d9ca9e2cbab90ac79
Signed-off-by: Bertrand <bertrand.low@hcl.com>

Replacing DS read with cache read in ShowVlan CLI

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

Migrate InterfaceChildInfoListener listener to use the new Listeners FW

Migrate the InterfaceChildInfoListener listener to use the new Listeners
framework available since [1].

[1] https://git.opendaylight.org/gerrit/#/c/59177/

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

Cleanup

- Declare and rename itmCache as constant
- Other minor changes

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

Bug 9238: DataStoreJobCoordinator now logs JobEntry before thread death

This is just the "port" of
https://git.opendaylight.org/gerrit/#/c/63965/ from infrautils to
genius.

This is the last time I maintain this code in both places!
https://git.opendaylight.org/gerrit/#/q/topic:genius.DJC-to-infra.JC
coming real soon, now.

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

Use Executors.newSingleThreadExecutor in AsyncDataTreeChangeListenerBase

as well as the same in AsyncClusteredDataTreeChangeListenerBase

instead of their current use of ThreadPoolExecutor, which [AFAIK...] is
pointless here now, since we switched it (a long time ago) to always be
of fixed size 1 only, for some reason?

This is JUST a clean-up, to make this class more easy to understand.
This should (AFAIK) NOT change anything in its behaviour, so safe.

This is also to align it with e.g.
https://git.opendaylight.org/gerrit/#/c/63505/ (and perhaps more changed
like that to come) - seeing the exact same ExecutorService used in both
IMHO makes it clearer how similar this old and that new async DTCL are.

NB: This is Executors from infrautils not java.util.concurrent, but
that's just a wrapper that deals with the setUncaughtExceptionHandler
etc.

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

Add ReadWriteTransaction support

This allows ItmTunnelAggregationHelper’s worker to be fully converted
to using the managed transaction runner.

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

Fix currently broken master build

This was to due to just merged c/63372 VS c/63120:

This unfortunately just happened (sorry!) because of
https://git.opendaylight.org/gerrit/#/c/63120/7/mdsalutil/mdsalutil-testutils/pom.xml
VS
https://git.opendaylight.org/gerrit/#/c/63372/8/mdsalutil/mdsalutil-testutils/pom.xml

which wanted conflicting things (without cause conflicts in the git
sense).  What we want is <scope>compile, so this removes the
<scope>test - which were duplicates!

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

ManagedTransactionRunner utility to help close transactions

inspired by
https://git.opendaylight.org/gerrit/#/q/topic:transaction-helper, but:

  (1) instead of the cancel-if-lambda-didn't-submit design, forbid
lambda from doing submit() or cancel(), and instead let new transaction
utility do that.  This is better because it already opens the door for
non-single transactions in the future (see upcoming next chained change)

  (2) allow lambda to throw checked exceptions (see CheckedConsumer),
which will, obviously, lead to the Transaction to be canceled. This is
important as it helps avoid errors where using code forget this.

TODO: Make all places which topic:transaction-helper converted to using
TransactionHelper use this change's ManagedNewTransactionRunner instead.

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

Bug 8059: Fix lockmanager-impl OptimisticLockFailedExceptions logs

with true Test-Driven-Development: first simulated the problem using the
brand new shiny DataBrokerFailures' failSubmits() which together with
the infrautils LogCaptureRule caused LockManagerTest's new
testOptimisticLockFailedException() to fail due to (unwanted) ERROR logs
related to the OptimisticLockFailedExceptions, and then fixed it
properly.

This is an alternative / counter-proposal to (parts of)
https://git.opendaylight.org/gerrit/#/c/61526/

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

DataBrokerFailures with failSubmits(TransactionCommitFailedException e)

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

AbstractDataTreeChangeListener getDataBroker protected instead of public

The getDataBroker() method presumably is destined for use by subclasses?

Other classes should not use that to get a DataBroker, they should have
one injected for themselves.  (This is a general DI principle - like
"don't get services you require from other services" kind of thing.)

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

Clean up

- Remove some dead code
- Simplify some conditions
- Private constructor and final modifier for utility class
- Remove deprecated CheckedFuture
- Return results immediately without local assignment
- Strength access to methods & variables
- Other minor improvements and formatting

Other changes will follow this one.

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

Drop yangtools.version completely

as well as mdsal.version & mdsal.model.version

because it turns out that all this is already available
via BOM/s inherited from the binding-parent, which is the
parent of genius/commons/binding-parent, anyway.

ACK: This is based on an idea raise on IRC by shague!

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

Bump genius to use yangtools 1.2.0

Jira: releng-485
Change-Id: Ifb9cbc5c58d18484ae3aee98936d06dec3aa6750
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Bump odlparent 2.0.4 to 2.0.5

Change-Id: Ib8a87ed7dc7550082fde3abab9891b2ee105eb1f
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: Id5153bce435a6f8d144d30d66a14435e26b53506
Signed-off-by: Stephen Kitt <skitt@redhat.com>

FutureRpcResults

This is https://git.opendaylight.org/gerrit/#/c/53998/ generalized!

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

Add @PreDestroy (and final) to AbstractDataTreeChangeListener's close

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

Add gpe option to itm-config.xml

https://git.opendaylight.org/gerrit/60769 missed adding
the option into the config.xml.

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

Improve the Genius Listeners Framework

Improve the Listeners Framework created in [1] to ease registration
and un-registration processes for entities.

[1] https://git.opendaylight.org/gerrit/#/c/59177/

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

Remove un-used InterfaceManagerRpcService IMdsalApiManager & methods

* buildInstructions()
* makeTerminatingServiceFlow()
* makeLFIBFlow()
* getFlowRef()

and instance field IMdsalApiManager mdsalMgr.

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

Enforce findbugs maven plugin for module arputil-api

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

Lower log level for non errors

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

Mechanical: ItmTunnelAggregationHelper Eclipse Auto-Save clean-up

just so that this won't clutter
https://git.opendaylight.org/gerrit/#/c/63372/5/itm/itm-impl/src/main/java/org/opendaylight/genius/itm/confighelpers/ItmTunnelAggregationHelper.java@306

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

Fix problems detected by the FindBugs plugin

- FindBugs enforcement.

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

InterfaceManagerRpcService include exception cause in RPC result

and LOG.error SOME (but not all) of such problems (logging is as before)

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

Add a basic framework to create DataTreeChangeListeners

Add a basic framework to create DataTreeChangeListeners.

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

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>