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"

Merge "Add SFC relevant service binding constants"

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: I7ad598fe9604aae9ad1df71270f1cba5b212a634
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

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

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

Add SFC relevant service binding constants

Using the same service index as SCF, as they should not coexist, to
avoid shifting the following indexes.

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

Fix Improved error message for jobs

Change-Id: I96f04654089ef390bfc2bab3c07e48f854254e3d
Signed-off-by: Marcos López Samaniego <marcos.lopez.samaniego@ericsson.com>

Improved error message for jobs

Signed-off-by: eraulhe <raul.hernandez.sucuesta@ericsson.com>
Change-Id: Ia6942d70dd5186c9d19564ece507c08ca683bac0

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

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

Merge "Reverting Overriding in_port in table0 with Zero value"

Reverting Overriding in_port in table0 with Zero value

Description : It's been observed that the fix for Bug 6765 is resulting
flooding of ELAN packets when External IPs case.

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

Merge "Implement action types required for ping responder"

Add JavaDoc to AsyncDataTreeChangeListenerBase init() re. @PostConstruct

Clarification re. precise inheritance semantics of the JSR 250
@PostConstruct DI lifecycle annotation.  Further details are also
documented on
https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines#Nota_Bene

This investigate and clarification has come out of the problem
encountered and solved in
https://git.opendaylight.org/gerrit/#/c/45966/.

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

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

Merge "Performs a residual cleanup of ElanPseudoPort flows"

Bug 6765 : Overriding in_port in table0 with Zero value

Description : 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: Iab5b87558beecee76a011bad33e1867311dc4a29
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Depends-On: Ica0a1b2da27fc189b8906462a4420473215a22af
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Bug 6748: Added support for match on nxm_reg5

Change-Id: Ifb87cea2142e90d29fcb0859ab1c9afb265a8e47
Signed-off-by: Tomer Pearl <tomer.pearl@hpe.com>

Performs a residual cleanup of ElanPseudoPort flows

 + When an ElanPseudoPort is removed on the Northbound, we
   must ensure that no elanPseudoPort related flow is left
   behind in any DPN.

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

Merge "NPE in InterfaceTopologyStateListener"

Merge "VM Migration: Flows not programmed in new DPN"

Merge "Bug 6610 Moving ACL service as highest among all the services"

Implement action types required for ping responder

Implement action types: move_src_dst_ip, move_src_dst_eth,
set_icmp_type

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

NPE in InterfaceTopologyStateListener

If the dpid is null for both old and new bridges the
"else if" will throw an NPE.

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

VM Migration: Flows not programmed in new DPN

If remove event comes from older dpn is fast, then the
flows programmed in new dpn gets removed instead of skip.

Fix is to move the drop remove event to a sync block.

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

Merge "Arp cache feature changes"

Bug 6689 - long delays between vm boot and flow installation

Improve DataStoreJobCoordinator performance by:

1) Removing sleep() after hitting an empty job list
since it causes major performance hit on scale setups (tested with 48 cpus)
2) Fix signal waiting condition to take effect
3) Add logging

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

Arp cache feature changes

The Arp cache feature (a part of l3vpn) has been refractored. Also, now
the Arp entry/routes of a VM are deleted only after failure in sending ARP
probes to the VM.

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

Merge "AsyncDataTreeChangeListenerBase @PreDestroy close() for easier DI"

Merge "Add VxLAN-GPE to the interface types list handled by the IFM"

Bug 6493 - Interface-Manager performance optimizations

Description : When BFD status notification comes before openflow port add
notification, the event was ignored. Adding a cache to store bfd-state
which can be queried to update the oper-status when node-connector add
event comes.

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

AsyncDataTreeChangeListenerBase @PreDestroy close() for easier DI

and new @PostConstruct protected void init() { }

This goes together with the
https://git.opendaylight.org/gerrit/#/c/44554/
and https://git.opendaylight.org/gerrit/#/c/45248/ work, and when merged
will allow to further simplify and standardize e.g.
https://git.opendaylight.org/gerrit/#/c/45249/ one more step, as shown
in https://git.opendaylight.org/gerrit/#/c/45261/

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

Bug 6557 : NPE thrown during Interface-mgr RPCs call

Description : When an get-dpid-from-interface and get-port-from-interface
RPC call is made on a interface object for which interface-state doesn't
exists, NullPointerException is thrown. Instead of NPE, proper error
message is required to be logged.

Change-Id: Iac989c673957eb30eca1d842381e8f2f4edf0a5b
Signed-off-by: Chetan A G <chetan.arakere.gowdru@ericsson.com>
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Add VxLAN-GPE to the interface types list handled by the IFM

The VxLAN-GPE tunnel type was not on the list of interface types
handled by the Interface Manager, so the manager returned unknown
interface.

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

Bug 6610 Moving ACL service as highest among all the services

