Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shard unit tests #4666

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Shard unit tests #4666

wants to merge 22 commits into from

Conversation

imindich
Copy link
Contributor

@imindich imindich commented Jan 8, 2025

b/388125345

Run unit tests on linux using 10 shards. Each gtest binary is split up across all shards, and results are aggregated and reported in the test reporting job.

Using 10 shards, each shard takes <= 2 minutes 10 seconds to complete: https://github.com/youtube/cobalt/actions/runs/12681753920/job/35346881410?pr=4666

Running tests without sharding currently takes ~5 minutes 30 seconds: https://github.com/youtube/cobalt/actions/runs/12679757749/job/35341306163?pr=4667

If num_shards is not specified in a config json, the number of shards defaults to 1.

@imindich imindich force-pushed the experimental/sharding branch from cf71d1c to 01889e2 Compare January 8, 2025 20:41
@imindich imindich requested review from oxve, kaidokert and isarkis January 9, 2025 04:35
@imindich imindich marked this pull request as ready for review January 9, 2025 04:36
Copy link
Contributor

@oxve oxve left a comment

Choose a reason for hiding this comment

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

Nice!

.github/config/linux.json Outdated Show resolved Hide resolved
.github/config/linux.json Outdated Show resolved Hide resolved
.github/config/linux.json Outdated Show resolved Hide resolved
.github/workflows/main.yaml Outdated Show resolved Hide resolved
.github/workflows/main.yaml Show resolved Hide resolved
@imindich imindich force-pushed the experimental/sharding branch from d33bf2a to 9cbef37 Compare January 10, 2025 01:28
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