Skip to content

Commit c51fdd0

Browse files
authored
Merge pull request #2343 from Kobzol/estimation-no-failed
Do not consider completed requests with errors for request duration estimation
2 parents ac90432 + d0b8747 commit c51fdd0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

site/src/request_handlers/status_page_new.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ pub async fn handle_status_page(ctxt: Arc<SiteCtxt>) -> anyhow::Result<status_ne
2020
// Figure out approximately how long was the most recent master benchmark request
2121
let expected_duration = completed
2222
.iter()
23-
.filter(|req| req.request.is_master())
23+
.filter(|req| req.request.is_master() && req.errors.is_empty())
2424
.filter_map(|req| match req.request.status() {
2525
BenchmarkRequestStatus::Completed { duration, .. } => Some(duration),
2626
_ => None,

0 commit comments

Comments
 (0)