Skip to content

Commit 49be500

Browse files
bnoordhuiscodebytere
authored andcommitted
test: show child stderr output in largepages test
The test starts child processes. A recent change is suspected of causing flaky crashes on one of the alpine buildbots but we can't know for sure because the test hides the child's stderr. Refs: #31547 (comment) PR-URL: #31612 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
1 parent c3247fe commit 49be500

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

‎test/parallel/test-startup-large-pages.js‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,8 @@ const { spawnSync } = require('child_process');
88

99
{
1010
constchild=spawnSync(process.execPath,
11-
['--use-largepages=on','-p','42']);
11+
['--use-largepages=on','-p','42'],
12+
{stdio: ['inherit','pipe','inherit']});
1213
conststdout=child.stdout.toString().match(/\S+/g);
1314
assert.strictEqual(child.status,0);
1415
assert.strictEqual(child.signal,null);

0 commit comments

Comments
 (0)