]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
mgr/cephadm: fix staggered upgrade daemon type warning
authorAdam King <adking@redhat.com>
Wed, 9 Apr 2025 22:54:41 +0000 (18:54 -0400)
committerAdam King <adking@redhat.com>
Mon, 5 May 2025 15:00:47 +0000 (11:00 -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>
(cherry picked from commit 2d248199cda0257884d2aac6a5003118e86c5880)

src/pybind/mgr/cephadm/module.py

index c1b91abe8f45b438b4e1d0908cf808c16387e9cc..6ee03a1c696a5139dfb9b0b0dc08909e695db99a 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
@@ -3956,9 +3956,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: