fix: properly propigate internal errors #8113
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
I believe this addresses JavaScriptBach's issue in #8088
We weren't properly propagating unrecoverable errors to
turbo
's exit code. So things like writing a log file failing could cause a zero exit code even thoughturbo
had a failure that left us in a bad state. This PR now propigates internal errors up to therun
command so it will have a non-zero exit when these are encountered.Testing Instructions
Force an internal failure like making a log file owned by root
Before
(Note exit code and the fact that run summary is displayed)
After
(Note the non-zero exit code)
Closes TURBO-3034