Skip to content

fix(ci): push Helm Chart to GHCR - #720

Merged
wrenix merged 2 commits into
nextcloud:mainfrom
onedr0p:patch-1
May 15, 2025
Merged

fix(ci): push Helm Chart to GHCR#720
wrenix merged 2 commits into
nextcloud:mainfrom
onedr0p:patch-1

Conversation

@onedr0p

@onedr0ponedr0p commented Mar 30, 2025

Copy link
Copy Markdown
Contributor

Important

After this is merged and charts are released you might need to make the packages public from here

I also bumped some deps

Fixes: #713

>[!IMPORTANT]
> After this is merged and charts are released you will need to make the packages public from [here](https://github.com/orgs/nextcloud/packages)
I also bumped some deps
Fixes: nextcloud#713
Signed-off-by: Devin Buhl <onedr0p@users.noreply.github.com>
@wrenix

Copy link
Copy Markdown
Collaborator

I will test it ... but i am not able for the ghcr.io configuration as public

Comment thread.github/workflows/release.yaml
@wrenix

wrenix commented May 10, 2025

Copy link
Copy Markdown
Collaborator

The code looks good for me, i test it here:
https://github.com/wrenix/nextcloud-helm/pkgs/container/nextcloud-helm%2Fnextcloud

So we really need to clearify the break part

@wrenixwrenix changed the title ci: push Helm Chart to GHCRfix(ci): push Helm Chart to GHCRMay 15, 2025
@wrenix
wrenix enabled auto-merge (squash) May 15, 2025 21:13

@wrenixwrenix left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Okay looks nice

@wrenix
wrenix merged commit 2bf2d3a into nextcloud:mainMay 15, 2025
@onedr0p
onedr0p deleted the patch-1 branch May 15, 2025 23:06
@Syntax3rror404

Copy link
Copy Markdown

How can I pull it?

@onedr0p

Copy link
Copy Markdown
ContributorAuthor

It should be built and pushed on the chart next release.

@wrenixwrenix mentioned this pull request Jun 19, 2025
2 tasks
Sign up for freeto 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.

Request: Upload Helm chart to OCI registry

3 participants

@onedr0p@wrenix@Syntax3rror404