stx/audit-armada-app uses soon-to-be deprecated pip install --use-deprecated legacy-resolver

Bug #1966069 reported by Ghada Khalil
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Medium
Karla Felix

Bug Description

Brief Description
-----------------
Opened on behalf of <email address hidden>

The pip team is planning to depreciate the '--use-deprecated legacy-resolver'
install option. While a firm date has yet to be set, We need to be ready for
the change.

Our usage seems to be confined to 'tox.ini' files. When the upstream pip change
change is delivered, we can expect.
- Build failures due to failed unit tests
- Zuul to vote -q on code reviews in some repos

This LP tracks addressing the issue with the following files:
cgcs-root/stx/audit-armada-app/python-k8sapp-auditd/k8sapp_auditd/tox.ini

Severity
--------
Major:

Steps to Reproduce
------------------
Scan source code with ....
grep -r 'use-deprecated' $MY_REPO

Expected Behavior
------------------
Not found

Actual Behavior
----------------
Found 8 files using the offending option

Reproducibility
---------------
100%

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

Branch/Pull Time/Commit
-----------------------
March 9, 2022

Last Pass
---------
N/A

Timestamp/Logs
--------------
N/A

Test Activity
-------------
Build

Workaround
----------
N/A

Ghada Khalil (gkhalil)
tags: added: stx.7.0 stx.apps stx.security
Changed in starlingx:
importance: Undecided → Medium
tags: added: stx.zuul
Ghada Khalil (gkhalil)
Changed in starlingx:
assignee: nobody → Ghada Khalil (gkhalil)
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Screening: stx.8.0 / there is no hard deadline for this. Doesn't appear that any other open-infra projects are actively addressing these issues either. Park until after stx.7.0 and target for stx.8.0

tags: added: stx.8.0
removed: stx.7.0
Ghada Khalil (gkhalil)
Changed in starlingx:
assignee: Ghada Khalil (gkhalil) → Karla Felix (kkarolin)
Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to audit-armada-app (master)

Reviewed: https://review.opendev.org/c/starlingx/audit-armada-app/+/847583
Committed: https://opendev.org/starlingx/audit-armada-app/commit/ede964b1832c1fdfe2cf5db675c49734fec02900
Submitter: "Zuul (22348)"
Branch: master

commit ede964b1832c1fdfe2cf5db675c49734fec02900
Author: Karla Felix <email address hidden>
Date: Fri Jun 24 10:33:29 2022 -0400

    Remove --use-deprecated legacy-resolver flag

    The pip team is planning to depreciate the
    '--use-deprecated legacy-resolver'install option.
    While a firm date has yet to be set, We need to be
    ready for the change.

    Test Plan:

    PASS: Verified that tox runs without issues
    PASS: Generate auditd package.
    PASS: Install audit package and check if pod is running.
    PASS: Check auditable events.

    Closes-Bug: 1966069

    Signed-off-by: Karla Felix <email address hidden>
    Change-Id: Icbe2fb324f220de194dc30686b65ad2d0d466b2d

Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
tags: added: stx.7.0
removed: stx.8.0
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Actually this will be included in stx.7.0 since the release branch was not created yet

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.