stx/integ pylint failure with new warning

Bug #1893988 reported by Don Penney on 2020-09-02
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
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) on 2020-09-02
Changed in starlingx:
importance: Undecided → Critical
assignee: nobody → Don Penney (dpenney)
tags: added: stx.3.0 stx.4.0 stx.5.0 stx.zuul
Ghada Khalil (gkhalil) wrote :

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

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

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)

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  Edit
Everyone can see this information.

Other bug subscribers