Re: [libvirt] [PATCH 1/2] storage: Need to ensure removal of voldef from driver list

2014-05-07 Thread John Ferlan
On 05/07/2014 08:52 AM, Ján Tomko wrote: > On 05/07/2014 02:08 AM, John Ferlan wrote: >> If for some reason a volume cannot be built in the pool (eg, the >> buildVol call fails), then the error code may or may not properly >> remove the volume from the pool->volumes.objs[] list. >> >> Removal fro

Re: [libvirt] [PATCH 1/2] storage: Need to ensure removal of voldef from driver list

2014-05-07 Thread Ján Tomko
On 05/07/2014 02:08 AM, John Ferlan wrote: > If for some reason a volume cannot be built in the pool (eg, the > buildVol call fails), then the error code may or may not properly > remove the volume from the pool->volumes.objs[] list. > > Removal from the list is dependent on the 'deleteVol()' bein

[libvirt] [PATCH 1/2] storage: Need to ensure removal of voldef from driver list

2014-05-06 Thread John Ferlan
If for some reason a volume cannot be built in the pool (eg, the buildVol call fails), then the error code may or may not properly remove the volume from the pool->volumes.objs[] list. Removal from the list is dependent on the 'deleteVol()' being present in the backend *and* calling it returning s