Re: [Freeipa-devel] [PATCH] 404 Do not add ipa-ca records on CA-less installs

2013-05-09 Thread Petr Viktorin
On 05/09/2013 02:06 PM, Martin Kosek wrote: This should get to 3.2 GA. -- ipa-dns-install crashed when it was run on a CA-less server. https://fedorahosted.org/freeipa/ticket/3617 This solves the issue, ACK -- PetrĀ³ ___ Freeipa-devel mailing

Re: [Freeipa-devel] [PATCH] 404 Do not add ipa-ca records on CA-less installs

2013-05-09 Thread Martin Kosek
On 05/09/2013 02:39 PM, Petr Viktorin wrote: On 05/09/2013 02:06 PM, Martin Kosek wrote: This should get to 3.2 GA. -- ipa-dns-install crashed when it was run on a CA-less server. https://fedorahosted.org/freeipa/ticket/3617 This solves the issue, ACK Thanks. Pushed to master.

Re: [Freeipa-devel] [PATCH] 404 Do not add ipa-ca records on CA-less installs

2013-05-09 Thread Jan Cholasta
On 9.5.2013 15:14, Martin Kosek wrote: On 05/09/2013 02:39 PM, Petr Viktorin wrote: On 05/09/2013 02:06 PM, Martin Kosek wrote: This should get to 3.2 GA. -- ipa-dns-install crashed when it was run on a CA-less server. https://fedorahosted.org/freeipa/ticket/3617 This solves the issue,

Re: [Freeipa-devel] [PATCH] 404 Do not add ipa-ca records on CA-less installs

2013-05-09 Thread Martin Kosek
On 05/09/2013 05:44 PM, Jan Cholasta wrote: On 9.5.2013 15:14, Martin Kosek wrote: On 05/09/2013 02:39 PM, Petr Viktorin wrote: On 05/09/2013 02:06 PM, Martin Kosek wrote: This should get to 3.2 GA. -- ipa-dns-install crashed when it was run on a CA-less server.

Re: [Freeipa-devel] [PATCH] 404 Do not add ipa-ca records on CA-less installs

2013-05-09 Thread Jan Cholasta
On 9.5.2013 17:57, Martin Kosek wrote: On 05/09/2013 05:44 PM, Jan Cholasta wrote: Sorry for this, but NACK. With this patch ipa-ca records are not created for existing masters unless ipa-dns-install is run on a replica which has CA configured. You should instead put the ldap.get_entries() call

Re: [Freeipa-devel] [PATCH] 404 Do not add ipa-ca records on CA-less installs

2013-05-09 Thread Rob Crittenden
Jan Cholasta wrote: On 9.5.2013 17:57, Martin Kosek wrote: On 05/09/2013 05:44 PM, Jan Cholasta wrote: Sorry for this, but NACK. With this patch ipa-ca records are not created for existing masters unless ipa-dns-install is run on a replica which has CA configured. You should instead put the