On 02.02.2016 12:18, Jan Cholasta wrote:
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:
https://fedorahosted.org/freeipa/ticket/5655
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
Honza.
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:
b9ae7690489368ead9f4983d386fa210dc265dfa
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.
LGTM + LGTM = ACK :)
Pushed to:
master: 612f4aa9003658f9a494ec327d50ec5a0592f7b4
ipa-4-3: d99552a8a9f855a7c5e00c4b0736061e05d6ed31
ipa-4-2: 3664efa31edf0dff6dd3410e2eccd12c9cd25782
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code