Bug 8031: Fix NPE from VrfEntryListener

Change-Id: I699b0ab3de8f6aa7079f1de969867a866a1cd042
Signed-off-by: Janki <jchhatba@redhat.com>

Correct several equals() bugs

Change-Id: I8f008515dc79cdd0c38e957a3917db1ce8d9ab86
Signed-off-by: Alon Kochba <alonko@hpe.com>

Elan base code changes

Change-Id: I58a9538826e7b9cbed2a05638e9308d2f2a4318b
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7988 - Cluster reboot fix

Elan Instance , Physical Switch , L2gateWay and L2gatewayConnection all
have dependency among each other.But if upon cluster reboot the order is
not proper (L2gw cache is updated at the last , then flows are not being programmed as expected.
To take care of the situation
Introduced new ElanInstance Listner (Was already present but not
registered with xml to be instantiated )
Handle scenario if L2gW shows up last after cluster reboot .
Handle scenario if l2gwConnection is updated but Elan instance is not
available.

Change-Id: I6194d7cb22d6659803b17078d57c8ad694c0c201
Signed-off-by: Akash Kumar Sahu <a.k.sahu@ericsson.com>

Fix ACL IPv6 flows to match on ipv6_src/ipv6_dst for remote SG

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

Added a new framework for selecting centralized SNAT router.

Added a new HA framework for selecting the centralized SNAT router which
will use a weighted round robin to select the NAPT switch

A pseudo port will be added once the router is elected as the designated
switch and will be delete on router delete.

The switch selection/release happens in response to external router
create/delete or snat enable/disable

The flow programming will be done in response to   changes in
centralized switch model RouterToNaptSwitch.

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

Add NPE protection for VpnFootprintService.removeOrUpdate...

 + Refactor the method to be a little bit more 'flatter' by adding
   early returns

 + Create the writeTx only when needed.

Change-Id: Ib760a20ed0ea561a05fbdd8d56715d1341cec7fd
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

AclServiceStatefulTest asyncEventsWaiter before coordinatorEventsWaiter

This came up during local investigation of today's aclservice test
failures following c/51783 (reverted in c/53827), and ultimately wasn't
the root cause - but doing this is probably still a good idea.

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

Bug 7606: Fix for missing table 110 flow in OVS 2.4 after VM live migration

Problem:
After live-migration of VM is completed, table 110 flow for migrated
instance missing in all nodes.

Fix:
* Skipped UPDATE event of Source OVS when VM is migrated.
* Skipped the cache update from UPDATE event.

Change-Id: Iddc15a38fc8c96e998a14622217957dcabdff438
Signed-off-by: arthi.b <arthi.b@hcl.com>

Added Conntrack based SNAT implementation.

Added an interface for the Snat operation.
The interface shall be implemented by the multiple snat implementations.
Only conntrack based implmentation uses this interface now.

AbstractSnatService has the flow programming which are common
across multiple implementations of SNAT. The code handles only flat/vlan
use cases now.

ConntrackBasedSnatService provides conntrack based SNAT implementation
for flat/vlan provider networks.

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

Bug 8019: when the neutron port acting as gateway is deleted, invisible ip
is not removed from FIB

When VpnInterface has a secondary adjacency like invisible ip, if tunnel
EP is deleted , VpnInterface Op DS is updated with empty nexthoplist.
During this time , invisible ip goes missing from VpnInterface Op DS due
to MDSAL.syncwrite() being done in updateVpnInterfaceOnTepDelete.

Another fix being done here is , invisible ip FIB entry is updated with empty
nexthop and route is withdrawn/advertized to BGP.

Change-Id: If98991940976defe16a7a988aa3071d87693f8e9
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

Bug 7997, 7939:  Groups not going in before the flows to switch.

+ Increased the delay between group and flow pushing to 1500ms.
+ Expanded some critical INFO statements.

Change-Id: I0c8ace294b29c08ebe88ec982cdcd12613a66d04
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

AclServiceTestBase with LogRule and (commented out) RunUntilFailureRule

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

Rename AclServiceStatefulTestIPv6 to AclServiceStatefulIPv6Test

so that 'mvn test' runs it (surefire is based on *Test naming)

but @Ignore it, as it's currently completely broken (TODO fix it)

and patch IdHelper to show cause of NPE occuring in this broken test

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

Add missing <scope>test to aclservice/impl/pom.xml

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

TunnelState changes for tunnels towards DCGW

Changes for programming the load balancing group for traffic towards
DC-GW.
 - When tunnel is added, based on the number of DC-GWs we create all
   possible load balancing groups for the traffic towards DC-GWs.
 - When tunnel is updated, we read the dpid-l3vpn-lb-nexthops and the
   l3vpn-lb-nexthops container to figure out the group which needs to be
   modified and update the bucket directly in that group.
 - When tunnel state is deleted, we delete the entry from both the
   containers and also delete the group if only one DC-GW is left.

This review considers only two DC-GWs for now.

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

NeutronVPN changes to propagate VNI to VPNManager

Adds support to populate subnetmaps DS with network attributes and
propagate the same to SubnetRouteHandler via notifications sent upon
updates to subnetmap DS

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

Revert "Bug 7775: Updated to use DataStoreJobCoordinator for flow programming"

This reverts commit ddfe3934e4df1dce0c54b32a1cd0b1106852852a.

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

Spec for L2vpn over vxlan using EVPN RT2

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

Spec to support IPv6 DC to Internet L3VPN connectivity using BGPVPN

This blueprint spec explains architecture for supporting IPv6 Datacenter
to Internet connectivity using BGPVPN in a SDN controlled by OpenDaylight.
Enhance option b) specification as explained in [0] so that it is possible to
have separated vpns for private inter-dc and for internet access. So that
private vpns keep being private, while an other vpn instance will be publicly
opened.

