fix(e2e): reduce playwright workers for staging to avoid FAPI rate limiting#8132
fix(e2e): reduce playwright workers for staging to avoid FAPI rate limiting#8132jacekradko merged 1 commit intomainfrom
Conversation
…miting The generic suite runs 212 tests with 4 workers, each spinning up long-running apps against the same staging instance. This overwhelms the staging FAPI rate limits causing 'Too many requests' errors. Add E2E_WORKERS env var support to playwright config and set it to 2 for the staging workflow.
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe pull request introduces configuration for controlling the number of parallel workers in end-to-end tests. An environment variable 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📝 Coding Plan
Comment |
Summary
E2E_WORKERSenv var support to playwright config, falling back to existing behaviorE2E_WORKERS=2in the staging E2E workflow to reduce concurrent FAPI requestsgenericsuite runs 212 tests with 4 workers (50% of 8 vCPUs), each spawning long-running apps against the same staging instance — this causes "Too many requests" rate limiting on the staging FAPITest plan
E2E_WORKERSis respected by playwright config50%/70%)Summary by CodeRabbit