Chore(UI): test.slow() for CSVImportWithQuotesAndCommas to cover 1.13 AUT timing#27885
Chore(UI): test.slow() for CSVImportWithQuotesAndCommas to cover 1.13 AUT timing#27885
Conversation
…ver slow responses in 1.13 AUT runs The test consistently lands at the 60s test-timeout boundary on 1.13 AUT (image n56+) where the backend CSV-import flow takes ~50s end-to-end — matching the precedent set by #27782 for customProperties. test.slow() triples the timeout to 180s so transient AUT jitter no longer flakes the test. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
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! |
Code Review ✅ ApprovedIncreases the timeout for CSVImportWithQuotesAndCommas to 180s via test.slow() to accommodate timing jitter in 1.13 AUT runs. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
🟡 Playwright Results — all passed (14 flaky)✅ 3714 passed · ❌ 0 failed · 🟡 14 flaky · ⏭️ 79 skipped
🟡 14 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 |
Summary
test.slow()toCSVImportWithQuotesAndCommas.spec.tsso the single test in this spec gets a 180s timeout instead of the default 60s.b053fecfea), which did the same forcustomProperties.Why
On 1.13 AUT, the CSV-import end-to-end flow consistently lands at ~48–52s — right at the 60s test-timeout boundary — and times out under any extra cluster jitter. Walking the recent run history:
Same test on regular (non-AUT) 1.13 Playwright passes in ~24s, and the same test on main AUT (run
25235690014, May 1) passes in 37.4s. The slow path is specific to 1.13 AUT (legacy task/feed system + bounded-executor backpressure under upgrade-loaded sample data).test.slow()is the unblocker; the underlying backend latency is a separate investigation.Test plan
CSV Import with Commas and Quotes - All Entity Types › Create glossary with CSV, export it, create new glossary and import exported datapasses within the new 180s budget.🤖 Generated with Claude Code