[GH Request] Cannot merge PRs. CodeCov uploads failing despite retries

[jesperhodge]

Firm Name

2U

Urgency

High (< 3 days)

Problem/Request

It looks like pipelines are failing across different PRs and different repos (course-authoring MFE, learning MFE) due to inability to upload to CodeCov. I know in the past retrying solved this, but I retried 10 times now on my PR and it didn't fix this.
Here's some examples:

• feat: home studio search filters frontend-app-authoring#846 (problem was first observed here)
• https://github.com/openedx/frontend-app-course-authoring/actions/runs/8604487493/job/23601521237#step:6:38
• https://github.com/openedx/frontend-app-course-authoring/actions/runs/8617944055/job/23619770035#step:6:38
• https://github.com/openedx/frontend-app-learning/actions/runs/8597532108/job/23612808929?pr=1346
• https://github.com/openedx/frontend-app-learning/actions/runs/8608881905/job/23591952269?pr=1348

Could you look into this?

Reasoning

This blocks our ability to merge PRs

[openedx-workflow-automation]

Thank you for your report! @openedx/axim-oncall will triage within a business day. Simple requests usually take 2-3 business days to resolve; more complex requests could take longer.

[sarina]

Hi @jesperhodge - this is related to a change that Codecov made regarding per-repo tokens. @feanil and @brian-smith-tcril are going to take the lead on figuring out how we're going to address this. Apologies that this is happening.

[jesperhodge]

Thanks for looking into it!

[mariajgrimaldi]

Hi folks. Should we move to python-coverage-comment-action? Or are there more advantages to using the codecov service? There might be use cases that I'm missing though. A few weeks ago I picked up the work Ned started for the replacement: openedx/openedx-events#323

[feanil]

@mariajgrimaldi @brian-smith-tcril pointed out that codecov has support for JS which the alternative that we are looking at does not support, so we may need to do more research before we can move off of codecov.

[feanil]

@jesperhodge This should be fixed once openedx/frontend-app-authoring#950 and openedx/frontend-app-learning#1351 land, can you please review and feel free to merge if they look good to you.

[mariajgrimaldi]

@feanil: is that secret common to all openedx repositories? Or do we have to set it up manually? I tried updating codecov here but it didn't work:
https://github.com/openedx/openedx-events/actions/runs/8662684410/job/23755234691?pr=339

[feanil]

@mariajgrimaldi annoyingly it is per repository, I've added the scertes for openedx-events and filters as well to unblock that codecov update over there if you want to make the github actions updates on those repos.

[DawoudSheraz]

The job is failing for edx-val as well openedx/edx-val#496. I followed openedx/i18n-tools@939d4e3 but as @feanil mentioned, it seems the token would need to be configured for each repository separately.

[mariajgrimaldi]

It's also failing in edx-ora2:
https://github.com/openedx/edx-ora2/actions/runs/8690285459/job/23829862863#step:10:57 - PR openedx/edx-ora2#2203
https://github.com/openedx/edx-ora2/actions/runs/8690282326/job/23829851090#step:10:57 - PR openedx/edx-ora2#2204

These errors seem to be consistent with the other errors reported here.

[feanil]

edx-ora2 and edx-val have both been updated with the new tokens.

PR for edx-ora2 to update the workflow: openedx/edx-ora2#2206

[feanil]

The relevant PRs for the original request have merged so this is all set. If we find more repos with the same issue, please create new requests and we'll update the repos with tokens.