Skip to content

test: further limit the concurrency of WPTRunner for RISC-V - #64027

Open
kxxt wants to merge 1 commit into
nodejs:mainfrom
kxxt:wpt-lower-concurrency-riscv
Open

kxxt wants to merge 1 commit into
nodejs:mainfrom
kxxt:wpt-lower-concurrency-riscv

Conversation

@kxxt

@kxxt kxxt commented Jun 20, 2026 •

Copy link
Copy Markdown
Member

Even when the concurrency of WPTRunner is limited to 10, flaky WPT tests are still observed on RISC-V with Sv39.

For example, test/wpt/test-compression.js failed in https://github.com/riscv-forks/node-riscv/actions/runs/27756747452 and https://github.com/riscv-forks/node-riscv/actions/runs/27544719447, where the cause of the failure is a failing mmap of roughly 8GiB:

[pid 954955] mmap(NULL, 8589996032, PROT_NONE, ...

Further limit the concurrency of WPTRunner to fix flaky tests. See #60591 for the reason of limiting the WPT test concurrency on RISC-V.

CC @nodejs/platform-riscv64

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure. labels Jun 20, 2026
Even when the concurrency of WPTRunner is limited to 10, flaky WPT tests
are still observed on RISC-V with Sv39.

For example, test/wpt/test-compression.js failed in
https://github.com/riscv-forks/node-riscv/actions/runs/27756747452 and
https://github.com/riscv-forks/node-riscv/actions/runs/27544719447,
where the cause of the failure is a failing `mmap` of roughly 8GiB:

    [pid 954955] mmap(NULL, 8589996032, PROT_NONE, ...

Further limit the concurrency of WPTRunner to fix flaky tests.
See nodejs#60591 for the reason of limiting the WPT test concurrency on
RISC-V.

Signed-off-by: Levi Zim <rsworktech@outlook.com>
@panva panva added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. labels Oct 23, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Oct 23, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants