Track pr/parent_sha with try parent

This avoids missing the enqueue into pending builds, which then makes us not
post the comment after finishing a benchmark.

Author: Mark-Simulacrum

site/src/load.rs

@@ -24,8 +24,8 @@ pub enum MissingReason {
     Master {
         pr: u32,
         parent_sha: String,
+        is_try_parent: bool,
     },
-    TryParent,
     Try {
         pr: u32,
         include: Option<String>,
@@ -214,6 +214,7 @@ fn calculate_missing_from(
                 MissingReason::Master {
                     pr: c.pr.unwrap_or(0),
                     parent_sha: c.parent_sha,
+                    is_try_parent: false,
                 },
             )
         })
@@ -235,11 +236,31 @@ fn calculate_missing_from(
         // Enqueue the `TryParent` commit before the `TryCommit` itself, so that
         // all of the `try` run's data is complete when the benchmark results
         // of that commit are available.
-        if let Some((try_parent, _)) = master_commits
+        if let Some((try_parent, metadata)) = master_commits
             .iter()
             .find(|(m, _)| m.sha == parent_sha.as_str())
         {
-            missing.push((try_parent.clone(), MissingReason::TryParent));
+            let (pr, parent_sha) = if let MissingReason::Master {
+                pr,
+                parent_sha,
+                is_try_parent: _,
+            } = &metadata
+            {
+                (*pr, parent_sha.clone())
+            } else {
+                unreachable!(
+                    "non-master missing reason in master_commits: {:?}",
+                    metadata
+                );
+            };
+            missing.push((
+                try_parent.clone(),
+                MissingReason::Master {
+                    pr,
+                    parent_sha,
+                    is_try_parent: true,
+                },
+            ));
         }
         missing.push((
             Commit {
@@ -351,7 +372,11 @@ mod tests {
                     sha: "foo".into(),
                     date: database::Date(time),
                 },
-                MissingReason::TryParent,
+                MissingReason::Master {
+                    pr: 77,
+                    parent_sha: "bar".into(),
+                    is_try_parent: true,
+                },
             ),
             (
                 Commit {
@@ -373,6 +398,7 @@ mod tests {
                 MissingReason::Master {
                     pr: 11,
                     parent_sha: "345".into(),
+                    is_try_parent: false,
                 },
             ),
         ];

site/src/request_handlers/next_commit.rs

@@ -9,7 +9,10 @@ pub async fn handle_next_commit(ctxt: Arc<SiteCtxt>) -> next_commit::Response {
     let next_commit = if let Some((commit, missing_reason)) = next_commit {
         // If we're going to run a master commit next, make sure
         // it's been enqueued in the pull_request_build table
-        if let MissingReason::Master { pr, ref parent_sha } = missing_reason {
+        if let MissingReason::Master {
+            pr, ref parent_sha, ..
+        } = missing_reason
+        {
             let conn = ctxt.conn().await;
             // TODO: add capability of doing the following in one step
             // to avoid possibile illegal inbetween states.

site/static/status.html

@@ -175,7 +175,9 @@
                 return `${reason_to_string(reason.InProgress)} - in progress`;
             } else if (reason["Master"] != undefined && reason.Master.pr) {
                 return `<a href="https://github.com/rust-lang/rust/pull/${reason["Master"].pr}">
-                    #${reason["Master"].pr}</a>`;
+                    #${reason["Master"].pr}</a>${
+                    reason.Master.is_try_parent ? " - Try commit parent" : ""
+                    }`;
             } else if (reason["Master"] != undefined && reason.Master.pr == 0) {
                 return "Master";
             } else if (reason["Try"] != undefined && reason.Try.pr) {