chore(docs): provide clear guidance on DPU configuration for site-controller nodes#3048
Open
spydaNVIDIA wants to merge 1 commit into
Open
chore(docs): provide clear guidance on DPU configuration for site-controller nodes#3048spydaNVIDIA wants to merge 1 commit into
spydaNVIDIA wants to merge 1 commit into
Conversation
Contributor
Summary by CodeRabbit
WalkthroughThis change updates the site controller DPU requirements across the quick-start and hardware prerequisite documentation. It now requires fully provisioned BlueField-3 DPUs, DPU mode only, and the listed firmware, optics, connectivity, and download details. ChangesSite controller DPU requirements docs
🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
🔍 Container Scan SummaryNo Grype artifacts were found to aggregate. |
|
🌿 Preview your docs: https://nvidia-preview-pull-request-3048.docs.buildwithfern.com/infra-controller |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/getting-started/prerequisites/hardware.md`:
- Line 31: The connectivity example in the hardware prerequisites doc uses curl
with -k, which bypasses TLS verification; update the example in the connectivity
check to use plain curl so it actually validates the trust path. Keep the
guidance aligned with the surrounding prerequisites content and adjust the
example text in the relevant markdown section so it remains realistic and safe
for operators.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 3db2954a-cf23-41db-8c23-3f7f286a687e
📒 Files selected for processing (2)
docs/getting-started/prerequisites/hardware.mddocs/getting-started/quick-start.md
✅ Files skipped from review due to trivial changes (1)
- docs/getting-started/quick-start.md
polarweasel
requested changes
Jul 1, 2026
polarweasel
left a comment
polarweasel
left a comment
Contributor
There was a problem hiding this comment.
Minor stuff, including arguing with Coderabbit 😁
Comment on lines
+27
to
+29
| Today, site-controller nodes must have Bluefield-3 DPUs. Ensure the following requirements are met: | ||
| - Verify the correct DPU power cable has been ordered from the server vendor. | ||
| - The Bluefield-3's operating mode is DPU mode (not NIC mode). Today, NIC mode is not supported. |
Contributor
There was a problem hiding this comment.
Suggested change
| Today, site-controller nodes must have Bluefield-3 DPUs. Ensure the following requirements are met: | |
| - Verify the correct DPU power cable has been ordered from the server vendor. | |
| - The Bluefield-3's operating mode is DPU mode (not NIC mode). Today, NIC mode is not supported. | |
| Site-controller nodes must have Bluefield-3 DPUs. Ensure the following requirements are met: | |
| - You have the correct DPU power cable from the server vendor. | |
| - The Bluefield-3's operating mode is DPU mode. NIC mode is not supported. |
| - The Bluefield-3's operating mode is DPU mode (not NIC mode). Today, NIC mode is not supported. | ||
| - For BF3 DPUs, verify link speed and optics: BF3 runs at 200 Gb, so match ports to 200 Gb-capable optics, fiber, or DACs. | ||
| - A basic onboard NIC for management is sufficient--no extra ConnectX NICs are needed. | ||
| - Verify that the DPU can connect to the outside world (curl -k https://www.google.com) |
Contributor
There was a problem hiding this comment.
Use curl -I to get a quick connection report (just the HTTP status code and some headers). Also... why not ping NVIDIA instead of Google?
Suggested change
| - Verify that the DPU can connect to the outside world (curl -k https://www.google.com) | |
| - Verify that the DPU can connect to the outside world (curl -I https://www.nvidia.com) |
| - For BF3 DPUs, verify link speed and optics: BF3 runs at 200 Gb, so match ports to 200 Gb-capable optics, fiber, or DACs. | ||
| - A basic onboard NIC for management is sufficient--no extra ConnectX NICs are needed. | ||
| - Verify that the DPU can connect to the outside world (curl -k https://www.google.com) | ||
| - The DPUs are at the latest supported firmware version: DOCA 2.9.3 and HBN 2.4.3 |
Contributor
There was a problem hiding this comment.
Do you want to include these numbers here and have to maintain them, or include links to their release pages instead?
Comment on lines
+40
to
+44
| - Flash the DPU firmware to the latest supported version using the BlueField Firmware Bundle. Latest supported firmware versions: | ||
|
|
||
| | DOCA | HBN | | ||
| | ----- | ----- | | ||
| | 2.9.3 | 2.4.3 | |
Contributor
There was a problem hiding this comment.
Same question as previous file: if you include version numbers, you have to maintain them, vs linking to release pages and letting the reader figure out what they need to do. UNLESS...do we not support the very latest releases?
| | ----- | ----- | | ||
| | 2.9.3 | 2.4.3 | | ||
|
|
||
| - Configure the Bluefield-3 device in DPU mode (operating mode). We do not support having DPUs in NIC mode today. |
Contributor
There was a problem hiding this comment.
Suggested change
| - Configure the Bluefield-3 device in DPU mode (operating mode). We do not support having DPUs in NIC mode today. | |
| - Configure the Bluefield-3 device in DPU mode (operating mode). We do not currently support NIC mode. |
|
|
||
| - Configure the Bluefield-3 device in DPU mode (operating mode). We do not support having DPUs in NIC mode today. | ||
| - Ensure the DPU ARM OS is booted and reachable via its management interface. | ||
| - Verify that the DPU can connect to the outside world (curl -k https://www.google.com) |
Contributor
There was a problem hiding this comment.
Same as previous file...
Suggested change
| - Verify that the DPU can connect to the outside world (curl -k https://www.google.com) | |
| - Verify that the DPU can connect to the outside world (curl -I https://www.nvidia.com) |
Comment on lines
50
to
+52
| Refer to the NVIDIA DOCA documentation and the BlueField Firmware Bundle download archive for firmware flashing instructions and supported firmware versions: | ||
|
|
||
| [https://developer.nvidia.com/doca-2-9-2-lts-ovs-doca-download-archive?deployment_platform=BlueField&deployment_package=BF-FW-Bundle](https://developer.nvidia.com/doca-2-9-2-lts-ovs-doca-download-archive?deployment_platform=BlueField&deployment_package=BF-FW-Bundle) | ||
| [https://developer.nvidia.com/doca-2-9-3-download-archive?deployment_platform=BlueField&deployment_package=BF-FW-Bundle](https://developer.nvidia.com/doca-2-9-3-download-archive?deployment_platform=BlueField&deployment_package=BF-FW-Bundle) |
Contributor
There was a problem hiding this comment.
Same question, really. :) Can we just link to the docs and the release page(s) directly, instead of having to increment here every time these get updated?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
chore(docs): provide clear guidance on DPU configuration for site-controller nodes
Related issues
#2992
Type of Change
Breaking Changes
Testing
Additional Notes