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

renovate: Fix Hubble release digest regex #24477

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

gandro
Copy link
Member

@gandro gandro commented Mar 20, 2023

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.

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>
@gandro gandro added release-note/ci This PR makes changes to the CI. needs-backport/1.11 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Mar 20, 2023
@gandro gandro requested review from a team as code owners March 20, 2023 17:53
@gandro gandro requested review from jibi and bimmlerd March 20, 2023 17:54
@gandro
Copy link
Member Author

gandro commented Mar 21, 2023

This needs to be merged before Renovate will be able to update the Hubble digests properly. I'm blocking the other PRs for now.

@gandro
Copy link
Member Author

gandro commented Mar 21, 2023

Integration tests failed with a new flake #24491 - restarting.

@gandro gandro removed the request for review from jibi March 21, 2023 09:13
@gandro
Copy link
Member Author

gandro commented Mar 21, 2023

Integration tests failed again, this time with #23297

@aanm aanm merged commit 30a783f into cilium:master Mar 21, 2023
@gandro
Copy link
Member Author

gandro commented Mar 21, 2023

Seems like this is not working:

DEBUG: Failed to look up github-release-attachments package cilium/hubble (cilium/hubble)(packageFile="images/cilium/download-hubble.sh", dependency="cilium/hubble")
{
  "baseBranch": "master"
}

@NikAleksandrov NikAleksandrov mentioned this pull request Mar 23, 2023
29 tasks
@NikAleksandrov NikAleksandrov added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Mar 23, 2023
@tklauser tklauser mentioned this pull request Mar 28, 2023
5 tasks
@tklauser tklauser mentioned this pull request Mar 28, 2023
6 tasks
@julianwiedmann
Copy link
Member

Seems like this is not working:

DEBUG: Failed to look up github-release-attachments package cilium/hubble (cilium/hubble)(packageFile="images/cilium/download-hubble.sh", dependency="cilium/hubble")
{
  "baseBranch": "master"
}

Was this resolved? If it's not working as intended, should we still backport to stable? 🤔

@gandro
Copy link
Member Author

gandro commented Mar 28, 2023

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 github-releases will cause renovate to open broken PRs on all stable branches without it (due to the change in Renovate's behavior). Once we understand what the issue is, it may be fixable by just updating renovate.json5 on master (which applies to all branches).

@tklauser tklauser added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.12 labels Mar 29, 2023
@gandro
Copy link
Member Author

gandro commented Mar 29, 2023

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:

               {
                 "depName": "cilium/hubble",
                 "currentValue": "v0.11.2",
                 "currentDigest": "7e1496bdc937dcf34de2f282edf62f91edc7ed1435ad78401035dc04e6b66f90",
                 "datasource": "github-release-attachments",
                 "replaceString": "# renovate: datasource=github-release-attachments depName=cilium/hubble digestVersion=v0.11.2\nhubble_sha256[arm64]=\"7e1496bdc937dcf34de2f282edf62f91edc7ed1435ad78401035dc04e6b66f90\"",
                 "updates": [],
                 "packageName": "cilium/hubble",
                 "warnings": [
                   {
                     "topic": "cilium/hubble",
                     "message": "Failed to look up github-release-attachments package cilium/hubble"
                   }
                 ],
                 "versioning": "semver"
               }
DEBUG: POST https://api.github.com/graphql = (code=ERR_NON_2XX_3XX_RESPONSE, statusCode=401 retryCount=0, duration=267) (repository=cilium/cilium, baseBranch=master)
DEBUG: Datasource unauthorized (repository=cilium/cilium, baseBranch=master)
       "datasource": "github-release-attachments",
       "packageName": "cilium/hubble",
       "url": "https://api.github.com/graphql"
DEBUG: Failed to look up github-release-attachments package cilium/hubble (repository=cilium/cilium, baseBranch=master, packageFile=images/cilium/download-hubble.sh, dependency=cilium/hubble)
DEBUG: Failed to look up github-release-attachments package cilium/hubble (repository=cilium/cilium, baseBranch=master, packageFile=images/cilium/download-hubble.sh, dependency=cilium/hubble)
DEBUG: Failed to look up github-release-attachments package cilium/hubble (repository=cilium/cilium, baseBranch=master, packageFile=images/cilium/download-hubble.sh, dependency=cilium/hubble)

Investigating weather this is a misconfiguration on our side, or simply a renovate bug (since github-release-attachments doesn't seem to be used in the wild)

@gandro
Copy link
Member Author

gandro commented Mar 29, 2023

Seems like a Renovate bug. Opened an issue upstream: renovatebot/renovate#21230

@gandro gandro deleted the pr/gandro/renovate-fix-hubble-gh-release branch April 4, 2023 15:45
@jibi jibi added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. labels Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants