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. -- To view, visit https://gerrit.osmocom.org/c/osmo-gbproxy/+/29715 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gbproxy Gerrit-Branch: master Gerrit-Change-Id: Ieddf20a5eafaa9738d5d1e9a9aae320e48aa2efc Gerrit-Change-Number: 29715 Gerrit-PatchSet: 3 Gerrit-Owner: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged