Skip to content

Commit

Permalink
Merge pull request ceph#54528 from yuvalif/wip-yuval-63532
Browse files Browse the repository at this point in the history
rgw/notifications: cleanup all coroutines after sending the notification

reviewed-by: cbodley
  • Loading branch information
yuvalif committed Nov 19, 2023
2 parents b9b9ec8 + 63e1489 commit 329e2a1
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/rgw/rgw_op.cc
Original file line number Diff line number Diff line change
Expand Up @@ -7283,14 +7283,15 @@ 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);
if (ret < 0) {
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)
Expand Down

0 comments on commit 329e2a1

Please sign in to comment.