-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Dockerfile.upi.ci: Drop pip+pyopenssl installs #4784
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
Conversation
|
/test e2e-gcp |
|
/test e2e-gcp-upi |
|
Thanks for this, @cgwalters. |
|
Here is a link to the PR giving some detail regarding why we are explicitly using an old pip version. |
|
Ah OK yep from that PR:
And that's almost certainly because RHEL8 switched to python3 by default, but gcloud is python2. This PR explicitly still installs for python2, so should still work. (But I didn't explicitly test the gcloud parts, trying that out now) EDIT: done, |
|
One thing that would probably help at some point is to have separate per-infra UPI CI containers so that we're not mashing together all the aws, gcp, azure, matchbox etc tooling. |
|
Err wait, if I |
|
OK compounding this it's not even clear to me that we need pyopenssl anymore, see https://cloud.google.com/sdk/crypto |
We shouldn't need this anymore, see https://cloud.google.com/sdk/crypto We have a larger conflict here because the gcloud sdk wants python2, azure wants python3. Something changed in the pip CDN that broke our use of the older pip. (hopefully) Closes: openshift#4783
|
/test e2e-gcp-upi |
Currently, pyopenssl is needed when we call This is the error when pyopenssl is not installed. |
|
Fun times. |
|
I'm fine with leaving e2e-gcp-upi broken (for now). There are a lot of things blocked on this error. |
|
Ah. Uggh. Well...I dunno. I think we could plow forward with trying to port this to rhel8, where everything is better maintained? |
staebler
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: staebler The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
One thing that's definitely going on here is the RHEL Python maintainers have been trying to ditch python2 for a long time, and then RHEL7 is even older. There it's moved to a SCL which would require running everything else in the context of that scl which would get awkward. Testing out rhel8 things seem to work much better (as I was originally doing) in a |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
The UPI CI image is not used for the e2e-aws-upgrade job. |
|
@staebler: Overrode contexts on behalf of staebler: ci/prow/e2e-aws-upgrade DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@cgwalters: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
We shouldn't need this anymore, see
https://cloud.google.com/sdk/crypto
We have a larger conflict here because the gcloud sdk wants
python2, azure wants python3.
Something changed in the pip CDN that broke our use of the
older pip.
(hopefully)
Closes: #4783