-
Notifications
You must be signed in to change notification settings - Fork 10
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
Bug Report: Successful codecovcli upload-process
failing to process by codecov which introduces unnecessary friction and lack of observability
#569
Comments
Hi @chriscerk Can you please share the full SHA? |
@drazisil-codecov |
@chriscerk can you open a ticket and share the full CLI output from your CI? Preferably with verbose enabled? Something is not happening correctly (don't think it's anything you are doing wrong), I but I don't know what...yet. You can mention this thread, if you want. I'll get it :) |
Same here, I can provide some examples as they're public repositories I use Codecov GitHub Action to upload the reports there. And then have some PR checks that are mandatory to merge to the An example from today: An example from another repository, 2 days ago: In both cases I forced a CI/CD run again by ammending the commit to change the date of the commit + force-pushing and it worked. |
@adrian-codecov Here are a few examples of uploads failing because |
Thanks, @davidlj95 ! Very helpful to be able to confirm that the CLI is calling all the steps in order. Definitely a server-side race condition. :-/ |
#485 is related |
Today happened again. Here are the details: An example from today: Let me know if I'm spamming and I won't add moar comments 😛 Just providing info to confirm the issue stil exists + provide details to diagnose the issue |
Just bumping this thread so it hopefully gets attention. and to re-iterate from the ticket:
We've had a lot of frustration from our teams and I don't have the time to track down every single instance of this. |
@souredoutlook , can we get some eyes on this, since I'm no longer in the support department? |
@drazisil-codecov / @souredoutlook we're seeing more issues with codecov processing today again and it has been extremely disruptive. We'd appreciate some eyes on this. One example of many:
yet |
Also if code coverage doesn't process within a reasonable timeframe - it would much less disruptive for our teams to have a codecov yaml config setting to handle these processing delays - similar to For example, This is in addition to accurately tracking the status as mentioned above a few times. |
This is happening again... both yesterday and today. Please review the above recommendations so that internal incidents with codecov don't create unnecessary friction. |
codecovcli upload-process
failing to process by codecovcodecovcli upload-process
failing to process by codecov introducing unnecessary friction and lack of observability
codecovcli upload-process
failing to process by codecov introducing unnecessary friction and lack of observabilitycodecovcli upload-process
failing to process by codecov which introduces unnecessary friction and lack of observability
@chriscerk - I'm the PM on Codecov. We're working on ways to improve the commits page to show processing status of individual reports - and surface errors more visibly on the PR comment and UI. You can follow along, and leave feedback here -- codecov/engineering-team#3229 That said, the point about |
Describe the bug
A clear and concise description of what the bug is.
Environment
To Reproduce
Steps to reproduce the behavior:
codecovcli --verbose upload-process
Expected behavior
A clear and concise description of what you expected to happen.
Uploading code coverage to work.
⭐ If you have metrics on:
That would be interesting to review since we see "Currently no uploads" which according to the codecov cli should not be true.
Showing the status in the UI would be great, eg if it is queued vs. processing vs. completed
update: from comment below
Also if code coverage doesn't process within a reasonable timeframe - it would much less disruptive for our teams to have a codecov yaml config setting to handle these processing delays - similar to informational: true.
For example,
informational_on_processing_delay: true.
This is in addition to accurately tracking the status as mentioned above a few times.
Screenshots
n/a
Additional context
This has been working fine for awhile, but since the past week this has been extremely flaky and blocking many engineering teams.
The text was updated successfully, but these errors were encountered: