brick has some pep8 violations that prevent clean tests in nova

Bug #1211935 reported by Walt Boring on 2013-08-13
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Medium
Walt Boring

Bug Description

Cinder's brick has a few small pep8 violations that aren't caught/thrown/shown in flake8 tests in cinder. Copying brick into nova and running the flake8 tests show some violations.

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

Changed in cinder:
status: New → In Progress

Reviewed: https://review.openstack.org/41766
Committed: http://github.com/openstack/cinder/commit/7c899c9b7750fe822a47496243ae075f9d3dc51b
Submitter: Jenkins
Branch: master

commit 7c899c9b7750fe822a47496243ae075f9d3dc51b
Author: Walter A. Boring IV <email address hidden>
Date: Tue Aug 13 11:53:06 2013 -0700

    Fixes some unseen flake8 violations.

    The failures aren't visible when run from
    cinder. Nova has different settings for
    flake8. This patch fixes the violations
    that nova's flake8 tests.

    fixes Bug #1211935

    Change-Id: I80209130de23842a0a35cacb7d59f166e7dba89f

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