Revert "Add try build on check run start and complete"

This reverts commit 1eb681f.

Author: Kobzol

.sqlx/query-33cf1b803a0b658f197ab26d87e2c6c4fe42128cb481a234372626f35392e809.json

@@ -1,8 +1,6 @@
 use std::sync::Arc;
 
 use anyhow::{Context, anyhow};
-use octocrab::params::checks::CheckRunOutput;
-use octocrab::params::checks::CheckRunStatus;
 
 use crate::PgDbClient;
 use crate::bors::RepositoryState;
@@ -35,9 +33,6 @@ pub(super) const TRY_MERGE_BRANCH_NAME: &str = "automation/bors/try-merge";
 // This branch should run CI checks.
 pub(super) const TRY_BRANCH_NAME: &str = "automation/bors/try";
 
-// The name of the check run seen in the GitHub UI.
-const TRY_BUILD_CHECK_RUN_NAME: &str = "Bors try build";
-
 /// Performs a so-called try build - merges the PR branch into a special branch designed
 /// for running CI checks.
 ///
@@ -92,34 +87,10 @@ pub(super) async fn command_try_build(
     {
         MergeResult::Success(merge_sha) => {
             // If the merge was succesful, run CI with merged commit
-            let build_id =
-                run_try_build(&repo.client, &db, &pr.db, merge_sha.clone(), base_sha).await?;
+            run_try_build(&repo.client, &db, &pr.db, merge_sha.clone(), base_sha).await?;
 
             handle_label_trigger(repo, pr.number(), LabelTrigger::TryBuildStarted).await?;
 
-            // Create a check run to track the try build status in GitHub's UI.
-            // This gets added to the PR's head SHA so GitHub shows UI in the checks tab and
-            // the bottom of the PR.
-            let check_run = repo
-                .client
-                .create_check_run(
-                    TRY_BUILD_CHECK_RUN_NAME,
-                    &pr.github.head.sha,
-                    CheckRunStatus::InProgress,
-                    CheckRunOutput {
-                        title: "Bors try build".to_string(),
-                        summary: "".to_string(),
-                        text: None,
-                        annotations: vec![],
-                        images: vec![],
-                    },
-                    &build_id.to_string(),
-                )
-                .await?;
-
-            db.update_build_check_run_id(build_id, check_run.id.into_inner() as i64)
-                .await?;
-
             repo.client
                 .post_comment(
                     pr.number(),
@@ -205,18 +176,17 @@ async fn run_try_build(
     pr: &PullRequestModel,
     commit_sha: CommitSha,
     parent_sha: CommitSha,
-) -> anyhow::Result<i32> {
+) -> anyhow::Result<()> {
     client
         .set_branch_to_sha(TRY_BRANCH_NAME, &commit_sha, ForcePush::Yes)
         .await
         .map_err(|error| anyhow!("Cannot set try branch to main branch: {error:?}"))?;
 
-    let build_id = db
-        .attach_try_build(pr, TRY_BRANCH_NAME.to_string(), commit_sha, parent_sha)
+    db.attach_try_build(pr, TRY_BRANCH_NAME.to_string(), commit_sha, parent_sha)
         .await?;
 
     tracing::info!("Try build started");
-    Ok(build_id)
+    Ok(())
 }
 
 enum MergeResult {
@@ -337,16 +307,13 @@ fn auto_merge_commit_message(
 
 #[cfg(test)]
 mod tests {
-    use crate::bors::handlers::trybuild::{
-        TRY_BRANCH_NAME, TRY_BUILD_CHECK_RUN_NAME, TRY_MERGE_BRANCH_NAME,
-    };
+    use crate::bors::handlers::trybuild::{TRY_BRANCH_NAME, TRY_MERGE_BRANCH_NAME};
     use crate::database::operations::get_all_workflows;
     use crate::github::CommitSha;
     use crate::tests::mocks::{
         BorsBuilder, Comment, GitHubState, User, Workflow, WorkflowEvent, default_pr_number,
         default_repo_name, run_test,
     };
-    use octocrab::params::checks::{CheckRunConclusion, CheckRunStatus};
 
     #[sqlx::test]
     async fn try_success(pool: sqlx::PgPool) {
@@ -894,74 +861,4 @@ try_failed = ["+foo", "+bar", "-baz"]
             })
             .await;
     }
-
-    #[sqlx::test]
-    async fn try_build_creates_check_run(pool: sqlx::PgPool) {
-        run_test(pool.clone(), |mut tester| async {
-            tester.create_branch(TRY_BRANCH_NAME).expect_suites(1);
-            tester.post_comment("@bors try").await?;
-            tester.expect_comments(1).await;
-
-            tester.expect_check_run(
-                &tester.default_pr().await.get_gh_pr().head_sha,
-                TRY_BUILD_CHECK_RUN_NAME,
-                "Bors try build",
-                CheckRunStatus::InProgress,
-                None,
-            );
-
-            let check_run = tester.get_check_run().await?;
-            insta::assert_snapshot!(check_run.summary, @"");
-            insta::assert_snapshot!(check_run.text, @"");
-
-            Ok(tester)
-        })
-        .await;
-    }
-
-    #[sqlx::test]
-    async fn try_build_updates_check_run_on_success(pool: sqlx::PgPool) {
-        run_test(pool.clone(), |mut tester| async {
-            tester.create_branch(TRY_BRANCH_NAME).expect_suites(1);
-            tester.post_comment("@bors try").await?;
-            tester.expect_comments(1).await;
-
-            tester.workflow_success(tester.try_branch()).await?;
-            tester.expect_comments(1).await;
-
-            tester.expect_check_run(
-                &tester.default_pr().await.get_gh_pr().head_sha,
-                TRY_BUILD_CHECK_RUN_NAME,
-                "Bors try build",
-                CheckRunStatus::Completed,
-                Some(CheckRunConclusion::Success),
-            );
-
-            Ok(tester)
-        })
-        .await;
-    }
-
-    #[sqlx::test]
-    async fn try_build_updates_check_run_on_failure(pool: sqlx::PgPool) {
-        run_test(pool.clone(), |mut tester| async {
-            tester.create_branch(TRY_BRANCH_NAME).expect_suites(1);
-            tester.post_comment("@bors try").await?;
-            tester.expect_comments(1).await;
-
-            tester.workflow_failure(tester.try_branch()).await?;
-            tester.expect_comments(1).await;
-
-            tester.expect_check_run(
-                &tester.default_pr().await.get_gh_pr().head_sha,
-                TRY_BUILD_CHECK_RUN_NAME,
-                "Bors try build",
-                CheckRunStatus::Completed,
-                Some(CheckRunConclusion::Failure),
-            );
-
-            Ok(tester)
-        })
-        .await;
-    }
 }

src/bors/handlers/trybuild.rs

@@ -1,9 +1,6 @@
 use std::sync::Arc;
 use std::time::Duration;
 
-use octocrab::params::checks::CheckRunConclusion;
-use octocrab::params::checks::CheckRunStatus;
-
 use crate::PgDbClient;
 use crate::bors::CheckSuiteStatus;
 use crate::bors::RepositoryState;
@@ -196,18 +193,6 @@ async fn try_complete_build(
 
     handle_label_trigger(repo, pr.number, trigger).await?;
 
-    if let Some(check_run_id) = build.check_run_id {
-        let (status, conclusion) = if has_failure {
-            (CheckRunStatus::Completed, Some(CheckRunConclusion::Failure))
-        } else {
-            (CheckRunStatus::Completed, Some(CheckRunConclusion::Success))
-        };
-
-        repo.client
-            .update_check_run(check_run_id as u64, status, conclusion, None)
-            .await?;
-    }
-
     let message = if !has_failure {
         tracing::info!("Workflow succeeded");
         try_build_succeeded_comment(&workflows, payload.commit_sha, &build)

src/bors/handlers/workflow.rs

@@ -15,9 +15,9 @@ use super::operations::{
     get_prs_with_unknown_mergeable_state, get_pull_request, get_repository, get_repository_by_name,
     get_running_builds, get_workflow_urls_for_build, get_workflows_for_build,
     insert_repo_if_not_exists, set_pr_assignees, 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_mergeable_state,
-    update_pr_try_build_id, update_workflow_status, upsert_pull_request, upsert_repository,
+    unapprove_pull_request, undelegate_pull_request, update_build_status,
+    update_mergeable_states_by_base_branch, update_pr_mergeable_state, update_pr_try_build_id,
+    update_workflow_status, upsert_pull_request, upsert_repository,
 };
 use super::{ApprovalInfo, DelegatedPermission, MergeableState, RunId, UpsertPullRequestParams};
 
@@ -181,13 +181,13 @@ impl PgDbClient {
         branch: String,
         commit_sha: CommitSha,
         parent: CommitSha,
-    ) -> anyhow::Result<i32> {
+    ) -> anyhow::Result<()> {
         let mut tx = self.pool.begin().await?;
         let build_id =
             create_build(&mut *tx, &pr.repository, &branch, &commit_sha, &parent).await?;
         update_pr_try_build_id(&mut *tx, pr.id, build_id).await?;
         tx.commit().await?;
-        Ok(build_id)
+        Ok(())
     }
 
     pub async fn find_build(
@@ -214,14 +214,6 @@ impl PgDbClient {
         update_build_status(&self.pool, build.id, status).await
     }
 
-    pub async fn update_build_check_run_id(
-        &self,
-        build_id: i32,
-        check_run_id: i64,
-    ) -> anyhow::Result<()> {
-        update_build_check_run_id(&self.pool, build_id, check_run_id).await
-    }
-
     pub async fn create_workflow(
         &self,
         build: &BuildModel,

src/database/client.rs

@@ -315,7 +315,6 @@ pub struct BuildModel {
     /// The base commit SHA that this build is merged with (e.g., main branch HEAD).
     pub parent: String,
     pub created_at: DateTime<Utc>,
-    /// The ID of the check run associated with the build.
     pub check_run_id: Option<i64>,
 }
 

src/database/mod.rs

@@ -946,24 +946,6 @@ pub(crate) async fn upsert_repository(
     .await
 }
 
-pub(crate) async fn update_build_check_run_id(
-    executor: impl PgExecutor<'_>,
-    build_id: i32,
-    check_run_id: i64,
-) -> anyhow::Result<()> {
-    measure_db_query("update_build_check_run_id", || async {
-        sqlx::query!(
-            "UPDATE build SET check_run_id = $1 WHERE id = $2",
-            check_run_id,
-            build_id
-        )
-        .execute(executor)
-        .await?;
-        Ok(())
-    })
-    .await
-}
-
 /// Removes the auto build associated with a PR and deletes the build record (if one exists).
 pub(crate) async fn delete_auto_build(
     executor: impl PgExecutor<'_>,

src/database/operations.rs

@@ -1,18 +1,14 @@
 use anyhow::Context;
 use octocrab::Octocrab;
-use octocrab::models::checks::CheckRun;
 use octocrab::models::{App, Repository};
-use octocrab::params::checks::{CheckRunConclusion, CheckRunOutput, CheckRunStatus};
 use tracing::log;
 
 use crate::bors::event::PullRequestComment;
 use crate::bors::{CheckSuite, CheckSuiteStatus, Comment};
 use crate::config::{CONFIG_FILE_PATH, RepositoryConfig};
 use crate::database::RunId;
 use crate::github::api::base_github_html_url;
-use crate::github::api::operations::{
-    ForcePush, MergeError, create_check_run, merge_branches, set_branch_to_commit, update_check_run,
-};
+use crate::github::api::operations::{ForcePush, MergeError, merge_branches, set_branch_to_commit};
 use crate::github::{CommitSha, GithubRepoName, PullRequest, PullRequestNumber};
 use crate::utils::timing::{measure_network_request, perform_network_request_with_retry};
 use futures::TryStreamExt;
@@ -169,39 +165,6 @@ impl GithubRepositoryClient {
         .map_err(|_| MergeError::Timeout)?
     }
 
-    /// Create a check run for the given commit.
-    pub async fn create_check_run(
-        &self,
-        name: &str,
-        head_sha: &CommitSha,
-        status: CheckRunStatus,
-        output: CheckRunOutput,
-        external_id: &str,
-    ) -> anyhow::Result<CheckRun> {
-        measure_network_request("create_check_run", || async {
-            create_check_run(self, name, head_sha, status, output, external_id)
-                .await
-                .context("Cannot create check run")
-        })
-        .await
-    }
-
-    /// Update a check run with the given check run ID.
-    pub async fn update_check_run(
-        &self,
-        check_run_id: u64,
-        status: CheckRunStatus,
-        conclusion: Option<CheckRunConclusion>,
-        output: Option<CheckRunOutput>,
-    ) -> anyhow::Result<CheckRun> {
-        measure_network_request("update_check_run", || async {
-            update_check_run(self, check_run_id, status, conclusion, output)
-                .await
-                .context("Cannot update check run")
-        })
-        .await
-    }
-
     /// Find all check suites attached to the given commit and branch.
     pub async fn get_check_suites_for_commit(
         &self,