-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
renovate: Fix Hubble release digest regex #24477
renovate: Fix Hubble release digest regex #24477
Conversation
Renovate v35 had a breaking change in how it computes the digest of GitHub releses. Previously, it would use the digest of the release attachements, but now it's just using a git sha as the "digest". This commit changes the data source for the Hubble artifacts to use the data source which preserves the old behavior. Ref: renovatebot/renovate#20178 Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
This needs to be merged before Renovate will be able to update the Hubble digests properly. I'm blocking the other PRs for now. |
Integration tests failed with a new flake #24491 - restarting. |
Integration tests failed again, this time with #23297 |
Seems like this is not working:
|
Was this resolved? If it's not working as intended, should we still backport to stable? 🤔 |
It was not resolved, due to the lack of cycles on my side. I need to investigate why renovate is refusing to update the attachment digests. Having said that, I think we should still backport the shellscript change here, since the previously used |
Status update. I've attempted running renovate locally to check what is happening, and it seems like renovate itself is unable to talk to the GitHub API:
Investigating weather this is a misconfiguration on our side, or simply a renovate bug (since |
Seems like a Renovate bug. Opened an issue upstream: renovatebot/renovate#21230 |
Renovate v35 had a breaking change in how it computes the digest of GitHub releases. Previously, it would use the digest of the release attachements, but now it's just using a git sha as the "digest". This commit changes the data source for the Hubble artifacts to use the data source which preserves the old behavior.
Ref: renovatebot/renovate#20178
Note: It might makes sense to merge the outstanding renovate PRs on this file first to not confuse Renovate further.chore(deps): update dependency cilium/hubble to v0.11.3 (v1.12) - autoclosed #24490chore(deps): update dependency cilium/hubble to v0.11.3 (v1.13) - autoclosed #24468renamed renovate PR (updates cilium/hubble on master) #24452