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"

Merge "Change OFP feature dependency"

Merge "Uncomment install/removeFlow"

Clean code: format, typo, new lines (very minor)

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

Merge "Fix potential null issue by propagating Optional"

Merge "Load config system files in etc/opendaylight/karaf"

Modification in NeutronPortChangeListener.java

It is always suggested that one listener should not be function of some
other listener.This commit takes care of this in
NeutronPortChangeListener.java file.

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

Uncomment install/removeFlow

I cannot write a test for aclservice if actually it does NOTHING. The
current implementation just pushes some bits from here to there
internally, but ends up doing nothing.  I'm working on a showcase test
that will demonstrate how I would test a module such as aclservice. It
will show how to test the EFFECT of an API, and for that, it must DO
something real... ;-)

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

Change OFP feature dependency

 - changed odl-openflowplugin-nsf-services to odl-openflowplugin-flow-services
   since the first will be removed. Actual behaviour should be still preserved.

Change-Id: I05b20c96be446c157e362fa0f5fd66d93cd771c2
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Supporting DHCP as a Service

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

Merge "ACL ingress/egress service bindings"

Dhcp Tenant Support for VM's on CSS and TOR

Change-Id: Ic795115317af112c61aa57ef64306cc1cda10ef0
Signed-off-by: sathish kumar B T <sathish.kumar.tippeshappa@ericsson.com>

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

+ Added L2GwTransportZoneListener. When a new transport zone (of vxlan
type) is created, add all l2gw devices to the transport zone. This would
mesh all the nodes in the transport zone.

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

ACL ingress/egress service bindings

+ ACL ingress/egress service binding implementation is done but currently
the call to bind/unbind services are commented.
This should be uncommented once ACL related flow programming is complete.

+ Service priorities have been updated for L3VPN and ELAN.

+ Added table miss entries for both ingress and egress ACL tables during
node up.

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

Amendments to displayNeutronPorts CLI + listed changes

Following things are taken care of:

a. Displaying neutron ports without fixed IPs
b. Displaying neutron ports with multiple fixed IPs
c. Displaying neutron ports with IPv6 fixed IPs
d. Additional error handling to fail only one the failing ports, not
all, in case of any.

Also, done changes for the following:
i) neutron port addition/removal/updation in case of fixedIP not
present is handled.
ii) Add/remove of port to cache modified to be populated for all use cases

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

Merge "Fixes for floatingIP and cluster reboot issue for int/ext VPN"

Merge "Auto-creation of bridges in vpnservice"

Fix potential null issue by propagating Optional

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

Fix in NeutronVpnProvider.java

This commit provides fix for null pointer exception due to the reason
floatingIpListener was being used by portListener, before it was
initialized.

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

Fixes for floatingIP and cluster reboot issue for int/ext VPN

1. FloatingIP support for assc to fixedIP while creation added
2. Cluster reboot issue for internal VPN fixed by separating out int/ext
VPN (assc/dissc of router to/from VPN) codepiece and checking for presence
of VPN upon reboot first.
3. Minor indentation changes

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

Merge "Remove x2 catch (Exception e) logger.error"

Merge "Fix bad single { instead {} in logger message"

Merge "Remove (apparently) un-used AclService interface"

Auto-creation of bridges in vpnservice

When a new OVSDB node is detected, the
event is caught and we make sure that
br-int and br-ex (if l3 is configured)
exist. Note that br-ex is not technically
required and once devstack is modified
to not check for it, we should go back
in to this code and no longer create it.

This code add dependencies to the
following ovsdb project utility bundles
and jars:
- southbound-api
- utils.mdsal-utils
- utils.southbound-utils
- utils.config

Bug for removing checks for br-int when
running devstack:
https://bugs.launchpad.net/networking-odl/+bug/1599853

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

Merge "Fix very bad exception handling in AclServiceUtils read()"

Remove (apparently) un-used AclService interface

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