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"

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>

Fix bad single { instead {} in logger message

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

Merge "remove unnecessary @Override close() "

Merge "Remove redundant "public" from Java interface"

Merge "Added existsVPN API and refactored NeutronVPN"

Fix typo in factory-method name

Change-Id: I7bfd07cf254a4cf3917ae1c9a5aae90bf74efb22
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Signed-off-by: Alexis de Talhouët <alexis.de.t@gmail.com>

Added existsVPN API and refactored NeutronVPN

1. existsVPN API added in VPNMgr, removed from NeutronVPN
2. Indentation, logging, import statement changes
3. Removed unused router-dpns yang model
4. Added notifications for router assc/dissc to/from VPN
5. Refactored a couple of validations

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

Remove x2 catch (Exception e) logger.error

Unless I'm missing something, this has no real added value (as the
caller would ultimately catch runtime exceptions anyway), and could
potentially hide other errors.

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

Fix very bad exception handling in AclServiceUtils read()

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

Remove redundant "public" from Java interface

I'm about to enforce this as a Checkstyle rule in
https://git.opendaylight.org/gerrit/#/c/41302/

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

remove unnecessary @Override close()

In InterfaceEventListener, the super.close() already has LOG. (And it's
bad anyway to use only e.getMessage() instead of e anyway; I'll be
extending Checkstyle to not permit this anymore.)

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

Load config system files in etc/opendaylight/karaf

There is where config system file should be loaded

Change-Id: Iec1a91f89f9f0effd95283c4d95d9d56fafa43a7
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Signed-off-by: Alexis de Talhouët <alexis.de.t@gmail.com>

Merge "Instead of augment hwvtep, added a new entity which references the logical switch ETree YANG model cosmetic change"

Fix breakage due to aclservice yang models changes

This patch changes yang models https://git.opendaylight.org/gerrit/#/c/41234/2
and broke netvirt.

Change-Id: Iae6c2623a58d6e9da6f24f7c70c023dc5d7e6a7a
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Initial changes for IPv6 Service"

Merge "clean aclservice files"

Merge "added Security Rule listener to handle coversion from security rule to acl model"

Instead of augment hwvtep, added a new entity which references the logical switch
ETree YANG model
cosmetic change

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

clean aclservice files

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

Merge "Support for IT ping feature"

remove unused features to shorten build time

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

Support for IT ping feature

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

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

Merge "Multiple fixes in neutronvpn code"

Merge "Bug 5860 Fix port event wrong ordering"

Merge "ACL yang updated to include allowed-address-pairs with Interface augmentation"

Multiple fixes in neutronvpn code

Commit includes:

1> Fixing NPE in NeutronVpnUtils when no network-type is provided from
northbound

2> NVPN: neutron northbound support for gateway mac for external
networks

3> Use vif_type to determine portname prefix

4> NeutronVPN: Handling fixed-floating IP dissociation for floatingIP
delete

5> NeutronVPN additional -ve validations via ECM for network/router
assc/dissc

  a. Added validations for the following:

  a1) Associate network(s) to vpn:
    i) network already assc to another VPN
    ii) network's subnet added to some router as interface

  a2) Dissociate network(s) from vpn:
    i) network's subnet added to some router as interface

  a3) Associate router to vpn:
    i) router assc to another vpn
    ii) network corresponding to router's subnet interface already assc to VPN

  a4) Dissociate router from vpn:
    i) router assc to another vpn

  b. Updating VPNMaps once assc/dissc are done in assc/dissc network(s) from
     VPN

6> NeutronVPN: adding yang model and notifications for router-dpn
map

  a. Added a container to maintain mapping between neutron router and
  DPN(s) on which vpn-interfaces for router are present.

  b. Added notifications for router associated/dissociated to/from vpn

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

Merge "Neutron port listener updated to handle security group configurations."

ACL yang updated to include allowed-address-pairs with Interface augmentation

Neutron supports default anti-spoofing capabilities with
allowed-address-pair Neutron port attribute which also gets
enabled/disabled with port_security_enabled. As this is based on SG flag,
updating ACL yang to support same with Interface augmentation.

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

Bug 5860 Fix port event wrong ordering

If the southbound event arrives before the northbound event the
tenant network is not there yet and therefore the flows for
this port will not be created.

This patch add new logic to the PortHandler (northbound event)
to verify if the southbound event was already arrived and to
add the missing flows.

Change-Id: Id8ad03b6de8afff0b64ca3af3457281ddbaf0f74
Signed-off-by: oshvartz <oshvartz@redhat.com>

Merge "Added support for fixed Security group."

added Security Rule listener to handle coversion from security rule to acl model

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

Neutron port listener updated to handle security group configurations.

During Neutron port add/update, Security group details are updated to corresponding ACL Config DS.

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

Initial changes for IPv6 Service

Change-Id: Ib10708e7010360eacb00f89655f0dbb59359a6c5
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>
Signed-off-by: Mohnish Anumala <Mohnish_Anumala@Dell.com>

Add blueprint wiring for neutron renderer

Change-Id: I00fcd2f7d661ec38802974fc39da4a4009b37d43
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Added support for fixed Security group.

1) Added DHCP and Arp rules
2) Added default connection tracking rules.

The rules will be added to the ovs only after integrating with genius.

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

Add blueprint wiring for openstack/net-virt-providers

Change-Id: If0d6691d9cd77666c65c2c4a2751acfbae660572
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for openstack/net-virt

Change-Id: I0d32881db4c1ee939f87441d75e38dbe1c34374f
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Add blueprint wiring for openstack/net-virt-sfc

Change-Id: I0358dbbeacdf3a734b721012a73fa790cdeb57ff
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Remove getModuleName()/getInstanceName() in IT

see: https://git.opendaylight.org/gerrit/#/c/39891/

Change-Id: I08725e0ce15697fee8ad76f7ce5d51bfdfd1f816
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Fix breakage due to SFC yang models changes

This patch modify many yang models used by Netvirt-SFC
https://git.opendaylight.org/gerrit/#/c/40209/

- ietf-access-control-list.yang is now in MDSAL
review mismatch and pom dependency mismatch

- ietf-packet-fields.yang
has been move to MDSAL
is using a newer version of ietf-yang-type

Change-Id: I6bec81c70cd5ef2e056eb923cf5ebebd7d8ddaf1
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Add postman collection with restconf requests to get datapath id's and create transport zone when using the VPN Service NetVirt."

Merge "IT fixes due to blueprint migration"