openstack/python-openstackclient
Code Issues Proposed changes

Merge "Handle log message interpolation by the logger in identity/"

Browse Source
This commit is contained in:
Jenkins 2017-02-14 08:23:21 +00:00 committed by Gerrit Code Review
commit 3b562ffa90
15 changed files with 24 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openstackclient/identity/v2_0/ec2creds.py
View File

@ -122,8 +122,8 @@ class DeleteEC2Creds(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete EC2 credentials with " LOG.error(_("Failed to delete EC2 credentials with "
"access key '%(access_key)s': %(e)s") "access key '%(access_key)s': %(e)s"),
% {'access_key': access_key, 'e': e}) {'access_key': access_key, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.access_keys) total = len(parsed_args.access_keys)

4
openstackclient/identity/v2_0/endpoint.py
View File

@ -102,8 +102,8 @@ class DeleteEndpoint(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete endpoint with " LOG.error(_("Failed to delete endpoint with "
"ID '%(endpoint)s': %(e)s") "ID '%(endpoint)s': %(e)s"),
% {'endpoint': endpoint, 'e': e}) {'endpoint': endpoint, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.endpoints) total = len(parsed_args.endpoints)

4
openstackclient/identity/v2_0/service.py
View File

@ -114,8 +114,8 @@ class DeleteService(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete service with " LOG.error(_("Failed to delete service with "
"name or ID '%(service)s': %(e)s") "name or ID '%(service)s': %(e)s"),
% {'service': service, 'e': e}) {'service': service, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.services) total = len(parsed_args.services)

3
openstackclient/identity/v3/consumer.py
View File

@ -73,8 +73,7 @@ class DeleteConsumer(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete consumer with name or " LOG.error(_("Failed to delete consumer with name or "
"ID '%(consumer)s': %(e)s") "ID '%(consumer)s': %(e)s"), {'consumer': i, 'e': e})
% {'consumer': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.consumer) total = len(parsed_args.consumer)

4
openstackclient/identity/v3/credential.py
View File

@ -99,8 +99,8 @@ class DeleteCredential(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete credentials with " LOG.error(_("Failed to delete credentials with "
"ID '%(credential)s': %(e)s") "ID '%(credential)s': %(e)s"),
% {'credential': i, 'e': e}) {'credential': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.credential) total = len(parsed_args.credential)

3
openstackclient/identity/v3/domain.py
View File

@ -111,8 +111,7 @@ class DeleteDomain(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete domain with name or " LOG.error(_("Failed to delete domain with name or "
"ID '%(domain)s': %(e)s") "ID '%(domain)s': %(e)s"), {'domain': i, 'e': e})
% {'domain': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.domain) total = len(parsed_args.domain)

4
openstackclient/identity/v3/ec2creds.py
View File

@ -140,8 +140,8 @@ class DeleteEC2Creds(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete EC2 credentials with " LOG.error(_("Failed to delete EC2 credentials with "
"access key '%(access_key)s': %(e)s") "access key '%(access_key)s': %(e)s"),
% {'access_key': i, 'e': e}) {'access_key': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.access_key) total = len(parsed_args.access_key)

3
openstackclient/identity/v3/endpoint.py
View File

@ -122,8 +122,7 @@ class DeleteEndpoint(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete endpoint with " LOG.error(_("Failed to delete endpoint with "
"ID '%(endpoint)s': %(e)s") "ID '%(endpoint)s': %(e)s"), {'endpoint': i, 'e': e})
% {'endpoint': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.endpoint) total = len(parsed_args.endpoint)

4
openstackclient/identity/v3/federation_protocol.py
View File

@ -102,8 +102,8 @@ class DeleteProtocol(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete federation protocol " LOG.error(_("Failed to delete federation protocol "
"with name or ID '%(protocol)s': %(e)s") "with name or ID '%(protocol)s': %(e)s"),
% {'protocol': i, 'e': e}) {'protocol': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.federation_protocol) total = len(parsed_args.federation_protocol)

4
openstackclient/identity/v3/identity_provider.py
View File

@ -115,8 +115,8 @@ class DeleteIdentityProvider(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete identity providers with " LOG.error(_("Failed to delete identity providers with "
"name or ID '%(provider)s': %(e)s") "name or ID '%(provider)s': %(e)s"),
% {'provider': i, 'e': e}) {'provider': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.identity_provider) total = len(parsed_args.identity_provider)

3
openstackclient/identity/v3/mapping.py
View File

@ -132,8 +132,7 @@ class DeleteMapping(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete mapping with name or " LOG.error(_("Failed to delete mapping with name or "
"ID '%(mapping)s': %(e)s") "ID '%(mapping)s': %(e)s"), {'mapping': i, 'e': e})
% {'mapping': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.mapping) total = len(parsed_args.mapping)

3
openstackclient/identity/v3/policy.py
View File

@ -82,8 +82,7 @@ class DeletePolicy(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete policy with name or " LOG.error(_("Failed to delete policy with name or "
"ID '%(policy)s': %(e)s") "ID '%(policy)s': %(e)s"), {'policy': i, 'e': e})
% {'policy': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.policy) total = len(parsed_args.policy)

3
openstackclient/identity/v3/region.py
View File

@ -87,8 +87,7 @@ class DeleteRegion(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete region with " LOG.error(_("Failed to delete region with "
"ID '%(region)s': %(e)s") "ID '%(region)s': %(e)s"), {'region': i, 'e': e})
% {'region': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.region) total = len(parsed_args.region)

4
openstackclient/identity/v3/service.py
View File

@ -103,8 +103,8 @@ class DeleteService(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete consumer with type, " LOG.error(_("Failed to delete consumer with type, "
"name or ID '%(service)s': %(e)s") "name or ID '%(service)s': %(e)s"),
% {'service': i, 'e': e}) {'service': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.service) total = len(parsed_args.service)

4
openstackclient/identity/v3/service_provider.py
View File

@ -108,8 +108,8 @@ class DeleteServiceProvider(command.Command):
except Exception as e: except Exception as e:
result += 1 result += 1
LOG.error(_("Failed to delete service provider with " LOG.error(_("Failed to delete service provider with "
"name or ID '%(provider)s': %(e)s") "name or ID '%(provider)s': %(e)s"),
% {'provider': i, 'e': e}) {'provider': i, 'e': e})
if result > 0: if result > 0:
total = len(parsed_args.service_provider) total = len(parsed_args.service_provider)