Revert "Disable building sfc by default"

This reverts commit 03b638ad869fc47929e08dfd0464eee13ad357c2.

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

Enable building of sfc by default

Change-Id: I7e1dace00f188e9426aee9b433cdce365155d7ab
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

fix BindingToNormalizedNodeCodec NoClassDefFoundError in sfc.classifier

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

Missed one listener migration to genius/tools

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

Migrate from genius mdsalutil-api/utils to tools-api/utils

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

Clean up VPN data on VPN delete.

1. When a VRFTable is not created in FIB due to VPN addition, the VRFTable will
not be available during VPN deletion.
2. The prefix-to-interface, l3-nexthop datastores will not have any data for
a vpn, until a vpn-interface is up and running in the network.
Under such conditions VPN deletion causes
ModifiedNodeDoesNotExistException on trying to delete such datastores
which causes entire transaction which handles clean-up to fail, leave
stale-data in the system.

This further causes new VPN-create with same RDs to fail. This review
fixes the issue.

Change-Id: I3011d81907eb32693de3079bd118cfaaf1b9dfcc
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>

NETVIRT-1158 ping to floating IPs of VMs start working only after
pinging from within VM to the outside first

When the packet returned to pipeline after sending to conntrack had
metadata which ofplugin was not able to parse. Now the metadata
associated with conntrack is cleared before resubmitting dispatcher at
table 214.

table=214, priority=62030,ct_state=-new-est+rel-inv+trk,ct_mark=0x1/0x1
actions=ct_clear,resubmit(,17)
table=214, priority=62030,ct_state=-new+est-rel-inv+trk,ct_mark=0x1/0x1
actions=ct_clear,resubmit(,17)

Change-Id: I0c01bb2da5fdc95f6f44e03e2f2d21881a55958b
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
(cherry picked from commit 2c548a85205e2bef3d004e6811423acba277b86e)

Revert "Revert "NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between vm in same n/w in same compute is not working in stateful SG mode.""

This reverts commit 5cc1d18d2fae5d5590b80e82416170641a786ff0.

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

NETVIRT-1181: 3 issues related to port-udpate with Allowed address pairs
are addressed in this fix.
1. AAP added as port-udpate, is added for L2Broadcast allow rule.
2. AAP deleted as port-update, is handled to remove L2Broadcast allow
rule only if there is no other aap with same mac.
3. AAP deleted as port-udpate, is handled to remove Dhcp v4 & v6 allow
rule only if there is no other aap with same mac

Change-Id: Id422f2dd828265d12da4cc03b41f6cd4e6967569
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

Add fluorine and oxygen spec dirs

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

NETVIRT-942, NETVIRT-945 : Conflicting modification for path...

*Conflicting path modification exception is seen for
elan forwarding tables when two methods, removeElanInterface
and removeElanStateForInterface, are trying two remove the
same elan forwarding tables entry

*This is fixed by adding djc for each elan interface for
remove method

Change-Id: I8139e53fe08a236348975e653bf7cad2a32f65da
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Convert ManagerListener to non-static HwvtepNodeHACache

This one popped up after the original patches to convert to
non-static HwvtepNodeHACache were submitted while ago.

Let's get this patch and the subsequent genius patch that
removes the static version in ASAP to finally put this to bed.

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

NETVIRT-1173 : Prevent SNAT packets unnecessary looping

Problem Description:
======================
We noticed lot of SNAT packets are
getting punted to ODL which is causing delayed installation of certain
flows.

On analysis, it's been obseved that under one more scenario, packets
unnecessarily getting looped for 4 sec between ODL and OVS (threshold time we wait to flow
install before making drop decision)

1) N1-> subnet1 (10.0.0.0/24)
2) R1 and subnet1 added to R1.
3) VPN1(100:1) and R1 added to this VPN.
4) Ext-NET1 created and associated with Ext-BGPVPN.
5) Router-gw-set R1 with Ext-Net1

With this set-up, if following configuration is done,
certain packets are looped due to existing NAT bug

6) create N2(20.0.0.0/24) and associated this with VPN1.
7) Initiate TCP/UDP traffic from VMs of N2.

When first packet punted to ODL Controller, we make an
entry(<routerid>:<VM-IP>:<TCP/UDP Port>) in a map and try to find if
internal-to-external mapping available which will not be available for
N2's subnet(as this is not associated with R1) and we just drop that
packet. But, at this point we not taking care of removing of earlier added
entry. Hence, when the 2nd packet is punt to ODL Controller, this packet simply loops
between ODL<->OVS for 4 sec(same will happened with subsequent packets
too).

Solution Proposal:
==================
Changes done to prevent such packet being send back to OVS which is
causing congestion between ODL<->OVS.

Change-Id: I3215a5230f49e05e820b057d6fba415acad084a8
Signed-off-by: Shaik Zakir <shaikzakir119@gmail.com>

NETVIRT-1188: ACL: Stats not working for ingress direction

Fixed this problem by using lport tag in Cookie for drop flows.

Earlier, metadata matches which was available in pipeline egress side (VM
Ingress) is no longer supported. So, added lport details in Cookie itself
now to uniquely identify drop flows. Aligning with aforementioned changes,
removed metadata matches from direct stats input which only has Cookie
matches now.

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

NETVIRT-1068: Upstreaming fixes 2

Issue-1 : When VM is configured with extra-route, refcount in l3nexthop is
incremented. It gets incremented further due to following reasons:

(a) After initial extra-route configuration using command - neutron
router-update RouterA destination=IP-A,nexthop=prefix-A , if another
update is done using command - neutron router-update RouterA
destination=IP-A,nexthop=prefix-B , neutron router listener calls update
on prefix-A as well as prefix-B. On prefix-A , secondary adj (IP-A) is
removed , where as its added on prefix-B. This back-to-back update creates
race-condition in Vrf Engine , leading inconsistencies in l3nexthop,
VpnExtraRoute, VpnInterfaceOp DS. (b) After initial extra-route
configuration using command - neutron router-update RouterA
destination=IP-A,nexthop=prefix-A, if cluster reboot is performed ,
TEP-ADD event triggered the update of FIB entry for IP-A. Update call in
FIB leads to increase in refcount of l3nexthop for prefix-A.

After refcount has reached high number due to issue-(a) and (b) , if VM is
migrated , group will not be programmed on destination DPN,, thus leading
to VM becoming un-reachable.

Fix: For issue-(a) , a temporary fix of 2sec delay is introduced in
neutron. A better fix/design need to be thought to avoid race condition.
For issue-(b) , after cluster reboot , secondary-adj FIB updated is
avoided due to wrong check in updateVpnInterfaceOnTepAdd method. refcount
is removed from l3nexthop yang because , after cluster reboot , due to
multiple Add/Update replays for a given prefix, refcount goes higher than
it should be. Hence prefixes using a group themselves are stored in
l3nexthop , so that even if spurious Add/Update are triggered post cluster
reboot for same prefix , its not updated.

Change-Id: I4618303345db1241b1826b018424b3df0f8bd9ec
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1187 Imported FIB Route is not being translated to flows

Caused due to NPE by calling a getter on a null extra-route object while
processing an imported FIB.
Also, remote table-21 flow creation was broken for imported routes.

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

Migrate HwvtepHACache users to HwvtepNodeHACache

The static HwvtepHACache instance is replaced by the
HwvtepNodeHACache singleton service so migrate users.

Change-Id: I3bab48856df6d1ccb4aad0d34c90b4d68d64d668
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Java 8 migration

* Use method references instead of lambda expressions where
  appropriate.
* Use lambda expressions instead of anonymous functions where
  appropriate.
* Use Map::forEach instead of streaming over entry sets and retrieving
  matching values.
* Use Map::computeIfAbsent instead of manually handling missing
  entries.

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

Drop unnecessary explicit types

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

Java 5 migration

* Use foreach loops instead of indexed for loops or iterating while
  loops.
* Use String::contains instead of String::indexOf.
* Use StringBuilder instead of StringBuffer, or a collector when the
  StringBuffer is only used to join a string collection.

This patch also simplifies the return value handling in
FibHelper::doesPrefixBelongToSubnet.

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

Ignore xtend-gen files

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

Removal of deprecated ACL table constants.

With new ACl pipeline old ACL table constants are not deprecated same were
present in Unit tests. Renaming them with new pipeline table names.

Change-Id: I7419897ce5970d14539a0fe442b0abc3700df195
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

Disable building sfc by default

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

NETVIRT-1155: L3nexthop group count is not decremented for extra-route

Change-Id: I666c065ee72bbf604f596ec6c51a745f9466091f
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
(cherry picked from commit 596786022adfadabe39b7a4fd30b1a1e3c87139b)

ELAN Service Recovery

Below service recovery options are supported
for ELAN:
* ELAN service recovery (deregister and register
  ElanInterface Listener again)
* ELAN Interface recovery

Change-Id: I7ded46eb34ec6c7942cd4a7903f2ad0a2a01df5d
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Add debug log for port status update

Make this puppy a little more debug-able.

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

NETVIRT-1166:IPv6 NA packet is setting with wrong solicited flag

Problem Description:
====================
Currently ODL IPv6 Service is doing Neighbor Advertisement (NA) for
Neighbor Solicitation (NS) request.
NS packet may contains IPv6 Source Address as below possible way:
:: --> Unspecified
SLL --> Source Link Local Address

When NS packet is punted to the ODL controller, It will prepare the NA
packet and it will send this NA to NS initiated nodes.
NA packet contains IPv6 Destination Address as below possible way based on
the NS packet received.
FF02::1 --> All node multicast IP address (FLAGS: R:1, S:0, O:1)
Dest_LLA --> Destination LLA taken from NS (FLAGS: R:1, S:1, O:1)

Currently ODL controller is setting Solicited Flag=1 for All node
multicast IPv6 address. Which is wrong.

Solution:
===========
ODL IPv6 Service Defined constant with wrong value is the root cause of
this issue. To address this issue have made the changes properly with IPv6
constant.

Change-Id: I0b0868ad78a904c8e7ea4d0d157b16d6ef309974
Signed-off-by: karthikeyan <karthikeyangceb007@gmail.com>

Remove config import from inter-vpn-link.yang

CSS remnant - not needed

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

Remove last use of AbstractDataBrokerTest

This has been deprecated in favour of
AbstractConcurrentDataBrokerTest. Note that BgpManagerTest’s tests are
actually useless since they assert that 1 equals 1, but that’s
something for another patch.

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

Remove all dlux dependencies

These were already disabled, this patch removes them.

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

Opening update server port with specified IP address (if configured)
update server port shall depend on EoS-entity-BGP.

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

Remove deprecated AsyncWriteTransaction#commit method

This method has been deprecated for quite some time and now has a
default implementation so can be safely removed from sub-classes.

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

NETVIRT-1169 : Upgrade - GW-MAC flows are sometimes not programmed on some of the DPNs

Caused due to OLFE between vrfEntryListener.populateFibOnNewDpn() and
vrfENtryListener.createRemoteFibEntries().
This fix makes use of RetryingManagedNewTransactionRunner to ensure that
transaction is successful.

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

NETVIRT-1170: Deleting VM with a learnt IP does not clean up
prefix-to-interface DS for the VM

A check that ignored non-primary adjacencies while processing an
operational vpn interface delete was causing this failure. This is
observed in cases where a non-primary adjacency is THE LAST adjacency to
get deleted for the VM interface.

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

RELENG-63: Move COE bundle to odl-netvirt-coe

odl-netvirt-impl currently includes the coe-impl bundle, which causes
issues for two reasons:

* coe-impl depends on coe-northbound-api (from the coe project), but
  odl-netvirt-impl doesn't depend on that (directly or indirectly); this
  results in the odl-netvirt-impl including a coe bundle which is
  incorrect;
* the coe-northbound-api is included in coe’s odl-coe-api feature, so
  the above results in two features containing the coe-northbound-api,
  which is also incorrect.

Moving coe-impl to odl-netvirt-coe ensures that odl-netvirt-impl itself
stays independent of COE, and ensures that no NetVirt feature directly
includes a COE bundle.

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

NETVIRT-1062: refactor SubnetmapChangeListener

* SubnetmapChangeListener:

    don't need to call vpnSubnetRouteHandler.onSubnetAddedToVpn method
    second time if Subnetmap.getInternetVpnId attribute was set.
    According to initial design,
    vpnSubnetRouteHandler.onSubnetAddedToVpn method doesn't take in
    account InternetVpnId, when it creates/updates/removes
    SubnetOpDataEntry for the given Subnetmap.

* ExternalSubnetVpnInstanceListener:

    fix potential race condition. It is better at first to create an
    Internal VPN instance for the given provider subnet and only then
    try to update all corresponding objects.

* NeutronvpnManager:

  ** updateVpnInternetForSubnet:
    should always keep set VpnId attribute for updating Subnetmap.

  ** associateExtNetworkToVpn and disassociateExtNetworkFromVpn:
    to avoid race conditions it is better at first to update BgpVpnType
    of VPN instance and only then to update associated/disassociated
    SubnetMaps with a new InternetVpnId/null.

JIRA: NETVIRT-1062
Change-Id: Ia388b9112f2ca683864c2c15d8ac44ed0e0c25b8
Signed-off-by: Valentina <valentina.krasnobaeva@6wind.com>

Service-Recovery: Below service recovery actions are supported for ACL
(a) ACL service recovery: As part of this, all ACL listeners are
re-registered (unregistered and registerd again)
(b) ACL Interface recovery
(c) ACL Instance recovery

Updated with ACL-Instance recovery changes.

Service recovery actions supported are defined in SRM module with below
patch set:
https://git.opendaylight.org/gerrit/#/c/68987/

Depends-On: I4c91cd52e7595b92b494356dfa715dbdf15da3c5

Change-Id: I5517effbec7f00e8fd48e656d9443f525fc74eeb
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Fixed typo changing "doesnt" to "doesn't"

My first delivery to the Netvirt project :-).

Change-Id: If4596d1031f15258cd490f4145e40d13725cde78
Signed-off-by: Ariel Adam <aadam@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Revert "Service-Recovery: Below service recovery actions are supported for ACL (a) ACL service recovery: As part of this, all ACL listeners are re-registered (unregistered and registerd again) (b) ACL Interface recovery (c) ACL Instance recovery"

This reverts commit fb00e158d0f667046d89d66c996ace9b423af444.

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

Revert "NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between vm in same n/w in same compute is not working in stateful SG mode."

This reverts commit 137537a26188f2139ee44eb35ad6ac945eb26a8b.

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

NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between
vm in same n/w in same compute is not working in stateful SG mode.

Replace the dummy commit with zone 0 with ct_clear action. This clears
the ct state in the current packet and will submit the packet to ct
again. So in cases where ct(table=x, zone=2) action twice in the
pipeline( with same zone) , the second one also reaches the netfilter.

Requires ovs2.9 with kernel module installed.

Depends-on: Change-Id: I163ed99d5af8d3643749ac2a278f5c8170d7ae29
Depends-on: Change-Id: If86203e73a5ec516ff487f3f5c33fbd7dbfd12e1
Change-Id: Iab842c6232476d499a3d8c8602d5b2f57f9e4e1f
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Unit Test Cases Changes

With the new table changes in ACL current ACL unit tests are not working,
changes required accordingly to make them work.

Change-Id: Ia127caf792420fce141978c7309c524e0fb537b0
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

NETVIRT-1076: refactor arrow anti-patterns

* NeutronvpnUtils:

    ** getSubnetMapsforNetworkRoute() method is replaced by
       getPrivateSubnetsToExport(), which obtains the list of private
       subnets from router in more simple way;
    ** addExternalNetworkToVpn() and removeExternalNetworkFromVpn() were
       moved to NeutronvpnManager to avoid import collisions.

* NeutronvpnNatManager:

    ** use getPrivateSubnetsToExport() instead of
       getSubnetMapsforNetworkRoute();
    ** fix log messages.

* NeutronvpnManager:

    ** refactor arrow anti-patterns in associateNetworksToVpn() and
       dissociateNetworksFromVpn() methods;
    ** add a support of use-case, when BGPVPN can be associated with more
       than one Provider Network;
    ** fix log messages;

JIRA: NETVIRT-1076
Change-Id: Ifea2e0d2c2631ac87a3883f5a59d5fd3f189f812
Signed-off-by: Valentina <valentina.krasnobaeva@6wind.com>

Stop using MDSALDataStoreUtils

This patch removes all uses of MDSALDataStoreUtils, either using
SingleTransactionDataBroker, or a managed transaction.

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

CLI to print DHCP caches.

Added dhcpcli to print dhcp caches .

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

Service Recover for QoS Service as a whole & QoS Policy Instance.

For QoS service Recovery, listeners in the QoS service will be
re-regisered. Following command will do service recovery for QoS
service.
-----
srm:recover service qos
-----

For Qos Policy Instance Recovery, The policy id giving in the following
command will be reapplied on all the networks and port on which it was
applied earlier.
-----
srm:recover instance policy <policy-id>
-----

Change-Id: If02bc0a610d216f4387832ac21dac80aafb7d70e
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>

Switch to using lfdocs-conf to pull in theme

lfdocs-conf is a new project that allows us to centrally manage
project theme configuration without having to duplicate it
everywhere or have the "docs" project become a large submodule
project.

We will push to ReadTheDocs separately and then create a new path
for this doc under docs.opendaylight.org/projects/integration-packaging

For further details refer to:
http://docs.releng.linuxfoundation.org/en/latest/project-documentation.html

Change-Id: Iefbb38dd3fa4714c5018d6553bfde41e45e2bd37
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Service-Recovery: Below service recovery actions are supported for ACL
(a) ACL service recovery: As part of this, all ACL listeners are
re-registered (unregistered and registerd again)
(b) ACL Interface recovery
(c) ACL Instance recovery

Updated with ACL-Instance recovery changes.

Service recovery actions supported are defined in SRM module with below
patch set:
https://git.opendaylight.org/gerrit/#/c/68987/

Depends-On: I4c91cd52e7595b92b494356dfa715dbdf15da3c5

Change-Id: I21be0f9ed05eb3bf17ddb422ed10196e122b5cc8
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

NETVIRT-1148:SNAT Traffic is not working with Dual Stack Router

Problem Description:
======================
Existing IPv4 NAT functionality is not working with Dual Stack Router
configuration scenario. Packets are getting dropped at ODL Controller.

Solution Proposal:
==================
Due to IPv4 and IPv6 Subnets are part of the single SNAT router, ODL
controller is not able to handle IPv6 Subnet properly. Due to this reason
IPv4 SNAT traffics are not working. To address this issue have made the
changes in IPv4 NAT service code to skip the IPv6 subnet process for
NATing. Since IPv6 NAT is not required. All IPv6 address are GUA
address's.

Change-Id: I7be6a999542e156047368203eee8e865b2c2eaac
Signed-off-by: karthikeyan <karthikeyangceb007@gmail.com>

NETVIRT-1144: PNF failure: Ping External Network PNF from Vm Instance 1

The rotuer<->bgpvpn swap procedure was being triggered even for addition
of a new adjacency to the vpn-interface, leading to a delay of two
seconds(the sleep present in the swap logic) in learning the PNF route.

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

NETVIRT-1132: NPE in TunnelInterfaceStateListener

Fix NPE caused by auto boxing by initializing
boolean in yang model to "false" so that the
Boolean object will not be null when checked,
for example, in if statements.

Details of analysis can be found in the JIRA
https://jira.opendaylight.org/browse/NETVIRT-1132

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

csit error log cleanup

when l2gw use case is not executed oper elan instance is not present
do not put error log if oper elan instance is absent.

Change-Id: Ie462ee96ce9239d5159b3a5809e0cce1558423f8
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1134 : ModifiedNodeDoesNotExistException bgp/networks in CSIT

This exception was being thrown when a delete was performed on a
non-existent node.
This was because of the data-remaining in the data-stores that were
created during the initial_create-networks on devstack bringup.

This patch aims at proper handling of deletion of those bgp routes that
have no nexthop.
It also prevents withdrawal of extra-routes from bgp when such the router
for the extra-routes is not associated to a bgpvpn

Change-Id: Ia666014c03b3f3454878487d3d42ef756a8bd2b2
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1141: Remote ACL count missing in acl stats data

While displaying the stats for acl it is not considering remote acl
drop in the count.
Providing implementation to consider that also.

Jira Id:https://jira.opendaylight.org/browse/NETVIRT-1141

Change-Id: I72fa8ad261ada769a65ae7e992e670523ed99b83
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

Switch to InstanceIdDataObjectCache

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

Revert "NETVIRT-1134 : ModifiedNodeDoesNotExistException bgp/networks in CSIT"

This reverts commit eead978ce7038d1709ff0456134650ca603acf72.

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

NETVIRT-1146 : Exception IllegalArgumentException due to null subnet-id
during extra-route vrf-entry deletion

A bug in one of the checks during extra-route creation was leading
to exceptions in the delete path.
Label-Mapper is a bgpvpn exclusive data-store and should not be populated
for router based interal-vpn adjacencies. Such a population lead to
iRT/eRT code execution during the delete extra route life-cycle for plain
router case, leading to this exception.

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

NETVIRT-1144 : Using JC for ExternalNetGroup Installation

Description:
Due to OptimisticLockFailedException during installtion of external
network subnet group, the installation of default FIB rule for
external-subnet CIDR been not installed resulting in PNF UC failure.

Change done to use JobCordinator fo installing this group there by
preventing multiple listeners attempting to install the same group
symultanously.

Change-Id: I667077eb2750e9e48019e5a2ad70520d3ada4d8b
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

NETVIRT-1134 : ModifiedNodeDoesNotExistException bgp/networks in CSIT

This exception was being thrown when a delete was performed on a
non-existent node.
This was because of the data-remaining in the data-stores that were
created during the initial_create-networks on devstack bringup.

This patch aims at proper handling of deletion of those bgp routes that
have no nexthop.

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

NETVIRT-1045 No connectivity between instances on different
networks connected to the same router when using VLAN setup

A pseudo port is added to ElanDPNList  when a vlan becomes a part of the
VPN for all the dpn which has the VPN. This ensures that the patch
ports to the physical interfaces are created and necessary FIB flows are
programmed.

The pseudo port will be add a subnet becomes a part of the VPN or when
the first vpn port is added to the DPN.

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

statemanager: document the unused dependencies

StateManager appears to have a number of service dependencies which it
doesn’t use; but its whole point is to signify that the set of
services it depends on is ready, as a proxy for the system as a whole
being ready. This patch suppresses unused warnings.

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

Enable fingbugs-slf4j enforcement in binding-parent

This ensures we won’t have any regressions in log handling in modules
based on binding-parent. We will update the dependency on odlparent
once the 3.0.3 or 3.1.0 release is available.

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

Fix logging issues in sfc

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

Fix logging issues in vpnmanager

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

Fix logging issues in alarm

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

Fix logging issues in cloud-servicechain

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

Fix logging issues in neutronvpn

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

Fix logging issues in natservice

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

Fix logging issues in coe

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

Fix logging issues in fibmanager

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

Fix logging issues in dhcpservice

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

Fix logging issues in bgpmanager

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

Fix logging issues in aclservice

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

NETVIRT-1142: Fix mistakenly switched port pair ports

When translating a port pair to data plane locators, the ingress port
was set as the reverse DPL and the egress port as forward DPL which is
inverse as to what it should be: the ingress port is the forward DPL and
the egress port is the reverse DPL.

Change-Id: I1251aaad0ce3fee336b8544858ce43c5b057c4f0
JIRA: NETVIRT-1142
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

NETVIRT-1142: sfc classifier bind to proper egress interface

SFC classifier binds on the SF egress interface of the last RSP hop to
be able to properly route the traffic to its original destination in
case it is on the same compute as the original source of the traffic.

With the introduction of directional data plane locators for symmetric
paths in SFC, the ingress and egress interfaces of a SF might be
different and the classifier has to take care to bind to the correcti
one depending on the direction of the path, wether it is the reverse
direction or the forward direction of the symmetric path.

Before this patch, the classifier binds to any interface of the SF, as
only one was expected.

Change-Id: I98090ea21d14a62cd19e5ad9065e8c9a90e03c11
JIRA: NETVIRT-1142
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

NETVIRT-1139: Invert matches for reverse path classification

When an ACL translated from a flow classifier includes a logical
destination port, ACL matches should be inverted to classify traffic on
that port to the reverse path, while the original matches are used to
classify traffic on the logical source port to the forward path.

Change-Id: Iaaff8106d9bfa6d1e8bd346b9c6f2f0ff0659817
JIRA: NETVIRT-1139
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

NETVIRT-1138: Fix classifier on multiple RSPs per SFP

When there are more than two RSPs per SFP, it is possible that the
classifier will classify uplink/downlink traffic into paths that are not
symmetric to each other.

This fix includes:

- Translator no longer explicitly creates the RSPs. SFC will
  automatically create the required RSPs for a SFP. This prevents that
  there are more than two RSPs per SFC as long as no third party
  creates more RSP, which will not be possible once the RSP create rpc
  is removed in Fluorine.
- Classifier will ensure that there is no more than two RSPs for a SFP
  and that they are symmetric to each other.
- Since the RSPs are no longer created explicitly, a new listener on the
  SFP state is required to react at the time the RSPs are automatically
  created.

Change-Id: Ifd44ab12ab12464351ecb4cf45bf9b9704accdf6
JIRA: NETVIRT-1138
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>

fix logging issues in statistics

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

logging issues in ipv6service

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

Fix logging issues in policyservice

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

Fix logging issues in qosservice

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

logging issues in elanmanager

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

Fix more bad logging in vpnmanager

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

Fix broken Create multiple VPNs tests

Deletion of bgpvpn with router associations should move vpn-interfaces to router

On deletion of bgpvpn with router associations, vpn-interfaces under
that bgpvpn will get updated with router as its vpn-instance name.
This will trigger remove and add of vpn-interface logic.
At present, former happens but latter does not.

Since, vpn-interace operational data is moved from operational
vpn-interface object to vpn-interface-op-data keyed on rd, interface name
vpn-interface remove/add can proceed without inter-dependency.

This fix removed batched processing of vpn-interfaces in swap scenarios
and does remove and add then and there in update event handling.

Partial-Bug: NETVIRT-1106
Change-Id: Id2b4b07e6c99fb3b5ecdc0f6063fdbd8a854e415
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>

fix bad Exception logging in TunnelInterfaceStateListener

and a bunch of other clear bugs in logging not related to exception

We will be enforcing this locally by adopting the findbugs-slf4j
bug-pattern proposed earlier in odlparent locally in netvirt already;
watch Change-Id: I91f21504c9a18580733a1eedd3ba379de79d37c2.

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

NETVIRT-1128: Add sfc back into netvirt features

Change-Id: Ie57e2d4ade222548931ada8185a590ab01cd6c2a
JIRA: NETVIRT-1128
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1122 - L3 ping between pods not working

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

Sync nitrogen with pike

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

Add CSIT update doc for OS version changes

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

oxygen updates to support matrix

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

fix delete Direct with switchdev port

Change-Id: I3165d5341770296052ded02751aabc3bee654cf3
Signed-off-by: Moshe Levi <moshele@mellanox.com>
Co-authored-by: Hamdy Khader <hamdyk@mellanox.com>

NETVIRT-1126: avoid shipping the MySQL connector

This strips the MySQL connector from the Karaf distribution we build,
to avoid shipping a GPL component in the distribution.

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

NETVIRT-1123: Remote ACL issue during cross SG rules

Remote ACL table was not programmed when port is the first port on
the dpn for a remote ACL in the below scenario.

VM1(sg1) <-> VM2(sg2)

sg1 -> allow icmp to/from sg2
sg2 -> allow icmp to/from sg1

Change-Id: I7991daa98730a9922aa52665489345a43b66f2c8
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Bump versions by x.(y+1).z for next dev cycle

Change-Id: Ice1563468d8e2b032ff2c2399c87a91ecd84369e
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Remove unused parameters

This patch removes unused parameters (except where they’re obviously
WIP).

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

yang chages for : Faster DC-GW Failure Detection (L3VPN with BGP)

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

ACL: Handle AAP update

During port-update with add of allowed-address-pair (IP/MAC),
corresponding flows were not added in table=210 for egressAclDhcpAllowClientTraffic()
and egressAclDhcpv6AllowClientTraffic(). This issue is addressed with the
patch.

Also, avoided configuring flows related AAPs with multicast addresses in
VM egress anti-spoofing table (table=210) as the source address will never be
multicast address.

Change-Id: Ibe66cc65a484864c53a085b70d2aa37486e662d4
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

avoid race in elan bc group update

elan broad cast group update happens from
1) first elan dpn interface add (runs in oper default shard leader node)
2) l2gw connection add (runs in eos/cluster singleton owner node)

To avoid the race between these two
add the l2gw tep to operational elan external teps and keep the job key as
elan name

handle the tunnel ip change of l2gw gracefully

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

statemanager: use transaction manager

This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed.

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

Drop unused exceptions in elanmanager

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