openstack/oslo.concurrency
Code Issues Proposed changes

Remove log translations

Browse Source 
Log messages are no longer being translated.
This removes all use of the _LE, _LI, and _LW
translation markers to simplify logging and to
avoid confusion with new contributions.

See:
http://lists.openstack.org/pipermail/openstack-i18n/2016-November/002574.html
http://lists.openstack.org/pipermail/openstack-dev/2017-March/113365.html

Change-Id: Iec2ef31f8543d51c8b6e1aed53f950800813c30b
This commit is contained in:
loooosy 2017-03-24 14:44:22 +08:00 committed by ChangBo Guo(gcb)
parent ef7bfd5771
commit f4b6622407
2 changed files with 2 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
oslo_concurrency/_i18n.py
View File

@ -20,13 +20,3 @@ _translators = oslo_i18n.TranslatorFactory(domain='oslo_concurrency')
# The primary translation function using the well-known name "_" # The primary translation function using the well-known name "_"
_ = _translators.primary _ = _translators.primary
# Translators for log levels.
#
# The abbreviated names are meant to reflect the usual use of a short
# name like '_'. The "L" is for "log" and the other letter comes from
# the level.
_LI = _translators.log_info
_LW = _translators.log_warning
_LE = _translators.log_error
_LC = _translators.log_critical

4
oslo_concurrency/lockutils.py
View File

@ -30,7 +30,7 @@ from oslo_utils import reflection
from oslo_utils import timeutils from oslo_utils import timeutils
import six import six
from oslo_concurrency._i18n import _, _LI from oslo_concurrency._i18n import _
LOG = logging.getLogger(__name__) LOG = logging.getLogger(__name__)
@ -158,7 +158,7 @@ def remove_external_lock_file(name, lock_file_prefix=None, lock_path=None,
try: try:
os.remove(lock_file_path) os.remove(lock_file_path)
except OSError: except OSError:
LOG.info(_LI('Failed to remove file %(file)s'), LOG.info('Failed to remove file %(file)s',
{'file': lock_file_path}) {'file': lock_file_path})