Re: [Freeipa-devel] [PATCH] 874 suppress managed netgroups as indirect members of hosts
On Fri, 2011-09-16 at 17:24 -0400, Rob Crittenden wrote: > JR Aquino wrote: > > On Sep 14, 2011, at 1:39 PM, Rob Crittenden wrote: > > > >> Suppress managed netgroups as indirect members of hosts. This enhances a > >> previous patch that I did for hostgroups. > >> > >> rob > > > > Works as advertised: > > > > ACK > > > > Martin, ok to push? > > rob Agreed. Pushed to master, ipa-2-1. Martin ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 874 suppress managed netgroups as indirect members of hosts
JR Aquino wrote: On Sep 14, 2011, at 1:39 PM, Rob Crittenden wrote: Suppress managed netgroups as indirect members of hosts. This enhances a previous patch that I did for hostgroups. rob Works as advertised: ACK Martin, ok to push? rob ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 874 suppress managed netgroups as indirect members of hosts
On Sep 14, 2011, at 1:39 PM, Rob Crittenden wrote: > Suppress managed netgroups as indirect members of hosts. This enhances a > previous patch that I did for hostgroups. > > rob Works as advertised: ACK ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 874 suppress managed netgroups as indirect members of hosts
Martin Kosek wrote: On Wed, 2011-09-14 at 16:39 -0400, Rob Crittenden wrote: Suppress managed netgroups as indirect members of hosts. This enhances a previous patch that I did for hostgroups. rob This works fine. I just one suggestion for the code - the function suppress_netgroup_memberof() function was already implemented in the last patch: https://fedorahosted.org/freeipa/changeset/ca1ca17cb61516dff6933b1b0381b32e1e38d44c for hostgroup. I suggest making this function more general and calling it from both host and hostgroup objects. Martin I looked at that. For the hostgroup once you find your own entry you can exit, for hosts you have to look at all netgroups. The dn comparison is also very different. These could be handled as arguments but I think the code would be less clear so I chose quasi-duplication. rob ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 874 suppress managed netgroups as indirect members of hosts
On Wed, 2011-09-14 at 16:39 -0400, Rob Crittenden wrote: > Suppress managed netgroups as indirect members of hosts. This enhances a > previous patch that I did for hostgroups. > > rob This works fine. I just one suggestion for the code - the function suppress_netgroup_memberof() function was already implemented in the last patch: https://fedorahosted.org/freeipa/changeset/ca1ca17cb61516dff6933b1b0381b32e1e38d44c for hostgroup. I suggest making this function more general and calling it from both host and hostgroup objects. Martin ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel