Cinder

Brick's iser.py uses locals()

Bug #1209006 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Walt Boring
Cinder 2013.2 "havana"

Bug Description

Running flake8 against brick, and the iser.py uses locals()

cinder/brick/iser/iser.py:170:36: H501 Do not use locals() for string formatting

Tags: brick iser
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/40507
Committed: http://github.com/openstack/cinder/commit/9ca3fc0c0906c2c04425c50ac848e5e04aa723f9
Submitter: Jenkins
Branch: master

commit 9ca3fc0c0906c2c04425c50ac848e5e04aa723f9
Author: Walter A. Boring IV <email address hidden>
Date: Tue Aug 6 14:43:35 2013 -0700

    Remove locals() from iser

    This patch removes the locals() call
    to fix a flake8 violation.

    Fixes Bug #1209006

    Change-Id: Ida4b498544a2c28ffb4912d402579f8bb7372309

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.