diff options
| author |   Steven Pisarski <s.pisarski@cablelabs.com> | 2018-02-01 14:33:22 +0000 |
|---|---|---|
| committer |   Gerrit Code Review <gerrit@opnfv.org> | 2018-02-01 14:33:22 +0000 |
| commit | 97c6c9550b522edb3eaa1875fe6d6c46aa105f1d (patch) | |
| tree | 6e1ac4e98ab92c61ff01615f09f5fb90f7fb8710 /.gitreview | |
| parent | 2bfa69008e45748fd7dd692562d5a123883e1879 (diff) | |
Update git submodules
* Update docs/submodules/snaps from branch 'master'
- Merge "openstack: nova_utils_tests: Use API timeout argument instead of sleep()"
- openstack: nova_utils_tests: Use API timeout argument instead of sleep()
On slow environments, time.sleep(10) is not enough for attach/detach API
calls to propage properly leading to failures like the following one
(reduced api_check test)
2018-01-31 18:53:19,297 - functest.ci.run_tests - INFO - Running test case 'api_check'...
2018-01-31 18:54:27,434 - functest.ci.run_tests - INFO - Test result:
+-------------------+------------------+------------------+----------------+
| TEST CASE | PROJECT | DURATION | RESULT |
+-------------------+------------------+------------------+----------------+
| api_check | functest | 01:07 | FAIL |
+-------------------+------------------+------------------+----------------+
We can workaround this by using the nova_utils.{de,at}tach_volume}
'timeout' argument to wait up to 2 minutes for the attach/detach call to
complete. This doesn't affect normal exception whilst it helps get the
test going on slow systems. WIth this change, the api_check (reduced
test) passes as follows:
2018-01-31 18:48:35,469 - functest.ci.run_tests - INFO - Running test case 'api_check'...
2018-01-31 18:50:56,705 - functest.ci.run_tests - INFO - Test result:
+-------------------+------------------+------------------+----------------+
| TEST CASE | PROJECT | DURATION | RESULT |
+-------------------+------------------+------------------+----------------+
| api_check | functest | 02:20 | PASS |
+-------------------+------------------+------------------+----------------+
Change-Id: I2245adb84d0e6d4e3350d17d0c2e44baf5202d51
Signed-off-by: Markos Chandras <mchandras@suse.de>
Diffstat (limited to '.gitreview')
0 files changed, 0 insertions, 0 deletions