]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: make osd_fast_shutdown_notify_mon option true by default
authorSatoru Takeuchi <satoru.takeuchi@gmail.com>
Thu, 18 Nov 2021 20:48:18 +0000 (20:48 +0000)
committerLaura Flores <lflores@redhat.com>
Fri, 25 Mar 2022 17:49:23 +0000 (12:49 -0500)
osd_fast_shutdown_notify_mon option is false by default. So users suffer
from error log flood, slow ops, and the long I/O timeouts on voluntary OS
shutdown before they are aware of the existence of this option. Let's
make this option true by default.

Fixes: https://tracker.ceph.com/issues/53328
Signed-off-by: Satoru Takeuchi <satoru.takeuchi@gmail.com>
(cherry picked from commit 729a5b85a6586b47d16acbba2cf8e765e498cd65)

 Conflicts:
src/common/options/global.yaml.in
- global.yaml.in does not exist in Pacific; rather,
  these configs were handled in options.cc.

src/common/options.cc

index 4992b53ff81d02bf75ab2acc95857db96a2f76c6..b0387f65bf8cc1c4da2d8b3294f386ffd218a3fe 100644 (file)
@@ -3804,7 +3804,7 @@ std::vector<Option> get_global_options() {
     .set_long_description("Setting this to false makes the OSD do a slower teardown of all state when it receives a SIGINT or SIGTERM or when shutting down for any other reason.  That slow shutdown is primarilyy useful for doing memory leak checking with valgrind."),
 
     Option("osd_fast_shutdown_notify_mon", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
-    .set_default(false)
+    .set_default(true)
     .set_description("Tell mon about OSD shutdown on immediate shutdown")
     .set_long_description("Tell the monitor the OSD is shutting down on immediate shutdown. This helps with cluster log messages from other OSDs reporting it immediately failed.")
     .add_see_also({"osd_fast_shutdown", "osd_mon_shutdown_timeout"}),