]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
mgr/cephadm: make SMB and NVMEoF upgrade last in staggered upgrade
authorAdam King <adking@redhat.com>
Thu, 2 May 2024 17:35:41 +0000 (13:35 -0400)
committerAdam King <adking@redhat.com>
Fri, 10 May 2024 16:19:35 +0000 (12:19 -0400)
commit5e7a3c2147d87c1fc5be71acbadedefb70e024bf
treebbe071b359c615dbb3713c860743a6021364ee26
parent8a001cb867d92d504075004b3a0c9d6368beba3b
mgr/cephadm: make SMB and NVMEoF upgrade last in staggered upgrade

This needs to happen as some work on the NVMEoF side (still unmerged
as of writing this) will make the NVMEoF daemon dependent on the mon.
Prior to this patch, in a staggered upgrade, all daemons not using the
ceph image were upgraded after the mgr since we typically only care
about the default image changing or potential changes to how we handle
our systemd units which only needs the mgr to be upgraded to be applied.
This NVMEoF dependency on the mon changes this and we can no longer
upgrade it directly after the mgr. This patch changes it so the NVMEoF
daemon is instead upgraded after all ceph image daemons have been
upgraded in a staggered upgrade scenario. Non-staggered upgrades
are unaffected as the NVMEoF daemon was already upgraded near the
end in that scenario. The SMB dameon has no reason it needs to be
upgraded later, but it's in the (small) pool of daemons that don't
use the ceph image and aren't for monitoring, so it's been affected
by this as well.

NOTE: This is a bit of an ugly patch imo and shows that a refactoring
of the upgrade code is likely required. Hopefully this patch is more
of a stopgap until that larger effort can be made

Fixes: https://tracker.ceph.com/issues/65809
Signed-off-by: Adam King <adking@redhat.com>
src/pybind/mgr/cephadm/upgrade.py