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>

Fix NPE in updateVpnInterfacesForUnProcessAdj

Issue: NETVIRT-1353

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

Subnet routing for hidden IPv6 addresses

As part of implementation of the spec, this patch handles:

+ Code has been enhanced to support subnet routing even for hidden
  IPv6 addresses.
+ Hidden IPv6 addresses are learnt via NA responses.

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

JIRA: NETVIRT-1213

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

Migrate to serviceutils/tools and serviceutils/srm

serviceutils repo has been created, and some of the utilities from
genius are moved over there. This patch is to migrate netvirt to
use the utils from serviceutils, so that we can remove the older utils
from genius asap

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

Punt IPv6 NA packets to controller

As part of implementation of the spec, this patch handles:

+ Punting IPv6 NA packets from hidden IPs to the controller. NA packets
  from Fixed IPs will be resubmitted to dispatcher table.
+ Provides RPC to send NS packet to an OF group. For this feature,
  NS packets will be sent to ELAN remote broadcast group.
+ Receives NA response, deserializes, publishes NA notifications to the
  registered users.

JIRA: NETVIRT-1213

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

IPV6 adress lost after ODL upgrade.

periodic IPV6 RA msgs missing after ODL upgrade from x to y , due to
which ipv6 address lost on all vm's.

Issue: NETVIRT-1248

Description:
During ODL upgrade , if cluster owner changes , the RA thread is not
getting restarted , due to which unsolicited RA msgs are not sent
from ODL.
     On every node-reboot, there is RE-connection of OVS(s) happens.
Once node is UP, MD-SAL replays configuration.
During which each listener can fire at random intervals and leading
inconsistent data store.
    This is observed only in a setup where 150 VNF's,
mulitiple networks, subnets, ...
       The issue is observed only in CLUSTERED-ODL upgrade.The same
issue was also reproduced without upgrade, "By rebooting
CLUSTERED-ODL nodes one after another sequentially".

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

The netviet sfc classifier get null vxlan-gpe port

Sometimes the ovs has two vxlan-gpe ports,
and one vxlan-gpe port number is null.The
patch avoid to get the null vxlan-gpe port
number.

Change-Id: I02063a7227520b1a55ae11c8b8d87d6ee5f01f80
Signed-off-by: Qiuzheng <rqzcsu123@163.com>

Do not apply rate limit rule on subport

Fix details: QoS rate limit rule is applicable on a neutron
port (at trunk level) and can not be applied at subport level.
If port has multiple subports then its total limit of all subports.
Only QoS DSCP policy can be applied on sub port.

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

Align Felix configadmin version with Karaf

Karaf 4.1.5 uses version 1.8.16 of configadmin, use that here too.

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

Re-enable FindBugs enforcement in elanmanager-impl

Now that we’ve upgraded to a version of ODL Parent with a fix for
ODLPARENT-146, we can re-enable FindBugs.

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

Dualstack support for L3VPN-two router design

According to dualstack specification [0], this commit introduces
following changes:

* neutronvpn.yang: Use "router-ids" leaf-list instead of "router-ids"
  leaf in grouping "vpn-instance" and "vpnMaps" container. Two neutron
  routers can be attached to one L3 BGPVPN instance at the same time:
  the first one with only IPv4 ports and the second one with only IPv6
  ports.

* NeutronvpnManager, NeutronvpnUtils: Support routerIds list in all
  basic methods. Control the number of associated routers in "add" and
  "handleRoutersUpdate" methods. Consequently assign each router
  accordingly.

* IVpnLinkServiceImpl: Appropriate changes, due to support of routerIds
  list in neutronvpn.yang.

* NatUtil:

  * "getRouterIdfromVpnInstance": control, that we get routerId from
    appropriate IPv4 Subnetmap, which is corresponding to given vpnId;
  * "getVpnForRouter" support routerIdsList, due to  changes in
    neutronvpn.yang.

* neutronvpn: rpc L3VPN support for assoc/disassoc routers list

Change yang model to associate or dissociate two routers with a vpnid in
the same curl query.

Change-Id: Iade14e52baee2da4a82f01231c7ea6f8051e520b
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

Port aclservice to Datastore-constrained txes

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

Fix For NETVIRT-1310

Change-Id: Ia6764eb1ad20da9e6245a89d82771c678f000e9b
Signed-off-by: gvrangan <venkatrangang@hcl.com>

CSIT:bound-service-state is not present for IPv6

Issue:
=======
The following warning message was throwing multiple times
from IPv6 service module in all CSIT gate job ran.

2018-04-05T17:51:37,507 | INFO  |
opendaylight-cluster-data-notification-dispatcher-593 |
FlowBasedServicesConfigListener  | 252 -
org.opendaylight.genius.interfacemanager-impl- 0.4.1.SNAPSHOT |Service
Binding Entry removed for Interface: 7642ee10-978f-4511-9416-
2697fa813982,ServiceName: ipv6.7642ee10-978f-4511-9416-2697fa813982,
ServicePriority 7 2018-04-05T17:51:37,508 | WARN  |
ForkJoinPool-1-worker-3 |FlowBasedServicesConfigListener  | 252 -
org.opendaylight.genius.interfacemanager-impl - 0.4.1.SNAPSHOT |
bound-service-state is not present for
interface:7642ee10-978f-4511-9416-2697fa813982, service-mode:class
org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager
.servicebinding.rev160406.ServiceModeIngress,
service-name:ipv6.7642ee10-978f-4511-9416-2697fa813982,
service-priority:7

Log_Ref:
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-upstream-stateful-oxygen/401/odl_1/odl1_karaf.log.gz

Solution:
==========
RCA: In IPv6service module network_dhcp interface also treated as a
VM host interface and it is doing IPv6 service binding and unbinding
of that interface is leading the problem for above warning message.

As part of this fix, ignoring network_dhcp interface for
binding/unbinding of IPv6 service.

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

Fix commons/checkstyle declaration

This bumps the odlparent reference to 3.1.2 and makes sure we are
using an empty relativePath.

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

Add Equinox Common to the Xtend Maven plugin

As pointed out by Robert Varga in
I556c934db07148f05f701d1c541fd1f1bfc34219,
org.eclipse.platform:org.eclipse.equinox.common:3.10.0 appears to be
required by the Xtend Maven plugin now.

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

Fixes stale fib flows for the g/w and PNF

The mip adjacency in the external interface and LearntVpnVipToPortData
for  external n/w is removed when the last external router is deleted.

This removes the stale flows in table21 and 51 and OLFE in vrfTables
update.

NETVIRT-1157 Stale flows after clearing gateway info from a router.

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

Bump odlparent to 3.1.2

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

To fix errors seen in CSIT run in NAT

Change-Id: I61456ca77615c89cea50a4a113b4245a2af121b7
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>

Remove unused import

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

Port statemanager to datastore-constrained txes

JIRA: NETVIRT-1344
JIRA: NETVIRT-1318
Change-Id: Iaefd93dded121c056713d6ae9451153910a98b07
Signed-off-by: Sam Hague <shague@redhat.com>

Port dhcpservice to datastore-constrained txes

This uses TransactionAdapter for calls to Genius, which will need to
be fixed once Genius migrates.

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