fix(preprod): send date fields to update Github check-run times (#103088)

We weren't sending the `started_at` and `completed_at` fields to Github
so the time always said 0s.

Processing:
<img width="214" height="60" alt="Screenshot 2025-11-10 at 2 54 32 PM"
src="https://github.com/user-attachments/assets/9abb4d64-41e5-4556-aa90-d70dd0de5060"
/>

Completed:
<img width="200" height="56" alt="Screenshot 2025-11-10 at 2 55 30 PM"
src="https://github.com/user-attachments/assets/d6d8f018-fb53-4c09-95ee-f531b7ce2485"
/>

Resolves EME-586

Author: trevor-e

bin/preprod/trigger_size_status_check

@@ -6,6 +6,7 @@ configure()
 
 import logging
 import sys
+from datetime import timedelta
 
 logger = logging.getLogger(__name__)
 
@@ -501,6 +502,12 @@ def main() -> None:
                 head_build_config.name if head_build_config else "None",
             )
 
+            # Simulate processing time by backdating the creation timestamp
+            # This ensures completed_at != started_at in the status check
+            preprod_artifact.date_added = preprod_artifact.date_added - timedelta(minutes=5)
+            preprod_artifact.save(update_fields=["date_added"])
+            logger.info("  ✓ Backdated artifact creation by 5 minutes (simulating processing time)")
+
             # Create size metrics if requested
             if create_metrics:
                 create_size_metrics(preprod_artifact, build_config, metrics_state)

src/sentry/preprod/vcs/status_checks/size/tasks.py

@@ -2,6 +2,7 @@
 
 import logging
 from abc import ABC, abstractmethod
+from datetime import datetime
 from typing import Any
 
 from sentry.constants import ObjectStatus
@@ -42,14 +43,23 @@
 )
 def create_preprod_status_check_task(preprod_artifact_id: int) -> None:
     try:
-        preprod_artifact = PreprodArtifact.objects.get(id=preprod_artifact_id)
+        preprod_artifact: PreprodArtifact | None = PreprodArtifact.objects.get(
+            id=preprod_artifact_id
+        )
     except PreprodArtifact.DoesNotExist:
         logger.exception(
             "preprod.status_checks.create.artifact_not_found",
             extra={"artifact_id": preprod_artifact_id},
         )
         return
 
+    if not preprod_artifact or not isinstance(preprod_artifact, PreprodArtifact):
+        logger.error(
+            "preprod.status_checks.create.artifact_not_found",
+            extra={"artifact_id": preprod_artifact_id},
+        )
+        return
+
     logger.info(
         "preprod.status_checks.create.start",
         extra={"artifact_id": preprod_artifact.id},
@@ -115,6 +125,10 @@ def create_preprod_status_check_task(preprod_artifact_id: int) -> None:
 
     target_url = get_preprod_artifact_url(preprod_artifact)
 
+    completed_at: datetime | None = None
+    if GITHUB_STATUS_CHECK_STATUS_MAPPING[status] == GitHubCheckStatus.COMPLETED:
+        completed_at = preprod_artifact.date_updated
+
     check_id = provider.create_status_check(
         repo=commit_comparison.head_repo_name,
         sha=commit_comparison.head_sha,
@@ -125,6 +139,8 @@ def create_preprod_status_check_task(preprod_artifact_id: int) -> None:
         summary=summary,
         external_id=str(preprod_artifact.id),
         target_url=target_url,
+        started_at=preprod_artifact.date_added,
+        completed_at=completed_at,
     )
     if check_id is None:
         logger.error(
@@ -301,6 +317,8 @@ def create_status_check(
         text: str | None,
         summary: str,
         external_id: str,
+        started_at: datetime,
+        completed_at: datetime | None = None,
         target_url: str | None = None,
     ) -> str | None:
         """Create a status check using provider-specific format."""
@@ -318,6 +336,8 @@ def create_status_check(
         text: str | None,
         summary: str,
         external_id: str,
+        started_at: datetime,
+        completed_at: datetime | None = None,
         target_url: str | None = None,
     ) -> str | None:
         with self._create_scm_interaction_event().capture() as lifecycle:
@@ -373,6 +393,12 @@ def create_status_check(
             if mapped_conclusion:
                 check_data["conclusion"] = mapped_conclusion.value
 
+            if started_at:
+                check_data["started_at"] = started_at.isoformat()
+
+            if completed_at:
+                check_data["completed_at"] = completed_at.isoformat()
+
             if target_url:
                 if target_url.startswith("http"):
                     check_data["details_url"] = target_url