From: Yuval Lifshitz Date: Thu, 16 Nov 2023 15:47:06 +0000 (+0000) Subject: rgw/notifications: cleanup all coroutines after sending the notification X-Git-Tag: v17.2.8~138^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=e76ee07ebc10f3cd5c78567c016c01c14227324a;p=ceph.git rgw/notifications: cleanup all coroutines after sending the notification this is fixing a regression from: 6b6592f50b6b81fa13a330bcb91273ba7f25c0c9 Fixes: https://tracker.ceph.com/issues/63580 Signed-off-by: Yuval Lifshitz (cherry picked from commit 63e14893cc7dc6a3aec92c0867ce5e3e67f44b29) --- diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc index 1056c1ea043..ffb36c4d816 100644 --- a/src/rgw/rgw_op.cc +++ b/src/rgw/rgw_op.cc @@ -6993,7 +6993,6 @@ void RGWDeleteMultiObj::handle_individual_object(const rgw_obj_key& o, optional_ op_ret = 0; } - send_partial_response(o, del_op->result.delete_marker, del_op->result.version_id, op_ret, formatter_flush_cond); // send request to notification manager int ret = res->publish_commit(this, obj_size, ceph::real_clock::now(), etag, version_id); @@ -7001,6 +7000,8 @@ void RGWDeleteMultiObj::handle_individual_object(const rgw_obj_key& o, optional_ ldpp_dout(this, 1) << "ERROR: publishing notification failed, with error: " << ret << dendl; // too late to rollback operation, hence op_ret is not set here } + + send_partial_response(o, del_op->result.delete_marker, del_op->result.version_id, op_ret, formatter_flush_cond); } void RGWDeleteMultiObj::execute(optional_yield y)