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

Bug #1966073 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/ptp-notification-armada-app/python-k8sapp-ptp-notification/k8sapp_ptp_notification/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)
Changed in starlingx:
importance: Undecided → Medium
tags: added: stx.7.0 stx.apps stx.networking stx.zuul
Ghada Khalil (gkhalil)
Changed in starlingx:
assignee: nobody → Cole Walker (cwalops)
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: Cole Walker (cwalops) → Karla Felix (kkarolin)
Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ptp-notification-armada-app (master)

Reviewed: https://review.opendev.org/c/starlingx/ptp-notification-armada-app/+/848898
Committed: https://opendev.org/starlingx/ptp-notification-armada-app/commit/542b4678709d2adc8ef4da78985bd0722bf3a2ba
Submitter: "Zuul (22348)"
Branch: master

commit 542b4678709d2adc8ef4da78985bd0722bf3a2ba
Author: Karla Felix <email address hidden>
Date: Wed Jul 6 15:26:38 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 ptp-notification package.
    PASS: Upload and apply ptp-notification package and check if pod
          is running.

    Closes-bug:1966073

    signed-off-by: Karla Felix <email address hidden>
    Change-Id: I854cfb8b72bb9af069b8c0c378f934cf189c6345

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
Ghada Khalil (gkhalil) wrote :

This change will be included in stx.7.0 since it merged before the release branch creation

tags: added: stx.7.0
removed: stx.8.0
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.