[Freeipa-devel] [PATCH 0073] Remove support for IPA deployments with no persistent search

[Tomas Babej]

On Thursday 11 of July 2013 11:19:44 Tomas Babej wrote:
> On Friday 21 of June 2013 13:52:40 Ana Krivokapic wrote:
> > On 06/12/2013 02:28 PM, Tomas Babej wrote:
> > > Hi,
> > >
> > > Drops the code from ipa-server-install, ipa-dns-install and the
> > > BindInstance itself. Also changed ipa-upgradeconfig script so
> > > that it does not set zone_refresh to 0 on upgrades, as the option
> > > is deprecated, but rather removes it altogether.
> > >
> > > https://fedorahosted.org/freeipa/ticket/3632
> > >
> > > Tomas
> > >
> > >
> > > _______________________________________________
> > > Freeipa-devel mailing list
> > > Freeipa-devel at redhat.com
> > > https://www.redhat.com/mailman/listinfo/freeipa-devel
> > 
> > 1) ipa-server-install (with no DNS), followed by ipa-dns-install now fails,
> > because you missed one reference to options.persistent_search in ipa-dns-install:
> > 
> >     if options.serial_autoincrement and not options.persistent_search:
> >         parser.error('persistent search feature is required for '
> >                      'DNS SOA serial autoincrement')
> > 
> > 2) I wonder if we can also remove the '--zone-notif' option from
> > ipa-server-install and ipa-dns-install. It is already deprecated so maybe this
> > is a good time to drop it altogether?
> > 
> > 3) You can remove the 'persistant_search' attribute of the BindInstance class,
> > and just hardcode the value to "yes" in the '__setup_sub_dict()' method.
> > 
> 
> Updated patch adresses all 3 issues.
> 
> Tomas

And it comes here.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130711/15958a1e/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-tbabej-0073-2-Remove-support-for-IPA-deployments-with-no-persisten.patch
Type: text/x-patch
Size: 12088 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130711/15958a1e/attachment.bin>