gp: Log ext failure with file and line number

Rather than dumping a traceback when there is a
failure, simply log the file name, line number
and the error message instead. This is much
cleaner.

Signed-off-by: David Mulder <dmulder@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>

diff --git a/python/samba/gp/gpclass.py b/python/samba/gp/gpclass.py
index 4737bfe07380e3c3fb507bdb9e52d8a19795c5b9..bec1ec5fecf8897c7caa1032ded264589774f58f 100644 (file)
--- a/python/samba/gp/gpclass.py
+++ b/python/samba/gp/gpclass.py
@@ -677,8 +677,10 @@ def apply_gp(lp, creds, store, gp_extensions, username, target, force=False):
                                 del_gpos, changed_gpos)
         except Exception as e:
             log.error('Failed to apply extension  %s' % str(ext))
-            log.error('Message was: %s: %s' % (type(e).__name__, str(e)))
-            log.debug(traceback.format_exc())
+            _, _, tb = sys.exc_info()
+            filename, line_number, _, _ = traceback.extract_tb(tb)[-1]
+            log.error('%s:%d: %s: %s' % (filename, line_number,
+                                         type(e).__name__, str(e)))
             continue
     for gpo_obj in gpos:
         if not gpo_obj.file_sys_path: