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>

Spec for NetVirt QoS alert

This spec file describes the changes to be made
in the Carbon release to support QoS Alert.

Change-Id: I6306a9f0f7a28ad524b39fb2b84a01ace71b5f14
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Netvirt:Yang changes for-Neutron Qos Alert support

***** Commiting only yang as first patch ****
Spec Review Link:
https://git.opendaylight.org/gerrit/50689

Adds support to monitor the drop counts and log alert message when
Qos rate limit rule if applied.

Supports:
- Configuration to start or stop monitoring the port statistics
- Configuration of rx drop count threshold value in %
- Configuration of location and name of log file
- Configuration of polling interval

Description:
- As a result of Qos rate limit rule, ingress packets may get dropped
  by the switch.
- If monitoring is enabled, port statistics are queried after polling interval
  and alert message is logged in a log file if packet drop % is more
  than configured threshold value.
- Threshold value is specified in % of drop counts
- All configurations are applied at controller level.
- Default or initial configuration available in netvirt-qosalert-config.xml
    * qos-alert-enabled - false
    * qos-drop-packet-threshold - 5%
    * qos-alert-log-file - qosalert/qos-alert.log
    * qos-alert-poll-interval - 2 minutes

Spec Review Link:
https://git.opendaylight.org/gerrit/50689

Testing procedure:
1. Create neutron network,subnet and instance.
2. Create neutron qos policy.
3. Associate policy to the port/network
4. Send traffic more than the limit specified in policy.
5. Configure the threshold value and enable port statistics monitoring
6. Check for alert log messages in the file if packet drop % cross the
   threshold value.

Result:
- Port statistics polling is started when qos-alert-enabled is set to true
- Alerts messages are logged when packet drop % cross the threshold value

Change-Id: I5b2675a2c036835d4ac2e5bd585c3da8e9b268e8
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

BGP Manager changes to support ECMP.

This review ensures that when 2 DC-GW advertises routes for the same
destination prefix, the vrfEntry DS are populated correctly with both the
route paths.

The stale fib entry flat map now includes nextHop also in its key, to
align with the new vrfEntry model.

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

Yang changes for federation plugin

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

Bug 7787 - missing flows in T21

when VPN interface up before tunnel info is available in operational DS,
it causes missing flows in T21

Fix: always handle the add event inside TunnelInterfaceStateListener -
 even for a tunnel which state is Down

Change-Id: I41c3efbeeb5740788f58ced57a698a73ef70b466
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

yang change to make remove deprecated leaf-elements

Description : It's been observed that in the review
https://git.opendaylight.org/gerrit/#/c/49093/, the whole
neutronvpn-config container been made as deprecated instead
of required leaf element.

1) Changes done to remove 'use-transport-zone' and 'use-of-tunnels'
config from neutronvpn-config(as no usage found in neutronvpn).
2) Removed the default values for these config defined in
netvirt-neutronvpn-config.xml
3) Defined default value for VNI Spec specific configs.
4) Re-defined the neutronvpnConfig and passing its reference to
neutronvpnManager in neutronvpn.xml(as it will be  refered
accessing VNI Spec specific configs)

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

Use the right service name when binding service

Making ELAN and ACL bind services with the right service name

Change-Id: I022a0ff4aaff4772870413890ce8a9665af5a309
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Yang changes for IPv6 Neighbor Discovery Support

This patchset includes the yang changes required to implement
IPv6 North-South support for FLAT/VLAN provider network spec [1].

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

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

Yang changes to support tunnel updates in ECMP

New YANG model to update load balancing next hop group buckets according
to VxLAN/GRE tunnel status.

Subsequent review will make use of these containers.

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

BUG:7896 OptimisticLockFailedException

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

Bug 7875: NPE due to subnetmap create DjC trailing update

Removed troubling DjC.
Also have made sure that such errors are caught at NeutronVPN
level itself by separating out subnetmap create and update.
Update will not create subnetmap node with partial info, instead will
return if subnetmap node is absent.

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

correcting the spec for providing ECMP over BGP VPN

RST generated doc had some warnings. Warnings are fixed and some
foramtting issues are resolved.

Change-Id: I22cef70c9af3165c16f2cd960639da12297297f4
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Bug 7714: VPN Operational Interfaces not getting removed at all.

This can be a potential problem for temporary CSIT failures
that happens on VPNService suite.

The VPNInterface is not getting removed at all permanently from
VpnInterface Op DS. When VM interface has secondary adj like extra route
or invisible ip , the order in which cleanupvpninterface worker DJC is
called not guaranteed.

The regression was caused by review:

https://git.opendaylight.org/gerrit/#/c/51224/
VPNInterfaces not deleted from Oper DS

Change-Id: I1b2a1d139030f707cec93eb5a5e6a690e010e336
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

Fixing checkstyle error in FibManager

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

Bug 7387 : Netvirt: qos policy applied on the network, not applied on
newly created ports of same network

Use case:

1.Network is created and updated with QoS policy.
2.Port is created on the same network.
3.Now this port is intended to have the QoS policy as the network has.

RCA:

The scenario of  application of network qos policy to the newly created
port  was missing

Code fix:

1.Created a new class called QosInterfaceStateChangeListener.java which
listens on the interface state oper events
2.Handled the scenario of  application of network qos policy to the newly
created port in the add( ) of QosInterfaceStateChangeListener.java

Application of network qos policy to the newly created port condition was
missing. Fixed it by adding the same in NeutronPortChangeListener.java

Change-Id: I3211a698e9a27cfefe090fbbf2ab120f8626de76
Signed-off-by: A Vamsikrishna <a.vamsikrishna@ericsson.com>

Bug 7856: Reverse SNAT flows order to minimize race possibility

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

Bug 7418 Run local group creation as async task with key equal to subsequent tasks.

Change-Id: I515b3b874b50f8e53a931e1a3ee3a2e30f877f8e
Signed-off-by: Konsta Pozdeev <konsta.pozdeev@hpe.com>

Spec for Remote ACLs Indirection Table

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

Added netvirt countrs

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

Fix NPE from VpnDpnToTransportZoneListener

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

Bug 7445: Improve the performance on bulk create.

    * Reduced read calls when SG associated with the VMs.
    * We have used cache mechanism to store neutron port information
      that was read from DS.

Change-Id: I3b243ddf3656e48392593c6fc1086713d1fd1af3
Signed-off-by: karthikeyen.p <karthikeyen.p@hcl.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fix for GwMac flow deletion during interface delete

* Without the fix, the mac address used for deleting the flow
  was obtained after quering interface state again. This
  returned null resulting in gwmac table not getting deleted.
* With the fix, interface state is not queried again. Interface
     state passed during interface state deletion notification is
  used to obtain mac address.

Change-Id: I1c091dc8edb722ab58fdf05f6cba1729a583209e
Signed-off-by: eceghkl <manu.b@ericsson.com>

vni-based-l2-l3-nat spec updated with openstack-semantics config

A few formatting changes done alongwith introducing the config.

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

Yang changes for vni-l2-l3-nat

Changes are as per approved in the spec at:
https://git.opendaylight.org/gerrit/#/c/52241

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

Bug 7281: Support auto-TZ for non-neutron northbounds.

Support non-neutron networks for creation of TZ.
Listen to Elan and Vpn changes instead of NeutronPorts.
This will be used for unimgr.

Change-Id: I24ae67888352b9a5f941f74abf4db019e3cf439e
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Bug 7831 : BgpRouter receives unnecessary events

Suppressed unwanted advertisement of routes, that were being advertised due to a
breakage in the VPN Engine.
When BGP Client is not configured, route/vrf advertisement used to fail
and throw lot of exception. If BGP Client is not connected, the
process of dispatching routes/vrf to DC-GW via BGP Client should not happen
at all.

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

External Routers yang changes - external ips will contain also subnetId

This is part of the hairpinning tasks.
This include only changes in the yang model for external routers.
Changes the external-ips to be a list of ip and subnet.

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

corrected the population of BGP Total Prefixes counter

