laforge submitted this change.

View Change


Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
config/sys.config: print prefix before msg, if present

It's sometimes desirable to add some context to logging messages,
like the FSM name and identifier, for instance. With this patch,
this can be easily achieved by passing an arbitrary prefix string
to a logging function/macro:

%% EXAMPLE: adding metadata to a specific log macro
?LOG_INFO("The planet is fine...", #{prefix = "EARTH"}).

or by setting it for all logging of a process:

%% EXAMPLE: adding context for the whole process
logger:set_process_metadata(#{prefix => "E-RAB FSM"}).

Change-Id: Ib88801b8d38976b98a02ed06e1638e0e9056bafb
---
M config/sys.config
1 file changed, 23 insertions(+), 0 deletions(-)

diff --git a/config/sys.config b/config/sys.config
index 5ccd43d..cf8ff34 100644
--- a/config/sys.config
+++ b/config/sys.config
@@ -27,6 +27,7 @@
single_line => false,
template => [time, " ", color, "[", level, "]",
{pid, [" ", pid, ""], ""},
+ {prefix, [" ", prefix, " ::"], ""},
" ", msg,
{mfa, [" (", mfa, ":", line, ")"], ""},
reset, "\n"]}}}}]}]}].

To view, visit change 37327. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ib88801b8d38976b98a02ed06e1638e0e9056bafb
Gerrit-Change-Number: 37327
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged