Fix: remote results not downloaded on failed actions, such as interrupted tests.
RELNOTES: none
TESTED=presubmit
@@ -123,9 +123,7 @@ final class RemoteSpawnRunner implements SpawnRunner {
// TODO(ulfjack): Download stdout, stderr, and the output files in a single call.
passRemoteOutErr(remoteCache, result, policy.getFileOutErr());
- if (result.getExitCode() == 0) {
remoteCache.downloadAllResults(result, execRoot);
- }
return new SpawnResult.Builder()
.setStatus(Status.SUCCESS) // Even if the action failed with non-zero exit code.
.setExitCode(result.getExitCode())
