| commit | 23923f0eba8f79b2a7015dd97b97323c1ab1db02 | [log] [tgz] |
|---|---|---|
| author | Felipe Monteiro <felipe.monteiro@att.com> | Fri Mar 17 02:15:07 2017 +0000 |
| committer | Felipe Monteiro <felipe.monteiro@att.com> | Fri Mar 17 02:15:07 2017 +0000 |
| tree | 33be6a62b9dbc19a35bb49673ae97ab484120498 | |
| parent | e83a257b06a79ff3fc8f8d857f3a7e24728ca118 [diff] |
Partially revert removal of time.sleep if v3 auth enabled in conf. In a previous patch [0], an attempt was made to speed up Patrole tests by skipping over time.sleep(1) if v3 auth was enabled in tempest.conf. However, this resulted in more tests failing than otherwise should be for non-admin roles (i.e. Member role). This is because: "Fernet tokens are not subsecond aware and Keystone should only be precise to the second. Sleep to ensure we are passing the second boundary before attempting to authenticate." [1][2] This patch guarantees that time.sleep(1) is always executed, regardless of identity auth version used by Tempest. While this prolongs the test runs in the gates, this change is necessary to guarantee correct test execution for non-admin roles. [0] https://review.openstack.org/#/c/444967/ [1] https://github.com/openstack/tempest/blob/master/tempest/api/identity/v2/test_users.py [2] https://github.com/openstack/tempest/blob/master/tempest/api/identity/v3/test_users.py Change-Id: I007190906e710f540ed0e44e76bd17047fcddea4 Closes-Bug: #1673497