Skip to content

Commit 073221b

Browse files
authored
Merge pull request #2194 from rust-lang/revert-2188-dont-panic-s3
Revert "Do not panic on S3 file upload failures"
2 parents 84a737f + e33734c commit 073221b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

collector/src/compile/execute/bencher.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -338,9 +338,9 @@ impl SelfProfileS3Upload {
338338
let start = std::time::Instant::now();
339339
let status = self.0.wait().expect("waiting for child");
340340
if !status.success() {
341-
log::error!("S3 upload failed: {status:?}");
342-
} else {
343-
log::trace!("uploaded to S3, additional wait: {:?}", start.elapsed());
341+
panic!("S3 upload failed: {:?}", status);
344342
}
343+
344+
log::trace!("uploaded to S3, additional wait: {:?}", start.elapsed());
345345
}
346346
}

0 commit comments

Comments
 (0)