Skip to content

Commit dd5415d

Browse files
Support Multiple EFS file systems
Signed-off-by: Hanwen <[email protected]>
1 parent 916acd3 commit dd5415d

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

attributes/default.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -498,8 +498,8 @@
498498
default['cluster']['log_group_name'] = "NONE"
499499
default['cluster']['disable_hyperthreading_manually'] = 'false'
500500
default['cluster']['volume_fs_type'] = 'ext4'
501-
default['cluster']['efs_shared_dir'] = 'NONE'
502-
default['cluster']['efs_fs_id'] = nil
501+
default['cluster']['efs_shared_dirs'] = ''
502+
default['cluster']['efs_fs_ids'] = ''
503503
default['cluster']['cluster_admin_user'] = 'pcluster-admin'
504504
default['cluster']['cluster_admin_user_id'] = node['cluster']['reserved_base_uid']
505505
default['cluster']['cluster_admin_group'] = node['cluster']['cluster_admin_user']

cookbooks/aws-parallelcluster-config/recipes/efs_mount.rb

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,10 @@
1616
# limitations under the License.
1717

1818
# Get shared_dir path and mount EFS filesystem
19-
efs_shared_dir = node['cluster']['efs_shared_dir'].split(',')[0]
20-
21-
# Check to see if EFS is created
22-
if efs_shared_dir != "NONE"
2319

20+
efs_fs_id_array = node['cluster']['efs_fs_ids'].split(',')
21+
efs_shared_dir_array = node['cluster']['efs_shared_dirs'].split(',')
22+
efs_fs_id_array.zip(efs_shared_dir_array).each do |efs_fs_id, efs_shared_dir|
2423
# Path needs to be fully qualified, for example "shared/temp" becomes "/shared/temp"
2524
efs_shared_dir = "/#{efs_shared_dir}" unless efs_shared_dir.start_with?('/')
2625

@@ -35,7 +34,7 @@
3534

3635
# Mount EFS over NFS
3736
mount efs_shared_dir do
38-
device "#{node['cluster']['efs_fs_id']}.efs.#{node['cluster']['region']}.#{node['cluster']['aws_domain']}:/"
37+
device "#{efs_fs_id}.efs.#{node['cluster']['region']}.#{node['cluster']['aws_domain']}:/"
3938
fstype 'nfs4'
4039
options 'nfsvers=4.1,rsize=1048576,wsize=1048576,hard,timeo=30,retrans=2,noresvport,_netdev'
4140
dump 0

system_tests/dna.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
"postinstall": "NONE",
1111
"postinstall_args": "NONE",
1212
"region": "us-east-2",
13-
"efs_fs_id": "NONE",
14-
"efs_shared_dir": "NONE,NONE,NONE,NONE,NONE,NONE,NONE,NONE,NONE",
13+
"efs_fs_ids": "",
14+
"efs_shared_dirs": "",
1515
"fsx_fs_id": "NONE",
1616
"fsx_mount_name": "",
1717
"fsx_dns_name": "",

0 commit comments

Comments
 (0)