daniel has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gbproxy/+/29715 )

Change subject: Improve log levels ......................................................................

Improve log levels

* Set log levels to LOGL_NOTICE by default * Set log message about pooling being disabled to debug

Change-Id: Ieddf20a5eafaa9738d5d1e9a9aae320e48aa2efc --- M src/gb_proxy.c M src/gb_proxy_main.c 2 files changed, 4 insertions(+), 2 deletions(-)

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve msuraev: Looks good to me, but someone else must approve laforge: Looks good to me, approved lynxis lazus: Looks good to me, approved

diff --git a/src/gb_proxy.c b/src/gb_proxy.c index 919ed61..a7554a6 100644 --- a/src/gb_proxy.c +++ b/src/gb_proxy.c @@ -278,7 +278,7 @@ return NULL; }

- LOGPSGSN(sgsn, LOGL_INFO, "Pooling disabled, using first available SGSN\n"); + LOGPSGSN(sgsn, LOGL_DEBUG, "Pooling disabled, using first available SGSN\n"); } else { /* Pooling is enabled, try to use the NRI for routing to an SGSN * See 3GPP TS 23.236 Ch. 5.3.2 */ diff --git a/src/gb_proxy_main.c b/src/gb_proxy_main.c index 1a4e951..ba49a77 100644 --- a/src/gb_proxy_main.c +++ b/src/gb_proxy_main.c @@ -237,12 +237,14 @@ [DGPRS] = { .name = "DGPRS", .description = "GPRS Packet Service", - .enabled = 1, .loglevel = LOGL_DEBUG, + .enabled = 1, + .loglevel = LOGL_NOTICE, }, [DOBJ] = { .name = "DOBJ", .description = "GbProxy object allocation/release", .enabled = 1, + .loglevel = LOGL_NOTICE, .color = "\033[38;5;121m" }, };

2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.