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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: skip helm chart upload if it already exists #320

Merged
merged 1 commit into from
Apr 19, 2023

Conversation

kbudde
Copy link
Collaborator

@kbudde kbudde commented Apr 18, 2023

Chart releaser action is failing since last release on every commit to master.
Don't know why. Maybe ignoring the issue is the way 馃し馃徑

@zhan9san
Copy link
Collaborator

CR_SKIP_EXISTING is fine. It would be more great if the action supports ignoring some directoies/files in chart directory like .gitignore, .helmignore and README.md

It is because the chart-release-action detects there are changes in chart directory but the version in chart is not updated. It will try to release a version, but the version exists.

https://github.com/helm/chart-releaser-action/blob/db5211eb38c65dcd18cbb57e987df03fce2a40f9/cr.sh#L257-L258

I create an issue in chart-releaser-action.

helm/chart-releaser-action#139

@kbudde kbudde merged commit 88cd542 into master Apr 19, 2023
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

Successfully merging this pull request may close these issues.

None yet

2 participants