Skip to content

fix(chart): agent container path pointing to skyhook not nodewright#250

Merged
ayuskauskas merged 1 commit into
mainfrom
fix/chart
May 22, 2026
Merged

fix(chart): agent container path pointing to skyhook not nodewright#250
ayuskauskas merged 1 commit into
mainfrom
fix/chart

Conversation

@ayuskauskas
Copy link
Copy Markdown
Collaborator

Description

Change agent from skyhook to nodewright

Checklist

  • I am familiar with the Contributing Guidelines.
  • My commits are signed off (git commit -s) per the DCO.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@ayuskauskas ayuskauskas merged commit 6a75900 into main May 22, 2026
30 of 31 checks passed
@ayuskauskas ayuskauskas deleted the fix/chart branch May 22, 2026 21:41
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: 778f0a0c-b006-4af1-b6d3-1c50ca31e026

📥 Commits

Reviewing files that changed from the base of the PR and between 046fe2e and 4b3caae.

📒 Files selected for processing (1)
  • chart/values.yaml

📝 Walkthrough

Walkthrough

This PR updates the Helm chart values to reference the agent container image from a new repository location. The controllerManager.manager.agent.repository value is changed from ghcr.io/nvidia/skyhook/agent to ghcr.io/nvidia/nodewright/agent. The agent image tag and digest values remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • lockwobr
  • mskalka
  • rice-riley
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/chart

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

This was referenced May 22, 2026
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26313346177

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Warning

No base build found for commit 046fe2e on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 81.704%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 9461
Covered Lines: 7730
Line Coverage: 81.7%
Coverage Strength: 9.59 hits per line

💛 - Coveralls

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