From 240a328ae86bfb5a81ec5e435630a0ba734505d8 Mon Sep 17 00:00:00 2001 From: Seena Fallah Date: Mon, 12 Aug 2024 19:32:25 +0200 Subject: [PATCH] rgw: increase log level for enoent caused by clients 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 --- src/rgw/rgw_op.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc index 52b407cc9bd34..b1b79ef3b1f4b 100644 --- a/src/rgw/rgw_op.cc +++ b/src/rgw/rgw_op.cc @@ -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; -- 2.39.5