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>

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

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

ACL: Updated to cache interface/interface state details

    - Updated to cache only required interface/interface state details in
      AclInterface object
    - AclInterface object used now as reference in all listeners for
      programming ACL flows instead of Interface/Interface State object
      from config/operational DS
    - Resolved NullPointerException observed while ACL flows were deleted
    - Updated AclDataUtil to use UUID of SG everywhere instead of SG name

Change-Id: If1a12e9a746aa1bf0360aa82caa3a26f6118d420
Signed-off-by: unknown <shashidhar.raja@ericsson.com>

fix compilation error

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

Add IT to netvirt-vpnservice

Writing IT that are available in the old Netvirt project.
The first patch include 2 tests:
1)validate basic connection between ovs and controller.
2)validate ping between 2 ports (using docker).

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

Port Pair/Group & Chain Listener registration and translator hookup

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

OpenStack Networking SFC Flow classifier translator

Patch contains translator code that converts OpenStack Networking SFC
flow classifier neutron yang models to OpenDaylight ACL Flow classifier
yang models

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

OpenStack Networking SFC Neutron Northbound Yang Model translator

This project will be hosting a code that will convert OpenStack
Networking SFC Neutron Northbound Yand Models to OpenDaylight
ACL/ SFC yang models.

Patch contains the project structure that hooks in all the
required dependencies that is required for the translator code.

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

Merge "bug-6326 use new sig for setBridgeController"

Merge "Added Port Range and Ipv6 matches"

Set vpn-id/router-id metadata so it would not to override split-horizon bit

Depends on genius commit https://git.opendaylight.org/gerrit/#/c/42918/

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

bug-6326 use new sig for setBridgeController

depends on: https://git.opendaylight.org/gerrit/42979

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

Added Port Range and Ipv6 matches

1)Added port range match using nicira extension.
2)Added IPV6 source and destination matches
3)UT Fixes

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

Merge "Formatting changes"

Global use of table ids from NwConstants.java

This review includes following things:

1> After merging of https://git.opendaylight.org/gerrit/#/c/42940/, build
of Netvirt started failing. Because ElanStatisticsImpl.java was using one
constant from IfmConstants.java, which was removed in mentioned commit.

2> Removed few table ids from FibConstants.java, NATConstants.java and
VpnSubnetRouteHandler.java then using directly from NwConstants.java.

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

Merge "Periodic RA transmission"

Merge "Add a match to nd_target for IPv6 NS punt flows"

Formatting changes

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

Periodic RA transmission

Support added for periodic transmission of the RA messages
from the router interfaces available in the compute nodes.
The unsolicitied RA is sent out only on the interfaces
where the same network is available.

Pending items:
1. Configurable RA interval
2. UT for the periodic transmission
(Will be added in separate patch)

Change-Id: I7b498e88ba6801f564c931abc6b5dfeb2d0b14ed
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>

Add a match to nd_target for IPv6 NS punt flows

Previous patchset that added support for IPv6 ServiceBinding
programs a generic flow to send all Neighbor Solicitation requests
to the controller. This was done since the support to match nd_target
was missing in genius matchutil at that time.

We now have support for matching nd_target in genius. This patch
modifies the punt flows to send only NS packets that correspond to
the Router interface. Neighbor Soliciation for other VMs would be
answered by the respecive VMs on the network.

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

Merge "Fixes related to ACL service."

Merge "Fix of NPE in DhcpServiceCounters and NatServiceCounters."

Fixes related to ACL service.

+ Egress ACL table is split into two tables (40 and 41).
+ Ingress ACL table is split into two tables (251 and 252).
+ Ingress/Egress filter tables are made to match on lport tag.
+ Packets matching ingress/egress ACL filters are re-submitted to
corresponding dispatcher tables for the next service.
+ Egress (w.r.t switch) acl service priority is changed to value 6.
+ Few more fixes.

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

Fix of NPE in DhcpServiceCounters and NatServiceCounters.

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

Global use of Table ids from NwConstants.java

In my previous patches for global use of table ids, I missed one table id
to remove from NATConstants.java.

Removing that particular table id in this patch and using directly from
NwConstants.java.

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

Merge "Using Table Ids from NwConstants.java"

Merge "First use of the infrautils counters in netvirt."

Using Table Ids from NwConstants.java

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

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

Add external ports to L3VPN models

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

First use of the infrautils counters in netvirt.

Added install and remove flows counters for Dhcp and SNAT default flows.

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

Support update/delete operations for provider networks
Add external ports to remote BC group buckets

Depends on genius commit https://git.opendaylight.org/gerrit/#/c/41943/

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

Merge "Write netvirt:1 to mdsal when netvirt is ready"

Write netvirt:1 to mdsal when netvirt is ready

This new module will write netvirt:1 to mdsal when all the
netvirt services have started. There are 10 services that
are required. Some services are in the Service Registry and
can be referenced in the blueprint.xml. Other services can
only be found via the config subssystem beans. Once we move
the rest of netvirt to blueprint we can deprecate the config
related code.

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

Create ELAN interface for physnet ports

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

Create physnet bridges+ports and patch to br-int

NB: This commit depends on changes in the ovsdb
project: https://git.opendaylight.org/gerrit/#/c/41946/

AND NOW ALSO:
https://git.opendaylight.org/gerrit/#/c/42693/

Assume "provider_mappings=physnet1:ABC". ABC will either be (in this
order) (1) the name of a port already configured on br-int, (2) the name
of a bridge already configured on OVS, or (3) the name of a physical
interface. So for each option:

(1) ABC is a port already on br-int
At OVSDB node add/update event:
  no op
When adding the ElanInterface to the ElanInstance:
  ABC is the name of the port, use that

(2) ABC is the name of a bridge on OVS
At OVSDB node add/update event:
  patch the bridge to br-int, br-int side port name is ABC-patch
When adding the ElanInterface to the ElanInstance:
  ABC-patch is the name of the port, use that

(3) ABC is the name of a NIC
At OVSDB node add/update event:
  add the port directly to br-int
When adding the ElanInterface to the ElanInstance:
  ABC is the name of the port, use that

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

Merge "Flat/VLAN network type support"

Merge "Support Service Binding for ipv6service"

Merge "Security Group: Allowed address pair changes"

Merge "Add NetvirtItUtils.verifyFlowByFields"

Merge "Fix and disable cloud-servicechain config"

Support Service Binding for ipv6service

This patch supports ServiceBinding with genius for an interface
that is having an IPv6 address and is connected to a router.

IPv6Service uses genius NWConstants for the serviceIndex value.

Pending (will be taken up via a different patch):
  1. Flow to punt Neighbor Solicitation currently does not include
     the nd_target, as support is missing in MatchUtils.
  2. ServiceBinding for existing ports when a v6 subnet is added to
     a network and associated to the router.

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

Fix and disable cloud-servicechain config

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

InterVpnLinkNodeListener log message is happening about
8 times per second.

Changing it from info to debug.

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

Added license headers in cloudservicechain test

Also included fix from https://git.opendaylight.org/gerrit/42760
(reflection of NwConstants change in Genius) to enable
successful compilation.

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

Merge "Fixed NPO and and some other changes"

Security Group: Allowed address pair changes

    - Updated Neutron port listener to pass Neutron port's MAC + Fixed Ips
      as allowed address pairs to Acl Service
    - Moved all SG utility methods to NeutronvpnUtils

Change-Id: I44467157e9b5085b198d1a107f2f121c7ef814a4
Signed-off-by: unknown <shashidhar.raja@ericsson.com>

Removed unnecessary changes

Dependencies of interfcemanager-shell not required on pom.xml
of vpnservice in netvirt.

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

Flat/VLAN network type support

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

Merge "Updated to support allowed address pair with ACL service"

Fixed NPO and and some other changes

1)Fixed NPO in case of empty access list

2)Added code to support ether type.

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

interfacemgr-shell was renamed to interfacemanager-shell

See:
https://git.opendaylight.org/gerrit/42028/

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