[libvirt] [PATCH 0/2] Check domain status before virsh suspend or resume

[Daniel P. Berrange]

On Mon, Apr 15, 2013 at 12:49:12PM -0400, John Ferlan wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=906644
> 
> Added checks to both virsh suspend and virsh resume for the domain to be
> in a the right state before trying the suspend/resume. Similar checks to
> examples/domsuspend/suspend.c.

IMHO this is just a pointless bug request. State checks don't
belong in virsh for a start, since that makes it inherantly
racey. While the drivers do check for whether the domain is
running, they explicitly chose not to raise an error if the
VM is already paused, when pause is executed & vica-verca


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|