feat(preprod): Remove reads from deprecated fields and use PreprodArtifactMobileAppInfo table by rbro112 · Pull Request #105847 · getsentry/sentry

rbro112 · 2026-01-07T20:30:48Z

Updates all reads from now-deprecated PreprodArtifact fields (build_number, build_version , app_icon_id and app_name) to use PreprodArtifactMobileAppInfo fields

rbro112 · 2026-01-07T20:31:06Z

feat(preprod): Remove deprecated/now unused fields from PreprodArtifact #105848
feat(preprod): Remove reads from deprecated fields and use PreprodArtifactMobileAppInfo table #105847 👈 (View in Graphite)
feat(preprod): Add backfill migration for PreprodArtifactMobileAppInfo #105883
feat(preprod): Add PreprodArtifactMobileAppInfo creation to write path #105846
feat(preprod): Add PreprodArtifactMobileAppInfo to enable decoupling of mobile app specific info from PreprodArtifact #105731
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

src/sentry/preprod/api/models/project_preprod_build_details_models.py

        build_configuration=(
            artifact.build_configuration.name if artifact.build_configuration else None
        ),
-        app_icon_id=artifact.app_icon_id,
+        app_icon_id=(
+            artifact.mobile_app_info.app_icon_id if hasattr(artifact, "mobile_app_info") else None
+        ),
        apple_app_info=apple_app_info,
        android_app_info=android_app_info,
    )


src/sentry/preprod/build_distribution_utils.py

+    build_number = (
+        artifact.mobile_app_info.build_number if hasattr(artifact, "mobile_app_info") else None
+    )


…of mobile app specific info from PreprodArtifact (#105731) In prep for using PreprodArtifact for things beyond mobile apps (snapshots), this creates a separate table, `PreprodArtifactMobileAppInfo`, which will contain info specific to mobile apps. The process to land this and deprecate/remove the fields it's meant to replace is a bit complicated, so will summarize here: - [THIS PR] Add the table - #105846 add the write path to create new `PreprodArtifactMobileAppInfo` rows alongside `PreprodArtifact` rows - #105883 add a backfill to fill `PreprodArtifactMobileAppInfo` for all existing `PreprodArtifact`s - #105847 Remove reads from deprecated `PreprodArtifact` fields and use `PreprodArtifactMobileAppInfo` instead - #105848 remove deprecated fields and add a migration to delete the columns

src/sentry/preprod/api/endpoints/project_preprod_artifact_update.py

+        build_version = (
+            head_artifact.mobile_app_info.build_version
+            if hasattr(head_artifact, "mobile_app_info")
+            else None
+        )
+        build_number = (
+            head_artifact.mobile_app_info.build_number
+            if hasattr(head_artifact, "mobile_app_info")
+            else None
+        )


src/sentry/preprod/api/endpoints/project_preprod_check_for_updates.py

trevor-e · 2026-01-13T18:02:39Z

src/sentry/testutils/factories.py

-            mobile_app_info_fields["app_name"] = app_name
-
-        if mobile_app_info_fields:
-            PreprodArtifactMobileAppInfo.objects.create(


Should this call create_preprod_artifact_mobile_app_info() now? I'm surprised no tests failed.

Oh I see you updated the tests to now call self.create_preprod_artifact_mobile_app_info() as well. We could create it as part of this method as a convenience if the fields are provided IMO.

Yeah I was torn here, I actually wanted to test cases where they explicitly weren't provided. I'll add back the convenience of it but leave mostly as-is.

src/sentry/preprod/api/endpoints/project_preprod_check_for_updates.py

+                build_version=mobile_app_info.build_version,
+                build_number=mobile_app_info.build_number,
                release_notes=(
                    preprod_artifact.extras.get("release_notes")
                    if preprod_artifact.extras
                    else None
                ),
-                app_name=preprod_artifact.app_name,
+                app_name=mobile_app_info.app_name,
                download_url=get_download_url_for_artifact(preprod_artifact),
                created_date=preprod_artifact.date_added.isoformat(),
            )


…nfo table

The preprodBuildCount query was reading from the deprecated PreprodArtifact.build_version field instead of the new PreprodArtifactMobileAppInfo.build_version field. This was missed when PR #105847 updated other endpoints to use the new model. Co-Authored-By: Claude <noreply@anthropic.com>

…6527) The preprodBuildCount query was reading from the deprecated `PreprodArtifact.build_version` field instead of the new `PreprodArtifactMobileAppInfo.build_version` field. This was missed when PR #105847 updated other endpoints to use the new model. This caused `preprodBuildCount` to always be `0` for new uploads (after PR #105847 was deployed), which affects: - Mobile Builds tab visibility on release detail pages for non-mobile projects - The count badge displayed on the Mobile Builds tab ### Before - Python project with mobile builds, not showing Mobile Builds tab <img width="643" height="314" alt="Screenshot 2026-01-19 at 16 26 40" src="https://github.com/user-attachments/assets/3fb9f5e9-9a5b-4280-b684-ec9233c01f15" /> ### After <img width="613" height="254" alt="Screenshot 2026-01-19 at 17 54 22" src="https://github.com/user-attachments/assets/b8b6b3ab-3ee5-4d33-b9ac-41c377e15e4b" /> ### Before - Mobile project, not showing count on Mobile Builds tab <img width="609" height="229" alt="Screenshot 2026-01-19 at 16 21 19" src="https://github.com/user-attachments/assets/bb100549-78d3-4111-bd44-093f813ec32c" /> ### After <img width="617" height="238" alt="Screenshot 2026-01-19 at 16 21 48" src="https://github.com/user-attachments/assets/9dfef683-86a0-409a-b61f-9d200701856a" /> Co-authored-by: Claude <noreply@anthropic.com>

rbro112 requested a review from a team as a code owner January 7, 2026 20:30

github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 7, 2026

rbro112 mentioned this pull request Jan 7, 2026

feat(preprod): Add PreprodArtifactMobileAppInfo to enable decoupling of mobile app specific info from PreprodArtifact #105731

Merged

This was referenced Jan 7, 2026

feat(preprod): Add PreprodArtifactMobileAppInfo creation to write path #105846

Merged

feat(preprod): Remove deprecated/now unused fields from PreprodArtifact #105848

Closed

vercel bot deployed to Preview January 7, 2026 20:33 View deployment

sentry bot reviewed Jan 7, 2026

View reviewed changes

rbro112 changed the base branch from ryan/add_preprodartifactmobileappinfo_creation_to_write_path to graphite-base/105847 January 8, 2026 00:57

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from f0842f3 to 5810afa Compare January 8, 2026 00:57

rbro112 changed the base branch from graphite-base/105847 to ryan/add_backfill_migration_for_preprodartifactmobileappinfo January 8, 2026 00:57

rbro112 mentioned this pull request Jan 8, 2026

feat(preprod): Add backfill migration for PreprodArtifactMobileAppInfo #105883

Merged

vercel bot deployed to Preview January 8, 2026 01:00 View deployment

rbro112 changed the title ~~Remove reads from deprecated fields and use PreprodArtifactMobileAppInfo table~~ feat(preprod): Remove reads from deprecated fields and use PreprodArtifactMobileAppInfo table Jan 8, 2026

rbro112 force-pushed the ryan/add_backfill_migration_for_preprodartifactmobileappinfo branch from 63b7c29 to 8958da1 Compare January 8, 2026 17:37

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from 5810afa to d4c4a00 Compare January 8, 2026 17:37

vercel bot deployed to Preview January 8, 2026 17:40 View deployment

sentry bot reviewed Jan 8, 2026

View reviewed changes

src/sentry/preprod/build_distribution_utils.py Outdated

Comment on lines +18 to +20

build_number = (

artifact.mobile_app_info.build_number if hasattr(artifact, "mobile_app_info") else None

)

This comment was marked as outdated.

Sign in to view

rbro112 force-pushed the ryan/add_backfill_migration_for_preprodartifactmobileappinfo branch from 8958da1 to bb33e35 Compare January 8, 2026 18:46

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from d4c4a00 to 0e97769 Compare January 8, 2026 18:46

vercel bot deployed to Preview January 8, 2026 18:48 View deployment

sentry bot reviewed Jan 8, 2026

View reviewed changes

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from 0e97769 to 658b42d Compare January 8, 2026 23:10

rbro112 force-pushed the ryan/add_backfill_migration_for_preprodartifactmobileappinfo branch from bb33e35 to 2bac7c1 Compare January 8, 2026 23:10

vercel bot deployed to Preview January 8, 2026 23:13 View deployment

rbro112 force-pushed the ryan/add_backfill_migration_for_preprodartifactmobileappinfo branch from 2bac7c1 to 11992b0 Compare January 9, 2026 00:11

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from 658b42d to c01fd24 Compare January 9, 2026 00:11

vercel bot deployed to Preview January 9, 2026 00:15 View deployment

sentry bot reviewed Jan 9, 2026

View reviewed changes

src/sentry/preprod/api/endpoints/project_preprod_check_for_updates.py Show resolved Hide resolved

sentry bot reviewed Jan 13, 2026

View reviewed changes

src/sentry/preprod/api/endpoints/project_preprod_check_for_updates.py Outdated Show resolved Hide resolved

vercel bot deployed to Preview January 13, 2026 04:50 View deployment

trevor-e approved these changes Jan 13, 2026

View reviewed changes

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from a6b9e1b to 7f5005d Compare January 13, 2026 23:44

vercel bot deployed to Preview January 13, 2026 23:47 View deployment

sentry bot reviewed Jan 13, 2026

View reviewed changes

vercel bot deployed to Preview January 14, 2026 00:02 View deployment

rbro112 added 14 commits January 13, 2026 18:04

Remove reads from deprecated fields and use PreprodArtifactMobileAppI…

2f5e864

…nfo table

One more

885f625

Tweaks

4aea357

Updates

a20249d

Fix tests

6bb7c20

Fixes

f0f2569

mor

04e7a5d

Tweak

6549a3f

Hopefully the end

8a00097

Cleanups

8d2e9fb

Cleanup

bc636b2

Final feedback

dac1f1f

Fix

d04f04d

Fix

9998b19

rbro112 force-pushed the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch from bb67d90 to 9998b19 Compare January 14, 2026 02:11

vercel bot deployed to Preview January 14, 2026 02:14 View deployment

rbro112 merged commit 45eedfe into master Jan 14, 2026
66 checks passed

rbro112 deleted the ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table branch January 14, 2026 17:42

cameroncooke mentioned this pull request Jan 19, 2026

fix(releases): Use mobile_app_info for preprod build count query #106527

Merged

github-actions bot locked and limited conversation to collaborators Jan 30, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(preprod): Remove reads from deprecated fields and use PreprodArtifactMobileAppInfo table#105847

feat(preprod): Remove reads from deprecated fields and use PreprodArtifactMobileAppInfo table#105847
rbro112 merged 14 commits intomasterfrom
ryan/remove_reads_from_deprecated_fields_and_use_preprodartifactmobileappinfo_table

rbro112 commented Jan 7, 2026 •

edited

Loading

Uh oh!

rbro112 commented Jan 7, 2026 •

edited

Loading

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

Uh oh!

trevor-e Jan 13, 2026

Uh oh!

trevor-e Jan 13, 2026

Uh oh!

rbro112 Jan 13, 2026

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

rbro112 commented Jan 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rbro112 commented Jan 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

Uh oh!

trevor-e Jan 13, 2026

Choose a reason for hiding this comment

Uh oh!

trevor-e Jan 13, 2026

Choose a reason for hiding this comment

Uh oh!

rbro112 Jan 13, 2026

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

rbro112 commented Jan 7, 2026 •

edited

Loading

rbro112 commented Jan 7, 2026 •

edited

Loading