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

Switch to RHEL8 builder image #204

Merged

Conversation

cblecker
Copy link
Member

Switches and updates the base builder image to the OCP sanctioned RHEL8 image

@openshift-ci openshift-ci bot requested review from arjunrn and bdematte March 18, 2022 19:49
@2uasimojo
Copy link
Member

See doc for help with the tagging requirement to fix pr-check.

@cblecker
Copy link
Member Author

/test all

@cblecker
Copy link
Member Author

/retest

@cblecker
Copy link
Member Author

@2uasimojo Tag pushed per directions, but it's still failing it seems.

@cblecker
Copy link
Member Author

/retest

@cblecker
Copy link
Member Author

/test

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 25, 2022

@cblecker: The /test command needs one or more targets.
The following commands are available to trigger required jobs:

  • /test images
  • /test pr-check

Use /test all to run all jobs.

In response to this:

/test

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.

@2uasimojo
Copy link
Member

/test pr-check

@cblecker
Copy link
Member Author

/retest

@2uasimojo
Copy link
Member

Need to update that tag in a couple of other places. See e.g. #180

@2uasimojo
Copy link
Member

Just need to move the new tag onto the new commit (88fa32d) and re-push it to origin & upstream.

@cblecker
Copy link
Member Author

/retest

1 similar comment
@cblecker
Copy link
Member Author

/retest

@2uasimojo
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 25, 2022
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 25, 2022
@2uasimojo
Copy link
Member

clean rebase

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 25, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 25, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 2uasimojo, cblecker

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 25, 2022

@cblecker: all tests passed!

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.

@openshift-merge-robot openshift-merge-robot merged commit 2a6a579 into openshift:master Mar 25, 2022
@cblecker cblecker deleted the change-base-builder-image branch March 25, 2022 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants