fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/39415?usp=email )

Change subject: Add GSMTAP logging handler
......................................................................

Add GSMTAP logging handler

This patch integrates a logging handler [1][2] for Erlang's built-in
logger.  It sends logging messages via GSMTAP (udp/4729), allowing
capture of log messages alongside with other PDUs (S1AP, PFCP)
sent/received by the gateway.

The new handler is added to the example configuration, sending
all logging messages (DEBUG and above) to 127.0.0.1 by default.

[1] https://www.erlang.org/doc/apps/kernel/logger_handler.html
[2] https://gitea.osmocom.org/erlang/logger_gsmtap

Change-Id: Iea884e2ca146b852c64bd9f135d8c71f4c925f09
---
M config/sys.config
M rebar.config
M rebar.lock
M src/osmo_s1gw.app.src
4 files changed, 15 insertions(+), 3 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  osmith: Looks good to me, but someone else must approve




diff --git a/config/sys.config b/config/sys.config
index f73b33f..4e5a80c 100644
--- a/config/sys.config
+++ b/config/sys.config
@@ -31,10 +31,15 @@
  %% kernel config
  %% 
================================================================================
  {kernel,
-  [{logger_level, info},
+  [{logger_level, debug},
    {logger,
-    [{handler, default, logger_std_h,
-      #{formatter =>
+    [{handler, gsmtap, logger_gsmtap_h,
+      #{level => debug,
+        config => #{rem_addr => "127.0.0.1",
+                    app_name => "OsmoS1GW"}}},
+     {handler, default, logger_std_h,
+      #{level => info,
+        formatter =>
         {logger_color_formatter, #{legacy_header => false,
                                    single_line => false,
                                    template => [time, " ", color, "[", level, 
"]",
diff --git a/rebar.config b/rebar.config
index 47ff372..de8b023 100644
--- a/rebar.config
+++ b/rebar.config
@@ -7,6 +7,8 @@
 %% run-time deps
 {deps, [{logger_color_formatter,
          {git, "https://github.com/rlipscombe/logger_color_formatter.git";, 
{tag, "0.5.0"}}},
+        {logger_gsmtap,
+         {git, "https://gitea.osmocom.org/erlang/logger_gsmtap.git";, {branch, 
"master"}}},
         {pfcplib,
          {git, "https://github.com/travelping/pfcplib.git";, {branch, 
"master"}}},
         {enftables,
diff --git a/rebar.lock b/rebar.lock
index ccee08d..02a4edf 100644
--- a/rebar.lock
+++ b/rebar.lock
@@ -21,6 +21,10 @@
   {git,"https://github.com/rlipscombe/logger_color_formatter.git";,
        {ref,"f1c96f979e6350f8cd787d27fe9ff003cbf3416b"}},
   0},
+ {<<"logger_gsmtap">>,
+  {git,"https://gitea.osmocom.org/erlang/logger_gsmtap.git";,
+       {ref,"e0d803fc1b1b1b66ec0c6986d1c542423ebc0320"}},
+  0},
  {<<"parse_trans">>,{pkg,<<"parse_trans">>,<<"3.4.1">>},1},
  {<<"pfcplib">>,
   {git,"https://github.com/travelping/pfcplib.git";,
diff --git a/src/osmo_s1gw.app.src b/src/osmo_s1gw.app.src
index e64fb93..13240dd 100644
--- a/src/osmo_s1gw.app.src
+++ b/src/osmo_s1gw.app.src
@@ -8,6 +8,7 @@
                        kernel,
                        stdlib,
                        logger_color_formatter,
+                       logger_gsmtap,
                        pfcplib,
                        enftables,
                        exometer_core,

--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/39415?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Iea884e2ca146b852c64bd9f135d8c71f4c925f09
Gerrit-Change-Number: 39415
Gerrit-PatchSet: 10
Gerrit-Owner: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>

Reply via email to