BGP Minor fixes

Fix to :

1. set more bits when lot of routes are synced from QBGP
2. handle entity owener ship correctly

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

Add ARP Check Table in doc

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

Fix issue in natservice with IPv6 VxLAN tunnels

When using IPv6 VxLAN tunnels along with SNAT Conntrack mode,
we have seen that use-cases like SNAT and FIP are not working
due to an exception in natservice leading to some missing flows.
This patch fixes this issue.

Issue: NETVIRT-1401

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

Datastore-constrained txes: elanmanager

Change-Id: I92faf670c930604761fbe3506f20c1e5af7a9e21
JIRA: NETVIRT-1339
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Handle exceptions in BgpRouteVrfEntryHandler

* Executor service is not handling any new task after the exception.
* Fix is added to handle the exception properly.
* Added more TRACE logs are added for data collection

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

ECMP code changes for handling itm-direct feature

* ECMP select group doesn't contain remote buckets
* Tunnel operstate was queried earlier from interface state
* Moving it to tunnel state from itm

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

Troubleshoot netvirt datapath

Adding a document which can help people debug
the netvirt datapath in case of any failures in the deployment.
This is a WIP, and requires inputs from all of the netvirt
contributors, so that we can have a detailed description
on making debugging easier.

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

Remove references to mdsal-model-artifacts

This artifact is going away, migrate to mdsal-artifacts.

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

COE external ip ping failure

JIRA: NETVIRT-1396

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

Bump versions by x.(y+1).z for next dev cycle

Change-Id: Ia07cd2f52a89dab5e230cd56ba762eb28c3f3c87
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

NAT failure due to Napt Switch-Over not happened

Description :
Whenever a last VM part given VRF is deleted/migrated, it is required
to delete 'dpn-vpninterfaces-list' entry for given dpn-id from
'odl-l3vpn:neutron-router-dpns' Operational DS. This trigger
re-election of new Napt Switch which has VRF presence. But, when 2
VMs part of same VRF on a same DPN deleted simultaneously, then
there are chances that both of these event doesn't delete
'dpn-vpninterfaces-list' entry for the DPN which results in
non-triggering on new Napt Switch election. As a
result, the SNAT reverse traffic initiated from Non-Napt Switch
fail as FIB routes will not be available to carry back traffic
to non-napt switch.

Solution: Changes are done to make sure the 'dpn-vpninterfaces-list'
entry for DPN is deleted properly when last VM on that DPN is deleted
for a given VRF.

Issue : NETVIRT-1221

Change-Id: Idaebd8576762d12639d7bfbc26a89af94de4e991
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Controller based SNAT traffic fail after upgrade

Description: It's been observed certain below NAT config DS are not
populated properly during upgrade resulting in SNAT traffic failure.

1. ext-vpn missing in external-networks.
2. router-id-name data missing.

Changes are done to update the above NAT DS properly during upgrade.

Issue: NETVIRT-1280

Change-Id: Idfd1e4a44c646bedc15f0c3e24401418065feda0
Signed-off-by: shaik <shaik.b@altencalsoftlabs.com>

21->26 flow missing for NAT use case

Description
-----------
21->26 flow is missing with the below use case

Use case :

1. create network
2. create subnet
3. create vm
4. create router
5. add subnet to router
6. create external network and subnet
7. create external bgpvpn
8. associate external bgpvpn to external network
9. openstack router set router1 --external-gateway myextnet10
       --disable-snat
10. openstack router set router1 --external-gateway myextnet10
       --enable-snat

Notice that the 21->26 flow is missing

Changes are done to install the 21->26 flows when we disable and
enable SNAT

Issue : NETVIRT-1294

Change-Id: I39a2b4d20aec4694aa269c334dc5635e7d7acd7e
Signed-off-by: shaik <shaik.b@altencalsoftlabs.com>

Loosen DataObjectModification collection return

This allows the contract to be loosened so we can do type-safety
in implementation of DataObjectModification.

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

Remove app-config-pusher feature dependency

This openflowplugin feature is not used by netvirt.

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

Datastore txes : fibmanager

This switches fibmanager to datastore-constrained transactions, except
where they conflict with the resource batching manager.

This requires Ic63fd992b33e9e1cd4049bc129545a9830b7a8c0.

JIRA: NETVIRT-1340
Change-Id: Ia766e6089ed3fd8b70f0a3f03e636ac1000c2cbb
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Exception Handling in UpgradeStateListener

Change-Id: I8bf268e8392b9f7b3902de318833729f3dfca4f4
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Prefix to interface d.s not cleared

Issue:
======
Stale flows not removed for many data stores .
Stale entries were not removed for prefix-to-interface d.s
because of which dependent vpn related d.s were not cleared.

Analysis:
=========
Prefix-to-interface was wrongly getting deleted from
removePrefixToInterfaceAdj() in updtae() in VpnInterfaceManager.java
because of which in cleanUpOpDataForFib() ,prefixinfo was null and the
respective calls to delete prefix-to-interface and vpn-interace-op-data
was getting missed.
         Removed the call removePrefixToInterfaceAdj().

Issue: NETVIRT-1223

Change-Id: Id892b033014606e6c451cb2c5644d839ec2da5ac
Signed-off-by: nithi thomas <nithi.t@altencalsoftlabs.com>

Traffic drop b/w DC-GW and VM

Issue:
------
L3_LFIB_TABLE (20) flows were missing for the interfaces.

Fix:
----
Corrected if check to program L3_LFIB_TABLE flows in case of BGP-VPN.

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

Retain NAPT Switch after Upgrade for SNAT

Test Scenario1: All OVS are connected back after upgrade.

1) Take the back-up of the Config DS and bring down the controller.
2) Disconnect OVS(by blocking port 6640 & 6653)
3) Upgrade ODL, set upgrade-flag=true and reload the config DS.
4) Start connecting back the OVS with first connecting non-napt
switches followed by Napt-Switches.
5) set upgrade-flag=false once all the OVS are connected by to ODL.
6) Verify earlier Napt Switches continues to work as Napt Switches
and all earlier on-going SNAT session and new sessions are not
impacted.

Test Scenario2: Few OVS(which are elected as Napt) are not
connected back after upgrade.

1) Take the back-up of the Config DS and bring down the controller.
2) Disconnect OVS(by blocking port 6640 & 6653)
3) Upgrade ODL, set upgrade-flag=true and reload the config DS.
4) Start connecting back the OVS with first connecting non-napt
switches and do not connect few OVS elected as Napt Switches earlier.
5) set upgrade-flag=false.
6) Verify a new Napt Switch is elected for those router whose Napt
Switches are not been connected back after upgrade.
7) Verify new SNAT sessions are established and earlier on-going SNAT
session gets migrated to newly elected Napt Switch.

JIRA: NETVIRT-1222

Change-Id: I194124596e987c43703be37707235a6c88c91013
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Signed-off-by: shaik <shaik.b@altencalsoftlabs.com>
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Clean up MDSALManager exception handling

Now that the MDSALManager interfaces have been updated, we can adjust
NetVirt to use the exceptions as declared instead of dealing with
Exception.

Change-Id: I4b41025f54f63a66a01b1f9b50d4f12918863289
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bump to odlparent 3.1.3

Change-Id: I15b0586dbed9cec935ba8504a2e2f9beb7fd8593
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Avoid duplicate processing code with events

When we use event callbacks to react to the appearance of
currently-missing data, we might as well use that as the nominal flow
too: if the data is present, the event callback's underlying listener
will fire immediately, and the callback will be handled straight
away. This avoids duplicating code whenever an event callback is
used.

If performance testing reveals that fast paths are needed, we can
handle that centrally in the event callback mechanism, instead of
repeating it at every call site.

Change-Id: I49791d810946988c704a9cfe24297c977c1c8c84
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Remove unused exceptions

This drops exception declarations which are never used.

Change-Id: I754690dcd1aa17dcd9b56fb290dcf173d2f6fc08
Signed-off-by: Stephen Kitt <skitt@redhat.com>

VPN-to-Router Associate/Disas Performance Improve

Addressing few review comments which got it
from existing merged patch[0].

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

Issue: NETVIRT-1182

Change-Id: I98ca89e082c3f339e75b8de3f1ad2eb83cebfbbc
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

rm runtime logging change in QosAlertGenerator

JIRA: CONTROLLER-1845
Change-Id: I4b92254b0b47a718f9920166b5dbfc177cbf8dc7
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Conntrack headers fails to parse.

The extra headers added to the packet when it return from ovs after a
conntrack operation causes deserialization error when the packet is
punted to the controller.

Added ct_clear action once the packet is returned from conntrack in
table=47

table=47, n_packets=50, n_bytes=4887,
priority=6,ct_state=+snat,ip,metadata=0x30d40/0xfffffe
actions=load:0x30d48->OXM_OF_METADATA[0..23],
load:0->NXM_OF_IN_PORT[],ct_clear,resubmit(,21)

NETVIRT-1383 Traffic to the PNF from a VM without FIP fails(SNAT
conntrack)

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

Add support for official OVS NSH implementation

Change-Id: Id0adf67fa3e39da46d667efe1c0bc13c34c49443
JIRA: TSC-114
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

vpnmanager-impl compilation issue

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

Fix exception in addSwitch

In a fresh deployment when the first switch is
added to the controller and when there are no
neutron resources (like ExtRouters) are present,
the current implementation in addSwitch was
causing ExpectedDataObjectNotFoundException.
This patch fixes it by changing the syncRead
to syncReadOptional while querying the datastore.

Issue: NETVIRT-1382

Change-Id: I4393a7ad2fa896eda2e1cbb88e3c34c9ffcc0c56
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
(cherry picked from commit acb65d76fce6e6a361fd59a72fe33f067f3d27c1)

Use IetfYangUtils to canonize addresses

IetfYangUtils provides optized utility methods for converting
Mac/PhysAddress to its canonical lower-case format. Use it wherever
possible. This prevents unnecessary object duplication.

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

Do not bounce Ipv6Prefix through String

Rather than converting IpPrefix to String and then back, just use
Ipv6Prefix.

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

Use IpAddress-based createHwvtepPhysicalLocator

Use IpAddress-based createHwvtepPhysicalLocatorAugmentation()

This removes the need to de-encapsulate IpAddress string by retaining
IpAddress.

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

Fix typos in log message

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

Do not use char[]-based union access

Both the constructor and getValue() method are going away, do not
use them.

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

VPN changes for IPv6 ND based aliveness monitor

+ Learnt hidden IPv6 addresses are monitored using ND based aliveness
  monitor.
+ Tried to have common code for both IPv4 and IPv6 whereever possible.

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

JIRA: NETVIRT-1213

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

VNI based L3 forwarding support for BGPVPN

Description: As part of this spec implementation earlier, VNI based
forwarding for BGPVPN was kept out of scope. This Change addresses to
support VNI based forwarding of packet over VXLAN tunnel covering Router
associated and Network Associated BGPVPN and iRT/eRT use-cases.

Additional changes to preventing backpulling of neutron subnet-map to
retieve network-related attributes in vpn-engine is been addressed.

Handled missing destination mac address in the remote fib flow for
iRT/eRT use-cases.

Issue : NETVIRT-1209

Change-Id: Ib61d0c02129de980348436cdeba7939488c2da6e
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Refactor VpnManager module.

+ Refactoring VpnManager to support subnet routing for hidden IPv6
  addresses.
+ Moved all IP learning classes to package org.opendaylight.netvirt
  .vpnmanager.iplearn.
+ Renamed few classes to Ip*** instead of Arp*** to keep it common for
  both IPv4 and IPv6.
