From a9a3a79767401ba1165fde9aeb8820c012ef5470 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Tue, 6 Sep 2011 08:14:35 +1000 Subject: [PATCH] s4-s3-upgrade Handle expected errors, error out on unexpected ones Autobuild-User: Andrew Bartlett Autobuild-Date: Wed Sep 7 02:22:56 CEST 2011 on sn-devel-104 --- source4/scripting/python/samba/upgrade.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/source4/scripting/python/samba/upgrade.py b/source4/scripting/python/samba/upgrade.py index 7e16eeee2c0..2bb5ccccbe1 100644 --- a/source4/scripting/python/samba/upgrade.py +++ b/source4/scripting/python/samba/upgrade.py @@ -208,10 +208,12 @@ def add_users_to_group(samdb, group, members, logger): try: samdb.modify(m) except ldb.LdbError, (ecode, emsg): - if ecode == ldb.ERR_NO_SUCH_OBJECT: - logger.warn("Could not add member '%s' to group '%s' as either group or user record doesn't exist: %s", member_sid, group.sid, emsg) + if ecode == ldb.ERR_ENTRY_ALREADY_EXISTS: + logger.info("skipped re-adding member '%s' to group '%s': %s", member_sid, group.sid, emsg) + elif ecode == ldb.ERR_NO_SUCH_OBJECT: + raise ProvisioningError("Could not add member '%s' to group '%s' as either group or user record doesn't exist: %s" % (member_sid, group.sid, emsg)) else: - logger.warn("Could not add member '%s' to group '%s': %s", member_sid, group.sid, emsg) + raise ProvisioningError("Could not add member '%s' to group '%s': %s" % (member_sid, group.sid, emsg)) def import_wins(samba4_winsdb, samba3_winsdb): -- 2.11.4.GIT