Skip to content

fix: Don't skip IB and NVLink checks during ingestion for no-dpu hosts#845

Open
Matthias247 wants to merge 1 commit intoNVIDIA:mainfrom
Matthias247:nodpu
Open

fix: Don't skip IB and NVLink checks during ingestion for no-dpu hosts#845
Matthias247 wants to merge 1 commit intoNVIDIA:mainfrom
Matthias247:nodpu

Conversation

@Matthias247
Copy link
Copy Markdown
Contributor

Description

On no-dpu hosts there is no need to wait until DPU configurations are applied since there are no DPUs.

However the instance provisiniong process still needs to wait until IB and NVLink are configured according to user requirements.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

On no-dpu hosts there is no need to wait until DPU configurations are applied since there are no DPUs.

However the instance provisiniong process still needs to wait until IB and NVLink are configured according to user requirements.

Signed-off-by: Matthias Einwag <meinwag@nvidia.com>
@Matthias247 Matthias247 requested a review from a team as a code owner April 8, 2026 00:26
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔐 TruffleHog Secret Scan

No secrets or credentials found!

Your code has been scanned for 700+ types of secrets and credentials. All clear! 🎉

🔗 View scan details

🕐 Last updated: 2026-04-08 00:27:59 UTC | Commit: 51a98ee

@Matthias247 Matthias247 enabled auto-merge (squash) April 8, 2026 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants