From fcc540bbdc5daa24990940124ec3bd439b05257d Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Tue, 1 Mar 2016 17:36:55 +0100 Subject: [PATCH] Fix connections to DS during installation Regression caused by commit 9818e463f5d0a91b300801ee7c8f31f25de402b2, admin_conn should be connected in method if there is no connection. https://fedorahosted.org/freeipa/ticket/5665 Reviewed-By: Petr Vobornik --- ipaserver/install/dsinstance.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/ipaserver/install/dsinstance.py b/ipaserver/install/dsinstance.py index 93af0ac0a..b7a480749 100644 --- a/ipaserver/install/dsinstance.py +++ b/ipaserver/install/dsinstance.py @@ -1067,6 +1067,9 @@ class DsInstance(service.Service): """ Add sidgen plugin configuration only if it does not already exist. """ + if not self.admin_conn: + self.ldap_connect() + dn = DN('cn=IPA SIDGEN,cn=plugins,cn=config') try: self.admin_conn.get_entry(dn) @@ -1085,6 +1088,9 @@ class DsInstance(service.Service): """ Add extdom configuration if it does not already exist. """ + if not self.admin_conn: + self.ldap_connect() + dn = DN('cn=ipa_extdom_extop,cn=plugins,cn=config') try: self.admin_conn.get_entry(dn)