stx/integ pylint failure with new warning

Bug #1893988 reported by Don Penney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Critical
Don Penney

Bug Description

Brief Description
-----------------
The pylint tox/zuul job in starlingx/integ is currently failing due to a new warning added in the most recent pylint release:

************* Module storage_topology.exec.storage_topology
tools/storage-topology/storage-topology/storage_topology/exec/storage_topology.py:175:8: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)

Severity
--------
Major: Designers cannot merge reviews, as the Zuul job fails

Steps to Reproduce
------------------
tox -e pylint

Expected Behavior
------------------
tox should pass with no failures

Actual Behavior
----------------
Fails with new warning

Reproducibility
---------------
Reproducible

Branch/Pull Time/Commit
-----------------------
starlingx/master, as of Sept 2, 2020
Impacts all release branches, as well

Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Critical
assignee: nobody → Don Penney (dpenney)
tags: added: stx.3.0 stx.4.0 stx.5.0 stx.zuul
Revision history for this message
Ghada Khalil (gkhalil) wrote :

marking as critical for all supported releases as this prevents code merges in the integ repo

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

Reviewed: https://review.opendev.org/749578
Committed: https://git.openstack.org/cgit/starlingx/integ/commit/?id=a944973da4f0a18e06ee26b5132d03e007e3e1d3
Submitter: Zuul
Branch: master

commit a944973da4f0a18e06ee26b5132d03e007e3e1d3
Author: Don Penney <email address hidden>
Date: Wed Sep 2 15:16:41 2020 -0400

    Filter new pylint warning W0707

    A new warning introduced in pylint 2.6, W0707 raise-missing-from, is
    causing tox and zuul pylint tests to fail. This update filters the new
    warning.

    Change-Id: Ifa4daf575d3b993d2fe65d895e5c0cd40a0cbcae
    Closes-Bug: 1893988
    Signed-off-by: Don Penney <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to integ (r/stx.4.0)

Fix proposed to branch: r/stx.4.0
Review: https://review.opendev.org/749588

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to integ (r/stx.3.0)

Fix proposed to branch: r/stx.3.0
Review: https://review.opendev.org/749589

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to integ (r/stx.3.0)

Reviewed: https://review.opendev.org/749589
Committed: https://git.openstack.org/cgit/starlingx/integ/commit/?id=320eca1799a944441036a0336c1c45593daf2eba
Submitter: Zuul
Branch: r/stx.3.0

commit 320eca1799a944441036a0336c1c45593daf2eba
Author: Don Penney <email address hidden>
Date: Wed Sep 2 15:16:41 2020 -0400

    Filter new pylint warning W0707

    A new warning introduced in pylint 2.6, W0707 raise-missing-from, is
    causing tox and zuul pylint tests to fail. This update filters the new
    warning.

    Change-Id: Ifa4daf575d3b993d2fe65d895e5c0cd40a0cbcae
    Closes-Bug: 1893988
    Signed-off-by: Don Penney <email address hidden>
    (cherry picked from commit a944973da4f0a18e06ee26b5132d03e007e3e1d3)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to integ (r/stx.4.0)

Reviewed: https://review.opendev.org/749588
Committed: https://git.openstack.org/cgit/starlingx/integ/commit/?id=30ea079a047a9990d48566f72ab42a2c11285da0
Submitter: Zuul
Branch: r/stx.4.0

commit 30ea079a047a9990d48566f72ab42a2c11285da0
Author: Don Penney <email address hidden>
Date: Wed Sep 2 15:16:41 2020 -0400

    Filter new pylint warning W0707

    A new warning introduced in pylint 2.6, W0707 raise-missing-from, is
    causing tox and zuul pylint tests to fail. This update filters the new
    warning.

    Change-Id: Ifa4daf575d3b993d2fe65d895e5c0cd40a0cbcae
    Closes-Bug: 1893988
    Signed-off-by: Don Penney <email address hidden>
    (cherry picked from commit a944973da4f0a18e06ee26b5132d03e007e3e1d3)

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.