Skip to content

Commit 3be9786

Browse files
Use the latest version of amazon-efs-utils and move to 2.x
1 parent 2b32e47 commit 3be9786

File tree

11 files changed

+19
-22
lines changed

11 files changed

+19
-22
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ This file is used to list changes made in each version of the AWS ParallelCluste
88
**CHANGES**
99
- On Ubuntu 22.04, install the Nvidia driver with the same compiler version used to compile the kernel.
1010
- Upgrade `aws-cfn-bootstrap` to version 2.0-32.
11+
- Upgrade amazon-efs-utils to version 2.1.0.
1112

1213
3.12.0
1314
------

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,12 @@
2020

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

2727
# On Amazon Linux 2, amazon-efs-utils and stunnel are installed from OS repo.
28-
package package_name do
28+
package full_package do
2929
retries 3
3030
retry_delay 5
3131
end

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
use 'partial/_mount_umount'
2222

2323
action :install_utils do
24-
package_name = "amazon-efs-utils"
24+
package_name = "amazon-efs-utils-#{new_resource.efs_utils_version}"
2525

2626
# Do not install efs-utils if a same or newer version is already installed.
2727
return if already_installed?(package_name, new_resource.efs_utils_version)

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

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,3 @@
2121
use 'partial/_redhat_based'
2222
use 'partial/_install_from_tar'
2323
use 'partial/_mount_umount'
24-
25-
def prerequisites
26-
%w(rpm-build make)
27-
end

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

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,3 @@
2121
use 'partial/_redhat_based'
2222
use 'partial/_install_from_tar'
2323
use 'partial/_mount_umount'
24-
25-
def prerequisites
26-
%w(rpm-build make)
27-
end

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

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

33
default_action :install_utils
44

5-
property :efs_utils_version, String, default: '1.34.1'
6-
property :efs_utils_checksum, String, default: '69d0d8effca3b58ccaf4b814960ec1d16263807e508b908975c2627988c7eb6c'
5+
property :efs_utils_version, String, default: '2.1.0'
6+
property :efs_utils_checksum, String, default: '2996bdd5387131d302310812fa1e07e1be00f80814a580f5dfeb27d68519fd24'
77

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

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,5 +24,5 @@ def install_script_code(efs_utils_tarball, efs_utils_package, efs_utils_version)
2424
end
2525

2626
def prerequisites
27-
%w(dkms)
27+
%w(dkms pkg-config libssl-dev rustc cargo )
2828
end

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,3 +22,7 @@ def install_script_code(efs_utils_tarball, efs_utils_package, efs_utils_version)
2222
yum -y install ./build/#{efs_utils_package}*rpm
2323
EFSUTILSINSTALL
2424
end
25+
26+
def prerequisites
27+
%w(rpm-build make rust cargo openssl-devel)
28+
end

cookbooks/aws-parallelcluster-environment/spec/unit/resources/efs_spec.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ def mock_already_installed(package, expected_version, installed)
6262

6363
it 'installs amazon-efs-utils' do
6464
is_expected.to install_utils_efs('install_utils')
65-
is_expected.to install_package('amazon-efs-utils').with(retries: 3).with(retry_delay: 5)
65+
is_expected.to install_package('amazon-efs-utils-1.2.3').with(retries: 3).with(retry_delay: 5)
6666
end
6767
end
6868

@@ -73,7 +73,7 @@ def mock_already_installed(package, expected_version, installed)
7373
end
7474

7575
it 'installs amazon-efs-utils' do
76-
is_expected.to install_package('amazon-efs-utils').with(retries: 3).with(retry_delay: 5)
76+
is_expected.to install_package('amazon-efs-utils-1.2.3').with(retries: 3).with(retry_delay: 5)
7777
end
7878
end
7979
end
@@ -179,8 +179,8 @@ def mock_already_installed(package, expected_version, installed)
179179
end
180180
cached(:required_packages) do
181181
{
182-
"redhat" => %w(rpm-build make),
183-
"rocky" => %w(rpm-build make),
182+
"redhat" => %w(rpm-build make rust cargo openssl-devel),
183+
"rocky" => %w(rpm-build make rust cargo openssl-devel),
184184
}
185185
end
186186

cookbooks/aws-parallelcluster-environment/test/controls/efs_spec.rb

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

55
only_if { !os_properties.redhat_on_docker? }
66

7-
describe file("#{node['cluster']['sources_dir']}/efs-utils-1.34.1.tar.gz") do
7+
describe file("#{node['cluster']['sources_dir']}/efs-utils-2.1.0.tar.gz") do
88
it { should exist }
9-
its('sha256sum') { should eq '69d0d8effca3b58ccaf4b814960ec1d16263807e508b908975c2627988c7eb6c' }
9+
its('sha256sum') { should eq '2996bdd5387131d302310812fa1e07e1be00f80814a580f5dfeb27d68519fd24' }
1010
its('owner') { should eq 'root' }
1111
its('group') { should eq 'root' }
1212
its('mode') { should cmp '0644' }

0 commit comments

Comments
 (0)