brick has some pep8 violations that prevent clean tests in nova

Bug #1211935 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
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.

Tags: brick
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/41766

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/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)
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.

Other bug subscribers

Remote bug watches

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