Skip to content

Conversation

@suzaku
Copy link

@suzaku suzaku commented Aug 17, 2014

We can use command in fig.yml to override the default command, but this won't work if the docker image we are working with specifies a ENTRYPOINT for its containers.

In my case, the Dockerfile of the main service has specified something like ["gunicorn", "-c", "config.py"] as its ENTRYPOINT, so I can't do things like python manage.py syncdb if I can't override the ENTRYPOINT.

Entrypoints can be overrdden by the --entrypoint option, this PR attempts to solve the problem by exposing this option in fig.

@salehe
Copy link

salehe commented Aug 17, 2014

Isn't this already supported in fig.yml with 'entrypoint' key? Does this PR intend to make this available on fig CLI?

@suzaku
Copy link
Author

suzaku commented Aug 17, 2014

@salehe Yes, since some images have specified ENTRYPOINT in their Dockerfiles, we cannot always override the command using CMD only.

@dnephin
Copy link

dnephin commented Aug 18, 2014

This is related to #363

Copy link
Author

Choose a reason for hiding this comment

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

@dnephin Thanks for replying. Isn't this a function that return some information about an already running container? What do you mean by "this"?

Copy link

Choose a reason for hiding this comment

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

ah, yes

Signed-off-by: Satoru Logic <satorulogic@gmail.com>
@suzaku
Copy link
Author

suzaku commented Aug 20, 2014

@salehe Are you a core developer of fig?

@salehe
Copy link

salehe commented Aug 20, 2014

No! I'm just a contributer @suzaku

@prologic
Copy link

+1

Copy link

Choose a reason for hiding this comment

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

This was intentionally 80 chars wide!

@bfirsh
Copy link

bfirsh commented Oct 1, 2014

Replaced by #511

@bfirsh bfirsh closed this Oct 1, 2014
xulike666 pushed a commit to xulike666/compose that referenced this pull request Jan 19, 2017
snapshot: Fix leaked mounts and temp directories
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants