]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: increase log level on abort_early 56949/head
authorSeena Fallah <seenafallah@gmail.com>
Fri, 12 Apr 2024 19:59:41 +0000 (21:59 +0200)
committerCasey Bodley <cbodley@redhat.com>
Wed, 17 Apr 2024 12:38:10 +0000 (08:38 -0400)
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 <seenafallah@gmail.com>
(cherry picked from commit 302e3c459acdffe51fb73e902f2b3c828faa23e0)

src/rgw/rgw_rest.cc

index 020309038eb3c39ba62b17d584733228e8a51d56..3a0652e82e2caf6bdf2f608ae5f0740713ab4f17 100644 (file)
@@ -679,13 +679,13 @@ void abort_early(req_state *s, RGWOp* op, int err_no,
   if (op != NULL) {
     int new_err_no;
     new_err_no = op->error_handler(err_no, &error_content, y);
-    ldpp_dout(s, 1) << "op->ERRORHANDLER: err_no=" << err_no
+    ldpp_dout(s, 20) << "op->ERRORHANDLER: err_no=" << err_no
                      << " new_err_no=" << new_err_no << dendl;
     err_no = new_err_no;
   } else if (handler != NULL) {
     int new_err_no;
     new_err_no = handler->error_handler(err_no, &error_content, y);
-    ldpp_dout(s, 1) << "handler->ERRORHANDLER: err_no=" << err_no
+    ldpp_dout(s, 20) << "handler->ERRORHANDLER: err_no=" << err_no
                      << " new_err_no=" << new_err_no << dendl;
     err_no = new_err_no;
   }