Upstreaming changes in Netvirt

(Dependent on https://git.opendaylight.org/gerrit/#/c/42540/)

This commit includes VPN Engine Concurrency Fixes also some SNAT and DNAT
regression issues.

1. We have made multiple fixes here for safe concurrency.

2. We have moved all helper methods that deal with VRFEntry
resource into FibManager from VPNManager.

3. We have gracefully handled deletion of bgpvpn regardless of
how many ever external routes are available in such vpn.

4. We have fixed NATService to directly use FibManager to
program SNAT/DNAT VRFEntries.

5. As of this review, all local VM routes WILL NOT be batched for
write into VRFEntry Config DataStore.

6. All the external routes imported from BGP, WILL BE batched and
written to VRFEntry Config DataStore.

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

Merge "Remove unused netvirt dependencies"

Merge "Fix for SingleFeatureTest wiring for neutron model"

Merge "code refactor for aclservice"

Remove unused netvirt dependencies

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

Fix for SingleFeatureTest wiring for neutron model

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

code refactor for aclservice

1. remove dupliate code in EgressAclServiceImpl and IngressAclServcieImpl
by using base class
2. constants are moved from AclServiceUtils to AclConstants
3. classes are moved to meaningful packages
4. class names are changed to make it more meaningful

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

Merge "NeutronVPN: Acquire lock from local JavaVM"

NeutronVPN: Acquire lock from local JavaVM

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

Upstreaming changes in netvirt

This commit includes following changes:

1> Fixing Port Up/Down Event handling for ELAN
2> Parallelization done in Arpcache

Parallelization of adding and removing arp cache done with the help of
DataStoreJobCoordinator.

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

Multiple fixes in various modules

This review includes following fixes or changes:

1> BGPManager using the batching infrastructure
to batch and apply VRFEntries to Config DS.
(both bgp advertisement and bgp withdrawals).

2> FibManager using the batching infrastructure
to batch and apply flow rules to Inventory Config DS.

3> VPN Refactoring - Concurrency for VPNInterfaceManager

4> Changed the code in dhcpservice such that we add only
direct and macvtap vnic_type to the neutron port cache.

5> Fix for bgp display commands to work in COMCLI

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

Add missing license headers

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

Fix build broken due to Checkstyle, and prepare for more CS to come

preparing for https://git.opendaylight.org/gerrit/#/c/41305/ with:
* Utility classes should have private constructor
* potential future problems re. hidden fields
* inner assignments

and also for the just merged
https://git.opendaylight.org/gerrit/#/c/40957/, which I think that
something here just got added and missed in the verification for
c/40957, so was about to cause build failures - so this one urgently
needs to get merged ASAP!

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

Merge "Bug 6066 - Improve the logging."

Merge "Fix Sonar (soon Checkstyle) constant name"

Merge "BGPManager FIXMEs fixed"

Merge "Vpnmanager module sync up"

Merge "Update eos to use service module: entity-ownership-service"

Bug 6066 - Improve the logging.

Change-Id: Ia695cde0f870768d85405d389a85d77fd636e46f
Signed-off-by: ananthip <ananthip@hcl.com>

Vpnmanager module sync up

This commit contains upstreaming of VpnManger module alongwith dependent
commits into fibmanager and neutronvpn manager

Change-Id: I631d9f3652c863180804cdf426d15e6e3ea1cfbc
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Fix Sonar (soon Checkstyle) constant name

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

Update eos to use service module: entity-ownership-service

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

Merge "Finish renaming odl-netvirt-vpnservice features."

BGPManager FIXMEs fixed

SInce odl-fib.yang is updated now, dependent classes in FIB are available,
hence the FIXMEs have been fixed.

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

Finish renaming odl-netvirt-vpnservice features.

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

NeutronVPN module sync up

Added 9 FIXMEs in VPNManager which came up as errors upon updating the
yang models for dependent NeutronVPN code.
This will be fixed in the upcoming VPNManager module sync up commit.

Fixed unit test

Change-Id: I203a26a5650a3b32ee5d0d3767894eed78a759f1
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Change in NextHopManager.java

After name modification of one of the constant in ITMConstants.java
file in the commit https://git.opendaylight.org/gerrit/#/c/42034/2 ,build
of netvirt was failing.

This review gives a fix for that.

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

Merge "Add line tracking for UT"

Add line tracking for UT

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

Merge "Add Host Config functionality for Neutron"

Merge "Upstreaming DHCP L2Gateway connection removal fix"

Merge "FibManager module sync up"

Merge "Reduce IT logging"

Merge "Skip v4 subnets/ports in ipv6service"

Skip v4 subnets/ports in ipv6service

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

Merge "Add new feature called odl-netvirt-openstack to replace odl-netvirt-vpnservice openstack"

FibManager module sync up

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

Reduce IT logging

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

Remove odl-netvirt-* features.

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

Add new feature called odl-netvirt-openstack to replace
odl-netvirt-vpnservice openstack

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

Upstreaming DHCP L2Gateway connection removal fix

- Earlier when the l2gwconnection config is removed, the dhcp flows were
  deleted but if it is added again before the Logical Switch gets removed
  from the TOR, an event is never received for adding back the DHCP
  entries and we will never have any designated DPN. This has been fixed
  by moving the remove logic to the Logical Switch listener itself.

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

The odl-sfcofl2 feature has been renamed

- Now its called odl-sfc-openflow-renderer

Change-Id: I7382e8aa910c8ca63c09d3972eea6eb93d7cb97c
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Add Host Config functionality for Neutron

Patch to read host configuration data from OVSDB
external_ids and populates ODL neutron northbound
mdsal data as per yang model defined in the patch
https://git.opendaylight.org/gerrit/#/c/36767/.
Complete details of the feature available at
https://github.com/openstack/networking-odl/blob/master/doc/source/devref/hostconfig.rst
Patch for original netvirt code
https://git.opendaylight.org/gerrit/#/c/40143/

Change-Id: I7da8c8e5cc18d661b21c70381990b7c2a7ebb40c
Signed-off-by: Ritu Sood <ritu.sood@intel.com>

Fixing minor compilation issue

- addTerminationPoint was ambiguous and needed a simple cast to compile

Change-Id: I581fbc117da4f3eb5c1434e25ddae3dc3cc687fc
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Merge "Adapt Natservice for ECMP"

Adapt Natservice for ECMP

Adapted Natservice BGP APIs to support ECMP. Minor changes in log messages and log levels to make it relevant to Natservice context.

Change-Id: I2f8f06b0c055d8013b3fed35f7056de10625a687
Signed-off-by: Sumanth MS <sumanth.ms@ericsson.com>

support ace(security rule) add/remove in egress/ingress service

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

Merge "TEST: DO NOT MERGE"

Merge "Fix IT failure when not running DockerOvs docker"

TEST: DO NOT MERGE

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

Temp disable failing ipv6 test

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

Fix IT failure when not running DockerOvs docker

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

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

Merge "Move old netvirt files into openstack dir"

Move old netvirt files into openstack dir

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

Merge "Bug 6066 - Improve the logging."

Merge "Multiple fixes in different module"

Merge "BGPVPN optional RD,L3 Type and Union of routelist"

Merge "Minor: Constants should be upper case"

Bug 6066 - Improve the logging.

Change-Id: If0537bf9318977d204cac4866ebec6c3ea7aef5f
Signed-off-by: ananthip <ananthip@hcl.com>

Multiple fixes in different module

This commit includes following changes and bug fixes:

  - Additional check when handling operational ds of vpn interface
  - Performance Issue: Association/Dissociation of router takes
    huge time.
  - All Internal VPN(ROUTER) Ip's not coming on deletion of L3VPN
  - NextHop for SubnetRoute not re-programmed in DCGW after a DPN
    disconnect.
  - NextHop for Subnet route not re-programmed on DC-GW.
  - Some of VM entries and Invisible IPs of VM are not in FIB
  - Fixed disassociate of floating ip
  - New Invisible IP is not reachable from
    DC/DC-GW after reboot

Change-Id: I3b0e6cc1fc9f8396d93b5391efaaefcac4b8eabd
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

BGPVPN optional RD,L3 Type and Union of routelist

Update to use idmanager form genius

Change-Id: I11c258d8f58ac6531b26f0f979dd93a863895da9
Signed-off-by: sathish kumar B T <sathish.kumar.tippeshappa@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Minor: Constants should be upper case

incl. LOG instead of logger; see also proposed new
https://git.opendaylight.org/gerrit/#/c/41289/

Change-Id: I049ee845062a1ff5b7f8e54be250d7bb9e25930c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Support Neighbor Advertisement functionality for router iface

This patch supports NA for router interface (including LLA).

Change-Id: I4d3d4c41dec12abef3b1d04c067a2e21930ed021
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>

Fix for local build to pass

After merging of following commit
https://git.opendaylight.org/gerrit/#/c/41817/
local build of netvirt was failing.

This patch gives a fix for that.

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

Merge "Fix natservice default-config file."

Merge "support for interface update and acl update"

Merge "L2GW: Updated to support dynamic port additions to TOR and stale entry cleanup"

Merge "BGPManager module sync up - 2"

Merge "Fix UI bundle dependency"

Merge "workaorund blueprint restart for NetvirtSfcIT"

Fix natservice default-config file.

Vpnservice fails to push config to natservice.

Interfacemanager moved to genius project.
There is no mdsalutil-api yang model in genius project, so
removed that config dependency.

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

support for interface update and acl update

1. handled interface update (security group added/deleted, port security
flag enable/disable)
2. handled acl update (security rule is added/deleted from security group)

Change-Id: I22b1aaf7d94dae014b3d140d57499a4bd7c95313
Signed-off-by: gujele <imanilyadav@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Bug 6066 - Improve the logging."

L2GW: Updated to support dynamic port additions to TOR and stale entry cleanup

- HwvtepPhysicalLocatorListener renamed to
  HwvtepTerminationPointListener is also being used to support
  dynamic port additions
- HwvtepTerminationPointListener updated to listen only for Hwvtep
  terminal point changes
- Fix of removing stale entries in network topology config
  datastore when HWVtep switch is removed

Change-Id: Ib5c54327b90547c32c6f2cc8423d215875e945a6
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

workaorund blueprint restart for NetvirtSfcIT

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

Fix UI bundle dependency

ui module does not need to be packaged in the distro, change
the scope of dependency to provided.

Change-Id: I0c2efbe682229c4fb853bab31c89de6a67048cac
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Fix for access-list model changes

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

BGPManager module sync up - 2

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

Bug 6066 - Improve the logging.

Change-Id: I4db00f7bfad567fa06aeeec13a62ec082c558677
Signed-off-by: ananthip <ananthip@hcl.com>
(cherry picked from commit 5ebe4d13e0bda843f479347d92c9728f358532b5)

Fix genius's odl-interface.yang revision

Change-Id: Ic10ede2c5c70bc21985b26c607b066072c5a8cc0
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: I96d4979bc79127a7c3a6e41920f7ce70967f61d7
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Merge "Fixes for the below NAT issues : 1) After cluster reboot, SNAT traffic is lost 2) Delete Router doesnt remove ext fixed IP fib entry and also corresponding entries from table 3) Inbound NAPT table (table 44) entries not removed after Outbound NAPT table (table 46) translation entries are removed when bgpVPN associated to router. 4) Packets punt to the controller for programming the NAPT entries are getting dropped. 5) Nullpointer exception in some cases, 6) Tracing hooks not available for the interthread communication which handles the NAT traffic."

Merge "Supporting DHCP as a Service"

Merge "BGPManager module sync up"

Merge "Multiple fix in different module"

Merge "Support Router Advertisement functionality when an RS is received."

Merge "ipv6service: Register listeners and configure handlers"

Merge "AclService Custom Security Group"

Merge "Neutron port listener updated to support allowed address pair with security group configurations."

Merge "Dhcp Tenant Support for VM's on CSS and TOR"

Fixes for the below NAT issues :
1) After cluster reboot, SNAT traffic is lost
2) Delete Router doesnt remove ext fixed IP fib entry and also
corresponding entries from table
3) Inbound NAPT table (table 44) entries not removed after Outbound NAPT
table (table 46) translation entries are removed when bgpVPN associated to
router.
4) Packets punt to the controller for programming the NAPT entries are
getting dropped.
5) Nullpointer exception in some cases,
6) Tracing hooks not available for the interthread communication which
handles the NAT traffic.

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

Merge "L2GW: Handling during delete and add of ITM:transport-zone."

BGPManager module sync up

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

Support Router Advertisement functionality when an RS is received.

This patch implements the following.
  1. When an Router Solicitation is received from a VM, it
     looks at the subnet/router info, frames the corresponding
     RA and does a packet-out. VM on receiving the Router Advt
     will be able to successfully configure the IPv6 address
     based on the advertised prefix.
  2. This patch supports SLAAC, DHCPv6-Stateless and DHCPv6-Stateful
     subnets along with support for Multiple prefixes.

Pending:
  1. Periodic advt of RA when a v6 Subnet is associated to the Router
     will be taken up in a different patch.
  2. IPv6 Service Binding will be added via a different patch.

Change-Id: Id889aae707818c10d7c2d6cc675bd22a99943c42
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>

Multiple fix in different module

This commit deals with following:

  - Modified VrfEntry yang model to support augmentation for elantag
  - Removed the rdtoElanOpData references from FibManager
  - Addressed review comments and removed rdToElanreferences
  - Fix for SubnetRoute handler

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

ipv6service: Register listeners and configure handlers

This patch includes the following changes.
1. Registration api's for Neutron Router, Subnet and Port changes.
2. Handlers for interface manager.

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

ELAN Performance improvement changes

     perform bulk write for flow/group programming for ELAN.

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

AclService Custom Security Group

1)TCP, UDP ,ICMP and Other protocl support added
2)ietf acl to flow converter added.

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

Neutron port listener updated to support allowed address pair with security group configurations.

Change-Id: I8328ba7c5154418d3f5de3aeadb48c0b1eeaba98
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

Merge "Modification in NeutronPortChangeListener.java"