delta/openstack/python-openstackclient.git/openstackclient/tests/functional/volume/v2/common.py, branch 3.8.2 opendev.org: openstack/python-openstackclient volume functest: ensure snapshots deleted when volume delete 2017-09-06T09:15:21+00:00 Akihiro Motoki amotoki@gmail.com 2017-05-21T03:36:23+00:00 5eb85674668e46fd657f150c77239b6a980eb2f5 Deleting snapshot may take time. The current volume API does not allow to delete volumes with snapshots, so if deleting snapshot may take time, a delete request for a parent volume will fail. This sometimes causes functional test failures in slow environments. wait_for_status() checks whether volume status is in error statuses but previously the expected error status was wrong. Cinder API uses lower case as volume status, so it did not work expectedly. Conflicts: openstackclient/tests/functional/compute/v2/common.py openstackclient/tests/functional/compute/v2/test_server.py openstackclient/tests/functional/volume/v1/test_volume.py openstackclient/tests/functional/volume/v2/test_volume.py openstackclient/tests/functional/volume/v3/common.py Change-Id: I095894ba39f23bf81d71351818d24dbb5ca459fb (cherry picked from commit eeb614c47759fa9a01e6d886ed07acceb8d9ff61) Closes-Bug: #1714977 
Deleting snapshot may take time. The current volume API does not allow
to delete volumes with snapshots, so if deleting snapshot may take time,
a delete request for a parent volume will fail.
This sometimes causes functional test failures in slow environments.

wait_for_status() checks whether volume status is in error statuses
but previously the expected error status was wrong. Cinder API uses
lower case as volume status, so it did not work expectedly.

Conflicts:
  openstackclient/tests/functional/compute/v2/common.py
  openstackclient/tests/functional/compute/v2/test_server.py
  openstackclient/tests/functional/volume/v1/test_volume.py
  openstackclient/tests/functional/volume/v2/test_volume.py
  openstackclient/tests/functional/volume/v3/common.py

Change-Id: I095894ba39f23bf81d71351818d24dbb5ca459fb
(cherry picked from commit eeb614c47759fa9a01e6d886ed07acceb8d9ff61)
Closes-Bug: #1714977
move all functional tests to tests module 2016-09-09T03:29:47+00:00 Steve Martinelli s.martinelli@gmail.com 2016-09-06T03:35:06+00:00 c14d3efe6162a58cb3cdcb2834ad2508e2525018 functional tests should be grouped with other tests (unit and integration tests). as part of this commit the "common" module was renamed to just "base", this was done for simplicity. the post_test_hook.sh file was also copied to the functional module since it should live there. a separate change to the infra repo will be made to call the new location, once that is merged we can remove the old one (a new change will also be posted for that) Needed-By: I49d54f009021d65c1ae49faf6b3f0a7acdadd7b3 Change-Id: Ie8c334f6223373b8e06df8bd8466500d2a2c8ede 
functional tests should be grouped with other tests (unit and
integration tests). as part of this commit the "common" module
was renamed to just "base", this was done for simplicity.

the post_test_hook.sh file was also copied to the functional module
since it should live there. a separate change to the infra repo
will be made to call the new location, once that is merged we
can remove the old one (a new change will also be posted for that)

Needed-By: I49d54f009021d65c1ae49faf6b3f0a7acdadd7b3
Change-Id: Ie8c334f6223373b8e06df8bd8466500d2a2c8ede