From 9021e57d83d7d35c95a43e13320b13d530453f03 Mon Sep 17 00:00:00 2001 From: Igor Fedotov Date: Fri, 2 Dec 2022 18:31:15 +0300 Subject: [PATCH] mon: avoid exception when setting require-osd-release more than 2 versions up. Request for additional confirmation instead. Fixes: https://tracker.ceph.com/issues/58156 Signed-off-by: Igor Fedotov --- src/mon/OSDMonitor.cc | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc index e9e7ce72a8b1d..3603d2cb7e8a3 100644 --- a/src/mon/OSDMonitor.cc +++ b/src/mon/OSDMonitor.cc @@ -11601,7 +11601,13 @@ bool OSDMonitor::prepare_command_impl(MonOpRequestRef op, err = 0; goto reply; } - ceph_assert(osdmap.require_osd_release >= ceph_release_t::pacific); + if (osdmap.require_osd_release < ceph_release_t::pacific && !sure) { + ss << "Not advisable to continue since current 'require_osd_release' " + << "refers to a very old Ceph release. Pass " + << "--yes-i-really-mean-it if you really wish to continue."; + err = -EPERM; + goto reply; + } if (!osdmap.get_num_up_osds() && !sure) { ss << "Not advisable to continue since no OSDs are up. Pass " << "--yes-i-really-mean-it if you really wish to continue."; -- 2.39.5