Skip to content

Fix Helm chart OCI publish path to match original registry location#3635

Open
ChrisJBurns wants to merge 1 commit intomainfrom
fix/helm-chart-publish-path
Open

Fix Helm chart OCI publish path to match original registry location#3635
ChrisJBurns wants to merge 1 commit intomainfrom
fix/helm-chart-publish-path

Conversation

@ChrisJBurns
Copy link
Collaborator

Summary

  • The helm-publish.yml workflow was using github.repository_owner (stacklok) instead of github.repository (stacklok/toolhive) as the OCI registry path prefix
  • This caused charts to publish to ghcr.io/stacklok/toolhive-operator instead of the original ghcr.io/stacklok/toolhive/toolhive-operator
  • Restores the original paths that match the old releaser-helm-charts.yml workflow that was replaced in Add unified Helm chart publishing to release workflow #3511

Test plan

  • Verify next release publishes charts to ghcr.io/stacklok/toolhive/toolhive-operator
  • Verify next release publishes charts to ghcr.io/stacklok/toolhive/toolhive-operator-crds
  • Confirm Cosign signing and verification commands use the correct paths

🤖 Generated with Claude Code

The helm-publish workflow was using github.repository_owner (stacklok)
instead of github.repository (stacklok/toolhive) as the OCI path prefix,
causing charts to publish to ghcr.io/stacklok/toolhive-operator instead
of the original ghcr.io/stacklok/toolhive/toolhive-operator.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ChrisJBurns ChrisJBurns requested a review from JAORMX as a code owner February 5, 2026 20:59
@github-actions github-actions bot added the size/XS Extra small PR: < 100 lines changed label Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.71%. Comparing base (533821b) to head (9d11979).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3635      +/-   ##
==========================================
- Coverage   65.78%   65.71%   -0.08%     
==========================================
  Files         411      411              
  Lines       40727    40727              
==========================================
- Hits        26794    26763      -31     
- Misses      11850    11880      +30     
- Partials     2083     2084       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants