Store duration in database

migrations/20251211185514_add_duration_to_build.down.sql

@@ -0,0 +1,2 @@
+-- Add down migration script here
+ALTER TABLE build DROP COLUMN duration;

migrations/20251211185514_add_duration_to_build.up.sql

@@ -0,0 +1,2 @@
+-- Add up migration script here
+ALTER TABLE build ADD COLUMN duration INTERVAL;

src/bors/handlers/workflow.rs

@@ -269,6 +269,8 @@ async fn maybe_complete_build(
     };
 
     db.update_build_status(&build, status).await?;
+    db.update_build_duration(&build, payload.running_time)
+        .await?;
     if let Some(trigger) = trigger {
         handle_label_trigger(repo, pr_num, trigger).await?;
     }

src/database/client.rs

@@ -1,3 +1,4 @@
+use chrono::Duration;
 use sqlx::PgPool;
 
 use crate::bors::comment::CommentTag;
@@ -18,8 +19,9 @@ use super::operations::{
     get_workflow_urls_for_build, get_workflows_for_build, insert_repo_if_not_exists,
     record_tagged_bot_comment, set_pr_assignees, set_pr_mergeability_state, set_pr_priority,
     set_pr_rollup, set_pr_status, unapprove_pull_request, undelegate_pull_request,
-    update_build_check_run_id, update_build_status, update_mergeable_states_by_base_branch,
-    update_pr_try_build_id, update_workflow_status, upsert_pull_request, upsert_repository,
+    update_build_check_run_id, update_build_duration, update_build_status,
+    update_mergeable_states_by_base_branch, update_pr_try_build_id, update_workflow_status,
+    upsert_pull_request, upsert_repository,
 };
 use super::{ApprovalInfo, DelegatedPermission, MergeableState, RunId, UpsertPullRequestParams};
 
@@ -233,6 +235,14 @@ impl PgDbClient {
         update_build_status(&self.pool, build.id, status).await
     }
 
+    pub async fn update_build_duration(
+        &self,
+        build: &BuildModel,
+        duration: Option<Duration>,
+    ) -> anyhow::Result<()> {
+        update_build_duration(&self.pool, build.id, duration).await
+    }
+
     pub async fn update_build_check_run_id(
         &self,
         build_id: i32,

src/database/operations.rs

@@ -1,4 +1,5 @@
 use chrono::DateTime;
+use chrono::Duration;
 use chrono::Utc;
 use sqlx::postgres::PgExecutor;
 
@@ -631,6 +632,24 @@ pub(crate) async fn update_build_status(
     .await
 }
 
+pub(crate) async fn update_build_duration(
+    executor: impl PgExecutor<'_>,
+    build_id: i32,
+    duration: Option<Duration>,
+) -> anyhow::Result<()> {
+    measure_db_query("update_build_duration", || async {
+        sqlx::query!(
+            "UPDATE build SET duration = $1 WHERE id = $2",
+            duration as _,
+            build_id
+        )
+        .execute(executor)
+        .await?;
+        Ok(())
+    })
+    .await
+}
+
 pub(crate) async fn create_workflow(
     executor: impl PgExecutor<'_>,
     build_id: i32,

tests/data/migrations/20251211185514_add_duration_to_build.sql

@@ -0,0 +1,17 @@
+UPDATE build
+SET
+    duration = '3h 8m 37s'
+WHERE
+    id = 1;
+
+UPDATE build
+SET
+    duration = '3h 11m 36s'
+WHERE
+    id = 2;
+
+UPDATE build
+SET
+    duration = '3h 12m 15s'
+WHERE
+    id = 3;