summaryrefslogtreecommitdiffstats
path: root/ipaserver/install
diff options
context:
space:
mode:
authorMartin Kosek <mkosek@redhat.com>2012-05-11 16:59:56 +0200
committerRob Crittenden <rcritten@redhat.com>2012-05-17 21:21:47 -0400
commitaa9ea477af7be67d59f204af4e0da67fd8cfd862 (patch)
tree12072ceae169c23a7db2c24143900a30da7cf32a /ipaserver/install
parent560f2ce8bd0525189e45ff7d8f8d4df11f9c20ff (diff)
downloadfreeipa-aa9ea477af7be67d59f204af4e0da67fd8cfd862.tar.gz
freeipa-aa9ea477af7be67d59f204af4e0da67fd8cfd862.tar.xz
freeipa-aa9ea477af7be67d59f204af4e0da67fd8cfd862.zip
Remove ipa-server-install LDAP update errors
python-ldap add_s method raises a NO_SUCH_OBJECT exception when a parent entry of the entry being added does not exist. This may not be an error, for example NIS entries are only added when NIS is enabled and thus the NIS entry container exists. The exception raised by python-ldap is also incorrectly processed in ipaldap's addEntry function and an irrelevant exception is re-raised instead. Fix LDAP updater to just log an information when an object cannot be added due to missing parent object. Also make sure that the addEntry function exception processing provides the right exception with a useful description. https://fedorahosted.org/freeipa/ticket/2520 https://fedorahosted.org/freeipa/ticket/2743
Diffstat (limited to 'ipaserver/install')
-rw-r--r--ipaserver/install/ldapupdate.py9
1 files changed, 8 insertions, 1 deletions
diff --git a/ipaserver/install/ldapupdate.py b/ipaserver/install/ldapupdate.py
index e803df8a2..61a2ae19f 100644
--- a/ipaserver/install/ldapupdate.py
+++ b/ipaserver/install/ldapupdate.py
@@ -649,7 +649,14 @@ class LDAPUpdate:
# addifexist may result in an entry with only a
# dn defined. In that case there is nothing to do.
# It means the entry doesn't exist, so skip it.
- self.conn.addEntry(entry)
+ try:
+ self.conn.addEntry(entry)
+ except errors.NotFound:
+ # parent entry of the added entry does not exist
+ # this may not be an error (e.g. entries in NIS container)
+ root_logger.info("Parent DN of %s may not exist, cannot create the entry",
+ entry.dn)
+ return
self.modified = True
except Exception, e:
root_logger.error("Add failure %s", e)