remove flake8 option in run_tests.sh

Bug #1212988 reported by Kui Shi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Undecided
Kui Shi

Bug Description

flake8 will read tox.ini as configuration, there is no need to keep the command line option in run_flake8 function.

We should keep the function run_flake8 simple, and change the [falke8] section of tox.ini to change the behavior.

Kui Shi (skuicloud)
Changed in glance:
assignee: nobody → Kui Shi (skuicloud)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

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

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

Reviewed: https://review.openstack.org/42327
Committed: http://github.com/openstack/glance/commit/63f7995c07b16f05e07929f1335de1b87f38c1b9
Submitter: Jenkins
Branch: master

commit 63f7995c07b16f05e07929f1335de1b87f38c1b9
Author: Kui Shi <email address hidden>
Date: Fri Aug 16 06:53:46 2013 +0800

    remove flake8 option in run_tests.sh

    flake8 read [flake8] section of tox.ini as options.

    function run_flake8 in run_tests.sh should be kept simple,
    and tox.ini controls the behavior of flake8.

    Fixes Bug #1212988

    Change-Id: I35d434ccea1ca2f3ff93657e4ca6dde93bbbf29e

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
milestone: none → havana-3
status: Fix Committed → Fix Released
Revision history for this message
Yang Yu (yuyangbj) wrote : I will be on vacation from 9/5 to 9/15, urgent call: 13811509950

I will be out of the office starting 2013-09-05 and will not return until
2013-09-15.

I will be on my marriage leave from 9/5 to 9/15, for any urgent issue
please call me before 9/7.

For daily work, please ask my scrum master Zhu Zhu for help.
For glance issue, please ask glance SME Feilong Wang for help.
For defect report, there will be no report next week.

Thierry Carrez (ttx)
Changed in glance:
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.