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

Add the possibility to specify the pip version in virtual envs #599

Merged
merged 1 commit into from
Feb 24, 2021

Conversation

SaschaDoering
Copy link
Contributor

Pull Request (PR) description

Hello,

there is already a pullrequest (#558) that has been opened for #559. But since September 2020 no further progress can be seen. Since this is currently blocking me a bit, I have tried myself on a solution. I hope that's the right way in this case.
I'll add the possibility to set the pip version in virtual envs to bypass a general update to latest. Currently it fails on some of our older Ubuntu installations and can be prevented by specifying a version.

Please let me know if I can support here with some more information.

Thanks

This Pull Request (PR) fixes the following issues

Fixes #559

@SaschaDoering
Copy link
Contributor Author

I'm not sure, but the CI for Puppet 5 on Ubuntu 20.04 failed probably because of a mistake that has nothing to do with my change.
The two CIs for Ubuntu 16.04 fail with the error that is the reason for my change.
This could be fixed by setting the Pip version. Should I put this in the Acceptance Tests only for Ubuntu 16.04? Or should I rather not fix it so that the tests are not pinned to an old Pip version?

kenyon
kenyon previously approved these changes Feb 6, 2021
@kenyon kenyon dismissed their stale review February 6, 2021 21:44

tests fail

@kenyon
Copy link
Member

kenyon commented Feb 6, 2021

I'm not sure, but the CI for Puppet 5 on Ubuntu 20.04 failed probably because of a mistake that has nothing to do with my change.
The two CIs for Ubuntu 16.04 fail with the error that is the reason for my change.
This could be fixed by setting the Pip version. Should I put this in the Acceptance Tests only for Ubuntu 16.04? Or should I rather not fix it so that the tests are not pinned to an old Pip version?

Yes, we do need the tests to pass.

@SaschaDoering
Copy link
Contributor Author

Thanks for your review @kenyon. I fixed the CIs for Ubuntu 16.04 by pinning the Pip version. The tests for Ubuntu 20 & Puppet 5 continues to fail with a network error. This seems to me to be a bug that I can't fix in this module. Or do I oversee something here?

@kenyon
Copy link
Member

kenyon commented Feb 13, 2021

Thanks for your review @kenyon. I fixed the CIs for Ubuntu 16.04 by pinning the Pip version. The tests for Ubuntu 20 & Puppet 5 continues to fail with a network error. This seems to me to be a bug that I can't fix in this module. Or do I oversee something here?

Yes, Puppet 5 on Ubuntu 20.04 always fails because there are no Puppet 5 packages for Ubuntu 20.04, but we have no way to exclude this combination from the test matrix.

@crazymind1337
Copy link
Member

@kenyon Can you merge the PR since already have approved it?

@bastelfreak bastelfreak added the enhancement New feature or request label Feb 23, 2021
@crazymind1337
Copy link
Member

I will merge this soon since it already has been approved by two persons despite I am from the same company like the requester.

@bastelfreak bastelfreak merged commit d7be32b into voxpupuli:master Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting Pip Version for Virtual Environments
4 participants