On Mon, 2010-04-19 at 11:28 +0200, Pavel Zuna wrote:
> On 04/14/2010 07:35 PM, Rob Crittenden wrote:
> > Pavel Zůna wrote:
> >> I noticed a few bugs when DNS was reinstalling:
> >>
> >> - Service.move_service returned None, because the service entry was
> >> already in the right place - BindInstance didn't expect that.
> >>
> >> - We were passing a unicode string to python-ldap although we know it
> >> hates that.
> >>
> >> - We were catching all exception alike when modifying the "dnsserver"
> >> role group. It's no longer an error if the DNS principal is already
> >> present.
> >>
> >> I think Martin has some work in progess on the bindinstance.py file,
> >> so please don't push until he acks it. He might want to included these
> >> changes in his own patch. I had to fix these to test my own code in
> >> the installer and posted the patch to point out the bugs.
> >
> > Interesting. Do we want to support re-installing the DNS server? Or
> > should we catch it and exit? Not crashing is definitely a good way to
> > start though :-)
> >
> > rob
> Improved version of the patch attached.
> 
> Pavel

Ack, pushed to master.
Martin

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

Reply via email to