Skip to content

fix: report size analysis processing before execution - #732

Closed
jamieQ wants to merge 1 commit into
mainfrom
fix/size-analysis-state-reconciliation
Closed

jamieQ wants to merge 1 commit into
mainfrom
fix/size-analysis-state-reconciliation

Conversation

@jamieQ

@jamieQ jamieQ commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Launchpad side changes related to getsentry/sentry#130246 (see that for more detail).

Refs EME-1341

@sentry

sentry Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
HackerNews com.emergetools.hackernews 3.8 (1) Release

Android

🔗 App Name App ID Version Configuration
Hacker News com.emergetools.hackernews 1.0.2 (13) Release

⚙️ launchpad-test-ios Build Distribution Settings

@sentry

sentry Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Size Analysis

2 components analyzed

iOS Builds

Name Configuration Version Download Size Install Size
HackerNews (iOS)
com.emergetools.hackernews
Release 3.8 (1) 6.5 MB (N/A) 9.7 MB (N/A)

Android Builds

Name Configuration Version Download Size Uncompressed Size
Hacker News (Android)
com.emergetools.hackernews
Release 1.0.2 (13) 2.9 MB (N/A) 6.6 MB (N/A)

Configure launchpad-test-ios status check rules

@linear-code

linear-code Bot commented Nov 6, 2026

Copy link
Copy Markdown

EME-1341

jamieQ added a commit to getsentry/sentry that referenced this pull request Nov 6, 2026
Size analysis metrics were never marked `PROCESSING` after launchpad
started processing them. If the processing jobs permanently failed (e.g.
the pods repeatedly OOM'd), this could result in stranded metrics that
were still marked `PENDING` and which aren't currently considered by the
cleanup cron (it only looks for items in the processing state). Improve
this a bit by having launchpad update the metric's state to processing
when appropriate. This PR is the sentry-side changes to handle that.

Launchpad PR to deploy after this:
getsentry/launchpad#732

Refs EME-1341
@jamieQ

jamieQ commented Nov 6, 2026

Copy link
Copy Markdown
Member Author

closing in favor of getsentry/sentry#130274

@jamieQ jamieQ closed this Nov 6, 2026
@jamieQ
jamieQ deleted the fix/size-analysis-state-reconciliation branch November 6, 2026 18:58
jamieQ added a commit to getsentry/sentry that referenced this pull request Nov 6, 2026
)

Sentry-only mitigation related to
#130246 (see that for more
detail). Alternative implementation to
getsentry/launchpad#732.

Refs EME-1341
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant