Repository navigation
fix(deps, ci): bump setup-sbt and playwright - #9100
Conversation
The ASF actions allowlist dropped the transitive carabiner-dev refs (install/ampel, download-and-verify, ampel-bootstrap) that setup-sbt v1.5.7 resolves through, so every job using it fails at "Set up job". v1.5.12 resolves through refs that are all on the current allowlist; its inputs are unchanged. Bump all 9 call sites.
If this limit is too low, the merge queue can enter a starvation loop where every job times out if runners are simply busy. Raising the limit stops this.
Automated Reviewer SuggestionsBased on the
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #9100 +/- ##
=========================================
Coverage 92.33% 92.33%
- Complexity 5045 5046 +1
=========================================
Files 1261 1261
Lines 54120 54120
Branches 6768 6768
=========================================
Hits 49974 49974
+ Misses 2486 2485 -1
- Partials 1660 1661 +1
☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
ubuntu-latest runners are moving to the Ubuntu 26.04 image, where
playwright 1.59.1 refuses to install Chromium ("Playwright does not
support chromium on ubuntu26.04-x64"), so the ubuntu frontend leg
fails before any test runs. 1.61.0 is the first release that knows
ubuntu26.04; bump to the current 1.64.1 and regenerate yarn.lock.
✅ No material benchmark regressions detected🟢 9 better · 🔴 0 worse · ⚪ 6 noise (<±5%) · 0 without baseline
Baseline detailsLatest main
Raw CSVconfig_idx,batch_size,schema_width,string_len,num_batches,total_ms,total_tuples,total_bytes,tuples_per_sec,mb_per_sec,lat_p50_us,lat_p95_us,lat_p99_us
0,10,10,64,20,292.87,200,128000,683,0.417,13998.52,20105.87,20105.87
1,100,10,64,20,1191.98,2000,1280000,1678,1.024,58496.18,69410.71,69410.71
2,1000,10,64,20,9766.63,20000,12800000,2048,1.250,486761.22,530636.08,530636.08 |
|
Original timeout PR is merged, so we don't need this PR anymore. |
|
If everything works out, we'll wake up tomorrow with a clean queue and this merged 🫠 |
What changes were proposed in this PR?
Nothing can merge into
mainright now. Three CI faults block it, and each fix needs the others to land first.carabiner-dev/actions/install/*refs thatsbt/setup-sbtv1.5.7 reaches, so every job that sets up sbt (amber,amber-integration,platform,platform-integration,Bench) has died at Set up job since 2026-10-19. This moves all 9 call sites to v1.5.12, whose whole chain is allowed. The action's inputs did not change.ubuntu-latestnow resolves to the Ubuntu 26.04 image, where Playwright 1.59.1 refuses to install Chromium ("Playwright does not support chromium on ubuntu26.04-x64"), so the ubuntu frontend leg fails before any test runs. This bumpsplaywrightto 1.64.1, which supports 26.04.check_response_timeout_minutes, which is 45. ASF's shared runners are backed up well past that: in the last merge group, Validate PR title, a 2-second job, waited 110 minutes for a runner. This raises the limit to 360 minutes, as DataFusion did in ci: raise the merge queue status check timeout to 360 minutes datafusion#26293.Required Checksstays red without both version bumps, and no PR gets through the queue without the higher limit, so all three go in one PR.Any related issues, documentation, discussions?
Closes #9095.
How was this PR tested?
Every changed file parses:
Walking the
uses:chain undersbt/setup-sbt@ecad189(v1.5.12) reachesampel/verify@32587e8,install/ampel@d6d91c5,install/download-and-verify@2fec8bdandinstall/ampel-bootstrap@16009dc, and all four are inapache/infrastructure-actions'approved_patterns.ymlas of 2026-10-23. Under v1.5.7,install/ampel@2fec8bd,install/download-and-verify@16009dcandinstall/ampel-bootstrap@619a474are not.The rest is CI. The labeler applies
cifor.github/**, which runs every stack that sets up sbt, the frontend legs, andBench. On this PR's previous head every job passed except the ubuntu frontend leg, which landed on ubuntu-26.04 and failed at the Playwright install. The new timeout only shows once this is merged.Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Opus 5.5)
🤖 Generated with Claude Code