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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

BGP-EoS leader flap leads to NPE.

Change-Id: Idfc5dec1416a93728eae2b9a307b456a3a6177cc
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

Refactoring and Duplicate Id detection

Change-Id: I4c76953a9dec8bf044374ae1a6dd6afb007bf4b2
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Remove illegal characters from yangs

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

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>

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>

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>

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>

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>

Switch some stragglers to odlparent 3.0.2

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

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>

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>

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>

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>

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>

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>

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>

Include AClStatefulTest

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

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>

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>

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>

Faster DC-GW Failure Detection (L3VPN with BGP)

Change-Id: I742ca73de057fddbf91f76ff9b3b18849a5774f8
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>

Add odl-jolokia to the NetVirt distribution

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

Fix getRenderedServicePath sfc test hanging

Change-Id: I8970f01d41313df7ce65ef83e4c7dfd166e20ea6
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

add aaa-cli-jar to distribution/karaf

see Ie47ea696190c9fad6472375007b2767db1f0fbed
and Ifdb3e87cee4cb80f0c753ead620077cbf98af0f8

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

Ignore hanging UT: getRenderedServicePath

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

OPNFLWPLUG-972 : Point to openflowplugin liblldp

Migrated liblldp from controller to openflowplugin

Change-Id: I2a0ef324d764d6908b4c9b03f3ac168eaac9f662
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

Re-enable SFC

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

Revert "Revert "Add hardware offload support for openvswitch""

This reverts commit 69ab0b21fe50db21443616a2a0e6bcd5e4806c11.

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

Revert "Add hardware offload support for openvswitch"

This reverts commit 29d3ba79843ef4dfdd1461f5c12c071bde1cbd8f.

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

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

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>

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>

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>

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>

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>

Minor: fix two log statements

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

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>

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>

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)

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>

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>

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>

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>

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>

add csit tools

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

Adjust for odlparent 3 / yangtools 2

This disables ElanServiceTest which is broken (see GENIUS-109), and
SFC to enable earlier testing.

Change-Id: I204aabf268275b1f9d359c85db4263ad07358d5e
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

use more common logger for alarm (i.e org.slf4j.Logger)

Change-Id: I3ea7f6ace69406facd421e10f22b13f2e9505198
Signed-off-by: Noel de Prandières <prandieres@6wind.com>

spec for sriov-hardware-offload

Change-Id: If4a137f24f65785daa2a651b70aaf1aa32ae043a
Signed-off-by: Moshe Levi <moshele@mellanox.com>
Co-Authored-By: Edan David <edand@mellanox.com>

Disable building policyservice

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

disable building odl-ovsdb-ui

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

Move SFC dependencies to a dedicated profile

This creates an “sfc” profile which must be enabled if SFC
dependencies are to be built. It *is* enabled by default, so the
current behaviour is preserved, but can be disabled by enabling the
“no-sfc” profile.

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

Convert VpnPseudoPortCache to non-static singleton

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

NETVIRT-1038 Fix bind logic in policymgr

TunnelStateChangeListener in policymanager seems to have
inconsistency between when it does bind and unbind.

Bind is called for vxlan tunnels while unbind for logicalGroups.

Fix is to call bind also for logicalGroups only as captured
in the feature spec [1].

[1]
http://docs.opendaylight.org/en/latest/submodules/netvirt/docs/specs/policy-based-path-selection.html#proposed-change

Change-Id: I25946ef80a313028427f49378494324f06a7c8bf
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Log level and message updated

Log level updated from warn to error and
added few more parameters into log messages of AclServiceUtils and
NeutronSecurityRuleListener classes to help in debugging.

Change-Id: Ia01c033a7b383de6bcf368f027d9b8e21f72b1a9
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Bug 8998 - sriov vm connectivity broken

Use the right dpn id while updating its broadcast group

Change-Id: Ifd2d3f3a445a2621cfc1aae74e48037e95fac5ed
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>

Convert L2GatewayCacheUtils to non-static singleton service

Crrated an interface, L2GatewayCache, with an associated
L2GatewayCacheImpl that is advertised as an OSGI service.
The L2GatewayCacheUtils had a couple update* methods that
only had one call site so I just inlined the code in the
caller to keep L2GatewayCache as a simple cache.

To avoid perpetuating circular bundle dependencies and having to
use optional availability on reference import, I put the
L2GatewayCacheImpl in a new bundle, cache-impl.

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

Remove unused static fields in HwvtepTerminationPointListener

The static fields, waitingJobsList and teps, are essentially no
longer used (ie have no effect now) so remove them.

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

Fix ConflictingModificationAppliedException with 3 retries

see https://jira.opendaylight.org/browse/NETVIRT-916

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

Update cni specification link in coe spec

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

Deletion of POD is not deleting corresponding ietf-interface

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

Updates to COE ELAN integration

- All pods of same namespace within the same node
  should be part of same ELAN.
- Pods of same namepsace, but under different node
  will not be part of same ELAN.
- Ietf interface as well as elan-interface names
  should have a format namespace:podName to match
  with the external id coming from CNI plugin

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

Adding compute node scale in scale out spec

Change-Id: I382addbc8db175bf3a1f07640d28f5a83791f80d
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Drop Karaf shell specifiers

We no longer need to support Karaf 3, and the karaf.shell
"[3.0.0,4.1)" declarations prevent us from upgrading to Karaf 4.1
(which is coming with odlparent 3).

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

Spec for weighted NAPT selection

Change-Id: If4e6540373a8caf2ef7f401c7ee11df7b9debf1d
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

NETVIRT-970: IPv6 Cluster Support

This patch updates the listeners in ipv6service to be
cluster aware and ensures that only the cluster owner
does the following.
  a. programming the necessary ipv6 control path flows.
  b. sending out periodic router advertisements.

Change-Id: I4b437b14e13da93701fcd94ac271ff3fe4426983
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

NETVIRT 1029:Route label mismatch for Subnet route
For an update of network,BGP was deleting the old network,sleeping for 5
secs and adding a new network to QBGP.But if the same network update
came in this 5 secs,it was creating issues.So removing 5 sec delay after
checking with QBGP guys

Change-Id: I9a4cb8f096b608e84e377eeb301d3bd52519b175
Signed-off-by: Loshmitha <loshmitha@ericsson.com>

Bug 8998 - Adding support for multiple PS

Following Listners do following job
onGlobalNodeAdd ------->copy child operational global node to parent node
copy parent config global node to child node
copy child ps operational node to parent ps
copy parent ps config to child ps config
onGlobalNodeUpdate ---->copy child operational global node update to parent
onGlobalNodeDelete ---->delete parent operational global node if all child got deleted
onPsNodeAdd ----------->copy child operational ps node to parent ps
                        copy parent config ps to child ps
onPsNodeUpdate -------->copy child update to parent
onPsNodeDelete -------->delete parent operation ps node if all child got deleted

Change-Id: I27d2e6896cadc38fd7c88e5d469ae89603e687c7
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>

Tools to troubleshoot netvirt

Change-Id: Ie008f8f67dbff3ca980e3b48af121275c98e2014
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Don't export packages in impl bundles

Add empty <Export-Package> to the maven-bundle-plugin configuration
to prevent any dependencies on the bundle and prevent @Singleton
annotated classes from being accidently included in another bundle's
blueprint XML.

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

Fix odlparent 3 Checkstyle issues

This doesn’t upgrade to odlparent 3, but fixes all the issues that the
new Checkstyle settings in odlparent 3 pick up; in particular, utility
classes need private constructors and need to be final, and parameters
and local variables mustn’t shadow fields.

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

Spec for coe-netvirt integration

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

Add findbugs plugin to the binding-parent pom

Now that virtually all of the sub-projects have findbugs enabled, we
can now add it to the binding-parent pom and remove it from each
sub-project pom. This has the added benefit that new sub-projects will
automatically pick it up.

Also fixed remaining violations in:

  aclservice-shell
  fibmanager-shell
  neutronvpn-shell

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

Eliminate circular dependencies and convert to BP annotations in elanmanager

ElanInstanceManager -> ElanInterfaceManager -> ElanUtils -> ElanInstanceManager

  Elimnated ElanUtils -> ElanInstanceManager by moving
  ElanInterfaceManager.getElanDPNByName to ElanUtils.

Eliminated ElanInterfaceManager.setElanUtils and
ElanUtils.getElanL2GatewayMulticastUtils that were put in to avoid circular
dependencies between ElanUtils, ElanInterfaceManager, ElanInterfaceManager and
ElanL2GatewayMulticastUtils. The circular dependencies were eliminated by
eliminating the dependencies between

  ElanL2GatewayMulticastUtils -> ElanInterfaceManager and
  ElanL2GatewayMulticastUtils -> ElanInstanceManager.

The former entailed moving
ElanInterfaceManager.updateRemoteBroadcastGroupForAllElanDpns et al
to ElanL2GatewayMulticastUtils.

The latter entailed the caller looking up and passing the ElanInstance to
updateRemoteBroadcastGroupForAllElanDpns via handleMcastForElanL2GwDeviceDelete,
handleMcastForElanL2GwDeviceAdd. The call sites for these are
DisAssociateHwvtepFromElanJob and LogicalSwitchAddedJob via
L2GatewayConnectionUtils, AssociateHwvtepToElanJob and HwvtepLogicalSwitchListener.

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

Convert InterVpnLinkCache to non-static singleton

InterVpnLinkCache is now an interface with InterVpnLinkCacheImpl
that is advertised as a service. InterVpnLinkCache is also used
by fibmanager-impl which, unfortuantely, creates a circular
service dependency with vpnamnager-impl so the service import
ws made optional. A better sution would be to put the
InterVpnLinkCacheImpl in a separate bundle.

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

NETVIRT-1023 - OptimisticLockFailedException: infrautils.jobcoordinator-impl

https://jira.opendaylight.org/browse/NETVIRT-1023

Looks like my patch below introduced a new exception.
 https://git.opendaylight.org/gerrit/#/c/65726/

Looks like only the TZ should be deleted, which will also delete
the VTEP. So, just delete the TZ.

Note: This exception only showed up in CSIT runs, did not see
any exceptions in local testing.

Change-Id: I4d6c0a180a2845b4658b6759222b1fc1e782f0db
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Updated to call bind/unbind from ACL interface state listener to avoid
stale flows during VM migration

Change-Id: I4b61f96b0672e4a70160926d762510f665eb7732
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Miscellaneous cleanup in natservice-impl

See inline comments.

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