On 2.2.2016 11:46, Jan Cholasta wrote:
On 2.2.2016 11:41, Martin Babinsky wrote:
On 02/02/2016 09:33 AM, Jan Cholasta wrote:
On 1.2.2016 14:54, Martin Basti wrote:

On 01.02.2016 13:55, Martin Babinsky wrote:

LGTM, works for me, tested on both ca-less server and CA-full server.

Because patch is touching certmonger I would like to get final ACK from

This is suspicious:

-    if is_ca_enabled():
-        http.configure_certmonger_renewal_guard()
+    http.configure_certmonger_renewal_guard()

Why is it necessary?

I don't know, you tell me:

What I meant is why is the change necessary, not why is the original
code necessary.

Ah, the original code didn't have the condition. LGTM then.

Jan Cholasta

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to