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>

adding portuuid to be retrieved as part of port statistics

Port statistics Listener gives nodeconnectorid.
To retrieve portuuid, portname is required.
Through "InterfaceInventoryStateListener" nodeconnectorId-portname map
is built.
Listener class is added upon
"odl-interface-meta/interface-child-info/interface-parent-entry" through
which interfaceparentname-interfacechildentry map is built.

Using these two maps, portuuid is retreived in port statistics.

Change-Id: I83ad72ce0ff21980b323f28aeb2cc177a4034eab
Signed-off-by: Viji J <viji.j@ericsson.com>
Signed-off-by: VijiJ <viji.j@ericsson.com>
Signed-off-by: Viji J <viji.j@ericsson.com>

Remove older s-n-d implementation.

Now that genius has moved to infrautils based
diagstatus, let us remove the older status-and-diagnostics
code which was remaining in IFM and ITM in bits and pieces.

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

added read method to ResourceBatchingManager

Reads the identifier of given resource type
If the identifier is going to be deleted , returns Optional.absent()
If the identifier is going to be added , returns the to be added DataObject
else goes to the datastore and reads it.

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

Add link to ITM scale spec to index

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

Spec for ITM Scale Improvements

Addressed review comments

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

Minor changes found by static analyzer

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

ACL: Defining new ACL tables

+ Defining new ACL tables as per the spec "Support for protocols that are not
supported by conntrack".
+ Includes metadata changes for ACL.

Change-Id: I95df598428f6351e2abb0b173a4318253c9e20bc
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Loading balancing port statistics collections

Currently in clustered controller nodes, stats query for
switches is happening on the master node due to DTCL
listener attachment, so the statistics RPC invocation
involves both local RPC and remote RPC calls. This creates
overhead in a scaled environment.

Fix is been provided to have CDTCL and applying EOS to find
the switches connected locally to the controller node.
Now, each controller node collects statisticsof its own
connected switches.Thus, avoiding remote RPC calls.
Additionally, added 10s of delay between each rpc call
of the switches stats query.
Also increased initial delay to be 5minutes to the overall
statistics collection.

Change-Id: I50b502992bb8182eab406342f467ef56b6ef7798
Signed-off-by: Viji J <viji.j@ericsson.com>
Signed-off-by: VijiJ <viji.j@ericsson.com>

Template for TestPlans

Also adds initial draft of interface-manager test plan

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

Adjust for odlparent 3 / yangtools 2

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

Fix IdManagerTest bug

a failing assert in a background thread in an ExecutorService is useless
and will do nothing; assert can only be used inside the main test
thread.  Also the ExecutorService must be logging its thread deaths.
also this is a prime example of where we really should use
LogCaptureRule to let any logged errors fails the test.

This will NOT fix GENIUS-105, just show the real root cause better.  But
now if you uncomment the RunUntilFailureClassRule & RunUntilFailureRule,
then you will hit the problem what occurred in GENIUS-105 during the
build locally, sooner or later (just let it run a few minutes).

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

Migration of arputil counters to use infrautils.metrics

instead of infrautils.counters

Code changes are done to use infrautils.metrics api and
cleaned up infrautils.counters dependency.

Change-Id: I73bf730d0f0e2f96fa4a84ab471e18f33305360c
Signed-off-by: Viji J <viji.j@ericsson.com>
Signed-off-by: VijiJ <viji.j@ericsson.com>

Make upgrading flag settable from a config file

File is called genius-mdsalutil-config.xml and looks like:
<config xmlns="urn:opendaylight:params:xml:ns:yang:mdsalutil">
  <upgradeInProgress>true</upgradeInProgress>
</config>

ALSO, fix that bundle is not reloaded when config changes on the fly.

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

EOS Openflow entitytype changes

1) EOS on openflow switches entity type got changed to "org.opendaylight.mdsal.ServiceEntityType"
2) Fix null pointer exception - putIfAbsent returns null when the first value is added.
   computeIfAbsent is replaced.
3) revisit of logs.

Change-Id: I13452abaaf382ac5ff9bda4bb1d1c7ad7278c5ec
Signed-off-by: VijiJ <viji.j@ericsson.com>
Signed-off-by: Viji J <viji.j@ericsson.com>
Signed-off-by: VijiJ <viji.j@ericsson.com>
Signed-off-by: Viji J <viji.j@ericsson.com>

DataObjectCache CacheProvider requireNonNull

just to avoid people who may start using this rather sooner than later
thinking "oh that's not [yet!] used, so I can just pass null" and then
breaking things when we (soon) do use it - better safe than sorry.

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

Minor: DataObjectCache final ListenerRegistration

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

Migrate listeners in id-manager-impl module to use new listeners fw

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

@Ignore testMultithreadedIdAllocationForSameKeyFromReleasedIds

just like the previous
testMultithreadedIdAllocationForSameKeyFromAvailableIds

because it is unstable and seems to fail autorelease, at least sometimes
(perhaps when the build servers are heavily loaded?); as seen e.g. in
https://lists.opendaylight.org/pipermail/genius-dev/2017-December/002197.html

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

Bug 8998 - ping to TOR vm fails post vm migration

fix npe in  accessing cache

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

Add DataObjectCache

This is intended as a non-static replacement for DataStoreCache.

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

DataTreeEventCallbackRegistrar for data tree instance call backs

Phase 1 is this first interface service API & simple implementation.

Phase 2 will be an optimized implementation (post usage).

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

Karaf CLI to print BFD Cache

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

mdsalutil: introduce new service index for VPNv6

In the case of two router setup configuration bound to the same port,
where IPv4 and IPv6 traffic is redirected to different router, one needs
a new service index that sticks to IPv6 Service. The changes here
include that new service

Change-Id: I4432277fe7361fdd98c193873031e67cff3f8a28
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Adjust to odlparent 3 Checkstyle settings

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

@Ignore testMultithreadedIdAllocationForSameKeyFromAvailableIds

in IdManagerTest, for occassional build issue GENIUS-105

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

Add K8S port pattern

K8S ports should be treated in the same way as Nova or Tunnel
Ports by Genius. Currently Genius prefixes dpnId to interface-name
if a portname does not match with a set of known port patterns.

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

Removed @Deprecated Guava checkedGet in SingleTransactionDataBroker

by replacing it with ListenableFutures.checkedGet from infrautils

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