Skip to content

Commit 067ecdf

Browse files
committed
Fix size analysis tag nullability and manual select_related
1 parent 265a229 commit 067ecdf

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

src/sentry/preprod/size_analysis/grouptype.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,8 @@ def _artifact_to_tags(artifact: PreprodArtifact) -> dict[str, str]:
6161
commit_comparison = artifact.commit_comparison
6262
if commit_comparison is not None:
6363
tags["commit_sha"] = commit_comparison.head_sha
64-
tags["branch"] = commit_comparison.head_ref
64+
if commit_comparison.head_ref is not None:
65+
tags["branch"] = commit_comparison.head_ref
6566
tags["repo"] = commit_comparison.head_repo_name
6667
if commit_comparison.base_sha:
6768
tags["base_sha"] = commit_comparison.base_sha

src/sentry/preprod/size_analysis/tasks.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -326,6 +326,7 @@ def manual_size_analysis_comparison(
326326
)
327327
.select_related("preprod_artifact")
328328
.select_related("preprod_artifact__mobile_app_info")
329+
.select_related("preprod_artifact__commit_comparison")
329330
)
330331
head_size_metrics = list(head_size_metrics_qs)
331332

@@ -337,6 +338,7 @@ def manual_size_analysis_comparison(
337338
)
338339
.select_related("preprod_artifact")
339340
.select_related("preprod_artifact__mobile_app_info")
341+
.select_related("preprod_artifact__commit_comparison")
340342
)
341343
base_size_metrics = list(base_size_metrics_qs)
342344

0 commit comments

Comments
 (0)