resync and ecmp testcase failure

This provide fix for resync and ecmp test case failures
at the CI master runs. Please have a look at the comments
in each testcase for more details.

Change-Id: I26c40da144e34a48ad44897988675db8d37c062a
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "fix for sdnvpn CI test failure"

Use bgvpn-delete and create instead of update

Testcases 1 and 4 fail because of netvirt bug [1]

The same functionality is now achieved by using
bgvpn-delete and bgpvpn-create cli commands instead
of bgpvpn-update

[1] https://jira.opendaylight.org/browse/NETVIRT-1067

Change-Id: Iaba6e06d0f39e356bf364b0640c707b0a83470d7
Signed-off-by: tomsou <soth@intracom-telecom.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

fix for sdnvpn CI test failure

Change-Id: Ie55ccbe68f57a9bb03efc8b6eebf23a0018a0bca
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Updating INFO file - new PTL

Updating project lead and primary contact to Peri after PTL election.

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

correct setup-openstack.yml destination path

Change-Id: I9c1c174529e1802beeab6b4e5ac91d18283aa521
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

correct XCI scenario variable

Change-Id: I4fcd3773457b6acb14720b84ba145ed9cac0368c
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

os-odl-bgpvpn scenario for xci deployment

This includes scenario os-odl-bgpvpn scenario specific configuration
files to make XCI to install Openstack (master and stable/pike) with
OpenDaylight configured as a BGP speaker.
XCI would automatically checkout sdnvpn repo and use scenarios
directory for the configuration during its execution time.

Change-Id: Ic86fcb171c8e97f7d3c3fb649ab0fb00368e6c6d
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Set unlimited router quotas

Update router quotas to fix bgpvpn scenario in CIq

JIRA:SDNVPN-208

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

Intra-Data Center ECMP testcase

This testcase add a scenario in which one extra route can be reachable
through two VMs which are in a same network attached to a router which
is later associated to a BGPVPN instance.
This extra route ip address is different from subnet of VMs.

After the extra route is configured with VM ips as nexthops, then when
third VM is trying to reach extra route, traffic would split between two
VMs. The traffic split can happen only in heavy loaded scenario, so this
testcase would test only reachability.

JIRA: SDNVPN-23

Change-Id: I4bb4e94f5c98c0f3c22bff7a6998af2a7a919f16
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Resync testcase description

JIRA: SDNVPN-200

Change-Id: Ida111d28591d99cc8d2a73ec1c14920800354a9b
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "Adding two new committers"

Merge "Updated from global requirements"

Adding two new committers

Adding Thomas Sounapoglou and Peri Palanisamy as new comitters to the project.
Both have made substantial contributions to the project and are very active.
Existing committers, please approve.

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

Merge "add resync testcase"

Updated from global requirements

Change-Id: I7a975d6cdeae1f7dbf982eca8f8de16fa1bc3d95
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

add resync testcase

It is an extention of testcase_11.py to check resync happens properly
between ODL and OVS when there is no changes from northbound

JIRA: SDNVPN-175

Change-Id: I63bd17f5e5fdab7d5ab90a2be2836e02e7dd54fa
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Adapt bgp daemon to run properly for testcase3

Since apex includes a new version of zrpcd compatible with Carbon
and Nitrogen [0], an adaptation is needed for bgp daemon enabling
and running within testcase3

Note that zrpcd run now as a process and not as a service

JIRA: SDNVPN-172

[0] https://jira.opnfv.org/projects/APEX/issues/APEX-508

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

Merge "remove flavor at the end of the test"

Merge "Fix instance boot when metadata exists"

Merge "Adapt documentation wo be comply with Apex Integration Guide"

Merge "Add tetcase suite overview"

remove flavor at the end of the test

JIRA: SDNVPN-122

Change-Id: I89f457e0d5a7c7c98bab223e722a5eb5a23387b6
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

use glance_client for image cleanup

Change-Id: Ia192a1b26c5f31b6d3465a060c3d8a3097e6ebd4
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Test groups are deleted after ovs reconnection

Create a new testcase (Testcase 11)

Testing procedure:
Create a Network with 2 VMs so that a group is added
to ovs. Disconnect ovs. Remove the Network and the VMs.
Connect ovs again. Check that the group is deleted from
it.

JIRA: SDNVPN-151

Change-Id: I4c6f5e16f837bbcc589cbbbf05896a2d17b9cd57
Signed-off-by: panageo2 <panageo@intracom-telecom.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Adapt documentation wo be comply with Apex Integration Guide

Since, Apex installation instructions have been changed, the corresponding section
into docs/release/scenarios/os-odl-bgpvpn/scenario.description.rst is also updated
according https://wiki.opnfv.org/display/apex/Integration+Guide

JIRA: SDNVPN-178

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

Add tetcase suite overview

Adds an overview of testcase suite, under:

docs/development/overview/index.rst

JIRA: SDNVPN-179

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

Fix instance boot when metadata exists

The way that a instance is considered as UP is pretty
strict and returns ERROR in case of instances with userdata

This patch differentiates the instances that are supposed to
boot till login prompt and those with userdata parameter

JIRA: SDNVPN-177

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

use openstack api's instead of CLIs

add two apis update_instance_quota_class and
update_nw_subnet_port_quota and make run_sdnvpn_tests.py to use these
APIs for setting quotas

JIRA:SDNVPN-188

Change-Id: I764bb6897e6c4c5abad4d252c1f3b4ccc742d52e
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "Remove argparse for adding argument -r"

Merge "Inheritance from Feature class"

Remove argparse for adding argument -r

Change-Id: Iaa2ddc3f655a2bf85785a99a519e0df78c298037
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Merge "Gather logs only if fuel or apex installer"

Adding scenario documentation

Adding the scenario documentation, which was missing from previous commit.

JIRA: SDNVPN-185

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

Gather logs only if fuel or apex installer

Change-Id: I11988ad86a916d1d7251aacf951c6362fda961ba
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Title adjusted

Removed Danube and simplified the title.

Change-Id: I0029b87f5382acecd3dd8e143139920ffd57c696
Signed-off-by: Sofia Wallin <sofia.wallin@ericsson.com>

Inheritance from Feature class

Change-Id: I81aa9582452ca43a4eab32ba34a640043c89043e
Signed-off-by: Jose Lausuch <jalausuch@suse.com>

Merge "Updating docs for Euphrates"

Updating docs for Euphrates

Moved previous user guide content into the right places,
updating the section structure along the way.

Started to edit the release notes but this needs more work.
This will be merged now to enable link checking by the docs team.
Remaining chan ges to be done in a separate review.

JIRA: SDNVPN-185

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

Merge "comment multiple subnet configuration"

Revert "write sdnvpn test result into db"

This reverts commit 2907946ee8931f749881e48e9601779be50b7e37.

Turned out the issue requires a different solution.

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

write sdnvpn test result into db

The details of the test results are not flushed into db which causes
test report showing empty data

JIRA: SDNVPN-183
JIRA: FUNCTEST-874

Change-Id: Ie539685990014149434d59e61490750ae318b3ba
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

comment multiple subnet configuration

* commenting multiple subnet configuration under a neutron network because ODL
doesn't have a support for it yet (netvirt bug: https://bugs.opendaylight.org/show_bug.cgi?id=9239)
* This has to be uncommented once the above bug is fixed.

Change-Id: I4c863fbdd951891035471f1a10cf18546f5218e0
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

SDNVPN-184: resources are not cleaned if tc fails

* moved the cleanup code inside the finally block in each test case

Change-Id: Ifc0fee58451923117c54040c716dad6827fbc7fa
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

make CommonConfig as singleton

* It is not necessary to instantiate CommonConfig object from each
testcase and can be made as singleton

Change-Id: I99d995486fbd88ea8a77ec7bd9e41a55047ebaca
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Fix logging

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

Fix the argument error

Change-Id: Idc4ba350a8bf77ec555109ad4e7a2649e2b0880d
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Fix testcase_10 module path

 Update config file with the full path for testcase 10

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

