L2Gateway: Fixes related to L2 gateway functionality.

+ Removed the check isL2GwDeviceConnected during l2 gw connection delete.
- This change ensures cleaning of data from cache and config DS
  which avoids having stale data when l2gw device is down.
- The stale data was blocking deletion of configs from
  l2gw device during subsequent l2gw connection (having
  same network id) deletes.

+ Few L2GatewayDevice class variables are updated by
  using Set instead of List to avoid duplicates. Also
  toString() method is overridden to print minimal info.

+ doesLocalUcastMacExistsInCache check is added during
  handling of the newly discovered L2gw device mac. This
  check avoids unnecessary configurations found during
  scalability testing.

Change-Id: Iab6be8d67645518e0e78f9ce883b305867c81958
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Merge "Enable NetVirt Maven site"

Enable NetVirt Maven site

This patch enables the Maven site, fixes up all the URLs and corrects
blocking Javadoc errors.

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

L2Gateway: Handling Hwvtep node tunnel ip update. Plus two more fixes.

+ Hwvtep node tunnel ip update is handled only in case of first time
configuration of tunnel IP (i.e., None to valid IP address).
Subsequent updates i.e., changing from one tunnel IP to another is not
handled.

+ Replaced HwvtepNodeListener with HwvtepPhysicalSwitchListener which will
handle Hwvtep node add/remove/update scenarios.

+ Changed logical switch listener to cluster-wide DCN listener to cache
ELAN L2 Gw details on all the cluster nodes when logical switch is
created.

+ Changed remote mcast mac listener to cluster-wide DCN listener.

Change-Id: Icb5efe578102f22650fc9b4ce4dcc8b7022398fe
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Merge "Remove unsued nic dependencies"

Merge "Remove fcapsmanager and application"

Remove unsued nic dependencies

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

Bug 5939 - Communications through external gateway not working

The problem happens because the GatewayMacResolver is not resolving the
MAC address for the external gateway.  GatewayMacResolver is failing
because externalSubnet == null in triggerGatewayMacResolver.  This is
happening due to changes from [0] (a fix for BUG-5291).

In [0], a change was made to only set GatewayIP if port.getDeviceOwner()
== "network:router_interface"; however, in our case,
port.getDeviceOwner() == "network:router_gateway", so the Gateway IP
isn't set.

In this fix, I also set GatewayIP if port.getDeviceOwner() ==
"network:router_gateway".

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

Change-Id: I49b089900e4f85581beb8634580439dca2413ce5
Signed-off-by: Andre Fredette <afredette@redhat.com>

Remove fcapsmanager and application

Fcaps manager and application have been moved to Genius
https://git.opendaylight.org/gerrit/#/c/39107/

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

use the right hop for more than one sf

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

Merge "Added support for enable/disable security on a port dynamically."

Merge "BUG-5813: Vxlan ports should not be removed in table 110 flow entry unless last VM instance removed from the openstack node.   * Before deleting the Vxlan port in flow entry it should     check whether the deleted vm instance is last or not.     If it is the last vm instance Vxlan port should be delete from     source node in flow entry else vxlan port shouldn't be delete."

updates for sfc models

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

Added support for enable/disable security on a port dynamically.

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

Change BGP thrift port to 7744

BGP Thrift port is 6644 which conlicts with NetIDE.
Refer: https://wiki.opendaylight.org/view/Ports

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

BUG-5813: Vxlan ports should not be removed in table 110 flow entry unless last VM instance removed from the openstack node.
  * Before deleting the Vxlan port in flow entry it should
    check whether the deleted vm instance is last or not.
    If it is the last vm instance Vxlan port should be delete from
    source node in flow entry else vxlan port shouldn't be delete.

Change-Id: Iadc9003ed81b1e8e475d629f80ccd961242ef859
Signed-off-by: hari.pr <hari.pr@hcl.com>

Merge "Fix Neutron VPN to handle Routers."

Merge "Added support in neutron and it utils for SGs"

Fix Neutron VPN to handle Routers.

This fix enables Neutron VPN module inside netvirt to
handle routers and router interfaces, thereby enabling
internal VPN to become functional.

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

Added support in neutron and it utils for SGs

Methods in NeutronUtils for creating SecurityGroups
Added param in NeutronNetItUtils.createPort for security groups

Initially commited in draft mode

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

Use Genius for all infra modules in vpnservice

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

Namespace changes for vpnservice modules

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

Fix CSIT failures

These changes bring in patch
https://git.opendaylight.org/gerrit/#/c/38361/ from vpnservice to address CSIT failures

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

Fix NPE in FibManagerProvider

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

Initial merge of vpnservice code

Based on vpnservice hash: commit f267eb8b08e303ccc40131361fe82f2ad5ea1628

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

Imported vpnservice as a subtree

Change-Id: I31c80009a31b0b97c1d10246e844be69e81125f4

Merge "Integration of fcaps applications"

Merge "IDManager fixes for restart scenario"

Integration of fcaps applications

Change-Id: If2fffa37ebed025b203b280ebf7b88a606c4823d
Signed-off-by: Viji J <viji.j@ericsson.com>

Merge "Integration of Fcaps framework"

Updated L2Gw changes in "neutronvpn", "elanmanager" and "dhcpservice" modules

- Used entity ownership service to handle l2gw requests only in one cluster node.
- Registered for entity ownership for elan entity to process all the l2gw requests.
- Taken out the datastore jobcoordinator jobs into separate classes.
- Handled external tunnel state events.

Change-Id: I39a2a24c8927e48fc7d0299de008cfff9ca6d2b4
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

Merge "Remove ovsdb related in resources"

Merge "Implementing Host Config data update"

Merge "Added isPortSecurityEnabled check to enable/disable SG."

Integration of Fcaps framework

Change-Id: I68429a51ba6cd8a7d49a426b78fad866efdf126e
Signed-off-by: Viji J <viji.j@ericsson.com>

Merge "Natservice module bug fixes"

postman: use 1 for netvirt table offset

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

Natservice module bug fixes

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

Vxlan/Gre co-existence,Alarms,tunnelstate,TR fixes

Change-Id: Ia253c413dc198dce371b6f87c2c67af2bdb324b5
Signed-off-by: HemaTG <hema.gopalkrishnan@ericsson.com>

Merge "DHCP Handling for TOR VM"

Fixes in interface manager and aliveness monitor

Removing HwVTEP tunnel entries from topology configuration and operational
datastores
cli command to display internal tunnels is fixed to show the interface
name along with the datapath node identifier
A check has been add to update interface operational state only of it
exists in the datastore
Tunnel to DC gateway should not have key "flow" set
Tap interface not getting deleted from interface operational datastore is
fixed
Interface operational status has been handled to get updated based on port
up/down events
Repopulation of monitor configuration after re-enabling tunnel monitor has
been fixed
Optimizations in aliveness monitor error handling code

Change-Id: I19f0ec5fb0133575e6e08a741f3f0f9c4b805ba7
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

DHCP Handling for TOR VM

- As the TOR floods the packets to the CSS mesh, some logic should be
  added such that only one CSS punts the DHCP packet to the controller and
  the other CSS drops the DHCP packet.
- The designated CSS is chosen for each TOR-ELAN pair and it is stored in
  the datastore.

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

Implementing Host Config data update

This patch implements the logic to fill host config data
based on the yang model defined in the Neutron Northbound
https://git.opendaylight.org/gerrit/#/c/36767/.

Updated patch after addressing review comments

Change-Id: I1ea6b315ab9de70dc1c1849ba905eee2eb51d94e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>

BFD monitoring changes

Enabling/Disabling BFD monitor was fixed. Updates to the  monitor
parameters were not being considered while writing to the data store.

Change-Id: Iaa9a71b353a78d8ac626a81dbe43e1587b8bbdf8
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Merge "BUG 5783: Ping test is FAILED from VM to Invisible IP which is hosted non-nexthop."

Tunnel monitoring changes

Code revision in Aliveness Monitor files related to fetching mac address
from data store.
Monitor profile creation for both the tunnel endpoints was fixed.
Changes in JUNIT test cases related to tunnel monitoring.

Change-Id: I0843ee855f854a3fd98d212516292d8b34a8409c
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Merge "LLDP monitor interval update fixes"

BUG 5783: Ping test is FAILED from VM to Invisible IP which
is hosted non-nexthop.

a. Ensure that SubnetRoutePacketHandler can
work in deployments where DPNs (i.e., Computes)
are connected to different PLs of an ODL Cluster
(i.e., different ODL instances).  More specifically,
this leads to a deployment where the nexthop
reports to one PL but the invisible IP is in a DPN
that is reporting to a different PL in a clustered
environment.

b. Ensure that SubnetRoutePacketHandler can
work with apply_actions rule used in FIB Table
(i.e., Table 22).

c. Create Table 22 (SubnetRoute Table) rules
as default Table Miss Flows instead of dynamically
creating them on the fly in FibManager.

Change-Id: I86e0b3ff8761b054603bc9bb7853340078ad653d
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Co-Authored-By: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Remove ovsdb related in resources

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

move back to non -li openflowplugin features

openflowplugin is removing the -li feature set in Boron
(M3 offset 2) and the non -li naming will be the only
ofp available.

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

Added isPortSecurityEnabled check to enable/disable SG.

The fixed and custom security group rules will be added only if
isPortSecurityEnabled is enabled. This applies for add/remove/update of
SecurityGroup/ security rules.

Cherry picked from ovsdb repo

Signed-off-by: Aswin Suryanarayanan <aswinsuryan@gmail.com>

Neutron->netvirt port rendering

Patch set 7
-----------
Remove unneeded mdsal-util pom changes.

Patch set 6
------------
Fix NetvirtIT.

Patch set 5
-------------------------------------
This patch addresses comments from patch set 4.

Add unit test code for NeutronPortDataProcessor.

This patch supports add/delete/update for
neutron ports and networks.

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

Bug 5781: Ping test is FAILED from DC-GW to Invisible IP
Bug 5782: Subnet route is not getting programmed for a Network

For issue 5781:
The packets received by SubnetRouteHandler are all MPLS Label
wrapped if the packets are initiated by DC-GW.  As a result,
such packets are ignored by SubnetRoutePacketHandler.

This fix changes the SubnetRoute rule in Table 20 (LFIB Table)
to pop-off the mpls label before shipping packet to the
controller.

For issue 5782:
The race to fetch the RD by both VPNManager and SubnetRouteHandler
results in RD fetch failing in SubnetRouteHandler.  The fix is
for SubnetRouteHandler will now rely on VPNInstance config produced by
NeutronVpn rather than using vpn-instance-to-vpn-id config mapping
produced by VpnManager.

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

Merge "Add postman for sff with ovs node in uuid format"

Merge "Basic neutron IT netvirt test"

Add postman for sff with ovs node in uuid format

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

Basic neutron IT netvirt test

Tests that flows are properly created for a neutron
network with a dhcp and two "vms"

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

Bug 5770: Ensuring proper cleanup of VPNControlpath.

1. Fix to ensure proper cleanup of Config and Oper DS
for VPN on removal of VPNs.

2. Fix for delayed release of VPN IDs by ensuring that
we release IDs for VPN and VPNInterfaces only when
the IDs are removed from respective OpDataStores.

Change-Id: Idda295e845af77355c2d7bdf0f60864782a9538c
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Co-Authored-By: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Update NetvirtSFC postman collection

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

TR: Local routes not appeared on deletion of VPN

-Datapath broken when router dissociated from VPN

-Enables tentative fixes to ensure clean up of datapath
on removal of an external VPN, when routers are present in it.

-Similarly enables a tentative fix to ensure clean up of datapath
on disassociation of routers from a VPN.

Change-Id: I224d39b33181d52c1332b1548a91eee167814a85
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Co-Authored-By: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Merge "Clean up subnet routes when vpn is out of a dpn."

add southound-impl-ui as part of openstack features to give restconf

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

IDManager fixes for restart scenario

* ID Entry list is added to parent instead of child. During id
* allocation, check is added for existing id in parent pool.
* Without this fix, each node in cluster used to check only in the
* child pool. And when leader changes during restart, child pool
* didn't have the corresponding id-entry, resulting in a new
* different id.

Change-Id: I479da293eb557915c6a20eaaacc364b5e90c2bc1
Signed-off-by: Manu B <manu.b@ericsson.com>

Merge "Support for SNAT and DNAT features in L3 forwarding services."

Support for SNAT and DNAT features in L3 forwarding services.

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

Use neutron-utils

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

Merge "Use dpl portId"

Merge "Remove the unused shared network validation check as its a bit misleading"

Merge "Postman for L2GW and L2GWConnection"

Use dpl portId

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

split it-utils: ovsdb-it-utils + netvirt-it-utils

Relies on https://git.opendaylight.org/gerrit/#/c/37485/

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

Remove the unused shared network validation check as its a bit misleading

ODL Neutron northbound is now MDSAL based (from Beryllium).
The canCreateNetwork/canUpdateNetwork checks are no longer
invoked in NetworkHandler of netvirt.

Observations with stable/Beryllium and master branch.
-----------------------------------------------------
In stable/beryllium and master branches an admin is able
to create a shared network. Two tenants are able to spawn
VMs (on the shared network), which successfully acquire
dhcp-ip and are able to communicate with the VM belonging
to the other tenant. Verified this use-case on a multi-node
setup.

It appears like shared networks are indeed supported. So, we
can remove this un-used validation as its misleading.

Observations with stable/Lithium SR4 release.
---------------------------------------------
An admin is unable to create a shared tenant network but shared
external networks are supported. Before removing the validation
for Lithium branch we have to see if there was any specific
reason why the check was inplace.

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

LLDP monitor interval update fixes

Updating LLDP monitor interval by deleting the old monitor profile and
creating a new one based on existing monitor profiles

Change-Id: I42c105e40eecfab02cb8da78327cd2733720ec5a
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Merge "JUnit test cases for: 1> SubnetOpDpnManager.java 2> VpnSubnetRouteHandler.java"

Clean up subnet routes when vpn is out of a dpn.

Fix to clean up subnet route flows on a dpn for a vpn
when the vpn is removed from that dpn.

Change-Id: I8fe43b3c80493f72fc08a1801d2a6cb81260f79a
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Co-Authored-By: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

JUnit test cases for:
1> SubnetOpDpnManager.java
2> VpnSubnetRouteHandler.java

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

Fixing Braodcast stroms

When a broadcast package is received
through the internal tunnel it should
not be broadcasted to all internal
tunnels again. It shall only be
broadcasted in the ElanLocal
Broadcast group.

Change-Id: I9c96d6c5ed05203133e0d980a277613191ffde5a
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>

Remove features-ovsdb

VPNService only needs southbound features. Use southbound-features instead of features-ovsdb.

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

Postman for L2GW and L2GWConnection

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

Merge "HwVTEP JUNITs, and fixing show:vxlan CLI"

Merge "L2 Gw connection support and Elan manager changes"

L2 Gw connection support and Elan manager changes

- Handled L2 Gw Connection CRUD related to Logical switch creation/Ucast/Mcast table updates in elanmanager module.
- Updated to use VXLAN provider segmentation id as the ELAN VNI for SR-IOV use cases.

Change-Id: If7bcc0a45a51773a75c7b7117a6a89e713e7eddc
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

Use the netvirt distro

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

Initial pass at changing groupId

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

Update .gitreview for new repo

netvirt is split from ovsdb

netvirt HEAD == 437e5ff1b33b47ae6db2026267510183adb3c674 maps to OVSDB SHA 723a6250573fea7b64fee8da12e7676d83ecd7cd

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

Do not deploy the karaf artifact

The karaf artifact is only used for testing. It does not need to be
released to the world.

Change-Id: I7f25d1310345c27417871fb94a79ef0348863dd5
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Switch from fig to docker-compose"

Merge "BUG-5291:Flows are not removed from control and compute nodes."

Merge "Test for NetvirtSFC config"

BUG-5291:Flows are not removed from control and compute nodes.

  * When the VM instance is deleted from control and compute nodes,
  * Found still flows are existed in table-40 and table-90 in L2 scenario.

Signed-off-by: hari.pr <hari.pr@hcl.com>
Change-Id: I6e0873e979cdbd8dc3037fe012477413faf95a92
(cherry picked from commit ad36780546ecbe1adbf9f081fca0f1eca7287713)

Test for NetvirtSFC config

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

Bug 5651 - southbound-impl failed to find utils.yang-utils in Boron

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

Switch from fig to docker-compose

docker-compose no longer supports fig.yml files, so finish the switch
completely.

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

Merge "OVS 2.5.1 docker image in fig.yml"

OVS 2.5.1 docker image in fig.yml

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

HwVTEP JUNITs, and fixing show:vxlan CLI

JUNIT Test cases for HwVTEPInterface Helpers.
Fix for show:vxlan command output when switch is stopped.

Change-Id: Ic1a4d561a8e508bef1b896fcb2f77eb5caab4d3e
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Use stateless transaction commands

Changing the transaction command model to one where all the required
information is passed to the execute() method makes the commands much
simpler to convert to a DataTreeChangeListener-based model.

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

SubnetRoute enhancements to VPN Service models

This commit deals with changes required to construct
internal data structures to provide SubnetRoute
facility in VPN Service.  With SubnetRoute facility
all packets intended for a cloud coming from a
DC gateway will be punted to one DPN that will be
considered as designated DPN.

Change-Id: Ie33ad0c62bf686496460b94bd4de6f9e28e1ae5b
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>
Co-Authored-By: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Co-Authored-By: Karan Raj Singh <karan.raj.s.singh@ericsson.com>
Co-Authored-By: Achuth Maniyedath <achuth.maniyedath@ericsson.com>

Merge "BUG-5648 - delete qos and queues from udpates"

Split listeners from data processors

Listening for changes is different from processing them, so separate
the two concerns. This should make the data processors easier to
test.

Change-Id: I7a385b13c37ac2d02f12799042fc956084bd3670
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>