laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved
bankd: Differentiate log levels, not everything is DEBUG

Change-Id: If3699642fee2d5d92fb164f7f9c41ccff02007cc
---
M src/bankd/bankd_main.c
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/bankd/bankd_main.c b/src/bankd/bankd_main.c
index 9e2d8f8..36fd651 100644
--- a/src/bankd/bankd_main.c
+++ b/src/bankd/bankd_main.c
@@ -170,7 +170,7 @@
struct client_slot cs;
RsproPDU_t *resp;

- LOGPFSM(srvc->fi, "Rx RSPRO %s\n", rspro_msgt_name(pdu));
+ LOGPFSML(srvc->fi, LOGL_DEBUG, "Rx RSPRO %s\n", rspro_msgt_name(pdu));

switch (pdu->msg.present) {
case RsproPDUchoice_PR_connectBankRes:
@@ -206,7 +206,7 @@
resp = rspro_gen_CreateMappingRes(ResultCode_ok);
goto send_resp;
} else {
- LOGPFSM(srvc->fi, "implicitly removing slotmap\n");
+ LOGPFSML(srvc->fi, LOGL_NOTICE, "implicitly removing slotmap\n");
bankd_srvc_remove_mapping(map);
}
}
@@ -243,10 +243,10 @@
} else {
rspro2client_slot(&cs, &rreq->client);
if (!client_slot_equals(&map->client, &cs)) {
- LOGPFSM(srvc->fi, "ClientId in removeMappingReq != map\n");
+ LOGPFSML(srvc->fi, LOGL_NOTICE, "ClientId in removeMappingReq != map\n");
resp = rspro_gen_RemoveMappingRes(ResultCode_unknownSlotmap);
} else {
- LOGPFSM(srvc->fi, "removing slotmap\n");
+ LOGPFSML(srvc->fi, LOGL_INFO, "removing slotmap\n");
bankd_srvc_remove_mapping(map);
resp = rspro_gen_RemoveMappingRes(ResultCode_ok);
}

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

Gerrit-Project: osmo-remsim
Gerrit-Branch: master
Gerrit-Change-Id: If3699642fee2d5d92fb164f7f9c41ccff02007cc
Gerrit-Change-Number: 27950
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged