Fix ITM UT failures

Yangtools has changed how IPAddresses are converted
to string. ITM code is using different methods to
convert String IPs to IpAddress. Fix is to use
IpAddressBuilder, which is v4/v6 anogstic, at all
places.

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

Use Map::forEach instead of streaming maps

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

add real Lifecycle management to DataObjectCache

by throwing an IllegalStateException if you get() or getAllPresent()
after close(), to prevent it from being used after it's closed, as was
attempted in Change-Id: I8d5f15757a805798f16b98b79b3ad03899b1130d (which
would lead to quite weird behaviour)

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

minor JavaDoc fix to ManagedNewTransactionRunner

It should not @link to a (now) only package local class.

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

Service-Recovery: support for elan-interface entity type
instead of elan-instance entity type

Change-Id: I723bd2ac707bf8b34d869b9b7abfdea4b616c47a
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Remove redundant type information

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

Clean up lambdas

Use method references and lambda expressions where appropriate.

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

Service Recovery Manager for ITM

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

Caches for newly introduced immutables in ITM

Change-Id: I8d5f15757a805798f16b98b79b3ad03899b1130d
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

enforce correct logging via findbugs-slf4j (GENIUS-118)

Modified genius/commons/binding-parent/pom.xml
to enable findbugs-slf4j enforcement in binding-parent.

This ensures we won’t have any regressions in log handling in modules
based on binding-parent.

Change-Id: I7030f32c3c5f16afb6ed7cd5e8c8ff96ca4865a2
Signed-off-by: Saurav Malani <sauravmalani1@gmail.com>
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix ItmAlarm Exception in logs

This is a regression caused by :

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

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

Fix logging issues in ITM

Change-Id: Ie2d697ce8026f3b75022ac7150457dde9f486582
Signed-off-by: sathwik boggarapu <b.sathwik@ericsson.com>

Fix logging issues in InterfaceManager

Change-Id: Ib77840b84f6190be7537c4a6f6f32d317cf89414
Signed-off-by: sathwik boggarapu <b.sathwik@ericsson.com>

Migrate ItmTunnelEventListener to use new listeners fw

Some additional changes:

- Make some variables final
- Remove dead code
- Make some methods private

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

GENIUS-102: interface manager: use transaction manager

Use the transaction manager to manage all write transactions.

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

Use List::sort instead of Collections::sort

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

Extend DataObjectCache to allow for custom key

Users want to be able to provide a custom key derived from the
DataObject instead of keying by InstanceIdentifier.

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

Fix abstract listeners

The deprecated method are now properly implemented in the base class

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

Fix logging issues in alivenessmonitor

Change-Id: I730efda2afc6ffbeed547737e47c980ae418ccd5
Signed-off-by: sathwik boggarapu <b.sathwik@ericsson.com>

Make recoverableListenerMap maintain a queue of recoverableListeners.

This was a review comment from Michael which was not addressed on the
original patch

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

Fix logging issues in SRM

Change-Id: I89a5ff35903202d549a91a510f323856d78b1542
Signed-off-by: sathwik boggarapu <b.sathwik@ericsson.com>

Fix logging issues in fcapsapplication

Change-Id: I4aa3358a1c4774f6eea83b7c8b7d2805d5565f5b
Signed-off-by: sathwik boggarapu <b.sathwik@ericsson.com>

Fixed issue when null is passed in CLI cmds

Due to the karaf migration, the tep CLIs were not working as expected when
null is passed as arguments in CLI cmds.

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

fix logging issues in mdsalutil

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

Add InstanceIdentifier to method add/update/remove in Genius' listeners

Some use cases require the Genius' listeners to provide the
InstanceIdentifier as a parameter in add, update and remove methods.

The change is made on the DataTreeChangeListenerActions interface by
making the new methods the default ones. The current methods are
deprecated and they will be removed once all the current implementations
are converted.

The AbstractDataTreeChangeListener will implement the old methods until
the migration to the new ones finishes.

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

Re-enable bindings of interfacemanager junits

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

Remove unnecessary boxing/unboxing

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

Revert "GENIUS-116: Efficient Handling of DPN disconnect"

This reverts commit 3cf5a4776c2b8dcd77049ecb3ba4e1b28bc1aa4c.

Change-Id: I7a377a9b4cd74fe9052843882e7b730a63cb0036
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

simplify TestableJobCoordinatorEventsWaiter

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

LockManager test failure

Looks like underlying error message has changed. After changing the
expected error message accordingly, test is passing.

Change-Id: I46829aa23c353c3c7a7de9968d54462f8d0ba7ba
Signed-off-by: Ankit <ankit.j.jain@ericsson.com>

fix ItmManagerRpcServiceTest ExternalTunnelBuilder destinationDevice

but ... the fact that this destinationDevice is a "String typed" instead
of "strongly typed" (a real YANG model) seems really wrong, to me.

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

Updating rst doc, how to run CSIT

Change-Id: I285bd34e687945801f1b1443d523a6c3b3908915
Signed-off-by: sathwik boggarapu <b.sathwik@ericsson.com>

JobCoordinatorEventsWaiter to wait for specific job count completion

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

Immutables for performance enhancements

Change-Id: I150d8edc58ad91724679dba15a1f269bd9e2e674
Depends-on: I250b75fea3ae29e8adb351693e7eeb0a596e18fe
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>

Add InstanceIdDataObjectCache

DataObjectCache will be extended to allow for custom key so
add implementation that keys by InstanceIdentifier.

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

Bug: GENIUS-115 OVS Nodes Connect Quickly, Not All Get TransportZone TEPs

1) In ItemTepAddWorker do NOT use cached DPNTEPsInfos, read from DS instead
2) In ItmInternalTunnelAddWorker do not queue the new DPNTEPsInfos via
ITMBatchingUtils, instead write synchronously and unbatched in that thread

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

Issues with SRM service recovery

service recovery logic was doing a close of the already registred
listeners. However the close method of the AsyncListener base classes
were shutting down the dataTreeChangeHandlerExecutor as well, which
will cause issues when we again register the listeners.

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

ifm changes for itm performance enhancements

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

add JavaDoc to DataTreeEventCallbackRegistrar onAddOrUpdate

in order to clarify null-ness expectation of the used BiFunction

also an appropriate @Nullable (has to be org.eclipse.jdt.annotation,
cannot be javax.annotation, because that is not a type annotation
suitable for annotating generics)

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

Add HwvtepNodeHACache interface

This is the first in a series of patches to eliminate the
static HwvtepHACache. Added an HwvtepNodeHACache interface
which HwvtepHACache temporarily implements until netvirt
is migrated to use HwvtepNodeHACache in the next patch.
The static HwvtepHACache instance is also temporarily
advertised as the HwvtepNodeHACache service.

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

Minor simplification in DataTreeEventCallbackRegistrarTest

the point of the testAddOrUpdateAdd(), contrary to the
testAddOrUpdateUpdate(), is that the listener will only get call once
(for the add); this change makes that more obvious when reading the
test.  It still passes.

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

Fix ActionNxConntrack.NxNat problems with xtendbeans (with a test)

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

Fix ActionNxConntrack.NxCtMark problem with xtendbeans (with a test)

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

Flourine idmanager build failures

sal-common-testutil has been removed from controller,
as the relevant modules have moved to mdsal

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

Remove sal-common-testutil dependency

This was previously used for EOS Tests, but since eos
has moved to mdsal repo, this is no longer needed.

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

Service-Recovery: Support needed for ACL feature introduced in SRM
framework.

(a) srm-types.yang updated with 'netvirt-acl', 'netvirt-acl-interface' and
'netvirt-acl-instance' definitions to support ACL service and instance
recovery operations. 'netvirt-acl-interface' and
'netvirt-acl-instance' are the two instance recovery options supported.
(b) SrmRpcUtils.java and SrmCliUtils.java updated to add required ACL definitions
to the internal maps maintained to support recovery operations.

Change-Id: I4c91cd52e7595b92b494356dfa715dbdf15da3c5
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Genius Specific Changes to support QoS with SRM

Adding QOS option as recoverable service & Qos policy as recoverable
instance.

Change-Id: I0e123bec31f663d8b6673b37c5858fbcc85b8b46
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>

Enable findbugs in commons binding-parent

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

Fix findbugs violations in fcapsapplication

- Inconsistent synchronization
- Write to static field from instance method

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

Fix findbugs violations in resourcemanager-impl

- Method ignores return value
- Incorrect lazy initialization of static field
- Dead store to local variable

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

Fix findbugs violations in itm-impl

- Method ignores return value
- Dead store of class literal
- Null pointer dereference
- No relationship between generic parameter and method argument
- Invocation of toString on an array
- Reliance on default encoding
- Field should be package protected
- Inconsistent synchronization
- Inefficient use of keySet iterator instead of entrySet iterator
- Method checks to see if result of String.indexOf is positive
- Dead store to local variable
- Write to static field from instance method
- Load of known null value
- Class implements same interface as superclass
- Redundant nullcheck of value known to be non-null
- Exception is caught when Exception is not thrown
- Useless object created
- Useless control flow

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

SRM refactoring

Realized that some code that went in interface-manager for service
recovery can be reused across modules, and hence moving some of those
common utilities to srm.

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

GENIUS-116: Efficient Handling of DPN disconnect

Dependent on OF plugin patch https://git.opendaylight.org/gerrit/#/c/63261/

The changes in openflowplugin includes, port removal event will come with proper
reason flag set which will enable interface-manager to distinguish between a port
removal and a "port removal due to node removal" from the info in
the incoming event itself.

Change-Id: I39c5cdc6f57052521cb0061ca97ab7518637a5fe
Signed-off-by: SujaT <suja.t@ericsson.com>

Fix findbugs violations in srm-shell

- Format string should use %n rather than \n
- Consider using Locale parameterized version of invoked method

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

DataTreeEventCallbackRegistrar: "add or update" param order

Since functions receiving add or update events need to
handle updates they have two params. As such, in event
of an add the created object should be the second
param passed to the function.

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

add new DataTreeEventCallbackRegistrarTest

this requires infrautils' I0d54b187fd58c6456708518a3886c3aedff3f86a

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

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

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

Fix for GENIUS-112.

This is a race condition, proper fix should remove Batching manager from add callback.

Change-Id: Ia9de21864e82b2c8ded1ea121de615cafa6dab19
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>

Update genius ITM Tunnel auto config spec

The TEP-IP is now obtained from other_config:local_ip,
so update the spec to reflect that change.

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

Revert "ifm changes for performance enhancements"

This reverts commit 51d5fab28b9fbd2c165bd0c803ad768441385d0f as discussed on the TSC mailing list: https://lists.opendaylight.org/pipermail/tsc/2018-February/008919.html

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

ifm changes for performance enhancements

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

NETVIRT-1068: Upstreaming fixes 3

When ARP response is received for the learnt IPs like MIP, its not
processed due to wrong check of table-id. Hence this check is removed. One
more bug was SrcIp and DestIP had junk chars which led formation of wrong
MonitoringKey. Due to this , MIP IP Arp Response was not processed.

Change-Id: If25bbc6fdc264ac866b5cd89c247c487d9f6ca6b
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>

NETVIRT-1009 - Enable genius auto-tz

https://jira.opendaylight.org/browse/NETVIRT-1009

Enable genius auto tunnels. There will be a separate
patch to disable netvirt dynamic tunnels.

https://git.opendaylight.org/gerrit/#/c/66109/1

This will be the default starting in oxygen.
Carbon and nitrogen will continue to have netvirt
dynamic tunnels as the default.

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

InterfaceManager Service Recovery

Adding recovery support for interface-manager.
Two types of recovery are supported:

1. Service Level Recovery - This triggers deregistration and
registration of all external facing listeners

2. Interface Instance Level Recovery - This triggeres a delete and
create of a particular interface configuration so that the state can
be fully reconstructed

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

Remove FlowBasedServicesInterfaceStateListener

Now that all applications are properly calling unbind-service,
the interfacestatelistener which by itself tries to remove
dispatcher table entries can be removed for a cleaner implementation

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

Migrating IFM PM counters to infrautils counters

Existing IFM PM counters were using JMX MBeans,
now new counters are written for IFM PM counters using
infrautils metrics counter APIs which will be common library
to populate JMX MBeans for these counters by leveraging
Codahale framework.

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

ItmRpcTestImpl throw UnsupportedOperationException instead return null

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

Fix more build failure due to openflowplugin lldp attribute name changes

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

Migrate VtepConfigSchemaListener to use new listeners fw

Additional changes:

- Remove useless JavaDoc
- Remove IllegalCatchs

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

Migrating FCAPSApplication PM Counters to Infrautils Counters

Change-Id: I2a4edeeb9deb72de15b8e4b75adf225843c4ddfb
Signed-off-by: Ankit <ank123it@gmail.com>

Migrate HwVtepTunnelsStateHandler to use new listeners fw

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

Build failures due to openflowplugin lldp attribute name changes

Change-Id: I9b06553ce1a0fcae7266492ea8fe176072d87874
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

use feature for inter-project dependency

Use feature to express inter-project dependency correctly instead of
bundle.

For details, refer to
https://lists.opendaylight.org/pipermail/release/2018-January/013604.html
https://lists.opendaylight.org/pipermail/release/2017-September/012356.html

This depends on below patches to export necessary bundle as feature.
https://git.opendaylight.org/gerrit/#/c/68214/
https://git.opendaylight.org/gerrit/#/c/68233/

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

NETVIRT-1066: Scale 150 DPN

Iterating through non-L2VLAN interfaces in L3VPN Tunnel
Interface State listener is unnecessary
For scaling to 150DPN , each TunnelIntfState add event causes a read of
list of interfaces on the given DPN.This list includes even non L2Vlan
type interfaces like tunnel-port etc. By ignoring these ports , read of
operational vpn-interface is avoided.

Change-Id: I3e658eb8eb78d39dc9ed0a7fc1750ce55ae6a7ec
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

remove wrong @Inject from DataStoreMetrics (and very minor clean up)

because instances of DataStoreMetrics are created explicitly via new by
the listeners in package org.opendaylight.genius.datastoreutils, but
never by Blueprint XML or annotations, the @Inject here is not needed,
and could be confusing to humans and tools in the future (and we could
never @Inject Class<?> anyway).

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

GENIUS-109: write interface state correctly

we need to provide the mandatory statistics.

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

added data change listener counters

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

Change log level from info to debug

Change-Id: I3e7fb7e274fe0cdb65aa06e31e3be6d97382a13a
Signed-off-by: Ankit <ankit.j.jain@ericsson.com>

LockManager lock() no longer eternally retries in case of errors

it should only retry if there is an OptimisticLockFailedException,
otherwise it should return a Future failed RpcResult.

this CHANGES the behavior of lock() to return a FAILED RpcResult in case
of a TransactionCommitFailedException caused by a AskTimeOutException
(before that, and other errors, returned a SUCCESS RpcResult); which, to
me, seems the Right Thing to do.

includes proper test driven reproducer for non-regression

much more work remains TBD in LockManager, see TODO in LockManagerTest;
anyone interested in taking this up?  Thank you!

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

Handle br-int dpid update for genius auto tunnels

In OOO deployments, genius auto tunnels does not work,
and vxlan tunnels are not created.

When br-int is added by netvirt, the ovsdb node add
comes in for br-int, without a dpid. The code bails
if br-int does not have a dpid (tep not added).
Then, a second notification comes in, as an ovsdbNodeUpdate,
with the dpid for the node (br-int). But, the current code
does not handle br-int updates, and the tep is never added.

This code change checks for dpid being added to br-int
in the update path, and triggers tep to be added when
this condition occurs.

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

GENIUS-104 Genius auto-tz: use local_ip for TEP IP

Currently, Genius auto tunnels is configured to read tep-ip from OVS
external_ids tuple in the Open_vSwitch table.  Details can be found here:

http://docs.opendaylight.org/en/stable-nitrogen/submodules/genius/docs/specs/itm-tunnel-auto-config.html

This change will be to use the existing, supported field local_ip,
from OVS other_config tuple in the Open_vSwitch table.

The main reasons for this change are:

1. Avoid duplicate data. Instead of introducing a new field, tep-ip, for
source VTEP IP, reusing existing local_ip.

2. Devstack and other installers already support local_ip. By reusing
this field, there should be no changes required in any of the
installers (devstack, OOO, etc).

So, going forward, local_ip will be used as the source IP for the VTEP,
instead of tep-ip.

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

IFM stats params configured by Blueprint

In interfacemanager-impl, new config file genius-ifm-config.xml is added
and corresponding yang for it.

It contains following params:
- ifm-stats-poll-enabled
- ifm-stats-def-poll-interval

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

Scale-In Scale-Out RPC call to get DPN info

Below changes exposes RPC call to get DPN details (DPN ID and Tep Ip of
the DPN) by passing the compute node.

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

minor fix InterfaceStateHelper import statement order

I'm not sure why Checkstyle in mvn hasn't picked up on this?!

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

Changed RPC to retrieve Vxlan DC GWY Ext. Tunnels

Changed the ITM RPC to look for external tunnels towards DC gateway based on
the tunnel type provided as input and removed the hard-coded MPLS Over GRE
tunnel type.
Fixed merge conflicts

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

Enforce deferred recycling of IDs by IDManager

Cause - IDManager allocates released ids first if ids are
avaliable in Released Ids holder. Hence, frequent recycling of ids by
IDManager is resulting in dataplane failures.

Following are changes done as part of this fix:

i) Allocate id from child pools available holder. If that gets exhausted, allocate id from released ids holder.
ii) If child pool is exhausted then get IdBlock from parent pools available ids holder. If that also gets exhaused, get IdBlock from parent pools released ids holder.

Change-Id: Ia8648d599c70480ebce15f05cbe85d13d841ba1d
Signed-off-by: Ankit <ankit.j.jain@ericsson.com>

remove IT, because SFT does that now

and InterfaceManagerIT really had nothing about InterfaceManager anyway

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

refactor ResourceManagerTest into correct .genius. sub-package

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

Migrate FlowBasedServicesNodeStateListener to use new listeners fw

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

Test Plan for ITM Scale Improvements

Find Spec for newly added feature ITM Scale Improvements here
https://git.opendaylight.org/gerrit/#/c/65809/

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

Add MetricProvider to JobCoordinatorTestModule

This requires Ie90562c2f8205bab8817d370d077def9a3419d52 and
I530d20946a80933b05371c5220def447be1ee5e6 in infrautils in order to be
able to compile, and must be merged together (with the 2nd, the first
can go in before).

NB: We cannot mock(MetricProvider.class, RETURNS_DEEP_STUBS), because
the TestableJobCoordinatorEventsWaiter relies on JobCoordinatorMonitor
and assumes a working getIncompleteTaskCount... which the mocked one
would not be.  So we use the (new) "real" TestMetricProviderImpl.

Change-Id: I80e3f5b6e8a601345cd2fbae79f0de22d0fa0e7d
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 8400 - ACL changes doesn't affect the
existing connections

Problem:
    Change in SG rules not affecting
the existing communications

Fix provided:
    When we delete the SG rule, added
a flow to reset the communication.

Change-Id: I40bb9c93b11b785e153e5751f083b628cec4b000
Signed-off-by: vinothb <vinothb@hcl.com>

Yang Definitions for Itm Scale Improvements

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

GENIUS-105 IdManagerTest testMultithreadedIdAllocationForSameKeyFromAvailableIds is brittle

Non thread safe idSet is used inside executor which is resulting in testMultithreadedIdAllocationForSameKeyFromAvailableIds
and testMultithreadedIdAllocationForSameKeyFromReleasedIds tests failures.

Making idSet thread safe fixes both the tests.

Change-Id: Ia90456793b46ec8924f6f831bf3553319387776f
Signed-off-by: Ankit <ankit.j.jain@ericsson.com>

Migrate HwVTEPConfigListener to use new listeners fw

Additional changes:

- Replace Callable objects by lambdas
- Swap order of helper's parameter to be consistent with the rest

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

OPNFLWPLUG-972 : Point to openflowpluign liblldp

Point to liblldp libraries from openflowplugin instead
of controller project (which will be removed soon)

Change-Id: Ie4feb5a9bb5d5552195f61356f9b27ec203d446d
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Migrate VlanMemberConfigListener to use new listeners fw

Addtional changes to simplify it:

- Remove unused parameters
- Replace Callables by lambdas

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

Replace DataStoreCache with DataObjectCache in itm-impl

There were 4 different caches to convert. 2 of them,
TunnelMonitorParams and TunnelMonitorInterval, are singleton
DataObjects use for global tunnel monitorng configuration.
These seem to be condidates for clustered-app-config however
dynamic updates would restrart the BP container but it doesn't
appear itm-impl is ready for that. So to preserve existing
behavior, they're DataObjectCache instances local to a new
TunnelMonitoringConfig class.

The other 2 caches were replaced by DataObjectCache subclasses,
DPNTEPsInfoCache and TunnelStateCache, and injected where needed.

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

Migrate TransportZoneListener to use new listeners fw

It includes the fixing of some issues found during the process.

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

Running Genius CSIT in a dev environment

Document on how to run genius csit in a dev
environment

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