summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrian Rosmaita <rosmaita.fossdev@gmail.com>2021-07-29 17:44:23 -0400
committerBrian Rosmaita <rosmaita.fossdev@gmail.com>2021-08-31 12:18:38 +0000
commitf7b35078861091edc3538429a436118b7ce30628 (patch)
tree1e7e716c11c1a4beae970b556948372a57305b5a
parentb568bfaf24b92a64ba02bcc4c801cc002e560290 (diff)
downloadpython-cinderclient-f7b35078861091edc3538429a436118b7ce30628.tar.gz
Unset tempest.lib timeout in functional tests
The test_cli.CinderBackupTests.test_backup_create_and_delete test is hitting timeout errors in the python-cinderclient-functional-py36 zuul job. This is happening because it's inheriting the OS_TEST_TIMEOUT value of 60 from the base testenv, and that value is being used by the tempest.lib class we inherit from as a timeout for each test. This is a problem for test_backup_create_and_delete because it creates a volume, waits for available, creates a backup, waits for available, deletes the volume, waits for deletion, deletes the backup, waits for deletion. Our functional tests have their own timeout handling, so turn off the tempest.lib timeout and use ours. An alternative to turning it off is to set it at a value that respects our timeout for our longest test, which would be: - time-to-available: 120 sec (x2) - time-to-deleted: 60 sec (x2) that is, 360 sec. Change-Id: I33399b4c094af2cc059da6e332f4c0a91e6ab57e (cherry picked from commit d04ded6a6ff4cc9ba8377a1b025493e939047b2c) (cherry picked from commit fa00aff41b266f46362b581ac48b4cec1692de71)
-rw-r--r--tox.ini5
1 files changed, 5 insertions, 0 deletions
diff --git a/tox.ini b/tox.ini
index fb1f64b..3956036 100644
--- a/tox.ini
+++ b/tox.ini
@@ -88,6 +88,11 @@ setenv =
OS_VOLUME_API_VERSION = 3
# must define this here so it can be inherited by the -py3* environments
OS_CINDERCLIENT_EXEC_DIR = {envdir}/bin
+ # Our functional tests contain their own timeout handling, so
+ # turn off the timeout handling provided by the
+ # tempest.lib.base.BaseTestCase that our ClientTestBase class
+ # inherits from.
+ OS_TEST_TIMEOUT=0
# The OS_CACERT environment variable should be passed to the test
# environments to specify a CA bundle file to use in verifying a