Julien Courtat [Fri, 24 Feb 2017 10:46:31 +0000 (11:46 +0100)]
Yang changes for Inter DC connectivity using BGPVPNv6
This patchset includes the yang changes required to implement
Spec to support IPv6 Inter DC L3VPN connectivity using BGPVPN [1].
[1] https://git.opendaylight.org/gerrit/#/c/50359
Change-Id: I70fb5460bde2a852dc965525fb37ae8003bea109
Signed-off-by: Julien Courtat <julien.courtat@6wind.com>
Periyasamy Palanisamy [Thu, 9 Mar 2017 06:44:07 +0000 (12:14 +0530)]
Bug 7826: proper elan djc job retries
When ELAN BC programming fails due to TransactionCommitFailedException
then there is no way of retrying djc job due to syncInstall(Delete)Group. It is fixed now and this is very much needed in a scaled setup because of OptimisticLockFailedException on the same group by multiple elan interfaces
Change-Id: Iaa0524edec3f25e0433a91c802f9c8da0d642962
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
eupakir [Thu, 9 Mar 2017 19:07:27 +0000 (00:37 +0530)]
Yang changes to support iRT/eRT for EVPN
Change-Id: I5ecf6bdbc5cd6b5101484a14cde66cec960c7ab2
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Miguel Perez [Thu, 9 Mar 2017 14:07:02 +0000 (15:07 +0100)]
Bug 7966: Fix route origin for some vrfEntries after VM migration
+ It's been observed that when a VM migrates and there are static
routes pointing to it, as it happens with the CGNAT, the origin
of these static routes are changed from 'static' to 'local'
Change-Id: I87751263b66080d8200f682b49ac0dd14062bc61
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>
Julien Courtat [Tue, 10 Jan 2017 10:13:51 +0000 (11:13 +0100)]
Spec to support IPv6 Inter DC L3VPN connectivity using BGPVPN
This blueprint spec explains architecture for supporting IPv6 Inter Data
Center connectivity using BGPVPN in a SDN controlled by OpenDaylight.
Change-Id: I8cb4d7bfcff6306f2552d2a73bfddb4983cc49c5
Signed-off-by: Julien Courtat <julien.courtat@6wind.com>
Abhinav Gupta [Wed, 15 Mar 2017 07:20:42 +0000 (12:50 +0530)]
Minor yang changes in neutronvpn.yang
1. rename to isBgpVpn > for easier code reading and clarity
2. change nw types to uppercase > to make use of existing utilities to
fetch network types and avoid changing cases.
Change-Id: I37dc47b0d1172fca34f8ae0d28973c4bae8f54d0
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Shai Haim [Wed, 22 Feb 2017 09:13:45 +0000 (11:13 +0200)]
Spec for DHCP Server dynamic allocation pool extension
Change-Id: I7f9d45b6dabbbb0e56e8db7f41c24f2cbaba537e
Signed-off-by: Shai Haim <shai.haim@hpe.com>
karthik [Sat, 18 Feb 2017 01:41:18 +0000 (07:11 +0530)]
Bug 7817 & Bug 7838: DHCP ARP flow is not added and irrelevant ARP flows
are installed in compute node.
* DHCP ARP flow is not added in Compute node
When VM's are created with multiple networks.
* VM's are created across networks, irrelevant ARP flows are
installed in compute node
* Fix: Compared with segmenationID and write the flows for table 20.
* This patch contains fix for both the Bug 7817 & Bug 7838.
Change-Id: Iab21d2fe391451dc1f6e9cc70db099105e990871
Signed-off-by: karthik <karthikeyen.p@hcl.com>
Tomer Pearl [Mon, 13 Mar 2017 14:44:44 +0000 (16:44 +0200)]
Enhancing adjacencies with PNF flag
This will be used to mark PNF adjacencies
This is part of "discovery of PNFs in Flat/Vlan
provider networks" spec.
Change-Id: I60a322691af6b662c9419421ede9f69814924623
Signed-off-by: Tomer Pearl <tomer.pearl@hpe.com>
Tomer [Wed, 11 Jan 2017 13:24:15 +0000 (15:24 +0200)]
Spec: Discovery of directly connected PNFs in Flat/VLAN provider networks
This features enables discovering and directing traffic to
Physical Network Functions (PNFs)
in Flat/VLAN provider and tenant networks, by leveraging Subnet-Route feature.
Updated the spec, hopefully with better explanations.
Added a note regarding preservation of src MAC of VMs in two tenant networks
use case, to prevent MAC movments in the underlay switches.
Change-Id: I556d7d0f28c2f2519cf4a4ba4a3250e821034e19
Signed-off-by: Tomer <tomer.pearl@hpe.com>
Tali [Tue, 28 Feb 2017 17:54:05 +0000 (19:54 +0200)]
Yang changes for policy service
Change-Id: Ia7572218c7f7375dc3ac2a0111793f36f9132013
Signed-off-by: Tali <tali.ben-meir@hpe.com>
Gobinath [Fri, 17 Feb 2017 10:19:34 +0000 (15:49 +0530)]
Changes for achieving ECMP in VMs present in different DPNs
* Split the traffic b/w extra routes behind the VMs present in the
* same DC in the different DPNs
The local fib entries of the extra route would now point to
the load balancing select group which would contain in turn the
buckets pointing to the local next hop groups(in case of
colocated vms and tunnels in case of VMs present behind
different DPNS).
The extra route handling for the imported routes.
Change-Id: I48ba107bd2b3f57884b6dae258221106e71ad251
Signed-off-by: Gobinath <gobinath@ericsson.com>
Signed-off-by: gobinath <gobinath@ericsson.com>
gobinath [Wed, 1 Mar 2017 05:30:55 +0000 (11:00 +0530)]
Fib changes for the UC1 & UC3 of ecmp
This patch contains the fib changes for the \
* UC1 - ECMP forwarding of traffic entering a DC and
* UC3 - ECMP forwarding of intra-DC traffic
* Split the traffic b/w extra routes behind the VMs present in the
same DC in the same DPN
The local fib entries of the extra route would now point to the load
balancing select group which would contain in turn the buckets
pointing to the local next hop groups.
Depends-On: https://git.opendaylight.org/gerrit/#/c/51999/
Change-Id: I85bd5263327b20cc030e34ca97c8336405935d36
Signed-off-by: Gobinath <gobinath@ericsson.com>
Signed-off-by: gobinath <gobinath@ericsson.com>
Sridhar Gaddam [Fri, 10 Mar 2017 12:36:26 +0000 (18:06 +0530)]
Bug 7952: ACLService to treat Ethertype=IPv6 and Protocol=icmp as a request for ICMPv6
In Neutron, "Protocol=icmp" is allowed to be configured with "Ethertype=IPv6"
for backward compatibility reasons. When such a rule is configured, Neutron
IPTables Firewall Driver (i.e., reference implementation) treats this as a
request for ICMPv6 and configures a rule accordingly. This patch implements
the same thing in ACLService. Please refer to the bug report for more details.
Change-Id: Id87c0eda599da4bc3669327bd36659b83d9e108b
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Aswin Suryanarayanan [Thu, 2 Mar 2017 13:19:21 +0000 (18:49 +0530)]
Updated SNAT Spec
The pipeline is updated,
1) The extra table 43 is removed and 44 will continue as inbound table.
2) Now metadata will be set before submitting the packet to the
netfilter using set_field.
Change-Id: I7fedf702bfb37cd8a7bbb9eab545225ec393a004
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Vishal Thapar [Thu, 16 Feb 2017 16:17:57 +0000 (21:47 +0530)]
AbstractDataChangeListener clean-up
Reference:
https://wiki.opendaylight.org/view/Weather#Genius_AbstractDataChangeListener_clean-up
Change-Id: If92640071cbbfcf65df6a7695a56e726c6eab682
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Riyazahmed D Talikoti [Wed, 15 Feb 2017 11:21:04 +0000 (16:51 +0530)]
EVPN RT2 Yang changes
1. Few review comments fixed
2. EVPN RT2 overall yang changes
Change-Id: Ie9fa2213d348f5b15b4ae52e4e65b79ebda89e4a
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>
Guy Regev [Mon, 16 Jan 2017 13:53:16 +0000 (15:53 +0200)]
Added statistics bundle
Change-Id: I2d10a9871c04373e944a07c1ff96bfd9a4f272fa
Signed-off-by: Guy Regev <guy.regev@hpe.com>
Alon Kochba [Thu, 23 Feb 2017 14:31:39 +0000 (16:31 +0200)]
Add Remote ACL Indirection table
Adds an indirection table into the ACL services,
as described in the spec:
https://git.opendaylight.org/gerrit/#/c/51462/
Depends-On: https://git.opendaylight.org/gerrit/#/c/52213/
Change-Id: I023dfda4bdc8da4554141d1fb3431872869ed5ab
Signed-off-by: Alon Kochba <alonko@hpe.com>
Sridhar Gaddam [Thu, 2 Mar 2017 18:04:57 +0000 (23:34 +0530)]
Update IPv6 N-S spec with neighbor-discovery.yang changes
Change-Id: I04bb19cffc3cef53ed6b7fbb29f96050bb90ad5a
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Viji J [Thu, 9 Mar 2017 09:36:41 +0000 (15:06 +0530)]
Rectified incorrect help usage displayed for BGP add-neighbor cli command
odl:configure-bgp -op add-neighbor displays incorrect help.
Change-Id: Ic1e7bd747a07de0fdc562eb59d8bc835558900ac
Signed-off-by: Viji J <viji.j@ericsson.com>
Koby Aizer [Thu, 2 Mar 2017 10:24:55 +0000 (12:24 +0200)]
Bug 7888: handle update of floating ip port
In Ocata, when a floating ip port is created, its deviceId may be
set to "PENDING", and an update will follow with the deviceId later on ([1])
This is causing both an uncaught exception in NeutronPortChangeListener.add()
which assumed deviceId is always a Uuid, and also there is no creation of
FloatingIpIdToPortMapping during update().
[1] https://review.openstack.org/#/c/396013/
Change-Id: Ide44ef016f0a8c576f02dfd28d9a4ee9d492841e
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>
cgowdru [Tue, 28 Feb 2017 09:13:36 +0000 (14:43 +0530)]
Bug 7809 - NAT snatGroupIdPool is overlapping with Elan Groups
Description : Currently the SNAT Group is defined as 200000-225000
and the elan group range also falls within this range(210000-220000).
Change done to move the snatGroupIdPool range to 225000-250000.
Change-Id: I680305c512221ca775fca8461a901f72f9e13e27
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Robert Varga [Wed, 8 Mar 2017 11:08:11 +0000 (12:08 +0100)]
BUG-7905: remove duplicate RPC declarations
Duplicate rpc identifiers are a violation of RFC6020 and only happen
to work due to BUG-7424 not being addressed. Fix this up by removing
duplicates.
Change-Id: I916a1607f3b315f9d9b8c8ed826aff0c1e679c6d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
eupakir [Wed, 9 Nov 2016 09:08:01 +0000 (14:38 +0530)]
Ethernet VPN Flow Programming for VM's within the same data center.
This review brings in enhancement to VRFEntryListener to manage flows to
realize L3 Forwarding over VXLAN tunnels. These code-changes cover the
following scenarios:
1) Intra-subnet connectivity between VM's on the same hypervisor within a
data center.
2) Intra-subnet connectivity between VM's on different hypervisors within
a data center.
3) Inter-subnet connectivity between VM's on same hypervisor within a data
center.
4) Inter-subnet connectivity between VM's on different hypervisors within
a data center.
Change-Id: Ieb787442d20cd6351ef0e04a5765afc5aa975721
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Vivekanandan Narasimhan [Wed, 8 Mar 2017 09:06:16 +0000 (14:36 +0530)]
Bug 7931: SubnetRoue re-election to be triggered on disconnected nodes
Subnet Route reelection algorithm is triggered today only when
the anchor ports on the subnet go to DOWN state (or) if such
anchor ports on the subnet are being removed.
However, there are situations where a switch hosting the anchor
port can be disconnected on the openflow control-path and can
continue to remain so indefinitely.
Under such circumstance, new non-neutron IPs become undiscoverable
as the switch hosting the anchor port cannot punt discoverable IP
packets to the controller.
So this fix attempts to always trigger reelection of nodes for
subnetRoute hosting as soon as the current anchor node(or switch)
is detected to be disconnected from the controller.
Change-Id: Idd8be8e2f1940c4840d7d52726ac5187ed33d28f
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
cgowdru [Wed, 22 Feb 2017 12:17:34 +0000 (17:47 +0530)]
Changes for supporting "opendaylight-vni-ranges" pool
Description:
1) Re-defined VPN_POOL range to 100000-130000
2) Adding Util API's for creating and deleting VNI Pool.
3) NeutronVpn APIs exposed to retreive 'opendaylight-vni-ranges' and
'enforce-openstack-semantics'
Change-Id: I61d518aa694416aca7b47b266c0f2c56edf350cf
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Periyasamy Palanisamy [Wed, 18 Jan 2017 07:31:34 +0000 (13:01 +0530)]
fix for mac movement issue
* updating only elan-interface-forwarding-entries when mac is learned
from vlan provider port which is already learned on VM port.
* updating MACs in elan ds and dpn when MAC is actually moved from VM port
to another VM port.
* handling MAC learning from VLAN provider port if it is a newly learned
mac
Change-Id: I4ff9d79463f0f238c5a8b345438514b93ed4bb17
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Tali [Tue, 7 Mar 2017 10:34:50 +0000 (12:34 +0200)]
Export ACL flow programming logic
Change-Id: Ia7ef604bb8a763eeb520c8459e23b9979d126160
Signed-off-by: Tali <tali.ben-meir@hpe.com>
karthikeyan [Wed, 22 Feb 2017 10:33:45 +0000 (16:03 +0530)]
Ethernet VPN (EVPN_RT5) Enhancement to Non-NAPT to NAPT
This review (1st Review) is part of EVPN_RT5 support in NAT feature.
Changes are made in the NAT code for accomplishing the below
Intra DC Communication:
=========================
For Non-NAPT to NAPT Traffic, Tunnel id will be setting with
Router lPortTag which will be carved out per router if it is part of
L3VpnOverVxlan else tunnel-id will be setting with existing router-id.
Change-Id: Idc5e8b1956311541ae7bae255d023c5f3543f9a3
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Poovizhi P [Tue, 28 Feb 2017 13:47:03 +0000 (19:17 +0530)]
Netvirt: Neutron Qos DSCP Marking support
Adds Neutron QoS DSCP support to Neutronvpn.
A new table 90 created for QOS DSCP in Genius.
Supports:
- add, update, remove of qos dscp marking rule assigned to Neutron
port/network
Description:
- When a DSCP rule is created in the Qos Policy which is bound to the
port, OF rule(table 90) to set tos filed in the IP with the DSCP vaule
is configured. Qos service is bound to the table 17.
- During the removal of policy from port, unbinding qos service, and
removal of flow is donw
Testing procedure:
1. Create neutron network,subnet and instance.
2. Create neutron qos policy and DSCP rules.
3. Associate policy to the port/network
4. Update the DSCP rule.
5. Disassociate the policy from port/netwok
Result:
- corresponding ovsdb port tos attributes are configured
appropriately.
- traffic flows with the DSCP value configured in the port.
Change-Id: I06d0d89c7144dcc27f01320c8ed63bfe6afebf27
Signed-off-by: Poovizhi P <poovizhi.p@ericsson.com>
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>
David Goldberg [Wed, 1 Mar 2017 15:36:33 +0000 (17:36 +0200)]
Yang changes for DHCP Server dynamic allocation
This change includes the yang changes needed to implement spec [1]
[1] https://git.opendaylight.org/gerrit/#/c/52156/
Change-Id: Ia331db0d5f2bf4e7fbe318debab4aa4a83ee1d56
Signed-off-by: David Goldberg <gdavid@hpe.com>
ehvkand [Wed, 1 Mar 2017 07:29:14 +0000 (12:59 +0530)]
Bug 7876 : After router association to L3vpn, one of the VM ip is not
removed from router interface to BGPVPN
When a VPN associated to router is deleted and a same VPN is associated
back to same router , there is race condition , where interface is not
found in VpnInterfaceOp and remove event is being ignored. Hence interface
remains in router VPN.
It is also seen that , one of interface from same subnet is swapped and
others are not. Its due to race condition that exist in NeutronVpnManager
where create and update can race with each other. Currently create uses
DJC , where as update doesn't .
Change-Id: I354c3ce93f965580c735f1153a043e2514d68af7
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>
Aswin Suryanarayanan [Thu, 2 Mar 2017 12:05:09 +0000 (17:35 +0530)]
Model for nat service config parameter.
The "nat-mode" will be used to switch between controller based and
conntrack based nat.
Change-Id: Ia616c2b1567a0d9e751fe9853274bc3c2a9e8365
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Jose-Santos Pulido Garcia [Thu, 2 Mar 2017 11:32:41 +0000 (12:32 +0100)]
Add tcp-md5-signature-option to ebgp's neighbors
An optional leaf has been added to ``neighbors`` list in ebgp module in
order to configure QBGP with the shared secret required by TCP MD5
Signature Option [RFC2385].
See spec qbgp-tcp-md5-signature-option [0]
[0] https://git.opendaylight.org/gerrit/#/c/52218 Ibdb0675c0045754e575b73deba2dfebde4b56bc5
Change-Id: I8f712840bd5b0b6d1647b972375c5e3080f1eadc
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>
Aswin Suryanarayanan [Mon, 6 Mar 2017 11:43:44 +0000 (17:13 +0530)]
Bug 7885 - CSIT Sporadic failures -
tempest.scenario.test_port_security_macspoofing_port
On port-security update bind/ubind service is now called.
Change-Id: I46cfae8b13a3f67647bfc501b3e55faf8c827a37
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Koby Aizer [Wed, 1 Mar 2017 10:02:16 +0000 (12:02 +0200)]
Bug 7878: provider interface MACs are installed on remote DPNs
MACs learnt on provider interfaces must not be installed on remote
DPNs.
This change includes 2 fixes:
1. handleInternalTunnelState event initiated installation on remote
DMAC flows for all elans in the remote DPN, without verifying that
this elan is not Flat/Vlan.
2. Protection on setupRemoteDmacFlow to make sure no other code flows
will install remote DMAC flows on MACs learnt on provider interfaces.
Change-Id: I5ae288ed525dc1ae2f5be1bec5215f2e8f01f8c5
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>
Ravindra Thakur [Thu, 2 Mar 2017 08:25:01 +0000 (13:55 +0530)]
Netvirt: Yang changes for vlan provider enhancement feature
Modified the elan:elan-instances container with attribute to indicate
whether the network has an external router attached to it.
Spec review link for the enhancement:
https://git.opendaylight.org/gerrit/#/c/52201/
Change-Id: I0f6fe5779a052804dc9549f97d958038fdb383bb
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>
Marcos López Samaniego [Tue, 28 Feb 2017 16:23:51 +0000 (17:23 +0100)]
Bug 7881 - Traffic drops when not matching UL SC starting
in a VPNPseudoPort
When programming the L3VPN to SCF pipeline, a new flow is needed in
table 17
Change-Id: I31cf7817eb38b47e5c665c4f0362d0baa9554af4
Signed-off-by: Marcos López Samaniego <marcos.lopez.samaniego@ericsson.com>
Arun Sharma [Thu, 19 Jan 2017 20:28:17 +0000 (01:58 +0530)]
Spec for NetVirt QoS alert
This spec file describes the changes to be made
in the Carbon release to support QoS Alert.
Change-Id: I6306a9f0f7a28ad524b39fb2b84a01ace71b5f14
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>
Arun Sharma [Tue, 17 Jan 2017 09:13:13 +0000 (14:43 +0530)]
Netvirt:Yang changes for-Neutron Qos Alert support
***** Commiting only yang as first patch ****
Spec Review Link:
https://git.opendaylight.org/gerrit/50689
Adds support to monitor the drop counts and log alert message when
Qos rate limit rule if applied.
Supports:
- Configuration to start or stop monitoring the port statistics
- Configuration of rx drop count threshold value in %
- Configuration of location and name of log file
- Configuration of polling interval
Description:
- As a result of Qos rate limit rule, ingress packets may get dropped
by the switch.
- If monitoring is enabled, port statistics are queried after polling interval
and alert message is logged in a log file if packet drop % is more
than configured threshold value.
- Threshold value is specified in % of drop counts
- All configurations are applied at controller level.
- Default or initial configuration available in netvirt-qosalert-config.xml
* qos-alert-enabled - false
* qos-drop-packet-threshold - 5%
* qos-alert-log-file - qosalert/qos-alert.log
* qos-alert-poll-interval - 2 minutes
Spec Review Link:
https://git.opendaylight.org/gerrit/50689
Testing procedure:
1. Create neutron network,subnet and instance.
2. Create neutron qos policy.
3. Associate policy to the port/network
4. Send traffic more than the limit specified in policy.
5. Configure the threshold value and enable port statistics monitoring
6. Check for alert log messages in the file if packet drop % cross the
threshold value.
Result:
- Port statistics polling is started when qos-alert-enabled is set to true
- Alerts messages are logged when packet drop % cross the threshold value
Change-Id: I5b2675a2c036835d4ac2e5bd585c3da8e9b268e8
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>
Kency Kurian [Thu, 23 Feb 2017 14:45:57 +0000 (20:15 +0530)]
BGP Manager changes to support ECMP.
This review ensures that when 2 DC-GW advertises routes for the same
destination prefix, the vrfEntry DS are populated correctly with both the
route paths.
The stale fib entry flat map now includes nextHop also in its key, to
align with the new vrfEntry model.
Change-Id: I26526561a74228925030f0086109dbe182b876c9
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
Tali [Wed, 22 Feb 2017 16:31:19 +0000 (18:31 +0200)]
Yang changes for federation plugin
Change-Id: Ic852e5343d843b7c7536c2e450f40e5cb75542bb
Signed-off-by: Tali <tali.ben-meir@hpe.com>
Olga Schukin [Mon, 27 Feb 2017 14:19:22 +0000 (16:19 +0200)]
Bug 7787 - missing flows in T21
when VPN interface up before tunnel info is available in operational DS,
it causes missing flows in T21
Fix: always handle the add event inside TunnelInterfaceStateListener -
even for a tunnel which state is Down
Change-Id: I41c3efbeeb5740788f58ced57a698a73ef70b466
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>
cgowdru [Fri, 3 Mar 2017 01:57:25 +0000 (07:27 +0530)]
yang change to make remove deprecated leaf-elements
Description : It's been observed that in the review
https://git.opendaylight.org/gerrit/#/c/49093/, the whole
neutronvpn-config container been made as deprecated instead
of required leaf element.
1) Changes done to remove 'use-transport-zone' and 'use-of-tunnels'
config from neutronvpn-config(as no usage found in neutronvpn).
2) Removed the default values for these config defined in
netvirt-neutronvpn-config.xml
3) Defined default value for VNI Spec specific configs.
4) Re-defined the neutronvpnConfig and passing its reference to
neutronvpnManager in neutronvpn.xml(as it will be refered
accessing VNI Spec specific configs)
Change-Id: Ied09b7080814ae9a014e827c464e5d07b23de4c7
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Faseela K [Mon, 6 Mar 2017 06:20:26 +0000 (11:50 +0530)]
Use the right service name when binding service
Making ELAN and ACL bind services with the right service name
Change-Id: I022a0ff4aaff4772870413890ce8a9665af5a309
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Sridhar Gaddam [Thu, 2 Mar 2017 14:42:46 +0000 (20:12 +0530)]
Yang changes for IPv6 Neighbor Discovery Support
This patchset includes the yang changes required to implement
IPv6 North-South support for FLAT/VLAN provider network spec [1].
[1] https://git.opendaylight.org/gerrit/#/c/49909/
Change-Id: I34a4034b2482d91243b15a21cba83c353a3b516c
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Kency Kurian [Thu, 2 Mar 2017 07:35:36 +0000 (13:05 +0530)]
Yang changes to support tunnel updates in ECMP
New YANG model to update load balancing next hop group buckets according
to VxLAN/GRE tunnel status.
Subsequent review will make use of these containers.
Change-Id: If75300674fbfe1ae27b6c6a43302caa5917b9daf
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
epgoraj [Wed, 1 Mar 2017 06:54:03 +0000 (12:24 +0530)]
BUG:7896 OptimisticLockFailedException
Change-Id: I977feeb0226613b1a93cfb8092d4afa88926979e
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Abhinav Gupta [Thu, 2 Mar 2017 10:01:37 +0000 (15:31 +0530)]
Bug 7875: NPE due to subnetmap create DjC trailing update
Removed troubling DjC.
Also have made sure that such errors are caught at NeutronVPN
level itself by separating out subnetmap create and update.
Update will not create subnetmap node with partial info, instead will
return if subnetmap node is absent.
Change-Id: I252d0164f123052d7090968325e214289b29daab
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
eswanit [Fri, 10 Feb 2017 08:48:11 +0000 (14:18 +0530)]
correcting the spec for providing ECMP over BGP VPN
RST generated doc had some warnings. Warnings are fixed and some
foramtting issues are resolved.
Change-Id: I22cef70c9af3165c16f2cd960639da12297297f4
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>
Vivekanandan Narasimhan [Thu, 2 Mar 2017 13:33:40 +0000 (19:03 +0530)]
Bug 7714: VPN Operational Interfaces not getting removed at all.
This can be a potential problem for temporary CSIT failures
that happens on VPNService suite.
The VPNInterface is not getting removed at all permanently from
VpnInterface Op DS. When VM interface has secondary adj like extra route
or invisible ip , the order in which cleanupvpninterface worker DJC is
called not guaranteed.
The regression was caused by review:
https://git.opendaylight.org/gerrit/#/c/51224/
VPNInterfaces not deleted from Oper DS
Change-Id: I1b2a1d139030f707cec93eb5a5e6a690e010e336
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>
Kency Kurian [Fri, 3 Mar 2017 06:43:32 +0000 (12:13 +0530)]
Fixing checkstyle error in FibManager
Change-Id: I6ab2a88236661f1866f9873167f0cf3e9168f304
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
A Vamsikrishna [Thu, 15 Dec 2016 08:56:18 +0000 (14:26 +0530)]
Bug 7387 : Netvirt: qos policy applied on the network, not applied on
newly created ports of same network
Use case:
1.Network is created and updated with QoS policy.
2.Port is created on the same network.
3.Now this port is intended to have the QoS policy as the network has.
RCA:
The scenario of application of network qos policy to the newly created
port was missing
Code fix:
1.Created a new class called QosInterfaceStateChangeListener.java which
listens on the interface state oper events
2.Handled the scenario of application of network qos policy to the newly
created port in the add( ) of QosInterfaceStateChangeListener.java
Application of network qos policy to the newly created port condition was
missing. Fixed it by adding the same in NeutronPortChangeListener.java
Change-Id: I3211a698e9a27cfefe090fbbf2ab120f8626de76
Signed-off-by: A Vamsikrishna <a.vamsikrishna@ericsson.com>
Alon Kochba [Tue, 28 Feb 2017 15:52:43 +0000 (17:52 +0200)]
Bug 7856: Reverse SNAT flows order to minimize race possibility
Change-Id: I145905151348d2fa2670756f24e17c16d6239885
Signed-off-by: Alon Kochba <alonko@hpe.com>
Konsta Pozdeev [Thu, 16 Feb 2017 11:46:11 +0000 (13:46 +0200)]
Bug 7418 Run local group creation as async task with key equal to subsequent tasks.
Change-Id: I515b3b874b50f8e53a931e1a3ee3a2e30f877f8e
Signed-off-by: Konsta Pozdeev <konsta.pozdeev@hpe.com>
Alon Kochba [Mon, 6 Feb 2017 12:00:09 +0000 (14:00 +0200)]
Spec for Remote ACLs Indirection Table
Change-Id: I8ea8fbbc2ce67f637dba40fe52b0361f67e4f6ab
Signed-off-by: Alon Kochba <alonko@hpe.com>
Guy Regev [Tue, 10 Jan 2017 10:02:31 +0000 (12:02 +0200)]
Added netvirt countrs
Change-Id: I809989d5dab91b54489fa16e244ce9cd888c7141
Signed-off-by: Guy Regev <guy.regev@hpe.com>
Tali [Wed, 1 Mar 2017 16:58:29 +0000 (18:58 +0200)]
Fix NPE from VpnDpnToTransportZoneListener
Change-Id: If4badb0de9733694d9397220e239a1fa9a4cd918
Signed-off-by: Tali <tali.ben-meir@hpe.com>
karthikeyen.p [Wed, 8 Feb 2017 04:30:16 +0000 (10:00 +0530)]
Bug 7445: Improve the performance on bulk create.
* Reduced read calls when SG associated with the VMs.
* We have used cache mechanism to store neutron port information
that was read from DS.
Change-Id: I3b243ddf3656e48392593c6fc1086713d1fd1af3
Signed-off-by: karthikeyen.p <karthikeyen.p@hcl.com>
Signed-off-by: Sam Hague <shague@redhat.com>
eceghkl [Mon, 27 Feb 2017 11:07:37 +0000 (16:37 +0530)]
Fix for GwMac flow deletion during interface delete
* Without the fix, the mac address used for deleting the flow
was obtained after quering interface state again. This
returned null resulting in gwmac table not getting deleted.
* With the fix, interface state is not queried again. Interface
state passed during interface state deletion notification is
used to obtain mac address.
Change-Id: I1c091dc8edb722ab58fdf05f6cba1729a583209e
Signed-off-by: eceghkl <manu.b@ericsson.com>
Abhinav Gupta [Tue, 28 Feb 2017 07:11:12 +0000 (12:41 +0530)]
vni-based-l2-l3-nat spec updated with openstack-semantics config
A few formatting changes done alongwith introducing the config.
Change-Id: I4f39a5a7371aa0e6a15c0f176252e8ff9dc83b5a
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Abhinav Gupta [Mon, 27 Feb 2017 11:54:17 +0000 (17:24 +0530)]
Yang changes for vni-l2-l3-nat
Changes are as per approved in the spec at:
https://git.opendaylight.org/gerrit/#/c/52241
Change-Id: I937202aa8388c5f23f8a85e84463c5d52f359535
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Yakir Dorani [Mon, 5 Dec 2016 15:32:28 +0000 (17:32 +0200)]
Bug 7281: Support auto-TZ for non-neutron northbounds.
Support non-neutron networks for creation of TZ.
Listen to Elan and Vpn changes instead of NeutronPorts.
This will be used for unimgr.
Change-Id: I24ae67888352b9a5f941f74abf4db019e3cf439e
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>
eupakir [Mon, 27 Feb 2017 10:44:11 +0000 (16:14 +0530)]
Bug 7831 : BgpRouter receives unnecessary events
Suppressed unwanted advertisement of routes, that were being advertised due to a
breakage in the VPN Engine.
When BGP Client is not configured, route/vrf advertisement used to fail
and throw lot of exception. If BGP Client is not connected, the
process of dispatching routes/vrf to DC-GW via BGP Client should not happen
at all.
Change-Id: I25b6abe7a67efb4621745933f869f7034eb9cd51
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
yair [Thu, 16 Feb 2017 11:05:25 +0000 (13:05 +0200)]
External Routers yang changes - external ips will contain also subnetId
This is part of the hairpinning tasks.
This include only changes in the yang model for external routers.
Changes the external-ips to be a list of ip and subnet.
Change-Id: I368d46b62d1b37aebcae5a99e02df15940e8a5e1
Signed-off-by: yair <yair.zinger@hpe.com>
eamakol [Mon, 27 Feb 2017 11:27:24 +0000 (16:57 +0530)]
corrected the population of BGP Total Prefixes counter
Change-Id: I679b9a4b50ce2204188d0bbe7a5d4c48aafd3aee
Signed-off-by: eamakol <amaranadh.kolla@ericsson.com>
Michael Vorburger [Sat, 14 Jan 2017 03:16:02 +0000 (04:16 +0100)]
AclServiceTestBase use assertFlowsInAnyOrder from genius
Change-Id: I62850b47bd47ba569c385ee138f129d053e57ff7
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Vishal Thapar [Mon, 27 Feb 2017 10:21:14 +0000 (15:51 +0530)]
Disable split horizon for Trunk subports
Pkts going from one subport to another on same VM will have the same
in/out ports. Default OVS splithorizon will not allow this behavior and
needs to be disabled for all subports on a trunk.
Change-Id: Id459fc0231e39f6a78b508daac3065fbb1b4cf28
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
karthikeyan [Mon, 27 Feb 2017 16:07:03 +0000 (21:37 +0530)]
Bug 7861: No ping response from FIP on 1st router when adding 2nd FIP
Problem Description:
====================
Table Miss Entry for SNAT/DNAT in FIB Table (table=21->26 flow) is
overriting when multiple router is present instead of creating new entry
for each VPN Instance.
Each router is associated with unique external network and each external
network is associated with unique internet BGP VPN Instance.
Example topology:
router-1/external-network-1/bgp-vpn-1
router-2/external-network-2/bgp-vpn-2
Both router-1 and router-2 are presented on single compute node. In this
scenario router-2 ping traffic only is working as expected. Since table 21
doesn't have the table miss entry for router-1 with bgp-vpn-1 instance.
Solution:
=========
In ExternalNetworkListener while creating table miss entry for SNAT/DNAT
in table 21, added "vpn-id" attribute also in creating flow reference as
same as implemented in the class
SNATDefaultRouteProgrammer.buildDefNATFlowEntity(). After this code change
in table 21 miss entry for SNAT/DNAT is created for each bgp-vpn instance.
Change-Id: Iac909e628f1a4134526e6df0b640149c8d45cd20
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>
Abhinav Gupta [Fri, 24 Feb 2017 10:19:02 +0000 (15:49 +0530)]
vni-l2-l3-nat spec update with yang models to supply VNI to VPNEngine
Change-Id: I90c4ffe16a9523dca7ec9322cebd4f2a552235b0
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Poovizhi P [Wed, 15 Feb 2017 10:23:54 +0000 (15:53 +0530)]
Qos implemented as separate service. All qos related codes from neutronvpn
are segregated and a separate qosservice module is formed.
Change-Id: Ie9fc4755f5270eeb3f654bf5d0676ae302c9f3e4
Signed-off-by: Poovizhi P <poovizhi.p@ericsson.com>
eupakir [Sat, 25 Feb 2017 14:08:38 +0000 (19:38 +0530)]
BUG 7727 : Local and Connected routes do not get imported
Route Origin for Subnet Routes and Local VM routes were changed from
STATIC to CONNECTED and LOCAL, respectively. Hence, the import/export RT
functionality for VPNs was broken.
Change-Id: I04d8326fcb0bddf77934a63c1eb459f23564323b
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
epgoraj [Mon, 27 Feb 2017 07:35:15 +0000 (13:05 +0530)]
BUG:7824 ModifiedNodeDoesNotExistException
Change-Id: I32b64af56b5b7c5ed8d2af9ed3ca5b47b68a153f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Aswin Suryanarayanan [Thu, 16 Feb 2017 14:40:51 +0000 (20:10 +0530)]
Updated the conntrack based SNAT spec
1)Added a new HA logic when conntrack mode is selected.
2)Added info about the config parameter to switch NAT mode
3)Added info about a new CLI for displaying the NAPT switch selected for
each router.
Change-Id: Idd8388276adf8e4bb7974aaa869d04b3a91e09de
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
yair [Tue, 21 Feb 2017 13:24:42 +0000 (15:24 +0200)]
Added to VrfEntry a new attribute of parentVpnRdName
This is part of the Hairpinning tasks also use in l3vpnOverVxlnEvpn tasks.
For floating IPs the route distinguisher will be subnetId
and the DPN information only exists for external network VPNs.
So, if will have information in the parentVpnName it will use it
to get the DPNs.
Change-Id: Ibf92c5a95e427c137c961a70c96acc1b6400b593
Signed-off-by: yair <yair.zinger@hpe.com>
Alon Kochba [Mon, 20 Feb 2017 13:21:20 +0000 (15:21 +0200)]
Cleanup errors for networks of unsupported type
Reword and lower debug level of network configurations
for networks without a provider extension or with an
unsupported type.
Change-Id: I3b58a79b2fb72fe2c56328f79e0962e366a299ff
Signed-off-by: Alon Kochba <alonko@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Periyasamy Palanisamy [Thu, 23 Feb 2017 06:53:47 +0000 (12:23 +0530)]
Bug 7824 - ModifiedNodeDoesNotExistException
* added potential fix to guard against above exception
while handling elan interface remove/interface state remove
handling
Change-Id: Iad5145eafd9398718171880825f1684189405398
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Somashekar Byrappa [Thu, 2 Feb 2017 14:21:30 +0000 (19:51 +0530)]
Bug 7725: AAP with prefix 0.0.0.0/0 not supported in ACL
+ Added a check. In case of 0.0.0.0/0 prefix, program flow with just "ip"
match skipping the ip-prefix (0.0.0.0/0).
+ This method is called to configure flows both in table 40 and 251.
+ Handled below issue:
- Create VM1 & VM2 with AAP 0.0.0.0/0 and default SG.
- Create VM3 with default SG without AAP.
- Delete VM2. Observe that +new+trk flow configured for VM2's 0.0.0.0/0 AAP
prefix gets deleted (related to remote SG) in 252 table even though VM1 still
exists.
Flow: cookie=0x6900000, duration=4737.343s, table=252, n_packets=0,
n_bytes=0,
priority=61010,ct_state=+new+trk,ip,metadata=0x10000000000/0x1fffff0000000000
actions=ct(commit,zone=5000),resubmit(,220)
+ Earlier flow priority was generated/used based on SG rule UUID. The
above case was not handled with this approach. So, with this fix now
every ct_state=+new+trk flow will have a unique flow priority within table.
Flow priorities are generated based on flow ID.
+ For every switch, two ID pools (one each for ingress and egress) are
maintained which increases the availability of flow priorities.
Change-Id: I646312e1324267913cbb2865a1a8164b23b40372
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>
Signed-off-by: Sam Hague <shague@redhat.com>
ehvkand [Fri, 17 Feb 2017 09:43:22 +0000 (15:13 +0530)]
Bug 7815: Using DJC for VpnManager Interface-state Listeners
With mdsalutil is having a THREAD_POOL_SIZE=1, all the add(), update(),
and remove() methods handling the application logic were on the same
thread. This in turn was delaying all the interface-state listeners.
Changes are done to make use of DataJobCordinator so that all the
VpnManager application logic get executed by a seperate thread so
that the main thread is freed-up for other listener execution.
Change-Id: I926e0742fdc7955450c9423ce5026f9177ce21bb
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>
Vyshakh Krishnan CH [Sat, 25 Feb 2017 07:31:22 +0000 (13:01 +0530)]
Bug 7852 : FIB not updated with DC-GW routes
Problem :
FIB not updated with DC-GW routes and DC-GW not updated with VM routes.
Analysis and Fix:
Recent commit has broken the way RT is handled in NeutronVPN.
Fixing the same.
Change-Id: I3678b491bdc743b24e3c4abcb9744accc640031b
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>
Shashidhar Raja [Thu, 23 Feb 2017 10:48:01 +0000 (16:18 +0530)]
Bug 7842: ACL: Arp flows missing in ACL tables for overlapping MAC address
Updated to use lport tag with flowname for Egress ACL ARP flows
Change-Id: I9e733b70cb968c16f57253a975b9aa6d74c1d085
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Shashidhar Raja [Wed, 22 Feb 2017 13:29:52 +0000 (18:59 +0530)]
Bug 7839: ACL: ACL flows are not deleted from source host during VM
migration
- In the current code, ACL flows are being added in Interface state
listener and deleted from Interface config listener. As during
migration, only interface state listener gets triggered, Config
Interface and Interface state listeners are re-formatted slightly as
explained below to fix this issue and also to support delete VM
without any problems.
Details of changes are as below:
(a) bind/unbind moved to Config interface listener
(b) All other ACL flows programming (add/delete) now being done from
Interface state listener
- Also, bind/unbind service code is updated to use
DataStoreJobCoordinator to avoid ConflictingModificationExceptions
Change-Id: I4cf0c0d58eecbceafbed740f0201e128c196f555
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Periyasamy Palanisamy [Mon, 13 Feb 2017 10:39:38 +0000 (16:09 +0530)]
releasing dcn thread once tunnel interface state dcn delivered
* This change is about releasing dataTreeChangeHandlerExecutor's thread
immediately after DCN is delivered to service module. This allows to
improve DataTreeChangeListener notification time when there are multiple
listerners listening to same data tree path.
* For ELAN, DJC is used to run tunnel interface state DCN related tasks in async
way using ElanUtils#getElanInterfaceJobKey
Change-Id: If3128e623ac3eae81fbec17320bf54d425b7f6f5
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
yair [Wed, 1 Feb 2017 15:25:27 +0000 (17:25 +0200)]
Creating default FIB router flows for external subnets.
This is part of the Hairpinning tasks.
It will install on the FIB table (21) default flow for every external subnet.
The flow match is based on the VPN ID for the external subnet.
Change-Id: I870696a3cbe186bb43a6997b6c8c6a577b2c008d
Signed-off-by: yair <yair.zinger@hpe.com>
Tali [Thu, 23 Feb 2017 13:35:25 +0000 (15:35 +0200)]
Bug 7843 - Missing buckets in ELAN BC group installation during OVS restart
L2 pipeline can contain malformed flows/groups if OVS restart occurs in parallel
due to oper DS temp delete during restart
Possible solution would be to check for config DS inventory node instead of oper DS
so flows would be restored with correct content after the OVS is up again
Change-Id: I5aee6c2af85bca1beb98dab52bfa9181975192b1
Signed-off-by: Tali <tali.ben-meir@hpe.com>
Kency Kurian [Wed, 8 Feb 2017 06:03:02 +0000 (11:33 +0530)]
Support for multiple route paths
- Modified odl-fib.yang to support multiple route-paths for the same
destination prefix.
- Added a FibHelper which provides util methods to construct VrfEntry.
Change-Id: I7bcd3b77ac452476b4190286c1f3e8815a1b8819
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
Hideyuki Tai [Thu, 26 Jan 2017 01:56:17 +0000 (01:56 +0000)]
Bug 7773: Objects should be compared with "equals()".
Additional changes:
* Removed unnecessary repetitive calls.
Change-Id: I607ce073ee9ec4289c80dccd96a4b5222e4d9772
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>
Signed-off-by: arthi.b <arthi.b@hcl.com>
Vishal Thapar [Fri, 25 Nov 2016 16:36:22 +0000 (22:06 +0530)]
Use OFtunnels when configuring automatic tunnels
Refer: https://git.opendaylight.org/gerrit/48694
Change-Id: I12b38946d779e267729ae6e6d1713529a86060cb
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Lorand Jakab [Wed, 22 Feb 2017 10:23:23 +0000 (12:23 +0200)]
Checkstyle: fix new ParenPad violations
Enforcing no space(s) after a left parenthesis and before a right
parenthesis is a proposed change [0] which can only be merged when all
violations are fixed in projects enforcing checkstyle. This patch fixes
new violations since [1] was merged.
[0] https://git.opendaylight.org/gerrit/#/c/51316/
[1] https://git.opendaylight.org/gerrit/#/c/51704/
Change-Id: Id1bcbad79a5db7ffd8ce1eafae020f2895034c68
Signed-off-by: Lorand Jakab <lojakab@cisco.com>
Akash [Mon, 13 Feb 2017 15:39:44 +0000 (21:09 +0530)]
Bug 7786 Delete and re add of access port handling
After deleting access port in l2gw device and upon readding , vlan
bindings were missing .
Fix is to resync upon such case from config topology.
Change-Id: I993616f6ea90312af2ae5830e444c1918f8b48bb
Signed-off-by: Akash <a.k.sahu@ericsson.com>
Miguel Perez [Mon, 16 Jan 2017 17:30:16 +0000 (18:30 +0100)]
Bug 7772 - Service Chaining is not being applied to VMs in the L3VPN
+ Problem: Ping from outside DC-GW to a VM located inside DC-GW
works well in the Uplink, that is, packets going from outside
DC-GW towards the VM are hijacked by SCF Pipeline and delivered
to VM. However, when VM responds (Downlink), the packets are sent
to DC-GW right away via L3VPN pipeline. Service Chaining is not
applied for the return path.
+ Solution: try to divert to SCF Pipeline the packets that come from
the VM too, not only the packets coming from the external Tunnel
port.
+ How: by adding a listener on VpnInterface on CONFIG DS.
When a VpnInterface is added, the listener checks if its VPN is
involved in a ServiceChain or not. If so, the interface is bound
to SCF too.
+ This commit is requiered in stable/boron SR3. The Gerrit change URL:
https://git.opendaylight.org/gerrit/51681
Change-Id: Id1051bc8ec180589583a76524c0b2d456ba7b85a
Signed-off-by: Marcos López Samaniego <marcos.lopez.samaniego@ericsson.com>
Yakir Dorani [Mon, 16 Jan 2017 18:19:05 +0000 (20:19 +0200)]
Add ability to punt LLDP packets to controller
Change-Id: Ib6bdd0377e055def3ab7c18f45ef2bf0905b9eb3
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>
Miguel Perez [Fri, 30 Dec 2016 18:59:06 +0000 (19:59 +0100)]
Cleanup ivpnLink unused code + small modifications
+ This is the 1st change to cleanup/refactor InterVpnLink code
+ 3 new utility methods in InterVpnLinkDataComposite
+ add.getName() replaced by IvpnLinkName in InterVpnLinkListener
Change-Id: Icf7e29d9f77f53cc3d06620f60e8fab7efd464da
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>
eupakir [Tue, 21 Feb 2017 15:35:21 +0000 (21:05 +0530)]
Adding debug statements to track caching of Operational Vpn Instances
Change-Id: I0bb17dec5714e8f68085bf1b70e2b2fa1f2f16ae
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Vishal Thapar [Mon, 16 Jan 2017 15:55:29 +0000 (21:25 +0530)]
Add Trunks support
This adds hooks to bring Neutron Trunk configuration into netvirt.
Genius/IFM already has required parameters to support this.
Change-Id: Ic087c0b9a7d1c27d880533331164ae3af0a1fabb
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Michael Vorburger [Mon, 6 Feb 2017 23:56:15 +0000 (00:56 +0100)]
Fixed NodeConnectedHandlerTest with AbstractConcurrentDataBrokerTest
use new AbstractConcurrentDataBrokerTest instead of @deprecated
AbstractDataBrokerTest, and manage the impacts this had in
NodeConnectedHandlerTest (which was a bit of a mess; e.g. we typically
shouldn't mix a Mockito mocked transaction with a real one... and we
need to wait for DataBroker submit() to process via a get() before
asserting something)
Bug: 7538
Change-Id: I15ed9908b03d56bd84c4f637fc83fc62328f6f11
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
cgowdru [Mon, 13 Feb 2017 08:07:38 +0000 (13:37 +0530)]
Bug 7780 : NAT RPC's for getting SNAT/DNAT translation information.
Description : With the Current DS structure, it's is bit difficult to
identify information about what is the current DNAT/SNAT configuration
done fo the given router-uuid/vpn-uuid.
It would be good if we have a RPC's which can provide a information as
below.
input- router-uuid/vpn-uuid
SNAT output - router-name, internal-ip/port, external-ip/port,protocol
DNAT output - router-name,internal-ip,floating-ip
Change-Id: I00b6576d2a752b53e94ad28c611a06794c65456c
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Alon Kochba [Sun, 12 Feb 2017 16:29:05 +0000 (18:29 +0200)]
Fix several NPEs showing up in CSIT
1. ElanUtils - an ELAN instance can have no segments
2. TEPChangeListener - TEP could be missing in interfaces-state
3. VpnInterfaceManager - gateway IP/MAC could be missing when creating
ARP responder flows.
4. BgpConfigurationManager - NextHop in list could be null
5. TransportZoneNoticationUtil - not all interfaces have a neutron port
Change-Id: Ibbf6e5834cd01389e29e36736073927cdd3dbb1a
Signed-off-by: Alon Kochba <alonko@hpe.com>
Jamo Luhrsen [Sat, 18 Feb 2017 02:05:17 +0000 (18:05 -0800)]
Adding missing <name> to poms
looking at the output of a build, it seems we missed a few of these. I think this
might catch the rest of them.
Change-Id: I6311c42f71f1a63bf61a4215a8325b5929c04ce0
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>