openstack/tempest
Code Issues Proposed changes

Remove deprecated allow_tenant_isolation option

Browse Source 
allow_tenant_isolation option of auth and compute groups has been
deprecated over 3 years since I78a4884e980ef7d0103639cb3792a54c69fb7761.
So we have spended much enough time for removing them, this patch does.

Change-Id: I0b0a5e16a5e3785579f95f75fa149b311d918e8c
This commit is contained in:
Ken'ichi Ohmichi 2018-07-13 22:53:03 +00:00
parent 8626ab7d47
commit 553d7cbddc
4 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
releasenotes/notes/remove-allow_tenant_isolation-option-03f0d998eb498d44.yaml Normal file
View File

@ -0,0 +1,6 @@
---
upgrade:
- |
Remove deprecated config option ``allow_tenant_isolation`` from
``auth`` and ``compute`` groups. Use ``use_dynamic_credentials`` directly
instead of the removed option.

2
tempest/api/identity/v2/test_users.py
View File

@ -84,7 +84,7 @@ class IdentityUsersTest(base.BaseIdentityV2Test):
new_pass = data_utils.rand_password() new_pass = data_utils.rand_password()
user_id = self.creds.user_id user_id = self.creds.user_id
# to change password back. important for allow_tenant_isolation = false # to change password back. important for use_dynamic_credentials=false
self.addCleanup(self._restore_password, user_id, old_pass, new_pass) self.addCleanup(self._restore_password, user_id, old_pass, new_pass)
# user updates own password # user updates own password

2
tempest/api/identity/v3/test_users.py
View File

@ -82,7 +82,7 @@ class IdentityV3UsersTest(base.BaseIdentityV3Test):
old_token = self.non_admin_client.token old_token = self.non_admin_client.token
new_pass = data_utils.rand_password() new_pass = data_utils.rand_password()
# to change password back. important for allow_tenant_isolation = false # to change password back. important for use_dynamic_credentials=false
self.addCleanup(self._restore_password, old_pass, new_pass) self.addCleanup(self._restore_password, old_pass, new_pass)
# user updates own password # user updates own password

6
tempest/config.py
View File

@ -61,11 +61,7 @@ AuthGroup = [
"users. This option requires that OpenStack Identity " "users. This option requires that OpenStack Identity "
"API admin credentials are known. If false, isolated " "API admin credentials are known. If false, isolated "
"test cases and parallel execution, can still be " "test cases and parallel execution, can still be "
"achieved configuring a list of test accounts", "achieved configuring a list of test accounts"),
deprecated_opts=[cfg.DeprecatedOpt('allow_tenant_isolation',
group='auth'),
cfg.DeprecatedOpt('allow_tenant_isolation',
group='compute')]),
cfg.ListOpt('tempest_roles', cfg.ListOpt('tempest_roles',
help="Roles to assign to all users created by tempest", help="Roles to assign to all users created by tempest",
default=[]), default=[]),