On Thu, Apr 22, 2010 at 10:49, Chris Lalancette <clalance@xxxxxxxxxx> wrote: > On 04/22/2010 10:45 AM, Stephen Shaw wrote: >> On Thu, Apr 22, 2010 at 07:55, Daniel P. Berrange <berrange@xxxxxxxxxx> wrote: >>> On Thu, Apr 22, 2010 at 09:39:57AM -0400, Chris Lalancette wrote: >>>> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c >>>> index 5f4adfd..4ec4bd6 100644 >>>> --- a/src/qemu/qemu_driver.c >>>> +++ b/src/qemu/qemu_driver.c >> >> <trim> >> >> Ok, I just applied that patch in the spec file and rebuilt. I ran >> virsh snapshot-create a couple time with virt-manager running and >> connected. Exited cleanly with no apparent problems. I also ran >> virsh snapshot-revert with no problems (hadn't tested this before so, >> I'm not saying there was a problem before) and impressed. Though it >> seems that being able to give a snapshot a name and not some random >> numerical string would be very/extremely useful. > > Great, thanks for the testing! I'll clean the patch up with the > recommendations from Dan and post it for 0.8.1. > > Note that you can specify a name when taking a snapshot; you just > need to provide the appropriate XML snippet. Something like the > following should work: > > # cat<< EOF > snapshot.xml > <domainsnapshot> > <name>myname</name> > <description>this is my description</description> > </domainsnapshot> > EOF > # virsh snapshot-create myguest snapshot.xml > > -- > Chris Lalancette > ah ok, It would be nice if you could just optionally append it to the end of the command ie. virsh snapshot-create 1 myname thanks for putting that patch together and getting that fixed. Something else I ran into as an aside was that if the driver type in the xml wasn't set it wouldn't create the snapshot. It would be cool if the code could detect the image type instead of relying solely on that xml attribute(?). Cheers, Stephen -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list