Karthikeyan Krishnan [Wed, 18 Jul 2018 06:18:46 +0000 (11:48 +0530)]
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>
Aswin Suryanarayanan [Fri, 13 Jul 2018 11:20:41 +0000 (16:50 +0530)]
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>
Aswin Suryanarayanan [Fri, 13 Jul 2018 17:41:47 +0000 (23:11 +0530)]
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>
Sam Hague [Thu, 19 Jul 2018 17:08:57 +0000 (17:08 +0000)]
Revert "Add support for OVS NSH implementation"
This reverts commit
c0d8e1872ffd45cb33b70e2fd753ee3b2d1795c1.
Change-Id: I7aa9d0c75c7e4aa11ec84d0002f405e8b04385b1
Signed-off-by: Sam Hague <shague@redhat.com>
Jaime Caamaño Ruiz [Wed, 6 Jun 2018 15:45:46 +0000 (17:45 +0200)]
Add support for official OVS NSH implementation
Change-Id: I1bac27d8c39639d84da0aa83ef1a5703c6c215f7
JIRA: TSC-114
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Karthikeyan Krishnan [Tue, 17 Jul 2018 14:11:34 +0000 (19:41 +0530)]
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>
Sushant Kumar Jha [Fri, 13 Jul 2018 14:13:35 +0000 (19:43 +0530)]
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>
Vinh Nguyen [Wed, 27 Jun 2018 19:07:51 +0000 (12:07 -0700)]
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>
Robert Varga [Tue, 17 Jul 2018 19:13:18 +0000 (21:13 +0200)]
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>
Faseela K [Tue, 17 Jul 2018 19:39:23 +0000 (01:09 +0530)]
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>
Somashekar Byrappa [Wed, 11 Jul 2018 07:39:56 +0000 (13:09 +0530)]
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>
Karthikeyan Krishnan [Tue, 17 Jul 2018 10:26:26 +0000 (15:56 +0530)]
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>
xcheara [Wed, 27 Jun 2018 14:51:09 +0000 (20:21 +0530)]
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>
eswanit [Wed, 11 Jul 2018 06:14:47 +0000 (11:44 +0530)]
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>
kiranvasudeva [Thu, 12 Jul 2018 10:08:40 +0000 (15:38 +0530)]
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>
Sam Hague [Sat, 14 Jul 2018 19:08:50 +0000 (15:08 -0400)]
Fix indents in l3vpn-robustness spec
Change-Id: Ic1a7eb5e1a442d07ac0d9e5e290c696b1a5d3747
Signed-off-by: Sam Hague <shague@redhat.com>
Stephen Kitt [Fri, 13 Jul 2018 14:15:45 +0000 (16:15 +0200)]
Datastore-constrained txes: aclservice
Change-Id: Id11d6576f9179aa5249a35346d3f5e9e61748339
JIRA: NETVIRT-1331
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Shashidhar Raja [Fri, 13 Jul 2018 12:58:44 +0000 (18:28 +0530)]
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>
eceghkl [Wed, 11 Jul 2018 12:32:39 +0000 (18:02 +0530)]
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>
ashwinbc [Tue, 10 Jul 2018 09:14:30 +0000 (14:44 +0530)]
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>
ashwinbc [Wed, 11 Jul 2018 08:42:58 +0000 (14:12 +0530)]
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>
HANAMANTAGOUD V Kandagal [Tue, 8 May 2018 12:56:53 +0000 (18:26 +0530)]
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>
Stephen Kitt [Tue, 10 Jul 2018 15:26:05 +0000 (17:26 +0200)]
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>
Sridhar Gaddam [Sun, 8 Jul 2018 15:22:34 +0000 (20:52 +0530)]
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)
Stephen Kitt [Mon, 9 Jul 2018 16:35:48 +0000 (18:35 +0200)]
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>
Vishal Thapar [Mon, 7 May 2018 05:55:53 +0000 (11:25 +0530)]
Neutron QoS blueprint
JIRA: NETVIRT-1242
Change-Id: If76b8cd7bb4c1d12076970f26e50bcb515351877
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Stephen Kitt [Mon, 9 Jul 2018 16:17:07 +0000 (18:17 +0200)]
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>
Aswin Suryanarayanan [Wed, 4 Jul 2018 08:30:50 +0000 (14:00 +0530)]
NPE in SNAT conntrack tempest.
NETVIRT-1359 NPE in SNAT conntrack tempest.
Change-Id: I7213aaaf98823f612a3ddf0bff78bbf9069b2f44
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
eswanit [Mon, 2 Jul 2018 11:49:11 +0000 (17:19 +0530)]
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>
Stephen Kitt [Mon, 9 Jul 2018 15:15:56 +0000 (17:15 +0200)]
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>
Stephen Kitt [Mon, 9 Jul 2018 09:32:09 +0000 (11:32 +0200)]
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>
Sam Hague [Mon, 9 Jul 2018 17:41:26 +0000 (13:41 -0400)]
Convert sfc.classifier to ds-constrained txs
Change-Id: I2494015d24b5b745af72deda60e997e95748afd8
JIRA: NETVIRT: 1347
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Mon, 9 Jul 2018 12:14:23 +0000 (08:14 -0400)]
Disable building unused cloud-servicechain
Change-Id: I2418033327968da6c12f3b316f4aee73e9d7ca21
Signed-off-by: Sam Hague <shague@redhat.com>
Stephen Kitt [Tue, 3 Jul 2018 15:06:20 +0000 (17:06 +0200)]
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>
Shashidhar Raja [Thu, 5 Jul 2018 14:22:24 +0000 (19:52 +0530)]
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>
eceghkl [Wed, 4 Jul 2018 06:14:32 +0000 (11:44 +0530)]
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>
eswanit [Thu, 5 Jul 2018 04:30:30 +0000 (10:00 +0530)]
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>
Sam Hague [Thu, 5 Jul 2018 15:36:22 +0000 (11:36 -0400)]
remove leftover netvirt-ui
Change-Id: Iaabdd2b220d2db5f1ae79dbe83fce792ec857bf9
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Thu, 5 Jul 2018 15:35:03 +0000 (11:35 -0400)]
disable statistics module
Change-Id: Ief07ad503ebec436fa9d8125c4ab5d8b62d898f1
Signed-off-by: Sam Hague <shague@redhat.com>
eupakir [Thu, 5 Jul 2018 05:41:03 +0000 (11:11 +0530)]
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>
Vishal Thapar [Thu, 5 Jul 2018 04:27:12 +0000 (09:57 +0530)]
Port neutronvpn to Datastore-constrained txes
JIRA: NETVIRT-1348
Change-Id: I77132c3eae83bcec70d977b266a7b1dc19c1e5fa
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Vishal Thapar [Thu, 5 Jul 2018 03:32:00 +0000 (09:02 +0530)]
Port QosService to Datastore-constrained txes
JIRA: NETVIRT-1343
Change-Id: I3e80d5dd6139b2cdc6c73a9162c9612650525a17
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Vinh Nguyen [Fri, 29 Jun 2018 21:01:39 +0000 (14:01 -0700)]
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>
eceghkl [Mon, 25 Jun 2018 14:39:42 +0000 (20:09 +0530)]
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>
Vishal Thapar [Wed, 4 Jul 2018 06:18:09 +0000 (11:48 +0530)]
Fix for invalid vpnid in default snat flow
JIRA: NETVIRT-1358
Change-Id: I3af41be559316c203e49fcfa9bd7fda809fdf59c
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Josh [Wed, 27 Jun 2018 07:28:14 +0000 (10:28 +0300)]
vpnmanager typed tx: VpnUtil
But w/out the VpnInterfaceManager parts
JIRA: NETVIRT-1346
Change-Id: Ic1d9c7d434a58128c130c9772d3941ed57ac0081
Signed-off-by: Josh <jhershbe@redhat.com>
eupakir [Tue, 10 Apr 2018 07:23:04 +0000 (12:53 +0530)]
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>
Dhiman [Wed, 20 Jun 2018 09:06:13 +0000 (14:36 +0530)]
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>
Faseela K [Mon, 2 Jul 2018 12:00:17 +0000 (17:30 +0530)]
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>
eupakir [Fri, 8 Jun 2018 16:32:09 +0000 (22:02 +0530)]
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>
Somashekar Byrappa [Fri, 8 Jun 2018 13:43:36 +0000 (19:13 +0530)]
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>
Michael Vorburger [Tue, 3 Jul 2018 17:35:51 +0000 (19:35 +0200)]
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>
ananth.y [Sat, 30 Jun 2018 02:41:00 +0000 (08:11 +0530)]
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>
Achuth Maniyedath [Thu, 28 Jun 2018 09:16:46 +0000 (14:46 +0530)]
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>
Faseela K [Fri, 29 Jun 2018 16:35:07 +0000 (22:05 +0530)]
Convert coe to datastore-constrained transactions
JIRA: NETVIRT-1336
Change-Id: I50804bb25f8d25bba58d4fdb25ef330a92ab0851
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Stephen Kitt [Tue, 3 Jul 2018 08:50:43 +0000 (10:50 +0200)]
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>
Michael Vorburger [Tue, 3 Jul 2018 14:56:48 +0000 (16:56 +0200)]
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>
eswanit [Mon, 11 Jun 2018 05:27:48 +0000 (10:57 +0530)]
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>
Aswin Suryanarayanan [Tue, 3 Jul 2018 14:09:13 +0000 (19:39 +0530)]
Add napt switch id in log
Change-Id: Ic77fbead06b2365a1a65e17923afe76cc2d54927
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
xcheara [Sat, 23 Jun 2018 15:25:53 +0000 (20:55 +0530)]
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>
eswanit [Fri, 25 May 2018 06:28:13 +0000 (11:58 +0530)]
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>
Aswin Suryanarayanan [Mon, 18 Jun 2018 16:57:06 +0000 (22:27 +0530)]
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>
Sridhar Gaddam [Wed, 27 Jun 2018 17:28:37 +0000 (22:58 +0530)]
Fix NPE in updateVpnInterfacesForUnProcessAdj
Issue: NETVIRT-1353
Change-Id: I3fe772517f59d02aa92fd7117842a72304205e40
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
(cherry picked from commit
08699284e9398bff2d0eeaca0edc3c55965291d0)
Somashekar Byrappa [Thu, 17 May 2018 12:42:52 +0000 (18:12 +0530)]
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>
Faseela K [Thu, 28 Jun 2018 14:40:40 +0000 (20:10 +0530)]
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>
Somashekar Byrappa [Mon, 30 Apr 2018 09:23:11 +0000 (14:53 +0530)]
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>
nithi thomas [Tue, 8 May 2018 12:55:46 +0000 (18:25 +0530)]
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>
Qiuzheng [Tue, 26 Jun 2018 06:17:46 +0000 (14:17 +0800)]
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>
Arun Sharma [Tue, 26 Jun 2018 09:42:43 +0000 (15:12 +0530)]
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>
Stephen Kitt [Fri, 29 Jun 2018 08:31:05 +0000 (10:31 +0200)]
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>
Stephen Kitt [Fri, 29 Jun 2018 08:25:15 +0000 (10:25 +0200)]
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>
Philippe Guibert [Mon, 16 Oct 2017 15:35:20 +0000 (16:35 +0100)]
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>
Vishal Thapar [Thu, 28 Jun 2018 07:26:47 +0000 (12:56 +0530)]
Port aclservice to Datastore-constrained txes
JIRA: NETVIRT-1331
Change-Id: Ia8aa244c4e2e7c535efc2cabb13348beb237823c
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
gvrangan [Fri, 18 May 2018 01:16:47 +0000 (06:46 +0530)]
Fix For NETVIRT-1310
Change-Id: Ia6764eb1ad20da9e6245a89d82771c678f000e9b
Signed-off-by: gvrangan <venkatrangang@hcl.com>
Karthikeyan Krishnan [Tue, 26 Jun 2018 09:39:15 +0000 (15:09 +0530)]
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>
Robert Varga [Thu, 28 Jun 2018 17:32:05 +0000 (19:32 +0200)]
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>
Stephen Kitt [Thu, 28 Jun 2018 17:46:58 +0000 (19:46 +0200)]
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>
Aswin Suryanarayanan [Fri, 27 Apr 2018 04:28:27 +0000 (09:58 +0530)]
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>
Stephen Kitt [Wed, 30 May 2018 13:47:33 +0000 (15:47 +0200)]
Bump odlparent to 3.1.2
Change-Id: I8872028713c3926ff314b2084af82f37b62714d9
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Edw7n [Mon, 25 Jun 2018 05:36:49 +0000 (11:06 +0530)]
To fix errors seen in CSIT run in NAT
Change-Id: I61456ca77615c89cea50a4a113b4245a2af121b7
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
Sam Hague [Wed, 27 Jun 2018 00:08:51 +0000 (20:08 -0400)]
Remove unused import
Change-Id: Idb4e0233ffa8442922d84a33b264be0f03487a51
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 26 Jun 2018 14:51:27 +0000 (10:51 -0400)]
Port statemanager to datastore-constrained txes
JIRA: NETVIRT-1344
JIRA: NETVIRT-1318
Change-Id: Iaefd93dded121c056713d6ae9451153910a98b07
Signed-off-by: Sam Hague <shague@redhat.com>
Stephen Kitt [Tue, 26 Jun 2018 09:50:56 +0000 (11:50 +0200)]
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>
Michael Vorburger [Mon, 25 Jun 2018 11:27:35 +0000 (13:27 +0200)]
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>
Vinh Nguyen [Wed, 13 Jun 2018 23:52:28 +0000 (16:52 -0700)]
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>
Vinh Nguyen [Sat, 23 Jun 2018 00:11:46 +0000 (17:11 -0700)]
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>
Josh [Sun, 17 Jun 2018 11:29:42 +0000 (14:29 +0300)]
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>
Stephen Kitt [Mon, 11 Jun 2018 15:09:00 +0000 (17:09 +0200)]
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>
Edw7n [Fri, 22 Jun 2018 09:05:22 +0000 (14:35 +0530)]
To prevent casting related errors
- as seen in sandbox runs
Change-Id: Ic9a32083b03ced92f6ffde00024d6ee42e44d879
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
Karthikeyan Krishnan [Thu, 14 Jun 2018 09:55:07 +0000 (15:25 +0530)]
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>
Vinh Nguyen [Fri, 8 Jun 2018 22:47:16 +0000 (15:47 -0700)]
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>
Meenakshi [Mon, 28 May 2018 12:29:09 +0000 (17:59 +0530)]
QoS Log clean-up
QoS log tags have been modified for better debugging.
Change-Id: I6ec119429363735ccbd8cadc4e4628b2fb983f49
Signed-off-by: Meenakshi <meenakshi.c@ericsson.com>
Vyshakh Krishnan CH [Tue, 12 Jun 2018 08:52:44 +0000 (14:22 +0530)]
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>
Sam Hague [Fri, 22 Jun 2018 14:29:09 +0000 (14:29 +0000)]
Revert "disable building sfc features"
This reverts commit
6f5db153417bb833d979cb90f90816522dce8323.
Change-Id: Id9124c906f786483f2c83244e7d8d862415796fa
Signed-off-by: Sam Hague <shague@redhat.com>
David Suarez [Sat, 16 Jun 2018 08:22:00 +0000 (10:22 +0200)]
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>
Josh [Thu, 21 Jun 2018 08:08:18 +0000 (11:08 +0300)]
Add missing parameters to RPC call
JIRA: NETVIRT-1133
Change-Id: I82d32cee4814f60fa986585fcaef2c5d8119c7de
Signed-off-by: Josh <jhershbe@redhat.com>
Vishal Thapar [Wed, 9 May 2018 05:25:12 +0000 (10:55 +0530)]
Uncaught IAE in SNAT
JIRA: NETVIRT-1249
Change-Id: Ic8e561d0ad52f6104220a97219dc13b5aff1aafc
Signed-off-by: Vishal Thapar <vthapar@redhat.com>
Ravindra Thakur [Fri, 18 May 2018 08:20:26 +0000 (13:50 +0530)]
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>
Edw7n [Fri, 4 May 2018 06:51:20 +0000 (12:21 +0530)]
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>
Karthikeyan Krishnan [Thu, 24 May 2018 04:56:46 +0000 (10:26 +0530)]
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>