]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: fix staggered upgrade daemon type warning 62755/head
authorAdam King <adking@redhat.com>
Wed, 9 Apr 2025 22:54:41 +0000 (18:54 -0400)
committerAdam King <adking@redhat.com>
Wed, 9 Apr 2025 22:54:41 +0000 (18:54 -0400)
This was meant to just be daemons that use the ceph image,
which everything under "gateway" daemons used to be before
the addition of nvmeof and smb. This patch updates it to
explicitly reference the daemon types that use the ceph image.

Signed-off-by: Adam King <adking@redhat.com>
src/pybind/mgr/cephadm/module.py

index 1a5389d026083707205548227b8eb6123896571c..2439c835cbacf4632cf32e711961a60275b5a14a 100644 (file)
@@ -97,7 +97,7 @@ from .inventory import (
 from .upgrade import CephadmUpgrade
 from .template import TemplateMgr
 from .utils import CEPH_IMAGE_TYPES, RESCHEDULE_FROM_OFFLINE_HOSTS_TYPES, forall_hosts, \
-    cephadmNoImage, CEPH_UPGRADE_ORDER, SpecialHostLabels
+    cephadmNoImage, SpecialHostLabels
 from .configchecks import CephadmConfigChecks
 from .offline_watcher import OfflineHostWatcher
 from .tuned_profiles import TunedProfileUtils
@@ -3913,9 +3913,9 @@ Then run the following:
             raise OrchestratorError('--daemon-types and --services are mutually exclusive')
         if daemon_types is not None:
             for dtype in daemon_types:
-                if dtype not in CEPH_UPGRADE_ORDER:
+                if dtype not in utils.CEPH_IMAGE_TYPES:
                     raise OrchestratorError(f'Upgrade aborted - Got unexpected daemon type "{dtype}".\n'
-                                            f'Viable daemon types for this command are: {utils.CEPH_TYPES + utils.GATEWAY_TYPES}')
+                                            f'Viable daemon types for this command are: {utils.CEPH_IMAGE_TYPES}')
         if services is not None:
             for service in services:
                 if service not in self.spec_store: