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

remove docker argument -d in DockerTimeout.sh #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

flywu
Copy link

@flywu flywu commented Apr 14, 2017

with -d ,docker can't bootup because of conflict of argument -d and argument --rm

@JeongtaekLim
Copy link

Dose removing -d option work?
I got error.

{ [Error: Command failed: time="2017-04-20T23:29:52+09:00" level=fatal msg="cannot enable tty mode on non tty input"
] killed: false, code: 1, signal: null } '' 'time="2017-04-20T23:29:52+09:00" level=fatal msg="cannot enable tty mode on non tty input" \n'

Instead, when I run with only -d option(deleting --rm), it works well.!

@flywu
Copy link
Author

flywu commented Apr 20, 2017 via email

@JeongtaekLim
Copy link

JeongtaekLim commented Apr 21, 2017

Thank you for reply.
Can you tell me your os?
Mine is Ubuntu 14.04 LTS

@flywu
Copy link
Author

flywu commented Apr 21, 2017 via email

@DollarAkshay
Copy link

This change is necessary.

Official Docs : https://docs.docker.com/engine/reference/run/#clean-up-rm

@fcmatteo
Copy link

I've tried in a Digital Ocean droplet with CentOS and removing -d didn't work, while removing --rm did work.

@asadm
Copy link
Contributor

asadm commented Nov 18, 2017

As I understand this issue -d and --rm are not conflicting anymore. The official docs also say:

By design, containers started in detached mode exit when the root process used to run the container exits, unless you also specify the --rm option. If you use -d with --rm, the container is removed when it exits or when the daemon exits, whichever happens first.

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.

6 participants