gerrit.opnfv Code Review - functest.git/log

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)

Print traceback as debug during cloudify tries

The traceback is printed as debug to avoid priting the first
unsuccessful tries in console. It will ease debugging issues as for [1]

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

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

Prevent from CloudifyClientError (key already exists)

It has rarely occured in Orange Openlab [1].

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/functest/core/cloudify.py", line 70, in execute
    self.cfy_client.secrets.create("foo", "bar")
  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: Ic61d0c729f13caa552a0b0a00dad35e50d130e01
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit b8a115906e4f6e598e7e1f1b59e2b1fb201af6a5)

Merge "Publish "Functest on steroids" (Plugfest-Gambia)" into stable/hunter

Switch from port to dport in site.yml

port is reserved in ansible

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

Publish "Functest on steroids" (Plugfest-Gambia)

It will be completed by demos:
  - deploy your own Functest CI/CD toolchains (Functest bot)
  - deploy your own CI/CD toolchains for your test case
  - run Functest on Raspberry PI

The presentation is avalaible in [1]

[1] http://testresults.opnfv.org/functest/gambia/

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

Update reveal.js to 3.7.0

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

Conform with new tag configs

It allows generating gate jobs.

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

Disable nose html output plugin

It doesn't run vs py36.

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

Update Xtesting to 0.61

It now generates html reports for unit tests (snaps).

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

Fix nits in ansible/site.yaml

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

Verify Ansible playbooks via ansible-lint

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

Allow pulling Neutron reviews or tags

It updates smoke/Dockerfile to allow pulling Neutron reviews/tags
(rally_jobs).

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

Deploy Functest CI toolchains

It eases verifying OpenStack by deploying the full testing toolchain
via Ansible. As opposed to the Releng model, all tests can be executed
in parallel.

It leverages on Xtesting Ansible role which currently deploys
Jenkins, Minio and S3www and loads all jobs in Jenkins.
Last components (MongoDB, TestAPI and dashboard) will be deployed in
the next update of the ansible role.

To deploy the testing toolchain:
$ ansible-galaxy install collivier.xtesting
$ ansible-playbook ansible/site.yml

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

Fix NEUTRON_RALLY_TAG in smoke

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

Remove duplicated code in vnfs

It mainly leverages on functest_utils instead of duplicating.

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

Remove Snaps NovaUtilsInstanceVolumeTests

It doesn't clean its resources and then breaks the gate after multiple
runs. It's the only way as Snaps is inactive and pinned in Functest.

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

Fix an invalid class name (vGPU)

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

Stop cleaning role

Functest may create default role if member doesn't exist.
It cannot be removed at the end of the testcase as it raises side
effects when testing in parallel.

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

Rename Rally blacklist.txt to blacklist.yaml

It eases editing it or verifyfing it via yamllint.

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

Fix cherry-pick mistakes

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

Switch to Clearwater Zamin (heat_ims)

Zamin is declared as the first "stable" release.
Cloudify descriptor is not ready for the update.

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

Protect vs build-args not consumed

BRANCH is useless in case of components which only inherits from tempest.
The build could simply fail depending on the docker version [1].

[1] https://build.opnfv.org/ci/job/opnfv-functest-components-latest-build/1/console

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

Add anchors in ci yaml files

It eases foctorizing between releng (1 multijob only) and functest
jjbs.

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

Check if at least 2 hypervisors (Shaker)

Shaker is now skipped if only one hypervisor.

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

Check if at least 1 hypervisors

Vmtp is skipped if only one hypervisor.

As Vtmp returns 0 when all is skipped, it protects vs issues when
generating charts.

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

Merge "Disable agent testing in tempest-based testcases" into stable/hunter

Remove snaps ansible test cases

They can't work vs all OPFV scenarios (MTU doesn't conform with Geneve)
and are deprecated. Updating snaps commit id [1] breaks all gates
closed to Gambia release.

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

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

Revert "Update Snaps commit id"

This reverts commit 68980ac76c433b1445e863bdedf3d0823bb66d63.

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

Disable agent testing in tempest-based testcases

It blacklists agent testing and two tests asking for VXLAN as overlay
if OVN (OVN supports GENEVE) [1]

blacklist.txt is renamed blacklist.yaml and is verified by yamllint.

[1] https://build.opnfv.org/ci/view/functest/job/functest-apex-baremetal-daily-gambia/33/console

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

Update Snaps commit id

