Skip to content

ci: raise the merge queue status check timeout to 360 minutes - #2638

Merged
alamb merged 1 commit into
apache:mainfrom
alamb:alamb/merge-queue-timeout
Oct 14, 2026
Merged

alamb merged 1 commit into
apache:mainfrom
alamb:alamb/merge-queue-timeout

Conversation

@alamb

@alamb alamb commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Rationale for this change

Same change as apache/datafusion#26293 (see apache/datafusion#26292 for background).

The merge queue evicts PRs whose checks have not reported within 60 minutes (the GitHub default), even when nothing failed and the remaining jobs are only waiting for a runner.

Each eviction discards CI that already passed, requires someone to re-queue the PR by hand, and reruns everything from scratch.

This is now happening in this repo too, for example on #2626: #2626 (comment) All Apache projects share a GitHub runner queue, and it has been saturated for the last few days.

What changes are included in this PR?

Adds a Merge Queue ruleset to .asf.yaml that sets check_response_timeout_minutes: 360.

Here are the current settings

andrewlamb@Andrews-MacBook-Pro-3:~/Software/sqlparser-rs$ gh api graphql -f query='{ repository(owner:"apache", name:"datafusion-sqlparser-rs") {
  mergeQueue(branch:"main") { configuration { checkResponseTimeout } } } }'
{
  "data": {
    "repository": {
      "mergeQueue": {
        "configuration": {
          "checkResponseTimeout": 3600
        }
      }
    }
  }
}

What is the testing strategy for this PR?

gh api graphql -f query='{ repository(owner:"apache", name:"datafusion-sqlparser-rs") {
  mergeQueue(branch:"main") { configuration { checkResponseTimeout } } } }'

Are there any user-facing changes?

No.

Written with assistance from Claude Code.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@alamb

alamb commented Oct 14, 2026

Copy link
Copy Markdown
Contributor Author

I'll give it a try and hopefully we can increase our throughput

@alamb
alamb enabled auto-merge October 14, 2026 20:06
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.16%. Comparing base (dacdbc9) to head (b541c5c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2638   +/-   ##
=======================================
  Coverage   81.16%   81.16%           
=======================================
  Files          42       42           
  Lines       33769    33769           
  Branches    33769    33769           
=======================================
  Hits        27408    27408           
  Misses       2797     2797           
  Partials     3564     3564           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alamb
alamb added this pull request to the merge queue Oct 14, 2026
Merged via the queue into apache:main with commit 5d653a2 Oct 14, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants