Clean always and change order

Change-Id: Ic9e0d374a4fe5278888b28fe2694c546840d55b4
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 88331de3fb9375d316acba7b160ddf2a9beab5b2)

Remove dependence inet

A new ubuntu image with quagga included was uploaded to
artifacts.opnfv.org/sdnvpn/
ubuntu-16.04-server-cloudimg-amd64-disk1.img

Change-Id: I7e48ebbc2b00b7e771f43b26303acc66c2c48457
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 9e47ddb9739bb3e6b2ca91615c71bcd94c954172)

Cleanup br-ex/br-quagga from testcase_3

After the peering between Quagga and ODL within testcase 3
cleanup br-ex (for fuel) or br-quagga (for apex) interfaces

JIRA: SDNVPN-13

Change-Id: I3e7bf91d4e47dfd7fe2a5068ed40ea4b2be48c6d
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit 0d8f3bf930bdd945d34188fb591a6deb03a999c5)

Merge "remove unused tempest blacklist" into stable/danube

remove unused tempest blacklist

Change-Id: Ica6365f41f1d4cf46244604b2ccb889bad051680
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 5bcc4c1a6ec2e592d3a0dac4b309b46e48f651a8)

Checking if hacked quagga bridge exsists and contiune

Change-Id: I8983ed687dabba0af2128c0ce06bd53aba22c819
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 2bfbe25b1df24d562789faa641c41e9cafec7efa)

Do not stop tests if exception in testcase

Change-Id: I9e3db350be4b1ca60d6feaf2961034b50b50deb5
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit fbe69370c477a7b01032df701fe96ea90ec60341)

Correct testcase 8 description

jira: SDNVPN-138

Change-Id: If56b58b003ce2e024883a015a964e3ff3ded53f5
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 9a52d17d9ac79c358647d87b5d540e869b25769d)

Add Labels

Labels serve as refs when referenced
from other documents.

Change-Id: I37b93bae1283d2ea8638fc49c0c40fe7badf61ad
Signed-off-by: ShubhamRathi <shubhamiiitbackup@gmail.com>
(cherry picked from commit d900e3f166536bfffdfba094757fbee4337630b8)

Fix security group rule conflict

-open_icmp_ssh function, that creates security
 group rules for icmp and http (here the name of
 the function is misleading),  is splitted into 2
 new functions one for each rule (open_icmp() and
 open_http_port()).
 Test cases that used the old function have been
 updated

-In the original implementation of open_icmp_ssh()
 and open_bgp_port(), the direction parameter was
 missing. This was leading to an error message

-A check for the existence of security group rule
 has been added and used before the creation of each
 security group rule. If a rule already exists,
 an info message is printed and the testcase
 continues

This change is strongly dependent on https://gerrit.opnfv.org/gerrit/#/c/31925
of Functest repo. NOT TO BE MERGED before

JIRA: SDNVPN-103

Change-Id: Icb96954556f6d7294cf3454f045dbca4b9be672d
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit 0e26e7dfd4ff41ae7c8389218bca138346864922)

Merge "remove testing docs" into stable/danube

Update url for ubuntu image downloading- testcase3

The url that the testcase3 uses to download ubuntu image
(https://cloud-images.ubuntu.com/releases/16.04/release/
ubuntu-16.04-server-cloudimg-amd64-disk1.img) is unstable.

This change updates it, using the one from the OPNFV artifact repo

JIRA: SDNVPN-133

Change-Id: Ic7265b572064f1df7ce30ab5249b1bdb61c63777
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit 70e78577e8736d17fe3d99c9c276d7ca9b6ea89b)

remove testing docs

Change-Id: I76af05c6b3dbd945d4c027ea3dc58ae1e5357418
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit a8ac0683dc4accfc4d300de20071989c0d1ffb63)

Reduce overview

Change-Id: Ifb8915d62ea26f44f29c4d3b84beecd78dceae9e
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit c063e49cec6afa6ba1ab743a86482768fe1cc28e)

Changing to new docs structure

Change-Id: I4c15e1635981b2b5fe5473be89b5abab78bfabd5
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 0f8220d5d96cc0a45be00e937c73ab938133e587)

Remove quagga fix in the testcase

Solves partly:
JIRA: SDNVPN-130

Change-Id: I1cf99e9294925545f6fece660983e32f6543f2b8
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
(cherry picked from commit 5960fab7230bebbffc439457d5542d086cb3f8af)

Fix testcase3 (peering subcase) for Apex

 - Fix runnning bgp related commands on Karaf:
   update util.run_odl_cmd() in order to be able to
   run commands on karaf client. Otherwise karaf
   commands fail
 - Ensure zrpcd/bgpd to run on controller side properly:
   Add missing quagga run folder, change owner and
   add user quagga into quaggavt group, in case of apex
   installer
 - Update the way of handling br-ex information and
   parsing out the controller external ip and mask:
   In case we have multiple ip/netmask assigned on br-ex
   the separation was faulty and an extra line, containing
   the second ip address, was added into ext_net_mask variable

JIRA: SDNVPN-118

Change-Id: I62468c94fd945ba30e419bd30d9cd85502f28eeb
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit db1ce6276c443dd9766458c516cc58c7f2e098d5)

Fix DNS use for testcase 3

Make the VMs use properly 8.8.8.8 nameserver

Change-Id: I8ebf0ff23fa78dcbc50eba9eb27262e690c56b66
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit 3570a9d39984c078f039726b98fc33d26ef94baa)

Merge "Set unlimited port quotas" into stable/danube

Set unlimited port quotas

Change-Id: I6de4bc921f1823f58395e27d804b863b79d23030
JIRA: SDNVPN-126
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit 7532973ae1c38e5d4f3fd6028f4688c72d41ea8d)

Testcase 3 Apex compatibility fixes

- Workaround a bug releng in getting opedaylight nodes
- Add some sudo commands because the path is not properly set otherwise
  it seems
- remove needless command that gets ip information from the compute,
  since that is already available from when the command was ran on the
  controller
- Handle attaching the quagga instance to Apex's br-ex ovs bridge

Change-Id: I870f1049d9cce696ec26376a09db5f9e9bac0bf9
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit b5b2a3abd74f84943cf3fab7ca7da591d326661c)

Merge "DNS fix for testcase 3" into stable/danube

Merge "Fix broken call to re.search()" into stable/danube

DNS fix for testcase 3

The CI fails because the openstack DNS does not work.
Make the VMs use 8.8.8.8

Change-Id: If0c45710cda402158171ce3436731d96bc346565
JIRA: SDNVPN-125
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit f7a6f2593a1b99a0bd0459093d606dd4bdbb999a)

Create flavor for Quagga ubuntu VM

- Implement a  new function in lib/utils ensures
(gets or creates) a new flavor existence
- Use this new function within Testcase3, in order
to create a flavor big enough to start ubuntu with

JIRA: SDNVPN-120

Change-Id: I5c7d9c855f083b96bfd056bb5d935baa851420c9
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit f368948e25f78c64c328662b52d6b139b02826b4)

Fix broken call to re.search()

Change-Id: I3d49c683c81e136967432acd1d28b6d305cfe90f
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit 28c5fc2391e133e4b76f1bed8675bc9610ebdf9b)

Fix most end-to-end issues with testcase 3 and reenable it

- Make the test work end-to-end on Fuel
- Use the instance log to verify quagga started
- Don't attempt to check for peering, as that is for
a future release
- Use the external IP of the controller for peering
- Fix not getting the output of OpenDaylight commands

Change-Id: Ia7bb533ab65e0fca6b7f48bb64133f6d8f3ff8ae
JIRA: SDNVPN-114
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit 6674049b09d02fe0dc7e11e007e710643cdd49ca)

Merge "Adding Labels" into stable/danube

Merge "Set unlimited instance quotas to fix CI" into stable/danube

Set unlimited instance quotas to fix CI

JIRA: SDNVPN-115

Change-Id: I4e24800e897702b8b5aa5ef0227925d66d552f02
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit 38e72ad1c9e4fbf4421e6314ab9a5468a5b29d34)

Adding Labels

Labels serve as anchors/ refs when referenced from other documents
This patch adds labels to docs/*/*/index.rst

Change-Id: Iaba02fccac1cbd164a44a7fef913572b1819b00c
Signed-off-by: ShubhamRathi <shubhamiiitbackup@gmail.com>
(cherry picked from commit 611651c82af07246775ec4aedb3ffeac74ea1ea7)

Implement a get VM IP function in utils to be used within test cases

Change-Id: Iaedbfb2fc626eb7d88badee2530593b43c0d1152
Signed-off-by: tomsou <soth@intracom-telecom.com>
(cherry picked from commit 7efb216ca8357533dd040cccc49cb5effec42615)

Merge "Fix typo in quagga script and call to glance" into stable/danube

Merge "Fix quagga peering by working around the NAT" into stable/danube

Fix quagga peering by working around the NAT

Quagga peering doesn't seem to work over the NATted openstack external
network. To work around this, directly patch the quagga instance to the
external network and give it a fixed IP in that network. Then, all the
peering traffic goes over that patched interface.

Change-Id: I8d9d8fef4e86839c821f83e564d5c49cc15e43ae
JIRA: SDNVPN-114
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit 1ad4cd4d35bcd0d4d9898650eb0452b184b55f77)

Fix typo in quagga script and call to glance

Change-Id: I80ce2f3b442bc7862486368822c8fe2cd13a0f1b
JIRA: SDNVPN-114
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit 7b1e45de5574cd35013910e746756d0718a45ceb)

Fix bgpd.conf path and duplication in tc_3

Change-Id: I78e521fddd8a156d5c9597d6307ac9b6f2571f5c
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
(cherry picked from commit 4c9d7aa555b450c5c3cb88debed7f2f62f884346)

Fix functest failing in apex case- testcase3

- Check the INSTALLER_TYPE environment variable
- Differentiate fuel from apex case

JIRA: SDNVPN-101

Change-Id: I10fa26f5c0b7f95601b29db86e756a20c9e921ec
Signed-off-by: tomsou <soth@intracom-telecom.com>

Merge "Set unlimited network and subnet quotas to fix CI"

Merge "Implement quagga peering"

Set unlimited network and subnet quotas to fix CI

Change-Id: I0804580fa5d89550911a5caf08dc00f9c9523bb1
JIRA: SDNVPN-100
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Implement quagga peering

- Create an openstack instance
- Assign it a floating IP
- Execute a quagga bootstrap script using cloud init
- Tell ODL to peer with the instance
- Start an instance in a VPN
- make sure the route was exchanged between the peered quagga and ODL

Change-Id: I73bcaec5425df2b953c2bceaca7d4f09ff28f3d0
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Update release notes for Danube

JIRA: SDNVPN-96
Change-Id: I0bb0d7f065bdd0f356b995f24374eb32f77344d1
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Implement testcase 3: Quagga-ODL integration

Testcase 3, which verifies OpenDaylight can start and
communicate with zrpcd/Quagga

- Verify that zrpdc service is running
- Issue bgp speaker start to OpenDaylight
- Verify that zrpcd has started bgpd

JIRA: SDNVPN-98

Change-Id: Ia0c497292a724161259c669425229d189a317ba2
Signed-off-by: tomsou <soth@intracom-telecom.com>
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

TC 8:Fix argument passed to create_glance_image

Change-Id: Ibed4febf84f4384ffcca951bd95d5284add454ea
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Enable floating IP and router assoc test

Change-Id: I2bc538fd055a14c5295f955ed700674cf60ad452
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Merge "Fix incompatibility with new tempest functest API"

Add and update license headers

Change-Id: I8ccb23c9ffa0d511427ad089b033973ffe96360f
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

Fixes tripleo_helper permissions

There were issues with some of the commands that need to be root.  Along
with logging into the undercloud when the tripleo_inspector script is
ran as a non root user.

Change-Id: Ie29b674bd0770759b3bcc22323b424fcc270eef6
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix incompatibility with new tempest functest API

Change-Id: I2f2bf9ed9dd775bec222d2faa241ffd1920aa516
JIRA: SDNVPN-93
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
Signed-off-by: tomsou <soth@intracom-telecom.com>

Merge "Updates ODL Pipeline scripts for CSIT"

Updates ODL Pipeline scripts for CSIT

Changes Include:
 - Change to TripleOInspector which only introspect a
   current Apex deployment and dump yaml config to be bundled with
   snapshots.
 - Add TripleOHelper which consists of all tripleO helper functions.
   Many this are done to virsh so the idea is to have at.
   Some point in time a libvirtHelper, or use another libvirt
   python lib. Thatsway it is a class so that we can inherit
   later on.
 - New argument for passing the SSH private key to use to connect to
   nodes is added to the service utils.
 - Some general clean up and consolidation of logic

JIRA: APEX-363

Change-Id: I792db0fac3f4e81969fe85c05fc298fe5af02537
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Test for router assoc and floating IP"

change cloner info path

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

Test for router assoc and floating IP

JIRA: SDNVPN-90
Change-Id: I81a38c3b707dd836512c9afbab93808cb6b91adc
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Use find_packages when installing sdnvpn

If we specify only the module 'sdnvpn', it
won't install the sub-modules 'lib' and 'functest'.
With find_packages there is no need to specify
the package names.

Change-Id: If7606a2cedbd6f7b8866747e748eadaa8b9bd5e7
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

Bug fixes after the Functest refactor

Change-Id: I20b9629bd3c9945e98edb94056f459d9f75eec5d
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

Update directory path

JIRA: FUNCTEST-673

Change-Id: I8430939c009942cd9f96e526c8acd2115267ed2f
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

Merge "Fix table formatting in testcase 7"

Merge "Modify tests to require 100% subtest success, fix tempest"

Change execution permission for scripts

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

Add post process script

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

Fix table formatting in testcase 7

Not enough '-'

Change-Id: Ia90a678cdf7022cbdbafb1d42531e7aaff716c2c
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Modify tests to require 100% subtest success, fix tempest

This is required to better align with functest, since that will handle overall
success/failure status. Also if a subestest failed, there were expected failures
that could seem like they pass making the overall success % look the same, which
would make the test look fine, even thought it failed.

Some tests check things that are known not to work. Disable these until they are
fixed upstream.

Also remove the success_criteria field from the config. Fail the test if any
subtest fails. This has the added benefit that it fixes the tempest suite, which
was still trying to read the value of `succes_criteria` that had been renamed to
`success_criteria`.

JIRA: SDNVPN-89
Change-Id: Iba5e5cc0db6c371f29a4cbba66ca16c4a1838e12
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Merge "Make sdnvpn a package"

Make sdnvpn a package

- Clean up the test/functest folder and move things to lib/ & artifacts/
- Add a new top level folder for the sdnvpn python code

Change-Id: I5fdc7fa5475fb800f488a17d3481158c9c4f84e1
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Merge "Execute script in lib folder/Use vjumb as Gateway"

Execute script in lib folder/Use vjumb as Gateway

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

Merge "Testcase for router/FloatingIP & net assoc mutual exclusivity"

Merge "Don't pass ips to get_ssh_output and check_ping"

Merge "Encapsulate some functionality in Results class"

Bugfix " in the environment part-3

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

Bugfix " in the environment part-2

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

Bugfix " in the environment

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

add setup script

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

small bugfixes for odl_reinstaller

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

small bugfixes for test_environment

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

test_environment upload the cloner info to jenkins

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

Merge "adding common/config"

adding common/config

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

Testcase for router/FloatingIP & net assoc mutual exclusivity

A testcase for ODL Bug 6962, testing whether a subnet with a router can be
network associated:
- Create two VMs, each in a subnet with a router
- Network assoc the two networks in a VPN iRT=eRT
- Try to ping from one VM to the other
- Assign a floating IP to a VM
- Ping it

JIRA: SDNVPN-88
Change-Id: I464f62b8bba55bc5d162071406637f6be846f223
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Don't pass ips to get_ssh_output and check_ping

Remove the IP arguments from these functions and use the VM objects passed to
them to find their IPs. This simplifies the calls.

Change-Id: Ideb08b6fdf0648870c2854f057708a88bb482d0c
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Encapsulate some functionality in Results class

- Move adding actions taken during tests to summary in a method in Results
- Provide add_success/failure methods that hide the column size manipulation

Change-Id: If7416f3f9399125fe21632328a49af285a4b6cc6
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Merge "adding odl-pipeline"

adding odl-pipeline

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

Merge "Move logging instance creation to create_instance"

Merge "Modified testcase_2 according to the Limitations of Boron"

Merge "Enable basic router assoc test (testcase 4)"

Merge "Remove push to DB of overall result of bgpvpn test case"

Move logging instance creation to create_instance

Change-Id: I299f2be3b127aefdd7036fa68fb934b09d278692
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Modified testcase_2 according to the Limitations of Boron

Change-Id: I79fa899a5deb917f5d87298b7c1b2dd17ad7770d
JIRA: SDNVPN-85
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Enable basic router assoc test (testcase 4)

Router assoc works now, enable it in the CI.

Change-Id: I095e5af048a597188169b87a71381208bb478aab
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Remove push to DB of overall result of bgpvpn test case

Functest will capture the return value of run_tests.py
and will report to the DB accordingly (PASS/FAIL)

JIRA: SDNVPN-83

This follows this approach:
https://wiki.opnfv.org/pages/viewpage.action?pageId=8685677#HowdoIintegrateafeaturetestcaseinFunctest?-3.Createapythonmoduleforyourtest

Change-Id: I664964038b59ab8f38946d2dfe586dbdd41e9906
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

Adapt new environment variable after refactor

This patch changes the env variable
repos_dir to REPOS_DIR

https://gerrit.opnfv.org/gerrit/#/c/24619/

Change-Id: I55e17ee09395a976e79b8ee1abf045787009f52e
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

Refactor config global variables

- Create config.py module to handle configuration parsing
- Remove globals from testcases

Change-Id: I16ed7bb138fde2d5d3c69703e3c2e569d49f0b37
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>

Merge "Fix router assoc in Boron"

Encapsulate sleeps in function

We have some sleeps into the testcase code. We can encapsulate them in a
utility function to refactor them out of the code until we figure out
the best way to remove them completely

Change-Id: I44c32d8329c2e319660518a8a46debc22f02c753
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>

Refactor common code across tests

- Move collecting test results in a method in Results class
- Create a util function for asserting the environment has
  enough compute nodes

Change-Id: Ib7b9dfab7d2e9f50d5f352d3eadba4d4717ee750
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Fix router assoc in Boron

As per I6b57eab89839d9e9122cd24b0f05737467439dd9:
- Add a route distinguisher
- Remove the router from the network that is network associated

Change-Id: I1399bb51c787df881529279ef7a1946a3c5c5591
JIRA: SDNVPN-66
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>

Fix network assoc tests in Boron

Network assoc in Boron with netvirt does not work for subnets that have
a router due to an inherent limitation in the implementation. See the
mailing list and related bug:
https://bugs.opendaylight.org/show_bug.cgi?id=6962
https://lists.opendaylight.org/pipermail/netvirt-dev/2016-October/001815.html
Also, if a subnet does not have a route distinguisher, it is not added
to the FIB, so association does not make traffic move between subnets.
This is intentional and an error is logged when a subnet is associated
to a bvpvpn without a route distinguisher.

This commit fixes the net assoc case and works around these issues by:
- Removing the routers from the subnets in testcase_1
- Adding a route distinguisher to the bgpvpn

JIRA: SDNVPN-74
Change-Id: I6b57eab89839d9e9122cd24b0f05737467439dd9
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>