It updates Snaps commit id to allow testing OVN [1]

It should be noted Snaps doesn't create any stable branch.
Then this patch can be cherry-picked asis in all branches.

It takes the package renaming into account.

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

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

Merge changes If9b61064,I0490eade into stable/hunter

* changes:
Increase Vmtp timeouts
Run vnfs in all scenarios

Increase Vmtp timeouts

It now conforms with other testcases booting ubuntu images.

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

Run vnfs in all scenarios

It allows fair comparisons between scenarios (e.g. ovs dpdk).
os-nosdn-calipso-noha could be verified VNFs as well [1].

[1] https://build.opnfv.org/ci/view/functest/job/functest-apex-baremetal-daily-gambia/32/console

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

Override tempest_conf.yaml in ci jobs

It also removes STORAGE_PROTOCOL (env var) which was removed.

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

Prepare OPNFV Functest releases

It sets version to 8.

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

Merge "Move tempest logics in tempest_conf.yaml" into stable/hunter

Update rally and rally-openstack tags

It switches from commit id to tag in case of rally-openstack.

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

Move tempest logics in tempest_conf.yaml

It allows removing OPNFV logics out of Functest (releng) and then
simplifies conf_utils.

tempest.conf is filled by the current default values.

functest-alpine.sh will be updated to allow modifying the default
values without breaking the OPNFV installers.

It deprecates the only tempest-related env var (STORAGE_PROTOCOL)

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

Temporarily disable Cinder volumes backup tests

From the time being, all OPNFV installers [1][2][3] don't support
Cinder volumes backup.

We will find a better way to handle the lack of OPNFV requiremements
(these tests pass succesfully in Orange Openlab).

At least, it will allow verifying tempest_full in OPNFV gates.

[1] Apex: https://artifacts.opnfv.org/logs/functest/lf-pod1/2018-10-25_02-17-25/tempest_full/tempest-report.html
[2] Compass: https://artifacts.opnfv.org/logs/functest/huawei-virtual9/2018-10-24_04-54-25/tempest_full/tempest-report.html
[3] Fuel: https://artifacts.opnfv.org/logs/functest/lf-pod2/gambia/2018-10-25_05-57-54/tempest_full/tempest-report.html

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

Temporarily disable VNC tests

At least Compass and Fuel configure Spice as console [1][2].
It disables vnc testing as it could fail for endusers as well.

[1] https://artifacts.opnfv.org/logs/functest/huawei-virtual9/2018-10-24_04-54-25/tempest_full/tempest-report.html
[2] https://artifacts.opnfv.org/logs/functest/lf-pod2/gambia/2018-10-25_05-57-54/tempest_full/tempest-report.html

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

Revert "Temporarily switch to collivier/abot_charm"

This reverts commit 204835a691869dc22031d207787f83a2a2937b0c.

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

Temporarily switch to collivier/abot_charm

It hosts a key fix which should be merged in the right repo.
This patch can be safely reverted as soon as [1] is merged.

[1] https://github.com/RebacaInc/abot_charm/pull/3

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

Disable Glance API v1

It has been deprecated since Newton [1].
Even if the related tests are skipped in latest tempest [2], API v1
has to be disabled in Gambia's tempest.conf [3].

[1] https://docs.openstack.org/glance/latest/user/glanceapi.html
[2] https://artifacts.opnfv.org/logs/functest/lf-pod1/2018-10-22_02-34-24/tempest_full/tempest-report.html
[3] https://artifacts.opnfv.org/logs/functest/lf-pod1/gambia/2018-10-23_10-34-07/tempest_full/tempest-report.html

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

Overprotect vmtp vs unversioned OS_AUTH_URL

It simply adds v3 to OS_AUTH_URL if required.
Vtmp should be updated to manage unversioned url [1].

https://build.opnfv.org/ci/view/functest/job/functest-apex-baremetal-daily-master/371/console

Co-Authored-By: Juha Kosonen <juha.kosonen@nokia.com>
Change-Id: I4806e811c0714b061b77f940934830cfb1835d74
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 5ac878fe7cccbfe5cb829488d0aedc307dd42885)

Harden Rally generate report

It converts html_file to str and raises all possible exceptions.
It sometimes fails when locally generating reports.

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

List neutron exts and service providers in connection_check

It eases debugging as well [1].

[1] https://artifacts.opnfv.org/logs/functest/lf-pod1/2018-10-20_12-23-41/patrole/tempest-report.html

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