citusdata / citusdata/test-automation

When an error happens in `common_setup` it's often difficult to tell what went wrong

Open
#35 0 comments 0 reactions 0 assignees View on GitHub
usability
Dominant language
C
Stars
15
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Because we run everything in parallel, the error often doesn't propagate all the way up until enough log lines have been emitted for the error to leave the scroll buffer. This can make debugging pretty hard!

Some solutions
- Don't emit to stdout unless an error happens, so we don't flood the scroll buffer
- Emit loglines but also write to a file the user can later inspect
- When an error happens, propagate the error faster and stop execution on other hosts
- If an error happens have the output from the failed command, and print it once the error finally propagates

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.