gerrit.opnfv Code Review - functest.git/log

Build arm* functest-vnf containers via travis-ci

It doesn't build juju deps if arm* as they can't be cross-compiled.
Then that deps won't be built in releng on native arm64 hosts
It's worth mentioning that containers have never been executed in
OPNFV gates.

That change allows running all other VNFs on Raspberry PI and
building containers without arm build servers.

Change-Id: I836a72f049e7f0e6ae704f5b829914692d5b2c09
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 0c3ae6e09acd16fc50a516975631dd6bb76b8c53)

Update all OpenStack refs to opendev.org

All gates are failing due wrong to redirections (see os-faults [1][2]).

[1] https://build.opnfv.org/ci/job/functest-opnfv-functest-tempest-latest-gate/96/console
[2] https://travis-ci.org/collivier/functest/builds/522071220

Change-Id: I5a8ca67f0d95df24daa9bdca2f23eb13d896f9d5
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 2ceebaa032f07c4162ed3701c6814dfe5cb683d4)

Updated from global requirements

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

Update Hunter Release Notes

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

Stop forcing admin_scope_domain = True

tempest fails vs devstack stable/stein and it passes successfully vs
devstack stable/rocky (Functest gates) without that option.

It may be added to tempest_conf.yaml if SUT requires it.

Change-Id: I4730c25b25403fd130d71166a8778e06fd479935
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit f038f387a8a0499d39b3fa5091c333b6b0ef5af7)

Speed up lxml build

It allows cross-compiling it via travis-ci (benchmarking) [1][2]

[1] https://lxml.de/installation.html
[2] https://travis-ci.org/collivier/functest/jobs/519737041

Change-Id: I31609268d77405f036924a010cfb3dab520e9ea3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 1d190eb4039194d049cfc97b456978853d0d77d1)

Verify py36 via travis-ci

Change-Id: I1180478c19d0f1c56e615a2af568cbbf4cc828d3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 8f1248484478813b7881e4642cf5ca6dbcf86f74)

Cross-compile arm and arm64 images

It allows building images for Raspberry PI via travis-ci.
It builds all arm64 images except functest-benchmarking [1].

[1] https://travis-ci.org/collivier/functest/jobs/519673994

Change-Id: I695c3afe8aacda63b7eae0d2672cd836c19e8ceb
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 5f525ebaf9d503aa641996fe2ac780dc9dda1774)

Update .travis.yml

It now selects Ubuntu Xenial and libssl1.0.0.
It fixes all issues about SSL [1].

[1] https://travis-ci.org/collivier/functest/builds/518874638

Change-Id: Ib7217afc1cfbe3a7c675961b37ad9c097afca037
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 43e0e08bc2a1a294c582bf79300c4ef34242974d)

Fix doc links

It removes 1 link to ask.openstack.org which is being stopped.
It also bypasses the issues due to LFN false certificates [1].

[1] https://travis-ci.org/collivier/functest/jobs/518170144

Change-Id: Iccfbc2d91aedb4617f2e29e9d03ba01e0fc58175
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 210803c3895ac9205c28868cc53761b20c2d6a01)

Apply blacklist for OVN noha scenario as well

Change-Id: I1dd51a54f39967fcf6b3a522e2a548f81e5646a9
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>
(cherry picked from commit 3509982923786f91c03d05f26f61df4505299722)

Update tab results

It also removes all refs to components and to the former binaries.

Change-Id: I4f7e3308673ba6ba4dd9c28a7a053e3d558c289d
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit e108f1ad405e788e0516a55875269adafb338c28)

Set cinder service type in rally tests

As a precondition for executing scenarios requiring volume, Rally
expects the presence of service named as "cinder". In a case when
there is only cinderv2 and cinderv3 configured in the system, Rally
fails to validate a testcase even the volume service is working ok.

Testcases in question changed to use v3.

Change-Id: I42e7f846e80d3647df274b2e03add94c9f0a34d6
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
(cherry picked from commit ebc2b41505299846faeb2163d34d820a8ef2bdb2)

Remove 1 Patrole test with a hardcoded vxlan id

It may fail when testing OpenStack in production or when running
testcases in parallel [1] because vxlan 200 is hardcoded [2]

[1] http://artifacts.opnfv.org/functest/functest-opnfv-functest-smoke-hunter-patrole-run-75/results/patrole/tempest-report.html
[2] https://github.com/openstack/patrole/blob/master/patrole_tempest_plugin/tests/api/network/test_networks_rbac.py#L208

Change-Id: Ibfaed6f3a95d016790baafebd203e49e152a889b
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 8ee279600186f2713b4c181965d8d290d2644cfb)

Update network context in selected rally tests

Network context not needed in following scenarios:

NeutronNetworks.create_and_delete_routers
NeutronNetworks.create_and_list_routers
NeutronNetworks.create_and_list_subnets
NeutronNetworks.create_and_update_routers
NeutronNetworks.create_and_update_subnets
NeutronNetworks.create_and_show_routers

Router not needed in network context in following scenarios:

NeutronNetworks.create_and_update_ports
NeutronNetworks.create_and_delete_ports
NeutronNetworks.create_and_delete_subnets
NeutronNetworks.create_and_list_ports
NeutronNetworks.create_and_show_ports
NeutronNetworks.create_and_show_subnets

This completes the patch skipping router creation [1].

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

Change-Id: I80ea145a21168fd984cc9ef88d50af227f0d6ed4
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
(cherry picked from commit b4b2e7f9a5a3cb3379b45ac4ddd0ef949974a698)

Skip router creation in selected rally tests

Do not create routers in those rally_sanity tests where the test does
not require it.

Change-Id: I6dd8a2b7579e11ca6ecc3b7383fa4d09a1e56ec3
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
(cherry picked from commit d5e56ad6becd1bb2aa7dafbfccb37c6b84512955)

Update ujson (Alpine 3.9)

ujson v1.35 can't work vs Alpine 3.9 [1].
upper-constraints are overriden to include the patch needed [2].

It fixes rally testcases (gnocchi).

[1] https://github.com/esnme/ultrajson/issues/326
[2] https://github.com/esnme/ultrajson/commit/d25e024f481c5571d15f3c0c406a498ca0467cfd

Change-Id: Iac9975b0669d5975c63d76ddefb64058e5183369
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit c6eb6eca4bb1919c8c943cf534b3e9c935e73118)

Fix other broken links

https://git.opnfv.org/cgit doesn't provide a correct certificate and
then sphinx linkcheck fails [1].

[1] https://travis-ci.org/collivier/functest/jobs/510587514

Change-Id: I884535766c39839927ea863d7fac62684ebb5dac
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit daf01598d6030a29ccddc3023ee94fe9b0e3b320)

Fix the broken link in docs

git.opnfv.org doesn't provide a correct certificate and then sphinx
linkcheck fails [1].

[1] https://travis-ci.org/collivier/functest/jobs/510587514

Change-Id: I4e9b8d07fa8e2da2d1b185668b834a8904cf35a3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit d35d774484de9e444a35322524357c142acd1859)

Remove snaps testcases from site.yml

Change-Id: I5f979b5ad704f8d9b35dd54deda33f910fe08a25
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 8be6277951925e0eeaecb078de2ec4c0b007a38a)

Fix manifest-tool builds

It now selects the last release to allow compiling it [1]

[1] https://travis-ci.org/collivier/functest-xtesting/jobs/510384544

Change-Id: If1c2c1d7bdcc5bc0a6a221dfe5f628b75f9d9608
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 34b5f05cefa2b28e2ff9835cdd7415a0023474fe)

Update Alpine tag in ansible/site.yml and build.sh

It complete the previous commit updating Alpine [1].

[1] https://git.opnfv.org/functest/commit/docker/core/Dockerfile?id=ee700ae5c7ab1fa3e761c330686f89f3ddefbfd3

Change-Id: Icec39d36c661ba3eb3e79c9abb592897f46efa22
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 43d4ce6bd5d0778fb51b014d33a04afafa6a18bb)

Update rest-client for IMS testing

It has to be updated to conform with Alpine 3.9 [1]

[1] http://artifacts.opnfv.org/functest/functest-opnfv-functest-vnf-latest-cloudify_ims-run-95/results/cloudify_ims/ims_test_output.txt

Change-Id: Ic4130d86789537867372f6fbdc026929c28370f2
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 6e4d9a3571bddf94065c9eb031b59731dad4b36d)

Update to Apine 3.9

It updates to the latest Alpine release [1]

[1] https://www.alpinelinux.org/posts/Alpine-3.9.0-released.html

Change-Id: I62ed47547dd8767ca50221840455ed0f7336b438
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit ee700ae5c7ab1fa3e761c330686f89f3ddefbfd3)

Remove hardcoded tempos in heat_ims

It adds loops to wait for services.
It sets dns forwarders to allow sending Heat signal at last
instructions.

Change-Id: I3ae29ddbaee5f8b2086078bd5245e140348f0dcf
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit ce305aa6d78ca62649d43b73b1d8f3b851584d81)

Merge "Update RefStack list to 2018.11" into stable/hunter

Set ext net and name server in Shaker templates

It sets the shaker args to override ext net and name servers [1].

[1] https://pyshaker.readthedocs.io/en/latest/tools.html

Change-Id: I1cc6d52837e3c370db5ff59f024e96d7423be63a
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 492f85284cbe3b78c66d60e187081701c8eb4869)

Update RefStack list to 2018.11

It sets max_microversion to 2.65 because if both values are not
specified, Tempest avoids tests which require a microversion [1].
Then test_keypairsv22_create_list_show_with_type is skipped and
refstack_defcore fails [2].

It also partially reverts "Allow building containers when
refstack.openstack.org is down" [3].

[1] https://docs.openstack.org/tempest/latest/sampleconf.html
[2] http://artifacts.opnfv.org/functest/functest-opnfv-functest-smoke-latest-refstack_defcore-run-147/results/refstack_defcore/tempest-report.html
[3] https://gerrit.opnfv.org/gerrit/#/c/67232/

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

Do not allow testcase skipping in refstack

Set the result of refstack_defcore as failed if there were skipped testcases
in a run.

Change-Id: Id782c2dee511cefbafd5b4440f386fd66f934a47
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
(cherry picked from commit cfd79ec4c39e215a25fc3d99fc5368e7d874f7bf)

Avoid killing create_number.py

create_numbers.py has been killed by other services as described in [1].
It's now called when ellis is up and running.

It adds few sleeps to ensure the right boot orders (signals must be send
before switching DNS).

[1] http://www.projectclearwater.org/using-start-stop-daemon-with-the-python-interpreter/

Change-Id: I87ad3828c10d3ccbc707ed8f4ab99987c6968f09
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 45708eda7ba8e4ddac85930dbb5defb14b9af88b)

Print monit summary before running IMS testing

Change-Id: I94310aa609d2c56609e0a5f0e46932af4aee02eb
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 7bc4e13a4c5409ed7d869ae5c0220feba70dbf65)

Add sleeps and debugs in heat_ims

It allows checking all the services and hardening the asynchronuous
configurations.

It will ease understanding the remaining race conditions in gates.

Change-Id: I1f72a828f4eb32c33f43066e5c71558067bd292f
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 26ddcc28113219b6180b36549995d94f075403fd)
(cherry picked from commit 49dd51392ced7555c59d94a21c7b2632e4cfe8e9)

Revert Snaps in upper-constraints.txt

Change-Id: Ie437b8ea6a33871516d42bf022a1f46cdd488d48
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit bf94db59bb7347dadcfc31accfaf2b71ec288285)

Boot Cassandra at the end of heat_ims stack

Cassandra sometimes doesn't start when it boots in parallel of the other services.
It now selects eth0 instead of forcing ipv6 on lo.

Change-Id: Id9f8a4e19a43e85fc8a04fbc4c8f138f0adea417
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 56245ada9aad65420f7958224e05c17cf4a1c59f)

Remove Snaps-based testcases

Snaps hasn't been synchronized for a while regarding requirements.
We do remove it due to the inactivity.

All test result tabs will be updated in a second change.

Change-Id: I834afd902829ed3883b0e88e92aa806ec43d6fcf
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit e44841210c53290207db9b958c66cb8019910cb7)

Add deps in clearwater.yaml

Ellis must be started right after dns to publish the shared config.

Change-Id: I99fb7ff88e55c5c6a963b3a6db65d8d83b212eda
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit d217430315c5ac571c0006c1178ff6ac50bd3d3a)

Allow building containers when refstack.openstack.org is down

It has been down for days and break our gates.

Change-Id: I2e18a95abaced050f85f6efb8e849e0a5d497ea1
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit a52900d8c926dc4370dea3b3ebe94d20d05e1357)

Remove inactive committers

It follows the last guidelines from TSC (1-year inactivity).
It stop listing contributors as it's worse than git log.

Change-Id: If8fb1ab7371d34798c9f72d8885f4ec092f138ac
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 575916bee58a32b1290ec8d1a3fd707dba541149)

Add dependencies in heat_ims

It avoids useless loops for cloud init messages.
It now conforms with clearwater-docker (disable ipv6 in cassandra).

Change-Id: I6f469cf48224296b2eaa079d342642c5caca65f5
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 464829a80878ede363537ec4020be30c21a5b859)

Remove SDNVPN as it doesn't participate to hunter

Change-Id: Iddeaa3890635103b66042a918ab66cabd8fdee60
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit f799428f8839a59973ba7edb3b6e66819e27e5ee)

Merge tempest.py and conf_utils.py

Change-Id: I597abdbb5bd3de023aa7b66fa27764ca2ec27f0d
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
(cherry picked from commit 226e0f36ee86202c262444cd0712250f4691ab49)

Modify the upstream Clearwater Heat files

It forces a single network for both management and signaling traffic.
It meets the OPNFV deployment requirements and the Functest SUT ones.

The security group will be improved in a second step to first quickly
fix the testcase in gambia. Private IPs may be registered in DNS to
avoid network address translations.

Change-Id: Ic19cf336ac5c2d07c52c6dd37b06271790145cf9
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 39ab5b6cab5d6c664dc96bbb92781a9eed0aa41d)

Stop overriding ssh default timeout

It should be modified by tempest custom yaml file instead.
It's safe as the default timeout is bigger than the previous one.

It also cleans the obsolete tempest/rally data in config_functest.yaml.

[1] https://docs.openstack.org/tempest/latest/sampleconf.html

Change-Id: Ibad02d8aaf90f154c82f3b60a6d3f04fc37bd7de
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 147f5a373b4c8659826f1f626bd9b6042afc3934)

Stop forcing non default operator role

Member is the default value [1] and SwiftOperator is not created by
default in OpenStack gates (devstack)

This role should be modified by tempest custom yaml file instead.

[1] https://docs.openstack.org/tempest/latest/sampleconf.html

Change-Id: Ie77b91e36a7011e5730cbf5998e029c07ec7fe2f
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 543884f5c2100878692260fd840624a549843238)

Change password generator

It adds lower and upper letters

JIRA: DOVETAIL-747

Change-Id: I4bdfcbf31a6dae4aa555fca3d42bbdec514a2b6b
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 470bcd99fdabf53d4652d13280329243d5f81c4a)

Pin neutron to the last Rocky version

networking sfc and bgpvpn depend on it.
If unpinned, neutron 14.0.0.0b2 is selected and raise side effects [1]

neutron 14.0.0.0b2 has requirement neutron-lib>=1.24.0, but you'll have neutron-lib 1.18.0 which is incompatible.
neutron 14.0.0.0b2 has requirement pyroute2>=0.5.3; sys_platform != "win32", but you'll have pyroute2 0.5.2 which is incompatible.

[1] https://build.opnfv.org/ci/job/functest-features-docker-build-amd64-hunter/125/console

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

Select SFC Hunter in stable/hunter

SFC team just published stable/hunter [1]

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

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

Conform heat_ims subnets with other testcases

It will be enhanced in a second step to calculate options from configs.

Change-Id: I91738e1aaad9a7049df44f734c4190def62fff91
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 96bb5dc834e3600a3bd0666f80f3298a420b37fa)

Reduce Heat Ims resources

It eases testing with less resources.
It conforms with the new values in the Cloudify part [1].

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

Change-Id: I1f6158a42fd8ed390c07e7c773bb246f81adef3b
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 57ef1f3dfd0b0d2d7a0646c75d5b295fbbe6aec9)

Ensure boot order in heat_ims

Ellis publishes the shared config which mostly fails when resources are
allocated in parallel (conflict). Then nothing works when the shared
config is blank.

It still remains failures in IMS testing which are still under analysis.

Change-Id: Ic54a2a7478e07a9291c76be8ffc57f04d199421e
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit b0d0e945ebdff62045fe74bceacbf731f5d1abfd)

Gather all skipped tempest test cases

This patch applies a change to the regex responsible for
identifying Tempest test cases. With the current regex some
skipped test cases are erroneously ignored.

The condition has been updated in order to match both format of
skipped testcases ('skip:' and 'skip [xxx]:')

Change-Id: Ie7144b01e07cae92918eede0324f3f1f2582c4aa
Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
(cherry picked from commit 692c50bbeb01b57c184cd8629a534a19f9b9ad73)

Remove useless vyos_vrouter output dir

It doesn't store any file.

Change-Id: I94801508a11edb8f0bbff7e4319c4d50411ec715
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 9262d3c0ee23aa5b79dbf790162134acccb5c42b)

Update VyOS to 1.1.8

It's worth mentionning that the filesystem is hugely reduced.

Change-Id: Ic3e8a5d601a62f1762cbd491c228b7f053905e35
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit d4efec0554d8ba66d71d67626630ff9cc0761da2)

Ease copying/pasting the private keys from logs

Change-Id: I8f1ccdb709dbb3e7a22a404c036714dd848dd87d
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit f6b2577f4e052f493f6359265fc473fe98830b62)

Reduce Cloudify Ims resources

It eases testing with less resources.

Change-Id: I301cdff0c8dfb2cf70ef53fc446749e1a70b0919
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit fd2375f17a6294210ffb48cdcb79e7d129ebe94d)

Stop downloading Cloudify container

It selects the Docker image archive as proposed by Cloudify.

Change-Id: I88b5333625a8d2cab140291552c1c8ca5bac3a36
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit fc5bccf4d29cd4bf705e3938fc189be686083f41)

Enrich Rally results output

This patch adds success/failure information of rally test cases to
summary output.

Change-Id: Ie4f9b4972a26e187bb7afdddfd5ce51b16f3db12
Signed-off-by: Stamatis Katsaounis <mokats@intracom-telecom.com>
(cherry picked from commit 13777d607dd219ace0157e560ff11c3cd6f1aad0)

Only run bgpvpn tempest api tests

It conforms with SFC regex and fits OPNFV criteria.
Running scenarios may force the use of bagpipe [1].

The SUT (functest-virtual) is being updated to verify SFC and BGPVPN
apis.

[1] https://docs.openstack.org/networking-bgpvpn/latest/user/drivers/bagpipe/index.html

Change-Id: I1442b9cac7eaba1aeef04e36a2f790bd29f90976
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 2b4e7955815ce27284043c03730fff6ace029a0d)

Harden upper-constraints.txt operations

It takes into account the wget exit values and avoids manipulating a
falsy empty file.

It won't help fixing the network outages in LF network which have been
highlighted by false failures in api_check (the wrong snaps package
is installed [1])

[1] https://build.opnfv.org/ci/job/functest-opnfv-functest-healthcheck-latest-api_check-run/100/console

Change-Id: Ic9b75965b1ffb1f7bb342ff216bda9933ca75f0b
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit fa8c50f4fd19576a2fde5b04077810d4ca91b330)

Generate xunit reports (rally and tempest)

It adds xunit reports for rally-based and tempest-based testcases.
It completes the reports provided by snaps (thanks to Xtesting).

All rally related operations are moved to rally.
It allows removing the rally dependency to tempest which was false.

Change-Id: Ia7d2476f58f4f68b7c88442e50cad844037a36e9
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 3393f2016483555c27d612c69ec11274cc8aa72a)

Update rally to support one global html

It isn't supported by Rally initally selected in Functest Hunter [1].

[1] https://build.opnfv.org/ci/job/functest-opnfv-functest-smoke-hunter-rally_sanity-run/15/console

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

Stop building components in travis-ci

It was removed by "Move rally_full in benchmarking" [1]

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

Change-Id: I7ade6f474364646bfe0f3ae1b2718749bd0ff26e
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 257af33f88e7f2ff55aa74b171455c83978610f1)

Specify which rally tests to run

This patch adds the ability to specify which rally test to run by
setting the appropriate configuration value.

Change-Id: I6cbc9d12aada4f8eab68d2219d94ae1391a79021
Signed-off-by: Stamatis Katsaounis <mokats@intracom-telecom.com>
(cherry picked from commit d0cb38d7ad5046700564f2700df9b8c92706b32c)

Publish tempest details into database

It switches from tempest.log to rally.log to conform with rally/stestr.

Change-Id: I00c2a268a2bc312a22a8d70278fad618c1c25235
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 4afaf09c94da6e98ca449a7495f158e2d30255e5)

Fix smoke testcase list

Rally_jobs were moved into benchmarking.
It completes "Move testcases in the right tiers" [1]

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

Change-Id: I1c2da77eff6682f85ab8504750077c9ee542edb8
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit dd217735edd2e596d396a2b8815b418548f78111)

Generate one single report for Rally testcases

It eases sharing one single report including all task results.
For the time being, rally_jobs only generates neutron.html.
Then it only adds one report for rally_full and rally_sanity.

We may consider generating only one json file but it asks for
a refactoring (result parsing) which can be done in a second
change.

Change-Id: I8fa75ae80750336950f355c2aedc3dd4dc621cc5
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 2df7a0728d687de9a24d8432ef30bbd2ecd771c8)

Increase timeout to reach vnf API

It increases the timeout in cloudify-based and clearwater-based
testcases to allow running them in parallel (functest-virtual) [1].

[1] https://build.opnfv.org/ci/job/functest-latest-gate/54/

Change-Id: Ic637cb46a1380a683cb1e2b633cb97ac3be6affa
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 38cba4aa2c2cdfd17ea1aa0c29ee14ed7b2bf035)

Set gnocchi as optional in Rally testcases

ci/testcases.yaml was copied/pasted when moving the rally testcases.
It completes "Set Gnocchi scenario as optional in rally tests" [1].

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

Change-Id: Ia23ae9764750775fe8983660c56ced06f3de977a
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 44db92e6135a573ed4bb23648ea1c73e8a70f91b)

Update ansible/site.yml (rally_full)

It conforms with "Move rally_full in benchmarking" [1]

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

Change-Id: I6382f6e612343896a1d817675e122d2c1e0b1a38
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 883d645814f2afd85bb37d042ab5c43565983f6a)

Move rally_full in benchmarking

It allows merging all benchmarking testcases in one tiers and saves
lots of time as they could all run in parallel.

Change-Id: I551179709d2ce862056c1f5fcc1e16ef0b776dd4
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit cec721055631734d2d1d39bd09cfa1404ee1d32e)

Remove go cache dir

It saves about 290M.

Change-Id: I603c689be989a603e933970f52510026026c7001
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 56c077dde1556b88030619cd4d26703060cb0824)

Select last git commit id from vyos blueprint

The pull request #4 fixing the blueprint [1] was merged [1].

[1] https://github.com/oolorg/opnfv-vnf-vyos-blueprint/commit/9b76d46a388d32d4985797620e67c2ed3315b3e4
[2] https://build.opnfv.org/ci/job/functest-opnfv-functest-vnf-hunter-gate/4/console

Change-Id: I3ddd86ee8c9fd30e68de74f06f40f782b1b04fd2
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 83a4d0ca9875890ddee2128976325c0c6a6263fe)

Update and enable Cloudify-based testcases

It switches from the Cloudify virtual machine to the community
container which allows deploying vyos_vrouter and cloudify_ims.

Last stable Cloudify OpenStack plugin (2.4.17) is now deployed.
It also updates the Cloudify python package to latest release.

It adds a delay before deleting the blueprint which may be
improved in a second change (it allows enabling the testcase in
the stable release).

Change-Id: Ie48a2c9d2badab0d7ba0b9e7845f453fad335ef7
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 97dde581982b9d49122aee3630328fb6064c8609)

Increase juju_epc timeout

It fails 4 times in functest-virtual (out of 34) due to a possible slow
bandwidth (e.g. [1]).

[1] https://build.opnfv.org/ci/job/functest-opnfv-functest-vnf-latest-juju_epc-run/24/console

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

Remove tempest_smoke from smoke

It completes "Move testcases in the right tiers" [1]

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

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

Move testcases in the right tiers

Here are the moves:
  - tempest_smoke in healthcheck
  - tempest_full and tempest_scenario in smoke
  - rally_jobs in benchmarking

