-
Notifications
You must be signed in to change notification settings - Fork 213
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
v4 fails to upload: "Failed to properly create commit" #1274
Comments
Same here, Codecov still states this is not required for public projects, but bumping to |
Same here |
I'm getting this as well. |
I believe this is due to this incident. @dereuromark are you still getting the issue? |
Yes, reran the workflow just now, same thing |
Same issue here, using codecov v4 and a PR from a fork: https://github.com/lab-cosmo/metatensor/actions/runs/7817731428/job/21326326361 EDIT: seems fixed after a restart, might have been transient. Here is the old log for reference:
|
@dereuromark you need to add the Codecov token
Instructions to do so are here |
Getting the same server error on all my repos |
We never had to enter any tokens for the last decade. Note that v3 is working just fine |
Remember that secrets are not passed to workflow triggered by PR from fork. |
Both PR and main branch CI worked so far with v3 afaik |
Yes. I point a problem with v4. Where token will not help for typical OSS. |
Hi @Czaki @dereuromark something I'd like to clarify here On requiring a token On impacting contribution flows We're aware that for open source contributors, the fork->commit-> PR workflow (which is by far the most common) would be impacted if we enforced token usage aggresively, so currently we DO NOT require forked repos to setup a token. You can read more on our blog (look for the section called On adding a single token for multiple repos This usecase is served by using Codecov's GLOBAL UPLOAD TOKEN. Here's how to set that up (docs) I hope this helps, please don't hesitate to reach out if you have challenges with this. |
@rohan-at-sentry I try to play with on:
workflow_run:
workflows: [Tests]
types:
- completed But it ends with codecov failing to determine PR/commit. If you could fix codecov action to work in workflow triggered by |
Is there a way to have a global token for a user, and not just an org? |
@Czaki I suspect that the token is not being set correctly in that example you shared. Is it possible for you to try the following and see if it works
|
Not currently... Can you describe your usecase? Maybe there's a way we can already help now |
In my case I have 30+ repos in my github user account (personal), is there a way to set up the env token once across all of them? I only saw sth like that for an org. Sorry to chime in. |
@rohan-at-sentry Why use env variable instead of pass in as a parameter? |
Oh I think I misunderstood the original question. @dereuromark - I believe if you navigate to https://app.codecov.io/account/gh/<your_GH_username> , and click the @gdalle if your usecase is similar to above, then you can try this as well |
Well, I generated a token there, not setting one
in all 30+ repos of my github account and CI |
@rohan-at-sentry After adding verbose, the upload passed. But I'm not sure if it is a proper solution. |
@Czaki if you are referring to 4DNucleome/PartSeg#1066 when you say adding verbose fixed it, I don't think it was the verbose option that made the upload run. I think it was the fact that it was your user (or pre-commit ci app) triggered the workflow that happen to have the verbose option in it. Apparently dependabot can't access repo secrets (see here) so the action didn't have the token to upload when dependabot tried to run it. Because that PR comes from the same repo, it needs a token (and so it failed). On the other hand, when your user (or pre-commit ci app) triggered the action would have access to the token, and the upload would then run. |
Apart from my personal account, which cannot be fixed for now, I also tried the org approach
The last message sounds like it actually still failed? |
it looks working now. I will provide it more time for test. |
With codecov action v3 or v4, even with _official_ tokenless support for open source repos, our CI sometimes fails because codecov hits a rate limit on github API. See issues with codecov action: * codecov/codecov-action#1292 * codecov/codecov-action#1274 The safest way to securely send our test coverage without failures is to configure the token as a github action secret and use that token to connect to codecov. More explanations can be found: * [here](https://about.codecov.io/blog/january-product-update-updating-the-codecov-ci-uploaders-to-the-codecov-cli/) Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
With codecov action v3 or v4, even with _official_ tokenless support for open source repos, our CI sometimes fails because codecov hits a rate limit on github API. See issues with codecov action: * codecov/codecov-action#1292 * codecov/codecov-action#1274 The safest way to securely send our test coverage without failures is to configure the token as a github action secret and use that token to connect to codecov. More explanations can be found: * [here](https://about.codecov.io/blog/january-product-update-updating-the-codecov-ci-uploaders-to-the-codecov-cli/) Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@jmgrady can you confirm this is working for you now? For everyone else, please note that this thread is getting a little bit crazy. There are a few root causes that we have since patched. If you are still experiencing this issue, please open a new issue so that we can track. |
@thomasrockhu-codecov I am still seeing the problem. I am using
The failure message matches the one in my original post. |
PR from dependabot causes an error, so comment this out until the problem is resolved. See codecov/codecov-action#1274
Having reports uploaded is not impotant, especially PR for dependency update. See following about bug for this option: codecov/codecov-action#1274
* Fix CodeQL check As of now, we need to manually setup golang for 1.22. github/codeql#15647 (comment) * ci: ignore Codecov token not found error PR from dependabot causes an error, so comment this out until the problem is resolved. See codecov/codecov-action#1274 * Remove fail_ci_if_error option Having reports uploaded is not impotant, especially PR for dependency update. See following about bug for this option: codecov/codecov-action#1274
codecov/codecov-action#1274 Signed-off-by: Stephen Whitlock <stephen@jshwisolutions.com>
## Overview Fixed codecov action to inherit secrets, required for uploading coverage reports. CI coverage was broken after #1519 due to codecov/codecov-action#1274 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Chores** - Enhanced security and management of sensitive information in the CI workflow by allowing the test job to inherit secrets from the parent workflow. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
added uploaded action token: ${{ secrets.CODECOV_TOKEN }} Update test-coverage.yaml removed # token: ${{ secrets.CODECOV_TOKEN }} to confirm that it is not useful Update test-coverage.yaml https://github.com/codecov/example-python/blob/main/.github/workflows/ci.yml https://app.codecov.io/gh/JohnGavin/hellow/new Update test-coverage.yaml from r-lib/covr#430 Update test-coverage.yaml v3 -> v4 uses: actions/upload-artifact@v4 Update test-coverage.yaml v4->v3 second try Update test-coverage.yaml trying again Update test-coverage.yaml v4 -> v3 uses: actions/upload-artifact@v3 Update test-coverage.yaml v4 to v3 codecov/codecov-action#1274 Add GitHub links to DESCRIPTION Initial commit
Use
|
* Address numpy 2.0 warning: numpy/numpy#26620 * Address issue with codecov uploading: codecov/codecov-action#1274
The codecov action GitHub failed because of "Error: Codecov: Failed to properly create commit: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1" One possible solution for this according to https://docs.codecov.com/docs/adding-the-codecov-token#github-actions is to add the codecov token found in Codecov to the GitHub repo. However, handwriterRF is a CSAFE-ISU repo, not one of my personal repos. Because CSAFE-ISU are not a personal repos, Codecov would charge money. Another possible solution is codecov/codecov-action#1274
After updating
codecov/codecov-action
to v4 in our GitHub workflows, the coverage reports fail to upload. The upload fails because of a connection error or timeout. The GitHub Action output is:The text was updated successfully, but these errors were encountered: