[cov,ci] Collect test coverage on CI#29297
Merged
Merged
Conversation
4fd1763 to
6c2c9ca
Compare
98e4ea9 to
6c81581
Compare
ef304ce to
0c8abcd
Compare
timothytrippel
approved these changes
Feb 19, 2026
Contributor
|
We should wait for dependent PRs to merge first before rebasing and rerunning tests. |
Change-Id: If7b15974a6f1c5a3786ab7b3075148f57e08e201 Signed-off-by: Yi-Hsuan Deng <yhdeng@google.com>
Change-Id: I9c33829bc72444078496993a45d224b76be91800 Signed-off-by: Yi-Hsuan Deng <yhdeng@google.com>
Change-Id: I276075ba17b709d0643e183f45f15b823a29a8d5 Signed-off-by: Yi-Hsuan Deng <yhdeng@google.com>
ee6befb to
3688165
Compare
This commit introduces code coverage collection and reporting to the CI pipeline. Change-Id: Iffdf510d5cf714d6364e335fa494ea36ac4242b1 Signed-off-by: Yi-Hsuan Deng <yhdeng@google.com>
This commit adds the `coverage` tag to the `fpga_params` of several smoke and functional tests. This allows these tests to be automatically selected by the CI pipeline to validate the coverage collection setup on FPGA targets. Change-Id: I86d8e28ee885cf683d961dd5e1eaf44a100e8ad2 Signed-off-by: Yi-Hsuan Deng <yhdeng@google.com>
timothytrippel
approved these changes
Feb 23, 2026
a733dbb
into
lowRISC:earlgrey_1.0.0
66 of 72 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR introduces code coverage collection for CI.
Reports will be available as GitHub Action artifacts and also published to GCS bucket.
Currently, several FPGA tests fail when running in coverage mode. To validate the CI setup, we have tagged a few passing tests with the
coveragetag and only run those tagged tests. Thesecoveragetags are temporary and will be removed later once all test failures are resolved or appropriately marked with "coverage_broken" tags.To maximize collected coverage, the workflow utilizes the
--keep-goingflag, allowing it to proceed even if individual tests fail.We expect all unit tests and OTBN tests to pass with the following two dependent PRs:
To showcase the post-submit coverage workflow, a test run is performed on previous version with PR trigger in the ci-coverage.yml.
Test run result: https://github.com/lowRISC/opentitan/actions/runs/22249013778