fix(ci): limit Playwright concurrency - #731
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 798cf81ab5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const oauthTasks = buildOAuthTasksForSetGroup(userSetKeys); | ||
| // Apply one global concurrency limit. Running a separate batch loop per set | ||
| // group allowed up to 20 OAuth browser contexts to start at once in CI. | ||
| const tokenUpdates = await collectTokensInBatches(browser, oauthTasks); |
There was a problem hiding this comment.
Update the Playwright concurrency documentation
This replaces the five parallel OAuth set groups with one globally batched queue, while playwright/README.md:203-209 still describes the removed grouping behavior; the README also says workers always scale to USER_SETS.length at lines 216 and 261 despite the new four-worker CI cap. Update those standing instructions in this change so operators do not diagnose or size E2E runs using obsolete concurrency behavior.
AGENTS.md reference: AGENTS.md:L68-L68
Useful? React with 👍 / 👎.
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
32154b0 to
f9af52d
Compare
COMPLETES #< INSERT LINK TO ISSUE >
This pull request addresses
< DESCRIBE THE CONTEXT OF THE ISSUE >
by making the following changes
< DESCRIBE YOUR CHANGES >
Change Type
The following scenarios were tested
< ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >
The GAI Coding Policy And Copyright Annotation Best Practices
Checklist before merging
Make sure to have followed the contributing guidelines before submitting.