Exception in SubnetRoute Feature

1. Handle exception in subnetroute feature
specifically for the
SubnetRouteInterfaceStateChangeListener.

2. Handle removal of subnetroutes when
subnets are removed from the router (or)
when the router is disassociated from VPN.

3. Increase the retry count to 3 when trying
deletion of VPNs.

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

Merge "ACL: Adding tracked flows for each of allowed address pair" into stable/boron

Merge "fix blue print to init InterfaceStateToTansportZoneListener" into stable/boron

Merge "IPv6Service Binding using elanTag" into stable/boron

Merge "Using DataStoreJobCoordinator" into stable/boron

Merge "stateless security groups" into stable/boron

Merge "BUG 6363: Further performance improvements for VPN Engine" into stable/boron

IPv6Service Binding using elanTag

This patch implements the following.
1. We do Service binding for a port only when a v6 address is
   configured on the port. IPv6Address could be part of the port
   when its initially created or later (i.e, port updated) after
   the VM is spawned. Both cases are handled and the port is
   unbinded when the v6Address-is-removed/port-deleted. Once
   Service Binding is done, packets would hit the IPV6_TABLE(45)
   for further processing.
2. Optimizes the IPv6 punt flows by matching on the elanTag.
   Previous patch implements punt flows based on the VM macaddress,
   this is not optimal as we have to add VM specific punt flows when
   a new VM is added. In this patch, we take into account the elanTag
   (i.e., network-id) and configure punt flows for
   - icmpv6 Router solicitation requests
   - icmpv6 Neighbor solicitation requests for router interface
3. Multiple IPv6 subnets can be added/deleted (even when existing VMs
   are present) on the network and those edge cases are handled in
   this patchset. IPv6Service maintains a cache of (dpnIds <--> List[ports])
   per network. We use this info, while programing flows when IPv6 subnets
   are added/deleted to handle the edge cases.

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

Using DataStoreJobCoordinator

- Made changes such that the Interface Events make use of
  DataStoreJobCoordinator.
- Batching transactions.

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

stateless security groups

1. refactor Egress/Ingress security group to allow inheritance of Stateless implementation
2. adding MethodInvocationParamSaver to allow asserts on parameters on invoked mocks.
3. current implementation differ between stateless Vs statefull is by
   additional configuration in aclservice-config.yang.
4. blueprint adaptation to support inheritence in security group
4. some check-style fixes

Change-Id: I5a73d5087ada651162ee92a42beeea8d9aaf97ef
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

ACL: Adding tracked flows for each of allowed address pair

+ For tracked packets, now matching on IP + MAC instead of lport tag

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

BUG 6363: Further performance improvements for VPN Engine

This fix removes waits in some critical places for
MDSAL operation to complete:
a. InterfaceStateChangeListener
b. VpnInterfaceManager

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

fix blue print to init InterfaceStateToTansportZoneListener

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

Add blueprint wiring to natservice

(cherry-picked from commit 954b3f14841bccd4a86bf668890124ce98c47707)

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

Merge "Ignore VpnSubnetRouteHandlerTests" into stable/boron

Ignore VpnSubnetRouteHandlerTests

Change-Id: I5fa97eb10860293d386fd0bfafe5c0e9be84bd0a
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 528fecc384ac71cb2a0e4b384119cece52ff67ca)

Remove dependency on RpcProviderRegistry

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

Update .gitreview for stable/boron

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

Merge "Add ovsdb/netvirt ui to vpnservice netvirt code."

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

Merge "Add blueprint wiring for bgpmanager"

Merge "Add blueprint wiring for vpnmanager"

Merge "Fix NPE caused by ElanServiceProvider"

Merge "OpenStack Port Chain to Service Function Chain translator"

Add ovsdb/netvirt ui to vpnservice netvirt code.

Use karaf feature odl-netvirt-ui to load.

Change-Id: Ibfda6264b58379d47c58af3f74d8a6651cb30a29
Signed-off-by: Andre Fredette <afredette@redhat.com>

Fix NPE caused by ElanServiceProvider

A null value was being passed into genius ultimately
causing a npe.

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

Add blueprint wiring for bgpmanager

(cherry-picked from commit e295724bb946593448669e4c7427aa087c5c5600)

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

Add blueprint wiring for vpnmanager

(cherry-picked from commit 1650830536abf2c503d5448e3f7f3155aed5c6d6)

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

Merge "Minor fixes for floating ip NAT flows - Use apply actions instead of write actions for floating ip port MAC change - Sync install group to ext-net - Create FIB entry to PDNAT for floating ips"

