new release of rpm breaks tox that uses sysinv

Bug #1949804 reported by Al Bailey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Critical
Al Bailey

Bug Description

Brief Description
-----------------
A new version of rpm was released on pypi on Nov 4, 2021 and it breaks our ability to import sysinv in some environments.

Severity
--------
Critical

Steps to Reproduce
------------------
submit a code change to sysinv (or to any repo that runs zuul and tries to import sysinv as a dependency. ie: integ)

Expected Behavior
------------------
zuul should work

Actual Behavior
----------------
fails to install the rpm dependency, so zuul and tox fail

Reproducibility
---------------
100% (but its based on the nodeset and what is cached in its pip cache. Something cached from yesterday will still work)

System Configuration
--------------------
N/A

Branch/Pull Time/Commit
-----------------------
Nov 4, 2021

Last Pass
---------
Nov 3, 2021

Timestamp/Logs
--------------

2021-11-04 14:13:47.637435 | ubuntu-focal | Collecting rpm
2021-11-04 14:13:47.641424 | ubuntu-focal | Downloading https://mirror-int.dfw.rax.opendev.org/pypifiles/packages/99/af/a43b0a3fe88278384f3b194c656825df712f645109f1fbd43c52314d9d3c/rpm-0.0.3.1.tar.gz (8.2 kB)
2021-11-04 14:13:47.648886 | ubuntu-focal | Preparing metadata (setup.py): started
2021-11-04 14:13:47.877978 | ubuntu-focal | Preparing metadata (setup.py): finished with status 'error'
2021-11-04 14:13:47.878323 | ubuntu-focal | ERROR: Command errored out with exit status 1:
2021-11-04 14:13:47.878354 | ubuntu-focal | command: /home/zuul/src/opendev.org/starlingx/integ/.tox/pylint/bin/python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-b_ofhqhu/rpm/setup.py'"'"'; __file__='"'"'/tmp/pip-install-b_ofhqhu/rpm/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-2toqd4hx
2021-11-04 14:13:47.878375 | ubuntu-focal | cwd: /tmp/pip-install-b_ofhqhu/rpm/
2021-11-04 14:13:47.878394 | ubuntu-focal | Complete output (5 lines):
2021-11-04 14:13:47.878411 | ubuntu-focal | Traceback (most recent call last):
2021-11-04 14:13:47.878428 | ubuntu-focal | File "<string>", line 1, in <module>
2021-11-04 14:13:47.878462 | ubuntu-focal | File "/tmp/pip-install-b_ofhqhu/rpm/setup.py", line 8, in <module>
2021-11-04 14:13:47.878482 | ubuntu-focal | raise RuntimeError("Install aborted - please install python-rpm from distributiong system.")
2021-11-04 14:13:47.878500 | ubuntu-focal | RuntimeError: Install aborted - please install python-rpm from distributiong system.

Test Activity
-------------
Submission

Workaround
----------
None. I have a fix.

Al Bailey (albailey1974)
Changed in starlingx:
assignee: nobody → Al Bailey (albailey1974)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to config (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/config/+/816713

Changed in starlingx:
status: New → In Progress
Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Critical
tags: added: stx.6.0 stx.config stx.zuul
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

Reviewed: https://review.opendev.org/c/starlingx/config/+/816713
Committed: https://opendev.org/starlingx/config/commit/5012653f5ebfa87218a592ac77d863cc1234fb37
Submitter: "Zuul (22348)"
Branch: master

commit 5012653f5ebfa87218a592ac77d863cc1234fb37
Author: albailey <email address hidden>
Date: Thu Nov 4 11:33:10 2021 -0400

    Removing rpm as a dependency for sysinv

    A new version of the python module was released Nov 4
    which breaks pip installs in some environments.

    The rpm dependency is not directly required by sysinv
    and so it has been removed.

    The source code changes in this review are only included
    to ensure that all zuul targets are invoked.

    Closes-Bug: 1949804
    Signed-off-by: albailey <email address hidden>
    Change-Id: I6786c3510bb7bac5f03adf3a3c3d377a090063ac

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
Al Bailey (albailey1974) wrote :

Change has merged.

A similar fix was added for integ repo, but that fix just does not pull in sysinv, and therefore does not fail for this package
https://review.opendev.org/c/starlingx/integ/+/816706

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.