Skip to content

ci: raise the test job timeout to 45 minutes as a stopgap (see Consiliency/pmcp#383) - #384

Merged
ViperJuice merged 2 commits into
mainfrom
ci/test-timeout-45
Oct 8, 2026
Merged

ViperJuice merged 2 commits into
mainfrom
ci/test-timeout-45

Conversation

@ViperJuice

@ViperJuice ViperJuice commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Raises timeout-minutes for the test matrix job in .github/workflows/test.yml from 30 to 45. Nothing else changes.

Why now: the 3.10 job on #372 head 21a7200 passed in 29m42s, 18 s under the limit. Earlier heads of #372 and #375 were cancelled at 30 minutes. Main itself takes 20–23 minutes per job.

Not a fix: this is a stopgap. Parallelising the suite (pytest-xdist or sharding) is still the real fix, tracked in #383, which stays open.

No src/ change, so no CHANGELOG entry is required.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

…#383)

The 3.10 job on #372 head 21a7200 passed in 29m42s, 18 s
under the limit; earlier heads were cancelled at 30 minutes. Main itself
takes 20-23 minutes. Parallelising the suite remains the real fix and is
still tracked in #383.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0119FzpAxp1zy5wNGodxvSvs
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

scripts/check_workflows.py accepted timeout-minutes only in 10..30, so the
workflows job would refuse the 45-minute test job. The ceiling exists to
stop the 360-minute default being re-created; 45 keeps that guarantee.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0119FzpAxp1zy5wNGodxvSvs
@ViperJuice
ViperJuice merged commit f15c975 into main Oct 8, 2026
13 checks passed
@ViperJuice
ViperJuice deleted the ci/test-timeout-45 branch October 8, 2026 08:49
ViperJuice added a commit that referenced this pull request Oct 9, 2026
…st); CHANGELOG conflicts resolved to main's newer text

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WBJPWpuagAwghQRCJ99two
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.

1 participant