proxy icon indicating copy to clipboard operation
proxy copied to clipboard

Add Ci workflow to compute code coverage

Open nezdolik opened this issue 4 months ago • 7 comments

Workflow computes code coverage for cilium-proxy source files using custom report generator. It relies heavily on bazel remote caching stored in gcs bucket and all reports (per PR) are uploaded to another gcs bucket and stored for 7 days. Sample report: https://storage.googleapis.com/cilium-proxy-coverage-reports/c08c59a/coverage/html/index.html

nezdolik avatar Dec 03 '25 15:12 nezdolik

looks like secret is not propagated to my fork workflow, synced offline with @sayboras on how to fix it

nezdolik avatar Dec 03 '25 15:12 nezdolik

looks like secret is not propagated to my fork workflow

I think with the pull request target setup, the secrets will not be visible in forked repo. Once you clean up the commit, I can create another temp branch to test it out.

sayboras avatar Dec 03 '25 22:12 sayboras

From offline discussion with @jrajahalme this feedback needs to be addressed:

Not sure I understand how this works, but I would expect that building test deps (//tests/...) before running the coverage build would not count the coverage on the pre-build test code, or will it be built again due to different compiler flags?

nezdolik avatar Dec 09 '25 14:12 nezdolik

@sayboras are you able to create a test branch for this PR?

nezdolik avatar Dec 11 '25 10:12 nezdolik

are you able to create a test branch for this PR?

Sure, however, check format seems failing.

sayboras avatar Dec 11 '25 11:12 sayboras

are you able to create a test branch for this PR?

Sure, however, check format seems failing.

let me fix it

nezdolik avatar Dec 11 '25 11:12 nezdolik

@sayboras format is fixed

nezdolik avatar Dec 11 '25 12:12 nezdolik