Cinder does have a hacking check ensuring that [1]. [1] https://github.com/openstack/cinder/blob/c96be71fd66aa9c2a977e6c0288c69a3d42c3b6a/cinder/hacking/checks.py#L225-L287
** No longer affects: cinder -- 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 Glance: New Status in glance_store: New Status in heat: New Status in Ironic: New Status in OpenStack Identity (keystone): New Status in neutron: Fix Released Status in OpenStack Compute (nova): In Progress Status in os-brick: In Progress Status in python-cinderclient: New Status in python-glanceclient: New Status in OpenStack Object Storage (swift): New Status in taskflow: 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 -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : [email protected] Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp
