-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
CI: Helm chart publishing still broken #14835
Comments
@morganrowse if you're still interested, looks like the saga isn't quite over |
Hi @trevorwhitney just got back from leave and now ready to look into this. I have a theory that this CI issue was introduced by some upstream changes to It looks like the core issue here follows this PR grafana/helm-charts#3418 @narqo just raising for visibility if you have not seen this for the Loki chart. Im not seeing any fundemental code differences between mimr and loki CI. Are there GitHub app or repo permission settings required to get this working as expected? |
It should just work since grafana/helm-charts#3424 — no special changes on your side needed (it works for k6-operator also). |
@narqo we're referencing thew workflow @ |
Awesome, thanks @morganrowse, I reverted the release pipeline to |
#14799 fixed the
helm-ci
action, allowing helm PRs to be merged. However, publishing releases is still broken:Last successful run was 3 weeks ago: https://github.com/grafana/loki/actions/runs/11374046600
The text was updated successfully, but these errors were encountered: