Skip to content

Commit

Permalink
fix parallel-ctest-containers creation of log tarballs on failure
Browse files Browse the repository at this point in the history
  • Loading branch information
spoonincode committed Apr 27, 2023
1 parent 99d1961 commit 9023583
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/actions/parallel-ctest-containers/dist/index.mjs

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions .github/actions/parallel-ctest-containers/main.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,8 @@ try {
stream.pipe(packer.entry(header, next));
}).on('finish', () => {packer.finalize()});

child_process.spawn("docker", ["export", tests[i]]).stdout.pipe(extractor);
stream.promises.pipeline(packer, zlib.createGzip(), fs.createWriteStream(`${log_tarball_prefix}-${tests[i]}-logs.tar.gz`));
child_process.spawn("docker", ["export", tests[i].name]).stdout.pipe(extractor);
stream.promises.pipeline(packer, zlib.createGzip(), fs.createWriteStream(`${log_tarball_prefix}-${tests[i].name}-logs.tar.gz`));
}
} catch(e) {
core.setFailed(`Uncaught exception ${e.message}`);
Expand Down

0 comments on commit 9023583

Please sign in to comment.