Change-Id: I679b9a4b50ce2204188d0bbe7a5d4c48aafd3aee
Signed-off-by: eamakol <amaranadh.kolla@ericsson.com>

AclServiceTestBase use assertFlowsInAnyOrder from genius

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

Disable split horizon for Trunk subports

Pkts going from one subport to another on same VM will have the same
in/out ports. Default OVS splithorizon will not allow this behavior and
needs to be disabled for all subports on a trunk.

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

Bug 7861: No ping response from FIP on 1st router when adding 2nd FIP

Problem Description:
====================
Table Miss Entry for SNAT/DNAT in FIB Table (table=21->26 flow) is
overriting when multiple router is present instead of creating new entry
for each VPN Instance.

Each router is associated with unique external network and each external
network is associated with unique internet BGP VPN Instance.

Example topology:
router-1/external-network-1/bgp-vpn-1
router-2/external-network-2/bgp-vpn-2

Both router-1 and router-2 are presented on single compute node. In this
scenario router-2 ping traffic only is working as expected. Since table 21
doesn't have the table miss entry for router-1 with bgp-vpn-1 instance.

Solution:
=========

In ExternalNetworkListener while creating table miss entry for SNAT/DNAT
in table 21, added "vpn-id" attribute also in creating flow reference as
same as implemented in the class
SNATDefaultRouteProgrammer.buildDefNATFlowEntity(). After this code change
in table 21 miss entry for SNAT/DNAT is created for each bgp-vpn instance.

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

vni-l2-l3-nat spec update with yang models to supply VNI to VPNEngine

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

Qos implemented as separate service. All qos related codes from neutronvpn
are segregated and a separate qosservice module is formed.

Change-Id: Ie9fc4755f5270eeb3f654bf5d0676ae302c9f3e4
Signed-off-by: Poovizhi P <poovizhi.p@ericsson.com>

BUG 7727 : Local and Connected routes do not get imported

Route Origin for Subnet Routes and Local VM routes were changed from
STATIC to CONNECTED and LOCAL, respectively. Hence, the import/export RT
functionality for VPNs was broken.

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

BUG:7824 ModifiedNodeDoesNotExistException

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

Updated the conntrack based SNAT spec

1)Added a new HA logic when conntrack mode is selected.
2)Added info about the config parameter to switch NAT mode
3)Added info about a new CLI for displaying the NAPT switch selected for
each router.

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

Added to VrfEntry a new attribute of parentVpnRdName

This is part of the Hairpinning tasks also use in l3vpnOverVxlnEvpn tasks.
For floating IPs the route distinguisher will be subnetId
and the DPN information only exists for external network VPNs.
So, if will have information in the parentVpnName it will use it
to get the DPNs.

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

Cleanup errors for networks of unsupported type

Reword and lower debug level of network configurations
for networks without a provider extension or with an
unsupported type.

Change-Id: I3b58a79b2fb72fe2c56328f79e0962e366a299ff
Signed-off-by: Alon Kochba <alonko@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7824 - ModifiedNodeDoesNotExistException

* added potential fix to guard against above exception
while handling elan interface remove/interface state remove
handling

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

Bug 7725:  AAP with prefix 0.0.0.0/0 not supported in ACL

+ Added a check. In case of 0.0.0.0/0 prefix, program flow with just "ip"
match skipping the ip-prefix (0.0.0.0/0).
+ This method is called to configure flows both in table 40 and 251.

+ Handled below issue:
  - Create VM1 & VM2 with AAP 0.0.0.0/0 and default SG.
  - Create VM3 with default SG without AAP.
  - Delete VM2. Observe that +new+trk flow configured for VM2's 0.0.0.0/0 AAP
    prefix gets deleted (related to remote SG) in 252 table even though VM1 still
    exists.
    Flow: cookie=0x6900000, duration=4737.343s, table=252, n_packets=0,
    n_bytes=0,
    priority=61010,ct_state=+new+trk,ip,metadata=0x10000000000/0x1fffff0000000000
    actions=ct(commit,zone=5000),resubmit(,220)

+ Earlier flow priority was generated/used based on SG rule UUID. The
  above case was not handled with this approach.  So, with this fix now
  every ct_state=+new+trk flow will have a unique flow priority within table.
  Flow priorities are generated based on flow ID.

+ For every switch, two ID pools (one each for ingress and egress) are
  maintained which increases the availability of flow priorities.

Change-Id: I646312e1324267913cbb2865a1a8164b23b40372
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7815: Using DJC for VpnManager Interface-state Listeners

With mdsalutil is having a THREAD_POOL_SIZE=1, all the add(), update(),
and remove() methods handling the application logic were on the same
thread. This in turn was delaying all the interface-state listeners.

Changes are done to make use of DataJobCordinator so that all the
VpnManager application logic get executed by a seperate thread so
that the main thread is freed-up for other listener execution.

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

Bug 7852 : FIB not updated with DC-GW routes

Problem :
FIB not updated with DC-GW routes and DC-GW not updated with VM routes.

Analysis and Fix:
Recent commit has broken the way RT is handled in NeutronVPN.
Fixing the same.

Change-Id: I3678b491bdc743b24e3c4abcb9744accc640031b
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

Bug 7842: ACL: Arp flows missing in ACL tables for overlapping MAC address

Updated to use lport tag with flowname for Egress ACL ARP flows

Change-Id: I9e733b70cb968c16f57253a975b9aa6d74c1d085
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Bug 7839: ACL: ACL flows are not deleted from source host during VM
migration

    - In the current code, ACL flows are being added in Interface state
      listener and deleted from Interface config listener. As during
      migration, only interface state listener gets triggered, Config
      Interface and Interface state listeners are re-formatted slightly as
      explained below to fix this issue and also to support delete VM
      without any problems.
      Details of changes are as below:
      (a) bind/unbind moved to Config interface listener
      (b) All other ACL flows programming (add/delete) now being done from
      Interface state listener

    - Also, bind/unbind service code is updated to use
      DataStoreJobCoordinator to avoid ConflictingModificationExceptions

Change-Id: I4cf0c0d58eecbceafbed740f0201e128c196f555
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

releasing dcn thread once tunnel interface state dcn delivered

* This change is about releasing dataTreeChangeHandlerExecutor's thread
immediately after DCN is delivered to service module. This allows to
improve DataTreeChangeListener notification time when there are multiple
listerners listening to same data tree path.
* For ELAN, DJC is used to run tunnel interface state DCN related tasks in async
way using ElanUtils#getElanInterfaceJobKey

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

Creating default FIB router flows for external subnets.

This is part of the Hairpinning tasks.
It will install on the FIB table (21) default flow for every external subnet.
The flow match is based on the VPN ID for the external subnet.

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

Bug 7843 - Missing buckets in ELAN BC group installation during OVS restart

L2 pipeline can contain malformed flows/groups if OVS restart occurs in parallel
due to oper DS temp delete during restart
Possible solution would be to check for config DS inventory node instead of oper DS
so flows would be restored with correct content after the OVS is up again

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

Support for multiple route paths

- Modified odl-fib.yang to support multiple route-paths for the same
  destination prefix.
- Added a FibHelper which provides util methods to construct VrfEntry.

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

Bug 7773: Objects should be compared with "equals()".

Additional changes:
 * Removed unnecessary repetitive calls.

Change-Id: I607ce073ee9ec4289c80dccd96a4b5222e4d9772
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>
Signed-off-by: arthi.b <arthi.b@hcl.com>

Use OFtunnels when configuring automatic tunnels

Refer: https://git.opendaylight.org/gerrit/48694

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

Checkstyle: fix new ParenPad violations

Enforcing no space(s) after a left parenthesis and before a right
parenthesis is a proposed change [0] which can only be merged when all
violations are fixed in projects enforcing checkstyle. This patch fixes
new violations since [1] was merged.

[0] https://git.opendaylight.org/gerrit/#/c/51316/
[1] https://git.opendaylight.org/gerrit/#/c/51704/

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

Bug 7786 Delete and re add of access port handling

After deleting access port in l2gw device and upon readding , vlan
bindings were missing .
Fix is to resync upon such case from config topology.

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