search

helm / chart-releaser

Hosting Helm Charts via GitHub Pages and Releases
Apache License 2.0
644 stars 107 forks source link

Multiple helm charts are not supported #411

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:

name: Run chart-releaser uses: helm/chart-releaser-action@v1.6.0 with: charts_dir: projects-charts skip_existing: true packages_with_index: true 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

Justin-DynamicD commented 3 weeks ago

Adding a voice to this. It looks like it's a problem where it simply attempts to push one package per commit without staying on head. If you simply bundled into one large commit or added a pull between each push it should work fine.