Skip to content

Commit ee9c994

Browse files
Tim Lanetilne
authored andcommitted
Revert workarounds for EFA installer issue
Revert "2.11: Update changelog" This reverts commit acb1ba0. Revert "Fix uninstall efa deps" This reverts commit 76bdb55. Revert "Remove rdma-core before EFA installation only on alinux2" This reverts commit 8e6c33f. Revert "Implement workaround for EFA installer issue" This reverts commit 1a52d00.
1 parent 52781cf commit ee9c994

File tree

2 files changed

+1
-11
lines changed

2 files changed

+1
-11
lines changed

CHANGELOG.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ This file is used to list changes made in each version of the AWS ParallelCluste
1515

1616
**BUG FIXES**
1717
- Lock the version of ``nvidia-fabricmanager`` package to the installed NVIDIA drivers to prevent updates and misalignments.
18-
- Fix EFA runtime installation on alinux2 when GPUDirect RDMA is enabled, by uninstalling ``rdma-core`` before installing EFA.
1918

2019
2.11.1
2120
-----

recipes/efa_install.rb

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -47,15 +47,6 @@
4747
end
4848
end
4949

50-
# Remove certain dependencies before installation to
51-
# work around an issue with the EFA installer.
52-
# Without the removal, runtime re-installation of EFA would fail on alinux2.
53-
package 'uninstall efa deps before installing efa' do
54-
action :remove
55-
package_name %w[rdma-core]
56-
only_if { (!efa_installed || efa_gdr_enabled?) && node['platform_family'] == 'amazon' }
57-
end
58-
5950
installer_options = "-y"
6051
# skip efa-kmod installation on not supported platforms
6152
installer_options += " -k" unless node['conditions']['efa_supported']
@@ -71,7 +62,7 @@
7162
./efa_installer.sh #{installer_options}
7263
rm -rf #{node['cfncluster']['sources_dir']}/aws-efa-installer
7364
EFAINSTALL
74-
only_if { !efa_installed || efa_gdr_enabled? }
65+
not_if { efa_installed && !efa_gdr_enabled? }
7566
end
7667

7768
# EFA installer v1.11.0 removes libibverbs-core, which contains hwloc-devel during install

0 commit comments

Comments
 (0)