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

Chore: playwright sharding #13318

Merged
merged 2 commits into from
Apr 6, 2024
Merged

Chore: playwright sharding #13318

merged 2 commits into from
Apr 6, 2024

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Apr 6, 2024

split playwright tests into two jobs

@naltatis naltatis marked this pull request as ready for review April 6, 2024 17:18
@andig
Copy link
Member

andig commented Apr 6, 2024

Oh, du willst es wissen :). Kan rein?

@naltatis
Copy link
Member Author

naltatis commented Apr 6, 2024

Ja, kann rein. Aktuell macht das Setup (Build/Install) leider noch nen Großteil der Zeit aus. Daher machen mehr Shards gerade nur wenig Sinn. Aber zumindest Extralaufzeit durch neue Tests wird so besser verteilt.

Weitere Optimierung wäre parallele Testausführung (in einem Worker). Das geht aktuell leider noch nicht wegen State und festen Ports. Aber evtl. ne Baustelle für später.

@naltatis naltatis merged commit 5d963e2 into master Apr 6, 2024
7 checks passed
@naltatis naltatis deleted the chore/playwright-shards branch April 6, 2024 22:09
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.

2 participants