From: Tomer Haskalovitch Date: Mon, 23 Mar 2026 23:27:43 +0000 (+0200) Subject: mgr/cephadm: support no pool value in spec for nvmeof gateway X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F68009%2Fhead;p=ceph.git mgr/cephadm: support no pool value in spec for nvmeof gateway if no value was provided we will use ".nvmeof" by default Signed-off-by: Tomer Haskalovitch --- diff --git a/src/pybind/mgr/cephadm/tests/services/test_nvmeof.py b/src/pybind/mgr/cephadm/tests/services/test_nvmeof.py index a2e8ef35874f..64961e7fa760 100644 --- a/src/pybind/mgr/cephadm/tests/services/test_nvmeof.py +++ b/src/pybind/mgr/cephadm/tests/services/test_nvmeof.py @@ -293,3 +293,8 @@ timeout = 1.0\n""" 'group name \'right.group\' and service id \'pool4.wrong.group\'' ): cephadm_module._apply_service_spec(mismatch_nvmeof_spec_group_service_id_with_dot) + + def test_nvmeof_service_spec_defaults_pool_to_metadata_pool(self): + spec = NvmeofServiceSpec(service_id='pool4.bla.group', group='group') + + assert spec.pool == ".nvmeof" diff --git a/src/python-common/ceph/deployment/service_spec.py b/src/python-common/ceph/deployment/service_spec.py index a6963d4dadcb..3979675c7556 100644 --- a/src/python-common/ceph/deployment/service_spec.py +++ b/src/python-common/ceph/deployment/service_spec.py @@ -1772,8 +1772,8 @@ class NvmeofServiceSpec(ServiceSpec): extra_entrypoint_args=extra_entrypoint_args, custom_configs=custom_configs) - #: RADOS pool where ceph-nvmeof config data is stored. - self.pool = pool + #: RADOS pool where ceph-nvmeof config data is stored (use '.nvmeof' as default). + self.pool = pool or '.nvmeof' #: ``addr`` address of the nvmeof gateway self.addr = addr #: ``addr_map`` per node address map of the nvmeof gateways