Merge lp://staging/~waigani/juju-core/suspend-resume-local-plugin into lp://staging/~go-bot/juju-core/trunk

Reviewers: mp+212743_code.launchpad.net,

Message:
Please take a look.

Description:
Local plugins suspend, resume

Add two plugins to the local provider: suspend and resume.
This branch sets up the boilerplate and help docs for the plugins.
The functionality of suspend and resume will be added in a later
branch once the following branch is landed:
lp:~waigani/juju-core/add-start-stop-to-container-manager-interface

https://code.launchpad.net/~waigani/juju-core/suspend-resume-local-plugin/+merge/212743

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/80160043/

Affected files (+119, -0 lines):
   A [revision details]
   M cmd/plugins/local/main.go
   M cmd/plugins/local/main_test.go
   A cmd/plugins/local/resume.go
   A cmd/plugins/local/suspend.go

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

https://codereview.appspot.com/80160043/diff/1/cmd/plugins/local/resume.go#newcode21
cmd/plugins/local/resume.go:21: resume starts the specified container
that has previously been shutdown.
s/shutdown/suspended/
?

https://codereview.appspot.com/80160043/diff/1/cmd/plugins/local/resume.go#newcode23
cmd/plugins/local/resume.go:23: If the container is kvm, resume will
execute "virsh start <container- name>".
Does the user really care about this? Seems a bit low-level to me.
If you do keep it, s/container- name/container-name/

https://codereview.appspot.com/80160043/diff/1/cmd/plugins/local/resume.go#newcode55
cmd/plugins/local/resume.go:55: return nil
Perhaps return an error saying not implemented.

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-
Again, this seems too low-level for user docs.

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

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/

WIP to keep queue tidy