]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: increase log level for enoent caused by clients 59172/head
authorSeena Fallah <seenafallah@gmail.com>
Mon, 12 Aug 2024 17:32:25 +0000 (19:32 +0200)
committerSeena Fallah <seenafallah@gmail.com>
Mon, 12 Aug 2024 17:32:25 +0000 (19:32 +0200)
RGWGetBucketPolicy, RGWGetBucketPublicAccessBlock, RGWGetBucketEncryption
logs with level 0 when the bucket doesn't have those documents. As
this is a normal client error, this can be logged with level 20 as
this does not indicate any serious error.

Fixes: https://tracker.ceph.com/issues/67500
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
src/rgw/rgw_op.cc

index 52b407cc9bd34a598121f277f4776ca61516cbf1..b1b79ef3b1f4b4545e9fb3a99a81b147be90399d 100644 (file)
@@ -8231,7 +8231,7 @@ void RGWGetBucketPolicy::execute(optional_yield y)
   rgw::sal::Attrs attrs(s->bucket_attrs);
   auto aiter = attrs.find(RGW_ATTR_IAM_POLICY);
   if (aiter == attrs.end()) {
-    ldpp_dout(this, 0) << "can't find bucket IAM POLICY attr bucket_name = "
+    ldpp_dout(this, 20) << "can't find bucket IAM POLICY attr bucket_name = "
         << s->bucket_name << dendl;
     op_ret = -ERR_NO_SUCH_BUCKET_POLICY;
     s->err.message = "The bucket policy does not exist";
@@ -8746,7 +8746,7 @@ void RGWGetBucketPublicAccessBlock::execute(optional_yield y)
   auto attrs = s->bucket_attrs;
   if (auto aiter = attrs.find(RGW_ATTR_PUBLIC_ACCESS);
       aiter == attrs.end()) {
-    ldpp_dout(this, 0) << "can't find bucket IAM POLICY attr bucket_name = "
+    ldpp_dout(this, 20) << "can't find bucket IAM POLICY attr bucket_name = "
                       << s->bucket_name << dendl;
 
     op_ret = -ERR_NO_SUCH_PUBLIC_ACCESS_BLOCK_CONFIGURATION;
@@ -8878,7 +8878,7 @@ void RGWGetBucketEncryption::execute(optional_yield y)
   const auto& attrs = s->bucket_attrs;
   if (auto aiter = attrs.find(RGW_ATTR_BUCKET_ENCRYPTION_POLICY);
       aiter == attrs.end()) {
-    ldpp_dout(this, 0) << "can't find BUCKET ENCRYPTION attr for bucket_name = " << s->bucket_name << dendl;
+    ldpp_dout(this, 20) << "can't find BUCKET ENCRYPTION attr for bucket_name = " << s->bucket_name << dendl;
     op_ret = -ENOENT;
     s->err.message = "The server side encryption configuration was not found";
     return;