Fix adapt path for import_module

    Fix the path to include each test case when the suite runs

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

Fix the config.yaml including for each testcase

At each testcase, adapt the way config.yaml is included via
sdnvpn.lib.config.TestcaseConfig().
The full path should be included, instead of just the testcase name

JIRA: SDNVPN-180

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

Adapt SDNVPN Functest test according to new framework

- Inherit from TestCase class in Functest
- Remove any push to DB method, since Functest
   takes care of it

After this change, we will be able to load this module
from Functest and make a python call instead of
a bash call.

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

Merge "Make tight the way an instance is considered as UP"

Merge "Add/Remove testcase"

Make tight the way an instance is considered as UP

An instance will be considered UP not only if a DHCP IP is
leased, but if the login prompt is present, as well

JIRA: SDNVPN-174

Change-Id: If68fd1436a54f057f37de777e22f1d6a1967d97c
Signed-off-by: panageo2 <panageo@intracom-telecom.com>

Fix import module functionality for each testcase

Fix the path, from where import_module runs each testcase

JIRA: SDNVPN-176

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

Merge "Removing inactive committers in accordance with Section 8 of the TSC charter."

Add/Remove testcase

This testcase investigates if communication
between a group of VMs is interupted upon
deleteon and creation of VMs inside this group.
Test case flow:
3 vms:
[1] on compute 1,
[2] on compute 1,
[3] on compute 2,
all vms ping each other.

vm [2] is deleted. Traffic is still flying
between [1] and [3]. A new [4] vm is added
to compute 1. Traffic is not interrupted
and [4] can be reached as well.

JIRA: SDNVPN-105

Change-Id: I93a65de2a5df83551e3115fd20b60241cd460e97
Signed-off-by: panageo2 <panageo@intracom-telecom.com>

Install run_sdnvpn_tests.py in $PATH

It eases how Functest runs it (see testcases.yaml [1]).
The former run_tests.py is renamed to avoid duplicated binaries (see
[2] and [3]).

[1] https://git.opnfv.org/functest/tree/functest/ci/testcases.yaml
[2] https://git.opnfv.org/sfc/tree/sfc/tests/functest/run_tests.py
[3] https://git.opnfv.org/promise/tree/promise/test/functest/run_tests.py

Change-Id: I50219cad42bf0e085c2edf2e1abe149c2e057a99
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Refactor imports and invocation of main function

In all testcases there are some inconcistencies of
import statements. Additionally the main function
of each testcase should be called through
sys.exit(main())

JIRA: SDNVPN-173

Change-Id: I6fdf0dc6598ca4789b6de6b50d1b8b85a8de923c
Signed-off-by: panageo2 <panageo@intracom-telecom.com>

Merge "Add test case 9"

Fix log gathering functionality

Some fixes on https://gerrit.opnfv.org/gerrit/#/c/37261/

JIRA: SDNVPN-170

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

Add test case 9

Development of a new test that checks all
nodes and succeeds if all OVS br-int interfaces
have fail_mode=secure

JIRA: SDNVPN-168

Change-Id: Iff14f60e2d25c1769cdec7cec126425937780eb7
Signed-off-by: panageo2 <panageo@intracom-telecom.com>

Removing inactive committers in accordance with Section 8 of the TSC charter.

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

[docs] Adding apex deployment config guide

JIRA: SDNVPN-171

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

Merge "Update release notes"

Update release notes

Uplifting docs to D-3. Update release-notes
to reflect the remaining issues we have not
been able to resolve.

JIRA: SDNVPN-169

Change-Id: I5997f1495fc37ecbc06a8c7b8071a7ba7f2c099c
Signed-off-by: panageo2 <panageo@intracom-telecom.com>

Adding log gathering

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

Fix the wrong path of sdnvpn

The new path should be /usr/local/lib/python2.7/dist-packages/sdnvpn,
rather than /home/opnfv/repos/sdnvpn

Change-Id: I22e019226e7bffe233b52c1a3ec6171b835e4bd3
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Merge "Adapt the floating ip handling in functest"

Merge "Add networking-bgpvpn module in SDNVPN requirements"

Merge "Transfer utils functions from functest"

Transfer utils functions from functest

Transfer functions:
  - create_bgpvpn
  - create_network_association
  - create_router_association
  - update_bgpvpn
  - delete_bgpvpn
  - get_bgpvpn
  - get_bgpvpn_routers
  - get_bgpvpn_networks

from functest/utils/openstack_utils.py to
sdnvpn/lib/utils.py

JIRA: SDNVPN-164
JIRA: FUNCTEST-840

Change-Id: Ic64d8ee85e74601b0fe51e1c452cf83fa6b0203b
Signed-off-by: panageo2 <panageo@intracom-telecom.com>

Adapt the floating ip handling in functest

Comply with the new floating ip management in functest

(https://gerrit.opnfv.org/gerrit/#/c/36415)

JIRA: SDNVPN-165

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

Merge "Uplifting docs to D-3"

Merge "Restore peering within testcase3 for Apex"

Add networking-bgpvpn module in SDNVPN requirements

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

Add requirements in sdnvpn

It now leverages on pbr to scan requirements.txt and inject them into
the install_requires.

Change-Id: Iae1517ae0bfd3575e9d16de4a2ca42e114d956fb
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Uplifting docs to D-3

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

Restore peering within testcase3 for Apex

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

Cleanup after testcase running

After every testcase running cleanup neutron and nova elements:
 - Allocated floating IPs (nova)
 - Created instances (nova)
 - Created images (nova)
 - Created router-subnet interfaces (neutron)
 - Created gateaway routers (neutron)
 - Created routers (neutron)
 - Created subnets (neutron)
 - Created networks (neutron)
 - Created bgpvpns (neutron)

JIRA: SDNVPN-145
JIRA: SDNVPN-148

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

Merge "Fix fail of pinging a floating IP- testcase8"

Fix failure in router association- testcase8

Due to some latency at the creation of the instance 8-1
somitimes the router association between  'sdnvpn-8-1-router' and net 'sdnvpn-8-2'
fails.

The case could be more robust, increasing the number of ping tries from
its current value (=1)

JIRA: SDNVPN-102

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

Fix fail of pinging a floating IP- testcase8

-Raise the upper bound for ping action

JIRA: SDNVPN-154

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

Fix the wrong image path for base image

The images are put under new directory while the code is still using
old path

Change-Id: I6dfa59ffe6bde316f155560449979bd10354b43c
Signed-off-by: helenyao <yaohelan@huawei.com>

Merge "Put the ubuntu images in new directory"

Update logger via logging.getLogger()

1. The module functest.utils.functest_logger has been deleted and
   logger has been reconfigured in functest [1].
2. Update all related modules to get loggers via logging.getLogger().

[1] https://gerrit.opnfv.org/gerrit/#/c/34019/

JIRA: FUNCTEST-824

Change-Id: I89f5d15342b5f257fbc2700655e255a4e28bd36f
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Fix the zrpcd running check within testcase3

Check specifically the "running" state of the zrpcd service

JIRA: SDNVPN-150

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

Put the ubuntu images in new directory

To adopt the new change of Functest[1], all images are put under
${HOME}/functest/images. Users are allowed to
pre-download the images and map the local folder to the new directory.

[1] https://gerrit.opnfv.org/gerrit/#/c/34033

Change-Id: Iafee8825af77dfce74ad2ace3346e3f2b70a7c96
Signed-off-by: helenyao <yaohelan@huawei.com>

add lable for both scenarios

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

removing multiple labels

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

Removing peering testcase for Apex

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

Merge "Stop and start quagga when reinstalling"

Stop and start quagga when reinstalling

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

Uplifting docs to D-2

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

Clean always and change order

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

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>

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>

remove unused tempest blacklist

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

Checking if hacked quagga bridge exsists and contiune

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

Correct testcase 8 description

jira: SDNVPN-138

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