Re: [PATCH] conf: storage: Fix duplicate check for gluster pools

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 30, 2015 at 06:48:45 -0400, John Ferlan wrote:
> 
> 
> On 06/30/2015 04:20 AM, Peter Krempa wrote:
> > The pool name has to be the same too to warrant rejecting a pool
> > definition as duplicate. This regression was introduced in commit
> > 2184ade3a0546b915252cb3b6a5dc88e9a8d2ccf.
> > 
> 
> How did it check duplicates before that commit? The difference there

It was not checked in any way :), thus ... 

> shows no checks being done on gluster at all... So semantically, not
> sure it's a regression...  That commit just added a check to ensure the
> host names and 'dir' weren't the same...  If anything it didn't go "far
> enough" to check the source.name.

If you have a volume with different name on the same server and you want
to use the same path there then libvirt will forbid that and that would
not be forbidden before.

$ git desc 2184ade3a0546b915252cb3b6a5dc88e9a8d2ccf
v1.2.14-168-g2184ade

As it was released already it is a regression in legitimate behaviour.

> 
> 
> > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1236438
> > ---
> >  src/conf/storage_conf.c | 11 ++++++++++-
> >  1 file changed, 10 insertions(+), 1 deletion(-)
> > 
> 
> While rereading virStorageBackendGlusterOpen, I'm now wondering... Does
> 'dir' have to be provided?  The virStorageBackendGlusterOpen indicates
> that 'dir' could be NULL and would default to "/"
> 
> ACK (although perhaps the STREQ_NULLABLE may need to be used for dir)

You are right, "dir" actually at least according to the code in the
gluster backend can be NULL, so I'll add the _NULLABLE part.

> 
> John

Thanks.

Peter

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]