yahoo-eng-team team mailing list archive

[OpenStack Infra <1600788@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/342616
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=ec729d6cc686f0af5c79cfd24cac99fc3de215fd
Submitter: Jenkins
Branch:    master

commit ec729d6cc686f0af5c79cfd24cac99fc3de215fd
Author: weiweigu <gu.weiwei@xxxxxxxxxx>
Date:   Fri Jul 15 15:26:02 2016 +0800

    Replace "LOG.info(_" with "LOG.info(_LI"
    
    Follow http://docs.openstack.org/developer/oslo.i18n/guidelines.html:
    If a common message is not being used, they should each betreated
    separately with respect to choosing a marker function.So this patch
    is to fix it.
    
    Change-Id: If7a74ce983ad0c7f4cab057cadd7d20ef3067fcd
    Closes-Bug: #1600788


** Changed in: magnum
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1600788

Title:
  If a common message is not being used, they should each be treated
  separately with respect to choosing a marker function

Status in Ceilometer:
  In Progress
Status in Glance:
  Fix Released
Status in glance_store:
  In Progress
Status in OpenStack Dashboard (Horizon):
  New
Status in Magnum:
  Fix Released
Status in neutron:
  Triaged
Status in oslo.log:
  Fix Released
Status in Solum:
  New

Bug description:
  Follow the
  http://docs.openstack.org/developer/oslo.i18n/guidelines.html

  For example, do not do this:

  # WRONG
  LOG.exception(_('There was an error.'))
  raise LocalExceptionClass(_('An error occured.'))
  Instead, use this style:

  # RIGHT
  LOG.exception(_LE('There was an error.'))
  raise LocalExceptionClass(_('An error occured.'))

  And oslo.log has the problem,we shuld correct it.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ceilometer/+bug/1600788/+subscriptions