Skip to content

server : test script : add timeout for all requests - #9282

Merged
ngxson merged 1 commit into
ggml-org:masterfrom
ngxson:xsn/fix_test_timeout
Sep 2, 2024
Merged

server : test script : add timeout for all requests#9282
ngxson merged 1 commit into
ggml-org:masterfrom
ngxson:xsn/fix_test_timeout

Conversation

@ngxson

@ngxson ngxson commented Sep 2, 2024

Copy link
Copy Markdown
Collaborator

Some steps in server test script are missing timeout config, which make it to use the default value of 300 seconds

This causes the the CI to randomly (and rarely) fail. Mostly due to the instability of hosted runner:

This PR add timeout everywhere in the script, the value is 600 seconds, which is double the default value.


@github-actions github-actions Bot added examples python python script changes server labels Sep 2, 2024
@ngxson

ngxson commented Sep 2, 2024

Copy link
Copy Markdown
Collaborator Author

Small note: I tested the stability of CI by re-running test multiple times (mixed between slow and default test configs).

In rare occasion, windows test failed due to ERROR: Server stopped listening. After more careful inspections, I believe that the problem comes from race condition inside the test script itself. Since it rarely happens and is windows-only, I won't go deeper right now (i.e. won't fix)

@ngxson
ngxson merged commit 48baa61 into ggml-org:master Sep 2, 2024
dsx1986 pushed a commit to dsx1986/llama.cpp that referenced this pull request Oct 29, 2024
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 15, 2024
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 18, 2024
Nexesenex pushed a commit to Nexesenex/croco.cpp that referenced this pull request Feb 25, 2025
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
phibya pushed a commit to ziee-ai/llama.cpp that referenced this pull request May 29, 2026
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
fukuro-kun pushed a commit to fukuro-kun/fukuro-llama-cpp-turboquant that referenced this pull request Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

examples python python script changes server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants