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@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list