proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-07-27 13:46:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

lib: move non-error from __log_err to __dbg

Browse Source 
Additionally, print `errmsg_if_any` in successful debug messages
if non-NULL.

fixes #16386 #16043

Signed-off-by: Christian Hopps <chopps@labn.net>
(cherry picked from commit 7afd7d99f2)
This commit is contained in:
Christian Hopps 2024-07-22 07:52:10 -04:00 committed by Mergify
parent c9bd77c918
commit e8506a06db
2 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/vty.c
View File

@ -3574,8 +3574,9 @@ static void vty_mgmt_set_config_result_notified(
errmsg_if_any ? errmsg_if_any : "Unknown");
} else {
debug_fe_client("SET_CONFIG request for client 0x%" PRIx64
" req-id %" PRIu64 " was successfull",
client_id, req_id);
" req-id %" PRIu64 " was successfull%s%s",
client_id, req_id, errmsg_if_any ? ": " : "",
errmsg_if_any ?: "");
}
if (implicit_commit) {
@ -3606,8 +3607,9 @@ static void vty_mgmt_commit_config_result_notified(
errmsg_if_any ? errmsg_if_any : "Unknown");
} else {
debug_fe_client("COMMIT_CONFIG request for client 0x%" PRIx64
" req-id %" PRIu64 " was successfull",
client_id, req_id);
" req-id %" PRIu64 " was successfull%s%s",
client_id, req_id, errmsg_if_any ? ": " : "",
errmsg_if_any ?: "");
if (errmsg_if_any)
vty_out(vty, "MGMTD: %s\n", errmsg_if_any);
}
@ -3638,8 +3640,9 @@ static int vty_mgmt_get_data_result_notified(
}
debug_fe_client("GET_DATA request succeeded, client 0x%" PRIx64
" req-id %" PRIu64,
client_id, req_id);
" req-id %" PRIu64 "%s%s",
client_id, req_id, errmsg_if_any ? ": " : "",
errmsg_if_any ?: "");
if (req_id != mgmt_last_req_id) {
mgmt_last_req_id = req_id;

4
mgmtd/mgmt_txn.c
View File

@ -919,7 +919,7 @@ static int mgmt_txn_create_config_batches(struct mgmt_txn_req *txn_req,
}
if (!chg_clients)
__log_err("No connected daemon is interested in XPATH %s",
__dbg("Daemons interested in XPATH are not currently connected: %s",
xpath);
cmtcfg_req->clients |= chg_clients;
@ -931,7 +931,7 @@ static int mgmt_txn_create_config_batches(struct mgmt_txn_req *txn_req,
if (!num_chgs) {
(void)mgmt_txn_send_commit_cfg_reply(txn_req->txn,
MGMTD_NO_CFG_CHANGES,
"No changes found to commit!");
"No connected daemons interested in changes");
return -1;
}