+ Refactored code in iplearn package to handle both IPv4 and IPv6.
+ Kept placeholders (TODO's) for supporting IPv6 subnet routing for
  hidden IPs.

JIRA: NETVIRT-1213

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

VPN-to-Router Associate/Disas Performance Improve

Problem Description:
====================
When BGP-VPN is getting associated with router, FIB entries
are getting programmed very slow. This will cause the
performance of the VPN Manager.

Solution:
=========
Delay root cause: Each VPN interface update is taking 2 sec time to
remove the old VPN ID and update the new VPN ID and VPN service
unbind and bound to new VPN Instance.

To avoid each VPN interface update 2sec time delay, have moved
the method "handleVpnSwapForVpnInterface" processing logic into
JobCoordinator to process each VPN interface update in parallel.
This way overall FIB entry update is consuming less time compare
to without having this code change.

Issue: NETVIRT-1182

Change-Id: Ic17cba3e325a0c63fead9d30ec857c0e7eb8dcfc
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

Remote group flow fails in conntrack SNAT

The flow to take the packet to the packet to the NAPT switch from
non-napt switch fails occasionally, since the group is not programmed in
time.

The flow is now programmed only when the group is available in
Config datastore.

NETVIRT-1352 CSIT sporadic failures Random ping failure in snat
conntrack mode

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

ARP flow fails due missing group

The flow is now programmed only when the group is available in Config
datastore.

NETVIRT-1371 VM associated to vxlan network is not reachable because of
ARP flow missing

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

Revert "Add support for OVS NSH implementation"

This reverts commit c0d8e1872ffd45cb33b70e2fd753ee3b2d1795c1.

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

Add support for official OVS NSH implementation

Change-Id: I1bac27d8c39639d84da0aa83ef1a5703c6c215f7
JIRA: TSC-114
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

L3VPN creation is throwing Exception

Issue:
======
When L3VPN is created via ODL REST API or Neutron BGP-VPN API
it thorws BGP-VPN creation failed exception from the neutronvpn
listener "NeutronBgpvpnChangeListener".

Solution:
=========
When L3VPN is created, we need to check properly whether any network
information also existing in the configuration or not.

With this code changes, handled properly for network presence in the
L3VPN configuration.

Issue: NETVIRT-1375

Change-Id: Ibf44f419124e313c646e272d49426e700c7ad833
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

syncRemoteAclTable removed as synchronized

Removed syncRemoteAclTable from synchronized context and added it to
JobCoordinator

Change-Id: I0064e3e839be0837093790db3d4f6771eca7d302
Signed-off-by: Sushant Kumar Jha <sushant.k@altencalsoftlabs.com>

Fix NaptFib entry incorrectly removed

NAPT_PFIB_TABLE (table 47) flows for external network/subnet
are incorrectly removed when one of routers unset from the
external subnet which still serves as gateway to other routers

The fix is to check if the external network/subnet no longer
gateways to other routers before removing the NaptFib flows

JIRA: NETVIRT-1351

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

Use minimal model imports

odl-mdsal-models is a big feature, pull in only the bits we need,
specifically draft-ietf-netmod-acl-model-07.

Change-Id: Ibeb19d3b59b72b537be499dfaf50595e94ed62b8
JIRA: MDSAL-363
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Avoid ofp topology discovery

openflowplugin is packaging lldp based topology discovery
along with "odl-openflowplugin-flow-services. This feature sends
periodic lldp messages on all the links available, to discover
the topology. Netvirt does not need this lldp feature, but is coming
by default with the feature we were including.

This patch removes the feature which was previously used, and
use only the appropriate openflowplugin features needed for netvirt
functionality.

JIRA: OPNFLWPLUG-1024

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

Handle unprocessed IPv6 NA flows.

This patch addresses below review comments:

https://git.opendaylight.org/gerrit/#/c/71567/19/ipv6service/impl/src/main/java/org/opendaylight/netvirt/ipv6service/IfMgr.java@634
https://git.opendaylight.org/gerrit/#/c/71567/19/ipv6service/impl/src/main/java/org/opendaylight/netvirt/ipv6service/IfMgr.java@661

JIRA: NETVIRT-1213

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

ECMP routes are not clearing in dualstack network

Problem Description:
====================
ECMP routes are not cleaned up properly in dual
stack network use cases.
The following neutron command is not working
properly for IPv6 address family in extra route
feature.

neutron router-update R1 --no-routes

Solution:
=========
FIBManager is not properly handled for both IPv4 and
IPv6 address family in case of dual stack network.
It is always added IP prefix length value is /32 instead of
validating whether it is IPv4 or IPv6 prefix. As a result
querying prefix-to-interface data store is returning null
value for valid IPv6 prefix. This is the root cause of the
problem for clearing all IPv6 extra routes.

As part of this fix, have handled properly for both IPv4 and IPv6
address family.

Issue: NETVIRT-1355

Change-Id: Iadb69e0c259d125c20c825edbdc7716495eb077a
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

LPort tag issue with controller-snat

Desciption: When packet is sent for SNAT translation
from Non-Napt Switch, the lport tag is overwritten to
tunnel lport tag(0->36 flow). During SNAT
translation, when we query for interface name from
if-indexes-interface-map DS for this tunnel lport tag
which is not available(with itm direct tunnel changes)
resulting in Exception. Changes are done to query the
dpn-id and in_port from the packet for Non-Napt Cases.

Change-Id: I9f9b6117b4acfcd3bb33adbfa86e2c25e1218976
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Fix for VM deletion issue

Route path entry is getting removed from
FIB table after VM deletion->

If multiple VMs are present in the same CSS and
an extra-route is configured with nexthop being
both such VMs, then if one of the VM is removed,
the entire route-path representing that CSS nexthop
is getting removed from FIB.

Consequence -> If further deletion of VM takes place,
then it won't update the flows and groups.

Fix -> This is fixed by checking if extra-route is present
behind at least one VM, then do not remove or update fib entry
for route-path representing that CSS nexthop, just update
vpntoextraroute and prefixtointerface DS.

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

Remove redundant method invocation

updateRemoteAclFilterTable is invoked in both BIND and ADD, removing
duplicate method invocation and updated corresponding test results.

Change-Id: I6f74d4a16df5448cadc739ac54fb0e02f02092ca
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

Fix indents in l3vpn-robustness spec

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

Datastore-constrained txes: aclservice

Change-Id: Id11d6576f9179aa5249a35346d3f5e9e61748339
JIRA: NETVIRT-1331
Signed-off-by: Stephen Kitt <skitt@redhat.com>

NETVIRT-1369: Conntrack headers are not removed

ct_clear was not used with ct submit in tables 217 and 247. This was
causing failures when same data packets were punted to ODL controller.

Updated ACL to add ct_clear with ct submit in tables 217 and 247.

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

Delete unProcessedElanInterfaces properly

* unProcessedElanInterfaces list was not deleted.
* Fix is added to delete it after the usage.
* Also delete is handled when elaninterface is deleted.

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

Fix for Stale entry in table 21 for same dest

Description: While associating bgpvpn to router having extra routes,
VrfEntryListener update is invoked which should ideally remove extra
routes as new vpn instance is created along with rd, but check condition
added to remove extra routes during interface down prevented deleting
extra routes resulting in stale entries. Have removed check condition to
solve this issue.

Change-Id: Ic693435ce7a937bd1a9f781f4007449f986fa426
Signed-off-by: ashwinbc <ashwin.b.c@ericsson.com>

Fix for NPE in TunnelInterfaceStateListener

Description: This is to supplement a fix
https://jira.opendaylight.org/browse/NETVIRT-1132
which is already in place. Deleting unused leaf in yang and related code

Change-Id: I341a72dedc33c5a1105abe9f06dbba35e167e5a2
Signed-off-by: ashwinbc <ashwin.b.c@ericsson.com>

Spec for L3VPN Robustness Improvements

This spec is to improve robustness of the L3VPN Engine
for specific use-cases that require IP-Serialization
within L3VPN.

JIRA: NETVIRT-1204

Change-Id: I626a7abc276810eef8e8cd65603c6bccf638e63e
Signed-off-by: enavive <vivek.konnect@gmail.com>
Signed-off-by: Vivekanandan Narasimhan <vivek.konnect@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <vivek.konnect@gmail.com>

Allow MDSALManager exceptions

The exception-handling story in MDSALManager is currently
underwhelming: we basically swallow errors, or encapsulate them in
RuntimeException. To allow this to be changed, NetVirt has to allow
for exceptions coming from MDSALManager; this patch implements this.

Once MDSALManager’s exceptions have been defined, this will be
revisited to clean everything up and use specific exceptions.

JIRA: NETVIRT-1318
Change-Id: I66ad8c2643a03d64ccb4abed94628498636f9152
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix invalid dpnId configured as NAPT Switch

Issue: NETVIRT-1361

This patch schedules a primaryNAPTSwitch for the routers
which do not have any allocation and also ensures that an
invalid switch is not allocated for a router.

Change-Id: Ib2355cb935c1014915270196b86ae4eb10ec8a72
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
(cherry picked from commit 6b49f701e57981db6d942053783d138bf3027a98)
(cherry picked from commit f7d129498326a7336c40c8783fea35228cc672fd)

Datastore txes: natservice, part 3

This switches calls to VpnManager to datastore-constrained
transactions, and completes the natservice migration.

Change-Id: Ib441b99be37177b121010ba08da99256fb740e51
JIRA: NETVIRT-1342
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Neutron QoS blueprint

JIRA: NETVIRT-1242
Change-Id: If76b8cd7bb4c1d12076970f26e50bcb515351877
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Datastore txes: vpnmanager external interface

This patch adds datastore-contrained variants of all the
transaction-carryying methods in IVpnManager. In some cases the
transactions aren’t used in the implementation, so the replacement
method drops the transaction entirely.

Change-Id: Ie169a320e7bc7ca80f80687b0d7aa31c63f0a5e6
JIRA: NETVIRT-1346
Signed-off-by: Stephen Kitt <skitt@redhat.com>

NPE in SNAT conntrack tempest.

NETVIRT-1359 NPE in SNAT conntrack tempest.

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

Fix for populating datastore properly

While updating used RDs, deletion of
extra-route was being performed on wrong
datastore, fixing this one by using operational
vpn interface op data.

Maintaining different transaction for config
extraroute-rds-map datastore.

Removing redundant delete of extra-route
from delAdjFromVpnInterface method since
it is already being taken care by remove
and update of FIB.

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

Datastore txes: natservice, part 2

This switches the remaining managed transactions to
datastore-contrained transactions, apart from transactions which
involve VpnManager (which isn’t ready yet).

JIRA: NETVIRT-1342
Change-Id: I560a0b2cf0ca012239a0b1759e6a9706ac4357f1
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Datastore txes: natservice, AbstractSnatService

This patch migrates AbstractSnatService and related classes and
methods to datastore-constrained transactions. It also splits a number
of add/remove methods into separate add and remove methods,
simplifying the remove methods as a result.

JIRA: NETVIRT-1342
Change-Id: I75b28b2edea2c576cf2c73f11a38b292e6c8063c
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Convert sfc.classifier to ds-constrained txs

Change-Id: I2494015d24b5b745af72deda60e997e95748afd8
JIRA: NETVIRT: 1347
Signed-off-by: Sam Hague <shague@redhat.com>

Disable building unused cloud-servicechain

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

MDSALManager clean-up part 1

This patch uses the new methods in MDSALManager to clean up
TransactionAdapter calls.

Change-Id: I86436249ea9163c7a9f8c68d4c1fb64a807a0729
JIRA: NETVIRT-1318
Signed-off-by: Stephen Kitt <skitt@redhat.com>

syncDelete replaced with non sync delete

An attempt is made to remove syncDelete as sync operation is not needed
during DS remove operation in this case

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

Port ELAN service to datastore-constrained txns

* Most of the txRunners are converted to use DS constrained txns.
* Excluded methods are below
- Same tx used for both config and operational
- Methods where blocking calls are used.

JIRA: NETVIRT-1339

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

Tunnel update changes for ECMP

When tunnel is brought down, then instead of updating
routh paths, just refresh fib tables since route path
might be present already.

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

remove leftover netvirt-ui

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

disable statistics module

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

NETVIRT-1054 : FATAL:fib-entries container missing

If fib:show command is executed the very first time
after karaf bringup, when no routes have yet been
populated, the container will not be created. A GET
will then throw a 404.
In this case, printing a fatal messge to console
is not required. The commmand will only print an
empty table.

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

Port neutronvpn to Datastore-constrained txes

JIRA: NETVIRT-1348
Change-Id: I77132c3eae83bcec70d977b266a7b1dc19c1e5fa
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Port QosService to Datastore-constrained txes

JIRA: NETVIRT-1343
Change-Id: I3e80d5dd6139b2cdc6c73a9162c9612650525a17
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Remove vpn-portip-to-port entry for Floating IP

Remove vpn-portip-to-port entry for floating IP
after the Floating IP is deleted

JIRA: NETVIRT-1354

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

ECMP: Change bucket weight from 0 to 1

* OVS expect non zero weight for select group to work
* Changed bucket weight from 0 to 1

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

Fix for invalid vpnid in default snat flow

JIRA: NETVIRT-1358
Change-Id: I3af41be559316c203e49fcfa9bd7fda809fdf59c
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

vpnmanager typed tx: VpnUtil

But w/out the VpnInterfaceManager parts

JIRA: NETVIRT-1346

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

NETVIRT-1201 Upstream fixes to master --- 8

This fix does not associate subnet to vpn,
if another subnet with the same cidr is already associated
to any configured vpn with the same route-targets.
This is currently handled only for network association
to vpn.

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

Local Bucket Entry deleted

Verify ECMP traffic distribution with weighted buckets -
Controller Restart with CSS in standalone Mode
========================================================

After Controller Restart the Local Bucket Entry was getting deleted.
This was happening bacause of multiple simultanious event was causing
ConflictModifiactionException while updating the group entry. Now
we are using DJC to submit a job using SAME key for adding group and
flows, so that it would go to the samed jobqueue and writes get
executed serially, first the froups and then the flows), instead of
parallely and no comflict in writing by diffent thread to the same
node.

Change-Id: I23ad23d1db830213feb992897ee01de6148b156a
Signed-off-by: Dhiman <dhiman.ghosh@ericsson.com>

Add COE service-implementation mode

COE has currently two approaches for service
implementation. It can either use kubernetes's default
kube proxy mode, or its own custom odl-kube-proxy
mode. This patch adds a mechanism to choose between the
two options available.

Change-Id: I686eb9de5eb86cef9058e3802a19b8367333f21c
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Convert Vpn Utilities to Singleton

1) Change static methods to instance methods
2) Inject Utility instances into dependent classes

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

Refactor Ipv6Service module.

+ IPv6 generic/common functionality has been moved from Ipv6Service
  module to ipv6util module (Genius) in patch [1].
+ This patch handles removing of the corresponding code in Ipv6Service
  module.
+ VpnManager dependency on Ipv6Service has been updated to refer to
  ipv6util (Genius).

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

JIRA: NETVIRT-1213

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

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

remove checkstyle dependency from commons

because it now doesn't contain any custom Checkstyle checks code
anymore, just XML configuration, so it doesn't need to have the
core Checkstyle JAR as a dependency - there is nothing to build,
this just packages that XML as a JAR.

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

Fix For NETVIRT-1310

Problem:
- Regression bug that occured while modification
  of port chain with multiple flow classifiers.

Solution:
- Removed redundant code.

Change-Id: I5eaa66e101c0f3521710913d875c17d1e218dfff
Signed-off-by: ananth.y <ananth.y@hcl.com>

NETVIRT-1330: Fix ARP responder flows.

Use Router interface IP for ARP responder flows arp_tpa.

Change-Id: I2c4fefb1367776cde9299933b6554edc6b1d7dce
Signed-off-by: Achuth Maniyedath <achuth.maniyedath@gmail.com>

Convert coe to datastore-constrained transactions

JIRA: NETVIRT-1336

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

Switch to sevntu forbidden method check

The sevntu checkstyle checks introduced a forbidden method check in
version 1.28, use that instead of our own version.

Change-Id: I52e3558ac7f76efcfb092557b50216ad12fb2361
Signed-off-by: Stephen Kitt <skitt@redhat.com>

really fix the recent xtend-maven-plugin problem

the initial fix in I1982e1d4aeadfdb30e73f68cd5833661f148ccd5 & Co.
didn't quite do it (or worked for a few days and failed again?!)

see ODLPARENT-156 and https://github.com/eclipse/xtext/issues/1231
and https://bugs.eclipse.org/bugs/show_bug.cgi?id=536445.

JIRA: ODLPARENT-156
Change-Id: I31814763e10d205308b105da98e270fcc54a90ef
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix to advertise VRF routes from DC-GW

VRF routes from DC-GW is not advertised
to ODL primary RDs

Previously, when some routes are advertised from DC-GW
to quagga. since it was imported by a wrong RD which is
not primary, destination prefix (advertised by DC-GW)
in vrf-entry of fib was under wrong RD.

This is fixed now by checking if RD equals primary RD
(while advertising routes to quagga bgp), then send
an import RTs list, otherwise send an empty list.

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

Add napt switch id in log

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

Conflicit Modification Exception on napt-Switch

Description :
During External-network suite clean-up, its been observed that both
the listener RouterDpnChangeListner.remove() and
ExternalRouterListener.remove() triggered symultanously both of
which updating router-to-napt-Switch DS resulting in conflicting
modifiction exception.

Changes are done for RouterDpnChangeListner to use DJC with router
lock to prevent such conflicting Modification exception.

Issue: NETVIRT-1040

Change-Id: I35ec9e2eaffb537e1f99361cdccda70ec4214d4d
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Only raise alarm when RDs less than nexthops

In checkAlarmExtraRoutes, when number of RDs are lesser
than number of nexthops, ECMP code was not getting
triggered.

Hence, when number of RDs are lesser
than number of nexthops, then just raise the alarm.

Previously, in routesError, there were few routes which
were raised in alarm and not supposed to be used,
but now, we are not maintaining any error routes.

Plus, we already have a patch
https://git.opendaylight.org/gerrit/#/c/55726/
Here, if number of RDs are lesser than number of CSSs,
then the extra-routes, which are configured behind the CSS
for which RD is not allocated, are saved.
These unprocessed extra-routes will be handled during
vpn-instance update when new RDs are added.

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

OLFE in vpn-to-dpn-list

NETVIRT-1270 Conflicting modification for path
vpn-to-dpn-list/vpn-to-dpn-list

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