You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think some of those are from test/parallel/test-runner-coverage.js which has specific tests checking coverage is produced when the NODE_V8_COVERAGE environment variable is not set.
I think some of those are from test/parallel/test-runner-coverage.js which has specific tests checking coverage is produced when the NODE_V8_COVERAGE environment variable is not set.
If I exclude test/parallel/test-runner-coverage.js from parallel there still at least one more test writing out coverage:
For test-runner-coverage.js - When NODE_V8_COVERAGE is set the report will be written into the dest directory, currently for test-runner-coverage.js is set to tmpdir.path maybe what we need to do is to clear the report directory after the test is finished to prevent the report fill up the disk space, will take another when I wake up.
For test-runner-coverage.js - When NODE_V8_COVERAGE is set the report will be written into the dest directory, currently for test-runner-coverage.js is set to tmpdir.path maybe what we need to do is to clear the report directory after the test is finished to prevent the report fill up the disk space, will take another when I wake up.
I think those cases are working as expected -- but that test has variants that deliberately do not set NODE_V8_COVERAGE and those are causing writes to the system tmp dir which are never cleaned up.
Activity
richardlau commentedon Aug 14, 2024
/tmpon test-equinix_mnx-smartos20-x64-3 is full:It's full of
node-coverage-*directories. I've removed them:richardlau commentedon Aug 14, 2024
I've cleared out
/tmp/node-coverage-*on test-equinix_mnx-smartos20-x64-4 as well:targos commentedon Aug 14, 2024
Good catch!
So there's a bug in our tests. They shouldn't be using the system's tmpdir.
richardlau commentedon Aug 14, 2024
FWIW I ran
make -j88 test-cion a dev machine I have access to and it created these directories/files in/tmp:richardlau commentedon Aug 14, 2024
I think some of those are from
test/parallel/test-runner-coverage.jswhich has specific tests checking coverage is produced when theNODE_V8_COVERAGEenvironment variable is not set.richardlau commentedon Aug 14, 2024
If I exclude
test/parallel/test-runner-coverage.jsfrom parallel there still at least one more test writing out coverage:targos commentedon Aug 14, 2024
Can you try to change the permissions so only root is allowed to write to
/tmp. Hopefully we can identify the tests by observing failures.richardlau commentedon Aug 14, 2024
Not on that machine, but should be able to in a container.
richardlau commentedon Aug 14, 2024
richardlau commentedon Aug 14, 2024
With an actual unwritable
/tmp:targos commentedon Aug 15, 2024
test-cli-node-options.jstest-runner-coverage.jsandtest-runner-output.mjstest-trace-events-http.jsis affected by reading the code.jakecastelli commentedon Aug 15, 2024
For
test-runner-coverage.js- WhenNODE_V8_COVERAGEis set the report will be written into the dest directory, currently fortest-runner-coverage.jsis set totmpdir.pathmaybe what we need to do is to clear the report directory after the test is finished to prevent the report fill up the disk space, will take another when I wake up.richardlau commentedon Aug 15, 2024
I think those cases are working as expected -- but that test has variants that deliberately do not set
NODE_V8_COVERAGEand those are causing writes to the system tmp dir which are never cleaned up.15 remaining items
test_runner: improve code coverage cleanup
richardlau commentedon Sep 11, 2024
/tmpon test-equinix_mnx-smartos20-x64-3 was full again. I have deleted all of thosenode-coverage-*directories.test_runner: improve code coverage cleanup
test_runner: improve code coverage cleanup
test_runner: improve code coverage cleanup
test_runner: improve code coverage cleanup
test_runner: improve code coverage cleanup
test_runner: improve code coverage cleanup
targos commentedon Dec 20, 2024
Is this resolved?
richardlau commentedon Dec 20, 2024
I just ran
make test-ciwithmainand that still appears to be creatingnode-coverage-*directories under/tmp.