-
Notifications
You must be signed in to change notification settings - Fork 4.9k
feat: custom sharding weights #38624
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
Conversation
This comment has been minimized.
This comment has been minimized.
pavelfeldman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm module cli flag.
| - type: <[null]|[Object]> | ||
| - `total` <[int]> The total number of shards. | ||
| - `current` <[int]> The index of the shard to execute, one-based. | ||
| - `weights` ?<[Array]<[int]>> The shard weights. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not expose them for now.
docs/src/test-sharding-js.md
Outdated
| ## Rebalancing Shards | ||
|
|
||
| ``` | ||
| npx playwright test --shard=1/4,26/24/25/25 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say --shard=1/4 --shard-weights=26:24:25:25 to keep things readable
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Test results for "MCP"2727 passed, 116 skipped Merge workflow run. |
Test results for "tests 1"1 failed 1 flaky34396 passed, 689 skipped Merge workflow run. |
Inspired by #38598, alternative to #38588. Let's play around with it and see how it works for our own CI.