It avoids conflicts between benchmarking tools and few functional
tests (short timeouts).

Then it should allow running in parallel all testcases in one tiers.
It also allows cleaning the additional steps defined in releng.

rally_full could move in benchmarking in a second step.

Change-Id: I0bc657557a61c7d9bf7679b4f2dbda08c36728ca
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 1d0f0bbf773aaaf7139b76442a9e59d13bd32830)

Update Cloudify images

The former image can't be downloaded and then breaks all gates.
It will be completed by changes in vnf if needed.

Change-Id: Ie364aa2315257157a5e09eb4ba121926a60e7b26
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit f501b520f80d573a77cdae941fe19db216b94307)

Override tempest upper-contraints

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

Update tempest to fix interfaces_by_fixed_ip

It includes [1] which allows testing even if one shared network is
defined.

[1] https://review.openstack.org/#/c/615425/3

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

Remove CreateInstanceVolumeTests (snaps_smoke)

The volume and the virtual machine created are not deleted.

Change-Id: I238fcb0839a2f609333080bc8c88b84327d51475
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 14d26a963665f17dc01d95bb996e1ae91ac44f03)

Decrease volume_size in all Rally tests

Devstack allocates a 24G disk image by default [1].
It can be easily reached by setting volume size to 5G or 10G.

[1] https://github.com/openstack-dev/devstack/blob/master/stackrc#L779

Change-Id: I7c35d8f333718d6c192a9f12a714e626d60b1895
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 890d9a764bf88752d36983802f375d5e29c2bee2)

Allow running latest tempest_full

It allows running test_user_update_own_password [1]

[1] http://artifacts.opnfv.org/functest/functest-opnfv-functest-components-latest-tempest_full-run-8/results/tempest_full/tempest-report.html

Change-Id: I9f273ba58871f1e16886af623dcbb91f5c20c2f6
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 9cd8ca6db84fd80f17b2d9c37d66d6a861130e7a)

Wait for apps being active (juju_epc)

It seems that juju-wait doesn't wait for all apps.
Juju_epc passes in functest-virtual thanks to that additional tries.

Change-Id: I161902c6308cfa6323c21e09140d16e036768ece
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 900b51c6885339c68cd0762f2b10363c436fbafb)

Clean project/user if snaps tests are skipped

Change-Id: I88a58516a3620cc6b19013313672ad1a80791e70
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit c9b7fe1da2432581d30bb166d381ac72fce33e97)

Merge "Stop using urljoin when adding v3" into stable/hunter

Merge "Set Gnocchi scenario as optional in rally tests" into stable/hunter

Remove Snaps NeutronUtilsFloatingIpTests

It has been fixed in Snaps [1] but we can't sync its requiremets [2].
Then we have to remove the test.

[1] https://gerrit.opnfv.org/gerrit/#/c/66625/
[2] https://gerrit.opnfv.org/gerrit/#/c/66627/1

Change-Id: Ied405092cb4992bc04ca13c1536d1e8cee056ace
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 17b788c5340e7d76f6c37397cd8aa35c8db3f4c2)
(cherry picked from commit bda196b00cd7567aad33bde12f66536bd1143bc3)

Stop using urljoin when adding v3

urljoin works here only if url ends by /.
It replaces identity by v3 when running vs devstack.

Change-Id: Id896f2b54dfcabadaad1137deef91b1cb07c697d
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 9e4e6f049fa7d5cce8db61dfa2ebebc55f91c1d8)

Set Gnocchi scenario as optional in rally tests

The execution of an optional scenario is skipped if the corresponding
openstack service does not exist.

Change-Id: I8721f26ebc253ce02138b85d6c88e659c5afae80
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
(cherry picked from commit 3b7b393ca643bcf7db0361bcc4414b74fff953be)

Remove test_get_endpoint_with_each_interface

It checks all 3 endpoints for image but nothing forces to define them.
Here is the only endpoint created by devstack.

glance | image | RegionOne
| | public: http://192.168.1.10/image

[1] keystoneauth1.exceptions.catalog.EndpointNotFound: internal endpoint for image service in RegionOne region not found

Change-Id: If501b849fbbed6df98742db21e924094c1df9c7f
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 9675c5a0cb2ccd698451a5930708263be5eba527)

Slightly improve heat_ims

It increases timeouts which could be reached if shared bandwidth
(functest-virtual [1]).

[1] http://artifacts.opnfv.org/functest/functest-opnfv-functest-vnf-latest-heat_ims-run-4/results/functest.log

Change-Id: I29adcd9ea88bbccbaef9ba7f8e155d37a3217433
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 9bb9d8d495317fe7e3aaeb7082d9662f211eafec)

Force the right external network if juju

It may select a falsy external network [1].

[1] https://build.opnfv.org/ci/job/functest-opnfv-functest-vnf-latest-juju_epc-run/4/console

Change-Id: Ic403d0cfb4bc829275617d6b1338311e8e7ea894
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit f28094ae0ecfedc3c7c3f32cc1cfc1eff12ea773)

Fix one typo in docstring (tempest.py)

Change-Id: I46053c92660139f4004d9cec8d1cfb6817cc0e07
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit bf976b4f606860630ffa967fe70a36a7e5954828)

Take control over Rally logics

Rally selects the first external network [1] which could raise side
effects if:
- another testcase is creating a dummy external network in //
- all external subnets are not reachable from jumphost
It's now overriden by EXTERNAL_NETWORK as defined in Functest

Rally creates a new shared network if one already exists [2].
As juju now allows other shared networks, it can be safely created by
Functest to allow only one logic and to handle provider networks,
different name servers, etc...

[1] https://github.com/openstack/rally-openstack/blob/master/rally_openstack/verification/tempest/config.py#L146
[2] https://github.com/openstack/rally-openstack/blob/master/rally_openstack/verification/tempest/context.py#L85

Change-Id: Icf8c08077d4b0a9eb1c2e1b7309c62957b0a3b63
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit ee52195f4476d3debf9c3f92c6d808256a199500)

Move logics out of TempestCommon.__init__()

It conforms with best programming practices.
It allows muting console and increasing log level (debug).

Change-Id: I0e0cfef6abe3b354a43817fcb242c71da701e237
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit b482b494ce695e83cca0a10e7750200fd1be42ea)

Remove former ci jobs

They can now be generated via the new ansible role.

Change-Id: I7746d08b940e8d6906cb023c39bc87d8dee441cc
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 12023f6a1a040c07f805553a1f5c65ab3940d638)

Remove fds in all branches

FDS didn't participate to Gambia.
Removing it in all branches avoids compexifying the jjbs.

This patch will be reverted as soon as the support is ready.

Change-Id: I9fb293b2972e0deb318ac9d3d08377039ea247e4
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 2fd07adb36288e9247ad73a7344340c36754132e)

Check if key exists instead of leveraging on update_if_exists

It seems falsy (see line 72)

2019-01-07 11:50:32,550 - functest.core.cloudify - DEBUG - try 9: Cloudify Manager isn't up and running
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/functest/core/cloudify.py", line 72, in execute
    "foo", "bar", update_if_exists=True)
  File "/usr/lib/python2.7/site-packages/cloudify_rest_client/secrets.py", line 86, in create
    response = self.api.put('/secrets/{0}'.format(key), data=data)
  File "/usr/lib/python2.7/site-packages/cloudify_rest_client/client.py", line 263, in put
    timeout=timeout)
  File "/usr/lib/python2.7/site-packages/cloudify_rest_client/client.py", line 224, in do_request
    verify=self.get_request_verify(), timeout=timeout)
  File "/usr/lib/python2.7/site-packages/cloudify_rest_client/client.py", line 158, in _do_request
    self._raise_client_error(response, request_url)
  File "/usr/lib/python2.7/site-packages/cloudify_rest_client/client.py", line 119, in _raise_client_error
    response=response)
  File "/usr/lib/python2.7/site-packages/cloudify_rest_client/client.py", line 131, in _prepare_and_raise_exception
    status_code, error_code=error_code, response=response)
CloudifyClientError: 409: <Secret id=`foo` tenant=`default_tenant`> already exists on <Tenant name=`default_tenant`>

Change-Id: I6f4e2719ad2e55e997ebbd158dce4c6a204b2c36
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 73f4aee1a22e502a7775a9be43829174f18fbc84)