Skip to content

Conversation

@dagil-nvidia
Copy link
Contributor

@dagil-nvidia dagil-nvidia commented Nov 25, 2025

Remove Frontend Image from Support Matrix

Summary

Removes the Dynamo Frontend Images section from the support matrix documentation.

Changes

  • Removed Dynamo Frontend Images section (lines 105-106)
  • This section mentioned the dynamo-frontend container as a standalone implementation new in v0.7.0

Rationale

The frontend image reference was included prematurely and should not be in the main branch support matrix at this time.

Testing

  • Documentation builds without errors
  • No linter errors
  • Section removed cleanly without affecting surrounding content

Signed-off-by: Dan Gil dagil@nvidia.com

Summary by CodeRabbit

  • Documentation
    • Updated support matrix with new Helm Chart support for Dynamo Graph
    • Removed outdated multi-architecture frontend Docker image references from build support section

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Dan Gil <dagil@nvidia.com>
@copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 25, 2025

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.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 25, 2025

Walkthrough

The support matrix documentation was updated by removing the Dynamo Frontend Images bullet from the Build Support section and adding a new Helm Chart entry for Dynamo Graph under Helm Charts.

Changes

Cohort / File(s) Summary
Documentation Updates
docs/reference/support-matrix.md
Removed Dynamo Frontend Images entry and its "New as of Dynamo v0.7.0" note from Build Support section; added new Helm Chart entry for Dynamo Graph

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Straightforward documentation maintenance with no logic changes
  • Simple removal and addition of reference entries

Poem

🐰 A hop through the docs, so bright and clear,
We tidied the matrix, removed what's old here,
New Helm Charts gleam in the garden we tend,
The support path's charted—a rabbit's delight, my friend!

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removing the frontend image entry from the support matrix documentation.
Description check ✅ Passed The description covers the key sections (Overview, Details, Rationale, Testing) but lacks explicit 'Where should the reviewer start?' and 'Related Issues' sections from the template.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a77558d and 2d4db6a.

📒 Files selected for processing (1)
  • docs/reference/support-matrix.md (0 hunks)
💤 Files with no reviewable changes (1)
  • docs/reference/support-matrix.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Build Documentation
  • GitHub Check: Build and Test - dynamo

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@dagil-nvidia dagil-nvidia enabled auto-merge (squash) November 25, 2025 21:18
@dagil-nvidia
Copy link
Contributor Author

/ok to test 2d4db6a

@dagil-nvidia dagil-nvidia self-assigned this Nov 25, 2025
@dagil-nvidia dagil-nvidia merged commit 445b4bd into main Nov 25, 2025
26 of 27 checks passed
@dagil-nvidia dagil-nvidia deleted the dagil/remove-frontend-from-support-matrix branch November 25, 2025 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants