openstack/osc-lib
Code Issues Proposed changes

Replace string format arguments with function parameters

Browse Source 
There are files containing string format arguments inside logging messages.
Using logging function parameters should be preferred.

Change-Id: I15b405bf4d4715263fe1e1262982467b3d4bc1f4
Closes-Bug: #1321274
This commit is contained in:
ting.wang 2016-02-20 14:09:40 +08:00
parent fae6c4c7bd
commit d3b24007b8
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openstackclient/api/auth.py
View File

@ -106,7 +106,7 @@ def select_auth_plugin(options):
# The ultimate default is similar to the original behaviour,
# but this time with version discovery
auth_plugin_name = 'osc_password'
LOG.debug("Auth plugin %s selected" % auth_plugin_name)
LOG.debug("Auth plugin %s selected", auth_plugin_name)
return auth_plugin_name
@ -130,7 +130,7 @@ def build_auth_params(auth_plugin_name, cmd_options):
auth_plugin_class = None
plugin_options = set([o.replace('-', '_') for o in get_options_list()])
for option in plugin_options:
LOG.debug('fetching option %s' % option)
LOG.debug('fetching option %s', option)
auth_params[option] = getattr(cmd_options.auth, option, None)
return (auth_plugin_class, auth_params)

4
openstackclient/common/clientmanager.py
View File

@ -167,8 +167,8 @@ class ClientManager(object):
elif 'tenant_name' in self._auth_params:
self._project_name = self._auth_params['tenant_name']
LOG.info('Using auth plugin: %s' % self.auth_plugin_name)
LOG.debug('Using parameters %s' %
LOG.info('Using auth plugin: %s', self.auth_plugin_name)
LOG.debug('Using parameters %s',
strutils.mask_password(self._auth_params))
self.auth = auth_plugin.load_from_options(**self._auth_params)
# needed by SAML authentication