From 38825ff73b6ce9ad330b4580831a39928926e698 Mon Sep 17 00:00:00 2001 From: Greg Farnum Date: Thu, 30 Sep 2021 18:08:00 +0000 Subject: [PATCH] mon: remove an incomplete and unnecessary wait_for_readable() bailout Not sure how this got in here[1], but without a return statement it's not functioning, and it's happily unnecessary as we don't reference the OSDMap. [1]: We do CRUSH parsing and I was probably expecting to compare it to the actual map at some point during development, but that's not happening. Signed-off-by: Greg Farnum --- src/mon/MonmapMonitor.cc | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/mon/MonmapMonitor.cc b/src/mon/MonmapMonitor.cc index ade6624d415..35cf2bca89c 100644 --- a/src/mon/MonmapMonitor.cc +++ b/src/mon/MonmapMonitor.cc @@ -1020,9 +1020,6 @@ bool MonmapMonitor::prepare_command(MonOpRequestRef op) goto reply; } - if (!mon.osdmon()->is_readable()) { - mon.osdmon()->wait_for_readable(op, new Monitor::C_RetryMessage(&mon, op)); - } vector argvec; map loc; cmd_getval(cmdmap, "args", argvec); -- 2.39.5