konveyor / tackle2-ui

Tackle (2nd generation) UI component.
Apache License 2.0
8 stars 43 forks source link

[Enhancement/CI] Fix or remove the codecov step of the repo CI #1820

Closed sjd78 closed 3 months ago

sjd78 commented 7 months ago

In the ci-repo.yaml workflow, the codecov action is used to upload test coverage info to Codecov service and have the report posted back to the PR. This process has not been working a quite some time, with the error often looking like this:

==> Uploader SHASUM verified (e3ea28a069fe174eb99ac2ffb523bfb55d5d29a7cfd93ef48cde12e9b552b492  codecov)
info - 2024-04-05 14:34:31,824 -- ci service found: github-actions
info - 2024-04-05 14:34:32,070 -- The PR is happening in a forked repo. Using tokenless upload.
info - 2024-04-05 14:34:32,227 -- Process Commit creating complete
error - 2024-04-05 14:34:32,227 -- Commit creating failed: {"detail":"Tokenless has reached GitHub rate limit. Please upload using a token: https://docs.codecov.com/docs/adding-the-codecov-token. Expected available in 67 seconds."}

There are 2 solutions:

  1. Remove the codecov action and stop using it all together.
  2. Setup a token for the uploads.

Option 2 would require:

Update 31-May-24: Other CI changes seems to have enabled the codecov plugin on PRs. However, it also looks like the reports on the PRs are based on reports submitted from the target branch (i.e. main). The codecov steps in the repo-level CI is disabled for normal merges. This probably should also change.

konveyor-ci-bot[bot] commented 7 months ago

This issue is currently awaiting triage. If contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance. The triage/accepted label can be added by org members.