[libvirt] [libvirt-glib 1/4] gobject: don't try to use pool's volumes before a successful refresh

[Christophe Fergeau]

On Wed, Nov 14, 2012 at 02:05:11PM +0100, Zeeshan Ali (Khattak) wrote:
> On Wed, Nov 14, 2012 at 10:31 AM, Christophe Fergeau
> <cfergeau at redhat.com> wrote:
> > On Tue, Nov 13, 2012 at 06:45:56PM +0100, Zeeshan Ali (Khattak) wrote:
> >> On Tue, Nov 6, 2012 at 1:45 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
> >> > gvir_storage_pool_refresh must be called and must be successful before
> >> > trying to use gvir_storage_pool_get_volume,
> >> > gvir_storage_pool_get_volumes and gvir_storage_pool_create_volume.
> >> > As the storage pool refresh can fail for reasons external to
> >> > libvirt/libvirt-gobject, the library user should check _refresh
> >> > errors. This commit outputs runtime warnings when these functions
> >> > are called and GVirObjectStoragePool::priv::volumes is NULL.
> >>
> >> Looks good. ACK!
> >
> > Just this patch or the whole series?
> 
> I've been reviewing/ACK'ing each patch separately so this one is only
> for this patch. :) Let me know if I missed a patch.

You missed the 3 other patches in this series then if I'm not mistaken ;)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20121114/fbe184c5/attachment-0001.sig>