Skip to content

ci : add PYTEST_WORKERS=1 to fix server-self-hosted job - #28603

Merged
danbev merged 2 commits into
ggml-org:masterfrom
danbev:ci-self-hosted-pytest-workers
Sep 8, 2026
Merged

ci : add PYTEST_WORKERS=1 to fix server-self-hosted job#28603
danbev merged 2 commits into
ggml-org:masterfrom
danbev:ci-self-hosted-pytest-workers

Conversation

@danbev

@danbev danbev commented Sep 8, 2026

Copy link
Copy Markdown
Member

Overview

This commit adds the PYTEST_WORKERS=1 environment variable to the hf-jobs-t4-small:cuda13 runner steps.

Additional information

This is an attempt to address CI failure of this job that I might have introduced in Commit 42f0225 ("server : use pytest-xdist for server tests (#28298)").

Refs: https://github.com/ggml-org/llama.cpp/actions/runs/34126971262/job/101757819134

Requirements

This commit adds the `PYTEST_WORKERS=1` environment variable to the
hf-jobs-t4-small:cuda13 runner steps.

This is an attempt to address CI failure of this job that I might have
introduced in Commit 42f0225
("server : use pytest-xdist for server tests (ggml-org#28298)").

Refs: https://github.com/ggml-org/llama.cpp/actions/runs/34126971262/job/101757819134

@CISC CISC left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah, I was wondering what that failure was about...

@github-actions github-actions Bot added the devops improvements to build systems and github actions label Sep 8, 2026
@CISC

CISC commented Sep 8, 2026

Copy link
Copy Markdown
Member

I think metal job needs it too?

Edit: or strangely enough, at least it fails identically sometimes:
https://github.com/ggml-org/llama.cpp/actions/runs/34200062554/job/101976568181

@danbev

danbev commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

I think metal job needs it too?

Ah yes it might, I'll take a look 👍

@danbev

danbev commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

I tried to run this manually on my fork but it got queued so trying here:
https://github.com/ggml-org/llama.cpp/actions/runs/34215251161

New run which includes the metal-job updates as well:
https://github.com/ggml-org/llama.cpp/actions/runs/34217920814 ✔️

@ggerganov

Copy link
Copy Markdown
Member

The matal job likely needs it as well: https://github.com/ggml-org/llama.cpp/actions/runs/34183828952/job/101928096964

@danbev
danbev marked this pull request as ready for review September 8, 2026 11:32
@danbev
danbev requested a review from a team as a code owner September 8, 2026 11:32
@danbev
danbev requested a review from ggerganov September 8, 2026 11:33
@danbev
danbev merged commit 1744c6b into ggml-org:master Sep 8, 2026
6 checks passed
@danbev
danbev deleted the ci-self-hosted-pytest-workers branch September 8, 2026 11:36
x1250 pushed a commit to x1250/llama.cpp that referenced this pull request Sep 9, 2026
* ci : add PYTEST_WORKERS=1 to fix server-self-hosted job

This commit adds the `PYTEST_WORKERS=1` environment variable to the
hf-jobs-t4-small:cuda13 runner steps.

This is an attempt to address CI failure of this job that I might have
introduced in Commit 42f0225
("server : use pytest-xdist for server tests (ggml-org#28298)").

Refs: https://github.com/ggml-org/llama.cpp/actions/runs/34126971262/job/101757819134

* apply same changes to server-metal steps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants