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-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F62356%2Fhead;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 dbea56d4de78..2c7a48a1edd5 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 583e2d1098e1..62211fa6c160 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");