Skip to content
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

Second up rewrites entry point #514

Closed
kojiromike opened this issue Oct 1, 2014 · 1 comment
Closed

Second up rewrites entry point #514

kojiromike opened this issue Oct 1, 2014 · 1 comment

Comments

@kojiromike
Copy link

The second time you fig up, fig rewrites the entry point on a container. To reproduce this issue, start with the following fig file:

whomovedmyentrypoint:
    image: scratch
    entrypoint: fail

Then, run fig up twice, inspecting the entry point each time:

$ fig up; docker inspect --format='{{.Config.Entrypoint}}' $(docker ps -ql); fig up; docker inspect --format='{{.Config.Entrypoint}}' $(docker ps -ql)
Creating foo_whomovedmyentrypoint_1...
Cannot start container 5206410efc5dce757cc0146d5dd3b28ac3cf37e23100015ea8565adf832edeb1: exec: "/fail": stat /fail: no such file or directory
[/fail]
Recreating foo_whomovedmyentrypoint_1...
Cannot start container 3fc0d214031cf7a0f955501f43e546381139610bd30ee8df98b9d938579707b9: exec: "echo": executable file not found in $PATH
[echo]

The use case for not changing the entry point is that the minimal container just for housing volumes (documented by Docker as Data Volume Containers) is little more than scratch with directories and a single, no-op executable like true-asm.

@dnephin
Copy link

dnephin commented Jan 21, 2015

Tracking this in #613, the workaround for now is to use busybox or a container that has /bin/echo

@dnephin dnephin closed this as completed Jan 21, 2015
t0mk added a commit to t0mk/dnscock that referenced this issue Feb 12, 2015
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 a pull request may close this issue.

2 participants