Skip to content

fix: Update OCI registry publishing in release workflow - #2294

Open
TerryHowe wants to merge 2 commits into
goharbor:mainfrom
TerryHowe:fix/oci-registry-publishing
Open

fix: Update OCI registry publishing in release workflow#2294
TerryHowe wants to merge 2 commits into
goharbor:mainfrom
TerryHowe:fix/oci-registry-publishing

Conversation

@TerryHowe

Copy link
Copy Markdown

Description

This PR fixes the OCI registry publishing issue in the release workflow.

Changes

  • Remove Docker Hub push that fails with authentication errors due to missing credentials
  • Fix GHCR push to use the organization path (ghcr.io/goharbor) instead of personal account (ghcr.io/${{ github.actor }})
  • Properly authenticate using GITHUB_TOKEN which is available in GitHub Actions

Fixes

Closes #2265

@TerryHowe
TerryHowe force-pushed the fix/oci-registry-publishing branch from c8fac53 to c59accd Compare January 7, 2026 17:03

@bupd bupd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @TerryHowe

I believe we should fix the env vars rather than removing the dockerhub push.

looks like this has been failing for a past couple of months

Image

@bupd

bupd commented Jan 7, 2026

Copy link
Copy Markdown
Member

@MinerYang thoughts ??

@TerryHowe

Copy link
Copy Markdown
Author

Heya @bupd been a minute. I was thinking delivering charts to ghcr would almost certain work and that would unblock the problem. If someone really wants to deliver to docker.io, that could be a follow up.

@MinerYang

Copy link
Copy Markdown
Collaborator

Sorry everyone, I was just back from a sick leave. Will take a look recently.

@TerryHowe

Copy link
Copy Markdown
Author

ping

@bupd

bupd commented Feb 19, 2026

Copy link
Copy Markdown
Member

cc // @MinerYang

@bupd bupd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Comment thread .github/workflows/publish_release.yaml Outdated
@sillock1

sillock1 commented Mar 4, 2026

Copy link
Copy Markdown

Is there anything else we need to do to get this merged? I'd very much like to use the OCI registry as the helm repository reference

@TerryHowe
TerryHowe force-pushed the fix/oci-registry-publishing branch from a8bd432 to 8c60b29 Compare March 5, 2026 12:12
@AntonOfTheWoods

Copy link
Copy Markdown

@cthtrifork , is this now good to go from your perspective?

@cthtrifork

Copy link
Copy Markdown

Sorry everyone, I was just back from a sick leave. Will take a look recently.

I am not the maintainer, but let us know if we can further!

@q-b

q-b commented Jun 17, 2026

Copy link
Copy Markdown

@MinerYang is there anything left to do in order to get this merged?

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the GitHub Actions release workflow to publish the Helm chart as an OCI artifact to GitHub Container Registry (GHCR), addressing failures caused by incorrect registry targets and missing credentials for Docker Hub.

Changes:

  • Removes Docker Hub OCI login/push from the release workflow.
  • Updates the GHCR OCI push target from the triggering user namespace to the repository owner namespace.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/publish_release.yaml Outdated
Comment thread .github/workflows/publish_release.yaml Outdated
TerryHowe added 2 commits July 7, 2026 07:47
Remove Docker Hub push that fails with authentication errors due to
missing credentials. Fix GHCR push to use GITHUB_REPOSITORY_OWNER
for the organization path instead of github.actor.

Closes goharbor#2265

Signed-off-by: Terry Howe <terrylhowe@gmail.com>
Signed-off-by: Terry Howe <terrylhowe@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Release Harbor Helm Chart as OCI artifact

8 participants