Skip to content
New issue

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

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

Already on GitHub? # to your account

[ci] add skip_existing to chart-releaser #5195

Merged
merged 1 commit into from
Jan 18, 2025
Merged

[ci] add skip_existing to chart-releaser #5195

merged 1 commit into from
Jan 18, 2025

Conversation

jkroepke
Copy link
Member

@jkroepke jkroepke commented Jan 17, 2025

What this PR does / why we need it

Should solve the following error https://github.com/prometheus-community/helm-charts/actions/runs/12834237724/job/35791102326 mention by #5190 (comment)

The fix is documented here: https://github.com/helm/chart-releaser/blob/cb3ed7107e0adfb5eae170bfc01bf61ee811e87a/README.md#common-error-messages

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Signed-off-by: Jan-Otto Kröpke <github@jkroepke.de>
@jkroepke
Copy link
Member Author

@GMartinez-Sisti you may have to approve as well, since you are a CODEOWNER :)

@jkroepke jkroepke merged commit 8cce564 into main Jan 18, 2025
6 checks passed
@jkroepke jkroepke deleted the jkroepke-patch-1 branch January 18, 2025 12:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants