baremetal-runtimecfg: use UBI8 as base#7391
baremetal-runtimecfg: use UBI8 as base#7391vrutkovs wants to merge 1 commit intoopenshift:masterfrom
Conversation
base 4.5 is using UBI7, which has python2. Scripts are not python2 compatible, so UBI8 needs to be used so that scripts would run in container
openshift-ci-robot
added
the
size/XS
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: vrutkovs The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
vrutkovs
changed the title
openshift-ci-robot
added
the
do-not-merge/work-in-progress
vrutkovs
changed the title
openshift-ci-robot
removed
the
do-not-merge/work-in-progress
|
@vrutkovs this should probably go into 4.5 and 4.6 as well |
|
/hold probably not needed since we're going to rebase fcos branch on master soon |
openshift-ci-robot
added
the
do-not-merge/hold
|
/retest |
|
We can't do this because we can't use UBI8 containers on RHEL7 workers. Or am I missing something? |
|
/close |
|
@LorbusChris: Closed this PR. 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. |
3 participants
base 4.5 is using UBI7, which has python2. Scripts are not python2
compatible, so UBI8 needs to be used so that scripts would run in
container.
Required for openshift/machine-config-operator#1521. This fixes OKD on openstack, where scripts are running in the container.
Fixes openshift/machine-config-operator#1430
/cc @celebdor