Skip to content

Bound release network operations and recover Sentry finalization#579

Open
baron wants to merge 3 commits into
mainfrom
publish/release-network-bounds
Open

Bound release network operations and recover Sentry finalization#579
baron wants to merge 3 commits into
mainfrom
publish/release-network-bounds

Conversation

@baron

@baron baron commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • bound Sentry release API connect and total request times
  • reconcile ambiguous transport and HTTP 5xx outcomes through authoritative GET observation before retrying or failing closed
  • add resumable Sentry finalization recovery
  • bound SwiftFormat and promotion downloads with true aggregate remaining-budget deadlines
  • isolate failed-attempt curl stdout so successful --write-out results cannot be corrupted
  • preserve current-main shared release_sentry_symbols authority and Tip-Sentry lifecycle

This current-main replacement supersedes #573, which became merge-conflicted after the Tip-Sentry changes landed. #573 is intentionally left untouched for audit until a maintainer chooses to close it.

Review

  • implemented and integrated in the dedicated PR6 worktree
  • original plan and implementation were both reviewed by Fable
  • after the current-main conflict integration, Fable was consulted again on the final stable commit
  • final exact-Fable verdict: no must-fix issues remain
  • an independent fallback review found three load-bearing retry/deadline issues; all were fixed and re-reviewed before the final Fable pass

Validation

  • python3 -m unittest Scripts.test_release_tooling: 87 tests, all passed (final Fable review run)
  • focused Sentry mutation, deadline-budget, promotion write-out, and formatter fixtures passed
  • bash -n on all changed shell scripts passed
  • Python compile checks passed
  • mandatory commit and push preflights passed, including guardrails and outgoing-range secret scan
  • git diff --check: clean
  • exact outgoing range: three commits, four files

Validation boundary

This is a source/tooling-only change; live/debug app validation is intentionally deferred. Exact-head hosted merge-ref CI is required before merge.

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