Skip to content

Commit 302e3c4

Browse files
committed
rgw: increase log level on abort_early
The function is typically invoked on client errors like NoSuchBucket. Logging these errors with level 1 may initially suggest a significant issue, when in fact it's just a client error. Consider raising the logging level to 20 for better clarity. Fixes: https://tracker.ceph.com/issues/65469 Signed-off-by: Seena Fallah <[email protected]>
1 parent c2d8c31 commit 302e3c4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/rgw/rgw_rest.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -679,13 +679,13 @@ void abort_early(req_state *s, RGWOp* op, int err_no,
679679
if (op != NULL) {
680680
int new_err_no;
681681
new_err_no = op->error_handler(err_no, &error_content, y);
682-
ldpp_dout(s, 1) << "op->ERRORHANDLER: err_no=" << err_no
682+
ldpp_dout(s, 20) << "op->ERRORHANDLER: err_no=" << err_no
683683
<< " new_err_no=" << new_err_no << dendl;
684684
err_no = new_err_no;
685685
} else if (handler != NULL) {
686686
int new_err_no;
687687
new_err_no = handler->error_handler(err_no, &error_content, y);
688-
ldpp_dout(s, 1) << "handler->ERRORHANDLER: err_no=" << err_no
688+
ldpp_dout(s, 20) << "handler->ERRORHANDLER: err_no=" << err_no
689689
<< " new_err_no=" << new_err_no << dendl;
690690
err_no = new_err_no;
691691
}

0 commit comments

Comments
 (0)