Skip to content

Commit f3d9236

Browse files
author
Csaba Daradics
committed
Create mechanism to be able to add custom mount options to AFS shares
Applies to /sapmnt/<SID> and /hana/shared in HA/non-HA deployments
1 parent cb85f3b commit f3d9236

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

deploy/ansible/roles-sap-os/2.6-sap-mounts/tasks/2.6.0-afs-mounts.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@
276276
src: "{{ item.src }}"
277277
path: "{{ item.path }}"
278278
fstype: "{{ item.type }}"
279-
opts: "{{ afs_mnt_options }}"
279+
opts: "{{ afs_mnt_options }}{{ ',' ~ afs_mnt_extra_options if afs_mnt_extra_options is defined }}"
280280
state: mounted
281281
loop:
282282
- { type: 'nfs4', src: '{{ sap_mnt }}/sapmnt{{ sap_sid | upper }}', path: '/sapmnt/{{ sap_sid | upper }}' }
@@ -286,7 +286,7 @@
286286
src: "{{ item.src }}"
287287
path: "{{ item.path }}"
288288
fstype: "{{ item.type }}"
289-
opts: "{{ afs_mnt_options }}"
289+
opts: "{{ afs_mnt_options }}{{ ',' ~ afs_mnt_extra_options if afs_mnt_extra_options is defined }}"
290290
state: unmounted
291291
loop:
292292
- { type: 'nfs4', src: '{{ sap_mnt }}/sapmnt{{ sap_sid | upper }}', path: '/sapmnt/{{ sap_sid | upper }}' }
@@ -303,7 +303,7 @@
303303
src: "{{ item.src }}"
304304
path: "{{ item.path }}"
305305
fstype: "{{ item.type }}"
306-
opts: "{{ afs_mnt_options }}"
306+
opts: "{{ afs_mnt_options }}{{ ',' ~ afs_mnt_extra_options if afs_mnt_extra_options is defined }}"
307307
state: mounted
308308
loop:
309309
- { type: 'nfs4', src: '{{ sap_mnt }}/sapmnt{{ sap_sid | upper }}', path: '/sapmnt/{{ sap_sid | upper }}' }
@@ -427,7 +427,7 @@
427427
'type': 'shared',
428428
'temppath': 'shared',
429429
'mount': "{% if site | default('SITE1') == 'SITE1' %}{{ hana_shared_mountpoint[0] }}{% else %}{{ hana_shared_mountpoint[1] }}{% endif %}",
430-
'opts': 'vers=4,minorversion=1,sec=sys',
430+
'opts': '{{ afs_mnt_options }}{{ "," ~ afs_mnt_extra_options if afs_mnt_extra_options is defined }}',
431431
'path': '/hana/shared',
432432
'owner': '{{ sidadm_uid }}',
433433
'permissions': '0775',
@@ -450,7 +450,7 @@
450450
'type': 'shared',
451451
'temppath': 'shared',
452452
'mount': "{{ hana_shared_mountpoint[0] }}",
453-
'opts': 'vers=4,minorversion=1,sec=sys',
453+
'opts': '{{ afs_mnt_options }}{{ "," ~ afs_mnt_extra_options if afs_mnt_extra_options is defined }}',
454454
'path': '/hana/shared',
455455
'owner': '{{ sidadm_uid }}',
456456
'permissions': '0775',

deploy/ansible/roles-sap/5.8-hanadb-scaleout-pacemaker/tasks/5.8.4.0-clusterPrep-ScaleOut-RedHat.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
# We set this to prevent code overflow. Since pacemaker code for both AFS and ANF are the same barring NFS mount options, we parameterize this parameter.
1919
- name: "5.8 HANA Pacemaker Scaleout - NFS Compatibility - set mount options based on NFS source"
2020
ansible.builtin.set_fact:
21-
nfs_mount_options: "{% if NFS_provider == 'ANF' %}defaults,rw,hard,rsize=262144,wsize=262144,proto=tcp,noatime,_netdev,nfsvers=4.1,lock,sec=sys{% else %}noresvport,defaults,rw,hard,proto=tcp,noatime,nfsvers=4.1,lock{% endif %}"
21+
nfs_mount_options: "{% if NFS_provider == 'ANF' %}defaults,rw,hard,rsize=262144,wsize=262144,proto=tcp,noatime,_netdev,nfsvers=4.1,lock,sec=sys{% else %}noresvport,defaults,rw,hard,proto=tcp,noatime,nfsvers=4.1,lock{% if afs_mnt_extra_options is defined %},{{ afs_mnt_extra_options }}{% endif %}{% endif %}"
2222
directory_path: "{% if NFS_provider == 'ANF' %}/shared{% else %}{% endif %}"
2323

2424
# Ref : https://learn.microsoft.com/en-us/azure/sap/workloads/sap-hana-high-availability-scale-out-hsr-rhel?tabs=lb-portal#create-file-system-resources

0 commit comments

Comments
 (0)