Skip to content

Split E2E tests into three Playwright shards - #8298

Closed
isaacroldan wants to merge 1 commit into
graphite-base/8298from
isaac/e2e-three-shards
Closed

Split E2E tests into three Playwright shards#8298
isaacroldan wants to merge 1 commit into
graphite-base/8298from
isaac/e2e-three-shards

Conversation

@isaacroldan

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

Fixes #0000

WHAT is this pull request doing?

How to test your changes?

Post-release steps

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

Shard 1 consistently takes 1.5-2.5x as long as shard 2 (6.5-12m vs
4-5m). A third shard rebalances the file distribution and shortens the
E2E chain that the cleanup job waits on.

isaacroldan commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Aug 11, 2026
@isaacroldan
isaacroldan changed the base branch from isaac/shard-macos-unit-tests to graphite-base/8298 August 11, 2026 10:54
@isaacroldan

Copy link
Copy Markdown
Contributor Author

Folded into #8297.

@isaacroldan
isaacroldan deleted the isaac/e2e-three-shards branch August 11, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant