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

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 https://github.com/doctrine/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 https://github.com/doctrine/orm/pull/8335

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 https://github.com/tremor-rs/tremor-runtime/pull/576

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: https://github.com/doctrine/DoctrineMigrationsBundle/pull/379

#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: https://github.com/greg0ire/DoctrineFixturesBundle/pull/1

#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! https://github.com/doctrine/DoctrineMigrationsBundle/pull/374
Same here: https://github.com/doctrine/orm/pull/8348 !

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! https://github.com/tremor-rs/tremor-runtime/pull/619 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?