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

fix(codecov): increase codecov upload attempt #1186

Merged
merged 2 commits into from
Oct 2, 2023

Conversation

gfanton
Copy link
Member

@gfanton gfanton commented Oct 2, 2023

multiple workflows were failing randomly with Unable to locate build via Github Actions API error like:

based on this issue#3954 from community.codecov.com adding codecov upload token should resolve most issues. However, in some rare instances where API limits are still reached, a re-upload attempt should be made.

This PR introduces a retry action to allow codecov to reattempt the upload if it fails the first time.

try to avoid `Unable to locate build via Github Actions API` by increasing
codecov upload attempt
based on this issue:
https://community.codecov.com/t/upload-issues-unable-to-locate-build-via-github-actions-api/3954

Signed-off-by: gfanton <[email protected]>
@gfanton gfanton requested a review from a team as a code owner October 2, 2023 14:32
@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (28455d8) 47.00% compared to head (13747d4) 47.02%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1186      +/-   ##
==========================================
+ Coverage   47.00%   47.02%   +0.02%     
==========================================
  Files         365      365              
  Lines       61156    61156              
==========================================
+ Hits        28748    28761      +13     
+ Misses      30052    30041      -11     
+ Partials     2356     2354       -2     
Flag Coverage Δ
gno.land-_test.gnokey 0.00% <ø> (ø)
gno.land-_test.gnoland 88.14% <ø> (ø)
gno.land-_test.pkgs 27.88% <ø> (ø)
gnovm-_test.cmd 45.89% <ø> (ø)
gnovm-_test.gnolang.native 63.09% <ø> (ø)
gnovm-_test.gnolang.other 16.63% <ø> (ø)
gnovm-_test.gnolang.pkg0 17.98% <ø> (ø)
gnovm-_test.gnolang.pkg1 8.21% <ø> (ø)
gnovm-_test.gnolang.pkg2 9.87% <ø> (ø)
gnovm-_test.gnolang.realm 41.68% <ø> (ø)
gnovm-_test.gnolang.stdlibs 53.53% <ø> (ø)
gnovm-_test.pkg 25.94% <ø> (ø)
tm2-_test.flappy ∅ <ø> (∅)
tm2-_test.pkg.amino 58.32% <ø> (ø)
tm2-_test.pkg.bft 63.65% <ø> (+0.01%) ⬆️
tm2-_test.pkg.others 59.26% <ø> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@moul moul merged commit 39ab2da into gnolang:master Oct 2, 2023
173 checks passed
moul pushed a commit that referenced this pull request Oct 3, 2023
It appears that the retry system introduced in #1186 was not sufficient,
as we still hit API limits when running multiple checks concurrently.
This PR merges all coverage files into a single upload at the of each
testing workflows. As a result, we now have 3 uploads instead of 16,
which should drastically reduce the number of Codecov upload failures.

Note: It still appears to fail randomly, but we might need to wait some
time until our API rate decreases. I believe it's still preferable to
have 3 uploads rather than 16, which seemed to overwhelm Codecov.

---------

Signed-off-by: gfanton <[email protected]>
gfanton added a commit to gfanton/gno that referenced this pull request Nov 9, 2023
multiple workflows were failing randomly with `Unable to locate build
via Github Actions API` error like:
-
https://github.com/gnolang/gno/actions/runs/6349517658/job/17247838334?pr=1179
-
https://github.com/gnolang/gno/actions/runs/6378366330/job/17308780300?pr=1117
- ...

based on this
[issue#3954](https://community.codecov.com/t/upload-issues-unable-to-locate-build-via-github-actions-api/3954)
from `community.codecov.com` adding `codecov upload token` should
resolve most issues. However, in some rare instances where API limits
are still reached, a re-upload attempt should be made.

This PR introduces a retry action to allow codecov to reattempt the
upload if it fails the first time.

---------

Signed-off-by: gfanton <[email protected]>
gfanton added a commit to gfanton/gno that referenced this pull request Nov 9, 2023
…#1190)

It appears that the retry system introduced in gnolang#1186 was not sufficient,
as we still hit API limits when running multiple checks concurrently.
This PR merges all coverage files into a single upload at the of each
testing workflows. As a result, we now have 3 uploads instead of 16,
which should drastically reduce the number of Codecov upload failures.

Note: It still appears to fail randomly, but we might need to wait some
time until our API rate decreases. I believe it's still preferable to
have 3 uploads rather than 16, which seemed to overwhelm Codecov.

---------

Signed-off-by: gfanton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants