daniel submitted this change.

View Change

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
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(-)

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 change 29715. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-gbproxy
Gerrit-Branch: master
Gerrit-Change-Id: Ieddf20a5eafaa9738d5d1e9a9aae320e48aa2efc
Gerrit-Change-Number: 29715
Gerrit-PatchSet: 3
Gerrit-Owner: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: msuraev <msuraev@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged