-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Dockerfile.upi.ci: Explicitly use rhel7 for now #4106
Conversation
It looks like the base image switched to RHEL8, but that lost things like python2 that this image uses. Keep this code on life support by explicitly using rhel7 for now.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
(I didn't test this locally since it has hardcoded references to the api.ci internal rhel mirror - could probably hack that but figured I might as well just try in CI) |
/retest |
Oh I think I see, the value in this repo is being overridden by the one in the release repo. |
@cgwalters: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
See also #4098 |
maybe this was fixed by that PR, so we can close this. Feel free to re-open if you think that's not the case. /close |
@abhinavdahiya: Closed this PR. In 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. |
It looks like the base image switched to RHEL8, but that
lost things like python2 that this image uses. Keep this
code on life support by explicitly using rhel7 for now.