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

Build container from rhel7 #4

Merged
merged 1 commit into from Jul 11, 2019
Merged

Build container from rhel7 #4

merged 1 commit into from Jul 11, 2019

Conversation

elfosardo
Copy link

Change base image from centos7 to rhel7 for ocp container

Change base image from centos7 to rhel7 for ocp container
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 10, 2019
@dhellmann
Copy link

When would we update this to RHEL8?

@elfosardo
Copy link
Author

@dhellmann apparently there are no builders for rhel8 at the moment and there's no ETA to have that for now

@dhellmann
Copy link

@elfosardo ok, if we can't yet then we'll just keep that as a todo item

Copy link

@dhellmann dhellmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 10, 2019
@hardys
Copy link

hardys commented Jul 10, 2019

Yeah IMO we should definitely avoid doing anything in the short-medium term which deviates significantly from our current test matrix, which is pretty much all centos7 (so rhel7 should work fine, modulo any dependency issues etc)

@elfosardo elfosardo merged commit a1f6b38 into master Jul 11, 2019
@elfosardo elfosardo deleted the build-from-rhel branch July 11, 2019 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants