Skip to content

test(sparql-qlever): raise Docker-backed test timeouts to 120s#367

Merged
ddeboer merged 1 commit intomainfrom
fix/ci-qlever-test-timeout
Apr 29, 2026
Merged

test(sparql-qlever): raise Docker-backed test timeouts to 120s#367
ddeboer merged 1 commit intomainfrom
fix/ci-qlever-test-timeout

Conversation

@ddeboer
Copy link
Copy Markdown
Member

@ddeboer ddeboer commented Apr 29, 2026

Summary

  • The last 3 CI runs on main failed with @lde/sparql-qlever:test > test/server.test.ts > Server > start > starts and stops QLever timing out at 20s. The Docker-backed importer.test.ts (30s) trips on some runs too.
  • Root cause: both tests use DockerTaskRunner.run(), which pulls adfreiburg/qlever:commit-a14e0a0 inline. On cold GitHub runners the pull alone exceeds the test budget.
  • Fix: raise the per-test timeouts to 120s on the two tests that pull the QLever image. No CI-side coordination needed.

Test plan

  • CI on this PR passes the @lde/sparql-qlever:test step.
  • After merge, monitor the next main CI run to confirm the QLever timeout no longer recurs.

@ddeboer ddeboer force-pushed the fix/ci-qlever-test-timeout branch from 693501c to 9a80efc Compare April 29, 2026 07:12
@ddeboer ddeboer changed the title ci: pre-pull QLever Docker image to avoid sparql-qlever test timeouts test(sparql-qlever): raise Docker-backed test timeouts to 120s Apr 29, 2026
@ddeboer ddeboer merged commit 6666a16 into main Apr 29, 2026
2 checks passed
@ddeboer ddeboer deleted the fix/ci-qlever-test-timeout branch April 29, 2026 07:15
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