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>

Add netvirt-features

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

NETVIRT-1037: Upstream Fixes --- Review-6

1) Support L3 Connectivity for subnets with no-gateway option set
2) Introduce jobCoordinator retries for critical tasks
3) Fix potential exception causing code
4) Avoid unnecessary request for ids much before its usage.

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

neutronvpn: update bgpvpn fallback pipeline rules for ipv6

When associating a BGPVPN to an external network, the identifier
associated to that BGPVPN is already used for all the underlying
external subnets declared in that network.

For IPv6 GUA subnets that are "behind that external network", that is
to say behind a router which has that external network linked, that same
identifier is used to write a specific fallback route for those IPv6 GUA
to access the Internet through BGPVPN. Indeed, when the router has no
more entries in its FIB table, a fallback rule is written for IPv6 GU
traffic against possibly BGPVPN IPv6 entries. That fallback rule
resubmits the packet to the FIB table, but with an ID set to the BGPVPN
id. Like that, all IPv6 traffic that has an entry associated may be
redirected to the correct DC-GW.

The modification here consists in adding that zero priority fallback,
rule, in the case a BGPVPN is present on external network, on the
presence of the first IPv6 subnet; removing the fallback rule, if the
BGPVPN is detached or the last subnet ipv6 is removed.

Change-Id: Iebd754ae10fa6db08512980a98e07c3a02ebb6d5
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>

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>