Arun Sharma [Fri, 8 Sep 2017 07:47:39 +0000 (13:17 +0530)]
NETVIRT-1079:Do not add a static route to Ubuntu
VM when nexthop IP is set to its own VM IP
Problem: Host route with next hop ip X is sent to VM with ip X.
This creats a loop.
Fix: Skip self route in DHCP response i.e. if next hop is self IP
then do not add that route in DHCP response.
Change-Id: I27a1ef1b6774659e817506535da2a0ed3546ed53
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>
Stephen Kitt [Tue, 13 Feb 2018 11:44:35 +0000 (12:44 +0100)]
elanmanager: use transaction manager
This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed. It also reworks a number of methods to use read/write
transactions, ensuring that changes are made based on still-relevant
data.
This doesn’t address transactions in ElanInterfaceManager.
Change-Id: Id38c7482cf2403a7751f0faa6134be7374e4a072
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Thu, 15 Feb 2018 11:06:03 +0000 (12:06 +0100)]
Split up creation/deletion, clean transactions
This patch splits up creation/deletion methods in
HwvtepNodeDataListener, avoiding the use of boolean parameters to
control method behaviour.
It also ensures the read transactions are closed, and cleans up
generics as far as possible, along with removing uses of
ResourceBatchingManager.
Change-Id: I648be29aed85fed71acb344236ecfa40497f84fc
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Deepthi V V [Mon, 15 Jan 2018 08:42:38 +0000 (14:12 +0530)]
Handle vpn removal from subnet correctly.
Network-id in subnetmaps was set to null causing remove subnet from vpn
event not being processed correctly. This fix ensures that network-id is
not removed from subnet.
Also, we do not store router-interfaces in subnetmap port list. Hence
removed logic which checks for device-id info.
Change-Id: I46955683a552b580b9ab3bb9930c98918c758a9c
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Somashekar Byrappa [Tue, 20 Feb 2018 12:29:50 +0000 (17:59 +0530)]
NETVIRT-1110: Fixed stale ACL service binding entries
Issue:
Stale ACL related service binding entries were found during CSIT run.
Solution:
ACL listeners are updated to handle even if the order of interface
events changes.
Change-Id: I69feca731e720e1ef50cd2a049c06fb4829da03b
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>
Sam Hague [Fri, 21 Jul 2017 20:49:33 +0000 (16:49 -0400)]
Add feature and compatibility docs
Change-Id: I4becffa7fe574a8f2bcff3dc4523bc40b034c0f4
Signed-off-by: Sam Hague <shague@redhat.com>
Stephen Kitt [Tue, 20 Feb 2018 16:45:11 +0000 (17:45 +0100)]
More transaction clean-up in ACL
This involves splitting a couple of add/delete methods too, and
converting a number of AclServiceUtils methods from static methods to
instance methods.
Change-Id: I769d4024d853fbfca99a3d9c253247ae2a35dd27
Signed-off-by: Stephen Kitt <skitt@redhat.com>
K.V Suneelu Verma [Fri, 16 Feb 2018 17:28:12 +0000 (22:58 +0530)]
removed halisteners
reused the existing node listeners to propogate child updates.
now the data gets copied from node listener only
added config manager listener to reconstruct ha cache upon restart
even if the node is not connected back post restart
fixed npe in ha ps node creation
copied db version of child ps node to parent ps node
Change-Id: Ic07bb876d05f02114e920ff270386b7f5324100b
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
Stephen Kitt [Tue, 20 Feb 2018 17:04:05 +0000 (18:04 +0100)]
sfc-classifier: use transaction manager
This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed.
Change-Id: I2852263856984b0228added475a99cb023fc962c
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Victor Pickard [Thu, 30 Nov 2017 19:36:56 +0000 (14:36 -0500)]
NETVIRT-1009 - Disable netvirt dynamic tunnels
https://jira.opendaylight.org/browse/NETVIRT-1009
Disable netvirt auto tunnels. There will be a separate
patch to enable genius auto-tz.
https://git.opendaylight.org/gerrit/#/c/66110/1
This will be the default starting in oxygen.
Carbon and nitrogen will continue to have netvirt
dynamic tunnels as the default.
Change-Id: I896c4d579cc105dedf4f9aa92275ba691850d214
Signed-off-by: Victor Pickard <vpickard@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Tom Pantelis [Wed, 27 Dec 2017 18:26:42 +0000 (13:26 -0500)]
Remove ElanInstanceManager#getElanInstanceByName method
This method always reads from the DS - we should use the
ElanInterfaceCache instead.
Change-Id: Ia3cb307a417d3ab4584c13e615c005bbed8de9ac
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Wed, 27 Dec 2017 17:59:58 +0000 (12:59 -0500)]
Refactor static WAITING_JOB_LIST in ElanInstanceListener
The WAITING_JOB_LIST functionality was moved to ElanInstanceCache.
Change-Id: I00910628748e59faf34bb19ccfb251f753b961ab
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Tue, 26 Dec 2017 16:40:23 +0000 (11:40 -0500)]
Refactor ElanUtil interface maps to a non-static singleton
Created an ElanInterfaceCache class to replace the static
elanInterfaceLocalCache and elanInstanceToInterfacesCache Maps
in ElanUtils.
Change-Id: I0f362fe749e1ab9e13ac2f0a92667d4336d43e72
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Sun, 24 Dec 2017 20:19:25 +0000 (15:19 -0500)]
Refactor elanInstancToDpnsCache to a non-static singleton
Created an ElanInstanceDpnsCache class to replace the static
elanInstancToDpnsCache Map in ElanUtils.
Change-Id: Iec19acb843ce74806db04ac9805771e69788b6d0
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Tom Pantelis [Sun, 24 Dec 2017 04:04:22 +0000 (23:04 -0500)]
Refactor elanInstanceLocalCache to a non-static singleton
Created an ElanInstanceCache class derived form DataObjectCache
to replace the static elanInstanceLocalCache Map in ElanUtils.
Change-Id: I4b8f489b7bdae70f3ebae456715bab81ab7e76e8
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
eaksahu [Mon, 18 Dec 2017 12:55:53 +0000 (18:25 +0530)]
Bug 8998 - stale l2gw connection cleanup
Upon cluster reboot fill the l2gw connection cache
by reading from config datastore.
Process l2gw connection even after dependent elan deletion.
Change-Id: Iee993e8f4e2bc6322b394543cb43408361d46e5c
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>
Loshmitha [Fri, 16 Feb 2018 09:07:59 +0000 (14:37 +0530)]
NETVIRT-1103 : display issue in ip bgp vpnv4 all summary command
White spaces inbetween the text was getting deleted which caused the
display issue,so changing the regex in replaceAll to resolve the issue
Signed-off-by: Loshmitha <loshmitha@ericsson.com>
Change-Id: I48c5019fec167649f09766531a5f3e7b29c9d42f
Signed-off-by: Loshmitha <loshmitha@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Fri, 16 Feb 2018 14:34:17 +0000 (09:34 -0500)]
Remove redundant names in paths
Change-Id: I443d41c2a994646aa4c2c2d61f114c3c50fb6d83
Signed-off-by: Sam Hague <shague@redhat.com>
Stephen Kitt [Thu, 15 Feb 2018 09:55:11 +0000 (10:55 +0100)]
ELAN: skip remote unicast MACs
The HWVTEP node data listener has a log message saying “Skipping
remote ucast macs to parent”, but it doesn’t actually skip. This patch
adds the apparently appropriate return, and ensures the transactions
are closed (without using the transaction manager yet, that’s coming
in a future patch).
Change-Id: I9a1fa2d5f3edeb4376a60c31e3a2e27b666bb034
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Sam Hague [Thu, 15 Feb 2018 20:53:07 +0000 (15:53 -0500)]
move vpnservice and cleanup poms
Change-Id: I16824568c0e89e33abc299d16e8d397c2cd95a90
Signed-off-by: Sam Hague <shague@redhat.com>
Vishal Thapar [Thu, 15 Feb 2018 16:07:13 +0000 (21:37 +0530)]
Fix build faliures due to OFPlugin checktyle fixes
OFPlugin checkstyle fixes have renamed some elements
used in Netvirt, which has broken netvirt builds. This
fixes it by using new names.
Change-Id: I53adf5f1db12bb5e6e179bca5c670faead8c67e6
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Sam Hague [Wed, 14 Feb 2018 21:02:42 +0000 (16:02 -0500)]
use feature for inter-project dependency
Change-Id: I7470d6dde622d8fa24df9fd4be156ce3085b6112
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Wed, 14 Feb 2018 21:02:24 +0000 (16:02 -0500)]
use feature for inter-project dependency
Change-Id: Ib503f61a94f4ee106794c83da9aaa5f7bcae9762
Signed-off-by: Sam Hague <shague@redhat.com>
Somashekar Byrappa [Tue, 9 Jan 2018 13:16:04 +0000 (18:46 +0530)]
ACL: Part-II implementation of the spec.
+ This is part-II implementation of the spec "Support for protocols that are not supported by conntrack".
+ Handles overlapping IP addresses while processing remote ACLs.
+ Handles ACL update scenarios.
+ Handles overlapping SG rule scenarios.
+ Fixed issues found during manual testing.
+ UT will be handled separately as part of https://git.opendaylight.org/gerrit/#/c/66553/.
Depends-On: Ie82fa8bf1eb139039247adb2321a53babe8fdc83
Change-Id: I44485f483fed7a00f3c7fa3601ba1b47b396c30a
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>
Somashekar Byrappa [Mon, 6 Nov 2017 10:16:11 +0000 (15:46 +0530)]
ACL: Support for non-conntrack supported traffic.
+ This is implementation for spec "Support for protocols that are not
supported by conntrack".
+ This involves redesign of entire ACL pipeline.
+ UT is disabled in this patch. It will be handled separately as part of https://git.opendaylight.org/gerrit/#/c/66553/.
This patch currently addresses (a) and (c) of below three issues as
mentioned in the spec:
a. Enhance ACL to support protocols like OSPF, VRRP etc that are not
supported by conntrack in stateful mode.
b. Handle overlapping IP addresses while processing remote ACLs.
c. Optimization for Remote ACL by reducing number of flows even for ports
having multiple ACLs.
Patch-1:
+ ACL default flows are changed as per the new pipeline.
Patch-2:
+ Updated bind service to exclude writing ELAN/VPN ID into metadata.
+ Removed dependency on vpnmanager and it is not required anymore.
Patch-3:
+ Rebased
Patch-4:
+ Fixed compilation issues. Removed VPN-ID related code.
Patch-5:
+ Handled programming general fixed flows.
+ Handled programming port specific fixed flows.
Patch-6:
+ Rebased
Patch-7:
+ Handled programming ACL commit flows.
+ Refactored: Moved programAceSpecificFlow() logic from
Stateful***AclServiceImpl to AbstractAclServiceImpl class.
Patch-8:
+ Handled programming ACL dispatcher table.
Patch-9:
+ Updated programming ACL dispatcher table.
+ Handled programming Remote ACL table.
Patch-10:
+ Rebased
Patch-11:
+ Code-cleanup
Patch-12:
+ Rebased
Patch-13:
+ Handled port update. Update for port-security-enabled, AAP and SG
changes.
Patch-14:
+ Rebased
Patch-15:
+ Handle port-update with allowed-address-pair change.
Patch-16:
+ Used DJC to program ACL node default flows.
+ Used single transaction for programming ACL node default flows.
+ Refactored AclNodeListener.
Patch-17:
+ Refactored to keep single level of abstraction as no other
security-group modes are supported except stateful.
+ Updated to display cache aclTagMap.
Patch-18:
+ This patch was mistakenly uploaded by Nishchya.
Patch-19:
+ Rebased.
Patch-20:
+ Changed references from METADATA_MASK_REMOTE_ACL_ID to
METADATA_MASK_REMOTE_ACL_TAG.
Patch-21:
+ Rebased.
Patch-22:
+ Rebased.
Patch-23:
+ Rebased.
Patch-24:
+ Rebased. Aligned [0] with the latest ACL pipeline.
[0] https://git.opendaylight.org/gerrit/#/c/66788/
Patch-25:
+ Rebased.
Depends-On: I95df598428f6351e2abb0b173a4318253c9e20bc
Change-Id: Ie82fa8bf1eb139039247adb2321a53babe8fdc83
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>
eaksahu [Fri, 15 Dec 2017 10:57:45 +0000 (16:27 +0530)]
Bug 8998 - Stale l2gw connection cleanup
upon cluster reboot if the device is not connected
it is not shown up in the cache or oper datastore.
Now when l2gw connection delete request is received it has no clue
on which device it has to delete the things and ignores the
delete request.
when the device connects later, cleanup job is scheduled
which will look for stale logial switches and deletes them.
any logical switch which is not present in networks dump or
l2gwconnections dump is treated as stale logical switch.
cleanup job is scheduled instead of firing immediately because
the network datastore may not be loaded yet as the cluster may
be just started.
cleanup if the device never shows up is not done.
Change-Id: I84d5e956ab6a94988f9ddf8faa8f3cf62f5c07d1
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>
HANAMANTAGOUD V Kandagal [Fri, 15 Sep 2017 10:14:06 +0000 (15:44 +0530)]
For scaling to 150DPN , each TunnelIntfState add event causes a read of list of
interfaces on the given DPN.This list includes even non L2Vlan type
interfaces like tunnel-port etc. By ignoring these ports , read of
operational vpn-interface is avoided.
Depends-on: https://git.opendaylight.org/gerrit/63160
Change-Id: I3dd089f9409a8701ccedb64b3a85b1f40862499f
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Achuth Maniyedath [Sat, 23 Sep 2017 12:50:14 +0000 (18:20 +0530)]
NETVIRT-1033: Adding ArpResponder flows for SR-IOV VMs
DHCP Neutron Port ARP handling for SR-IOV VMs.
Handled removal of stale flows entry from ArpResponderTable.
New pipeline for processing Neutron port DHCP ARPs
Table 0 -> Table 18 -> Table 38 -> Table 81
Change-Id: Id5ee3160240cfcb6caec5790734d9f003c16640e
Signed-off-by: Achuth Maniyedath <achuth.maniyedath@gmail.com>
Signed-off-by: Vijayalakshmi Chickkamenahalli Nagaraju <vijayalakshmi.c@altencalsoftlabs.com>
Stephen Kitt [Wed, 14 Feb 2018 17:34:59 +0000 (18:34 +0100)]
Avoid shadowing fields
A couple of NeutronvpnUtils methods have parameters named dataBroker,
which shadow the field by the same name. These parameters aren’t
necessary any longer, this patch drops them.
Change-Id: I234fe5a2607911121cb49d6d48006517227f9d45
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Sam Hague [Wed, 14 Feb 2018 13:30:34 +0000 (08:30 -0500)]
Add netvirt-features
Change-Id: I8e2afcf08d523219f488c1b9ec9ad82330ba4557
Signed-off-by: Sam Hague <shague@redhat.com>
eupakir [Mon, 12 Feb 2018 14:34:42 +0000 (20:04 +0530)]
NETVIRT-1037: Upstream Fixes --- Review-6
1) Support L3 Connectivity for subnets with no-gateway option set
2) Introduce jobCoordinator retries for critical tasks
3) Fix potential exception causing code
4) Avoid unnecessary request for ids much before its usage.
Change-Id: Ia3a41c9a1aaccfd1a4d1d8baa059892636b01efb
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Philippe Guibert [Tue, 28 Nov 2017 09:44:26 +0000 (09:44 +0000)]
neutronvpn: update bgpvpn fallback pipeline rules for ipv6
When associating a BGPVPN to an external network, the identifier
associated to that BGPVPN is already used for all the underlying
external subnets declared in that network.
For IPv6 GUA subnets that are "behind that external network", that is
to say behind a router which has that external network linked, that same
identifier is used to write a specific fallback route for those IPv6 GUA
to access the Internet through BGPVPN. Indeed, when the router has no
more entries in its FIB table, a fallback rule is written for IPv6 GU
traffic against possibly BGPVPN IPv6 entries. That fallback rule
resubmits the packet to the FIB table, but with an ID set to the BGPVPN
id. Like that, all IPv6 traffic that has an entry associated may be
redirected to the correct DC-GW.
The modification here consists in adding that zero priority fallback,
rule, in the case a BGPVPN is present on external network, on the
presence of the first IPv6 subnet; removing the fallback rule, if the
BGPVPN is detached or the last subnet ipv6 is removed.
Change-Id: Iebd754ae10fa6db08512980a98e07c3a02ebb6d5
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Sridhar Gaddam [Tue, 13 Feb 2018 20:34:42 +0000 (02:04 +0530)]
NETVIRT-1085: Fix regression in IPv6PeriodicTimer Unsolicited RAs
While addressing the findbugs in Ipv6Service (I635ff43cffded41c7a32079214cff48a057f4fac)
there was a regression introduced, where the periodic unsolicited RAs are no longer sent
out. This patch fixes this issue.
Change-Id: I73faf5d50b7237732675fcd951926ecf4a9b492d
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Stephen Kitt [Tue, 13 Feb 2018 11:48:19 +0000 (12:48 +0100)]
Return the tagged ELAN instance
ElanUtils::updateOperationalDataStore() is given an ElanInstance to
update, but because YANG instances are immutable, it has to create a
new one to add the ELAN tag. As a result, calling code has to waiting
for the surrounding MD-SAL transaction to finish, and then retrieve
the updated ElanInstance from the MD-SAL... This patch makes the
update method return the updated ElanInstance, potentially avoiding a
MD-SAL round-trip and enabling more asynchronous operations.
Change-Id: Ia83d65b2d0953c97976d37af0bb4299841925b8e
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Sridhar Gaddam [Tue, 13 Feb 2018 10:04:52 +0000 (15:34 +0530)]
IPv6Service fix stale flows during VM migration
This patch fixes the stale flows during VM migration by
invoking bind/unbind from Ipv6 Interface State Listener.
Change-Id: I74c4d591aad3ce4785580237bfae54f7d1ea2bfa
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
eupakir [Mon, 22 Jan 2018 12:17:30 +0000 (17:47 +0530)]
NETVIRT-1037: Upstream Fixes --- Review-5
1) Fixes cleanup of Table 19 flows for network-based-bgpvpns.
2) Also removes the check for schedule-to-remove flag during
vpn-op-interface removal, covering that gap with DJC retries.
3) Enhances a few log statements.
Change-Id: If48f84edfced7945db9a5a8b941d0bb04bcc3c7d
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Stephen Kitt [Mon, 12 Feb 2018 15:31:32 +0000 (16:31 +0100)]
dhcpservice: use transaction manager
This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed.
Change-Id: I3559cf891fddc7ee89afb8516718b2247eeff86d
Signed-off-by: Stephen Kitt <skitt@redhat.com>
eupakir [Wed, 6 Sep 2017 12:41:11 +0000 (18:11 +0530)]
NETVIRT-1037: Upstream Fixes --- Review-4
Introduces a few useful fixes in the VPN Engine.
Particularly, some of the yang-based flags used during the VPN deletion
life-cycle are not cleared properly if the deletion fails, leading to
stale vpn op interface entries.
Change-Id: I817338174bfda0ea881a066484d944f013138b8e
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Tom Pantelis [Thu, 21 Dec 2017 04:46:20 +0000 (23:46 -0500)]
Move VpnInstanceOpDataEntry cache from DataStoreCache to DataObjectCache
The DataStoreCache for VpnInstanceOpDataEntry instances was accessed by
VpnUtil.isVpnIntfPresentInVpnToDpnList which was only called by the
VpnInterfaceManager so the method was moved to VpnInterfaceManager which
now also contains the VpnInstanceOpDataEntry DataObjectCache instance.
The VpnOpInstanceCacheManager DTCL was removed since the DataObjectCache
has its own internal DTCL.
Change-Id: I96cbe45eb33c5a6b595d18dde34a457c4ce3dbc5
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Philippe Guibert [Fri, 19 May 2017 09:14:53 +0000 (11:14 +0200)]
neutronvpn: fallback default rule implementation
This utility function is used by external networks to create an IPv6
fallback rule for external VPN.
This API should support the creation and destruction of fallback rule.
Change-Id: Ib197b8ba293cae4b9cc3c9732c812e0922056644
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Michael Vorburger [Tue, 13 Feb 2018 09:18:33 +0000 (10:18 +0100)]
fix completely broken distribution/karaf
we broken this in I34584eccd14da25fde24779247e78db167d7587a,
following Iaf56adcb93490f9b29e34a50d33b324bb24a9b07; this fixes it.
Change-Id: Ie09a938bc1d36319a06acf8f46f00599d0ed0550
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Kiran Kumar Darapu [Thu, 18 Jan 2018 06:18:31 +0000 (11:48 +0530)]
Migrating BGP PM Counters to Infrautils Counters
Existing BGP PM counters were using JMX MBeans,
Now new BGP PM counters were imoplemented using
Infrautils metrics counters APIs which will be common library
to populate JMX MBeans for these counters by leveraging
Codahale framework.
Change-Id: Ib458d151ffed331ccd2b717ad3abef0ebecf3f80
Signed-off-by: Kiran Kumar Darapu <kiran.kumar@altencalsoftlabs.com>
Siva Kumar Perumalla [Mon, 12 Feb 2018 16:34:38 +0000 (22:04 +0530)]
BGP-EoS leader flap leads to NPE.
Change-Id: Idfc5dec1416a93728eae2b9a307b456a3a6177cc
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>
Stephen Kitt [Mon, 12 Feb 2018 09:28:09 +0000 (10:28 +0100)]
cloud-servicechain: use transaction manager
This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed.
Change-Id: I42434cde5080a9bf940f8b329f962f5fe0b2aaff
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Mon, 12 Feb 2018 09:43:37 +0000 (10:43 +0100)]
coe: use transaction manager
This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed.
Change-Id: Ida638029de40029ff93353665233d156a4164ff4
Signed-off-by: Stephen Kitt <skitt@redhat.com>
eupakir [Tue, 12 Dec 2017 14:03:26 +0000 (19:33 +0530)]
NETVIRT-1037: Upstream Fixes --- Review-3
This reviews stores gwMac and lportTag into operational vpn interfaces,
in-case retrieval of the same through ietf-interfaces state fails while
handling an interface remove event
Change-Id: Idbeeb9486087786ec38003d198d6ffc30fe0daf5
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
eupakir [Tue, 12 Dec 2017 10:48:49 +0000 (16:18 +0530)]
NETVIRT-1037: Upstream Fixes --- Review-2
If an extra-route/learnt IP has the same IP as a primary adjacency on the
interface, it is not processed.
Change-Id: Ib2bd26da897896233e8ed5fb668780d0d0ce3d79
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
eupakir [Tue, 12 Dec 2017 09:21:58 +0000 (14:51 +0530)]
NETVIRT-1037: Upstream Fixes --- Review-1.
1) Handle VrfEntry updates only for static routes/cluster reboot scenario
2) Fix a few Exceptions seen in the logs during TEP events
Change-Id: Ia992bda38a61b6787170aa7884f189584f82ca6d
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
HANAMANTAGOUD V Kandagal [Tue, 23 Jan 2018 12:25:00 +0000 (17:55 +0530)]
NETVIRT-1068: Upstreaming fixes
Issue-1: When VpnInterface has a secondary adjacency like invisible ip, if tunnel
EP is deleted , VpnInterface Op DS is updated with empty nexthoplist.
During this time , invisible ip goes missing from VpnInterface Op DS due
to MDSAL.syncwrite() being done in updateVpnInterfaceOnTepDelete. Another
fix being done here is , invisible ip FIB entry is updated with empty
nexthop and route is withdrawn/advertized to BGP
Issue-2: prefix-A part of subnet-A/network-A/VPN-A. The refcount for
prefix-A is 1. When VPN-B is importing from VPN-A , refcount is 2. But
when prefix-B part of VPN-B is deleted , refcount was not decremented.
For self-imported routes , when the prefix is deleted , refcount is
decremented.
Change-Id: I2cabeaa1ae23e356b2d17a527e9bbc6a5b21afd0
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Deepthi V V [Fri, 12 Jan 2018 07:13:33 +0000 (12:43 +0530)]
l3vpn app data clean up on vm delete.
L3VPN application data is not cleaned up on vm delete and
dissociate network/router scenarios. This review fixes the issue.
Change-Id: Id1d7c89206c5bea77925320b67c0e12d6ffbf2bf
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Stephen Kitt [Wed, 22 Nov 2017 08:34:22 +0000 (09:34 +0100)]
ACL: use transaction manager for write transactions
This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed.
Change-Id: Id148f5f5dc1bee69805e096136d509f4f6c3209a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Siva Kumar Perumalla [Thu, 8 Feb 2018 13:22:10 +0000 (18:52 +0530)]
update thrift version in feature.xml
from 0.9.1 to 0.9.3 as BgpConfigurationManager is dependent on 0.9.3
Change-Id: I6a0c7acf6c7775f59bbce6b0c82aeb606618a795
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>
Julien Courtat [Wed, 8 Feb 2017 17:16:42 +0000 (18:16 +0100)]
IPv6 BGPVPN DC to internet detailed description
This blueprint spec change introduces a new solution that relies on
external network from neutron router. The spec analyses the problem and
gives a solution.
Change-Id: I30c90380fa9b3cf2d8a7705090fa57b7d6043608
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Philippe Guibert [Tue, 22 Aug 2017 10:32:46 +0000 (11:32 +0100)]
Bug-9029: Bgp Manager / End of Rib feature complements
This commit change brings some complement of changes in the generated
thrift files.
Change-Id: I13e8c066039b9c27452e09915fbc518f62ef681f
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Jamo Luhrsen [Tue, 10 Oct 2017 16:29:56 +0000 (09:29 -0700)]
Remove security group mode config knob
Stateful is the only security group mode we have now.
Change-Id: Id30042b1a2ca1ef371079bd43640262613265924
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
Vishal Thapar [Fri, 27 Oct 2017 10:00:13 +0000 (15:30 +0530)]
Optimize NAPT switch selection logic
NaptSwitch selection logic today sorts switches in descending order of
weight and then uses iterator to find last entry. By changing the
comparator and convreting it into ascending order, we can return the first
entry without needing to iterate through list of switches.
Change-Id: I6620d9e5a92ea60d100ed2d767e73fb1bdef998e
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Vishal Thapar [Tue, 23 Jan 2018 08:13:06 +0000 (13:43 +0530)]
Refactoring and Duplicate Id detection
Change-Id: I4c76953a9dec8bf044374ae1a6dd6afb007bf4b2
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Josh [Mon, 29 Jan 2018 08:42:54 +0000 (10:42 +0200)]
Remove illegal characters from yangs
Change-Id: I32f933305dbfcc759501ce7e238e61a3d16698a0
Signed-off-by: Josh <jhershbe@redhat.com>
Aswin Suryanarayanan [Thu, 18 Jan 2018 08:02:09 +0000 (13:32 +0530)]
NETVIRT-1063 SNAT flows fails to install sporadically(conntrack SNAT)
The vpind now retrieved using the subnet uuid from the router.
Change-Id: I6e8c0ce53349e40a336db12b247822f33c483c75
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Stephen Kitt [Fri, 9 Feb 2018 16:55:59 +0000 (17:55 +0100)]
Use NwConstants in ...StatefulIPv6.xtend
This should avoid regressions once we start using the Xtext
representation of IPv6 stateful flows entry objects.
Change-Id: I221269f410f5e85bdefe72491977bb62ca3cc559
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Fri, 9 Feb 2018 09:57:41 +0000 (10:57 +0100)]
Drop obsolete “as short” .xtend conversions
With the switch to constants, many “as short” conversions are no
longer necessary.
Change-Id: I4aee4183486a29c081ca8121dda81bf81d6aeed8
Signed-off-by: Stephen Kitt <skitt@redhat.com>
karthikeyan [Thu, 1 Feb 2018 11:41:42 +0000 (17:11 +0530)]
NETVIRT-1074: DNAT traffic is not working if SNAT is Disabled
Problem Description:
======================
FIP to External Network or FIP to FIP traffic is not working if SNAT is
disabled for the router. Since L3_FIB_TABLE (21) -> PSNAT_TABLE(26) flow
is missing on the FIP associated switch.
Solution:
=========
If SNAT is disabled, we need to take care of DNAT (FIP) traffic should
work as expected. By default when router-gw is set with external
network/subnet SNAT is enabled and default NAT flow on Table 21 will be
getting programmed. In case of SNAT is disabled this flow won't get
installed on the required FIP configured switches. Hence FIP traffic is
getting dropped at Table 21. Have added the logic to program the NAT
default flow in Table 21 if SNAT is disabled UC as well.
Change-Id: Ie6c66c9571d5698408317718bfaa61ce677472c4
Signed-off-by: karthikeyan <karthikeyangceb007@gmail.com>
Vyshakh Krishnan CH [Fri, 9 Feb 2018 05:00:12 +0000 (10:30 +0530)]
Setting BGP default log to debugging
BGP default log was set to error and due to which qbgp logs are not
shown. making it to debugging now.
Change-Id: I66826d34ce2cb6448261b6c9287fac63eacdcda0
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>
Stephen Kitt [Mon, 5 Feb 2018 09:56:16 +0000 (10:56 +0100)]
Switch some stragglers to odlparent 3.0.2
Change-Id: I98d03a63001971dc526d89d3e2df580bf304c10b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
kaoru sueda [Wed, 29 Nov 2017 07:36:40 +0000 (16:36 +0900)]
NETVIRT-984: Fix of issue that l2gw connection creation failed in L2GW HA environment.
Issue:
Exception occurred in l2gw connection creation and caused OVSDB
transaction is not executed to the end in L2GW HA environment.
Cause:
"replication-mode" is not set in HWVTEP logical-switches entry in
network topology config DS. And this caused the
NoSuchElementException in l2gw connection creation.
Fix:
Fix this by set replication-mode in the above entry.
Change-Id: I022e493abdbd20d2279540be790bf018c8dc2032
Signed-off-by: kaoru sueda <k-sueda@zj.nes.jp.nec.com>
VinothB [Thu, 1 Feb 2018 18:32:20 +0000 (13:32 -0500)]
NETVIRT-1044 fix for Exception in karaf when delete neutron port
Null check for port's dpId is missing in this particular method whereas
all other methods having null check for dpId in AbstracAclServiceImpl
Change-Id: I8c4f5768af2a04b7921915264b6f5089acb9a7bb
Signed-off-by: VinothB <vinothb@hcl.com>
eaksahu [Wed, 13 Dec 2017 11:06:01 +0000 (16:36 +0530)]
Bug 8998 - Vlan Binding missing on connection add
missing child entries when connection is deleted and added immediately.
Since we are using batch manager previous delete
is still not finished by the time next add request comes.
Since the data is not deleted yet this add request will become no op.
delete from batch manager eventually succeeds and we end up entry missing.
make sure that add request does not proceed until the previous delete
request has not completed.
Change-Id: I88c887a4f40ad96ae2230e3a5362e8cc1cbde87f
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>
Noel de Prandières [Tue, 5 Dec 2017 11:10:44 +0000 (12:10 +0100)]
vpnmanager: subnetopdataentry context creation
Update Adjacencies to BGPVPN Internet, if the subnetmap handles refers
to an IPv6 Subnet.
Enhance the subnet-show command to display all the subnetmaps and
subnetmapopdataentries.
Change-Id: I7efba2cc94e65c1862aee96e3e3591a3d14bc630
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Valentina Krasnobaeva [Mon, 7 Aug 2017 14:21:23 +0000 (16:21 +0200)]
neutronvpn: subnetmap update for internet-vpn-id support
Add internet-vpn-id leaf in the subnetmap list. This model change was
needed to provide a capability to associate an external network with L3
BGPVPN.
Update internet-vpn-id field in subnetmap before associating provider
neutron network with L3BGP VPN.
Add an enumerate in vpn instance op data entry, to qualify the
kind of BGPVPN: BGPVPN external, or BGPVPN internet.
Change-Id: Icab050c89a25beb12b5be3a586d84ee97a7eb593
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Philippe Guibert [Fri, 8 Sep 2017 14:10:33 +0000 (15:10 +0100)]
neutronvpn: unlock multiple vpn configuration per iface
This change unlocks the ability of neutronvpn to attach a vpn
interface to two different vpns, or one router and a L3VPN, for
instance.
This is here the case for an internetVPN that is attached to a VPN
interface, in addition to the router.
Change-Id: I7c65b3ce981db0072ba2ab23323d44f2cf689f21
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Stephen Kitt [Wed, 7 Feb 2018 14:17:40 +0000 (15:17 +0100)]
Add aaa-cli dependency for the distribution
We need to *guarantee* that the aaa-cli JAR is available for
maven-dependency-plugin to copy it correctly; adding the dependency
explicitly ensures that the JAR will be in the local repository.
Change-Id: I34584eccd14da25fde24779247e78db167d7587a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Aswin Suryanarayanan [Fri, 19 Jan 2018 15:17:50 +0000 (20:47 +0530)]
Include AClStatefulTest
Change-Id: I899af0163ab811ed8bdece7ee5555c43b3340fbb
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
vinothb [Wed, 13 Sep 2017 23:10:00 +0000 (19:10 -0400)]
Bug 8400 - ACL changes doesn't affect the existing connections
Problem:
Change in SG rules does not affect the existing traffic
Fix provided:
* "ct_mark" flag of conntrack module will be used to identify the
change made in acl rules of specific instance.
* When a Acl rule removed from the instance, Netvirt will reset the
value of "ct_mark" flag value. Reset of ct_mark value will skip the further
packets of existing traffic to not be matched by conntrack flow and these
packets will be dropped by the DROP flow.
Change-Id: If55070a7810b580af1cb922abde2880cf434b371
Signed-off-by: vinothb <vinothb@hcl.com>
Faseela K [Tue, 30 Jan 2018 07:51:50 +0000 (13:21 +0530)]
Pass MetricProvider to JC Test Module
This requires Ie90562c2f8205bab8817d370d077def9a3419d52 and
I530d20946a80933b05371c5220def447be1ee5e6 in infrautils in order to be
able to compile, and must be merged together (with the 2nd, the first
can go in before).
Change-Id: If3c3bcdb8a972c7e57cff20fd24d5addefa7746b
Signed-off-by: Faseela K <faseela.k@ericsson.com>
cgowdru [Tue, 9 May 2017 09:23:10 +0000 (14:53 +0530)]
Conntrack-based SNAT support for GRE/VXLAN Type
Description : Provided changes for supporting External-Network
Provider-Type of GRE and VXLAN using Conntrack-based SNAT.
Change-Id: I7b1115173f765159ff4d0538c0c6e04f82932f90
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Siva Kumar Perumalla [Tue, 12 Dec 2017 15:10:59 +0000 (20:40 +0530)]
Faster DC-GW Failure Detection (L3VPN with BGP)
Change-Id: I742ca73de057fddbf91f76ff9b3b18849a5774f8
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>
Stephen Kitt [Wed, 31 Jan 2018 12:49:08 +0000 (13:49 +0100)]
Add odl-jolokia to the NetVirt distribution
Change-Id: I1cf93bfa2e36024d16c3e188edf331bdfe312917
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Jaime Caamaño Ruiz [Wed, 31 Jan 2018 16:01:04 +0000 (17:01 +0100)]
Fix getRenderedServicePath sfc test hanging
Change-Id: I8970f01d41313df7ce65ef83e4c7dfd166e20ea6
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Michael Vorburger [Wed, 31 Jan 2018 11:25:37 +0000 (12:25 +0100)]
add aaa-cli-jar to distribution/karaf
see Ie47ea696190c9fad6472375007b2767db1f0fbed
and Ifdb3e87cee4cb80f0c753ead620077cbf98af0f8
Change-Id: Iaf56adcb93490f9b29e34a50d33b324bb24a9b07
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Sam Hague [Tue, 30 Jan 2018 19:18:18 +0000 (14:18 -0500)]
Ignore hanging UT: getRenderedServicePath
Change-Id: I224fa3d43b7b4811f9891753ad43abde579d19ae
Signed-off-by: Sam Hague <shague@redhat.com>
D Arunprakash [Thu, 11 Jan 2018 11:55:17 +0000 (17:25 +0530)]
OPNFLWPLUG-972 : Point to openflowplugin liblldp
Migrated liblldp from controller to openflowplugin
Change-Id: I2a0ef324d764d6908b4c9b03f3ac168eaac9f662
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
Stephen Kitt [Fri, 19 Jan 2018 10:22:49 +0000 (11:22 +0100)]
Re-enable SFC
Change-Id: I7f259d1932ce3e6d64d1dd75253f46d616cd363b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Sam Hague [Tue, 30 Jan 2018 00:34:22 +0000 (00:34 +0000)]
Revert "Revert "Add hardware offload support for openvswitch""
This reverts commit
69ab0b21fe50db21443616a2a0e6bcd5e4806c11.
Change-Id: If6121416f62ea031185ae2e740bd999f2d0b9ea7
Signed-off-by: Sam Hague <shague@redhat.com>
Sam Hague [Tue, 30 Jan 2018 00:31:43 +0000 (00:31 +0000)]
Revert "Add hardware offload support for openvswitch"
This reverts commit
29d3ba79843ef4dfdd1461f5c12c071bde1cbd8f.
Change-Id: Ib42e63bc268b3777e6703c62d62042152d0977d9
Signed-off-by: Sam Hague <shague@redhat.com>
Moshe Levi [Wed, 12 Jul 2017 19:50:17 +0000 (22:50 +0300)]
Add hardware offload support for openvswitch
Handle 'direct' vnic type with 'switchdev' (hw offload) capability
ports in NeutronPortChangeListner.
Change-Id: I0d2a5aa32b9f909d5cf56a1c220ca2cdcbc80071
Signed-off-by: Moshe Levi <moshele@mellanox.com>
Co-Authored-By: Edan David <edandavi@gmail.com>
Depends-On: I65644f32551124c61e30c02831ac4193d12471d2
Faseela K [Sat, 16 Dec 2017 16:50:46 +0000 (22:20 +0530)]
Clean-up older s-n-d implementation
Now that ELAN is using infrautils.diagstatus
let us clean-up the older s-n-d implementation
in ELAN
Change-Id: I2c10a5adb8eadb132e1804d38c935da640ec5cb9
Signed-off-by: Faseela K <k.faseela@gmail.com>
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Stephen Kitt [Thu, 21 Dec 2017 14:15:29 +0000 (15:15 +0100)]
GENIUS-109: make checkDmacSameDPN more robust
This sorts the actions before comparing the flows, avoiding issues
when upstream changes cause the action order to change in the data
structure.
Change-Id: Ia4b62e2af5bc432f076432422236adacb097885b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
xcheara [Tue, 2 Jan 2018 08:14:31 +0000 (13:44 +0530)]
ODL SNAT upgrade issue - flow configured before group
The default fib route's action is to send to the
external network group which must be configured on the
switch before the flow is configured. Added
Change-Id: I9c4d8e649ad5966fbe536831a1b516b31763163a
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Josh [Tue, 2 Jan 2018 03:47:46 +0000 (05:47 +0200)]
Upgrade+conntrack, default FIB flow missing
N.B.: Depends on https://git.opendaylight.org/gerrit/#/c/65894/
The default fib route can not be configured before
the VpnInstanceOpDataEntry's dpn<->vpn mapping exists.
I've added code that waits for that to show up.
Change-Id: I7a4e75b04b171b5a6c80c74e75a410312ef3dee6
Signed-off-by: Josh <jhershbe@redhat.com>
Josh [Thu, 28 Dec 2017 10:28:15 +0000 (12:28 +0200)]
Upgrade: Remote FIB entry missing for ext. gateway
Root cause of this is that the ARP to discover the
external gateway's mac is triggered at router/subnet
creation time and the code assumes the ovsdb models
are in place. However, in the case of upgrade they are
not yet present as the switches have not yet reconnected.
Fix: At the point where the upgrading flag is unset we
know that all switches have reconnected. Listen for that
transition and trigger the ARP for all routers.
Change-Id: I6cc6045937e2d9a936596e1a8bc36b7a0cbb70e3
Signed-off-by: Josh <jhershbe@redhat.com>
Josh [Tue, 19 Dec 2017 10:28:31 +0000 (12:28 +0200)]
Minor: fix two log statements
Change-Id: Ie5c9de5eddde36eb026481453dd88a757e919fe1
Signed-off-by: Josh <jhershbe@redhat.com>
Josh [Tue, 19 Dec 2017 10:21:12 +0000 (12:21 +0200)]
SNAT Ext. Router scheduling broken w/ upgrade
Depends on this gerrit from genius:
https://git.opendaylight.org/gerrit/#/c/65894/
SNAT requires that all the nodes be in the operational datastore
at the time the external router configurations are pushed to ODL.
At this point the external router is "scheduled", that is, a
switch is chosen to get the flows and track the sessions. In the
case of upgrade, there are in fact no nodes connected to ODL.
Solution:
Wait until all the nodes are connected and only then schedule
the external routers.
1) if the upgrading flag is set, don't schedule external routers
2) When the listener fires, schedule all the external routers
Change-Id: Ib349450905ef9c33fe3640753bb88e65c8ebf967
Signed-off-by: Josh <jhershbe@redhat.com>
Josh [Wed, 13 Dec 2017 10:31:10 +0000 (12:31 +0200)]
Upgrade: race cond. btwn elan-dpn-interfaces and arp-responder for external ifaces
Depends on this gerrit from genius:
https://git.opendaylight.org/gerrit/#/c/65894/
A comment from inside the code:
The following through the end of the function deals with
an upgrade scenario where the neutron configuration
is restored before the OVS switches reconnect. In such a
case, the elan-dpn-interfaces entries will be
missing from the operational data store. In order to
mitigate this we use DataTreeEventCallbackRegistrar
to wait for the exact operational md-sal object we need
to contain the external interface we need.
Change-Id: Id02564a13e7f85933867e21c51e36c32bc60cd4e
Signed-off-by: Josh <jhershbe@redhat.com>
Josh [Tue, 16 Jan 2018 15:59:35 +0000 (17:59 +0200)]
NETVIRT-1065 Handle new bridges
If a provider network's value is a bridge name we
patch it to br-int. If the bridge does not exist prior
to the discovery of the provider mapping ODL will assume
the provider net is a local port and add it to br-int.
However, a user can remove that port and then add the bridge
and now the code will detect that addition and create the
patch ports.
Change-Id: Ieae1a37ccf90caa51566864be14d840319429e61
Signed-off-by: Josh <jhershbe@redhat.com>
(cherry picked from commit
216a424f69d254cbafe86a06cd90a42ed7fb6d2e)
Noel de Prandières [Fri, 19 Jan 2018 14:04:50 +0000 (15:04 +0100)]
FIXUP neutronvpn.shell.ConfigureL3VpnCommand issue
ConfigureL3VpnCommand is not ready yet to use rpcRegistry
Change-Id: If39a18edca4e29853209dffacca4f76cfba320bc
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Moshe Levi [Wed, 1 Nov 2017 22:31:29 +0000 (00:31 +0200)]
Update openstack-guide security group section
This commit update the openstack-guide to reflect the
current implementation. It removes the old security-group-mode
and adds openstack commands on how to spwan vm without security
groups.
Change-Id: I93b0e20453c0b4910120a3a4b7aceb0b0d4937c1
Signed-off-by: Moshe Levi <moshele@mellanox.com>
Jaime Caamaño Ruiz [Thu, 18 Jan 2018 17:55:34 +0000 (18:55 +0100)]
Support symmetric chain classification
Symmetric chain classification through networking-sfc requires a flow
classifier that defines both a source and a destination logical port.
This patch adds support for such scenario.
RedirectToSfc can now specify a SFP name. Classification for the
forward RSP of that SFP will be done on the source port. Classification
for the reverse RSP of that SFP will be done on the destination port.
Sfc translator has been modified to specify a SFP name in
RedirectToSfc instead of a RSP name.
When RedirectToSfc specifies a RSP name, if the RSP is not reverse,
classification will be done in the source port. Classification will
happen in the destination port otherwise. No classification will be
done if no source port is specified for a reverse RSP, and viceversa.
If a network is specified instead, all the ports of the network will be
considered source ports if the RSP is not reverse, and destination
ports otherwise, meaning that classification will happen on those ports
wether the RSP is forward or reverse. This maintains legacy behavior
with respect network classification.
Change-Id: I86d85a58688367591784f4361aacd095e9cbccf0
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Jaime Caamaño Ruiz [Mon, 15 Jan 2018 15:19:49 +0000 (16:19 +0100)]
Support directional-dpls & cleanup sfc-translator
This patch adds support for the new directional-dpl sfc spec. It allows
to setup both the egress and ingress ports of a port pair as distinct
service funcion data plane locators so that they are used appropriately
depending on the direction of a symmetric path.
This patch also fully transitions to the logical sff configuration model
cleaning up code that applied to the old configuration model. The
logical sff configuration model makes thing much simpler: neutron port
uuids are configured directly in sfc as logical interface data plane
locators without the need to fetch aditional data about them.
Change-Id: Ibee6e2707878932ae1efbfd4ab0eae8a78056437
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Faseela K [Fri, 12 Jan 2018 07:10:14 +0000 (12:40 +0530)]
Add restconf to coe feature
As we are starting with csit activities for coe, it will be useful
if restconf is also included inside the user-facing feature
Change-Id: I6d0bda34c83528026a1618cd57b5917d50ad9189
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Sam Hague [Tue, 19 Dec 2017 15:42:19 +0000 (10:42 -0500)]
add csit tools
Change-Id: I6d7df190633fef95ea3ed3b560933f96733409de
Signed-off-by: Sam Hague <shague@redhat.com>