[0] https://drive.google.com/file/d/0BxAspfn9mEi8OEtvVFpsZXo0ZlE/view

Change-Id: I99b6718d6868fe8b0c49081e2f4cf3897ea1e3f6
Signed-off-by: Julien Courtat <julien.courtat@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Add demo for openstack + new netvirt

Change-Id: I4d9c816ca20c2659f870d7767f01667254ea42ce
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Fixing compilation breakage.

- There were dependencies on Genius that were changed.

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

Bug 7775: Updated to use DataStoreJobCoordinator for flow programming

Updated syncFlow() in AbstractAclServiceImpl.java to use DJC while flow is
added/removed to serialize/parallelize flow programming based on device
id. This also avoids possibility of OptimisticLockException getting
generated from ACL while flows are programmed.

Also, Interface state listener updated to use DJC to serialize/parallelize
interface state processing changes.

This review is dependent on below genius patches:
https://git.opendaylight.org/gerrit/#/c/53143/
https://git.opendaylight.org/gerrit/#/c/53352/

Depends-On: I8c15af6607f83a08354520872e4a5ff0ea16d2bf
Change-Id: I578cdf332d8e5575ac31b3b7c5fdf9d7ea86013d
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Bug 7913: QosInterfaceStateChangeListener IllegalArgumentException

Fixed it by avoiding IllegalArgumentException through regexp check.

Change-Id: I8c474eae6629a7219a524a5de3cd92e087c6e746
Signed-off-by: A Vamsikrishna <a.vamsikrishna@ericsson.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 7816: NullPointerException while create a router in external network provider

 * Problem:  Patch port for br-ex is not updated with port name = "patch-ex" into
             localcache during stack up for follower node, thus NullpointerException.
 * Solution: Patch port name is constructed by passing external bridge name and
             as it was constructed on fly in getPatchPortName() in VLANProvider.java,
             patch port name entry was not stored in 'patchPortNames' map.
             However, 'patchPortNames' map will have default value as "patch-ext" for
             patch port name.Default value will be used
             when 'ovsdb.l3.fwd.enabled=yes' in custom properties file.

Change-Id: I9e446529c1962b41c9e97644d59a35dea4606740
Signed-off-by: hari.pr <hari.pr@hcl.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8054: Fix IndexOutOfBoundsException in VrfEntryListener

Issue:
The exception occurs in case handling cleanupDpnForVpn in case of the
prefix not being an extra route as the usedRds is being tried to be
accessed to fetch the extraRoute(it is absent for the non-extra route
prefixes) and so throws the exception.

Fix:
A condition has been indroduced to fetch extraRoutes only if the prefix is
extra route else extraRoutes is set as absent.

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

Add setPeerSecret to thrift service BgpConfigurator

A new function 'setPeerSecret(ipAddress, secret)' has been
added to thrift service 'BgpConfigurator' inorder to
configure the shared secret needed by TCP MD5 Signature
Option.

See spec qbgp-tcp-md5-signature-option [0].

[0] https://git.opendaylight.org/gerrit/#/c/52218 Ibdb0675c0045754e575b73deba2dfebde4b56bc5

Changelog:

- qbgp.thrift: add function 'setPeerSecret' to service
  'BgpConfigurator'. Add BGP_ERR_NOT_SUPPORTED error code.

- Update generated file BgpConfigurator.java.

- Update generated file qbgpConstants.java.

- BgpRouterException.java: add specific messages for
  setPeerSecret.  Add BGP_ERR_NOT_SUPPORTED message.

- BgpRouter.java: overload BgpRouter.addNeighbor.

Change-Id: I3e19372e29dd1e83f1ea6e394bf7cc941de9d9a6
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Code changes to support qos-rule-types

Change-Id: I80587e96ed7d200b788a3f8ac981e7b46b5d3f47
Signed-off-by: chandrag <chandramohan.grandhi@tcs.com>

Bug 7912 - ACL issue related to AAP with prefix 0.0.0.0/0

+ Supporting AAP with prefix 0.0.0.0/0 for remote security group rules
would lead to a potential security breach. This would result in allowing
the traffic from all the IPs for a VM.

+ This fix removes support for AAP with 0.0.0.0/0 as part of remote
security group rules/flows configuration.

+ Support for AAP with 0.0.0.0/0 will be retained only for anti spoofing
flows which are configured in table 40/251.

Change-Id: Ib1e17cd3007b05f6c6d2bb4c8514c667380c1b7e
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Fix RemoveVpnPseudoPortJob logger name

 + Since it gets very confusing when reading the logs

 + And I take the chance to add some more debugging traces to
   VpnServiceChainUtils.

Change-Id: Ic6b6712f2c4c74d041fd68a73cfb6ef0bd8b5caf
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Bug 7606: Fix for missed tunnel flows, after VM live migration

   * Skip deleting the flows in table 110 and table 20 when processing an
    interface deletion event from OVSDB in the case that the interface
    deletion is due to VM migration.
   * Remove the Port Name from flowName so that, the same entry gets
    updated during the update notification processing while live migration.

Change-Id: I2b917e50e07af4ed840d811a9c3f0a131281bd12
Signed-off-by: VinothB <vinothb@hcl.com>
Signed-off-by: arthi.b <arthi.b@hcl.com>

Fix the metadata programming/recovery for subnetroute

The following genius review
https://git.opendaylight.org/gerrit/#/c/53546

alters the subnetroute metadata mask. So we have
to fix subnetroute handlers to align with the
new metadata mask for subnetroute.

Change-Id: I9855c20f0af4c51b03b76ece2604cbf328739236
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Add --tcp-md5-password to bgp CLI

Optional option '--tcp-md5-password' has been added to karaf
console commands 'odl:configure-bgp' and 'odl:bgp-nbr'.

See spec qbgp-tcp-md5-signature-option [0].

[0] https://git.opendaylight.org/gerrit/#/c/52218 Ibdb0675c0045754e575b73deba2dfebde4b56bc5

Changelog:

- BgpConfigurationManager.java: Overload
  'BgpConfigurationManager.addNeighbor'.

- BgpManager.java: overload 'BgpManager.addNeighbor'.

- ConfigureBgpCli.java: add option --tcp-md5-password to command
  'odl:configure-bgp'.

- Neighbor.java: add option --tcp-md5-password to command
  'odl:bgp-nbr'.

Change-Id: I933177239b54fbc370d4a97901cf423cade36def
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

First version of New Netvirt SFC classifier

- Only contains the class structure and minimal impl so far
- Fixed code review comments

Change-Id: Idc9ae549b5a42a2c528e3f14b80d56a1030d19d5
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7863 - Add Layer 4 Match for flow entries for TCP/UDP security
           group rule with no min/max

The methods Ingress/EgressAclService::ingress/egressAclTcp/Udp
incorrectly assume that portRangMin/Max values are always specifed.
This is not true when using openstack command lines to
define SG rule for TCP/UDP with no min/max ports specified.
As a result, no layer 4 match are added to the flow entries.

The fix is to add logic to handle null min/max values for the
above scenarios.

Change-Id: I537f51fdd56d8aca3711a38cd5fd2c5eba5d9af9
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Spec for TCP MD5 Signature Option of QBGP

This spec provides documentation of "Support for TCP MD5 Signature Option
configuration of Quagga BGP".

Change-Id: Ibdb0675c0045754e575b73deba2dfebde4b56bc5
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Bug 8023 Handling ELAN remote DMAC programming correctly

* Fixing the issue related to synchronization key for programming remote
dmac entry
* code cleanup while handling tunnel add DCN
* added required info logs to collect the logs to know about possible
races. Will get it removed once this issue is completely solved.

Change-Id: I82d3cc1a44696d9dfae531fac818295ba75d28a9
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Blueprint for new-sfc-classifier

- Added Genius integration info provided by Jaime
  Camaaño Ruiz
- Work items added by Miguel Duarte De Mora Barroso
- Addressed code review comments
- Added detail about ACL network-uuid match field
- Added detail about services, priorities, and last hop
- Finalized the source directory, karaf feature name,
  a few other details.
- Added classifier and sff pipelines
- Minor syntactical improvements
- Ingress SFC service wont resubmit to Ingress, only to Egress
- Added link to Google doc slides for reference only

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

Fixed element counters doc format

Change-Id: I11a53567173f6ad2a7b9320ce24b4c0958ed268e
Signed-off-by: Guy Regev <guy.regev@hpe.com>

Cross compute traffic for external network vpn will go over flat/vlan provider network port

This is part of the Hairpinning tasks.
Set the egress actions to go to the provider network for floating IPs.
Change floating IPs FIB table flows to use the provider network port as action.

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

Ethernet VPN (EVPN_RT5) Enhancement to FIB for NAT

This code review (5th Review) is part of EVPN_RT5 enhancement to NAT
Feature.

This code changes is responsible for installing remote FIB (L3_FIB_TABLE) entries for external IPs
(Floating IPs and External Fixed IPs) for EVPN_RT5 (VXLAN) networks only.

Change-Id: I49a43b26f6aa869a126714e46e6ab202c3dfefbb
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Hairpinning pipeline, extSubnetId as vpnId and resubmit to FIB.

This is part of the hairpinning tasks - Changes for flat/VLAN external networks:
Resubmit SNAT and outbound NAPT flows to FIB table.
L3_GW_MAC_TABLE floating IP flows add set vpnId metadata to extSubnetId.
PSNAT_TABLE floating IP flows - set vpnId metadata to extSubnetId.
SNAT_TABLE floating IP flows - match vpnId metadata of extSubnetId.
FIB_TABLE floating IP flows - match vpnId metadata of extSubnetId.
OUTBOUND_NAPT_TABLE routerGW flows - set metadata to routerGwSubnetId.
NAPT_PFIB_TABLE change match to be extSubnetId instead of networkId.
INBOUND_NAPT_TABLE routerGW flows - set metadata to routerGwSubnetId.

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

Ethernet VPN (EVPN_RT5) Enhancement to DNAT

This code review (3rd Review) is part of EVPN_RT5 enhancement to NAT
Feature.

This code changes is responsible for below table flow creation and
removal.

1) Install the flow 36->25 (SNAT VM on DPN1 is responding back to FIP VM on DPN2)
   {SNAT to DNAT traffic on different Hypervisor}

2) Install the flow 21->25 (FIP VM1 to FIP VM2 Traffic on Same Hypervisor)
   {DNAT to DNAT on Same Hypervisor}

3) Install the flow 19->25 (DC-GW is responding back to FIP VM)
   {DNAT Reverse traffic})

Note: This review is dependent on below review
https://git.opendaylight.org/gerrit/#/c/51338/

Change-Id: Ic9a9546804f808223d53dd67e3a969e0d26ca793
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Bug 7939:  VpnService Suite and Tempest failures

This fix is an add-on patch for the below review

https://git.opendaylight.org/gerrit/#/c/53355/

and is in a direction to solve the OperTxn gets
failing when configTxn changes are triggered.

Also we fix here the routerInterface VrfEntry
management such that when we clearly know its a
router-interface VRFEntry, we donot do any further
processing than what is required for the same.

Change-Id: I8efc5c41f26e9e6dd04c8638b2cfaa94a2050384
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Added element counters api bundle

Change-Id: I5790c5f9943399575e9f78705107c39af1bf8f7f
Signed-off-by: Guy Regev <guy.regev@hpe.com>

Port sfc-translator to new netvirt.

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

AbstractDataChangeListener clean-up (2)

Previous patch missed classes using AbstractDataChangeListener

Reference:
https://wiki.opendaylight.org/view/Weather#Genius_AbstractDataChangeListener_clean-up

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

Migrate l3vpn service docs to netvirt

Change-Id: I8686a9b127ea746ecb5bb9031bed70e9fec287d0
Signed-off-by: Colin Dixon <colin@colindixon.com>

Support for BGP multipath apis

Incorporates thrift idl changes for supporting BGP Multipath feature.
Specifically adds:
a) enableMultipath(1:af_afi afi, 2:af_safi safi)
b) disableMultipath(1:af_afi afi, 2:af_safi safi)
c) multipaths(1:string rd, 2:i32 maxPath)
d) Sets maximum number of paths for a given rd for multipath().
Karaf shell command support also incorporated.
Incorporated review comments.i Fixed checkstyle issues.
Disabled checkstyle checks for auto-generated files
Corrected copyright year in touched files

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

Ethernet VPN (EVPN_RT5) Enhancement to SNAT_HA

This code review (4th Review) is part of EVPN_RT5 enhancement to NAT
Feature.

This code changes is responsible for SNAT HA during NAPT Switch
down/restart scenario

1) Remove all the SNAT flows from old NAPT switch

2) Install all the SNAT flows on newly elected NAPT switch

Note: This review is dependent on below review
https://git.opendaylight.org/gerrit/#/c/51338/

Change-Id: I4201096772af11b111f7387bb1bfce1535ee0ca5
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Netvirt: Spec for enhancement to VLAN provider network support

Change-Id: Iaba120d7b3e2174fc59913546a82575e0ce8ba97
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>

Ethernet VPN (EVPN_RT5) Enhancement to SNAT

This code review (2nd Review) is part of EVPN_RT5 enhancement to NAT Feature.

This code changes is responsible for below table flow creation and
removal.

1) Install the flow 36->44 (FIP VM on DPN1 is responding back to external fixed IP on DPN2)
   {DNAT to SNAT traffic on different Hypervisor}

2) Install the flow 19->44 (FIP VM on DPN1 is responding back to external fixed IP on DPN1 itself)
   {DNAT to SNAT traffic on Same Hypervisor}

3) Install the flow 25->44 (If there is no FIP Match on table 25 (PDNAT_TABLE) then default flow to INBOUND_NAPT_TABLE)

Change-Id: Ibb210891fe5dd8d6e18527535a7510dde0eb0273
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Ethernet VPN Flow Programming for VMs in different Data Centers

This review brings in the code changes to program flows as part of
Ethernet VPN RT5. It handles inter-subnet connectivity between VM's in
different data centers.

Change-Id: Iec208441eedc4ba77c2d86d775984153c8e8ec08
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Added spec for element to element counters

Change-Id: Ib4d8a4558aaa8a83d54a91496a0b43d78c32b5dd
Signed-off-by: Guy Regev <guy.regev@hpe.com>

Bug 7984: IDLE_TIMEOUT check required in onFlowRemoved.

Description: When FlowRemoved notification is received,
We are taking care of deleting the INBOUND_NAPT_TABLE entry
and releasing the allocated external port for an external-ip.
This is valid only if onFlowRemoved received due to OUTBOUND_NAPT_TABLE
removed beause of IDLE_TIMEOUT. When Switch goes down, we re-elect
a new NAPT switch and re-install ongoing session flows. It is required
not to release the allocated external port during Switch down, but
should be done only during IDLE_TIMEOUT.

Initially, we have the check but been removed with review
https://git.opendaylight.org/gerrit/#/c/49214/. Also plugin support
to get the RemovedReason was not available with onFlowRemoved.

Now, we have the plugin support to get the removed reason which can be
used. https://git.opendaylight.org/gerrit/#/c/50335/

Change-Id: I5708fbd1755719ea0cdabe29eaf0787076a46f24
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Spec for Listener Dependency Helper

LDH(Listener Dependency Helper) avoids application overhead
of maintaining and checking for dependent objects

patch-7: removed tabs

Change-Id: Iaecd19f1fe17ead2cfb3f07407cca3d8aa79b4cd
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@ericsson.com>

Add odl-netvirt-federation feature to vpnservice features

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

Spec to support cross site connectivity with federation service

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

Spec to support policy based path selection for multiple VxLAN tunnels

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

Code adjustments for ITM yang changes for multiple VxLAN tunnels

NB: This change must be submitted together with genius yang changes
https://git.opendaylight.org/gerrit/#/c/51770/

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

Bug 7939, 7938, 7968, 7997: Potential fix for the four L3VPN bugs

While creating LocalFibEntry, localDpnId list is being returned empty
leading to local FIB entry not getting installed.
The fix ensures we log an error statement in case it happens and tries
to achieve processing config and inventory transactions for the
vpninterface once operational transaction is processed.

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

Yang changes for Inter DC connectivity using BGPVPNv6

This patchset includes the yang changes required to implement
Spec to support IPv6 Inter DC L3VPN connectivity using BGPVPN [1].

[1] https://git.opendaylight.org/gerrit/#/c/50359

Change-Id: I70fb5460bde2a852dc965525fb37ae8003bea109
Signed-off-by: Julien Courtat <julien.courtat@6wind.com>

Bug 7826: proper elan djc job retries

When ELAN BC programming fails due to TransactionCommitFailedException
then there is no way of retrying djc job due to syncInstall(Delete)Group. It is fixed now and this is very much needed in a scaled setup because of OptimisticLockFailedException on the same group by multiple elan interfaces

Change-Id: Iaa0524edec3f25e0433a91c802f9c8da0d642962
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Yang changes to support iRT/eRT for EVPN

Change-Id: I5ecf6bdbc5cd6b5101484a14cde66cec960c7ab2
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Bug 7966: Fix route origin for some vrfEntries after VM migration

 + It's been observed that when a VM migrates and there are static
   routes pointing to it, as it happens with the CGNAT, the origin
   of these static routes are changed from 'static' to 'local'

Change-Id: I87751263b66080d8200f682b49ac0dd14062bc61
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Spec to support IPv6 Inter DC L3VPN connectivity using BGPVPN

This blueprint spec explains architecture for supporting IPv6 Inter Data
Center connectivity using BGPVPN in a SDN controlled by OpenDaylight.

Change-Id: I8cb4d7bfcff6306f2552d2a73bfddb4983cc49c5
Signed-off-by: Julien Courtat <julien.courtat@6wind.com>

Minor yang changes in neutronvpn.yang

1. rename to isBgpVpn > for easier code reading and clarity
2. change nw types to uppercase > to make use of existing utilities to
fetch network types and avoid changing cases.

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

Spec for DHCP Server dynamic allocation pool extension

Change-Id: I7f9d45b6dabbbb0e56e8db7f41c24f2cbaba537e
Signed-off-by: Shai Haim <shai.haim@hpe.com>

Bug 7817 & Bug 7838: DHCP ARP flow is not added and irrelevant ARP flows
                     are installed in compute node.

   * DHCP ARP flow is not added in Compute node
     When VM's are created with multiple networks.
   * VM's are created across networks, irrelevant ARP flows are
     installed in compute node
   * Fix: Compared with segmenationID and write the flows for table 20.
   * This patch contains fix for both the Bug 7817 & Bug 7838.

Change-Id: Iab21d2fe391451dc1f6e9cc70db099105e990871
Signed-off-by: karthik <karthikeyen.p@hcl.com>

Enhancing adjacencies with PNF flag

This will be used to mark PNF adjacencies

This is part of "discovery of PNFs in Flat/Vlan
provider networks" spec.

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

Spec: Discovery of directly connected PNFs in Flat/VLAN provider networks
This features enables discovering and directing traffic to
Physical Network Functions (PNFs)
in Flat/VLAN provider and tenant networks, by leveraging Subnet-Route feature.

Updated the spec, hopefully with better explanations.

Added a note regarding preservation of src MAC of VMs in two tenant networks
use case, to prevent MAC movments in the underlay switches.

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

Yang changes for policy service

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

Changes for achieving ECMP in VMs present in different DPNs

    * Split the traffic b/w extra routes behind the VMs present in the
        * same DC in the different DPNs

    The local fib entries of the extra route would now point to
    the load balancing select group which would contain in turn the
    buckets pointing to the local next hop groups(in case of
    colocated vms and tunnels in case of VMs present behind
    different DPNS).

    The extra route handling for the imported routes.

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

Fib changes for the UC1 & UC3 of ecmp

    This patch contains the fib changes for the \
    * UC1 - ECMP forwarding of traffic entering a DC and
    * UC3 - ECMP  forwarding of  intra-DC traffic
    * Split the traffic b/w extra routes behind the VMs present in the
      same DC in the same DPN

    The local fib entries of the extra route would now point to the load
    balancing select group which would contain in turn the buckets
    pointing to the local next hop groups.

    Depends-On: https://git.opendaylight.org/gerrit/#/c/51999/

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

Bug 7952: ACLService to treat Ethertype=IPv6 and Protocol=icmp as a request for ICMPv6

In Neutron, "Protocol=icmp" is allowed to be configured with "Ethertype=IPv6"
for backward compatibility reasons. When such a rule is configured, Neutron
IPTables Firewall Driver (i.e., reference implementation) treats this as a
request for ICMPv6 and configures a rule accordingly. This patch implements
the same thing in ACLService. Please refer to the bug report for more details.

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

Updated SNAT Spec

