Code review comment for lp://staging/~waigani/juju-core/suspend-resume-local-plugin

Revision history for this message
William Reade (fwereade) wrote :

LGTM as a skeleton, but please don't land do-nothing commands (or at
least make sure they're not accessible)

https://codereview.appspot.com/80160043/diff/1/cmd/plugins/local/suspend.go
File cmd/plugins/local/suspend.go (right):

https://codereview.appspot.com/80160043/diff/1/cmd/plugins/local/suspend.go#newcode23
cmd/plugins/local/suspend.go:23: If the container is kvm, suspend will
execute "virsh destroy <container-
On 2014/03/26 01:20:31, axw wrote:
> Again, this seems too low-level for user docs.

Agreed. Would be really nice to put this sort of thing in the actual
documentation somewhere, though...

https://codereview.appspot.com/80160043/

« Back to merge proposal