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

ci: publish the tufaceous manifest #5403

Merged
merged 1 commit into from
Apr 3, 2024
Merged

ci: publish the tufaceous manifest #5403

merged 1 commit into from
Apr 3, 2024

Conversation

iliana
Copy link
Contributor

@iliana iliana commented Apr 3, 2024

While working on updating the script we use for downloading TUF repos over in meta, I was hitting unexpected 403 errors. This is because the script wants to also download the manifest.toml, and it can only get the URL for that artefact by hitting GitHub's check runs endpoint to get the URL from the job status posted there by Buildomat, and GitHub rate limits anonymous API requests significantly.

The fewer things we ask of GitHub the better, I think.

@iliana iliana requested a review from jgallagher April 3, 2024 19:59
@iliana iliana merged commit fd67302 into main Apr 3, 2024
20 checks passed
@iliana iliana deleted the iliana/tuf-repo-manifest branch April 3, 2024 23:33
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

3 participants