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

Added support for restart_policy parameter #712

Closed
wants to merge 1 commit into from

Conversation

kennu
Copy link

@kennu kennu commented Dec 7, 2014

This patch adds the restart_policy parameter to services. The parameter
is passed to the Docker API when starting containers. Its recognized
values are the same as in the Docker CLI ("always" and "on-failure[:max-retry]").

For future compatibility, the policy can also be specified as a
dictionary, which is passed directly to the Docker API (after converting
the keys to CamelCase).

This feature seems to have been requested at least in issue #478.

Signed-off-by: Kenneth Falck kennu@iki.fi

This patch adds the restart_policy parameter to services. The parameter
is passed to the Docker API when starting containers. Its recognized values
are the same as in the Docker CLI ("always" and "on-failure[:max-retry]").

For future compatibility, the policy can also be specified as a
dictionary, which is passed directly to the Docker API (after converting
the keys to CamelCase).

This feature seems to have been requested at least in issue docker#478.

Signed-off-by: Kenneth Falck <kennu@iki.fi>
@dnephin
Copy link

dnephin commented Dec 7, 2014

See #594

@bfirsh
Copy link

bfirsh commented Dec 9, 2014

This has now been merged. Thanks anyway, @kennu!

@bfirsh bfirsh closed this Dec 9, 2014
@kennu
Copy link
Author

kennu commented Dec 9, 2014

Nice that it's in there, one form or another :-)

@cosminonnet
Copy link

Hi, thanks for this feature!
Is the "restart_policy" parameter available in Fig 1.0.1?
If not, how can I start using it?

@cosminonnet cosminonnet mentioned this pull request Dec 13, 2014
@tomkersten tomkersten mentioned this pull request Jan 7, 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 this pull request may close these issues.

None yet

4 participants