Skip to content

Conversation

EddyMM
Copy link
Contributor

@EddyMM EddyMM commented Sep 26, 2022

Description of changes

Upgrades EFA installer from 1.17.2 to 1.18.0

  • Efa-driver: efa-1.16.0-1
  • Efa-config: efa-config-1.11-1
  • Efa-profile: efa-profile-1.5-1
  • Libfabric-aws: libfabric-aws-1.16.0~amzn4.0-1
  • Rdma-core: rdma-core-41.0-2
  • Open MPI: openmpi40-aws-4.1.4-2

Tests

  • Created a Custom AMI with the cookbook changes and confirm EFA (1.18.0) was successfully installed

References

Checklist

  • Make sure you are pointing to the right branch and add a label in the PR title (i.e. 2.x vs 3.x)
  • Check all commits' messages are clear, describing what and why vs how.
  • Make sure to have added unit tests or integration tests to cover the new/modified code.
  • Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@EddyMM EddyMM added the 3.x label Sep 26, 2022
@EddyMM EddyMM marked this pull request as ready for review September 26, 2022 17:41
@EddyMM EddyMM requested review from a team as code owners September 26, 2022 17:41
CHANGELOG.md Outdated
- Upgrade third-party cookbook dependencies:
- selinux-6.0.5 (from selinux-6.0.4)
- nfs-5.0.0 (from nfs-2.6.4)
- Upgrade EFA to `1.18.0`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we change it to:
Upgrade EFA installer to version 1.18.0
and also mentioned the version of - EFA driver, EFA configuration, EFA profile, Libfabric, RDMA core, Open MPI version? See an example from this PR: https://github.com/aws/aws-parallelcluster-cookbook/pull/1481/files

- Efa-driver: `efa-1.16.0-1`
- Efa-config: `efa-config-1.11-1`
- Efa-profile: `efa-profile-1.5-1`
- Libfabric-aws: `libfabric-aws-1.16.0~amzn4.0-1`
- Rdma-core: `rdma-core-41.0-2`
- Open MPI: `openmpi40-aws-4.1.4-2`

Signed-off-by: Eddy Mwiti <eddmwiti@amazon.com>
@chenwany chenwany merged commit 3987786 into aws:develop Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants