Bug: 7483 Implement IPv6 component tests in ACLService

This is based on and inspired by the similar IPv4 tests

Change-Id: I2a5416992d178c8bfd99c5822005462f0536e44f
Signed-off-by: Janki <jchhatba@redhat.com>

Fix potential NPEs in ELAN tunnel handling

We encountered null ElanInfos in CSIT after deletion
of all the interfaces via REST in BFD Monitoring suite,
this should not prevent other elanDpns from processing.

Change-Id: I465b62cb5133e6aea5326eb4219fc82a70eab70e
Signed-off-by: Alon Kochba <alonko@hpe.com>

adding lport tag for temporary mac learning

* loading lport tag into NxmNxReg1 while binding elan service
to the interface.
* adding NxmNxReg1 match for table 49 using relearn action

Change-Id: I2914c08157bf33fdb033b7656f1692b6be3ea690
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Improve log messages.

* Correct wrong messages.
* Change the log level of some messages into the appropriate level.

Change-Id: I3e238611ebc9c9fc42aa800caa2a44621880c9aa
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

BUG 7717 Fix OOM when defining large number of networks

Use NodeId instead of Node as key for the cache
ProviderNetworkManagerImpl::nodeToProviderMapping
The size bridge Node object is huge when there are
large number of network/subnets on the bridge.

Change-Id: I6956318477dfa524ce20680d12bba3c7818540ec
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Update netvirt guide - correct DB DROP procedure

Change-Id: I99d19c118d05ade5a9c77f6a98f75abb1b8c7ddc
Signed-off-by: Alon Kochba <alonko@hpe.com>

Bug 7790 - Attempting to install RNH on local DPN for FIB with custom instructions

On VxLAN network there is no severe side affect other than trashing the log
For VLAN/flat networks this is a major issue since VrfEntryListener is not able to
find any local DPN hence all FIB entries are considered RNH and the actual local
FIB entry created using FibRpcService.createFibEntry() will be overridden by the
faulty RNH on the local DPN.

Change-Id: If703d71a6d92c1fb5bff2d0c5fec28f3519649c9
Signed-off-by: Tali <tali.ben-meir@hpe.com>

ECMP: L3VPN Yang changes and extra route rd selection.

* This change contains selection of rd for an extra route from
  the list of configred rds. A new rd is allocated for an extra
  route if the new extra route is behind a different CSS.
* The selected rd for an extra route is advertised to quagga
* Vpn-to-extraroute container is modifed to include vrf-id
* New yang container is added for storing rds used for an
  extra-route

Change-Id: I4d427d8308d047a9667cd4247e53742df473b53a
Signed-off-by: eceghkl <manu.b@ericsson.com>

Fix BFD regression

When tunnel and elan-interface goes up in parallel, there is a race in writing
DMAC flows. The result could be DMAC flow with drop action for the affected interface

Change-Id: I080e222025d59f0b482cd6c07563c168646afc76
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Use Objects equals instead of == where necessary

This might actually fix bugs and change behavior,
there were places where Java objects were compared
using == instead of equals(), this patch corrects
those cases.

Change-Id: I6996e168966fb660d62cca3e4c81635a6c1f6b06
Signed-off-by: Alon Kochba <alonko@hpe.com>

Bug 7775: Using DJC for NAT Interface-state Listeners

Description : With mdsalutil is having a THREAD_POOL_SIZE=1, all the
add(), update(), and remove() methods handling the application logic were
on the same thread. This in turn was delaying all the interface-state
listeners.

Changes are done to make use of DataJobCordinator so that all the NAT
application logic get executed by a seperate thread so that the main
thread is freed-up for other listener execution.

Change-Id: I0c7f3f70715b01a2dcb54d251a36a2f9ff21357e
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Spec for VNI based L2 switching, L3 forwarding and NATing

1. Added specification document for VNI based L2 switching, L3
forwarding and NATing
2. Linked spec to specs/index

This spec is a first step in the direction of enforcing datapath
semantics that uses tenant supplied VNI values on VXLAN Type networks
created by tenants in Openstack Neutron.

Earlier the datapath semantics for VXLAN traffic from tenants were
realized by using either LportTags (or) MPLS Labels being carried as
VNI values, even though the networks created by the tenant were of
type VXLAN.

Please note that implementation of Spec DOES NOT COMPLETELY remove
usage of LPortTags (or) ElanTags (or) MPLS Labels.

It retains the use of MPLSLabels for realizing an L3 BGPVPN
(regardless of type of networks put into such BGPVPN that may include
networks of type VXLAN) both on the wire and within the OVS Datapaths.

It also retains the use of LPortTags within the OVS Datapath of a
hypervisor, for streaming traffic to the right virtual endpoint on that
hypervisor (note: not on the wire).

It retains the use of ELANTags within the OVS Datapath to handle
local/remote L2 broadcasts (note: not on the wire)

On the cloud data network wire, all the VXLAN traffic for basic L2
Switching within a VXLAN network and L3 Forwarding across VXLAN-type
networks using Routers will use tenant supplied VNI values for such
VXLAN networks.

Complete removal of use of LPortTags and MPLS Labels for VXLAN-type
networks has large scale design/pipeline implications and thus need to
be attempted as future initiatives via respective specs.

Change-Id: I708243b75fb83ce94064b7214af2709ee82914f4
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Creating vpn-instance for external subnets for flat/vlan.

This is the first task for the Hairpinning issue.
Added new model to odl-nat for external-subnets.
For each subnet of flat/vlan external network will create vpn-instance.

Change-Id: Ie3cb34f0746c9a06d0b2fda009fb69da3522c8e6
Signed-off-by: yair <yair.zinger@hpe.com>

BUG 7729: Remove redundant tunnel drop flow in table 110

- Remove methods: L2ForwardingService::programLocalTableMiss
  and L2ForwardingService::programTunnelMiss. Both methods
  generate the same tunnel drop flows in table 110.
  The drop flows for individual tunnels are not needed
  since there is a default drop flow in the same table.

- Remove the method L2ForwardingService::programVlanFloodOut
  which is not used.

Change-Id: I0fb826094b6c03e58fb89c95600e7b84bc2c774f
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>
(cherry picked from commit a3a91c1c01d294a8e36662284c4cc82a2749c2e1)

Spec to support IPv6 North-South support for Flat/VLAN Provider Network

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

Fix regression in auto tunnels - listen on Interface

This patch https://git.opendaylight.org/gerrit/49908
introduced a behavior change in how parentRefs are used.
Refactor the auto-tunnel behavior to listen on Interface
from config instead of Interface State, to take into
account any change in parentRefs and in any order.

Change-Id: Iee459634fab97eba5f76dbe7ae70caa00a48f3b4
Signed-off-by: Alon Kochba <alonko@hpe.com>

Checkstyle: fix new ParenPad violations

Enforcing no space(s) after a left parenthesis and before a right
parenthesis is a proposed change [0] which can only be merged when all
violations are fixed in projects enforcing checkstyle. This patch fixes
new violations since [1] was merged.

[0] https://git.opendaylight.org/gerrit/#/c/51316/
[1] https://git.opendaylight.org/gerrit/#/c/51414/

Change-Id: Id68cac28db2911f552fd9951a027bbcfd99faf76
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Enhancement to support list of rds for l3vpn

* This review contains enhancement to support list of rds while
    creating l3vpn instances.
* Primary rd is obtained from the list of rds which will be used
  as route distinguisher for primary adjacencies.

Change-Id: Ib8d0dd41712c734b5868c5dc87ee0e54e547ff34
Signed-off-by: eceghkl <manu.b@ericsson.com>

Fixing checkstyle changes in BgpManager.

After the checkstyle fixes, if stale path is not passed NPE is thrown.
Fixed that by adding a variable which defaults to 0 if stale path is not
provided.

Change-Id: Ia84ec51913dcb2465b9fa408b1a37be08ea33b3c
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

BUG7748: Subnet-op-data empty after cluster reboot

SubnetAddToVpn notification is fired from add() of SubnetMapListener
also instead of firing from update() only.The reason for doing this is
after reboot this event comes in subnetmap add().

Change-Id: I766e3836cbe8b0eafbf5fbb52791ca96eb6ccfa9
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Bug 7680: Fix Nexthop when advertising to DCGW

 + for static routes whose nexthop is a VM, the leaked route to the other
   VPN was being advertised to DCGW with the DPN where VM is located as
   the nexthop of the route. However, the nexthop must be the DPN where
   the InterVpnLink has been placed.

Change-Id: I1097e9c992c66053f499807fd411334a7113b509
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Fixed Netvirt Build Breakage

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

bug 7599  hwvtep ucast mac add performance improv

Introduced new listners to listn at subtree of Node
Listens for the node children data updates and propagates the updates
between ha child and parent nodes.
When an operational child node data is updated, it is copied to parent
When a config parent node data is updated , it is copied to all its
children.
Introduced Manager listners , which execute HA operations when non-ha node
becomes HA node.

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

Ethernet VPN Enhancement in BGP and FIB APIs

This review brings in API changes in FIB and BGP manager to support BGP
Ethernet VPN feature in OpenDaylight. This is the fifth in a series of
reviews with 'EVPN_RT5' as the topic.

This review brings in the following things :
1) FIB manager API changes to supply additional parameters for VPN to BGP,
when an L3VPN over VxLAN is created.
2) BGP manager API changes to confirm to thrift API changes and to supply
routes to FIB when BGP gets routes belonging to IP-VRF table of an EVPN.

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

Bug 7680: leaked routes not advertised to DC-GW

 + It's been found that the routes that are leaked to DC-GW
   at InterVpnLink creation time (well, at InterVpnLink added
   callback) are correctly advertised to DC-GW. However, the
   routes that are leaked after the InterVpnLink has been
   created are not being advertised to DC-GW.

 + These second type of leaked routes are handled on fibmanager,
   but fibmanager does not have access to BgpManager. So the
   solution consists in moving part of the logic from fibmanager
   to vpnmanager where the BgpManager is accessible.

 + InterVpnLinkService class, which was just an internal class in
   vpnmanager-impl, is renamed to InterVpnLinkLocator whose only
   responsibility is to search for the most suitable DPNs to
   place new InterVpnLinks. The name change is to avoid confusion
   with IVpnLinkServiceImpl which is the implementation of the
   public interface 'IVpnLinkServce'.

 + Besides the renaming of InterVpnLinkService, most of its code
   is moved to IVpnLinkServiceImpl, and that's why this change
   may llok big.

 + Note: there is a <wip> [1] for taking the route leaking out
   of the fibmanager completely.

 [1] https://git.opendaylight.org/gerrit/#/c/48599/

Change-Id: I1a7311345307716ef223c707b22c6ecec66d97a2
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Use OVSDB external-id for Neutron port parent ref

This patch removes the logic when a neutron port is added,
to link it to the OVSDB termination point via parentRef.
Filling in this parentRef will be handled by Genius in both orders.

We no longer do the previous behavior of trying to guess the parentRef
by the VIF type - the parentRef will be left empty until such a
termination point is added.

Depends-On: https://git.opendaylight.org/gerrit/49906
Depends-On: https://git.opendaylight.org/gerrit/49907
Change-Id: I86f1f964c363af982c2460d40ef6b7afb7f56e59
Signed-off-by: Alon Kochba <alonko@hpe.com>

Add project info to name in pom files

Refer to for details on this activity:
https://trello.com/c/EGYoCYXX/11-automated-autorelease-build-failure-notification

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

Bug 7544: L2 control packets that should not be forwarded are forwarded

Currently L2 service is flooding L2 control protocols, which shouldn't be forwarded.
This change installs drop rules for them.

Depends-On: https://git.opendaylight.org/gerrit/#/c/50489
Change-Id: Ifb363f7a641b25ec90e15b649ee0dc720ffd1dc4
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

bug 7599 added l2gw validate cli

Added a helper cli to validate various config and operational
data models.
Checks if the config and operational datastore differ
Checks if the ha parent node and child nodes differ
Checks if the logical switches/ mcast macs and vlan bindings
are created for the configured l2gw connection

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

Revert "bug 7538 fix broken netvirt build"

This reverts commit 76e234c0c44b5449dd0aef8f9cffc195fff0ee3b.

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

Ethernet VPN Enhancement to Vpn Engine and FIB.

This review brings in code changes in VPN Engine and FIB to support
BGP Ethernet VPN feature in OpenDaylight. This is the third review
with 'EVPN_RT5' topic branch.

This review brings in the following :
1) Code changes to add FIB entries and advertising routes to BGP,
appropriate to the type of VPN being processed.

Change-Id: I92ec69843151b93720a53c17436878707aa0a2b4
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 7372 - Supress error of NAPT switch selection failure before router-dpn association

NAPT switch selection will take place once the router-dpn association is made
All missing NAPT/non-NAPT flows will be recovered as the result of selection

This review fixes only the log levels.
There can still be some missing NAPT flows in this scenario
The resolution for this is still WIP and covered by
https://bugs.opendaylight.org/show_bug.cgi?id=6866

Change-Id: I494cca69aceb8cc4d564e9bc4d1419894a224de6
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Setup SMAC on routed packets destined to virtual endpoints

All L3 Routed packets destined to virtual endpoints in the datacenter
managed by ODL do not carry a proper source-mac address in such frames
put out to virtual endpoints. With the fix all the packets destined to
virtual endpoints shall have gateway mac as source mac. If gateway mac
is not recoverable for the subnet on which the virtual endpoint resides,
then the connected mac address of that virtual endpoint will be used as
the source mac address.  Here the connected mac address represents the
mac-address on the tapxxx (or) the vhuxxx port on OVS Datapath for that
virtual endpoint.

Use VrfEntry yang model to represent the source mac.

Sample Group Entry:
group_id=150001,type=all,bucket=actions=set_field:fa:16:3e:a1:4b:61->eth_src,set_field:fa:16:3e:21:b8:d3->eth_dst,load:0x300->NXM_NX_REG6[],resubmit(,220)

Spec Reference link:
https://git.opendaylight.org/gerrit/#/c/49807

Change-Id: I2e75215fd75f051135d702ab63cb792b6cf4cb55
Signed-off-by: karthik.prasad <karthik.p@altencalsoftlabs.com>
Signed-off-by: Achuth Maniyedath <achuth.m@altencalsoftlabs.com>

bug 6589 l2gw cluster reboot fixes

registerd the group listener
created the l2gw cache upfront
fixed npe in terminationPoint command

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

Checkstyle: fix ParenPad violations

Enforcing no space(s) after a left parenthesis and before a right
parenthesis is a proposed change [0] which can only be merged when all
violations are fixed in projects enforcing checkstyle. This patch fixes
those violations.

[0] https://git.opendaylight.org/gerrit/#/c/51316/

Change-Id: I8266ea1698dae071b1ce42128f534eb2ff5e266e
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

NeutronVPN: Error out if VxLAN/VLAN network configured without seg-id

This adds an error log based on the presence of segmentation-id when VxLAN
or VLAN based neutron networks are configured via Neutron.

Additionally, some cleanup done:
1. Moved some impl specific utils to NeutronVpnUtils
2. Moved constants to NeutronConstants
3. Renamed SubnetMapListener class to SubnetMapChangeListener
4. Removed  unused xml dependency from QosListener

Change-Id: I2d22773e4e7bdfa927c01d679542fa36146e625a
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Bug 7714 - Vpn Interface not deleted from oper DS

sometimes Vpn Interface not deleted from operational/l3vpn:vpn-interfaces model,
if there is more than one adjacency to this vpn interface.

Change-Id: I45e4ca20db6c5aa9f5c280fea8726a5ed8aa654b
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

Checkstyle: fix vpnmanager-shell and enforce

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

bug 7538 fix broken netvirt build

using the same thread executor

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

Bug 7669: Add multi-provider network support to NetVirt for L2 Gateway.

Multi-provider network support is needed to be able to support
connecting VLAN based networks to VXLAN based networks,
using L2GW/HWVTEP.

Slides describing the network, and use cases, can be
found here:

https://docs.google.com/presentation/d/13OqyCHwGQABlcno9lVVY2KHVZdNMYz3JiiTRrQj-n98/edit?usp=sharing

Testing
-------
Setup and successfully tested use cases 1-3, as described in
slides above.

Note: There is a corresponding Nova bug that must be fixed
for this to work. Basically, Nova needs to add support
for multi-provider networks. This work is in progress.

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

Clean-up: simplify some if statements

Generally speaking, reduce

    if (condition) { return true; } else { return false; }

to

    return condition;

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

Replace mockito-all by mockito-core (see Bug 7662)

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

Bug 7720: create/delete VPN CLI handling addition/removal of subnets

Following changes have been done as part of the bug fix:

1. Addition of subnets to VPN done via associateNetworks RPC
2. Deletion of subnets from VPN done via dissociateNetworks RPC
3. (1) is done only if createVPN succeeds
4. Deletion of VPN is done only if (2) succeeds

Change-Id: I21d25afa6b3ba56dae56abe2536afcfe5f0476b4
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

spec for providing ECMP over BGP VPN

Change-Id: I08b3376b29882d00efd27be3cd4887a3c8b8e349
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Remove unused utils.config from neutronvpn

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

Remove unused utils.config from aclservice

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

set it version to match other netvirt versions

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

Use binding-parent rather than config-parent for api bundles

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

Pull checkstyle enforcement from parent poms

vpnmanager-shell defines the severity at warning because it has
not been updated for checkstyle yet. This will be removed once
it is ready.

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

Remove duplicate lockmanager-api in pom

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

Spec for Conntrack Based SNAT in netvirt

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

Ethernet VPN Enhancement to NeutronVpn.

This review brings in code changes in neutronvpn to support
BGP L3VPN support over VxLAN in OpenDaylight. This is the second review
with 'EVPN_RT5' as the topic.

This review bring in the following things :
1) Code changes to service instance and interface creation for L3VPN over
VxLAN in addition to the existing L3VPN over MPLSoGRE.
2) Minor yang change to use a generic VpnInstance data-model in
NeutronVpn, when handling vpn instances through RPC

Change-Id: Ic46cfb8886eb429172151f8f0239a0166468e3ca
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>

Thrift interface changes to support BGP EVPN.

Thrift interface changes to include information on
protocol type, encapsulation type and the layer type
in the method APIs is incorporated. APIs now include
arguments to support ethernet specific arguments
relevant to evpn feature. OnUpdatePushRoute() and
OnUpdateWithdrawRoute() also changed to support above
mentioned arguments.

Change-Id: I07ce11319dd57de98dfd0776f11628e03e55209a
Signed-off-by: Sumanth MS <sumanth.ms@ericsson.com>

Fixed compilation issues related to "throws Exception".

Compilation issues due to https://git.opendaylight.org/gerrit/#/c/51073/

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

Yang Model Changes for Ethernet VPN.

This review brings in all the yang model changes required to support BGP
L3VPN support over VxLAN in OpenDaylight. This is a first in a series of
review that will have 'EVPN_RT5' as the topic.

This review has yang model changes in the following modules:
1) NeutronVpn
2) Vpn
3) FIB
4) Model-BGP

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

ACL: Operation Improvements

Updated ACL to program "+trk+new" and "+trk+inv" DROP flows with lport tag
as metadata for each of the VM (Neutron port) being added to support drop
packet stats at port level.

Spec reference: https://git.opendaylight.org/gerrit/#/c/49086/

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

Checkstyle: fix vpnmanager-api and enforce

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

Checkstyle: fix dhcpservice-api and enforce

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

Checkstyle: fix bgpmanager-api and enforce

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

Bug 7674 Update of DHCP flag support for subnet-update configuration

Change-Id: Ibe318e207dfeb4ea95ac99df733fd31c0f8d1dd6
Signed-off-by: Vijayalakshmi Chickkamenahalli Nagaraju <vijayalakshmi.c@altencalsoftlabs.com>

bgpmanager Checkstyle, enforcing

Passing the PrintStream to Commands is rather inelegant but it feels
like the most straightforward fix.

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

ACL: Support for statistical counters

This feature provides statistics related to drop packets due to ACL for
the specified ports.

+ Provided new RPC: acl-live-statistics:get-acl-port-statistics
+ Uses OFplugin's RPC (opendaylight-direct-statistics:get-flow-statistics)
to get flow statistics.
+ This is currently supported for ACL stateful mode only.

Spec reference:
https://git.opendaylight.org/gerrit/#/c/49086/

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

BgpCounters re-write

This reworks BgpCounters using slightly more idiomatic approaches
(without re-designing it though).

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

de-static-ify Timer field in ElanL2GatewayUtils

I don't see any reason why this needs to be static?

On the contrary, it will only lead to problems in tests.

Also introducing close() and Timer cancel() & purge(), just to be safe.

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

Bug 7601: Cleanup Elan instances when a network is deleted

The following patch in Neutron[1] modifies the Network delete sequence.
Prior to a network delete event, we now see an update event that removes
the networkType. So, in delete event, when we try to check for the
networkType, it would be empty and there is an exception. Because of this,
elanInstances are not getting deleted. In this patch, we avoid the check
for networkType since the network itself would not be created in the first
place if its not a supported networkType.

[1] https://review.openstack.org/#/c/317358/

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

BUG 7488 Add option to disable auto bridge creation

This adds a new param to elan-config, auto-create-bridge that defaults to
true. When false, netvirt will neither try to create br-int if it doesn't
exist, nor add any configuration if it exists. This is to be used in
scenarios where br-int is already created and configured.

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

Bug 7667: SNAT table 46&44 are not getting programmed when private BGPVPN

Problem Description:
=====================
SNAT: Table 46(OUTBOUND_NAPT) and table 44 (INBOUND_NAPT) are not getting
programmed (SNAT Forward Traffic and SNAT Reverse Traffic) with NAT
translation when router is
associated with Private/Internal BGP-VPN.Due to this issue, SNAT traffic
is getting failed when interval bgp-vpn is configured.

Root Cause of the Problem:
==========================
As part of bug (Bug 7478) fix SNAT traffic is used to send router-gw mac
address instead of VM mac. With this code changes router gw-mac was
getting from external-router Yang Model using router name as key.

When private/internal bgp-vpn is configured router-id (Internal Private
VPN ID) should be replaced with created private/internal bgp-vpn id. With
this approach when querying external-router Yang model for getting
router-gw mac address using new router-id (bgp-vpn id) will not match with
router-name.

Solution:
===========
When querying external-router yang model for getting router-gw mac address
it should use old router-id instead of bgp-vpn id. For doing this change
in NAPTEventHandler re-aligned the router-gw mac address code to
appropriate place to be working as expected.

Change-Id: Ieefd799d6b2349464bfb737e3fcc8101036c3619
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Bug 7700: create-l3vpn (REST/CLI) should not allow another VPN to use the same VPNID

Change-Id: I4481db51b17e92b2df7afab18b41bcb4e1a23df0
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

System Configurable Parameters for EVPN

The following parameters have been initially made available as
configurable for EVPN.These configurations can be made via the
RESTful interface:

1> Multi-homing-mode
2> Irb-mode

Change-Id: I09b55b1e021741d46f2d9bb1d2a0877cbbc81572
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

String handling cleanup

* Remove redundant toString() calls
* Remove explicit String constructor calls
* Use string concatenation where appropriate

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

Cleanup: replace Arrays.asList() where appropriate

... with Collections.singletonList (when a single argument is given).

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

Boxing cleanup

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

General clean-ups

Automated refactors:
* use List::sort instead of Collections::sort
* use foreach loops
* use switch statements instead of if/else if constructs
* use contains() instead of comparing indexOf()
* use Collection::removeIf instead of looping explicitly
* use Optional::ifPresent where possible
* use Map::computeIfAbsent instead of conditional puts
* use try with resources

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

Clean up lambdas

Use lambdas instead of functional interfaces, and use method
references instead of lambdas where possible.

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

Remove unnecessary generic types

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

Use Java declarations instead of Google Collections

Where appropriate, replace:
* Maps.newHashMap() with new HashMap<>()
* Lists.newArrayList() with new ArrayList<>() or
  Collections.singletonList()
* Sets.newHashSet() with new HashSet<>()
* Sets.difference() with explicit .removeAll() constructs (usually
  avoiding some collection instantiation too)
* Lists.newLinkedList() with new LinkedList<>()
* Maps.newConcurrentMap() with new ConcurrentHashMap<>()

Also
* replace some Guava filters/transforms with Java 8 streams
* replace some Guava predicates with Java 8 predicates
* remove unnecessary generic type qualifiers in affected classes
* clean up some Optional uses (.orNull(), direct Optional returns where
  appropriate)
* remove some String() constructors

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

Add missing UDP matches for SNAT

Fix regression introduced by MatchInfo refactoring,
part of https://git.opendaylight.org/gerrit/#/c/50518

Related fix for null MatchInfo here:
https://git.opendaylight.org/gerrit/#/c/51182/

Change-Id: I26aaf01648cbb27b703eb51b569be957803dc9df
Signed-off-by: Alon Kochba <alonko@hpe.com>

Fix and re-enable aclservice tests

This requires https://git.opendaylight.org/gerrit/51156 in Genius.

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

BUG7308: fix leaf to leaf traffic

Change-Id: Iabae4f9fb7a2665f41f705ad0a6516a2d96ec77b
Signed-off-by: David Goldberg <gdavid@hpe.com>

BUG-7461

Align Legacy Netvirt with New Netvirt's design of not
removing integration bridge from the config DS.

Change-Id: I9f5da8344747f854828cddedd8c91f633b7b666f
Signed-off-by: Bertrand <bertrand.low@hcl.com>

BUG 7591: Allow configuration of inactivity_probe and max_backoff for OVS

Depends-On: https://git.opendaylight.org/gerrit/50643
Change-Id: I7881934df38c32cd40e156944c3ee46804d5a4e4
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Spec for NetVirt QoS

This spec file describes the changes to be made
in the Carbon release to complete the implementation
of Neutron QoS API support for NetVirt.

QoS Policies with Bandwdith Limiting Rule support was
added in Boron.  Carbon will add support for the DSCP
marking rule and for reporting supported QoS rules
back to Neutron.

Change-Id: I7e10b702fbbe9eb214336f500c00a579cb441e37
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

BUG 7489: Add startup config file for elanmanager-config

This implements issue 2 mentioned in BUG 7489 by adding a new
configuration file in which user can specify default values for
elan-config at startup.

Note: Update-Strategy is set to None which means any runtime changes to
config will not be reflect till next restart

Path: etc/opendaylight/datastore/initial/config/
Config file name: netvirt-elanmanager-config.xml

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

Lower debug level when truncating provider port name

This print shows up a lot in the log, as it is
called on multiple events, whenever the configured
provider port name is longer than 15 chars.
Lowering debug level as it is mostly an internal
translation.

Change-Id: Ic317c96c66cd876e2fd3628b4985ea0abb6d5152
Signed-off-by: Alon Kochba <alonko@hpe.com>

Fix ElanStatusMonitorJMX failing upon bundle reinitialization.

Change-Id: I9e04e3ee1b9edc6000439e18b35c729f9ec0f92b
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Enable checkstyle for fibmanager

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

Enable checkstyle for natservice

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

Enable checkstyle for neutronvpn api and shell

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

Disable aclservice tests until fix provided for matchinfo

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

Enable checkstyle for neutronvpn

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

Handle the Genius InstructionInfo clean-up

I'd missed a couple of changes... One of the tests is disabled for
now, I'll fix it later.

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

SNAT tests failures

The rule with empty action list was installed to T47

Change-Id: I8fb81cafb2a7bfd392ce24b73dedf14ab29608bb
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

Fix vpnmanager-impl Checkstyle issues and enforce

This switches Checkstyle to enforcing mode on vpnmanager-impl, and
fixes all the issues apart from exception handling (which I left alone
since it would change the code paths). This includes:

* ordering imports
* removing unused imports
* formatting code (whitespace, line lengths...)

I fixed up some SLF4J uses when it helped with the line wrapping. I've
also fixed up some exceptions and migrated some MD-SAL code to
SingleTransactionDataBroker.

There's also some cleanup in IBgpManager to allow simpler exception
handling in vpnmanager-impl.

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

MatchInfo redesign

This uses the new MatchInfo subclasses and drops all references to
MatchFieldType.

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

InstructionInfo redesign

This removes all uses of InstructionType in favour of the new
InstructionInfo subclasses.

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

Bug 7660 Infinite loop while vpn instance removal

Sometimes VpnInstanceListener gets into endless loop because not all vpn
interfaces removed successfully.

Analysis:
The current interface counter value inside the function waitForOpRemoval()
can be less that original counter, but still not zero, then we are back to
beginning of the cycle without any action and sleep again.
The infinite loop doesn't allow to DataStoreJobCoordinator to run any
other job within same key.

Change-Id: I31ce3d38fa49212b7736888664a2d597d09f9666
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

Bug 7384: CSIT Exception: NPE in deleteVpnInterface

getNeutronPort is returning null, causing port.getUUid() to result
in an NPE.
Further analysis necessary as to how it happened in the first place
since port delete event for the port in concern hasn't come.
The introduced check should ensure avoiding this NPE for now.

Change-Id: Ia4298d3b9f2b57b1a94805bca3ae34dffa2df4fe
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Bug 7532 - arp responder rule sometimes missing after vm reboot

VRF entry is not removed during the port down because of wrong value (null string) in the next hop address list.

Change-Id: Ib0a0a84358cce116bdb9c4768ff9d9608f5c638b
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>

[BUG 7543] Replace the request function used by restangular

Restangular as two methods to fetch data: get and getList.
Those two methods was doing the same things, but it seems
that getList now expect a List of JSON and not an object
who caused the application to crash.

Change-Id: I4aa51860cd03f3f6a04d2322518c7c9c7a1b3092
Signed-off-by: Maxime Millette-Coulombe <mmcoulombe@inocybe.com>

Bug 7436: Handle VpnInterfaces of VpnInstance

The problem:
VpnOpDataSyncer.waitForVpnDataReady is using java wait method
that does not return the thread to the pool while waiting.
If there are more compute nodes then CPU threads, all threads
in DjC will wait and there will be not availabe thread to
execute the code which waitForVpnDataReady waits for.

The solution:
VpnInterfaceManager holds all vpn interfaces that cannot be processed till the
relevant vpn instance in ready.
Updated the synchronized lock blocks to user vpnInstanceName instead of this.

Added cleanup of removed interface

Change-Id: I66da144356e17e778f1d04e5720befbece565fd5
Signed-off-by: Tomer <tomer.pearl@hpe.com>