The pipeline is updated,
1) The extra table 43 is removed and 44 will continue as inbound table.
2) Now metadata will be set before submitting the packet to the
netfilter using set_field.

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

AbstractDataChangeListener clean-up

Reference:
https://wiki.opendaylight.org/view/Weather#Genius_AbstractDataChangeListener_clean-up

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

EVPN RT2 Yang changes

1. Few review comments fixed
2. EVPN RT2 overall yang changes

Change-Id: Ie9fa2213d348f5b15b4ae52e4e65b79ebda89e4a
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>

Added statistics bundle

Change-Id: I2d10a9871c04373e944a07c1ff96bfd9a4f272fa
Signed-off-by: Guy Regev <guy.regev@hpe.com>

Add Remote ACL Indirection table

Adds an indirection table into the ACL services,
as described in the spec:
https://git.opendaylight.org/gerrit/#/c/51462/

Depends-On: https://git.opendaylight.org/gerrit/#/c/52213/
Change-Id: I023dfda4bdc8da4554141d1fb3431872869ed5ab
Signed-off-by: Alon Kochba <alonko@hpe.com>

Update IPv6 N-S spec with neighbor-discovery.yang changes

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

Rectified incorrect help usage displayed for BGP add-neighbor cli command

odl:configure-bgp -op add-neighbor displays incorrect help.

Change-Id: Ic1e7bd747a07de0fdc562eb59d8bc835558900ac
Signed-off-by: Viji J <viji.j@ericsson.com>

Bug 7888: handle update of floating ip port

In Ocata, when a floating ip port is created, its deviceId may be
set to "PENDING", and an update will follow with the deviceId later on ([1])

This is causing both an uncaught exception in NeutronPortChangeListener.add()
which assumed deviceId is always a Uuid, and also there is no creation of
FloatingIpIdToPortMapping during update().

[1] https://review.openstack.org/#/c/396013/

Change-Id: Ide44ef016f0a8c576f02dfd28d9a4ee9d492841e
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Bug 7809 - NAT snatGroupIdPool is overlapping with Elan Groups

Description : Currently the SNAT Group is defined as 200000-225000
and the elan group range also falls within this range(210000-220000).
Change done to move the snatGroupIdPool range to 225000-250000.

Change-Id: I680305c512221ca775fca8461a901f72f9e13e27
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

BUG-7905: remove duplicate RPC declarations

Duplicate rpc identifiers are a violation of RFC6020 and only happen
to work due to BUG-7424 not being addressed. Fix this up by removing
duplicates.

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

Ethernet VPN Flow Programming for VM's within the same data center.

This review brings in enhancement to VRFEntryListener  to manage flows to
realize L3 Forwarding over VXLAN tunnels. These code-changes cover the
following scenarios:
1) Intra-subnet connectivity between VM's on the same hypervisor within a
data center.
2) Intra-subnet connectivity between VM's on different hypervisors within
a data center.
3) Inter-subnet connectivity between VM's on same hypervisor within a data
center.
4) Inter-subnet connectivity between VM's on different hypervisors within
a data center.

Change-Id: Ieb787442d20cd6351ef0e04a5765afc5aa975721
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Bug 7931:  SubnetRoue re-election to be triggered on disconnected nodes

Subnet Route reelection algorithm is triggered today only when
the anchor ports on the subnet go to DOWN state (or) if such
anchor ports on the subnet are being removed.

However, there are situations where a switch hosting the anchor
port can be disconnected on the openflow control-path and can
continue to remain so indefinitely.

Under such circumstance, new non-neutron IPs become undiscoverable
as the switch hosting the anchor port cannot punt discoverable IP
packets to the controller.

So this fix attempts to always trigger reelection of nodes for
subnetRoute hosting as soon as the current anchor node(or switch)
is detected to be disconnected from the controller.

Change-Id: Idd8be8e2f1940c4840d7d52726ac5187ed33d28f
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Changes for supporting "opendaylight-vni-ranges" pool

Description:

1) Re-defined VPN_POOL range to 100000-130000
2) Adding Util API's for creating and deleting VNI Pool.
3) NeutronVpn APIs exposed to retreive 'opendaylight-vni-ranges' and
'enforce-openstack-semantics'

Change-Id: I61d518aa694416aca7b47b266c0f2c56edf350cf
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

fix for mac movement issue

* updating only elan-interface-forwarding-entries when mac is learned
from vlan provider port which is already learned on VM port.
* updating MACs in elan ds and dpn when MAC is actually moved from VM port
to another VM port.
* handling MAC learning from VLAN provider port if it is a newly learned
mac

