Skip to content

Raise the CI database statement timeout to 10 minutes - #7184

Merged
roed314 merged 1 commit into
LMFDB:mainfrom
roed-math:ci-statement-timeout-10min
Aug 9, 2026
Merged

Raise the CI database statement timeout to 10 minutes#7184
roed314 merged 1 commit into
LMFDB:mainfrom
roed-math:ci-statement-timeout-10min

Conversation

@roed-math

Copy link
Copy Markdown
Contributor

Summary

Raise the default PostgreSQL statement timeout used by the test tox environment
from 120 seconds to 10 minutes.

Motivation

The two-minute timeout introduced in #7181 successfully prevents unbounded
queries, but it also acts as a sharp performance threshold against the shared
devmirror database. The completeness test has now crossed that threshold in
multiple runs even though the underlying logical result is correct (e.g.
https://github.com/LMFDB/lmfdb/actions/runs/31268853079/job/93204410081, which
canceled the null-data probe for an intrinsically impossible nf_fields
query while it was waiting on the timeout).

Ten minutes still catches a pathological statement far earlier than the
60-minute GitHub Actions job timeout while leaving substantially more headroom
for transient load.

Scope

This PR intentionally changes only tox.ini. The expensive completeness query
and CI matrix concurrency are addressed separately.

Verification

  • tox config -e tests resolves the default to statement_timeout=10min.
  • An exported PGOPTIONS value still overrides the default (checked with PGOPTIONS='-c statement_timeout=0' tox config -e tests).
  • git diff --check passes.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@roed-math

Copy link
Copy Markdown
Contributor Author

Note on the close/reopen in the timeline: the first CI run hit a transient psycopg.errors.ConnectionTimeout while collecting tests on the devmirror modular_curves shard (two full 12-job matrices were hitting devmirror simultaneously — the load situation #7185 caps). Since this branch has no admin rights to rerun jobs, closing and reopening retriggered CI on the same commit; the retriggered run is fully green (13/13 checks).

@roed314
roed314 merged commit 470e5be into LMFDB:main Aug 9, 2026
25 of 26 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