no conversion type in LOG.debug string

Bug #1220557 reported by Kiyohiro Adachi on 2013-09-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Kun Huang

Bug Description

method extend_volume in cinder/volume/drivers/rbd.py

> LOG.debug(_("Extend volume from %(old_size) to %(new_size)"),

Convert type 's' is required.

method extend_volume in cinder/volume/drivers/sheepdog.py

> LOG.debug(_("Extend volume from %(old_size) to %(new_size)"),

Convert type 's' is required.

Changed in cinder:
assignee: nobody → Kun Huang (academicgareth)

Fix proposed to branch: master
Review: https://review.openstack.org/45031

Changed in cinder:
status: New → In Progress

Reviewed: https://review.openstack.org/45031
Committed: http://github.com/openstack/cinder/commit/6956065f0b336c0d4edeba265fae8c9f7cddb736
Submitter: Jenkins
Branch: master

commit 6956065f0b336c0d4edeba265fae8c9f7cddb736
Author: Kun Huang <email address hidden>
Date: Wed Sep 4 18:42:16 2013 +0800

    fix log string in conversion type

    By using 'grep -rn "%([a-zA-Z_]*) "', we could find all cases of
    '%(variable)'. And we need add an 's' like '%(variable)s'.

    Closes-Bug: #1220557
    Change-Id: Ib28b7ddc5e5aabe2b82a499d9c242c1982e7f953

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-10-04
Changed in cinder:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-rc1 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers