Skip to content

feat(replays): Let the deletion script select which steps it runs - #121501

Open
gggritso wants to merge 2 commits into
masterfrom
georgegritsouk/replay-971-per-step-args-for-replay-deletion
Open

feat(replays): Let the deletion script select which steps it runs#121501
gggritso wants to merge 2 commits into
masterfrom
georgegritsouk/replay-971-per-step-args-for-replay-deletion

Conversation

@gggritso

@gggritso gggritso commented Aug 7, 2026

Copy link
Copy Markdown
Member

Makes each step of delete_replays individually selectable. archive, delete_seer_data and delete_blobs are keyword-only and default to True, so every existing caller keeps its current behaviour.

We sometimes perform one of these steps by hand, out of band, so it's useful to skip some steps (e.g., if we delete GCS contents manually).

getsentry side in https://github.com/getsentry/getsentry/pull/21408

References REPLAY-971

`delete_replays` archives, deletes Seer summaries and deletes RRWeb blobs. Each
step is now selectable, defaulting to on, so a caller that has already performed
one of them by hand can skip it.

Skipping the blob step also skips the legacy `ReplayRecordingSegment` cleanup,
which lives inside `delete_replays_script_async`.
@linear-code

linear-code Bot commented Aug 7, 2026

Copy link
Copy Markdown

REPLAY-971

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 7, 2026
@gggritso

gggritso commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 5cacf21. Configure here.

@gggritso
gggritso marked this pull request as ready for review August 7, 2026 04:10
@gggritso
gggritso requested a review from a team as a code owner August 7, 2026 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant