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>

Merge "Prepare run_tests to be called directly to main()"

remove testing docs

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

Reduce overview

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

Changing to new docs structure

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

Remove quagga fix in the testcase

Solves partly:
JIRA: SDNVPN-130

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

Prepare run_tests to be called directly to main()

Instead of python -> bash -> python, call
the BGPVPN test using main()

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

Merge "Fix testcase3 (peering subcase) for Apex"

Merge "Fix DNS use for testcase 3"

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>

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>

Merge "Set unlimited port quotas"

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>

Set unlimited port quotas

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

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>

Merge "Create flavor for Quagga ubuntu VM"

Fix broken call to re.search()

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

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>

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

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>

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>

Merge "Adding Labels"

Set unlimited instance quotas to fix CI

JIRA: SDNVPN-115

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

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>

Fix typo in quagga script and call to glance

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

Fix bgpd.conf path and duplication in tc_3

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

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>

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>

Fix a shebang

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

Merge "Final edits for scenario description"

Final edits for scenario description

Removed some placeholders and todo notes, updated copyright and did a final readthrough.

Change-Id: I1357174c159ae1065c4452128152140bb6b00d11
Signed-off-by: Tim Irnich <tim.irnich@ericsson.com>

adding release notes for promise and parser

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

adding index.rst so that the release notes are build

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

Merge "update release notes cause of exclude of rally"

Merge "Refactoring"

update release notes cause of exclude of rally

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

Refactoring

- Wrap common methods that use globals in a Class
 `get_ping_status` and `add_to_summary` depended on globals for
  statefulness. The easiest way to refactor them was to convert them
  into methods and the globals they used into instance variables.
- Move `generate_userdata_common` and `generate_userdata_with_ssh` to
  the utils module.
- Make flavor a kwarg in create_instance.
  Add a defaults section in config.yaml with flavor. This value is used
  if flavor is not passed via kwargs.
- Substitute some sleeps with `wait_for_instances_up` which will
  wait until a dhcp lease success message appears in the instance's log.
- Add functions that wait until router and network association is
  completed. These do not replace the sleeps but make the tests more
  deterministic.
- Remove unused ping_timeout from config.yaml

Change-Id: Ia68ebf47aaa5a58c4e20267f1476db626f015a7c
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
Co-Authored-by: George Paraskevopoulos <geopar@intracom-telecom.com>

Remove trilling whitespace

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