diff options
| author | Stephen Finucane <stephenfin@redhat.com> | 2020-05-18 17:25:35 +0100 |
|---|---|---|
| committer | Stephen Finucane <stephenfin@redhat.com> | 2020-05-27 09:40:47 +0000 |
| commit | 21fecc7060179ab62564db901ea932a1f723020e (patch) | |
| tree | 35d346df86666b326e892a1ecff5035227679b1e /nova/volume/cinder.py | |
| parent | d565e7a09281b803c47bc8eec3bac9c4959b4987 (diff) | |
| download | nova-21fecc7060179ab62564db901ea932a1f723020e.tar.gz | |
trivial: Remove log translations
We neither need nor want these translated so remove them. A couple of
logs with unnecessary brackets or weird indentation were identified in
the process, and these are also modified. Note that exceptions must
still be translated. Refer to [1] for more details.
[1] https://docs.openstack.org/oslo.i18n/latest/user/guidelines.html
Change-Id: I4573a7c5f3a7b5716a15fbd15ad9336807843a03
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Diffstat (limited to 'nova/volume/cinder.py')
| -rw-r--r-- | nova/volume/cinder.py | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/nova/volume/cinder.py b/nova/volume/cinder.py index 95a131e190..5bb56bf610 100644 --- a/nova/volume/cinder.py +++ b/nova/volume/cinder.py @@ -800,8 +800,8 @@ class API(object): # NOTE: It is unnecessary to output BadRequest(400) error log, # because operators don't need to debug such cases. if getattr(ex, 'code', None) != 400: - LOG.error(('Create attachment failed for volume ' - '%(volume_id)s. Error: %(msg)s Code: %(code)s'), + LOG.error('Create attachment failed for volume ' + '%(volume_id)s. Error: %(msg)s Code: %(code)s', {'volume_id': volume_id, 'msg': six.text_type(ex), 'code': getattr(ex, 'code', None)}, @@ -826,8 +826,8 @@ class API(object): return translated_attach_ref except cinder_exception.ClientException as ex: with excutils.save_and_reraise_exception(): - LOG.error(('Show attachment failed for attachment ' - '%(id)s. Error: %(msg)s Code: %(code)s'), + LOG.error('Show attachment failed for attachment ' + '%(id)s. Error: %(msg)s Code: %(code)s', {'id': attachment_id, 'msg': six.text_type(ex), 'code': getattr(ex, 'code', None)}) @@ -873,8 +873,8 @@ class API(object): return translated_attach_ref except cinder_exception.ClientException as ex: with excutils.save_and_reraise_exception(): - LOG.error(('Update attachment failed for attachment ' - '%(id)s. Error: %(msg)s Code: %(code)s'), + LOG.error('Update attachment failed for attachment ' + '%(id)s. Error: %(msg)s Code: %(code)s', {'id': attachment_id, 'msg': six.text_type(ex), 'code': getattr(ex, 'code', None)}) @@ -890,8 +890,8 @@ class API(object): attachment_id) except cinder_exception.ClientException as ex: with excutils.save_and_reraise_exception(): - LOG.error(('Delete attachment failed for attachment ' - '%(id)s. Error: %(msg)s Code: %(code)s'), + LOG.error('Delete attachment failed for attachment ' + '%(id)s. Error: %(msg)s Code: %(code)s', {'id': attachment_id, 'msg': six.text_type(ex), 'code': getattr(ex, 'code', None)}) @@ -913,8 +913,8 @@ class API(object): attachment_id) except cinder_exception.ClientException as ex: with excutils.save_and_reraise_exception(): - LOG.error(('Complete attachment failed for attachment ' - '%(id)s. Error: %(msg)s Code: %(code)s'), + LOG.error('Complete attachment failed for attachment ' + '%(id)s. Error: %(msg)s Code: %(code)s', {'id': attachment_id, 'msg': six.text_type(ex), 'code': getattr(ex, 'code', None)}) |