confluentinc / ducktape

System integration and performance tests
10 stars 93 forks source link

Show stdout and stderr with failure in remote account #369

Open imcdo opened 1 year ago

imcdo commented 1 year ago

Currently on failure we don't attach stderr and stdout of a command (we do stderr sometimes but its not consistant. This pr achieves that by passing in stdout and stderr to the error.

It also solves the problem of potentially storing to much data in memory (lets say stdout of a long running task) by capturing stdout and stderr in ring buffers, and storing only the last 32 lines to be printed.

This pr is to follow up on this pr: https://github.com/confluentinc/ducktape/pull/234 by @stanislavkozlovski

cla-assistant[bot] commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.