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."

Applying the userguide template for the SDN VPN userguide documentation.

Added a draft of the scenario description template for consideration and review.
This is a draft migrating the existing userguide text into the proposed
feature project userguide format for Colorado.  Please review and comment,
correct and merge as needed.  (this is the prototype)

Change-Id: Ia82a0c3ba0f654efc2892aac3a3b2d217ed2ad27
Signed-off-by: ChristopherPrice <christopher.price@ericsson.com>

[DOC] Automatic Installation

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

Adding New Comitters

Change-Id: Ia237ab7347c789761e36bec1cf28372f8fbcefbc
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Colorado documentation: scenario install procedure

JIRA: DOCS-134

Adding structure of docs for installation
procedure for virtual deployment.

Introducing a scenario-specific directory level.

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

Return -1 if the tests fail

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

Remove logger from openstack_utils function calls

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

Report overall status of the test results to DB

The reporting Dashboard looks for the test case called
'bgpvpn'. Now, we report the overall status of the tests
into that test case and also each one separatelly.

http://testresults.opnfv.org/test/api/v1/results?case=bgpvpn

Also, fix the 3 flake8 violations

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

Create a blacklist of Tempest test cases

JIRA: FUNCTEST-369

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

Add support to push the results to the DB

JIRA: SDNVPN-19

The test scripts now return a json object with
the following format:

{"status": criteria, "details": results}

where
criteria: "PASS"|"FAILED" (result of the test case)
details: free output that will be printed to the console
and pushed to the DB

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

Merge "Fix testcase2"

Add tempest wrapper to enable the bgpvpn plugin for tempest

The tempest.conf is copied to bgpvpn_tempest.conf to be updated
with the needed parameters, in this case, only adding
bgpvpn=True.
The way to execute this module only is with the option -C.
For example:

run_tempest.sh -C bgpvpn_tempest.conf -t
  -N -- networking_bgpvpn_tempest

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

Fix testcase2

JIRA: SDNVPN-31

It seems due to the merge conflict, some of the lines got
reordered and the script failed.

Also, added the private key for the instances.

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

Bugfix run_tests fails to run python scripts

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

Adding test case 2

JIRA: SDNVPN-31

Test case 2 checks tenant IP address space separation.

Now done with this change.

Change-Id: I07ef04de4195b2892c837ac87e53146d133755c6

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

Add config for testcase 2

JIRA:SDNVPN-31

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

Add testcase1 implementation

JIRA: SDNVPN-30

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

Added directory structure for the tests and script templates.

JIRA: SDNVPN-19

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

Removing whitespace to remove build error messages when pulling sdnvpn docs.

Change-Id: Ie9728bafed45dbff444102c85ba10501130521ce
Signed-off-by: ChristopherPrice <christopher.price@ericsson.com>

Merge "Reverify"

Reverify

Adding release notes for Brahmaputra.3.0.

Change-Id: I262f4e0ab7ec785088601b48557555e07e03f870

Small update to configuration guide.

Change-Id: Ibb7efe2c6a476e205f806e99afdfd041f3740f36

Changing filenames to match convention and update index.rst.

Some additional content for feature config guide.

Fixing a typo in userguide.

Change-Id: I1f3056535052e74ec893b9f3264a154a145c2350

Adding link to BGPVPN user guide.

Change-Id: Id0d67c804d721baf41828b5aff01990b729f3406

Adding SDNVPN release documentation.

Change-Id: Iff53422b5e609a21e7165946d776e66fdfdbe431
Signed-off-by: ChristopherPrice <christopher.price@ericsson.com>