Change-Id: I4ff9d79463f0f238c5a8b345438514b93ed4bb17
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Export ACL flow programming logic

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

Ethernet VPN (EVPN_RT5) Enhancement to Non-NAPT to NAPT

This review (1st Review) is part of EVPN_RT5 support in NAT feature.

Changes are made in the NAT code for accomplishing the below

Intra DC Communication:
=========================
For Non-NAPT to NAPT Traffic, Tunnel id will be setting with
Router lPortTag which will be carved out per router if it is part of
L3VpnOverVxlan else tunnel-id will be setting with existing router-id.

Change-Id: Idc5e8b1956311541ae7bae255d023c5f3543f9a3
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Netvirt: Neutron Qos DSCP Marking support

Adds Neutron QoS DSCP support to Neutronvpn.
A new table 90 created for QOS DSCP in Genius.

Supports:
- add, update, remove of qos dscp marking rule assigned to Neutron
port/network

Description:
- When a DSCP rule is created in the Qos Policy which is bound to the
  port, OF rule(table 90) to set tos filed in the IP with the DSCP vaule
  is configured. Qos service is bound to the table 17.

- During the removal of policy from port, unbinding qos service, and
  removal of flow is donw

  Testing procedure:
  1. Create neutron network,subnet and instance.
  2. Create neutron qos policy and DSCP rules.
  3. Associate policy to the port/network
  4. Update the DSCP rule.
  5. Disassociate the policy from port/netwok

  Result:
  - corresponding ovsdb port tos attributes are configured
  appropriately.
  - traffic flows with the DSCP value configured in the port.

Change-Id: I06d0d89c7144dcc27f01320c8ed63bfe6afebf27
Signed-off-by: Poovizhi P <poovizhi.p@ericsson.com>
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Yang changes for DHCP Server dynamic allocation

This change includes the yang changes needed to implement spec [1]

[1] https://git.opendaylight.org/gerrit/#/c/52156/

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

Bug 7876 : After router association to L3vpn, one of the VM ip is not
removed from router interface to BGPVPN

When a VPN associated to router is deleted and a same VPN is associated
back to same router , there is race condition , where interface is not
found in VpnInterfaceOp and remove event is being ignored. Hence interface
remains in router VPN.

It is also seen that , one of interface from same subnet is swapped and
others are not. Its due to race condition that exist in NeutronVpnManager
where create and update can race with each other. Currently create uses
DJC , where as update doesn't .

Change-Id: I354c3ce93f965580c735f1153a043e2514d68af7
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

Model for nat service config parameter.

The "nat-mode" will be used to switch between controller based and
conntrack based nat.

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

Add tcp-md5-signature-option to ebgp's neighbors

An optional leaf has been added to ``neighbors`` list in ebgp module in
order to configure QBGP with the shared secret required by TCP MD5
Signature Option [RFC2385].

See spec qbgp-tcp-md5-signature-option [0]

[0] https://git.opendaylight.org/gerrit/#/c/52218 Ibdb0675c0045754e575b73deba2dfebde4b56bc5

Change-Id: I8f712840bd5b0b6d1647b972375c5e3080f1eadc
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Bug 7885 - CSIT Sporadic failures -
tempest.scenario.test_port_security_macspoofing_port

On port-security update  bind/ubind service is now called.

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

Bug 7878: provider interface MACs are installed on remote DPNs

MACs learnt on provider interfaces must not be installed on remote
DPNs.

This change includes 2 fixes:

1. handleInternalTunnelState event initiated installation on remote
DMAC flows for all elans in the remote DPN, without verifying that
this elan is not Flat/Vlan.

2. Protection on setupRemoteDmacFlow to make sure no other code flows
will install remote DMAC flows on MACs learnt on provider interfaces.

Change-Id: I5ae288ed525dc1ae2f5be1bec5215f2e8f01f8c5
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Netvirt: Yang changes for vlan provider enhancement feature

Modified the elan:elan-instances container with attribute to indicate
whether the network has an external router attached to it.

Spec review link for the enhancement:
https://git.opendaylight.org/gerrit/#/c/52201/

Change-Id: I0f6fe5779a052804dc9549f97d958038fdb383bb
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>

Bug 7881 - Traffic drops when not matching UL SC starting
in a VPNPseudoPort

When programming the L3VPN to SCF pipeline, a new flow is needed in
table 17

Change-Id: I31cf7817eb38b47e5c665c4f0362d0baa9554af4
Signed-off-by: Marcos López Samaniego <marcos.lopez.samaniego@ericsson.com>