Skip to content

Split CGO unit tests into 5 shards - #54848

Merged
pelikhan merged 1 commit into
mainfrom
copilot/task-9919-1036865607-80b38e9d-d8ee-487e-a86c-63d3bfa98611
Aug 22, 2026
Merged

Split CGO unit tests into 5 shards#54848
pelikhan merged 1 commit into
mainfrom
copilot/task-9919-1036865607-80b38e9d-d8ee-487e-a86c-63d3bfa98611

Conversation

Copilot AI commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

The CGO workflow ran unit tests across 4 matrix shards. This splits them into 5 to lower per-shard wall-clock time.

Changes

  • .github/workflows/cgo.yml: replaced the D-L and M-R shards with D-G, H-M, and N-R. The A-C and S-Z and other shards are unchanged, so the patterns still partition the full test-name space with no overlap or gaps.
- name: A-C
  pattern: ^Test[A-C]
  shard: a-c
- name: D-G
  pattern: ^Test[D-G]
  shard: d-g
- name: H-M
  pattern: ^Test[H-M]
  shard: h-m
- name: N-R
  pattern: ^Test[N-R]
  shard: n-r
- name: S-Z and other
  pattern: ^Test([S-Z]|[^A-R])
  shard: s-z

No other edits were needed: coverage/result artifacts are named from matrix.shard, and the aggregation job downloads them via the test-result-cgo-unit-* glob.

Note: branch protection required-status-check names derived from Unit tests (<name>) will change and may need updating.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Split CGO unit tests into 5 buckets Split CGO unit tests into 5 shards Aug 22, 2026
Copilot AI requested a review from pelikhan August 22, 2026 15:37
@pelikhan
pelikhan marked this pull request as ready for review August 22, 2026 15:37
Copilot AI balanced review requested due to automatic review settings August 22, 2026 15:37
@pelikhan
pelikhan merged commit c82314e into main Aug 22, 2026
@pelikhan
pelikhan deleted the copilot/task-9919-1036865607-80b38e9d-d8ee-487e-a86c-63d3bfa98611 branch August 22, 2026 15:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Splits CGO unit tests into five balanced shards to reduce per-shard execution time while preserving complete test-name coverage.

Changes:

  • Replaces two broad shards with three narrower shards.
  • Retains existing artifact aggregation behavior.
Show a summary per file
File Description
.github/workflows/cgo.yml Updates the CGO unit-test matrix from four to five non-overlapping shards.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.5

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.

3 participants