Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] pip.installed with upgrade=True and index_url doesn't work #61610

Closed
Lego3 opened this issue Feb 9, 2022 · 3 comments · Fixed by #61645
Closed

[BUG] pip.installed with upgrade=True and index_url doesn't work #61610

Lego3 opened this issue Feb 9, 2022 · 3 comments · Fixed by #61645
Assignees
Labels
Bug broken, incorrect, or confusing behavior Regression The issue is a bug that breaks functionality known to work in previous releases. severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around

Comments

@Lego3
Copy link
Contributor

Lego3 commented Feb 9, 2022

Description
Issue #59100 was closed as having done what was possible for Aluminium, and the situation has indeed improved. There is, however, still an issue when used together with the index_url argument.

Setup
With a state like this:

pip3:
  pip.installed:
    - name: pip
    - upgrade: True

Everything works as expected. But with

pip3:
  pip.installed:
    - name: pip
    - upgrade: True
    - index_url: https://pypi.python.org/simple

It always returns "the available upgrade doesn't fulfills the version requirements".

Expected behavior
These two should behave the same

@Lego3 Lego3 added Bug broken, incorrect, or confusing behavior needs-triage labels Feb 9, 2022
@welcome
Copy link

welcome bot commented Feb 9, 2022

Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey.
Please be sure to review our Code of Conduct. Also, check out some of our community resources including:

There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar.
If you have additional questions, email us at saltproject@vmware.com. We’re glad you’ve joined our community and look forward to doing awesome things with you!

@Lego3
Copy link
Contributor Author

Lego3 commented Feb 15, 2022

I had a quick look at the code myself, and it looks like this commit: eaf061a
put the code slightly out of place. At the moment the --index-url argument is put right after "pip", but it needs to go after "index versions" or "install".

@s0undt3ch might know more about the reasoning on the location in the code? I.e. could the new code go back to where "install" was previously appended to the command as well?

@OrangeDog OrangeDog added severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around and removed needs-triage labels Mar 14, 2022
@OrangeDog OrangeDog added this to the Approved milestone Mar 14, 2022
@OrangeDog OrangeDog added the Regression The issue is a bug that breaks functionality known to work in previous releases. label Mar 14, 2022
@anilsil anilsil modified the milestones: Approved, Sulfur v3006.3 Aug 24, 2023
@Ch3LL
Copy link
Contributor

Ch3LL commented Aug 24, 2023

closed by #61645

@Ch3LL Ch3LL closed this as completed Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Regression The issue is a bug that breaks functionality known to work in previous releases. severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants