Skip to content
This repository was archived by the owner on Aug 10, 2018. It is now read-only.

Don't override container's entrypoint#42

Open
dchambers wants to merge 1 commit intoayufan:masterfrom
dchambers:dont-override-entrypoint-in-kubernetes
Open

Don't override container's entrypoint#42
dchambers wants to merge 1 commit intoayufan:masterfrom
dchambers:dont-override-entrypoint-in-kubernetes

Conversation

@dchambers
Copy link

Bring the Kubernetes executor into line with the Docker executor, which never overrides entrypoint, but only cmd. In Kubernetes this is done using the args parameter instead of the command parameter.

Bring the Kubernetes executor into line with the Docker executor which never overrides `entrypoint`, but only `cmd` -- in Kubernetes this is done using the `args` parameter instead of the `command` parameter.
@dchambers
Copy link
Author

FYI: I've not been able to test this change at present. I created an automated build on Docker Hub, but this seems to grab pre-built binaries from S3 and there doesn't seem to be a CI configured to build these binaries or instructions for doing it manually.

If somebody gives me a steer I should be able to test that this change works in practice.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant