mgr/cephadm: fix Server_Scope in ganesha.conf

Otherwise, with newer ganesha versions (hence why we didn't
see this in testing) you get

ganesha.nfsd-2[sigmgr] config_errs_to_log :CONFIG :WARN :Config File (/etc/ganesha/ganesha.conf:17): Syntax error in statement

Signed-off-by: Adam King <adking@redhat.com>

diff --git a/src/pybind/mgr/cephadm/templates/services/nfs/ganesha.conf.j2 b/src/pybind/mgr/cephadm/templates/services/nfs/ganesha.conf.j2
index bd859fd986e6b2d85f7ae79fb8ecded9111d4a5c..8642c0ae162910394bdb9c6d05f3c53cfd55f59c 100644 (file)
--- a/src/pybind/mgr/cephadm/templates/services/nfs/ganesha.conf.j2
+++ b/src/pybind/mgr/cephadm/templates/services/nfs/ganesha.conf.j2
@@ -20,7 +20,7 @@ NFSv4 {
         Delegations = false;
         RecoveryBackend = "rados_cluster";
         Minor_Versions = 1, 2;
-        Server_Scope = {{ cluster_id }}-{{ namespace }}
+        Server_Scope = "{{ cluster_id }}-{{ namespace }}";
 {% if nfs_idmap_conf %}
         IdmapConf = "{{ nfs_idmap_conf }}";
 {% endif %}

diff --git a/src/pybind/mgr/cephadm/tests/test_services.py b/src/pybind/mgr/cephadm/tests/test_services.py
index c8682c3b5d172baf7a6c410463b9a1c07eca7435..aa09c68110eda4ef563645208e8e67f7ebc057ac 100644 (file)
--- a/src/pybind/mgr/cephadm/tests/test_services.py
+++ b/src/pybind/mgr/cephadm/tests/test_services.py
@@ -3347,7 +3347,7 @@ class TestIngressService:
             '        Delegations = false;\n'
             '        RecoveryBackend = "rados_cluster";\n'
             '        Minor_Versions = 1, 2;\n'
-            f'        Server_Scope = {cephadm_module._cluster_fsid}-foo\n'
+            f'        Server_Scope = "{cephadm_module._cluster_fsid}-foo";\n'
             '        IdmapConf = "/etc/ganesha/idmap.conf";\n'
             '}\n'
             '\n'