On Tue, 2012-05-22 at 12:15 +0200, Petr Viktorin wrote:
> On 05/10/2012 09:31 AM, Martin Kosek wrote:
> > bind-dyndb-ldap persistent search queries LDAP for all DNS records.
> > The LDAP connection must have no size or time limits to work
> > properly.
> >
> > This patch updates limits both for existing service principal
> > on updated machine and for new service principals added
> > as a part of DNS installation.
> >
> > https://fedorahosted.org/freeipa/ticket/2531
> >
> 
> Yes, the limits are set correctly on both install and upgrade.
> ACK
> 
> 
> 
> Minor style nitpicks:
> 
> > +            root_logger.critical("Could not modify principal's %s entry: 
> > %s" \
> > +                    % (dns_principal, str(e)))
> 
> The backshashes are unnecessary inside parentheses.
> 
> > +        dnsupdates[dns_service_dn] = {'dn' : dns_service_dn,
> > +                                      'updates' : limit_updates}
> 
> Don't put a space before a semicolon.

I removed the space before the colon.

Pushed to master.

Martin

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to