ceph-nfs: allow overriding NFS_CORE_PARAM

We already have config override variables for existing block (like
ganesha_ceph_export_overrides, ganesha_log_overrides, etc...) or a
global one (ganesha_conf_overrides) but redefining the NFS_CORE_PARAM
block in that variable will erase all previous values (currently only
Bind_Addr).

ganesha_core_param_overrides: |
        Enable_UDP = false;
        NFS_Port = 2050;

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1941775
Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit 9817d29543099ca640ce8b23da2ab9f26179cba5)

diff --git a/group_vars/nfss.yml.sample b/group_vars/nfss.yml.sample
index f156c7518136cad9596bbd29e769a4cd9a0027c9..21ed92e185c2e64add557083c3acd3db678f9427 100644 (file)
--- a/group_vars/nfss.yml.sample
+++ b/group_vars/nfss.yml.sample
@@ -110,6 +110,7 @@ dummy:
 #        #Entries_HWMark = 100000;
 #}
 #
+#ganesha_core_param_overrides:
 #ganesha_ceph_export_overrides:
 #ganesha_rgw_export_overrides:
 #ganesha_rgw_section_overrides:

diff --git a/roles/ceph-nfs/defaults/main.yml b/roles/ceph-nfs/defaults/main.yml
index 361cd36988e23f87ee60aaab59b076286fce1b94..38248fbee28e8ff61457fc07f809a38237b6e546 100644 (file)
--- a/roles/ceph-nfs/defaults/main.yml
+++ b/roles/ceph-nfs/defaults/main.yml
@@ -102,6 +102,7 @@ rgw_client_name: client.rgw.{{ ansible_facts['hostname'] }}
         #Entries_HWMark = 100000;
 #}
 #
+#ganesha_core_param_overrides:
 #ganesha_ceph_export_overrides:
 #ganesha_rgw_export_overrides:
 #ganesha_rgw_section_overrides:

diff --git a/roles/ceph-nfs/templates/ganesha.conf.j2 b/roles/ceph-nfs/templates/ganesha.conf.j2
index 10cd990dd3ec1f660f7e567465b51acd7e15e5a6..7e6fab6c5ed54af5eb38b159a67c1fb16fa80417 100644 (file)
--- a/roles/ceph-nfs/templates/ganesha.conf.j2
+++ b/roles/ceph-nfs/templates/ganesha.conf.j2
@@ -10,6 +10,7 @@ NFS_Core_Param
 {% if ceph_nfs_bind_addr is defined %}
        Bind_Addr={{ ceph_nfs_bind_addr }};
 {% endif %}
+{{ ganesha_core_param_overrides | default(None) }}
 }
 
 {% if ceph_nfs_disable_caching | bool or nfs_file_gw | bool %}