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

Cleanup volumes after docker_compose containers stop #4134

Merged
merged 1 commit into from
Feb 14, 2017
Merged

Conversation

mshaposhnik
Copy link
Contributor

@mshaposhnik mshaposhnik commented Feb 14, 2017

What does this PR do?

Fixes cmd_stop to cleanup volumes after container stopped.

What issues does this PR fix or reference?

codenvy/codenvy#1477

Changelog

Fixed CLI to cleanup volumes after container stopped.

Release Notes

In our quest for constant improvement we've tweaked the stop command in the CLI to cleanup volumes after the containers are stopped.

Docs PR

N/A

@mshaposhnik mshaposhnik added the kind/bug Outline of a bug - must adhere to the bug report template. label Feb 14, 2017
@bmicklea
Copy link

@mshaposhnik - in your title you say this is an option but it sounds like this is something that will always happen when you stop with CLI, is that right?

@benoitf
Copy link
Contributor

benoitf commented Feb 14, 2017

@bmicklea IMHO it's more "enable flag"

@mshaposhnik
Copy link
Contributor Author

@bmicklea Yes, techinaclly it's a docker-compose option. But we will use it permanently.

@mshaposhnik mshaposhnik changed the title Add option to cleanup volumes after docker_compose containers stop Cleanup volumes after docker_compose containers stop Feb 14, 2017
@bmicklea
Copy link

Is it possible to disable this and would anyone want to?

@mshaposhnik
Copy link
Contributor Author

@bmicklea very unlikely

@codenvy-ci
Copy link

@JamesDrummond JamesDrummond added this to the 5.3.0 milestone Feb 17, 2017
@JamesDrummond JamesDrummond mentioned this pull request Feb 17, 2017
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants