Codecov commit status check missing for PR when using codecov-action

Support Bug Fixes
1

Description

This is a clone of github issue

We noticed the commit status check started to disappear once we migrate from Travis CI to Github Action (using codecov/codecov-action for uploading coverage to codecov)

Repository

private

CI/CD

CI: Github Action
CD: N.A.

Uploader

Commit SHAs

e7443e6

Codecov Output

Steps to Reproduce

  1. Create a PR
  2. Wait for all commit status checks to show up

Expected behavior:
codecov commit status check shows up

Actual behavior:
codecov commit status does not show up

Flakiness?
We can consistently reproduce this on the same repo(s)

Additional Information

codecov status check is working fine for master commit

1 Like
2

Hi @sfdye when did you notice this change, and would you be able to provide the full commit SHA?

3

About a few weeks ago right when we migrate from Travis CI to Github action.

Full commit sha shared via DM.

4

Hello, I think we are experiencing the very same issue at GitHub - doctrine/orm: Doctrine Object Relational Mapper (ORM) . You marked the previous answer as a solution, did you somehow fix it?

5

@greg0ire, would you mind sharing a SHA here?

6

Sure! For instance you have 9e25a32d2c83091a447fa31c1e6cab3f8c81b259, from Update second-level-cache.rst by RomaVan · Pull Request #8335 · doctrine/orm · GitHub

Note that it also fails for doctrine/dbal (for instance 813a042a7f200324b4168a3b018f4e5fe7dfc4b5), so this isn’t just about projects newly migrated to Codecov. It used to work on the DBAL at some point.

7

We’re having the same issue, the pull request where we first noticed it is Add docker image and action for training docker image by darach · Pull Request #576 · tremor-rs/tremor-runtime · GitHub

It might be related but the entire tremor-rs org is no longer visible in codecov as a top level org either :

8

Hi @greg0ire and @Licenser, thanks for this. I think I have an idea where the issue is, I’ll be working with the product team to fix.

2 Likes
10

Hi @greg0ire and @Licenser, I want to check something, has it been working for you in the past 12 hours?

11

Hi Tom, thanks for the quick look into it. We had some runs that worked in the past 12 hours, but about 9h ago (~23:00 UTC yesterday) a run failed to report results again, this morning re-running the same job added the reports.

12

@Licenser, do you happen to have a commit SHA for the one that didn’t work?

13

@tom in my case it didn’t work for this other doctrine repository: Quote path placeholder for use with PCRE by greg0ire · Pull Request #379 · doctrine/DoctrineMigrationsBundle · GitHub

14

Another datapoint: you cannot see Github Actions on a PR unless there is already one it seems. What I usually do is I create a similar PR, but from my fork to my fork. It works on this repository: Maintenance work by greg0ire · Pull Request #1 · greg0ire/DoctrineFixturesBundle · GitHub

15

Got it, this is really helpful @greg0ire. I’ll keep digging in.

16

@greg0ire, I made a fix here, would you be able to see if this is still happening?

17

I just rebased this PR and it seems to work! Prevent deprecation with Symfony 5.1 and allow private services by smoench · Pull Request #374 · doctrine/DoctrineMigrationsBundle · GitHub
Same here: [GH-8229] Prevent Illegal Inheritance Override by beberlei · Pull Request #8348 · doctrine/orm · GitHub !

Thanks a lot!

1 Like
18

Hi Tom,
sorry for the late reply. Things got in the way :smiley: I hope you can sympathize.

I got a few commits that cause problems:

19

Oof @Licenser, would you be able to run the action with the verbose flag on?

20

Heya thanks for getting back to us :slight_smile: done! Add verbose flag to codecov action for debugging by Licenser · Pull Request #619 · tremor-rs/tremor-runtime · GitHub lets see what happens

21

@Licenser, thanks! I pulled up a few of the past few commits and am seeing statuses come through now. Is that the case or are you still seeing errant commits without statuses?