[MERGED] osmo-gsm-tester[master]: log.py: add a log target option to omit the list of deeper o...

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Mon May 8 10:12:26 UTC 2017


Neels Hofmeyr has submitted this change and it was merged.

Change subject: log.py: add a log target option to omit the list of deeper origins
......................................................................


log.py: add a log target option to omit the list of deeper origins

Change-Id: I8a0a508ee4aeffe940be906de0e431740653a08c
---
M src/osmo_gsm_tester/log.py
1 file changed, 6 insertions(+), 3 deletions(-)

Approvals:
  Pau Espin Pedrol: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo_gsm_tester/log.py b/src/osmo_gsm_tester/log.py
index 0da9bd5..1c069e7 100644
--- a/src/osmo_gsm_tester/log.py
+++ b/src/osmo_gsm_tester/log.py
@@ -61,6 +61,7 @@
     do_log_category = None
     do_log_level = None
     do_log_origin = None
+    do_log_all_origins = None
     do_log_traceback = None
     do_log_src = None
     origin_width = None
@@ -86,7 +87,7 @@
     def remove(self):
         LogTarget.all_targets.remove(self)
 
-    def style(self, time=True, time_fmt=DATEFMT, category=True, level=True, origin=True, origin_width=32, src=True, trace=False):
+    def style(self, time=True, time_fmt=DATEFMT, category=True, level=True, origin=True, origin_width=32, src=True, trace=False, all_origins=True):
         '''
         set all logging format aspects, to defaults if not passed:
         time: log timestamps;
@@ -105,13 +106,14 @@
         self.do_log_category = bool(category)
         self.do_log_level = bool(level)
         self.do_log_origin = bool(origin)
+        self.do_log_all_origins = bool(all_origins)
         self.origin_width = int(origin_width)
         self.origin_fmt = '{:>%ds}' % self.origin_width
         self.do_log_src = src
         self.do_log_traceback = trace
         return self
 
-    def style_change(self, time=None, time_fmt=None, category=None, level=None, origin=None, origin_width=None, src=None, trace=None):
+    def style_change(self, time=None, time_fmt=None, category=None, level=None, origin=None, origin_width=None, src=None, trace=None, all_origins=None):
         'modify only the given aspects of the logging format'
         self.style(
             time=(time if time is not None else self.do_log_time),
@@ -122,6 +124,7 @@
             origin_width=(origin_width if origin_width is not None else self.origin_width),
             src=(src if src is not None else self.do_log_src),
             trace=(trace if trace is not None else self.do_log_traceback),
+            all_origins=(all_origins if all_origins is not None else self.do_log_all_origins),
             )
         return self
 
@@ -187,7 +190,7 @@
 
         log_line = [compose_message(messages, named_items)]
 
-        if deeper_origins:
+        if deeper_origins and self.do_log_all_origins:
             log_line.append(' [%s]' % deeper_origins)
 
         if self.do_log_src and src:

-- 
To view, visit https://gerrit.osmocom.org/2511
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8a0a508ee4aeffe940be906de0e431740653a08c
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>



More information about the gerrit-log mailing list