Merge lp:~rvb/gwacl/fix-destroydeploy2 into lp:gwacl
Proposed by
Raphaël Badin
Status: | Merged |
---|---|
Approved by: | Raphaël Badin |
Approved revision: | 166 |
Merged at revision: | 163 |
Proposed branch: | lp:~rvb/gwacl/fix-destroydeploy2 |
Merge into: | lp:gwacl |
Diff against target: |
257 lines (+33/-102) 3 files modified
example/management/run.go (+4/-18) management.go (+9/-19) management_test.go (+20/-65) |
To merge this branch: | bzr merge lp:~rvb/gwacl/fix-destroydeploy2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gavin Panella | Approve | ||
Review via email: mp+173095@code.launchpad.net |
Commit message
Do not attempt to stop the VMs inside DestroyDeployment.
Description of the change
I found that it's quicker a more reliable to just delete the deployment instead of trying to shutdown the VMs when destroying a deployment. When we delete the deployment object, the VMs are shut down are removed. The only thing left are the disks.
Added bonus: the testing becomes simpler as we issue less API requests.
To post a comment you must log in.
I surprised this works, but looks good.