Skip to content

Commit 9c90810

Browse files
authored
Merge pull request #2403 from Kobzol/enqueue-logs
Add more logs around commit enqueueing
2 parents 343d823 + 20960fa commit 9c90810

File tree

1 file changed

+16
-2
lines changed

1 file changed

+16
-2
lines changed

site/src/request_handlers/github.rs

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -284,13 +284,27 @@ async fn handle_rust_timer(
284284
}
285285
}
286286

287-
let enqueued = enqueue_shas(
287+
let enqueued = match enqueue_shas(
288288
&ctxt,
289289
main_client,
290290
issue.number,
291291
valid_build_cmds.iter().map(|c| c.sha),
292292
)
293-
.await?;
293+
.await
294+
{
295+
Ok(enqueued) => enqueued,
296+
Err(error) => {
297+
log::error!("Failed to enqueue SHAs on {}: {error:?}", issue.number);
298+
main_client
299+
.post_comment(
300+
issue.number,
301+
"Failed to enqueue some commit SHAs. Maybe they were already benchmarked?"
302+
.to_string(),
303+
)
304+
.await;
305+
return Ok(github::Response);
306+
}
307+
};
294308
if enqueued.len() < valid_build_cmds.len() {
295309
use std::fmt::Write;
296310

0 commit comments

Comments
 (0)