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

Use token for Codecov #2696

Open
stefan6419846 opened this issue May 30, 2024 · 0 comments
Open

Use token for Codecov #2696

stefan6419846 opened this issue May 30, 2024 · 0 comments
Labels
Meta nf-ci Non-functional change: Continuous Integration

Comments

@stefan6419846
Copy link
Collaborator

At the moment, we are using tokenless uploads to Codecov. This regularly requires to restart the coverage upload job until it has been successful and the Codecov comment is updated on the PR.

Example error output:

info - 2024-05-30 13:29:27,496 -- The PR is happening in a forked repo. Using tokenless upload.
info - 2024-05-30 13:29:27,804 -- Process Commit creating complete
error - 2024-05-30 13:29:27,804 -- Commit creating failed: {"detail":"Tokenless has reached GitHub rate limit. Please upload using a token: https://docs.codecov.com/docs/adding-the-codecov-token. Expected available in 471 seconds."}

The workflow does not fail in this case, thus this is hard to spot if one is not looking for it.

For this reason, we should probably consider adding the corresponding token, which can only be done by users which have access to the GitHub and Codecov settings.

@stefan6419846 stefan6419846 added Meta nf-ci Non-functional change: Continuous Integration labels May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Meta nf-ci Non-functional change: Continuous Integration
Projects
None yet
Development

No branches or pull requests

1 participant