docs: drop stale Playwright wrapper/job-clock minute values from README - #32772
Conversation
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
✅ Playwright Results — workflow succeededValidated commit ✅ 556 passed · ❌ 0 failed · 🟡 1 flaky · ⏭️ 0 skipped · 🧰 0 lifecycle flaky PerformanceBlocking targets: ✅ met · Optimization targets: 🟡 in progress Shard-job maxima below are not the full workflow wall time; the linked run includes build, fixture, planning, and reporting. 🕒 Full workflow signal wall (to summary) 32m 39s ⏱️ Max setup 4m 17s · max shard execution 13m 58s · max shard-job elapsed before upload 17m 20s · reporting 5s 🌐 234.49 requests/attempt · 2.82 app boots/UI scenario · 27.09% common-shard skew Optimization targets still in progress:
🟡 1 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
Describe your changes:
Removed the hardcoded "25-minute
timeoutwrapper" and "35-minuteplaywright-cijob clock" figures from.github/playwright/README.md,replacing them with a pointer to
playwright-e2e-reusable.yml. Thesefigures were stale: the wrapper and job clock have since moved to 60m
and 75m respectively, so the README described a 25-minute ceiling that
no longer constrains the shard allocation budget.
A developer or agent reasoning about shard-budget headroom against the
README would believe the 21-minute allocation budget must stay under a
25-minute wrapper, when the real wrapper is 60m — leading to wrong
conclusions about how much headroom exists and whether a budget bump is
safe.
Introduced by commit 8fb5397 (@pmbrull, #32611)
Type of change:
High-level design:
N/A — small change.
Tests:
Not applicable — documentation-only change with no code or behavior
impact.
UI screen recording / screenshots:
Not applicable.
Checklist:
Doc Drift PRs can be configured here.