Skip to content

Commit c3acc41

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Ignore domain related config when using with keystone v2"
2 parents 3f5fd2f + b843240 commit c3acc41

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

openstackclient/common/clientmanager.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -200,12 +200,12 @@ def setup_auth(self, required_scope=True):
200200
# ignore all domain related configs.
201201
if (self._api_version.get('identity') == '2.0' and
202202
self.auth_plugin_name.endswith('password')):
203-
LOG.warning("Ignoring domain related configs "
204-
"because identity API version is 2.0")
205203
domain_props = ['project_domain_name', 'project_domain_id',
206204
'user_domain_name', 'user_domain_id']
207205
for prop in domain_props:
208-
self._auth_params.pop(prop, None)
206+
if self._auth_params.pop(prop, None) is not None:
207+
LOG.warning("Ignoring domain related configs " +
208+
prop + " because identity API version is 2.0")
209209

210210
# For compatibility until all clients can be updated
211211
if 'project_name' in self._auth_params:

0 commit comments

Comments
 (0)