Skip to content

fix: bump libredfish to v0.44.15#2854

Merged
krish-nvidia merged 2 commits into
NVIDIA:mainfrom
krish-nvidia:krish/smc-bmc-factory-reset
Jun 24, 2026
Merged

fix: bump libredfish to v0.44.15#2854
krish-nvidia merged 2 commits into
NVIDIA:mainfrom
krish-nvidia:krish/smc-bmc-factory-reset

Conversation

@krish-nvidia

Copy link
Copy Markdown
Contributor

This PR bumps libredfish to v0.44.15 which brings in the following change:

Related issues

#2121

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

Additional Notes

Signed-off-by: Krish Dandiwala <kdandiwala@nvidia.com>
@krish-nvidia
krish-nvidia requested a review from spydaNVIDIA June 24, 2026 19:14
@krish-nvidia krish-nvidia self-assigned this Jun 24, 2026
@krish-nvidia
krish-nvidia requested a review from a team as a code owner June 24, 2026 19:14
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 935ac823-807a-4b54-b764-69e42cf2fc42

📥 Commits

Reviewing files that changed from the base of the PR and between c34597c and 6832bb7.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

Summary by CodeRabbit

  • Chores
    • Updated a pinned workspace dependency to the latest patch release.
    • No user-facing features or behavior changes are included in this update.

Walkthrough

The workspace dependency pin for libredfish in Cargo.toml is updated from v0.44.14 to v0.44.15.

Changes

libredfish dependency bump

Layer / File(s) Summary
Dependency tag update
Cargo.toml
libredfish is repinned in the workspace dependencies from git tag v0.44.14 to v0.44.15.

Sequence Diagram(s)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the main change: bumping libredfish to v0.44.15.
Description check ✅ Passed The description matches the changeset and explains the dependency bump and its purpose.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@krish-nvidia
krish-nvidia enabled auto-merge (squash) June 24, 2026 19:16
@github-actions

Copy link
Copy Markdown

@krish-nvidia
krish-nvidia merged commit ecf44bd into NVIDIA:main Jun 24, 2026
55 checks passed
@github-actions

Copy link
Copy Markdown

🔍 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 265 6 24 98 7 130
machine-validation-runner 717 32 188 267 36 194
machine_validation 717 32 188 267 36 194
machine_validation-aarch64 717 32 188 267 36 194
nvmetal-carbide 717 32 188 267 36 194
TOTAL 3139 134 776 1172 151 906

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

@coderabbitai coderabbitai Bot mentioned this pull request Jun 25, 2026
10 tasks
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.

2 participants