From: Samuel Just <sjust@redhat.com>
Date: Fri, 13 Dec 2024 21:33:08 +0000 (-0800)
Subject: crimson/.../replicated_request: call handle.complete() once in finally
X-Git-Tag: testing/wip-mchangir-testing-PR60889-main-debug~15^2~4
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=02b70a62a43c86123b196dbd478be2c3c3e272d4;p=ceph-ci.git

crimson/.../replicated_request: call handle.complete() once in finally

Signed-off-by: Samuel Just <sjust@redhat.com>
---

diff --git a/src/crimson/osd/osd_operations/replicated_request.cc b/src/crimson/osd/osd_operations/replicated_request.cc
index 1eca6c1fec4..5e173cf1d51 100644
--- a/src/crimson/osd/osd_operations/replicated_request.cc
+++ b/src/crimson/osd/osd_operations/replicated_request.cc
@@ -82,15 +82,12 @@ seastar::future<> RepRequest::with_pg(
       });
     }).then_interruptible([this, pg] (auto) {
       return pg->handle_rep_op(req);
-    }).then_interruptible([this] {
-      logger().debug("{}: complete", *this);
-      return handle.complete();
     });
   }, [](std::exception_ptr) {
     return seastar::now();
   }, pg, pg->get_osdmap_epoch()).finally([this, ref=std::move(ref)] {
     logger().debug("{}: exit", *this);
-    handle.exit();
+    return handle.complete();
   });
 }