Merge "Add default flows validation to IT"

Merge "Fixes an issue with security rule with only ether field "

Fixes an issue with security rule with only ether field

Change-Id: I8594d53ea321b16c974d5a80e5c7cbf5ada8a802
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Add default flows validation to IT

The "validateDefaultFlows" method was added, so we can
remove the static sleep before running the ping test.

The method look for the following flows:
1) DHCP Table Miss Flow
2) IPv6 Table Miss Flow
3) ELAN sMac Table Miss Flow
4) ELAN dMac Table Miss Flow

Change-Id: I97217fc43203ae342411a35d38857c78358abc1a
Signed-off-by: Oded Shvartz <oshvartz@redhat.com>

Minor fixes for floating ip NAT flows
- Use apply actions instead of write actions for floating ip port MAC change
- Sync install group to ext-net
- Create FIB entry to PDNAT for floating ips

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

Add blueprint wiring for dhcpservice

Change-Id: I896f72477f6c4d86aac117bd6296e3c337e6133c
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Signed-off by: Alexis de Talhouët <alexis.de.t@gmail.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

OpenStack Port Chain to Service Function Chain translator

Change-Id: I40ab18ae038da8eba4422d515877889382e22fa8
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

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

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

Add blueprint wiring for fibmanager

(cherry-picked from commit ebd077e94cac73f88fcb07d882c18ad150a4b79b)

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

Merge "Fix blueprint cosntructors for AclService"

Fix blueprint cosntructors for AclService

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

Add blueprint wiring for neutronvpn

(cherry-picked from commit 0f5f9c3ddf31a77beaf9409f9b199ee15ffef2c3)

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

Merge "Populating future object used by DataStoreJobCoordinator"

Merge "Add blueprint wiring for aclservice"

Merge "fix dhcp to use interfacemanager rather than interfacemgr"

Merge "Netvirt changs for multiple egress service bindings"

Populating future object used by DataStoreJobCoordinator

* populating future object so that it can be used by DSJC
* DSJC uses te future fall retrying failed transactions
* Retry count increased.

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

Add blueprint wiring for aclservice

Change-Id: Id3f6dd3c3e0152554ccf75c4779384d4a5b0a75e
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Signed-off by: Alexis de Talhouët <alexis.de.t@gmail.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Netvirt changs for multiple egress service bindings

These are couterpart to Change 41712
https://git.opendaylight.org/gerrit/#/c/41712/

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

fix dhcp to use interfacemanager rather than interfacemgr

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

Fixed the NAT issue SNAT flows not getting configured

The model NeutronRouterDpns, which is one of the main models used
by SNAT, is not getting populated because of the dependent model
RouterInterfaces being empty. RouterInterfaces model is empty because
of the race condition b/n the RouterInterfacesListner thread
which is trying to read interface state before the thread which
updates the IetfInterfaces model. Hence in RouterInterfacesListener, we
have removed the check of operational state of Interfaces before populating the
RouterInterfaces model. Eventually InterfacesStateChangeListener will
read RouterInterfaces only when ietf-interface is updated and populates
the NeutronRouterDpns model

Change-Id: I2ffaa9d686d213bcd8187ac9e41c5ad2ae5945d8
Signed-off-by: Satish Dutt <satish.dutt@ericsson.com>

Fix IndexOutOfBoundsException in NetvirtIt

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

Merge "Add service names for priority configuration"

Merge "Send garp on floating ip"

Merge "Add blueprint wiring for ipv6service"

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

Merge "reduced the lock scope in neutron vpn utils"

Merge "Corrections on DHCP caches"

Merge "Add unit tests to AclEventListener"

Merge "elan local cache for ElanInstance/ElanInterface"

Send garp on floating ip

When a new floating IP is created or an existing one updated, we want to send a GARP on the external network.

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

Merge "Floating IP entries in SNAT table will be routed to ext-net"

Merge "Respond to ARP requests on floating IPs"

Merge "Adding Etree:Remove CLI Command"

Merge "Automatic tunnel configuration"

Add blueprint wiring for ipv6service

(cherry-picked from commit 943aa071e9b03931d8f3316cba116bddfcb59962)

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

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

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

Merge "Port Pair to Service Function translation"

reduced the lock scope in neutron vpn utils

instead of taking lock on the class level on whole method
lock is taken for add and remove blocks

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

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

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

Port Pair to Service Function translation

