Skip to content

Commit 9e029ef

Browse files
author
Himani Anil Deshpande
committed
[Efs-utils] Add efs-utils attributes
1 parent 39fe7b4 commit 9e029ef

File tree

5 files changed

+21
-8
lines changed

5 files changed

+21
-8
lines changed

cookbooks/aws-parallelcluster-environment/attributes/environment.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,11 @@
7373
default['cluster']['efa']['version'] = '1.38.1'
7474
default['cluster']['efa']['sha256'] = '83923374afd388b1cfcf4b3a21a2b1ba7cf46a01a587f7b519b8386cb95e4f81'
7575

76+
default['cluster']['efs']['version'] = '2.3.1'
77+
default['cluster']['efs']['sha256'] = 'ced12f82e76f9740476b63f30c49bd76cc00b6375e12a9f5f7ba852635c49e15'
78+
if platform?('amazon')
79+
default['cluster']['efs']['version'] = '2.1.0'
80+
end
7681
# TODO: Move to platform cookbook
7782
default['cluster']['spack_shared_dir'] = "#{node['cluster']['shared_dir']}/spack"
7883
default['cluster']['spack']['version'] = '0.20.2'

cookbooks/aws-parallelcluster-environment/resources/efs/efs_alinux2.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@
2222

2323
action :install_utils do
2424
package_name = "amazon-efs-utils"
25-
full_package = "#{package_name}-#{new_resource.efs_utils_version}"
25+
full_package = "#{package_name}-#{_efs_utils_version}"
2626
# Do not install efs-utils if a same or newer version is already installed.
27-
return if already_installed?(package_name, new_resource.efs_utils_version)
27+
return if already_installed?(package_name, _efs_utils_version)
2828

2929
# On Amazon Linux 2, amazon-efs-utils and stunnel are installed from OS repo.
3030
package full_package do

cookbooks/aws-parallelcluster-environment/resources/efs/efs_alinux2023.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@
2323
property :efs_utils_version, String, default: '2.1.0'
2424

2525
action :install_utils do
26-
package_name = "amazon-efs-utils-#{new_resource.efs_utils_version}"
26+
package_name = "amazon-efs-utils-#{_efs_utils_version}"
2727

2828
# Do not install efs-utils if a same or newer version is already installed.
29-
return if already_installed?(package_name, new_resource.efs_utils_version)
29+
return if already_installed?(package_name, _efs_utils_version)
3030

3131
# On Amazon Linux 2, amazon-efs-utils and stunnel are installed from OS repo.
3232
package package_name do

cookbooks/aws-parallelcluster-environment/resources/efs/partial/_common.rb

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,16 @@
22

33
default_action :install_utils
44

5-
property :efs_utils_version, String, default: '2.3.1'
6-
property :efs_utils_checksum, String, default: 'ced12f82e76f9740476b63f30c49bd76cc00b6375e12a9f5f7ba852635c49e15'
5+
property :efs_utils_version, String
6+
property :efs_utils_checksum, String
7+
8+
def _efs_utils_version
9+
efs_utils_version || node['cluster']['efs']['version']
10+
end
11+
12+
def _efs_utils_checksum
13+
efs_utils_checksum || node['cluster']['efs']['sha256']
14+
end
715

816
def already_installed?(package_name, expected_version)
917
Gem::Version.new(get_package_version(package_name)) >= Gem::Version.new(expected_version)

cookbooks/aws-parallelcluster-environment/resources/efs/partial/_install_from_tar.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
return if redhat_on_docker?
3131

3232
package_name = "amazon-efs-utils"
33-
package_version = new_resource.efs_utils_version
33+
package_version = _efs_utils_version
3434
efs_utils_tarball = "#{node['cluster']['sources_dir']}/efs-utils-#{package_version}.tar.gz"
3535
efs_utils_url = "#{node['cluster']['artifacts_s3_url']}/dependencies/efs/v#{package_version}.tar.gz"
3636

@@ -46,7 +46,7 @@
4646
mode '0644'
4747
retries 3
4848
retry_delay 5
49-
checksum new_resource.efs_utils_checksum
49+
checksum _efs_utils_checksum
5050
action :create_if_missing
5151
end
5252

0 commit comments

Comments
 (0)