Skip to content

[integ-test] Use EFA 1.49.0 to build image on Rocky 9.8#7464

Merged
hanwen-cluster merged 1 commit into
aws:integ-tests-3.15.1from
hanwen-cluster:integ-tests-3.15.1
Jul 1, 2026
Merged

[integ-test] Use EFA 1.49.0 to build image on Rocky 9.8#7464
hanwen-cluster merged 1 commit into
aws:integ-tests-3.15.1from
hanwen-cluster:integ-tests-3.15.1

Conversation

@hanwen-cluster

Copy link
Copy Markdown
Contributor

Description of changes

  1. Adds a DevSetting to use EFA 1.49.0 when building image with Rocky 9
  2. Removes the old DevSetting on enable_dcv, because it is no longer used anymore
  3. RHEL9 build is unchanged so that we are monitoring the build health on both RHEL 9.7, and Rocky 9.8 with DevSettings

Tests

  • build-image on all OSes have passed

References

  • Link to impacted open issues.
  • Link to related PRs in other packages (i.e. cookbook, node).
  • Link to documentation useful to understand the changes.

Checklist

  • Make sure you are pointing to the right branch.
  • If you're creating a patch for a branch other than develop add the branch name as prefix in the PR title (e.g. [release-3.6]).
  • 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.

1. Adds a DevSetting to use EFA 1.49.0 when building image with Rocky 9
2. Removes the old DevSetting on `enable_dcv`, because it is no longer used anymore
3. RHEL9 build is unchanged so that we are monitoring the build health on both RHEL 9.7, and Rocky 9.8 with DevSettings
@hanwen-cluster hanwen-cluster requested review from a team as code owners June 30, 2026 20:19
Cookbook:
ExtraChefAttributes: |
{"cluster": {"dcv": {"install_enabled": false}}}
{"cluster": {"efa": {"version": "1.49.0", "sha256": "cf2e9281a2328a243c76f911a490faed43ca0fecfe4733c25e34b2e92a32c309"}}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you use the--extra-chef-attribute for this? Any reason for hardcoding this if this is temporary lets add a commit message and comment saying so?

@himani2411 himani2411 Jul 1, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

On 2nd thought you cant use the --extra-chef-attribute as that will apply to all the tests.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why is this devsetting needed when you have aws/aws-parallelcluster-cookbook#3213 merged the cookbook change?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This PR targets 3.15.1, to make sure 3.15.1 can build-image with the latest EFA

@hanwen-cluster hanwen-cluster enabled auto-merge (rebase) July 1, 2026 15:19
@hanwen-cluster hanwen-cluster added the skip-changelog-update Disables the check that enforces changelog updates in PRs label Jul 1, 2026
@hanwen-cluster hanwen-cluster merged commit c7457c3 into aws:integ-tests-3.15.1 Jul 1, 2026
33 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog-update Disables the check that enforces changelog updates in PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants