On 02/09/14 11:59, Martin Basti wrote:
On 02/09/14 09:10, Jan Cholasta wrote:
Hi,

Dne 1.9.2014 v 16:57 Martin Basti napsal(a):
This patch allows to disable service in LDAP to prevents service to be
started by "ipactl restart"

Required by DNSSEC

Patch attached

I don't think the extra argument in ldap_enable is necessary. It should enable the service no matter if the entry existed before or not.

Similarly, in ldap_disable you should not raise an error when the entry is not found, because that already makes the service disabled.

Honza

Updated patch attached



_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
Please review, this should be in 4.1

--
Martin Basti

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to