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>

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>

Merge "Updating release notes for Colorado"

Use get_functest_config where appropriate

The latest functest enforces using `get_parameter_from_yaml` to only
get test specific parameters from the config file.
`get_functest_config` is given to get functest configuration parameters.

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

Updating release notes for Colorado

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

Refactor some common methods into utils

- generete_ping_userdata(): This method was exactly the same in two
  tests
- create_instance(): The various implementations of this method had
  almost the same signature, but not quite. This was fixed by adding
  extra arguments as needed. Also a global variable was used in the
  function, which was converted into a parameter.
- create_network(): The three implementations of create_network had
  three different signatures and returned different variables. They
  were all unified in a single create_network class and the calls to
  create_network were adjusted accordingly.

Revision 1: Fix flake8 violations
Revision 2: Rename logger in utils

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

Adapt some calls to functest_utils that don't require logger

Due to a small refactor of some functions in Functest,
the calls to those are outdated and fail.

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

Add Router Association testcase

JIRA: SDNVPN-49

This functional test is a variant of testcase 1. Instead of associating
Network 1 and Network 2 to the BGP VPN, we connect Network 1 to a
router and associate that router and Network 2 to the VPN.

The testcase is disabled for now.

Test setup procedure:
- Set up VM1 and VM2 on Node1 and VM3 on Node2, all having ports in the
  same Neutron Network N1 and having 10.10.10/24 addresses in subnet
  SN1. N1/SN1 are connected to router R1.
- Set up VM4 on Node1 and VM5 on Node2, both having ports in Neutron
  Network N2 and having 10.10.11/24 addresses in subnet SN2.

Test execution:
1. Create VPN1 with eRT<>iRT and associate R1 to it
 - Ping from VM1 to VM2 should work
 - Ping from VM1 to VM3 should work
- Ping from VM1 to VM4 should not work
2. Associate SN2 to VPN1
 - Ping from VM4 to VM5 should work
 - Ping from VM1 to VM4 should not work
 - Ping from VM1 to VM5 should not work
3. Change VPN1 so that iRT=eRT
 - Ping from VM1 to VM4 should work
 - Ping from VM1 to VM5 should work

Jira task in SDNVPN: https://jira.opnfv.org/browse/SDNVPN-49

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

Small final improvements in userguide.

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

Add more information how to manually configure

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

adding example to the userguide

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

Moving documentation content into new template

JIRA: DOCS-103, DOCS-134, SDNVPN-40

Moving documentation content into new feature description and user guide template.

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

Merge "Applying the userguide template for the SDN VPN userguide documentation."