[tahoe-lafs-trac-stream] [Tahoe-LAFS] #3568: codecov results for fork PRs are delivered before all coverage is submitted, claim massive loss of coverage

Tahoe-LAFS trac at tahoe-lafs.org
Thu Dec 17 17:04:29 UTC 2020


#3568: codecov results for fork PRs are delivered before all coverage is
submitted, claim massive loss of coverage
---------------------+---------------------------
 Reporter:  exarkun  |          Owner:
     Type:  defect   |         Status:  new
 Priority:  normal   |      Milestone:  undecided
Component:  unknown  |        Version:  n/a
 Keywords:           |  Launchpad Bug:
---------------------+---------------------------
 If a PR is submitted from a GitHub fork of tahoe-lafs then codecov makes
 its report comment very early - before most builds have finished - and the
 report is ugly.  It often claims something like a 30% loss of coverage
 spread across the whole project.

 It seems like codecov is generating a report from the coverage data
 uploaded by the *first* job to finish.  It typically updates its comment
 later when more data comes in but the first report is obnoxious,
 misleading noise (it will often be emailed out in addition to showing up
 on the PR page).

 Also, the codecov status item is not always updated.  For example, right
 now https://github.com/tahoe-lafs/tahoe-
 lafs/pull/939/checks?check_run_id=1571210288 shows 63% coverage even
 though the report on codecov itself https://codecov.io/gh/tahoe-lafs
 /tahoe-lafs/pull/939 says 92% coverage.  These pages are totally out of my
 control, of course, so who knows what they'll look like after any amount
 of time passes.

--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3568>
Tahoe-LAFS <https://Tahoe-LAFS.org>
secure decentralized storage


More information about the tahoe-lafs-trac-stream mailing list