Change-Id: I0f990ac6a6947d60e9f77793320c6fc62345b84e
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "updated to support RemoteSecurityGroup"

Merge "ACL: Updated match criterias for ARP flows"

Corrections on DHCP caches

- While handling the DHCP packets the DPNID and mac address of the
  interface was being fetched from a cache which was not updated using a
  clustered data change listener. Instead of this the interface object is
  retrieved from an API exposed by interface manager.
- There were few ConcurrentModificationException observed on the
  tunnelIpElanNameToVmCache in L2GW use case. Fixed that by using
  CopyOnWriteArraySet.

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

Add service names for priority configuration

Depends on https://git.opendaylight.org/gerrit/#/c/43126/

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

Add unit tests to AclEventListener

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

Merge "Fix for bug in elanmactable:show cli"

Automatic tunnel configuration

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

Floating IP entries in SNAT table will be routed to ext-net

- Create group entry to route egress traffic to the external port
- SNAT flow will forward matched floating IP packets to the new group entry
- The src mac will be set to the neutron port of the floating IP
- dst mac of the external GW will be set in the group table

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

Respond to ARP requests on floating IPs

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

Adding Etree:Remove CLI Command

Change-Id: I7ff8401590ca544813bcfd7b15b97a5145993938
Signed-off-by: Guy Sela <guy.sela@hpe.com>

elan local cache for ElanInstance/ElanInterface

A local cache has been created for ElanInstance/ElanInterface to reduce
the time taken for the Datastore read operations as a part of performance
improvement changes.

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

ACL: Updated match criterias for ARP flows

+ Switch ingress matches on arp+arp_sha+lport_tag
+ Switch egress matches on arp+lport_tag

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

Refactor moving tunnelinfstatechglistener subclass

1. Refactored moving tunnelinterfacestatechangelistener subclass out of
VPNInterfaceManager class.
2. Added itmRpcService and FibManager dependency for the above class.
3. Renamed a few misleading functions

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

Fix for bug in elanmactable:show cli

The timestamp in the cli "elanmactable:show" was strange. The timestamp
displayed in the format day,hours,minutes and milliseonds.Now the format
displayed is day,hours,minutes and seconds.

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

updated to support RemoteSecurityGroup

handled remote security group for interface add/delete, interface update
(SG add/delete and AllowedAddressPair add/delete) and SG update (SR
add/delete)

Change-Id: I20d9dc46dcc0279b7198339f0e1064faf5f082a4
Signed-off-by: gujele <imanilyadav@gmail.com>

Merge "Multiple fixes related to VPN concurrency"

Merge "Complete ETree implementation in netvirt. Extended ElanService to support Etree model."

Merge "Fix exception in IPv6 ifacelistener for a tunnel add notification"

Merge "Commented out frequently replayed BGPMIP errorlogs"

Merge "Fixes related to anit-spoofing rule to allow DHCP traffic."

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

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

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

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

Complete ETree implementation in netvirt. Extended ElanService to support Etree model.

Etree network adds another attribute for interfaces: Root or Leaf.
In the same Etree network:
Root interfaces can communicate with all interfaces.
Leaf interfaces can communicate only with Root interfaces.

Change-Id: I7855015db9c34757249a7461ebf37924969d398c
Signed-off-by: Guy Sela <guy.sela@hpe.com>

Merge "ACL: Updated to cache interface/interface state details"

Merge "Add IT to netvirt-vpnservice"

Fix exception in IPv6 ifacelistener for a tunnel add notification

This patch fixes the IllegalArgumentException in Ipv6ServiceInterfaceEventListener
when a tunnel is created.

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

Merge changes from topic 'openstack-sfc-translator'

* changes:
  Port Pair/Group & Chain Listener registration and translator hookup
  OpenStack Networking SFC Flow classifier translator
  OpenStack Networking SFC Neutron Northbound Yang Model translator

Multiple fixes related to VPN concurrency

1. Handle dpnToVpn list maintained by vpn engine correctly.
2. Make sure that SNAT flows appear when Internet VPN is created.
3. Print batches and batchsize for batchable resources as an INFO.
4. Fix the issue when VM holding the invisible IP is removed, the
invisible IP is removed along with the original VM IP.
5. Fixes extra-route add logic so that Extra-Routes configured via neutron
router now appear in FIB for further realization into flows.
6. Avoid using LabelRouteInfo for InternalVPN. This saves Operation DS
namespace.

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

Commented out frequently replayed BGPMIP errorlogs

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