Change-Id: I93a8f571746973e85cce3c179385ac09689d53cc
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Merge "Adding srcMac to odl-arputil since ARP src mac is currently set to the OF port's mac internally and cannot be set externnaly. (e.g. in the case of GARP for floating ip we would want to use the floatingIp port's logical mac)"

Adding srcMac to odl-arputil since ARP src mac is currently set to
the OF port's mac internally and cannot be set externnaly.
(e.g. in the case of GARP for floating ip we would
want to use the floatingIp port's logical mac)

Change-Id: I44f7afba8110745e50963a9a8d0d9d89d31e59fe
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Fixed the use of index NwConstants

There was a change to support L2 before L3
https://git.opendaylight.org/gerrit/#/c/43126/
Change the use of constnats to the new way of work

Change-Id: I59836f6791fd8c37aeedd9188306ff090a2219c5
Signed-off-by: yair <yair.zinger@hpe.com>

Merge "Fix Dependency Injection IdManager appears to have to have a LockManager"

Merge "Table Id constant for L3VPN GW Table"

Merge "Bug 6539 : Exception seen during OVS disconnect"

Merge "add send packetout from controller"

Bug 6539 : Exception seen during OVS disconnect

Description : Data Validation exception been observed for port-names
prefixed with dpnid(Ex : br-int,br-ex).This is due to removeIngressFlow
and unbindDefaultEgressDispatcherService API getting called on these port
which should be avoided during OVS disconnect.

Change-Id: Ie631a3e40dc39cd0255968813f8334cc24ba6327
Signed-off-by: Chetan A G <chetan.arakere.gowdru@ericsson.com>

Merge "Fix bug wherein console commands would not work with Karaf ssh session"

Table Id constant for L3VPN GW Table

* Changed id from 19 to 13

Change-Id: I93ab27797a05f540202166344c84fa8ebd3eb6bc
Signed-off-by: eceghkl <manu.b@ericsson.com>

Fix bug wherein console commands would not work with Karaf ssh session

Change-Id: I2bb59ab2eecb96644caad9cf1614bf6060e2753d
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Removing Blind imports across the module

This commit removes blind imports across the genius module.

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

add send packetout from controller

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

Bug 6493 :Interface-Manager performance optimizations

Description: starting to use batching for tunnel interface configuration
             separating out transaction batching across shards
 optimizing service binding calls for non northbound interfaces
 fixing exceptions in interface-manager while ODL was shutting down

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

Bug 6490 : After VM migration, L3VPN flows are failed to program

L3VPN is processing both remove and add event parallely.
Because of this race condition occurs and in ends up in
removing the flows programmed for table 21 on new DPN.

This fix has introduced a delay of 10sec for VM migration
events for remove and add in interfacemanager code.

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

Bug 6479 : Exception from Ifacemgr while feature load

Description: When feature odl-netvirt-openstack is installed, an
NullPointer Exception is observed. This is due to
InterfaceInventoryStateListener getting triggered before the
entityOwnershipService is initialized in IfmClusterUtil which is used by
this listener. Changes done to initialize this entityOwnershipService
first followed by registration of Listeners.

Change-Id: I4aa2f1178eca10754d0f935d477b4246a0f4821a
Signed-off-by: Chetan Aakere Gowdru <chetan.arakere.gowdru@ericsson.com>

Add learn table constatnts

Change-Id: I1e04e9c30663ff7287c569380d6b5384bcdf8b23
Signed-off-by: Slava <slava.radune@hpe.com>

Remove unused fields in close() that caused NPE

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

Remove req-cap for eos

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

Merge "Table Id constant for L3VPN GW Table"

Bug 6439 : lower-layer-if information is missing in Op ietf-interfaces:state for the Tap interfaces

Moving to clustered DTCL in interface-manager to avoid race conditions
encountered during scale testing, when both northbound and southbound events
are triggered simultaneously.

Change-Id: If4a98acf50312a64ae03aaccc4562e5782716b2f
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: unknown <efaseel@IN00116280.ericsson.se>
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Table Id constant for L3VPN GW Table

added table cookie and Id for L3VPN GW Table

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

Merge "add learn action"

Merge "Intro. new TestIMdsalApiManager implements IMdsalApiManager"

add learn action

this commit depends on https://git.opendaylight.org/gerrit/#/c/42702/

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

Merge "ACL: Removed deprecated ACL constants in NwConstants"

Fix Dependency Injection IdManager appears to have to have a LockManager

Therefore it's constructor should expect one, just like it does for
DataBroker (AKA "constructor injection") instead of having a separate
method for it (AKA "setter injection").

Unless a LockManager is optional for an IdManager, which doesn't seem to
be the case.

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

Merge "Bug 6388: Support of Non-unique names across OVS"

ACL: Removed deprecated ACL constants in NwConstants

+ All the references in netvirt is already updated to use non-deprecated
constants in the below commit.
https://git.opendaylight.org/gerrit/#/c/43640/

Change-Id: If55e16ba03fca52774f3ae01a19a70f1d44cdfeb
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Bug 6388: Support of Non-unique names across OVS

Description: When two nodes connect and those two nodes have interfaces
with the same name (They all do, br-int), each time one of those
interfaces gets added to the InterfaceState, it will erase the previous
one. Changes done to check VM migration only tap/vhu/tun ports and prefix
dpnid for all other type of interfaaces

Change-Id: I9e7d930e294836b2a9be6facf5e0d5847ce1cffe
Signed-off-by: Chetan Aakere Gowdru <chetan.arakere.gowdru@ericsson.com>

Merge "log cleanup - remove errors on resource manager pool init"

ArpUtilImpl fixes

- Add egress output actions when sending ARP request
- Fix NPE in getPortFromInterface

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

Intro. new TestIMdsalApiManager implements IMdsalApiManager

and publish a test-jar which is then used via <type> e.g. in aclservice
testing (moved here so that it will be available to other projects as
well later)

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

log cleanup - remove errors on resource manager pool init

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

.gitignore .factorypath created by m2e-apt

https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath
files all over the place.

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

Merge "Install flows into Table 36 only for LLDP."

Install flows into Table 36 only for LLDP.

ITM will not install flows into Table 36 for BFD monitoring.

Change-Id: I409fffbc49959715f69f4fd71e8ac20a400da638
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Bump ietf versions to ...10-SNAPSHOT

Bump versions according to:
https://lists.opendaylight.org/pipermail/release/2016-August/007731.html

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

ACL: Updated ACL related constants in NwConstants

+ As the pipeline is w.r.t switch, updated ACL ingress/egress constants to
align w.r.t switch instead of VM.
+ Earlier constants are deprecated, will be deleted when the references
are updated in netvirt.

Change-Id: Iea65815cdffa9f2de952b7b7bfbea8b5e476fbe3
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Bump versions by 0.1.0 for next dev cycle

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

Merge "Use odlparent dependency management for guava"

Merge "Merge L3Vpn service Id check for Arp Learning"

Use odlparent dependency management for guava

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

fixed some formatting problems

Change-Id: I85d3e24b1a280b840dedc265e8aa64da59154378
Signed-off-by: David Goldberg <gdavid@hpe.com>

Merge L3Vpn service Id check for Arp Learning

Change-Id: I965c3997b7437219f1f2bc449bf8ee0435499563
Signed-off-by: Achuth <achuth.maniyedath@ericsson.com>

GetEgressACtions giving wrong actions for tunnel interfaces

Description: Packets to egress via tunnel need not go via Egress
Dispatcher Table, since currently we don't have egress binding on tunnels.
Making getEgressACtions() API return normal egress actions for tunnel
ports, and returning goto egressTable action only for l2vlan interfaces

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

Merge "Having maximum id block size"

Merge "Fix cpu hog in JobQueueHandler thread"

Merge "Add service names for priority configuration"

Merge "Log message improvements in ResourceBatchingManager"

Add service names for priority configuration

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

Merge "Support for binding multiple egress services on the same interface"

Having maximum id block size

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

Log message improvements in ResourceBatchingManager

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

Fix cpu hog in JobQueueHandler thread

When the jobEntriesMap is empty, this thread consumes
as much CPU as it can, spinning and wasting cpu
cycles.

Add a small 500ms sleep, when the jobEntriesMap is empty.

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

fix compilation error due to a change in openflow plugin

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

Fix for bug in cli elaninterface:show

The cli "elaninterface:show" listed the operational state of the
interfaces as "UP" even after their DPN is disconnected from the
controller. Only handling for the states "UP" and "DOWN" were present in
the code. Changes have been made for handling of the "UNKNOWN" state.

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

Support for binding multiple egress services on the same interface

Description : Adding support for binding multiple egress services on the
same interface. Whenever an egress service is bound on an interface, the
following operations will happen:
1) an entry will be created in Table 220(Egress Dispatcher Table) to match
on the lport tag and service index, and take the packet to the respective
egress table pipeline
2)Based on the service priority, entries in table 220 will be reordered,
so that highest priority service will match first in the pipeline
3)Applications should do a resubmit from their pipeline, back to table
220,if the packet is not destined for their pipeline

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

Merge "Set VRF-ID metadata so it would not to override split-horizon bit"

Set VRF-ID metadata so it would not to override split-horizon bit

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

Using Table Ids from NwConstants.java

This review includes changes in entire genius module, to use table ids
from NwConstants.java.

This thing is already done in netvirt module.
https://git.opendaylight.org/gerrit/#/c/42776/

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

Merge "Updated NwConstants with ACL related constants"