]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ci.git/commit
mgr/orchestrator: block OSD specs with no service id
authorAdam King <adking@redhat.com>
Wed, 6 Dec 2023 15:58:03 +0000 (10:58 -0500)
committerAdam King <adking@redhat.com>
Wed, 6 Dec 2023 15:58:03 +0000 (10:58 -0500)
commit993ce910f41c69a84b5b3058da534ed0e8841a03
tree17123f9bb98d8fd60338cd90199d9a45080c9140
parent77e2e0207902812f9629a83b80c2071ff264ed78
mgr/orchestrator: block OSD specs with no service id

This is a copy of a code comment from this commit, but
it explains the change, so putting it here as well

There is a general "osd" service with no service id, but we use
that to dump osds created individually with "ceph orch daemon add osd"
and those made with "ceph orch apply osd --all-available-devices"
For actual user created OSD specs, we should promote users having a
service id so it doesn't get mixed in with those other OSDs. This
check is being done in this spot in particular as this is the only
place we can 100% differentiate between an actual user created OSD
spec and a spec we made ourselves to cover the all-available-devices case

Relates to: https://tracker.ceph.com/issues/63729

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