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>

InterfaceManagerService API without RpcResult

FutureRpcResults lets us easily and clearly make RPC implementations
be simple delegating wrappers to methods just returning a Future<Output>
witout the intermediary RpcResult that I find is often more worked
around than really used.

These non-RPC service implementation classes can then be registered as
regular local OSGi Services, and direct used locally.

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

Added a new API addL2GwInterface to handle adding TOR NodeId as external
destination endpoint

Change-Id: I929757008e451e285b4a029523c120256628d30c
Signed-off-by: Yugandhar Reddy Kaku <yugandhar.reddy.kaku@ericsson.com>

Id manager: use transaction manager

Use the transaction manager to manager all write transactions as far
as possible. A couple of uses are still left because they manipulate
state outside the transaction scope.

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

Use odl-daexim instead of odl-daexim-all

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

RetryingManagedNewTransactionRunnerImpl made private (incl. JavaDoc)

because upon starting to explain the retrying
ManagedNewTransactionRunner to people I realized that the 3 impls are
really my implementation detail; as a user what's important is the pure
close guaranteeing VS the retrying one.

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

Upgrade Race Condition: tap ports not getting table=0 flows

On occasion VM ports do not get table 0 and 220 flows. The immediate
cause of this is there is no interface state added for the neutron port
(name=the uuid) just for the "physical" port (name=tapXYZ). This happens
because of the following race condition:

TerminationPointStateListener triggers the creation of the child
interface for the external ids it receives from ovsdb. Before it does,
it retrieves an entry in InterfacemgrProvider#nodeIidToBridgeMap in order
to retrieve the node's dpid.  However, the entry in
InterfacemgrProvider#nodeIidToBridgeMap is created from
InterfaceTopologyStateListener and does not necessarily happen in time.
In fact I have seen cases where InterfaceTopologyStateListener is invoked
between two events to TerminationPointStateListener and the first port
will not get flows but the second will.

This fix simply reads from md-sal if the bridge is not present in the
cache yet. This works because the bridge and termination point
augmentations for the node are actually written to md-sal in the same
transaction, the bug has to do with the order of notifications. Note
that in regular application flow the bridge will show up WAY before
the termination points.

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

add maxRetries to RetryingManagedNewTransactionRunner

as well as the SingleTransactionDataBroker

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

Use @OsgiService annotation instead of XML

Following the DI best practices [1], the dataBroker <reference> entry
in lockmanager.xml resource file is replaced by its corresponding
annotation. Tests are changed accordingly.

[1] https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

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

Clean up

- Remove deprecated
- Declare some fields as final
- Remove some redundancies

Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>
Change-Id: I03c0dc552768ae13e1f0a350e3802ac9e52446f5

Migrate InterfaceStateListener listener to use the new Listeners FW

The change uses the synchronous version of the DCTL since it is already
using the JobCoordinator to offload work to other thread(s).

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

SingleTransactionDataBroker with RetryingManagedNewTransactionRunner

built-in retries transparently handle OptimisticLockFailedException

This probably will squash a number of bugs from
https://jira.opendaylight.org/browse/NETVIRT-994?jql=text%20~%20"OptimisticLockFailedException"%20and%20status%20%3D%20Open
in one go.

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

Remove DataStoreJobCoordinator

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

Remove CheckedConsumer from genius and use infrautils'

The one from infrautils has better exception declaration, via generics.

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

LockManager unlock() with RetryingManagedNewTransactionRunner

lock() and tryLock() should ideally also be switched to
(Retrying)ManagedNewTransactionRunner and FutureRpcResults, but that's a
little trickier, because of the return true/false in readWriteLock().

This is a partial counter proposal to
https://git.opendaylight.org/gerrit/#/c/61526/ related to
https://jira.opendaylight.org/browse/GENIUS-86.  It's not a complete
counter proposal; the changes other than the retry on unlock should
still be considered.

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

Drop obsolete Maven site configuration

See
https://lists.opendaylight.org/pipermail/odlparent-dev/2017-November/001492.html

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

Checkstyle errors while building on windows due to windows delimiters

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

Upgrading flag for full sync upgrades

This commit introduces a small utility that tracks whether or not
an upgrade is in progress. The purpose of this state is so that
code paths required during upgrade need not be entered during
regular application flow. The upgrade in progress state is toggled
via REST (or config xml). I tested it like this:

> curl -v -u admin:admin -X PUT -d @put-data -H \
"Content-Type: application/json" \
http://127.0.0.1:8181/restconf/config/genius-mdsalutil:config

...where the contents of the put-data file are:
{
"config": {
        "upgradeInProgress": false
         }
}

The REST call to retrieve the status is:

> curl -u admin:admin http://127.0.0.1:8181/restconf/config/genius-mdsalutil:config

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

Nicer (shorter) how to addErrorLogging in AlivenessProtocolHandlerLLDP

improves If342b11591f1dfe6e58fdca29b400531455669fd

using I6be0a0677659d9748e86995c8a4874e99e2c6493

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

Remove un-used fields, services and DEFAULT_CALLBACK in 3 ITM Workers

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

Fix possible transaction leak in ItmInternalTunnelDeleteWorker

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

Fix bad transaction leak in ItmInternalTunnelAddWorker

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

  523x transactions opened here, which are not closed:
    org.opendaylight.genius.itm.confighelpers.ItmInternalTunnelAddWorker.buildAllTunnels

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

GENIUS-86 : LockManager fixes

Root cause:

There were few AskTimeOut exception that was observed, which means the
data got eventually written to the DS. But because of the exception the
lock was being tried again to be acquired. Later, because of the first
write attempt that got succesful we were getting logs saying that the lock is
already present. This could be avoided by checking if the lock that is
already present in the DS is that of the same request.

Fix:

- For each request we can maintain an id so that if at all a timeOut
exception occurs, but eventually the write succeeds, we can identify
which request got successful by checking the owner field.
- While trying for lock, if lock is already present check for owner and
if it is the same owner then return from lock.

Change-Id: Ia2757981e92dbd48f3057c841f6ed6d710ba5bd2
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Fix FindBugs violations and enable enforcement in interfacemanager-api

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

Fix FindBugs violations and enable enforcement in mdsalutil-impl

Also converted FlowBatchingUtils from static to a singleton.

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

Enforce findbugs

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

Fix too long service binding INFO logs

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

Enable FindBugs enforcement in alivenessmonitor-impl

No violations found!

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

ItmInternalTunnel[Add|Delete]Worker with DataBroker as field instead arg

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

Fix FindBugs violations and enable enforcement in lockmanager-impl

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

Shut down the resource batcher before cleaning up

Currently, every time the batcher runs, it retrieves the information
it needs from a map. When we unregister a resource, we remove that
information before stopping the scheduled thread; this can result in
errors being logged because the thread fires between the removal and
the scheduler shutdown.

This patch switches the order, stopping the scheduler before cleaning
up the map.

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

Fix FindBugs violations and enable enforcement in idmanager-impl

See in-line comments

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