Re: [PATCH v2 18/20] network: Rename @filter to @aclfilter

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

 



On Wed, Aug 16, 2017 at 09:36:36AM +0200, Michal Privoznik wrote:
> On 08/15/2017 10:56 PM, John Ferlan wrote:
> > 
> > 
> > On 08/15/2017 11:32 AM, Michal Privoznik wrote:
> >> On 07/26/2017 05:05 PM, John Ferlan wrote:
> >>> Rename the virNetworkObjListFilter to be virNetworkObjListACLFilter
> >>> since that's more representative of what it is. Also modify the
> >>> variable @filter to be @aclfilter. In the future adding the ability
> >>> to describe a generic @filter routine for the Export functions
> >>> could be a useful thing.
> >>
> >> Well technically this is a filter. It's only that we use ACL filter
> >> function for it. But the implementation is generic enough for the cb to
> >> be called filter IMO. Therefore I'm not a fan of this one.
> >>
> >> Michal
> >>
> > 
> > Understood - I can drop it, but then it's different than what I've
> > already done in nwfilter, secret, nodedevice, and storage.
> 
> Darn, did those slip in? Frankly, I'm not a fan of this change.
> Therefore I'll no longer object to this change, but probably not ACK it
> either. I'd like others to chime in and express their opinion.

I agree with Michal and I've already pointed this out for the first
series [1].  In my second response I gave up :) but I still think
that it should be made as a generic filter.  Since now I'm not the only
one against this change we should consider "fixing" the other places
that already use aclfilter.

Pavel

[1] <https://www.redhat.com/archives/libvir-list/2017-July/msg00973.html>

Attachment: signature.asc
Description: PGP 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]
  Powered by Linux