Cleanup: remove redundant type declarations

Java 7 and 8 have improved type inference a lot, many type parameters
are no longer necessary.

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

Cleanup: remove redundant modifiers

* Replace <? extends Object> with <?>
* Remove public from interface members (they're public by default).
* Remove static and final from interface attributes (they're static
  final by default).
* Remove unnecessary semi-colons.
* Remove private from enum constructors (they're private by default).
* Remove static from enum declarations (they're static by default).

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

Merge "move interface utilities from ElanUtils and undeprecate Genius IIM"

Merge "Add egress split horizon drop flows for external interfaces"

Merge "Refactor and checkstyle changes."

TestDataStoreJobCoordinator with waitForAllJobs() methods req. in tests

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

Merge "Checkstyle compliance for DataStoreJobCoordinator, incl. clean JavaDoc"

Merge "ClusteringUtils Collections.singletonList instead Lists.newArrayList()"

Refactor and checkstyle changes.

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

Add pipeline doc for Genius and dependent projects

1. Adds pipeline.rst to capture current OF Pipeline as used by genius and
projects using Genius for app-coexistence.
2. Links pipeline.rst to genius/index

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

Merge "Link specs to genius/index"

move interface utilities from ElanUtils and undeprecate Genius IIM

Moving utilities currently in ElanUtils into genius.
Removing deprecate flag from InterfaceManager, since performance
issues were discovered with the RPC-only approach.

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

Merge "Migrate arputilimpl moudule to Blueprint."

Merge "Blueprint migration for idmanager"

Link specs to genius/index

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

Merge "Add template for design spec documents"

Blueprint migration for idmanager

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

Add template for design spec documents

Refer: https://git.opendaylight.org/gerrit/#/c/47966/

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

Add .tox to .gitignore

Running tox in docs folder generates .tox as output directory.

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

Migrate arputilimpl moudule to Blueprint.

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

Merge "BUG 6726 : Loss of traffic during ODL Cluster reboot"

BUG 6726 : Loss of traffic during ODL Cluster reboot

Since all ARP Traffics are punted to controller, when controller reboots
the packets are lost. The changes fixes traffic loss during cluster
reboot by adding ARP flows on DPN.
The changes includes addition of all NICIRA ARP Extension Actions to
ActionType.java
PS1: Removed Tabs
PS2: Incorporation of review comments
PS3: Incorporation of review comments
PS5: Removed Deprecated method and also removed whitespace.
PS6: Removal of whitespace
PS7: Incorporation of review comments
Change-Id: Ida59553187ba20b10bbc4e08d20e491d5dd72200
Signed-off-by: karthik.prasad <karthik.p@altencalsoftlabs.com>

Add missing odl-infrautils-inject to features.xml

This is required as Genius is starting to use adopt
https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines with
AbstractLifecycle e.g. in https://git.opendaylight.org/gerrit/#/c/47426/
and https://git.opendaylight.org/gerrit/#/c/48036/

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

Add Docs for Genius

This patch just initializes sphinx for Genius

Refer: https://git.opendaylight.org/gerrit/#/c/39142/

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

Merge "Prepare for enabling checkstyle fail on error"

Checkstyle compliance for DataStoreJobCoordinator, incl. clean JavaDoc

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

ClusteringUtils Collections.singletonList instead Lists.newArrayList()

This is clearer, shorter and AFAIK will save a little bit of memory,
because AFAIK an ArrayList will allocate at least DEFAULT_CAPACITY (10)
if not otherwise specified.

Also includes some otherwise unrelated minor ClusteringUtils CS cleanu.

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

alarmmanager Checkstyle clean-up (incl. exception handling fixes)

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

Prepare for enabling checkstyle fail on error

This patch adds the checkstyle plugin config to
fail on checkstyle errors. Uncomment the checkstyle.violationSeverity
line when ready to enable.

If just adding one bundle at a time rather than all the bundles using
the commons/parents, then copy the same plugin section to the bundle
poms and uncomment the severity. As all bundles are completed then
remove the bundle sections and just use the one in the commons/pom.xml.

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

Add egress split horizon drop flows for external interfaces

Add drop flows to egress dispatcher to drop packets originating from
an external interface, marked by SH metadata bit, if the packet is
being sent to an external interface.

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

Merge "Add info to log message with ARP response details on transmit"

Add info to log message with ARP response details on transmit

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

Intro. @Inject @Singleton etc. in some helper services, for testability

This should not have any impact to real runtime code, and is hopefully
low risk. In particular, this alone does *NOT* yet actually exploit
these annotations for auto. generating blueprint XML (because I'm not
activating the blueprint-maven-plugin here, yet).

But adding these is useful already, because it allows these helper
services to be easily used in dependency injection via Guice in
Component Tests.

These annotation can however, later also be used by the
blueprint-maven-plugin - once the respective projects fully switch.

This change also includes a bunch of otherwise completely unrelated
automated formatting and code clean-up which happened automatically as I
was touching these files.

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

Merge "BUG 6626 Packet IN handler thread in deadlock after high ARP rate"

Merge "Bug 6836 - No access to external network"

Bug 6836 - No access to external network

1) Additional race fix
2) Move future creation to be per worker

Change-Id: I02fe765021489c71a9c3549d2d41767c65578a50
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Merge "target-ide/ on .gitignore"

Merge "Migrate Resource Manager service to BluePrint"

target-ide/ on .gitignore

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

Bug 6836 - No access to external network

Upon first neutron network VLAN provider creation with a defined physnet
both VLAN trunk VLAN member are created for each compute node in the ietf-interface config DS.
As a result ietf-interface state for the VLAN member is attampted to be created simultaneously by

1) InterfaceConfigListener - triggered as a result of VLAN trunk creation in the config DS
2) VlanMemberConfigListener - triggered as a result of VLAN member creation in the config DS

