]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Logging not using oslo.i18n guidelines (brick)
authorSean McGinnis <sean_mcginnis@dell.com>
Thu, 9 Apr 2015 15:32:03 +0000 (10:32 -0500)
committerSean McGinnis <sean_mcginnis@dell.com>
Thu, 9 Apr 2015 15:32:03 +0000 (10:32 -0500)
commit976cfe77d1de86be1566f2e4b96f8f8d9b8e0008
treeb54289a3bdce166752ac94399ec13ece75e42192
parentc7ca4b95b56539dd560dc88038ab994d50c8394d
Logging not using oslo.i18n guidelines (brick)

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

Updates have already been made to the os-brick project.

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: I993a0a567383dfe6fca13dfcc76758ea3a46e1dd
Partial-bug: 1433216
cinder/brick/exception.py
cinder/brick/initiator/connector.py
cinder/brick/initiator/linuxfc.py
cinder/brick/initiator/linuxscsi.py
cinder/brick/local_dev/lvm.py
cinder/brick/remotefs/remotefs.py
cinder/hacking/checks.py