Fuel for OpenStack

pep8 updated case CI fails in stable OSTF branches

Bug #1428587 reported by Sebastian Kalinowski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Sebastian Kalinowski
Fuel for OpenStack 6.0-updates
5.1.x
Fix Released
Critical
Sebastian Kalinowski
Fuel for OpenStack 5.1.1-updates

Bug Description

New pep8 version introduced new checks that caused CI check to fail for patches like:

https://review.openstack.org/#/c/161512/
https://review.openstack.org/#/c/161233/

We don't want to fix them in stable branches since this is not critical. Solution is to pin pep8 version.

Tags: ostf
Sebastian Kalinowski (prmtl)
Changed in fuel:
milestone: 6.1 → 6.0.1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-ostf (stable/5.1)

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/161681

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ostf (stable/6.0)

Reviewed: https://review.openstack.org/161675
Committed: https://git.openstack.org/cgit/stackforge/fuel-ostf/commit/?id=b38332e6741fc4b0ef40a9b4fe9d2804ee6437cc
Submitter: Jenkins
Branch: stable/6.0

commit b38332e6741fc4b0ef40a9b4fe9d2804ee6437cc
Author: Sebastian Kalinowski <email address hidden>
Date: Thu Mar 5 11:41:43 2015 +0100

    Pin flake8 and pep8 version

    If falke8 and pep8 version will be allowed to update without control,
    it would cause future patchsets to fail CI check when new rules
    was added. It is a problem since we do not fix such things in stable
    branches.

    Closes-Bug: #1428587
    Change-Id: I94709d8f310e93b2cee617f3e4a43a64d22916b5

Bartłomiej Piotrowski (bpiotrowski)
Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ostf (stable/5.1)

Reviewed: https://review.openstack.org/161681
Committed: https://git.openstack.org/cgit/stackforge/fuel-ostf/commit/?id=3447260f7f718e3e85d130b2d20a348b3b68b99d
Submitter: Jenkins
Branch: stable/5.1

commit 3447260f7f718e3e85d130b2d20a348b3b68b99d
Author: Sebastian Kalinowski <email address hidden>
Date: Thu Mar 5 11:41:43 2015 +0100

    Pin flake8 and pep8 version

    If falke8 and pep8 version will be allowed to update without control,
    it would cause future patchsets to fail CI check when new rules
    was added. It is a problem since we do not fix such things in stable
    branches.

    Closes-Bug: #1428587
    Change-Id: I94709d8f310e93b2cee617f3e4a43a64d22916b5

Revision history for this message
Tatyanka (tatyana-leontovich) wrote :

verified, Thank you

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.
Subscribing...

Other bug subscribers

Remote bug watches

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