Fix redfish vendor vera rubin

[hakhondzadeh]

fix: Add RedfishVendor::VeraRubin to be compatible with libredfish changes.
Bump up the libredfish version

Related issues

Vera Rubin compute tray ingestion.

[coderabbitai]

Walkthrough

This PR extends Redfish vendor handling so P3809 and VeraRubin map to BMCVendor::Nvidia, routes VeraRubin through the direct root-password change path, and bumps the pinned libredfish dependency.

Changes

VeraRubin/P3809 NVIDIA BMC vendor mapping

Layer / File(s)	Summary
Vendor-to-BMC mapping and password rotation path crates/redfish/src/libredfish/conv.rs, crates/site-explorer/src/redfish.rs, Cargo.toml	bmc_vendor now maps P3809 and VeraRubin to BMCVendor::Nvidia; set_bmc_root_password routes VeraRubin through the same direct change_password_by_id path as NvidiaGBx00; the workspace libredfish git tag is updated to v0.44.18.

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

Possibly related PRs

• NVIDIA/infra-controller#2910: Related Redfish vendor handling for VeraRubin, including NVIDIA-aligned mapping behavior.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is concise and accurately reflects the main change: adding Vera Rubin support in Redfish vendor handling.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description check	✅ Passed	The description accurately matches the code changes: VeraRubin vendor handling and a libredfish version bump.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[coderabbitai]

🧹 Nitpick comments (1)

crates/redfish/src/libredfish/conv.rs (1)

150-169: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Add a value_scenarios! case for the new vendor aliases. Pin P3809 and VeraRubin to BMCVendor::Nvidia so this mapping stays covered by a table-driven test.

🤖 Prompt for 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.

In `@crates/redfish/src/libredfish/conv.rs` around lines 150 - 169, The bmc_vendor
mapping in conv.rs now includes new RedfishVendor aliases P3809 and VeraRubin,
but the table-driven coverage in value_scenarios! does not exercise them yet.
Add a new value_scenarios! case for bmc_vendor that includes these aliases and
asserts they map to BMCVendor::Nvidia, alongside the existing
vendor-to-BMCVendor conversions.

Source: Path instructions

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

Nitpick comments:
In `@crates/redfish/src/libredfish/conv.rs`:
- Around line 150-169: The bmc_vendor mapping in conv.rs now includes new
RedfishVendor aliases P3809 and VeraRubin, but the table-driven coverage in
value_scenarios! does not exercise them yet. Add a new value_scenarios! case for
bmc_vendor that includes these aliases and asserts they map to
BMCVendor::Nvidia, alongside the existing vendor-to-BMCVendor conversions.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 1306161d-e757-4e28-a576-bfe357d3f106

📥 Commits

Reviewing files that changed from the base of the PR and between a171cb1 and ea980de.

📒 Files selected for processing (2)

• crates/redfish/src/libredfish/conv.rs
• crates/site-explorer/src/redfish.rs

[github-actions]

🔍 Container Scan Summary

Service	Total	Critical	High	Medium	Low	Other
boot-artifacts-aarch64	3	0	0	3	0	0
boot-artifacts-x86_64	3	0	0	3	0	0
forge-admin-cli-x86_64	302	5	32	113	8	144
machine-validation-runner	769	25	208	278	37	221
machine_validation	769	25	208	278	37	221
machine_validation-aarch64	769	25	208	278	37	221
nvmetal-carbide	769	25	208	278	37	221
TOTAL	3384	105	864	1231	156	1028

Per-CVE detail lives in the per-service grype-* artifacts (JSON + SARIF). Severity counts only — no CVE IDs published here.