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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]