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>

coe: use transaction manager

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

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

NETVIRT-1037: Upstream Fixes --- Review-3

This reviews stores gwMac and lportTag into operational vpn interfaces,
in-case retrieval of the same through ietf-interfaces state fails while
handling an interface remove event

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

NETVIRT-1037: Upstream Fixes --- Review-2

If an extra-route/learnt IP has the same IP as a primary adjacency on the
interface, it is not processed.

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

NETVIRT-1037: Upstream Fixes --- Review-1.

1) Handle VrfEntry updates only for static routes/cluster reboot scenario
2) Fix a few Exceptions seen in the logs during TEP events

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

NETVIRT-1068: Upstreaming fixes

Issue-1: When VpnInterface has a secondary adjacency like invisible ip, if tunnel
EP is deleted , VpnInterface Op DS is updated with empty nexthoplist.
During this time , invisible ip goes missing from VpnInterface Op DS due
to MDSAL.syncwrite() being done in updateVpnInterfaceOnTepDelete. Another
fix being done here is , invisible ip FIB entry is updated with empty
nexthop and route is withdrawn/advertized to BGP

Issue-2:  prefix-A part of subnet-A/network-A/VPN-A. The refcount for
prefix-A is 1. When VPN-B is importing from VPN-A , refcount is 2. But
when prefix-B part of VPN-B is deleted , refcount was not decremented.
For self-imported routes , when the prefix is deleted , refcount is
decremented.

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

l3vpn app data clean up on vm delete.

L3VPN application data is not cleaned up on vm delete and
dissociate network/router scenarios. This review fixes the issue.

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

ACL: use transaction manager for write transactions

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

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

update thrift version in feature.xml
from 0.9.1 to 0.9.3 as BgpConfigurationManager is dependent on 0.9.3

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

IPv6 BGPVPN DC to internet detailed description

This blueprint spec change introduces a new solution that relies on
external network from neutron router. The spec analyses the problem and
gives a solution.

Change-Id: I30c90380fa9b3cf2d8a7705090fa57b7d6043608
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug-9029: Bgp Manager / End of Rib feature complements

This commit change brings some complement of changes in the generated
thrift files.

Change-Id: I13e8c066039b9c27452e09915fbc518f62ef681f
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Remove security group mode config knob

Stateful is the only security group mode we have now.

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

Optimize NAPT switch selection logic

NaptSwitch selection logic today sorts switches in descending order of
weight and then uses iterator to find last entry. By changing the
comparator and convreting it into ascending order, we can return the first
entry without needing to iterate through list of switches.

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

Refactoring and Duplicate Id detection

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

Remove illegal characters from yangs

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

NETVIRT-1063 SNAT flows fails to install sporadically(conntrack SNAT)
The vpind now retrieved using the subnet uuid from the router.

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

Use NwConstants in ...StatefulIPv6.xtend

This should avoid regressions once we start using the Xtext
representation of IPv6 stateful flows entry objects.

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

Drop obsolete “as short” .xtend conversions

With the switch to constants, many “as short” conversions are no
longer necessary.

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

NETVIRT-1074: DNAT traffic is not working if SNAT is Disabled

Problem Description:
======================
FIP to External Network or FIP to FIP traffic is not working if SNAT is
disabled for the router. Since L3_FIB_TABLE (21) -> PSNAT_TABLE(26) flow
is missing on the FIP associated switch.

Solution:
=========
If SNAT is disabled, we need to take care of DNAT (FIP) traffic should
work as expected. By default when router-gw is set with external
network/subnet SNAT is enabled and default NAT flow on Table 21 will be
getting programmed. In case of SNAT is disabled this flow won't get
installed on the required FIP configured switches. Hence FIP traffic is
getting dropped at Table 21. Have added the logic to program the NAT
default flow in Table 21 if SNAT is disabled UC as well.

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

Setting BGP default log to debugging

BGP default log was set to error and due to which qbgp logs are not
shown. making it to debugging now.

Change-Id: I66826d34ce2cb6448261b6c9287fac63eacdcda0
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

Switch some stragglers to odlparent 3.0.2

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

NETVIRT-984: Fix of issue that l2gw connection creation failed in L2GW HA environment.

Issue:
  Exception occurred in l2gw connection creation and caused OVSDB
  transaction is not executed to the end in L2GW HA environment.

Cause:
  "replication-mode" is not set in HWVTEP logical-switches entry in
  network topology config DS. And this caused the
  NoSuchElementException in l2gw connection creation.

Fix:
  Fix this by set replication-mode in the above entry.

Change-Id: I022e493abdbd20d2279540be790bf018c8dc2032
Signed-off-by: kaoru sueda <k-sueda@zj.nes.jp.nec.com>

NETVIRT-1044  fix for Exception in karaf when delete neutron port

Null check for port's dpId is missing in this particular method whereas
all other methods having null check for dpId in AbstracAclServiceImpl

Change-Id: I8c4f5768af2a04b7921915264b6f5089acb9a7bb
Signed-off-by: VinothB <vinothb@hcl.com>

Bug 8998 - Vlan Binding missing on connection add

missing child entries when connection is deleted and added immediately.

Since we are using batch manager previous delete
is still not finished by the time next add request comes.

Since the data is not deleted yet this add request will become no op.

delete from batch manager eventually succeeds and we end up entry missing.

make sure that add request does not proceed until the previous delete
request has not completed.

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

vpnmanager: subnetopdataentry context creation

Update Adjacencies to BGPVPN Internet, if the subnetmap handles refers
to an IPv6 Subnet.

Enhance the subnet-show command to display all the subnetmaps and
subnetmapopdataentries.

Change-Id: I7efba2cc94e65c1862aee96e3e3591a3d14bc630
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Signed-off-by: Sam Hague <shague@redhat.com>

neutronvpn: subnetmap update for internet-vpn-id support

Add internet-vpn-id leaf in the subnetmap list. This model change was
needed to provide a capability to associate an external network with L3
BGPVPN.

Update internet-vpn-id field in subnetmap before associating provider
neutron network with L3BGP VPN.

Add an enumerate in vpn instance op data entry, to qualify the
kind of BGPVPN: BGPVPN external, or BGPVPN internet.

Change-Id: Icab050c89a25beb12b5be3a586d84ee97a7eb593
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Signed-off-by: Noel de Prandières <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

neutronvpn: unlock multiple vpn configuration per iface

This change unlocks the ability of neutronvpn to attach a vpn
interface to two different vpns, or one router and a L3VPN, for
instance.

This is here the case for an internetVPN that is attached to a VPN
interface, in addition to the router.

Change-Id: I7c65b3ce981db0072ba2ab23323d44f2cf689f21
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Add aaa-cli dependency for the distribution

We need to *guarantee* that the aaa-cli JAR is available for
maven-dependency-plugin to copy it correctly; adding the dependency
explicitly ensures that the JAR will be in the local repository.

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

Include AClStatefulTest

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

Bug 8400 - ACL changes doesn't affect the existing connections

Problem:
    Change in SG rules does not affect the existing traffic

Fix provided:
    * "ct_mark" flag of conntrack module will be used to identify the
change made in acl rules of specific instance.
    * When a Acl rule removed from the instance, Netvirt will reset the
value of "ct_mark" flag value. Reset of ct_mark value will skip the further
packets of existing traffic to not be matched by conntrack flow and these
packets will be dropped by the DROP flow.

Change-Id: If55070a7810b580af1cb922abde2880cf434b371
Signed-off-by: vinothb <vinothb@hcl.com>

Pass MetricProvider to JC Test Module

This requires Ie90562c2f8205bab8817d370d077def9a3419d52 and
I530d20946a80933b05371c5220def447be1ee5e6 in infrautils in order to be
able to compile, and must be merged together (with the 2nd, the first
can go in before).

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

Conntrack-based SNAT support for GRE/VXLAN Type

Description : Provided changes for supporting External-Network
Provider-Type of GRE and VXLAN using Conntrack-based SNAT.

Change-Id: I7b1115173f765159ff4d0538c0c6e04f82932f90
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Faster DC-GW Failure Detection (L3VPN with BGP)

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

Add odl-jolokia to the NetVirt distribution

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

Fix getRenderedServicePath sfc test hanging

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

add aaa-cli-jar to distribution/karaf

see Ie47ea696190c9fad6472375007b2767db1f0fbed
and Ifdb3e87cee4cb80f0c753ead620077cbf98af0f8

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

Ignore hanging UT: getRenderedServicePath

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

OPNFLWPLUG-972 : Point to openflowplugin liblldp

Migrated liblldp from controller to openflowplugin

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

Re-enable SFC

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

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

This reverts commit 69ab0b21fe50db21443616a2a0e6bcd5e4806c11.

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

Revert "Add hardware offload support for openvswitch"

This reverts commit 29d3ba79843ef4dfdd1461f5c12c071bde1cbd8f.

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

Add hardware offload support for openvswitch

Handle 'direct' vnic type with 'switchdev' (hw offload) capability
ports in NeutronPortChangeListner.

Change-Id: I0d2a5aa32b9f909d5cf56a1c220ca2cdcbc80071
Signed-off-by: Moshe Levi <moshele@mellanox.com>
Co-Authored-By: Edan David <edandavi@gmail.com>
Depends-On: I65644f32551124c61e30c02831ac4193d12471d2