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>

Drop unused exceptions in vpnmanager

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

Drop unused exceptions in qosservice

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

Drop unused exceptions in neutronvpn

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

Drop unused exceptions in natservice

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

Drop unused exceptions in ipv6service

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

Drop unused exceptions in fibmanager

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

Drop unused exceptions in dhcpservice

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

Drop unused exceptions in bgpmanager

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

Drop unused exceptions in aclservice

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

NETVIRT-1079:Do not add a static route to Ubuntu

VM when nexthop IP is set to its own VM IP

Problem: Host route with next hop ip X is sent to VM with ip X.
         This creats a loop.

Fix: Skip self route in DHCP response i.e. if next hop is self IP
     then do not add that route in DHCP response.

Change-Id: I27a1ef1b6774659e817506535da2a0ed3546ed53
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

elanmanager: use transaction manager

This replaces all direct invocations of
DataBroker::newWriteOnlyTransaction() with calls to a
ManagedNewTransactionRunner, ensuring all such transactions are
closed. It also reworks a number of methods to use read/write
transactions, ensuring that changes are made based on still-relevant
data.

This doesn’t address transactions in ElanInterfaceManager.

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

Split up creation/deletion, clean transactions

This patch splits up creation/deletion methods in
HwvtepNodeDataListener, avoiding the use of boolean parameters to
control method behaviour.

It also ensures the read transactions are closed, and cleans up
generics as far as possible, along with removing uses of
ResourceBatchingManager.

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

Handle vpn removal from subnet correctly.

Network-id in subnetmaps was set to null causing remove subnet from vpn
event not being processed correctly. This fix ensures that network-id is
not removed from subnet.
Also, we do not store router-interfaces in subnetmap port list. Hence
removed logic which checks for device-id info.

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

NETVIRT-1110: Fixed stale ACL service binding entries

Issue:
Stale ACL related service binding entries were found during CSIT run.

Solution:
ACL listeners are updated to handle even if the order of interface
events changes.

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

Add feature and compatibility docs

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

More transaction clean-up in ACL

This involves splitting a couple of add/delete methods too, and
converting a number of AclServiceUtils methods from static methods to
instance methods.

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

removed halisteners

reused the existing node listeners to propogate child updates.
now the data gets copied from node listener only

added config manager listener to reconstruct ha cache upon restart
even if the node is not connected back post restart

fixed npe in ha ps node creation
copied db version of child ps node to parent ps node

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

sfc-classifier: use transaction manager

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

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

NETVIRT-1009 - Disable netvirt dynamic tunnels

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

Disable netvirt auto tunnels. There will be a separate
patch to enable genius auto-tz.

https://git.opendaylight.org/gerrit/#/c/66110/1

This will be the default starting in oxygen.
Carbon and nitrogen will continue to have netvirt
dynamic tunnels as the default.

Change-Id: I896c4d579cc105dedf4f9aa92275ba691850d214
Signed-off-by: Victor Pickard <vpickard@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Remove ElanInstanceManager#getElanInstanceByName method

This method always reads from the DS - we should use the
ElanInterfaceCache instead.

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

Refactor static WAITING_JOB_LIST in ElanInstanceListener

The WAITING_JOB_LIST functionality was moved to ElanInstanceCache.

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

Refactor ElanUtil interface maps  to a non-static singleton

Created an ElanInterfaceCache class to replace the static
elanInterfaceLocalCache and elanInstanceToInterfacesCache Maps
in ElanUtils.

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

Refactor elanInstancToDpnsCache to a non-static singleton

Created an ElanInstanceDpnsCache class to replace the static
elanInstancToDpnsCache Map in ElanUtils.

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

Refactor elanInstanceLocalCache to a non-static singleton

Created an ElanInstanceCache class derived form DataObjectCache
to replace the static elanInstanceLocalCache Map in ElanUtils.

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

Bug 8998  - stale l2gw connection cleanup

Upon cluster reboot fill the l2gw connection cache
by reading from config datastore.

Process l2gw connection even after dependent elan deletion.

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

NETVIRT-1103 : display issue in ip bgp vpnv4 all summary command
White spaces inbetween the text was getting deleted which caused the
display issue,so changing the regex in replaceAll to resolve the issue
Signed-off-by: Loshmitha <loshmitha@ericsson.com>
Change-Id: I48c5019fec167649f09766531a5f3e7b29c9d42f
Signed-off-by: Loshmitha <loshmitha@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Remove redundant names in paths

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

ELAN: skip remote unicast MACs

The HWVTEP node data listener has a log message saying “Skipping
remote ucast macs to parent”, but it doesn’t actually skip. This patch
adds the apparently appropriate return, and ensures the transactions
are closed (without using the transaction manager yet, that’s coming
in a future patch).

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

move vpnservice and cleanup poms

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

Fix build faliures due to OFPlugin checktyle fixes

OFPlugin checkstyle fixes have renamed some elements
used in Netvirt, which has broken netvirt builds. This
fixes it by using new names.

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

use feature for inter-project dependency

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

use feature for inter-project dependency

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

ACL: Part-II implementation of the spec.

+ This is part-II implementation of the spec "Support for protocols that are not supported by conntrack".
+ Handles overlapping IP addresses while processing remote ACLs.
+ Handles ACL update scenarios.
+ Handles overlapping SG rule scenarios.
+ Fixed issues found during manual testing.
+ UT will be handled separately as part of https://git.opendaylight.org/gerrit/#/c/66553/.

Depends-On: Ie82fa8bf1eb139039247adb2321a53babe8fdc83

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

ACL: Support for non-conntrack supported traffic.

+ This is implementation for spec "Support for protocols that are not
supported by conntrack".
+ This involves redesign of entire ACL pipeline.
+ UT is disabled in this patch. It will be handled separately as part of https://git.opendaylight.org/gerrit/#/c/66553/.

This patch currently addresses (a) and (c) of below three issues as
mentioned in the spec:
a. Enhance ACL to support protocols like OSPF, VRRP etc that are not
supported by conntrack in stateful mode.
b. Handle overlapping IP addresses while processing remote ACLs.
c. Optimization for Remote ACL by reducing number of flows even for ports
having multiple ACLs.

Patch-1:
+ ACL default flows are changed as per the new pipeline.

Patch-2:
+ Updated bind service to exclude writing ELAN/VPN ID into metadata.
+ Removed dependency on vpnmanager and it is not required anymore.

Patch-3:
+ Rebased

Patch-4:
+ Fixed compilation issues. Removed VPN-ID related code.

Patch-5:
+ Handled programming general fixed flows.
+ Handled programming port specific fixed flows.

Patch-6:
+ Rebased

Patch-7:
+ Handled programming ACL commit flows.
+ Refactored: Moved programAceSpecificFlow() logic from
Stateful***AclServiceImpl to AbstractAclServiceImpl class.

Patch-8:
+ Handled programming ACL dispatcher table.

Patch-9:
+ Updated programming ACL dispatcher table.
+ Handled programming Remote ACL table.

Patch-10:
+ Rebased

Patch-11:
+ Code-cleanup

Patch-12:
+ Rebased

Patch-13:
+ Handled port update. Update for port-security-enabled, AAP and SG
changes.

Patch-14:
+ Rebased

Patch-15:
+ Handle port-update with allowed-address-pair change.

Patch-16:
+ Used DJC to program ACL node default flows.
+ Used single transaction for programming ACL node default flows.
+ Refactored AclNodeListener.

Patch-17:
+ Refactored to keep single level of abstraction as no other
security-group modes are supported except stateful.
+ Updated to display cache aclTagMap.

Patch-18:
+ This patch was mistakenly uploaded by Nishchya.

Patch-19:
+ Rebased.

Patch-20:
+ Changed references from METADATA_MASK_REMOTE_ACL_ID to
METADATA_MASK_REMOTE_ACL_TAG.

Patch-21:
+ Rebased.

Patch-22:
+ Rebased.

Patch-23:
+ Rebased.

Patch-24:
+ Rebased. Aligned [0] with the latest ACL pipeline.
[0] https://git.opendaylight.org/gerrit/#/c/66788/

Patch-25:
+ Rebased.

Depends-On: I95df598428f6351e2abb0b173a4318253c9e20bc

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

Bug 8998 - Stale l2gw connection cleanup

upon cluster reboot if the device is not connected
it is not shown up in the cache or oper datastore.

Now when l2gw connection delete request is received it has no clue
on which device it has to delete the things and ignores the
delete request.

when the device connects later, cleanup job is scheduled
which will look for stale logial switches and deletes them.

any logical switch which is not present in networks dump or
l2gwconnections dump is treated as stale logical switch.

cleanup job is scheduled instead of firing immediately because
the network datastore may not be loaded yet as the cluster may
be just started.

cleanup if the device never shows up is not done.

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

For scaling to 150DPN , each TunnelIntfState add event causes a read of list of
interfaces on the given DPN.This list includes even non L2Vlan type
interfaces like tunnel-port etc. By ignoring these ports , read of
operational vpn-interface is avoided.

Depends-on: https://git.opendaylight.org/gerrit/63160

Change-Id: I3dd089f9409a8701ccedb64b3a85b1f40862499f
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>

NETVIRT-1033: Adding ArpResponder flows for SR-IOV VMs

DHCP Neutron Port ARP handling for SR-IOV VMs.
Handled removal of stale flows entry from ArpResponderTable.

New pipeline for processing Neutron port DHCP ARPs
Table 0 -> Table 18 -> Table 38 -> Table 81

Change-Id: Id5ee3160240cfcb6caec5790734d9f003c16640e
Signed-off-by: Achuth Maniyedath <achuth.maniyedath@gmail.com>
Signed-off-by: Vijayalakshmi Chickkamenahalli Nagaraju <vijayalakshmi.c@altencalsoftlabs.com>

Avoid shadowing fields

A couple of NeutronvpnUtils methods have parameters named dataBroker,
which shadow the field by the same name. These parameters aren’t
necessary any longer, this patch drops them.

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