From 520e2ed9c5b2cfe3e3231bd616639bddb16d6995 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 cadf9ccbe8ed0a20813af3fd671b18942a918b0b..4ad0f9e7def8a10b1eaffce1b3d9cadd9cdcc689 100644 --- a/ipaserver/install/dsinstance.py +++ b/ipaserver/install/dsinstance.py @@ -938,6 +938,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) @@ -956,6 +959,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) -- 2.5.0