Merge pull request ceph#58592 from NitzanMordhai/wip-nitzan-crimson-mgr-client-lost-connection

mgr/client: validate connection before sending

Reviewed-by: Matan Breizman <[email protected]>
Reviewed-by: Radosław Zarzyński <[email protected]>
Reviewed-by: Yingxin Cheng <[email protected]>

Author: Matan-B

qa/config/crimson_qa_overrides.yaml

@@ -9,8 +9,6 @@ overrides:
         osd pool default crimson: true
       osd:
         crimson osd obc lru size: 10
-      mgr:
-        mgr stats period: 30
     flavor: crimson
   workunit:
     env:

src/crimson/mgr/client.cc

@@ -165,6 +165,10 @@ void Client::report()
     }
     return with_stats.get_stats(
     ).then([this](auto &&pg_stats) {
+      if (!conn) {
+        logger().warn("report: no conn available; before sending stats, report skipped");
+        return seastar::now();
+      }
       return conn->send(std::move(pg_stats));
     });
   });