Bug 8137: Missing log entries in many places for NAT feature

Fixing the missing log entries for NAT feature. This will not impact any traffic.
Just to ensure in TRACE log to capture all the missing parameters.

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

Bug 8023 adding trace logs

Adding trace logs to find out root cause for not programming remote dmac
and bc group for auto tunnel configuration

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

Bug 7866 fixing remote bc group and mac learning

* Updating remote BC group correctly for VLAN provider networks
* catching exception while parsing ip address from packet ins.
otherwise, it would ignore the packet-in and mac learning doesn't happen

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

reading the bgp_sdnc_mip, sdnc_bgp_mip from custom.properties instead of
platform dependent cluster.conf

Change-Id: I4c1c00c99126fcc9306a880ddedd5db613683fcc
Signed-off-by: Shankar M <shankar.m@ericsson.com>

add missing acl in federation routed rpc

Change-Id: Icc0d58f606345601c069c1ab7687d85fd551432e
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

8121:  Exception in SG Acl while SFC acl is created/modified

   - Updated ACL event listener to act on the event only if it has
     SecurityRuleAttr augmentation

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

Add UTest BgpRouterAddNeighborTest

'BgpRouter.addNeighbor' is no longer a simple wrapper around
'BgpConfigurator.Client.createPeer' so a small UT has been
added to check 'createPeer' and 'setPeerSecret' are called
properly.

Changelog:

- BgpRouter.java: new protected factory method
  'makeTestingRouter'.

- BgpRouterAddNeighborTest.java: new UT class with the
  following tests: 'addNeighbor_noPass_happyCase',
  'addNeighbor_wPass_happyCase',
  'addNeighbor_noPass_createPeerFails',
  'addNeighbor_wPass_createPeerFails', and
  'addNeighbor_wPass_setPeerSecretFails'.

Change-Id: I851aa55a3aba6bd60e29fee3d9d190337317a6f3
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Subnet-Route support for L3VPN Over VxLAN

This review includes the code changes to support the subnet-route feature
for EVPN RT5

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

Refactor InterVpnLink. Part ONE

 + This is the first of a series of commits that intend to
   split this bigger one [1] into smaller so that it's easier
   to review.

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

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

ACLs separate priority pools for allow/deny

To support mixing of Allow and Deny ACEs, we need to
define separate pools, where the higher range is allocated
to the ACEs with behavior matching the default behavior.

Depends-On: https://git.opendaylight.org/gerrit/#/c/53894/
Change-Id: I83ca5a1a57f73971117aa766ae00cc9e22d8d197
Signed-off-by: Alon Kochba <alonko@hpe.com>

Bug 8135: Potential NPE in subnet route packet handler for subnetmap

Check for nulls when handling the subnetmap and some more minor fixes.

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

Fixing the 'error writing to datastore' in
CSIT.

* updated unbindService method
* added DJC in update method for elan-instance

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

Using InternetVPN Vni for Intra-DC communication

Description :

When Enforce Openstack Semantics is enabled, then VNI will be carved
out from the VNI pool per External-VPN and will be used as tun_id for
Intra-DC communication.else label generated per external-ip will be
used as default.

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

Ignore ACL unitests that fail on timing

https://git.opendaylight.org/gerrit/#/c/53002/ introduced sleeps in
unitests, causing netvirt compilation to sporadically fail.
Opened bug to handle properly:
https://bugs.opendaylight.org/show_bug.cgi?id=8136

Change-Id: Id840735142f6f574085c6899088ed32dfbfef891
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

Bug 7633: Karaf 4 net-virt-sfc features

This patch needs the SFC Karaf 4 features.

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

Remove subnetroute notifications, listen directly on snmap events

To make subnetroute engine support populating subnet-op-data at all times,
and not just be specific for non-ext-net BGPVPNs, two subsequent changes
are required.
1. Remove 2 hops to act on snmap events by eliminating notifs.
2. To populate subnet-op-data for all cases via add/remove/update, and
trigger subnetroute only for non-ext-net BGPVPNs.

This review handles part (1) of the problem.

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

Making ELAN to use StateTunnelList listener

* Currently ELAN uses Interface state DCN for tunnel events to program
remote DMAC and BC group when VxLAN tunnel is created later. So making it
to use listen StateTunnelList
* This change would be useful when ITM supports multiple VxLAN tunnels and
avoid race conditions

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

Refactor InterVpnLink. Part TWO

 + This change is independent from the other ones. Basically
   it changes some classes to retrieve data from the Cache instead
   of directly from MD-SAL

 + Removes some methods from InterVpnLinkUtil that are no longer
   needed.

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

Add custom policy ACL matches

Depends-On: https://git.opendaylight.org/gerrit/#/c/54004/

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

Router VNI as tun_id from Non-NAPT to NAPT

Description :

When Enforce Openstack Semantics is enabled, then VNI will be carved
out from the VNI pool per router and will be used as tun_id.
else router_id will be used by default(if Router is over L3VPNOverVXLAN
then RouterLport tag will be used as per EVPN)

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

Bug 7599 -hwvtep ucast mac consumption improvement

Adding DPN cache to make getting invoved DPN's faster instead of reading
from data store in case of each Mac added.

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

Extra routes support for L3VPN Over VxLAN

Code changes to handle extra-routes for L3VPN Over VxLAN (EVPN RT5).
Change-Id: I3b5b8bcea04f1f946225b23f16e146fa16f82d4b
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Plugin provider functionality

- The plugins provide the renderers with the
  necessary functionality to be able to render
- Converted back to Genius MDSALUtil
- Fixed a few issues in ClassifierUpdate and related classes

Change-Id: I85e23b0058a12fd795689791cac35efcffffe125
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Add statistics-config.xml to initial resources

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

Add neutronvpn-config.xml to initial resources

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

Add vpnmanager-config.xml to initial resources

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

Set update-strategy to none for all configs

The default update strategy is to restart the bundles
when any config is changed. No bundles support that
so disable that behavior until it is supported.

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

Implement remote ACL indirection table

Implementation of remote ACL indirection as described in spec:
https://git.opendaylight.org/gerrit/#/c/51462

Change-Id: Ifaccd2bab13c8b16eed2d1a17dd54cfa87f70ee3
Signed-off-by: Slava Radune <slava.radune@hpe.com>

Bug 7922 - Use counter to keep track of duplicate flow entries

Problem: Single flow entry associated with two more security rules
incorrectly deleted when one of the security rule is deleted.

Fix: Use counter to keeps track of duplicate security rules
in a particular VM. The counter is maintained in the flow cookie.
When writing a security flow, the flow counter is incremented
if the flow already exists, otherwise the flow cookie is set to '1'.
When removing a security flow, if the counter value is '1',
the flow is removed from the node, otherwise the flow counter is
decremented.

This changeset also:
- reversed some previous partial fixes for the same problem.
- refactor common Acl Service to AbstractAclService
- reformat/clean up code

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

Send ARP requests for PNFs with unresolved MACs.

In internal VPN scenario, when packets are sent to an PNF with an unresolved MAC,
ARP requests should be sent from a specific switch.

There are two cases:
1. Tenant network - The destination IP belongs to a subnet in a tenant network.
The arp should be sent from the designated switch of the destination subnet.
2. External network - The destination IP belongs to a subnet the external network.
The arp should be sent from the NAPT switch.

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

Egress policy service binding and policy table miss flows

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

Policy routing table flow installation

Route each combination of policy classifier and logical interface
to the corresponding policy classifier group

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

Policy classifier route group installation

this patch depends on
https://git.opendaylight.org/gerrit/#/c/53049/

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

Support federated security groups

Change-Id: I2d365dbff772dc90128b7e76b05382544bd2f26d
Signed-off-by: Guy Sela <guy.sela@hpe.com>

More NxMatchInfo re-design

Some uses of the old-style NxMatchInfo were added after the earlier
patch was completed, this patch migrates them to the reworked
sub-classes.

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

Support for sending Neighbor Solicitation for RouterGWMacAddress resolution

This patch implements the sendNeighborSolicitation rpc which can be triggered
by NeutronSubnetGwMacResolver code to resolve the RouterGWMacAddress. Changes
in NeutronSubnetGwMacResolver to invoke the rpc would be done in a future
patch that implements the necessary code to handle the response (i.e.,
Neighbor Advertisement for DCGW).

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

Fix NPE if underlay network is missing from TEP

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

Support ACL allow and deny behavior, add default behavior config

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

Netvirt: Karaf CLI for Qos Alert patch3

*** patch #3 - Adding karaf CLI for Neutron Qos Alert ***

Added following new karaf CLI

qos:enable-qos-alert <true|false>
qos:drop-packet-threshold <threshold value in %>
qos:alert-poll-interval <polling interval in minutes>

Spec Review Link:
https://git.opendaylight.org/gerrit/50689

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

Bug 8093:Fib entries for external Fixed Ips are not programmed

Problem Statement:
==================
In RouterDpnChangeLister, the extRouter parameter is passed as null and in
NatUtil.getExternalSubnetForRouterExternalIp() is called, router is passed
as null resulting in NullPointerException

Solution:
==========
If the router data is null, we need to get the routername from ConfigDS
and use that router name in NatUtil.getExternalSubnetForRouterExternalIp()
to avoid null pointer exception.

Note: Below code is added for external fixed IP information to be writing
into odl-fib entry for EVPN_RT5 use cases.

writeTx.submit()

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

Netvirt: Config Listener for Qos Alert patch2

*** patch #2 - Adding config listener for
Neutron Qos Alert ***

Supports:
  - Configuration to start/stop monitoring the port statistics
  - Configuration of rx drop count threshold value in %
  - Configuration of polling interval

  - Default or initial configuration available in netvirt-qosalert-config.xml
      * qos-alert-enabled - false
      * qos-drop-packet-threshold - 5%
      * qos-alert-poll-interval - 2 minutes

Spec Review Link:
https://git.opendaylight.org/gerrit/50689

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

Remove addNeighbor(ip,as)

Methods 'addNeighbor(ip,as)' of classes 'BgpRouter',
'BgpConfigurationManager', and 'BgpManager' are no longer
used.

Change-Id: If7e36c822d89827d6383fcf5352240601b22ae5d
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Bug 7633: Karaf 4 openstack features

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

hwvtep federation support

Change-Id: I1e3ef68ca45c9ab3998d2e0402989098c1aca137
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

Add types of classifier entries

On the initial implementation a single ClassifierEntry would support
all the necessary information needed to render a classification for a
specific match on a specific node for a specific input interface. This
setup was proving difficult when suppressing from the dataplane
rendered items that would be shared between different classifications
like for example initialization flows or interface bindings.

With this patch the classifier entry now can be of different types
of smaller informational items that allow for a much more flexible
update southbound.

Change-Id: Ic355c0bf11d7c0c6f7264efb955d1f501bf905fd
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Bug 7778: VM's FIP are not able communicate to each other
in external network provider

Problem: NORMAL instruction support need to be removed
from legacy netvirt

Soultion: Added a separate flow for broadcast, multicast, unicast of
internal to external communication

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

Update egress ACL service to match on lport-tag reg6 mask

Depends-On: https://git.opendaylight.org/gerrit/#/c/53891/
Change-Id: I06db66bb7bf68672f142f1349e91cb12d98b96bd
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Policy classifier table ACL flows

This patch depends on genius patch
https://git.opendaylight.org/gerrit/#/c/52933/

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

Enable SubnetRoute in non BGPVPN usecase

First commit in a series of commits, part of the discovery
of directly connented PNFs in Flat/Vlan provider networks.

This commit enables the creation of VrfEntry with a subnetroute
augmentation, in non BGPVPN use case, with respect to the recently
changes made by the hairpinning spec.

A VrfEntry is created in both external networks and private tenant
networks.

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

Create VrfEntry for PNF

This patch is the third one in a series of patches for
PNF discovery in Flat/Vlan provider networks.

In this patch we create a mip adjacency upon receving ARP
response from a PNF.

VpnInterfaceManager listen for that new adjacency and request the creation
of a matching VrfEntry.

VrfEntryListener Listens for the new VrfEntry and call
createRemoteFibEntry on all relevent switches from VpnToDpn list.

Depends-On: https://git.opendaylight.org/gerrit/#/c/53090/

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

Add DHCP Allocation Pool extension to the DHCP service

Change-Id: I9a0bf4b806824934cf86640b0878b26d338cfb31
Signed-off-by: Shai Haim <shai.haim@hpe.com>

OpenStackSFCTranslatorProvider close() the *Listener/s it created

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

align dhcp yang model in the spec with implementation

Change-Id: If497d228035275eb53f483f6bc5c969eb35697e7
Signed-off-by: Shai Haim <shai.haim@hpe.com>

Replace DataProcessors for classifier service

Change-Id: I696a80a86317143fa540dfb40700bd5895f929bd
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Use TcpMd5SignatureOption in replayNbrConfig

'BgpConfigurationManager.replayNbrConfig' replays the
current 'ebgp:bgp/neighbors' configuration when QBGP
restarts.  With this change it uses
'tcp-md5-signature-option' to set the MD5 password.

See spec qbgp-tcp-md5-signature-option [0].

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

Change-Id: I7119d030166ea953afc32bf0ad1a2bd47ec4dff5
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

React to TcpMd5SignatureOption in NeighborsReactor

When 'ebgp:bgp/neighbors' contains a 'tcp-md5-signature-option'
'BgpConfigurationManager.NeighborsReactor.add' adds a neighbor with
password.

See spec qbgp-tcp-md5-signature-option [0].

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

Change-Id: I892a2f989886c40d8957ccb6e681301e78117c90
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Sort specs index.rst, fix PNF spec formatting

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

Added element counters

Change-Id: Icd37eeedc92cfabd94e1a496833e899b202b20e9
Signed-off-by: Guy Regev <guy.regev@hpe.com>

Don't ignore .gitignore

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

Programming remote fib entries for BGP routes

This review consists of changes made for programming remote fib entry to
point to load-balancing group if there are more than one adjacency.

Also makes sure that the remote fib entry is not deleted during tunnel
remove event.

Depends-On: I87f9054ca7ba69762a57e8d18827c34e17c2f9dc
Change-Id: I8e148cbf940ce206d8af49c384401a118f03e67c
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Early implementation of classifier service

The idea of the classifier service is to replace the DataProcessor
approach. The rationale is that there is a huge amount of
heterogeneous events that may require adjustment of the global
operational state of the classifier. Having specific code to handle
each specific event would require a lot effort. The classifier service
works on this by having a generic processing that caters for any event
that may affect it's operational state.

The classifier service is based on the concept of ClassifierEntry, a tuple
that defines a classifier flow. A collection of this entries represents
the global state of the configuration and operational classifier, one
collection each. By comparing these two collections, we find the
entries that need to be added or removed from the operational state
based on the configuration state at any given time.

Change-Id: Icc33ac761297cea645a34f09d836d6364a3de0ca
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Revive missing features

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

NxMatchInfo redesign

This uses Genius' new NxMatchInfo subclasses for Nicira extension
matches. Requires https://git.opendaylight.org/gerrit/52179

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

Bug 7979: Fix issue where VM is unable to acquire address during IPv6 tests

Tempest IPv6 Scenario tests validate usecases like IPAM, multi_prefix,
dualnetworks etc. While running IPv6 tempest scenario tests, we have
noticed some exceptions, because of which the test-cases are failing.
This patch fixes those exceptions in the file ExternalRoutersListener,
by skipping the IPv6 subnets. The necessary implementation in
ExternalRoutersListener will be added as part of IPv6 external
connectivity blueprint.

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

Updating ArpNotificationHandler.

Updating code related to the new onArpResponseReceived
yang notification

Depends-On: https://git.opendaylight.org/gerrit/#/c/53087/

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

Added support for multiple local_ips for DPN

This is part of the Policy based path selection for multiple VxLAN tunnels tasks.
Now when creating/updating a transportzone it will check for the local_ips information
from the ovsdb other_config entry - example:
local_ips="10.0.43.159:MPLS,11.11.11.11:DSL,ip:underlay-network"

If it is exist it will create the TZ based on it, if not it will take it
from the local_ip information.
It add support for multiple local ips on different underlay networks.

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

Federation plugin

L2 and L3 cross site connectivity using the federation service
 https://wiki.opendaylight.org/view/Federation:Main

Implementation details in the following spec
 https://git.opendaylight.org/gerrit/#/c/52271/

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

Fixed gerrit link

Change-Id: I2dc11d8bfd4131cd8744cb5f0320258e9e20d6c1
Signed-off-by: Guy Regev <guy.regev@hpe.com>

EVPN RT2 advertise route API l2vni parameter added

Change-Id: I113bd9616856b79f0bfe2c22148c7c400768c7b1
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>

Bug 8031: Fix NPE from VrfEntryListener

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

Correct several equals() bugs

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

Elan base code changes

Change-Id: I58a9538826e7b9cbed2a05638e9308d2f2a4318b
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7988 - Cluster reboot fix

Elan Instance , Physical Switch , L2gateWay and L2gatewayConnection all
have dependency among each other.But if upon cluster reboot the order is
not proper (L2gw cache is updated at the last , then flows are not being programmed as expected.
To take care of the situation
Introduced new ElanInstance Listner (Was already present but not
registered with xml to be instantiated )
Handle scenario if L2gW shows up last after cluster reboot .
Handle scenario if l2gwConnection is updated but Elan instance is not
available.

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

Fix ACL IPv6 flows to match on ipv6_src/ipv6_dst for remote SG

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

Added a new framework for selecting centralized SNAT router.

Added a new HA framework for selecting the centralized SNAT router which
will use a weighted round robin to select the NAPT switch

A pseudo port will be added once the router is elected as the designated
switch and will be delete on router delete.

The switch selection/release happens in response to external router
create/delete or snat enable/disable

The flow programming will be done in response to   changes in
centralized switch model RouterToNaptSwitch.

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

Add NPE protection for VpnFootprintService.removeOrUpdate...

 + Refactor the method to be a little bit more 'flatter' by adding
   early returns

 + Create the writeTx only when needed.

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

AclServiceStatefulTest asyncEventsWaiter before coordinatorEventsWaiter

This came up during local investigation of today's aclservice test
failures following c/51783 (reverted in c/53827), and ultimately wasn't
the root cause - but doing this is probably still a good idea.

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

Bug 7606: Fix for missing table 110 flow in OVS 2.4 after VM live migration

Problem:
After live-migration of VM is completed, table 110 flow for migrated
instance missing in all nodes.

Fix:
* Skipped UPDATE event of Source OVS when VM is migrated.
* Skipped the cache update from UPDATE event.

Change-Id: Iddc15a38fc8c96e998a14622217957dcabdff438
Signed-off-by: arthi.b <arthi.b@hcl.com>

Added Conntrack based SNAT implementation.

Added an interface for the Snat operation.
The interface shall be implemented by the multiple snat implementations.
Only conntrack based implmentation uses this interface now.

AbstractSnatService has the flow programming which are common
across multiple implementations of SNAT. The code handles only flat/vlan
use cases now.

ConntrackBasedSnatService provides conntrack based SNAT implementation
for flat/vlan provider networks.

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

Bug 8019: when the neutron port acting as gateway is deleted, invisible ip
is not removed from FIB

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.

Change-Id: If98991940976defe16a7a988aa3071d87693f8e9
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

Bug 7997, 7939:  Groups not going in before the flows to switch.

+ Increased the delay between group and flow pushing to 1500ms.
+ Expanded some critical INFO statements.

Change-Id: I0c8ace294b29c08ebe88ec982cdcd12613a66d04
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

AclServiceTestBase with LogRule and (commented out) RunUntilFailureRule

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

Rename AclServiceStatefulTestIPv6 to AclServiceStatefulIPv6Test

so that 'mvn test' runs it (surefire is based on *Test naming)

but @Ignore it, as it's currently completely broken (TODO fix it)

and patch IdHelper to show cause of NPE occuring in this broken test

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

Add missing <scope>test to aclservice/impl/pom.xml

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

TunnelState changes for tunnels towards DCGW

Changes for programming the load balancing group for traffic towards
DC-GW.
 - When tunnel is added, based on the number of DC-GWs we create all
   possible load balancing groups for the traffic towards DC-GWs.
 - When tunnel is updated, we read the dpid-l3vpn-lb-nexthops and the
   l3vpn-lb-nexthops container to figure out the group which needs to be
   modified and update the bucket directly in that group.
 - When tunnel state is deleted, we delete the entry from both the
   containers and also delete the group if only one DC-GW is left.

This review considers only two DC-GWs for now.

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

NeutronVPN changes to propagate VNI to VPNManager

Adds support to populate subnetmaps DS with network attributes and
propagate the same to SubnetRouteHandler via notifications sent upon
updates to subnetmap DS

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

Revert "Bug 7775: Updated to use DataStoreJobCoordinator for flow programming"

This reverts commit ddfe3934e4df1dce0c54b32a1cd0b1106852852a.

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

Spec for L2vpn over vxlan using EVPN RT2

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

Spec to support IPv6 DC to Internet L3VPN connectivity using BGPVPN

This blueprint spec explains architecture for supporting IPv6 Datacenter
to Internet connectivity using BGPVPN in a SDN controlled by OpenDaylight.
Enhance option b) specification as explained in [0] so that it is possible to
have separated vpns for private inter-dc and for internet access. So that
private vpns keep being private, while an other vpn instance will be publicly
opened.

[0] https://drive.google.com/file/d/0BxAspfn9mEi8OEtvVFpsZXo0ZlE/view

Change-Id: I99b6718d6868fe8b0c49081e2f4cf3897ea1e3f6
Signed-off-by: Julien Courtat <julien.courtat@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Add demo for openstack + new netvirt

Change-Id: I4d9c816ca20c2659f870d7767f01667254ea42ce
Signed-off-by: Guo Ruijing <ruijing.guo@intel.com>

Fixing compilation breakage.

- There were dependencies on Genius that were changed.

Change-Id: I90d1b6dc409bc0c5fb47fda3170a22ddb54f1f38
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

Bug 7775: Updated to use DataStoreJobCoordinator for flow programming

Updated syncFlow() in AbstractAclServiceImpl.java to use DJC while flow is
added/removed to serialize/parallelize flow programming based on device
id. This also avoids possibility of OptimisticLockException getting
generated from ACL while flows are programmed.

Also, Interface state listener updated to use DJC to serialize/parallelize
interface state processing changes.

This review is dependent on below genius patches:
https://git.opendaylight.org/gerrit/#/c/53143/
https://git.opendaylight.org/gerrit/#/c/53352/

Depends-On: I8c15af6607f83a08354520872e4a5ff0ea16d2bf
Change-Id: I578cdf332d8e5575ac31b3b7c5fdf9d7ea86013d
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Bug 7913: QosInterfaceStateChangeListener IllegalArgumentException

Fixed it by avoiding IllegalArgumentException through regexp check.

Change-Id: I8c474eae6629a7219a524a5de3cd92e087c6e746
Signed-off-by: A Vamsikrishna <a.vamsikrishna@ericsson.com>
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 7816: NullPointerException while create a router in external network provider

 * Problem:  Patch port for br-ex is not updated with port name = "patch-ex" into
             localcache during stack up for follower node, thus NullpointerException.
 * Solution: Patch port name is constructed by passing external bridge name and
             as it was constructed on fly in getPatchPortName() in VLANProvider.java,
             patch port name entry was not stored in 'patchPortNames' map.
             However, 'patchPortNames' map will have default value as "patch-ext" for
             patch port name.Default value will be used
             when 'ovsdb.l3.fwd.enabled=yes' in custom properties file.

Change-Id: I9e446529c1962b41c9e97644d59a35dea4606740
Signed-off-by: hari.pr <hari.pr@hcl.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8054: Fix IndexOutOfBoundsException in VrfEntryListener

Issue:
The exception occurs in case handling cleanupDpnForVpn in case of the
prefix not being an extra route as the usedRds is being tried to be
accessed to fetch the extraRoute(it is absent for the non-extra route
prefixes) and so throws the exception.

Fix:
A condition has been indroduced to fetch extraRoutes only if the prefix is
extra route else extraRoutes is set as absent.

Change-Id: Ie6ac4e4121e8dc1e418d21f77ccf41882b78d968
Signed-off-by: gobinath <gobinath@ericsson.com>

Add setPeerSecret to thrift service BgpConfigurator

A new function 'setPeerSecret(ipAddress, secret)' has been
added to thrift service 'BgpConfigurator' inorder to
configure the shared secret needed by TCP MD5 Signature
Option.

See spec qbgp-tcp-md5-signature-option [0].

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

Changelog:

- qbgp.thrift: add function 'setPeerSecret' to service
  'BgpConfigurator'. Add BGP_ERR_NOT_SUPPORTED error code.

- Update generated file BgpConfigurator.java.

- Update generated file qbgpConstants.java.

- BgpRouterException.java: add specific messages for
  setPeerSecret.  Add BGP_ERR_NOT_SUPPORTED message.

- BgpRouter.java: overload BgpRouter.addNeighbor.

Change-Id: I3e19372e29dd1e83f1ea6e394bf7cc941de9d9a6
Signed-off-by: Jose-Santos Pulido Garcia <jose.santos.pulido.garcia@ericsson.com>

Code changes to support qos-rule-types

Change-Id: I80587e96ed7d200b788a3f8ac981e7b46b5d3f47
Signed-off-by: chandrag <chandramohan.grandhi@tcs.com>

Bug 7912 - ACL issue related to AAP with prefix 0.0.0.0/0

+ Supporting AAP with prefix 0.0.0.0/0 for remote security group rules
would lead to a potential security breach. This would result in allowing
the traffic from all the IPs for a VM.

+ This fix removes support for AAP with 0.0.0.0/0 as part of remote
security group rules/flows configuration.

+ Support for AAP with 0.0.0.0/0 will be retained only for anti spoofing
flows which are configured in table 40/251.

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

Fix RemoveVpnPseudoPortJob logger name

 + Since it gets very confusing when reading the logs

 + And I take the chance to add some more debugging traces to
   VpnServiceChainUtils.

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

Bug 7606: Fix for missed tunnel flows, after VM live migration

   * Skip deleting the flows in table 110 and table 20 when processing an
    interface deletion event from OVSDB in the case that the interface
    deletion is due to VM migration.
   * Remove the Port Name from flowName so that, the same entry gets
    updated during the update notification processing while live migration.

Change-Id: I2b917e50e07af4ed840d811a9c3f0a131281bd12
Signed-off-by: VinothB <vinothb@hcl.com>
Signed-off-by: arthi.b <arthi.b@hcl.com>