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
provider/gce: remove getSnapshot #4931
Conversation
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Build failed: Generating tarball failed |
Updates LP 1563628 getSnapshot was being used incorrectly with the expectation that it was creating an immutable copy of the environment at that point in time.
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Build failed: Tests failed |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Build failed: Tests failed |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Build failed: Tests failed |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
Build failed: Tests failed |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
worker/metrics/sender: fix mutex copy bug in test Fixes 1563628 Fixed accidental copy of a mutex in test. Fixes LP 1563628 when #4938, #4937 and #4931 are comitted. (Review request: http://reviews.vapour.ws/r/4376/)
Updates LP 1563628
getSnapshot was being used incorrectly with the expectation that it was
creating an immutable copy of the environment at that point in time.
(Review request: http://reviews.vapour.ws/r/4367/)