Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Test failures in 16.9.1 on Linux/rv64 #40152

Closed
mkszuba opened this issue Sep 19, 2021 · 0 comments · Fixed by #42674
Closed

Test failures in 16.9.1 on Linux/rv64 #40152

mkszuba opened this issue Sep 19, 2021 · 0 comments · Fixed by #42674
Labels
riscv64 Issues and PRs related to the riscv64 architecture.

Comments

@mkszuba
Copy link

mkszuba commented Sep 19, 2021

Version

v16.9.1

Platform

Linux 5.13.14 riscv64

Subsystem

No response

What steps will reproduce the bug?

  1. Compile nodejs-16.9.1 on a rv64 Linux box
  2. Run python tools/test.py --mode=release --flaky-tests=dontcare -J message parallel sequential

How often does it reproduce? Is there a required condition?

Every time so far.

What is the expected behavior?

All tests pass.

What do you see instead?

parallel/test-corepack-yarn-install and sequential/test-cpu-prof-dir-absolute fail, sequential/test-heap-prof times out. Attaching full test output.
nodejs-16.9.1-test.log.gz

Additional information

No response

@targos targos added the riscv64 Issues and PRs related to the riscv64 architecture. label Sep 19, 2021
ArchFeh added a commit to ArchFeh/node that referenced this issue Apr 9, 2022
In riscv hardware test-heap-prof.js caused timeout in test because of weak performance. So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: nodejs#40152
ArchFeh added a commit to ArchFeh/node that referenced this issue Apr 9, 2022
In riscv hardware test-heap-prof.js caused timeout in test because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: nodejs#40152
ArchFeh added a commit to ArchFeh/node that referenced this issue Apr 9, 2022
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: nodejs#40152
nodejs-github-bot pushed a commit that referenced this issue Oct 29, 2022
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: #40152
PR-URL: #42674
Reviewed-By: James M Snell <jasnell@gmail.com>
RafaelGSS pushed a commit that referenced this issue Nov 1, 2022
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: #40152
PR-URL: #42674
Reviewed-By: James M Snell <jasnell@gmail.com>
RafaelGSS pushed a commit that referenced this issue Nov 10, 2022
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: #40152
PR-URL: #42674
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this issue Dec 30, 2022
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: #40152
PR-URL: #42674
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this issue Dec 30, 2022
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: #40152
PR-URL: #42674
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this issue Jan 3, 2023
In riscv hardware test-heap-prof.js caused timeout in test.
Because of weak performance.
So there is a need to set TIMEOUT_SCALEFACTOR for riscv too.

Fixes: #40152
PR-URL: #42674
Reviewed-By: James M Snell <jasnell@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
riscv64 Issues and PRs related to the riscv64 architecture.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants