helm / chart-releaser-action

A GitHub Action to turn a GitHub project into a self-hosted Helm chart repo, using helm/chart-releaser CLI tool
https://github.com/helm/chart-releaser
Apache License 2.0
539 stars 197 forks source link

Multiple helm charts are not supported #198

Open sdoncila opened 1 month ago

sdoncila commented 1 month ago

Version: chart-releaser-action@1.6.0

Hello, I have multiple helm charts in my projects-charts folder and I get GIT errors if I modify more than 1. I use this the below settigns, because it is a private repository and for some reason if I don't use packages_with_index to upload also in the branch, my ArgoCD gets 404 while doing helm pull, but locally all is fine doing helm repo add:

Did anybody manage to have more than 1 helm chart change at a time?:

Successfully packaged chart in /home/runner/work/charts/pcharts/projects-charts/graf and saved it to: .cr-release-packages/appgraf-1.0.1.tgz Packaging chart 'projects-charts/ux-project-app'... Successfully packaged chart in /home/runner/work/charts/pcharts/projects-charts/ux-project-app and saved it to: .cr-release-packages/app1-sample-app-1.0.7.tgz Releasing charts... Preparing worktree (detached HEAD cbaf1d0) HEAD is now at cbaf1d0 Update index.yaml [detached HEAD 14a116f] Publishing chart package for app1-sample-app-1.0.7 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 app1-sample-app-1.0.7.tgz Pushing to branch "gh-pages" To https://github.com/HIDDEN/charts cbaf1d0..14a116f HEAD -> gh-pages Preparing worktree (detached HEAD cbaf1d0) HEAD is now at cbaf1d0 Update index.yaml [detached HEAD b947ab1] Publishing chart package for appgraf-1.0.1 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 appgraf-1.0.1.tgz Pushing to branch "gh-pages" To https://github.com/HIDDEN/charts ! [rejected] HEAD -> gh-pages (non-fast-forward) error: failed to push some refs to 'https://github.com/HIDDEN/charts' hint: Updates were rejected because a pushed branch tip is behind its remote hint: counterpart. If you want to integrate the remote changes, use 'git pull' hint: before pushing again.

All works fine if you only change 1 chart at a time, but it is very annoying when you do updates to multiple.

Thank you

0xThresh commented 4 weeks ago

I've run into the same issue here: https://github.com/open-webui/helm-charts/actions/runs/9388614861/job/25854221012

I'll report back if I find a fix.

Doncilas commented 4 weeks ago

Thanks for the response.

In the meantime I have dropped investigating chart release/gh-pages. I just don't like it, the whole implementation is a little messy, besides the bugs, why would you have a whole branch filled with charts? :)

On the other hand, considering that my projects are in AWS, I have found a cool way to host charts in AWS ECR https://docs.aws.amazon.com/AmazonECR/latest/userguide/push-oci-artifact.html

Super clean and implements beautifully with ArgoCD. Github has also OCI support, but again, it's way behind Gitlab and even AWS.

Best regards, Sergiu