Fuel for OpenStack

Merging to stable branches of fuel-web is blocked by failing pep8 checks on openstack-infra CI

Bug #1511787 reported by Igor Belikov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Igor Belikov
Fuel for OpenStack 8.0
5.0.x
Fix Committed
Critical
MOS Maintenance
Fuel for OpenStack 5.0-updates
5.1.x
Won't Fix
Critical
MOS Maintenance
Fuel for OpenStack 5.1.1-updates
6.0.x
Won't Fix
Critical
MOS Maintenance
Fuel for OpenStack 6.0-updates
6.1.x
Fix Released
Critical
MOS Maintenance
Fuel for OpenStack 6.1-updates
7.0.x
Fix Committed
Critical
Igor Belikov
Fuel for OpenStack 7.0-updates

Bug Description

After the merge of https://review.openstack.org/#/c/240440 voting pep8 checks are enabled for stable branches of fuel-web. These checks are currently failing, which leaves stable branches of fuel-web in a blocked state.

Alexey Shtokolov (ashtokolov)
Changed in fuel:
status: New → Confirmed
Revision history for this message
Igor Belikov (ibelikov) wrote :

https://review.openstack.org/#/c/240575/

Changed in fuel:
status: Confirmed → In Progress
Dmitry Pyzhov (dpyzhov)
tags: added: regression-8.0
tags: added: tech-debt
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/7.0)

Related fix proposed to branch: stable/7.0
Review: https://review.openstack.org/240864

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/6.1)

Related fix proposed to branch: stable/6.1
Review: https://review.openstack.org/240871

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/6.0)

Related fix proposed to branch: stable/6.0
Review: https://review.openstack.org/240873

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/5.1)

Related fix proposed to branch: stable/5.1
Review: https://review.openstack.org/240878

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/5.0)

Related fix proposed to branch: stable/5.0
Review: https://review.openstack.org/240886

Revision history for this message
Alexey Shtokolov (ashtokolov) wrote :

Patches for 7.0 and 5.0 are in progress:
https://review.openstack.org/240864
https://review.openstack.org/240886

Patches for 6.1 6.0 5.1 are blocked (bug https://bugs.launchpad.net/fuel/+bug/1512448 ) :
https://review.openstack.org/240871
https://review.openstack.org/240873
https://review.openstack.org/240878

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-web (stable/5.0)

Reviewed: https://review.openstack.org/240886
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=435b4bb2ef920356d996d8b05bc6f4de6a6b2259
Submitter: Jenkins
Branch: stable/5.0

commit 435b4bb2ef920356d996d8b05bc6f4de6a6b2259
Author: Igor Belikov <email address hidden>
Date: Mon Nov 2 17:43:25 2015 +0300

    Add tox.ini with pep8 tests to fix CI

    * Add tox.ini with pep8 tests so 'gate-fuel-web-pep8' is able to
      actually run
    * Ignore all existing pep8 errors and warnings to unblock merging in
      stable/5.0, but prevent any more pep8 errors and warnings
    * Add .tox to .gitignore

    Change-Id: Iad59da7f8e5af901e7c9fa830f3a6ebe81d021b3
    Related-Bug: #1511787

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-web (stable/7.0)

Reviewed: https://review.openstack.org/240864
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=25946c3323b7842c44f4173de80c0a8d83900302
Submitter: Jenkins
Branch: stable/7.0

commit 25946c3323b7842c44f4173de80c0a8d83900302
Author: Igor Belikov <email address hidden>
Date: Mon Nov 2 16:45:17 2015 +0300

    Add tox.ini with pep8 tests to fix CI

    * Add tox.ini with pep8 tests so 'gate-fuel-web-pep8' is able to
      actually run
    * Fix existing pep8 warnings

    Change-Id: Iad59da7f8e5af901e7c9fa830f3a6ebe81d021b3
    Related-Bug: #1511787

Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Revision history for this message
Alexey Shtokolov (ashtokolov) wrote :

Assigned to MOS-Maintenance, because patches for 6.1 6.0 5.1 are blocked by bug https://bugs.launchpad.net/fuel/+bug/1512448 :
https://review.openstack.org/240871
https://review.openstack.org/240873
https://review.openstack.org/240878

Only Maintenance Team can fix and merge to stable branches.

Changed in fuel:
assignee: Alexey Shtokolov (ashtokolov) → MOS Maintenance (mos-maintenance)
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
assignee: MOS Maintenance (mos-maintenance) → Igor Belikov (ibelikov)
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-web (stable/6.1)

Reviewed: https://review.openstack.org/240871
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=521237cf7a922e533c3895035a97071d1051f304
Submitter: Jenkins
Branch: stable/6.1

commit 521237cf7a922e533c3895035a97071d1051f304
Author: Igor Belikov <email address hidden>
Date: Mon Nov 2 17:12:48 2015 +0300

    Add tox.ini with pep8 tests to fix CI

    * Add tox.ini with pep8 tests so 'gate-fuel-web-pep8' is able to
      actually run
    * Ignore all existing pep8 errors and warnings to unblock merging in
      stable/6.1, but prevent any more pep8 errors and warnings

    Change-Id: Iad59da7f8e5af901e7c9fa830f3a6ebe81d021b3
    Related-Bug: #1511787

Revision history for this message
Denis Meltsaykin (dmeltsaykin) wrote :

Setting as Won't Fix for 6.0 and 5.1 updates as these branches have no delivery channel for fixes.

Aleksandra Fedorova (bookwar)
Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/4.1)

Related fix proposed to branch: stable/4.1
Review: https://review.openstack.org/262541

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (stable/4.1)

Change abandoned by Fuel DevOps Robot (<email address hidden>) on branch: stable/4.1
Review: https://review.openstack.org/262541
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (stable/5.1)

Change abandoned by Tony Breeds (<email address hidden>) on branch: stable/5.1
Review: https://review.openstack.org/240878
Reason: This branch (stable/5.1) is at End Of Life

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (stable/6.0)

Change abandoned by Tony Breeds (<email address hidden>) on branch: stable/6.0
Review: https://review.openstack.org/240873
Reason: This branch (stable/6.0) is at End Of Life

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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