update DHCP spec according to the desired feature capabilities

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

Update fibManager to use Singleton

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

Update cloud-servicechain to use Singleton

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

Disabling enforce-openstack-semantics flag

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

Bug 8025: Correct non-NAPT to NAPT tunnel ID

As part of the vni-based-l2-l3-nat changes, the tunnel
used was changed but the packet out sent should use the
same tunnel ID for NAPT to work from a VM on a non-NAPT
switch.

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

Fix natservice-impl for Karaf 4

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

Update dhcpservice to use Singleton

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

External networks subnetroute fix

Commit [1] changed the VpnSubnetRouteHandler APIs from using
yang based notifications to listening on subnetmap changes
from the datastore.

Subnetmap entities for subnets in Flat/VLAN external networks don't have a
vpn-name assigned to them, which caused a null pointer exception in
VpnSubnetRouteHandler after [1]

In this change, subnetmap is assinged a vpn-name, for subnets in
external Flat/VLAN networks, which is the subnet id itself.

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

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

Remove statemanager's ConfigStateManager

now that all of netvirt is using blueprint

This class (ConfigStateManager) does not seem to be used anywhere.

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

Update ipv6service to use Singleton

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

update AclLiveStatistics to use Singleton

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

statemanager StateManager @Singleton with @Inject instead of <bean> XML

https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

Change-Id: I9c5607b1eba5bcb9bf74b234a64c7583a272f09d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Command line added to display the current NAPT switches configured.

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

Bug 7633: Karaf 4 vpnservice-it features

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

Bug 7607 : Bgp route not getting sync-up with fib

Problem:
ODL bgp route not getting sync-up with fib table

Analysis and Fix:
In BGPManger, configs are replayed to quagga in BGP reactor.
BGP reactor gets trigered whenever ebgp.yang data model gets added.
on configuring a VPN also ebgp gets added and replay is called. But no bgp
configs are there to replay. Now when BGP config is added later, no replay
will happen and routes will be missed
So moving the config replay to ASID reactor.

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

Bug 7633: Karaf 4 vpnservice features

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

Correct errors in specs

The new classifier description had some typos and errors

Change-Id: I98a98c3e77e15ec55f0392b33029bfc6ad8c22ae
Signed-off-by: Manuel Buil <mbuil@suse.com>

Use correct metadata value for remote ACL

Add a utility to extract the correct metadata
value for writing and matching the remote ACL ID
according to the metadata mask.

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

Bug 8154 - log error for an extra-route if localNextHopInfo is unavailable

As per the comments on this patch https://git.opendaylight.org/gerrit/#/c/53459,
log an error when localnexthopinfo == null for extra-route

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

Bug 7633: Karaf 4 sfc-translator features

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

Bug 8023: returning if StateTunnelList is not internal

While handling add DCN for StateTunnelList in ELAN, making sure it doesn't
process DCN for external tunnel.

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

Fixes for extraroute deletion

* This contains fixes during extra route deletion and nova
  delete
* The fix includes complete clean up of
  -extraroute-rds container.
  -vpn-to-extraroute container
  -operational adjaceny container.

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

Use mask on reg6 metadata

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

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>