From 8acd1ed5b342567779b952f3224fdb122b3bd30e Mon Sep 17 00:00:00 2001 From: Ryan Anderson Date: Wed, 4 Dec 2024 16:41:33 -0500 Subject: [PATCH] Add efs access point id to update recipe mounting (#2844) --- CHANGELOG.md | 1 + .../recipes/config/update_shared_storages.rb | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e51d39f4b..ca74953e1 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -27,6 +27,7 @@ This file is used to list changes made in each version of the AWS ParallelCluste **BUG FIXES** - Fix an issue in the way we get region when manage volumes so that it can correctly handle local zone. +- Fix an issue where adding EFS filesystems with `AccessPointIds` during an update would fail. 3.11.1 ------ diff --git a/cookbooks/aws-parallelcluster-environment/recipes/config/update_shared_storages.rb b/cookbooks/aws-parallelcluster-environment/recipes/config/update_shared_storages.rb index dfe3245c6..46adab8c8 100644 --- a/cookbooks/aws-parallelcluster-environment/recipes/config/update_shared_storages.rb +++ b/cookbooks/aws-parallelcluster-environment/recipes/config/update_shared_storages.rb @@ -37,7 +37,7 @@ # get efs to unmount node.default['cluster']['unmount_efs_shared_dir_array'], node.default['cluster']['unmount_efs_fs_id_array'], = get_efs(UNMOUNT_ACTION) # get efs to mount - node.default['cluster']['mount_efs_shared_dir_array'], node.default['cluster']['mount_efs_fs_id_array'], node.default['cluster']['mount_efs_encryption_in_transit_array'], node.default['cluster']['mount_efs_iam_authorization_array'], node.default['cluster']['mount_efs_access_point_array'] = get_efs(MOUNT_ACTION) + node.default['cluster']['mount_efs_shared_dir_array'], node.default['cluster']['mount_efs_fs_id_array'], node.default['cluster']['mount_efs_encryption_in_transit_array'], node.default['cluster']['mount_efs_iam_authorization_array'], node.default['cluster']['mount_efs_access_point_id_array'] = get_efs(MOUNT_ACTION) # get fsx to unmount node.default['cluster']['unmount_fsx_fs_id_array'], node.default['cluster']['unmount_fsx_fs_type_array'], node.default['cluster']['unmount_fsx_shared_dir_array'], node.default['cluster']['unmount_fsx_dns_name_array'], node.default['cluster']['unmount_fsx_mount_name_array'], node.default['cluster']['unmount_fsx_volume_junction_path_array'] = get_fsx(UNMOUNT_ACTION) # get fsx to mount @@ -193,6 +193,7 @@ def get_raid(action) efs_fs_id_array(lazy { node['cluster']['mount_efs_fs_id_array'] }) efs_encryption_in_transit_array(lazy { node['cluster']['mount_efs_encryption_in_transit_array'] }) efs_iam_authorization_array(lazy { node['cluster']['mount_efs_iam_authorization_array'] }) + efs_access_point_id_array(lazy { node['cluster']['mount_efs_access_point_id_array'] }) action :mount not_if { node['cluster']['mount_efs_shared_dir_array'].empty? } end