Skip to content

Separate coverage job#7898

Open
maxtropets wants to merge 3 commits into
microsoft:mainfrom
maxtropets:f/cov-separate
Open

Separate coverage job#7898
maxtropets wants to merge 3 commits into
microsoft:mainfrom
maxtropets:f/cov-separate

Conversation

@maxtropets
Copy link
Copy Markdown
Collaborator

@maxtropets maxtropets commented May 16, 2026

As per #7897

CI: keep coverage on a separate, less frequent schedule

Companion to the ci.yml reorg that drops the coverage duplicate from PR
runs. Coverage is still useful, just not on every PR, so this adds a
dedicated `coverage.yml` workflow that runs on the same triggers as the
existing benchmarks workflow:

  - cron "0 8,20 * * 1-5"  (twice a day, Mon-Fri)
  - push to main
  - workflow_dispatch       (manual trigger)

The job is the old VMSS-C steps lifted verbatim: setup-ci, debug build
with -DCOVERAGE=ON, unit tests, full e2e bucket
(-LE benchmark|suite|unit), scripts/coverage.sh, and the same
`coverage-report-html` artifact upload + `$GITHUB_STEP_SUMMARY` write.
Runs on the same gha-vmss-d16av5-ci pool with the same caps, and uses
the same pinned action SHAs as the rest of the repo.

Net effect: PRs are ~32% faster to validate, coverage still gets
generated regularly and on demand, and the artifact name (and consumers
of it) are unchanged.

@maxtropets maxtropets self-assigned this May 16, 2026
@maxtropets maxtropets changed the title Separate coverage Separate coverage job May 16, 2026
@maxtropets maxtropets marked this pull request as ready for review May 16, 2026 12:20
@maxtropets maxtropets requested a review from a team as a code owner May 16, 2026 12:20
Copilot AI review requested due to automatic review settings May 16, 2026 12:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a dedicated GitHub Actions workflow for generating coverage outside PR CI, preserving the existing coverage build/test/report sequence on scheduled, main-branch push, and manual triggers.

Changes:

  • Adds coverage.yml with benchmark-like triggers.
  • Runs coverage build, unit/e2e tests, coverage report generation, summary writing, and artifact uploads.

Custom instructions used

  • .github/copilot-instructions.md

Comment thread .github/workflows/coverage.yml Outdated
./tests.sh --output-on-failure -L unit -j$(nproc --all)
shell: bash

- name: "End to end tests except partition"
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants