Skip to content

Commit

Permalink
Fixes go-runner path in docker container (#935)
Browse files Browse the repository at this point in the history
For 1.23 clusters kops changed the static pod manifests to run
/go-runner instead of relying on the entry point set in the dockerfile.
go-runner has been at the / of the upstream container for many versions now.
This should continue to work with previous versions as well

Kops PR:  kubernetes/kops#12494
  • Loading branch information
jaxesn committed May 13, 2022
1 parent 846b102 commit 28b40b4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion development/kops/eks-d.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ spec:
{{end}}
cloudProvider: aws
configBase: {{ .configBase }}
containerRuntime: docker
containerRuntime: containerd
etcdClusters:
- cpuRequest: 200m
etcdMembers:
Expand Down
2 changes: 1 addition & 1 deletion projects/kubernetes/release/1-23/GIT_TAG
Original file line number Diff line number Diff line change
@@ -1 +1 @@
v0.12.0
v0.13.0
4 changes: 2 additions & 2 deletions projects/kubernetes/release/docker/go-runner/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,5 @@ ARG RELEASE_BRANCH

COPY _output/$RELEASE_BRANCH/LICENSES /LICENSES
COPY $RELEASE_BRANCH/ATTRIBUTION.txt /ATTRIBUTION.txt
COPY _output/$RELEASE_BRANCH/bin/release-src/$TARGETOS-$TARGETARCH/go-runner /usr/local/bin/go-runner
ENTRYPOINT ["/usr/local/bin/go-runner"]
COPY _output/$RELEASE_BRANCH/bin/release-src/$TARGETOS-$TARGETARCH/go-runner /go-runner
ENTRYPOINT ["/go-runner"]

0 comments on commit 28b40b4

Please sign in to comment.