Skip to content

fix(ci): raise merge queue timeout - #9096

Merged
Humbedooh merged 1 commit into
apache:mainfrom
parshimers:imaxon/mqinc
Oct 23, 2026
Merged

Humbedooh merged 1 commit into
apache:mainfrom
parshimers:imaxon/mqinc

Conversation

@parshimers

Copy link
Copy Markdown
Member

What changes were proposed in this PR?

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.

Any related issues, documentation, discussions?

Closes #9095 and https://issues.apache.org/jira/browse/INFRA-28569

How was this PR tested?

It wasn't; it's specific to ASF infra. I got the idea from a helpful hint from Andrew Lamb of DataFusion.

Was this PR authored or co-authored using generative AI tooling?

No

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.
@github-actions github-actions Bot added the ci changes related to CI label Oct 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

  • Contributors with relevant context: @mengw15
    You can notify them by mentioning @mengw15 in a comment.

@aglinxinyuan aglinxinyuan left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
We need to find a way to merge #9064 first.

@codecov-commenter

codecov-commenter commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.33%. Comparing base (3ba45ee) to head (8158310).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #9096   +/-   ##
=========================================
  Coverage     92.33%   92.33%           
  Complexity     5045     5045           
=========================================
  Files          1261     1261           
  Lines         54120    54120           
  Branches       6768     6768           
=========================================
  Hits          49973    49973           
  Misses         2486     2486           
  Partials       1661     1661           
Flag Coverage Δ *Carryforward flag
access-control-service 77.38% <ø> (ø) Carriedforward from 3ba45ee
agent-service 99.16% <ø> (ø)
amber 87.31% <ø> (ø) Carriedforward from 3ba45ee
computing-unit-managing-service 60.48% <ø> (ø) Carriedforward from 3ba45ee
config-service 87.37% <ø> (ø) Carriedforward from 3ba45ee
file-service 82.00% <ø> (ø) Carriedforward from 3ba45ee
frontend 96.55% <ø> (ø)
notebook-migration-service 83.73% <ø> (ø) Carriedforward from 3ba45ee
pyamber 98.57% <ø> (ø)
workflow-compiling-service 72.43% <ø> (ø) Carriedforward from 3ba45ee

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aglinxinyuan

aglinxinyuan commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

@parshimers do you have permission to force merge this one? I don't think we can merge any PR without this.

@aglinxinyuan

Copy link
Copy Markdown
Contributor

@parshimers I combined this with #9064 in #9100 — your commit is cherry-picked as-is (authorship kept), since this PR can't pass the Scala checks without the setup-sbt bump. #9100's own merge group still runs under the 45-minute limit until the .asf.yaml change is on main, so we still need Infra to raise the timeout on the ruleset directly (INFRA-28569) or merge #9100 for us.

@parshimers

Copy link
Copy Markdown
Member Author

Sounds good. I do think we have to do something to force it in assuming the runner starvation stays like this for any longer. I am talking with infra about it. Either I might be given permission to, or they may do it for us.

@Humbedooh
Humbedooh merged commit 2407d79 into apache:main Oct 23, 2026
24 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci changes related to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Raise the merge queue check response timeout

4 participants