8000 Don't rm old containers, in case we want the log by bboreham · Pull Request #1939 · weaveworks/weave · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Jun 20, 2024. It is now read-only.

Don't rm old containers, in case we want the log #1939

Merged
merged 1 commit into from
Jan 29, 2016
Merged

Don't rm old containers, in case we want the log #1939

merged 1 commit into from
Jan 29, 2016

Conversation

bboreham
Copy link
Contributor

Fixes #1937

@awh awh self-assigned this Jan 29, 2016
@awh awh added this to the 1.5.0 milestone Jan 29, 2016
awh added a commit that referenced this pull request Jan 29, 2016
@awh
Merge pull request #1939 from /issues/1937-dont-rm
11ccada 
Don't rm old containers, in case we want the log
@awh awh merged commit 11ccada into master Jan 29, 2016
@awh awh deleted the issues/1937-dont-rm branch January 29, 2016 15:47
brb added a commit that referenced this pull request Apr 5, 2016
@brb
Do not fail if $COVERAGE is not set
41a5c6f 
Also, stop weave in end_suite by using `weave stop` which does not
remove the weave container (#1939).
@brb brb mentioned this pull request Apr 5, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@awh awh

Labels
None yet
Projects
None yet
Milestone
1.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@bboreham @awh
0