fix: pull in additional PNs to GB200xBF3 mlxconfig profile

[chet]

GB200 B3240 DPUs can report 900-9D3B6-00CN-PN0 in bootstrap Redfish data and 900-9D3B6-00CN-P_Ax after Scout records the device identity. The shared selector did not accept either value, so DPF and Non-DPF provisioning selected the generic BF3 profile for those devices even when the rack was identified as GB200.

This adds both exact identities to the shared GB200 B3240 selector. The separate GB200 rack check remains required, and other B3240 prefixes remain rejected.

Related issues

• Closes Recognize Additional GB200 B3240 Part Numbers for NVConfig Selection #5541
• Follows fix: add NVConfig profile support for Non-DPF GB200 provisioning #5444 and fix: apply GB200 NVConfig during DPF provisioning #5468
• Part of feat: (dpu-os): gracefully resolve physical HBN uplinks when extra interfaces are exposed #5029

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.)

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.)

• cargo test -p carbide-libmlx-model

• cargo make clippy

• cargo make format-nightly

• git diff --check

Additional Notes

The existing DPF and Non-DPF integrations already consume this shared selector, so this PR does not change their NVConfig assignments.

The expanded local Carbide lints run (--all-targets --all-features) reaches existing txn_held_across_await failures in unchanged crates/api-core/tests/integration/dns_resolution.rs; that file is identical to origin/main. The hosted lint task does not compile that integration target.

[chet]

@coderabbitai full_review, thanks!

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-08-28T23:42:18.845071Z	042a81c	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[coderabbitai]

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: db8252d5-395b-48fa-93f1-f8d4609ab0eb

📥 Commits

Reviewing files that changed from the base of the PR and between 77355c5 and 042a81c.

📒 Files selected for processing (1)

• crates/libmlx-model/src/nvconfig.rs

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

---

Summary by CodeRabbit

• Bug Fixes
  • Added support for two additional GB200/B3240 part numbers.
  • Updated validation and documentation to recognize the newly supported hardware identities.
  • Expanded test coverage for accepted part numbers.

Walkthrough

The NVConfig selector now accepts two additional GB200/B3240 part numbers. Documentation and table-driven tests reflect the expanded identity set.

Changes

GB200 B3240 identity support

Layer / File(s)	Summary
Identity list and selector validation crates/libmlx-model/src/nvconfig.rs	The selector accepts 900-9D3B6-00CN-PN0 and 900-9D3B6-00CN-P_Ax. Documentation lists both identities. Tests verify that both resolve to Gb200B3240V1 while retaining whitespace and case normalization coverage.

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

Merge Risk: ⚪ Minimal · up to 042a8

This change expands GB200 device matching for the intended identities without altering the existing rack check or provisioning assignments. No actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes address issue #5541 by adding both required exact GB200 B3240 identities, preserving the rack check and matching behavior, updating selector tests, and leaving integrations and NVConfig as…
Out of Scope Changes check	✅ Passed	The changes are limited to the shared selector, related documentation, and table-driven tests. They align with the linked issue objectives and introduce no unrelated scope.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files.
Description check	✅ Passed	The description clearly explains the added GB200 B3240 identities, the provisioning issue, preserved selector constraints, and validation performed.
Title check	✅ Passed	The title clearly identifies the fix: adding additional part numbers to the GB200xBF3 mlxconfig profile selector.

Full details: Linked Issues check

Explanation

The changes address issue #5541 by adding both required exact GB200 B3240 identities, preserving the rack check and matching behavior, updating selector tests, and leaving integrations and NVConfig assignments unchanged.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[coderabbitai]

@chet: I will perform a complete review of pull request #5544.

🐇🔍

✅ Action performed

Full review finished.