]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Logging not using oslo.i18n guidelines
authorSean McGinnis <sean_mcginnis@dell.com>
Fri, 20 Mar 2015 23:42:50 +0000 (18:42 -0500)
committerSean McGinnis <sean_mcginnis@dell.com>
Tue, 31 Mar 2015 03:06:54 +0000 (22:06 -0500)
commite1ef8aed8a2cb8680cb13ba8f6e4d1dc93899145
treea5ebbf50b36459304b3579a2ffd6c6c005bad429
parentd4dd0d5f30106b2dbe4552e52ebd2ab2d8e0ecfe
Logging not using oslo.i18n guidelines

Multi-patch set for easier chunks. This one addresses
the backup and common cinder directories.

There have been quite a few instances found where the
i18n guidelines are not being followed. I believe this
has helped lead to some of the confusion around how to
correctly do this. Other developers see this code and
assume it is an example of the correct usage.

This patch attempts to clean up most of those violations
in the existing codebase to hopefully help avoid some of
that confusion in reviews.

Some issues address:
* Correct log translation markers for different log levels
* Passing format values as arguments to call, not preformatting
* Not forcing translation via six.text_type and others

Guidelines can be found here:
http://docs.openstack.org/developer/oslo.i18n/guidelines.html

Hacking checks will not be able to identify all violations of
the guidelines, but it could be useful for catching obvious ones
such as LOG.info("No markers!").

Change-Id: I2560d9dee564f5a9b214fd63ccd2600d78a30464
Partial-bug: 1433216
cinder/backup/api.py
cinder/backup/chunkeddriver.py
cinder/backup/driver.py
cinder/backup/drivers/ceph.py
cinder/backup/drivers/tsm.py
cinder/backup/manager.py
cinder/backup/rpcapi.py
cinder/common/sqlalchemyutils.py
cinder/hacking/checks.py