No PEP8 check for fuelweb_test

Bug #1261547 reported by Ivan Kolodyazhny
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Igor Shishkin

Bug Description

There are not any pylint/pep8 checks for fuelweb_test in https://github.com/stackforge/fuel-main/ repo for new review requests (commits)

Revision history for this message
Evgeniy L (rustyrobot) wrote :

Right now fuelweb_tests have 110 errors

$ flake8 --ignore=H302,H802 fuelweb_test/ | wc -l
110

Maybe we can use openstack infra jobs for such verification.

Changed in fuel:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → 4.0
Mike Scherbakov (mihgen)
tags: added: low-hanging-fruit
tags: added: system-tests
Changed in fuel:
milestone: 4.0 → 4.1
assignee: nobody → Matthew Mosesohn (raytrac3r)
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :
Changed in fuel:
assignee: Matthew Mosesohn (raytrac3r) → Igor Shishkin (teran)
Revision history for this message
Igor Shishkin (teran) wrote :

running flake8 is already added by Matthew https://review.openstack.org/#/c/64514/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/64514
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=f8c0ba860a97674b737390b39a37082316112b8c
Submitter: Jenkins
Branch: master

commit f8c0ba860a97674b737390b39a37082316112b8c
Author: Matthew Mosesohn <email address hidden>
Date: Tue Feb 4 13:49:56 2014 +0400

    flake8 fixes for fuelweb_test

    Added flake8 check to run_test.sh for CI checks
    flake8 check ignores temporary files created when
    running fuelweb_test.
    Change-Id: I4bf99b63b1c690c0d86f0a20bc86fe9ecd9ccfb2
    Closes-Bug: #1261547

Changed in fuel:
status: In Progress → Fix Committed
Igor Shishkin (teran)
Changed in fuel:
status: Fix Committed → Fix Released
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.