← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1596829] Re: String interpolation should be delayed at logging calls

 

** Also affects: python-cinderclient
   Importance: Undecided
       Status: New

** Changed in: python-cinderclient
     Assignee: (unassigned) => haobing1 (haobing1)

** Also affects: python-glanceclient
   Importance: Undecided
       Status: New

** Changed in: python-glanceclient
     Assignee: (unassigned) => haobing1 (haobing1)

** Also affects: glance-store
   Importance: Undecided
       Status: New

** Changed in: glance-store
     Assignee: (unassigned) => haobing1 (haobing1)

** Also affects: ceilometer
   Importance: Undecided
       Status: New

** Changed in: ceilometer
     Assignee: (unassigned) => haobing1 (haobing1)

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

Title:
  String interpolation should be delayed at logging calls

Status in Ceilometer:
  New
Status in Cinder:
  New
Status in Glance:
  New
Status in glance_store:
  New
Status in neutron:
  In Progress
Status in OpenStack Compute (nova):
  In Progress
Status in os-brick:
  New
Status in python-cinderclient:
  New
Status in python-glanceclient:
  New

Bug description:
  String interpolation should be delayed to be handled by the logging
  code, rather than being done at the point of the logging call.

  Wrong: LOG.debug('Example: %s' % 'bad')
  Right: LOG.debug('Example: %s', 'good')

  See the following guideline.

  * http://docs.openstack.org/developer/oslo.i18n/guidelines.html
  #adding-variables-to-log-messages

  The rule for it should be added to hacking checks.

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


References