[Freeipa-devel] [PATCH 0043] Properly handle ipa-replica-install when its zone is not managed by IPA

[Ana Krivokapic]

On 03/26/2013 04:59 PM, Tomas Babej wrote:
> Hi,
>
> The ipa-replica-install script tries to add replica's A and PTR
> records to the master DNS, if master does manage DNS. However,
> master need not to manage replica's zone. Properly handle this use
> case.
>
> https://fedorahosted.org/freeipa/ticket/3496
>
>
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel

The patch works well and fixes the issue.

Just a couple of nitpicks:

1) "However, master need not to manage replica's zone." -- This sentence
sounds a little strange to me, but I am not a native speaker so I may be
wrong about that.

2) There are three PEP8 501 errors introduced by the patch, but given
the recent discussion on this subject, I think it is really up to you if
you want to take the time to fix these.

ACK from the functional perspective.

-- 
Regards,

Ana Krivokapic
Associate Software Engineer
FreeIPA team
Red Hat Inc.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130329/b6b3015d/attachment.htm>