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

Test Sharding #5075

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Test Sharding #5075

wants to merge 4 commits into from

Conversation

amerryma
Copy link

PR Checklist

Overview

This strategy uses file-based sharding. The entire set of test files is divided into different groups or "shards". Each shard contains a subset of the test files. The shards use a round robin approach using a simple modulus calculation. The way this works in the code is it simply filters out the tests by the shard value before proceeding to the rest of the mocha code.

A shard option has been added to the command line interface for specifying which shard to run and how many shards there are in total. This value is validated to ensure that bad values aren't inputted.

Sharding does not work when parallel mode is enabled.

Copy link

linux-foundation-easycla bot commented Jan 11, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@JoshuaKGoldberg
Copy link
Member

Marking as draft pending discussion in #4958.

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as draft March 4, 2024 17:51
@JoshuaKGoldberg JoshuaKGoldberg added the status: blocked Waiting for something else to be resolved label Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: blocked Waiting for something else to be resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 Feature: Test Sharding for CI Matrix Purposes
2 participants