-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Support entrypoint #1710
Comments
Yeah. Interesting. I get what they are doing with entrypoint vs command, Is there a common mode we can fall back on? Something like: if len(container.command) > 0 { I guess that breaks for anyone that relies on entrypoint and adds args to Uggh. So tacky. Can't we do better? On Thu, Oct 9, 2014 at 5:17 PM, Johan Euphrosine notifications@github.com
|
@bgrant0607 can't really say, that's part of the docker primitive that users can manipulate. Some docker image might expose an entrypoint that you want to override, i.e: add flags to an interpreter or prefix with a debugger. |
Can't flags be added without exposing entrypoint? Common Docker cases like starting a shell or debugger won't even work right now, since we don't have a way to open stdin (#1521). |
It depends, flags could be added w/ |
Docker's API is a mess here, but given that and given that people USE On Fri, Oct 10, 2014 at 10:41 AM, bgrant0607 notifications@github.com
|
/cc @erictune |
Closing in favor of #2316, where all the discussion is happening. |
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/types.go#L222 doesn't seem to allow it.
The text was updated successfully, but these errors were encountered: