crimson/osd: fix osd shutdown problem

when doing full-system shutdown, monitors may go down before OSDs in which case
the osd shutdown hangs waiting for monc to successfully send the markmedown msg
to monitors

Fixes: https://tracker.ceph.com/issues/46564
Signed-off-by: Xuehan Xu <xxhdx1985126@163.com>

diff --git a/src/crimson/osd/osd.cc b/src/crimson/osd/osd.cc
index 694317f94fe84d2f47c405c0459c3417236f231a..5986368a41dcac0743a87f772f841317b0497cf9 100644 (file)
--- a/src/crimson/osd/osd.cc
+++ b/src/crimson/osd/osd.cc
@@ -1265,21 +1265,24 @@ seastar::future<> OSD::prepare_to_stop()
 {
   if (osdmap && osdmap->is_up(whoami)) {
     state.set_prestop();
-    return monc->send_message(
+    const auto timeout =
+      std::chrono::duration_cast<std::chrono::milliseconds>(
+       std::chrono::duration<double>(
+         local_conf().get_val<double>("osd_mon_shutdown_timeout")));
+
+    return seastar::with_timeout(
+      seastar::timer<>::clock::now() + timeout,
+      monc->send_message(
          make_message<MOSDMarkMeDown>(
            monc->get_fsid(),
            whoami,
            osdmap->get_addrs(whoami),
            osdmap->get_epoch(),
            true)).then([this] {
-      const auto timeout =
-       std::chrono::duration_cast<std::chrono::milliseconds>(
-         std::chrono::duration<double>(
-           local_conf().get_val<double>("osd_mon_shutdown_timeout")));
-      return seastar::with_timeout(
-      seastar::timer<>::clock::now() + timeout,
-      stop_acked.get_future());
-    }).handle_exception_type([this](seastar::timed_out_error&) {
+       return stop_acked.get_future();
+      })
+    ).handle_exception_type(
+      [](seastar::timed_out_error&) {
       return seastar::now();
     });
   }