From: Oguzhan Ozmen Date: Fri, 11 Apr 2025 21:45:19 +0000 (+0000) Subject: rgw: metadata and data sync fairness notifications to retry upon any error case X-Git-Tag: v19.2.3~183^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=0d9ee21d0714b2f347e0794b107957b2ae689f94;p=ceph.git rgw: metadata and data sync fairness notifications to retry upon any error case This is a complementary fix to the earlier one described at #62156. When the sync shard notification fails due to any failures including timeout, this change keeps the loop going for both metadata and data sync. Fixes https://tracker.ceph.com/issues/70270 Signed-off-by: Oguzhan Ozmen (cherry picked from commit 766b7a14315ae7ca9645b462ee774570caeff64c) --- diff --git a/src/rgw/driver/rados/rgw_data_sync.cc b/src/rgw/driver/rados/rgw_data_sync.cc index dbea56d4de78e..2c7a48a1edd52 100644 --- a/src/rgw/driver/rados/rgw_data_sync.cc +++ b/src/rgw/driver/rados/rgw_data_sync.cc @@ -2364,8 +2364,7 @@ public: set_status("sync lock notification"); yield call(sync_env->bid_manager->notify_cr()); if (retcode < 0) { - tn->log(5, SSTR("ERROR: failed to notify bidding information" << retcode)); - return set_cr_error(retcode); + tn->log(5, SSTR("ERROR: failed to notify bidding information retcode=" << retcode)); } set_status("sleeping"); diff --git a/src/rgw/driver/rados/rgw_sync.cc b/src/rgw/driver/rados/rgw_sync.cc index 583e2d1098e1c..62211fa6c160d 100644 --- a/src/rgw/driver/rados/rgw_sync.cc +++ b/src/rgw/driver/rados/rgw_sync.cc @@ -1996,8 +1996,7 @@ public: set_status("sync lock notification"); yield call(sync_env->bid_manager->notify_cr()); if (retcode < 0) { - tn->log(5, SSTR("ERROR: failed to notify bidding information" << retcode)); - return set_cr_error(retcode); + tn->log(5, SSTR("ERROR: failed to notify bidding information retcode=" << retcode)); } set_status("sleeping");