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

[Feature Request] Support StopTimeout for Docker tasks #3544

Closed
angrycub opened this issue Nov 13, 2017 · 7 comments
Closed

[Feature Request] Support StopTimeout for Docker tasks #3544

angrycub opened this issue Nov 13, 2017 · 7 comments

Comments

@angrycub
Copy link
Contributor

Would it be possible to support the StopTimeout field on Docker container creation?

re: ZD-5428

@samart
Copy link

samart commented Nov 16, 2017

Thanks.

To add color to this ticket -

Use case:

Operators may need to bounce docker engine, and engine will observe a stop-timeout if set on "docker run". This helps to ensure there is consistent stopping behavior when either nomad does a container stop and also docker-engine doing the stop. This helps protect applications that need a little more time to shutdown gracefully. Otherwise, the default that docker provides is just 10 seconds. When set, docker engine will wait the for the container that has the longest timeout.

Thank you!

@dadgar
Copy link
Contributor

dadgar commented Nov 16, 2017

@samart Would you want this to be settable on its own or just be the same value as the kill_timeout. Have the two out of sync seems problematic.

@samart
Copy link

samart commented Nov 16, 2017

Hi Alex - it does make sense for it to be the same value and probably set automatically. But who knows if you will encounter the oddball use use where someone might see this as a problem.

But i am pro setting it automatically and that's a good feature IMHO.

@dadgar
Copy link
Contributor

dadgar commented Nov 16, 2017

Will need fsouza/go-dockerclient#693 to merge first

@davidsylvester
Copy link

davidsylvester commented Dec 5, 2017

@chelseakomlo As part of which release this feature would be available ?

@dadgar
Copy link
Contributor

dadgar commented Dec 5, 2017

It will be available shortly in 0.7.1

@github-actions
Copy link

github-actions bot commented Dec 5, 2022

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants