Skip to content

Make server e2e readiness timeout configurable#43

Merged
mudler merged 1 commit into
mudler:masterfrom
quartzjer:codex/server-e2e-timeout
Jul 1, 2026
Merged

Make server e2e readiness timeout configurable#43
mudler merged 1 commit into
mudler:masterfrom
quartzjer:codex/server-e2e-timeout

Conversation

@quartzjer

Copy link
Copy Markdown
Contributor

Summary

  • add PARAKEET_SERVER_E2E_READY_TIMEOUT to tests/server_e2e.sh
  • validate the timeout value before launching the server
  • set the CI server-e2e readiness timeout to 600s while keeping the job-level 20-minute timeout as the outer cap
  • include the configured timeout in the failure message

Why

PR #41 hit a server-e2e failure where the server was still downloading the first-run tdt_ctc-110m-q4_k.gguf model at the previous 180s readiness deadline. The log showed the 125 MB download around 82% when server_e2e: server not ready fired, then finishing seconds later. Since /health only becomes available after model download and load, the readiness wait needs to allow hosted-runner network variance.

Validation

  • bash -n tests/server_e2e.sh
  • tests/server_e2e.sh; test $? -eq 77
  • PARAKEET_SERVER_E2E=1 PARAKEET_SERVER_E2E_READY_TIMEOUT=bogus tests/server_e2e.sh /bin/true; test $? -eq 1
  • PARAKEET_SERVER_E2E=1 PARAKEET_SERVER_E2E_READY_TIMEOUT=0 tests/server_e2e.sh /bin/true; test $? -eq 1
  • git diff --check

Assisted-by: Codex:gpt-5 [Codex]

@mudler mudler left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

looking good, thanks!

@mudler mudler merged commit 2f880f6 into mudler:master Jul 1, 2026
7 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