Daniel P. Berrange wrote: > On Tue, Oct 21, 2008 at 03:57:13PM +0200, Chris Lalancette wrote: >> Currently, you can define a logical storage pool with something like the >> following: >> >> <pool type='logical'> >> <source> >> <name>MyVG</name> >> <device name='/dev/sdb'/> >> ... >> >> However, dumping out the XML for this same storage pool (with, say, virsh >> pool-dumpxml), gives: >> >> <pool type='logical'> >> <source> >> <name>MyVG</name> >> <device name='/dev/sdb'> >> </device> >> >> >> To make this more idempotent, do the <device name='/dev/sdb'/> form by >> default, >> and only do the <device>...</device> form if .nfreeExtent is defined for the >> storage pool. > > ACK
Committed this. -- Chris Lalancette -- Libvir-list mailing list Libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list