Re: [libvirt] [PATCH]Fix minor issues in logical storage backend

2008-09-08 Thread Daniel Veillard
On Thu, Sep 04, 2008 at 02:21:34PM +0100, Daniel P. Berrange wrote: On Thu, Sep 04, 2008 at 03:13:45PM +0200, Daniel Veillard wrote: So there is an initialization of the field. It's unclear to me if the two should be OR'ed, or should be kept as-is, if we have the pool name do we need to

Re: [libvirt] [PATCH]Fix minor issues in logical storage backend

2008-09-04 Thread Daniel Veillard
On Thu, Aug 28, 2008 at 06:15:27PM -0600, Jim Fehlig wrote: Hi All! I came across some problems trying to create a new LVM-based storage pool using this config pool type=logical nametest_vg/name source device path=/dev/sdb1/ /source target path/dev/test_vg/path

Re: [libvirt] [PATCH]Fix minor issues in logical storage backend

2008-09-04 Thread Daniel P. Berrange
On Thu, Sep 04, 2008 at 03:13:45PM +0200, Daniel Veillard wrote: On Thu, Aug 28, 2008 at 06:15:27PM -0600, Jim Fehlig wrote: Hi All! I came across some problems trying to create a new LVM-based storage pool using this config pool type=logical nametest_vg/name source

[libvirt] [PATCH]Fix minor issues in logical storage backend

2008-08-28 Thread Jim Fehlig
Hi All! I came across some problems trying to create a new LVM-based storage pool using this config pool type=logical nametest_vg/name source device path=/dev/sdb1/ /source target path/dev/test_vg/path /target /pool Volume group did not previously exist so I did virsh