Guard some TRACE logs in DJC

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

Merge "Bug 8800: TestIMdsalApiManager assertFlowsInAnyOrder fail safe"

Merge "Removing unnecessary TRACE log in service-binding"

Bug 8800: TestIMdsalApiManager assertFlowsInAnyOrder fail safe

It *MUST* fail in case that:

  assertThat(sortedFlows).containsExactlyElementsIn(sortedExpectedFlows);

failed but:

  assertEqualBeans(sortedExpectedFlows, sortedFlows);

passed; any such case is likely a bug in ch.vorburger.xtendbeans.

netvirt aclservice tests (or their expected flows) must be fixed before
this change can be merged.  This doesn't fix anything, it would just let
us discover this kind of problem much earlier, if things ever broke
again.

The new FlowEntityAssertBeansTest adds a non-regression test for the
reoot cause problem; it fails as of the time of writing, but will pass
once the ch.vorburger.xtendbeans dependency is bumped from 1.2.3 to the
1.3.0 in https://git.opendaylight.org/gerrit/#/c/59950.

See bug 8800 for full background and details.

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

Removing unnecessary TRACE log in service-binding

This TRACE log is printing the whole tree of a change notification.
There are other informative logs already in place after the initial
processing, and this one is not needed.

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

Making ActionConverterUtil log to DEBUG

As these logs are filling up netvirt CSIT logs

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

Bug 8800: Prevent xtendbeans NPE warn.log in InstructionApplyActions

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

Remove odl-genius-ui from artifacts and features

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

DO NOT install or deploy the karaf artifact

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

BUG 8528: Fix duplicate Apply Actions Instruction

When an egress service sends it's own Apply Actions instruction, Genius
will generate two Apply Actions instructions southbound, one of it's
own containing the action to set REG6 and a different one containing the
service supplied actions. This is not correct per Openflow specification
and OVS rejects the operation.

Fixed by bundling the set REG6 action and the service provided
actions into the same Apply Actions instruction, taking care of
not rearranging the order of the service actions.

Change-Id: I48735857418fed0025b172b73eb02be9c8e9bbbe
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Upgrade to odlparent 2.0.0, karaf4 and Guava21

1. Change odlparent-lite to 2.0.0
2. Delete genius-features and rename features4-genius to genius-features
3. Change single-feature-parent to 2.0.0
4. Cleanup in karaf/pom.xml for karaf4
5. Fixes for Guava21
6. Removed odl-genius-ui feature, removing dependency on dlux

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

Merge "Genius wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/"

Merge "avoiding logging full stack trace in case of id doesn't exist"

Genius wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/

Also have removed fake "availability=optional"

Change-Id: Ia4c21bbba98f01d938c97990f2a02dd9996a8490
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Don’t load odl-genius-ui on boot

This drops odl-genius-ui as the Karaf 4 distribution’s local feature,
so that it’s no longer loaded as a boot feature. The Genius feature
repository will still be part of the distribution.

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

changing karaf3 to karaf4 for genius features

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

avoiding logging full stack trace in case of id doesn't exist

* This fix is to avoid filling karaf logs exception with full of stack
trace if source of exception is known by its exception type and message.

Change-Id: I057647f9a4ac90c32476e8d9eb2494a1b9d3c4a2
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "Bug 8146:BFD tunnel monitoring interval update fix"

Remove uses of Optional for conditionals

Optional instances should never be constructed just to facilitate
conditional handling of the form

    Type variable = ...
    if (variable != null) {
        ...
    }

Turning this into

    Optional<Type> variable = Optional.ofNullable(...);
    variable.ifPresent(value -> ...);

causes more GC activity and makes the code harder to read (and
analyse).

This patch doesn’t address the issue of whether the values in question
are actually nullable.

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

Bug 8146:BFD tunnel monitoring interval update fix

Added support for dumping more caches
Caches were not getting cleared correctly. Fix to remove tunnels from Cache whenTEPs are deleted.

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

Merge "Use TestEntityOwnershipService that's now in controller"

Merge "Listener base classes should not throw exception if close()'d "

Migrate to odlparent 1.9.0

Change-Id: I40a03cc92255b08e54815050d15d487ef6c081bf
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fixing a typo in DJC log statement

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

Merge "Bug 8555 - Too many log info print to the console"

Use polymorphism instead of Optional

Optional shouldn’t be used instead of method polymorphism. This patch
deprecates
InterfaceManagerCommonUtils.createInterfaceChildEntry(String, String,
Optional) and replaces it with variants using either no transaction or
an explicit WriteTransaction.

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

Use TestEntityOwnershipService that's now in controller

following https://git.opendaylight.org/gerrit/#/c/58089/

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

Merge "Neutron Port Allocation For DHCP Service"

Merge "Bug 8008: NPE detected in IfMgrRpcService.getDpnInterfaceList"

Listener base classes should not throw exception if close()'d

for AsyncClusteredDataTreeChangeListenerBase and
AsyncDataTreeChangeListenerBase, by introducing a new
ShutdownLoggingExecutorService infra util. related to -1 discussion in
https://git.opendaylight.org/gerrit/#/c/57004/

This is an alternative proposal to the ShutdownLoggingExecutorService
idea from https://git.opendaylight.org/gerrit/#/c/57156/

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

Bug 8008: NPE detected in IfMgrRpcService.getDpnInterfaceList

 + This line is throwing NPEs because rpcResultBuilder is NULL under
   certain circumstances:

   return Futures.immediateFuture(rpcResultBuilder.build());

 + I'm fixing other potential NPE for the same price

Change-Id: Ib6bf5039234f114228eb7d6a9c4f3b1882831e5d
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Fix checkstyle problems not detected by the current version

This change is required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

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

Bug 8555 - Too many log info print to the console

Change-Id: I19133b5e7b2768ff5762bcf5d4a97994178a893b
Signed-off-by: 肖锋钢10072566 <xiao.fenggang@zte.com.cn>

Neutron Port Allocation For DHCP Service

Adding new ARP_CHECK_TABLE constant in NwConstants for Neutron Port
Allocation For DHCP Service.

New pipeline for DHCP & Gateway IP ARP Responder in ELAN.
LPort Dispatcher Table(17) -> ARP Check Table(43) -> ARPResponder Group
-> ELAN Base Table(48), ARP Responder Table(81), Controller
-> ELAN Base Table(48) (ARP Reply), Table 220( Floating IP ARP reply)

Change-Id: I2dce5e253d3c94180dd18e51125c5c18170b5c69
signed-off-by: Vijayalakshmi Chickkamenahalli Nagaraju <vijayalakshmi.c@altencalsoftlabs.com>
Signed-off-by: Achuth Maniyedath <achuth.m@altencalsoftlabs.com>

Migrate to odlparent 1.8.0-Carbon

Per request of odlparent project we are downgrading all Nitrogen
projects to use the released odlparent 1.8.0-Carbon to allow for the
odlparent project to start performing semver style releases.

Jira: RELENG-159
RT: 41406

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

Merge "@Immutable GroupEntity"

Merge "Simplify boolean expressions"

Merge "Make testutils a JAR instead of a POM"

EVPN RT2 : L2VNI Demux Table id

For EVPN RT2 a new table(table 24) is introduced to demux the l2vni from packets
coming from DC-GW

Spec is approved and is available at : https://git.opendaylight.org/gerrit/#/c/51693/

Change-Id: Ie23457df4d9d20a01f53e271c5b7ece77342225f
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

Merge "Utility api to configure icmpv6 type"

Merge "Make ActionInfo.toString() generic"

Remove TestableDataTreeChangeListenerModule from ResourceManagerTest

It's currently not used anyway, and (because it's not used!) it can
occasionally cause the "heisenbug" (i.e. arbitrary test failure on some
runs, but not always) problem "IllegalStateException: Test forgot an
awaitEventsConsumption()".

see also (new) doc on
https://wiki.opendaylight.org/view/BestPractices/Component_Tests#Async

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

Add missing @Override and serialVersionUID to genius.mdsalutil

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

Merge "Add Unit Test (only) for ActionDrop equals"

Merge "Add yangtools:testutils to mdsalutil-testutils"

Merge "Support for moving Src/Dest IPv6 addresses in NxRegMove action"

Merge "Support API that configures IPv6 address as sourceAddress"

Make testutils a JAR instead of a POM

and start to use it for holding test fakes of genius APIs

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

Add yangtools:testutils to mdsalutil-testutils

To make it possible to depend on mdsalutil-testutils in a new component
test for netvirt fibmanager, without having to declare a dependency to
yangtools:testutils (one should not have to).

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

Add Unit Test (only) for ActionDrop equals

as a reminder that ActionDrop's equals actually works just fine, even
though it's not overriden (because ActionInfo's equals() has a
getClass() != other.getClass()) ... this confused me for a moment, and a
suspected a classical hashCode / equals bug - but it's actually fine as
is - this new test serves as a regression assurance - and as a "reminder
in code".

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

Make ActionInfo.toString() generic

Subclasses don’t need to override it, so use the actual class name in
toString() (for example, ActionDrop can “just work”).

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

@Immutable GroupEntity

Contrary to the earlier 3a5d1e58dadf17654064234bdeb08597b7a48fe5 which
made FlowEntity @Immutable about 3 months ago, this change has no
impacts in downstream project netvirt (because all of its GroupEntity
instantiations go through MDSALUtil, which is adjusted here as well).

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

Replace LOGGER by LOG

Replace LOGGER by LOG to follow the
OpenDaylight recommendations [1].

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

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

Fix vlanIdModified test

OvsVlanMemberConfigUpdateHelper.vlanIdModified() has an apparent
copy-paste error in its null/non-null check. The whole test simplifies
down to !Object.equals(), so this patch just uses that.

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

Simplify ResourceManager

- Simplify the initialization process refactoring some methods and using
System.getProperty with default values
- Change names of constants
- Improved coverage to from 70% to 81%.

More refactoring in later change.

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

Merge "fix for indefinite wait on the resource lock"

Simplify boolean expressions

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

Merge "Avoid extending Object"

Replace toUpperCase().equals by equalsIgnoreCase()

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

Avoid extending Object

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

Merge "Bug7451 - Maintaining state information for bound-services in DS"

Merge "BUG 8332: Inconsistent tunnel port information in Genius"

Bug7451 - Maintaining state information for bound-services in DS

Use the  bound-service-state info while unbinding service so as to avoid race conditions
when interface-state and service-unbinding happens exactly at the same
time

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

fix for indefinite wait on the resource lock

* Currently idmanager manager mutates the allocated id list (to make it sorted) while writing it into datastore which causes ConcurrentModificationException.
This leads to resource lock is never released and makes release id thread to
wait indefinitley for the lock.
* added lock retry logic with timed future.get version to handle any missed signals

Change-Id: I5f271f6054a44aa50d19f3130a2d5a73230ff50c
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

BUG 8332: Inconsistent tunnel port information in Genius

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

Bug 7451 - Leftovers in dispatcher table when unbind and ietf-state
delete occurs simultaneously

Adding service-state-information to cache, whenever a service is bound to
an interface, and this information will be deleted only when the last
service on an interface is unbound. To make the first and last service
counting more robust, moving the boundservices config listener, one layer
up, at the services-info level, so that the event itself will come with
the number of services currently bound an interface. Reading the same info
from DS later on during the event processing, can cause incorrect data
being read from DS, when multiple parallel services might be getting bound
and unbound without any synchornization.

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

Utility api to configure icmpv6 type

Change-Id: I6c7f2def23e043872957d5272f0fe1217f323c8b
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Support for moving Src/Dest IPv6 addresses in NxRegMove action

Depends-On: https://git.opendaylight.org/gerrit/#/c/56969

Change-Id: Ib4ba7baf50416b4e6e43705fbf796f65f4e00aa7
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Support API that configures IPv6 address as sourceAddress

Change-Id: I23e2f04315ca023be9886d7e1570f78230b65dd7
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Merge "IMdsalApiManager API simplification"

Merge "Incorrect handling of admin state update for interface"

IMdsalApiManager API simplification

Having two installFlow/removeFlow, one returning void and the other
returning CheckedFuture, is completely pointless - they do the same
thing!  The void one is NOT synchronous / blocking anyway (it internally
does NOT wait with a get() on the CheckedFuture) - so simplify.

This change, coincidentally, is fully backward API compatible.  Callers
are free to ignore the new CheckedFuture returned - that's what the code
today already does anyway (in the implementation).

PS: This is the first step of perhaps more IMdsalApiManager API
simplifications to come (e.g. re. having variants taking BigInteger dpId
arguments, given that FlowEntity already has a dpId (only Flow does
not).

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

Remove <prerequisites><maven> from pom.xml to avoid WARNING

[WARNING] The project ... uses prerequisites which is only intended for
maven-plugin projects but not for non maven-plugin projects. For such
purposes you should use the maven-enforcer-plugin. See
https://maven.apache.org/enforcer/enforcer-rules/requireMavenVersion.html

This (^^^) is already done in odlparent.

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

Multiple VxLAN tunnels - egress service

- default egress service would be bound to the logical tunnel
- OF select group based on the actual list of tunnel members in the
  logical group
- set the oper status of logic tunnel according the states of the group members
- admin status of logic tunnel affects the states of the group members

Depends-On:
 https://git.opendaylight.org/gerrit/52871
 https://git.opendaylight.org/gerrit/52993
 https://git.opendaylight.org/gerrit/53603

Change-Id: Ica0801aa274615cedc36c60d661a0d5d411e1bd0
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

Incorrect handling of admin state update for interface

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

fixing junit in resourcemanager

There are some regression introduced in resource manager tests due to [1]
which causes junits in resourcemanager-impl to get blocked indefinitely.
It is addressed in this review

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

Change-Id: I261c14eefa2eb59839fa36eb0578cfe499c487bc
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

adding future.get for acquiring clustered lock

* LockManager manager uses busy spin with thread.sleep to acquire a lock
if there is a contention for it.
* Now thread.sleep is replaced with future.get() using ACDTCN on the lock

Change-Id: I4ce3816e150f1e75fedf3093d96ed41efc444fa5
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Merge "Bug 7826: Data validation failed for path"

Genius: configurable tos option for tunnels to support QoS DSCP marking

For supporting QoS DSCP marking feature, the outer header of the packets
egressing out of Ovs through vxlan/gre tunnels need to copy the DSCP bits
from the inner IP header.

The marking of inner header is done through OpenFlow rules configured on the
corresponding neutron port. For outer tunnel header, the tunnel interface tos
option in ovsdb  needs to be set as “inherit”.

For setting of tos a configurable parameter is introduced under ITM module.
By default the tos option is set to “0” as specified in Ovsdb specification.
For QoS DSCP marking feature, this should be set to “inherit"

link: https://git.opendaylight.org/gerrit/#/c/48949/
Change-Id: Ife923f74f5bd59648d40c1779478333632337952
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>

Merge "Bug 8277 : VXLAN tunnel flapping"

Remove ResourceManager.start method

The ResourceManager.start method has been removed
to improve initialization and testability. Its content
has been moved to the constructor.

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

Merge "@Immutable FlowEntity"

Bug 8277 : VXLAN tunnel flapping

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

Add component tests for Resource Manager Service

Add component tests for Resource Manager Service following this
guidelines [0].

Code coverage for resourcemanager-impl increased to more than 70%. This
coverage can improved by adding some negative test cases as well as
applying some re-factoring to the ResourceManager itself.

[0] https://wiki.opendaylight.org/view/BestPractices/Component_Tests

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

@Immutable FlowEntity

This change has impacts in downstream project netvirt, which are handled
in https://git.opendaylight.org/gerrit/#/c/54390/ (must be merged
simultaneously)

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

Merge "BUG 8107: Flow based tunnels not being handled correctly"

BUG 8107: Flow based tunnels not being handled correctly

OF tunnels don't get added to interface-state which causes failures.

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

Fix minor issues in idUtils

- Return expressions directly.

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

Junits failure on interfacemanager-impl due to timing issues

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

VLAN IF in a VM gets a Trunkport IP from the DHCP if there is no SubPort
created

If there is a VM which has two (2) TrunkPorts (vNICs) and both of these
TrunkPorts does have one SubPort created, and one of these ports has also 8
other VLAN interfaces defined inside the VM. These additional VLAN
interfaces which don't have matching SubPort do get IP address of the
TrunkPort from the DHCP server. Assumption is that the DHCP server (ODL)
should not answer to the DHCP request if it comes from a Trunk port with
VLAN tag / SubPort which is not defined.

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

Merge "ArpUtil Test"

Clean up ITM impl *Worker: Use "final" where possible, avoid parameters

Just a start, there is more TBD here re. static and all; e.g. the
Itm*Tunnel*Worker private static methods could probably be made
non-static and an the workers made to have an instance, or even just be
moved into the respective Itm*Worker classes, which seem to be the only
place using them.

This change was motivated by having had a closer look at this code due
to review comment in c/52871 to make something non-static which we
cannot yet easily, because of this.

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

ArpUtil Test

Change-Id: I084e89b7cf8af6090b3fde30412ac8a809c24647
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>

Enhancing junits for interface-manager phase6

Adding junits for interface-manager APIs

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

Cleanup IdManager and LockManager info messages

These modules are very active and output a large number
of logs in INFO level, which should be DEBUG level.

Change-Id: Id260b124ffeefa8ccf58d2c021c5c75b8ac589ab
Signed-off-by: Alon Kochba <alonko@hpe.com>

Merge "Fix 1 Checkstyle problem (red in Eclipse, missed in build)"

Merge "Spec Propose to utilize Ovsdb QoS queues"

Spec Propose to utilize Ovsdb QoS queues

Change-Id: I7f6d8e4237c829b49efd4b78d072b5cbd91f174a
Signed-off-by: Konsta Pozdeev <konsta.pozdeev@hpe.com>

Added rpc in itm-rpc to move commonly used method.

getEndpointIpAddressForDPN method is used by multiple modules
elanmanager, fibmanager, vpnmanager, natservice
Hence adding this method to itm-rpc.
We can use this same rpc call in all the above mentioned modules.

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

Fix 1 Checkstyle problem (red in Eclipse, missed in build)

required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

see also earlier https://git.opendaylight.org/gerrit/#/c/55170/

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