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>

remove counters-api dependency

following Id0af13c4fac1a657757235057ffce6e01cbd8e2f
which migrated from infrautils-counters to -meters,
we can now safely remove it from the POMs as well.

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

Remove vpn-portip-to-port DS after deleting VM

Issue: The vpn-portip-to-port entry associates with VM
remains after the VM is deleted. The issue is fixed by
deleting the vpn-portip-to-port entry along with the
adjacency when the VPN interface for the VM is deleted.

JIRA: NETVIRT-1309
Change-Id: Id4a1b630ffc9096ddd9d56bab1f5a10a59531761
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Remove vpn-portip-to-port after unset router GW

Issue: After router unsets external-gateway the
vpn-portip-to-port entry for router external gateway IP remains.

JIRA: NETVIRT-1313

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

Upgrade: wait for interface states for adjacencies

This issue was discovered in upgrade CSIT. The issue
is that usually L3 configuration comes in way after
tunnel interface states are present in the operational
database. However, in the case of upgrade this order
can be reversed. The following fix utilizes
DataTreeEventRegistrar to wait for the specific interface
states required to program the Vrfs. Note that this is
only done during upgrade. Also note that this fix has
been tested with the upgrade CSIT and works.

JIRA: NETVIRT-1312

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

Migrate from infrautils-counters to -meters

The former are deprecated and about to be removed.

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

To prevent casting related errors

- as seen in sandbox runs

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

Set Proper EtherType for MplsPopAction

Problem Description:
======================
When IPv6 L3VPN traffic is coming from DC-GW to OVS switch
via MPLSoGRE tunnel at table L3_LFIB_TABLE(20) is doing
removal of MPLS label and it is setting with ethertype
is 0x800(IPv4) always which is wrong for IPv6 traffic.

Example issue flow for
IPv6 Prefix(2001:db8:1111:0:f816:3eff:fed3:155a) and
MPLS Label (103021)
----------------------------------------------------
cookie=0x8000002, duration=87904.264s, table=20,
n_packets=5,n_bytes=340, priority=10,mpls,
mpls_label=103021 actions=pop_mpls:0x0800,group:152511

From the above example flow which is currently programmed on OVS
switch for IPv6 prefix and it is doing Pop MPLS label action
followed by setting with IPv4 etherType which is wrong for
IPv6 L3VPN traffic.

Solution:
=========
As part of this code changes, passing proper ethertype either
ETHTYPE_IPV4(0X0800) or ETHTYPE_IPV6(0x86dd) based on the
IpPrefix address family type.

Issue: NETVIRT-1307

Dependent Review:
==================
This issue is dependent on below GENIUS review

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

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

Remove learnt and Adjacent entries for GW after deleting external subnet

Learnt and adjacent entries for external subnet GW
remain in DS after the external subnet is deleted.
This prevents installing new learnt/adjacent/Fib entries
for external GW when the external subnet is re-created.
Hence the fix.

JIRA: NETVIRT-1301
Change-Id: I3cce5e5d284563d359551173c569ef53d616388e
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

QoS Log clean-up

QoS log tags have been modified for better debugging.

Change-Id: I6ec119429363735ccbd8cadc4e4628b2fb983f49
Signed-off-by: Meenakshi <meenakshi.c@ericsson.com>

Updating spec and yang based on QBGP apis

QBGP is not supporting updateBfd(). so its better to have one container
for all the BFD parameters so that multiple calls wont land into QBGP

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

Revert "disable building sfc features"

This reverts commit 6f5db153417bb833d979cb90f90816522dce8323.

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

Remove usage of SFC RPCs

Those RPCs were deprecated in Oxygen and they are removed in
Fluorine [1].

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

Change-Id: I945bc118910ff73dac82ab64f3f53804d1e65065
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Add missing parameters to RPC call

JIRA: NETVIRT-1133

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

Uncaught IAE in SNAT

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

Rate limiting for ARP packet punts

spec link:
http://docs.opendaylight.org/projects/netvirt/en/latest/specs/fluorine/controller-punt-protection.html

Issue: NETVIRT-1218

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

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

To include internal tunnel fetch

- to prevent NPE in ElanUtils.isTunnelInLogicalGroup
  when ItmScaling is enabled

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

Rate limit for subnet route punt packets for IPv6

Adds a learn action for subnet route punt flow which
installs a learn flow with higher priority than the
punt flow and matches on dst ipv6 address + vpn id
and drops the matching packets till the flow is
deleted after hard timeout expiry.

Depends-ON: https://git.opendaylight.org/gerrit/#/c/73025/

JIRA: NETVIRT-1213

Spec:
http://docs.opendaylight.org/projects/netvirt/en/latest/specs/fluorine/subnet-routing-for-hidden-ipv6.html

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

Helper patch for handling external router changes

While implementing IPv6 External Connectivity (FLAT/VLAN
based provider networks), we would need a listener to handle
changes to the external routers YANG model. It is seen that
the current implementation of ExternalRoutersListener is not
properly modularized. Making any further changes in that class
could introduce regressions for Controller, Conntrack based
SNAT and BGPVPN use-cases. So, this patch adds a new listener
SnatExternalRoutersListener for the Routers YANG model, but
only one of the two listeners would be active at a time as we
register the listener depending on the NatMode that is
configured in the deployment.

This patch mainly re-organizes the code and does not introduce
any functional changes. Its mainly a helper patch and the
necessary IPv6 pipeline changes would be done in a separate patch.

Change-Id: I424de2494abb1140b1f4d702d483498f9f072363
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
(cherry picked from commit 1d8d695790ffde338aa4c424cb0bc8a5db122f4b)
(cherry picked from commit 71e96d44231073cd0f300fa668f628e1cddb6a5d)

Update odltools

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

Add delay for EOR in BGP

Adding a delay option to wait for an EOR trigger from ODL

JIRA: NETVIRT-1231
Change-Id: Iac3ddfcdb04d722c005b2286fa450877e6877215
Signed-off-by: Loshmitha <loshmitha@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Add explicit org.osgi.core dependency - part 2

Change-Id: Iabe6eb19cfe0878c063dc75802dc25ca4a05bca7
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

disable building sfc features

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

SNAT Connectivity from non-FIP instances

When the NAPT switch is hosted on a node without the vlan footprint,
SNAT from the instance in the vlan on other nodes are failing.

We are now creating a vlan footprint in the NAPT switch by adding a
pseudo port in the elan-dpnlist for all the vlan added to the router
when the router g/w is set.

NETVIRT-1302 SNAT Connectivity from non-FIP instances to an external IP
fails when using VLAN setup (conntrack)

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

Add missing return in VrfEntryListener

Ia850f60e8497f449200aafb99b1cfcd3ec0c0107 dropped a return statement
from update()’s handling of updates to static routes, which could
result in extra processing if the update didn’t actually make any
changes.

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

Add explicit org.osgi.core dependency

This was picked up transitively from config-api but that
is going away.

Change-Id: Iab519494774ee665b0f618dfbcd401014e3385b5
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Use SingleTransactionDataBroker to read instead of VpnUtil read() - Part 5
of 5

Also replaces MDSALUtil.syncWrite() with SingleTransactionDataBroker
writes in the Vpn engine.
Uses ManagedNewTransactionRunner whereever applicable
Restricts VpnUtil.read() accessibility to within the class

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