Brick's iser.py uses locals()

Bug #1209006 reported by Walt Boring on 2013-08-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Low
Walt Boring

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

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

Changed in cinder:
status: New → In Progress

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) on 2013-09-05
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers