Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple helm charts are not supported #198

Open
sdoncila opened this issue May 17, 2024 · 3 comments
Open

Multiple helm charts are not supported #198

sdoncila opened this issue May 17, 2024 · 3 comments

Comments

@sdoncila
Copy link

sdoncila commented May 17, 2024

Version: [email protected]

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/[email protected]
    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

@0xThresh
Copy link

0xThresh commented Jun 5, 2024

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
Copy link

Doncilas commented Jun 6, 2024

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

@Justin-DynamicD
Copy link

FYI this is a bug with the core chart-releaser that this action leverages. The fix has been merged to main but not tagged/released offically yet:

helm/chart-releaser#314 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants