[Freeipa-devel] [PATCH] 0043 fix ipa-dns-install to not require DM password

[Simo Sorce]

On Thu, 2011-01-06 at 10:35 +0100, Jan Zelený wrote:
> Simo Sorce <ssorce at redhat.com> wrote:
> > This patch makes it possible to run ipa-dns-install and use the admin
> > kerberos credentials.
> > 
> > Fixes #686.
> > 
> > Simo.
> 
> Nack, I have some comments:
> 
> Exception handling (chunk #4):
> Those prints should go away. But the main thing: that particular part of code 
> doesn't seem to produce any exceptions, which should be handled
> 
> Function ldap_disconnect isn't used anywhere. That makes me wonder - is it 
> redundant or should it be somewhere in the code. I guess this is a policy 
> issue - either we want the connection to stay as long as possible or we want 
> to use it only for a certain set of commands and then disconnect it.

Attached new patch that fixes hunk #4.
Actually I ended up using ldap_disconnect() here as we need to test the
ldap connection anyway.
I also had to do minor changes to Bindinstance() as the code was
clearing self.fqdn after Service.__init__ set it.

Simo.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-simo-0043-2-Allow-ipa-dns-install-to-install-with-just-admin-cre.patch
Type: application/mbox
Size: 17961 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110106/2f570f25/attachment.mbox>