s4-s3-upgrade: Add --verbose option to print extra details
authorAmitay Isaacs <amitay@gmail.com>
Thu, 17 Nov 2011 07:51:14 +0000 (18:51 +1100)
committerAmitay Isaacs <amitay@gmail.com>
Fri, 18 Nov 2011 03:38:28 +0000 (14:38 +1100)
source4/scripting/python/samba/netcmd/domain.py
source4/scripting/python/samba/upgrade.py

index 9f77820855f92404c3e881ef28fe0362db538530..0e6083a8b0d54324074d456df4bf960858efc28e 100644 (file)
@@ -528,6 +528,7 @@ class cmd_domain_samba3upgrade(Command):
         Option("--targetdir", type="string", metavar="DIR",
                   help="Path prefix where the new Samba 4.0 AD domain should be initialised"),
         Option("--quiet", help="Be quiet", action="store_true"),
+        Option("--verbose", help="Be verbose", action="store_true"),
         Option("--use-xattrs", type="choice", choices=["yes","no","auto"], metavar="[yes|no|auto]",
                    help="Define if we should use the native fs capabilities or a tdb file for storing attributes likes ntacl, auto tries to make an inteligent guess based on the user rights and system capabilities", default="auto"),
     ]
@@ -535,7 +536,7 @@ class cmd_domain_samba3upgrade(Command):
     takes_args = ["smbconf"]
 
     def run(self, smbconf=None, targetdir=None, dbdir=None, testparm=None, 
-            quiet=False, use_xattrs=None, sambaopts=None, versionopts=None):
+            quiet=False, verbose=False, use_xattrs=None, sambaopts=None, versionopts=None):
 
         if not os.path.exists(smbconf):
             raise CommandError("File %s does not exist" % smbconf)
@@ -550,7 +551,9 @@ class cmd_domain_samba3upgrade(Command):
             raise CommandError("Please specify either dbdir or testparm")
 
         logger = self.get_logger()
-        if quiet:
+        if verbose:
+            logger.setLevel(logging.DEBUG)
+        elif quiet:
             logger.setLevel(logging.WARNING)
         else:
             logger.setLevel(logging.INFO)
index 5b0088737cb9177cde8195b3c1d23fa1d187da99..11999608aa1a66ec254d11295da18738b90d6289 100644 (file)
@@ -237,7 +237,7 @@ def add_users_to_group(samdb, group, members, logger):
             samdb.modify(m)
         except ldb.LdbError, (ecode, emsg):
             if ecode == ldb.ERR_ENTRY_ALREADY_EXISTS:
-                logger.info("skipped re-adding member '%s' to group '%s': %s", member_sid, group.sid, emsg)
+                logger.debug("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: