Skip to content

feat(container/AWS): Update to EFA Installer 1.50 - #13690

Open
erezzarum wants to merge 2 commits into
ai-dynamo:mainfrom
erezzarum:efa150
Open

feat(container/AWS): Update to EFA Installer 1.50#13690
erezzarum wants to merge 2 commits into
ai-dynamo:mainfrom
erezzarum:efa150

Conversation

@erezzarum

@erezzarum erezzarum commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Overview:

Update AWS EFA Installer to 1.50.

Details:

Update to AWS EFA Installer 1.50

Where should the reviewer start?

Related Issues

⚠️ This section is required. Choose one path below and delete the other.

🔗 This PR is linked to an issue:

  • Closes #XXXX

🚫 This PR is NOT linked to an issue:

  • Confirmed — no related issue

Open in Devin Review

Summary by CodeRabbit

  • Chores
    • Updated networking and communication components to newer versions.
    • Updated the default EFA version for the CUDA 13.0 environment.

Signed-off-by: Erez Zarum <erezz@amazon.com>
Signed-off-by: Erez Zarum <erezz@amazon.com>
@erezzarum
erezzarum requested a review from a team as a code owner August 23, 2026 11:43
@copy-pr-bot

copy-pr-bot Bot commented Aug 23, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@erezzarum
erezzarum deployed to external_collaborator August 23, 2026 11:43 — with GitHub Actions Active
@erezzarum
erezzarum deployed to external_collaborator August 23, 2026 11:43 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi erezzarum! Thank you for contributing to ai-dynamo/dynamo.

Just a reminder: The NVIDIA Test Github Validation CI runs an essential subset of the testing framework to quickly catch errors.Your PR reviewers may elect to test the changes comprehensively before approving your changes.

🚀

@github-actions github-actions Bot added external-contribution Pull request is from an external contributor feat container labels Aug 23, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

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.

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread container/compliance/native_packages.yaml
@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 7542afec-e322-42c8-88cd-05954d1ae729

📥 Commits

Reviewing files that changed from the base of the PR and between bee2a6e and cd26dc3.

📒 Files selected for processing (2)
  • container/compliance/native_packages.yaml
  • container/context.yaml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


Walkthrough

The container metadata updates the tracked libfabric, aws-ofi-nccl, and CUDA 13 EFA versions.

Changes

Container version metadata

Layer / File(s) Summary
Update communication and EFA versions
container/compliance/native_packages.yaml, container/context.yaml
The declared libfabric and aws-ofi-nccl versions are updated. The dynamo.cuda13.0 EFA default changes to 1.50.0.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to cd26d

This localized dependency update does not present an actionable merge-blocking risk and is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the primary change: updating the AWS EFA Installer to version 1.50.
Description check ✅ Passed The description includes the main required sections and accurately describes the EFA Installer update, although reviewer-start guidance and issue-template cleanup are incomplete.

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

container external-contribution Pull request is from an external contributor feat size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant