-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add a RHEL7 Dockerfile and standardize format #3
Add a RHEL7 Dockerfile and standardize format #3
Conversation
Dockerfile.rhel
Outdated
@@ -1,11 +1,10 @@ | |||
FROM openshift/origin-release:golang-1.10 as build | |||
FROM registry.svc.ci.openshift.org/openshift/release:golang-1.10 AS builder |
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.
Please look exactly like this commit (
openshift/csi-external-attacher#6). You want two docker files, one with .rhel
as an extension.
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.
Done
f13eeac
to
a262020
Compare
Dockerfile.openshift.rhel7
Outdated
FROM centos:7 | ||
COPY --from=build /go/src/github.com/coredns/coredns/coredns /usr/bin/coredns | ||
FROM registry.svc.ci.openshift.org/ocp/4.0:base | ||
COPY --from=builder /go/src/github.com/coredns/coredns/coredns /usr/bin/coredns |
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.
You can omit the trailing /coredns
and just have /usr/bin/
as the destination.
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.
(do that in both docker files
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.
Fixed
@@ -0,0 +1,13 @@ | |||
FROM registry.svc.ci.openshift.org/openshift/release:golang-1.10 AS builder |
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.
You can do Dockerfile
and Dockerfile.rhel
, unless there is a docker file in the repo already that you don't want to overwrite.
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.
There's already a Dockerfile
|
||
ENTRYPOINT ["/usr/bin/coredns"] | ||
|
||
LABEL io.k8s.display-name="CoreDNS" \ |
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.
Keep the two docker files identical except for the FROM lines.
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.
That's already the case...
/retest |
Test terraform failure looks suspiciously flaky... /retest |
/lgtm |
1. Why is this pull request needed and what does it do?
2. Which issues (if any) are related?
3. Which documentation changes (if any) need to be made?