Due to race condition in ietf-interface state for the VLAN memeber creation,
OptimisticLockFailedException can be sporadically get thrown and ietf-interface state
will not be added to the oper DS.

Suggested solution:
When renderers triggered by InterfaceConfigListener on the VLAN trunk call OvsInterfaceConfig*Helpers
to update VLAN member state, these jobs will be queued based on the trunk name to the DjC to prevent
concurrent write of VLAN member ietf-interface state with VlanMemberConfigListener

Change-Id: I0a89e73c6ff0acb9bdc369ab73e16df43e2f727b
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Migrate Resource Manager service to BluePrint

Migration of the resourcemanager-impl module to use BluePrint
framework.

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

Merge "Bug 7048 - Update to OF port does not change 220 flow"

Merge "Added postman collections for id-manager"

Bug 7048 - Update to OF port does not change 220 flow

Description : Wrong instance identifier was being used for unbind service.
Also, DJC synchronization key for unbind service should be parent interface, rather
than interface-name, else this will cause race conditions while deleting 220 flow.

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

Added postman collections for id-manager

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

BUG 6626 Packet IN handler thread in deadlock after high ARP rate

use offerNotification instead of putNotification. this will reject messages over the queue limit

Change-Id: Ia43ce75a8a6e7e3b1ed9e90704e03656e7cfdc11
Signed-off-by: shlomi <shlomi.alfasi@hpe.com>

Merge "Enhancing service binding logic to support more services"

Enhancing service binding logic to support more services

Description : Currently Genius supports only a maximum of 7 services per interface.
Enhancing this logic to support maximum 16 services per interface by making use of one
more bit from metadata field.

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

Merge "SFC table addition; Offset the other SFC tables."

SFC table addition; Offset the other SFC tables.

Added one missing SFC table. All other SFC tables had to be offset by 1.
(now they are using indexes 82 to 87, inclusive)

Change-Id: I691ec349c47b96db182b88bf6df51110bd72504e
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Adding resourcemanager postman collection

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

Fix for fcaps application module config push error

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

Merge "Bug 6765: Overriding in_port in table0 with Zero value"

Merge "Fixes for IT base"

Merge "Add pom for commons"

Merge "Bug 6278: Switch to use odlparent's karaf-parent"

Bug 6765: Overriding in_port in table0 with Zero value

Description :(Review Re-raised) OVS split-horizon logic will drop pkts
by default if value of in_port and out_port is same. But, if we have two
vlan-trunk member VM for which trunk interface is same, the connectivity
between these two VM cannot be established as OVS drops such packets.
So, it is required to override the value of in_port so that split_horizon
check is bypassed. The in_port value will be overrriden using load(ex:
load:0->NXM_OF_IN_PORT[]).

Change-Id: I3d7c457bd030b9d386037c4b6002284a83802b09
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Merge "IdManager Performance Improvements"

Fixes for IT base

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

Add pom for commons

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

Enhancements to improve DJC transaction retry mechanisms

Description : Adding linear time delay for each retries in case of job
execution failure in datastorejobcordinator

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

IdManager Performance Improvements

In order to avoid the wait time to write to the DS during allocation of id
each local id pool(pool per cluster) can be stored in memory using
concurrent hash map. This can be later synced with the DS using
DatastoreJobCoordinator. In order to achieve this the following has been
done:

- Created 5 new jobs to update the local pool info DS.
  * LocalPoolCreateJob - Writes the entire local pool content from the
    cache to the DS.
  * IdHolderSyncJob - Updates either of the idHolder(Available/Released)
  * CleanUpJob - Used to return back excessive ids if any to the parent
    pool. This will be scheduled after IDLE_TIME whenever an id is
    released since that is the only time there are chances of excessive
    ids to be present in the local pool.
  * UpdateIdEntryJob - To add/delete the idEntry in the parentPool.
  * LocalPoolDeleteJob - Deletes the local pool from DS.
- New classes to hold local id pool objects.
  * IdHolder - An interface which provides methods to allocateId, addId
    etc. which is implemented by AvailableIdHolder and ReleaseIdHolder.
  * IdLocalPool - Holds the local pool name and IdHolders for each pool.
- JUnits test cases have also been updated.
- Added a CLI for printing the local cache values.

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

Bug 6278: Switch to use odlparent's karaf-parent

Change-Id: Iad5fc802e638963d547e535eace90f07ce0f8519
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Upstreaming ITM cache impl and monitoring bug fix"

Add alivenessmonitor rpc

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

Upstreaming ITM cache impl and monitoring bug fix

Removed the Add and Remove External End Point CLI from this review
Fixed caches related failures in JUnit Tests
Retaining the oper-state along with tunnel-state in tunnels_state

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

Merge "Adds abstract base for Jobs that must be validated"

Bug 6791 Introduction of Async & Clustere listner

Two new listners are introducled which will be responsible for listning to
HA changes. Clustered Listner and normal Listner.
If its HA childen node or normal node then the particular
add/remove/update node will be executed accordingly.

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

Merge "blueprint for alivenessmonitor"

blueprint for alivenessmonitor

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

ODL Bug: 6095,
bundle:diag failing for ITM bundle.
UT:-
RemoveExternalEndpoint is pointing to a vpnservice package which is
causing the issue,
Started the Karaf and checked the bundle status and diag. coming up jjst
fine.

Change-Id: I76613223e5819154e6f505e42ba435500a8f5a28
Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>

Checkstyle and formatting.

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

SFC table addition + relocation

Added one missing SFC table. All SFC tables had to be rellocated
(now they are using indexes 82 to 86, both included) in order
to fit.

Change-Id: I0994994c89f751c533d16c56abe1280af7300098
Signed-off-by: Diego Granados <diego.jesus.granados.lopez@ericsson.com>

Default transport zone creation during the bootup

CSC to create a default transport zone at bootup
-Any switch that connects via OVSDB with no transport zone
configured will get added into the default transport zone
-This transport zone will have default monitoring and tunnel type
settings as per default values kept in the YANG file

Testcases:-
 1.Switch is connected with TZNAME="" and that node info is added to
 defaultTransport zone
 2.Once ODL controller is booted default transportzone is created

Change-Id: I3e510238e097e3a2eb3ab36d871477c46f01509a
Signed-off-by: Sathish Kumar Tippeshappa <sathishkumar.bt@altencalsoftlabs.com>

Merge "BUG 6726 : Loss of traffic during ODL Cluster reboot"

Merge "Make local variables creation and assignment in a single statement. Some other minor formatting (removing commented code, etc.)"

BUG 6726 : Loss of traffic during ODL Cluster reboot

Since all ARP Traffics are punted to controller, when controller reboots
the packets are lost. The changes fixes traffic loss during cluster
reboot by adding ARP flows on DPN.

PS10: Added actionKey as an optional parameter for
getEgressActionForInterface
PS:11: Removed white spaces
PS12: Added description for RPC Yang field
Change-Id: Id75c6bc4f68e88b56a7b5b5722866161a8590dec
Signed-off-by: karthik.prasad <karthik.p@altencalsoftlabs.com>

Merge "Arp cache feature changes"

Merge "Refactoring: MoreObjects2 is now in yangtools.utils"

Adds abstract base for Jobs that must be validated

 + Includes some state-checking stateful methods

 + Inheriting from Observable so that classes that enqueued the
   job can be informed about job changes.

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

Refactoring: MoreObjects2 is now in yangtools.utils

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

Bug 6776 - Bad instructions returned by genius RPC

When calling the 'getEgressInstructionsForInterface' genius RPC, the
correct actions are returned, but encapsulated within a write-actions
deferred action set, which lead to a openflow plugin error.

Since these actions have to be acted upon immediately, and not at the
end of the pipeline, which is when an write-actions instruction will act
upon the actions, the code is modified to encapsulate the actions in an
apply-actions instruction, whose effects are immediate.

Change-Id: Icaedf148b015d729f21300209f84eeeb5b9b76dd
Signed-off-by: Miguel Duarte <miguel.duarte.de.mora.barroso@ericsson.com>

Merge "Adding the Add/Remove ExternalEndpoint commands."

Make local variables creation and assignment in a single statement.
Some other minor formatting (removing commented code, etc.)

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

Arp cache feature changes

The aliveness monitor is used for the arp cache feature.
ARP monitor packets(ARP requests) are sent by the aliveness monitor and the
response to these are processed by the aliveness monitor.

The packets are checked whether they are from the L3 interface table. As
the ARP packets are now punted to controller from the L3 gateway mac table
(my mac changes), the packets are checked whether they come from that
table too.

Change-Id: Ic5ad946aac7a9f1a1210e55ba9ca39cb3c0865a6
Signed-off-by: Gobinath <gobinath@ericsson.com>

Merge "Bug 6786:  L3VPN is not honoring VTEP add or delete in operational cloud"

Bug 6786:  L3VPN is not honoring VTEP add or delete in operational cloud

Whenever VTEP IP on a DPN is added or deleted , the next-hop IP of VMs in
FIB entries on that DPN needs to be updated appropriately.

To know which all VM FIB entries to be modified , a new DpnToInterfaceList
operational datastore (DS) is being introduced in InterfaceManager.
InterfaceManager will update the DpnToInterfaceList  operational DS , when
VM is booted or deleted.

Whenever VTEP IP is added/deleted on a DPN , VPN would get a VxLAN tunnel
event. VPN will query the InterfaceManager using RPC for a given DPNId to
retrieve the list of VpnInterfaces(VM ports).

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

Adding the Add/Remove ExternalEndpoint commands.

Change-Id: I32f3c611a2641126c867ab4bc36423e647a41765
Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>

Merge " changes for REST API to make TZ specific parameters  (transport zone  name, tunnel type) mandatory."

BUG 6838: Retry Mechanism for Batched Transaction

If the existing batch mechanism fails, then the individual operations of
each of the transactions will be retried so that all other operations
other than the problematic one will succeed.

Change-Id: Iec94654f1e28d9c9088cb0a6e3c6f7dece41e138
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Merge "Bug 6642 - Improvising Batching code"

Merge "Add fcapsapplication-impl XML config to features/pom.xml"

Merge "Implement an action type nx_load_in_port"

Merge "Fixing overflow in long-to-IPv4 address conversion"

Merge "Revert "BUG 6838: Retry Mechanism for Batched Transaction""

Revert "BUG 6838: Retry Mechanism for Batched Transaction"

The corresponding netvirt patch is fairly large and conflicting with many other patches, so we will revert this patch until we work through the backlog.

This reverts commit 29f20d962ea8c3e76b4e81857f39b7d56cbe8769.

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

Bug 6589 adding support for hwvtep devices ha

Added the following utility methods
to add a node as ha child node of the other node
to check if a node is ha child node or not
to get list of child nodes for given parent node

When two southbound devices are in mlag, the northbound l2gw configuration
has to be synced in both the devices.
a node is marked as ha child node based on other_config columnn key ha_id in
manager table
Two ha devices will have same ha_id value.

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

Merge "BUG 6838: Retry Mechanism for Batched Transaction"

Merge "Replace some collection.size() > 0 for !collection.isEmpty() to improve readability. Some other minor changes."

Merge "Add support to the ITM to create Transport Zones with different UDP:   VxLAN: default port   VxLAN-GPE: 4880"