Skip to content

feat(snapshots): Use retention endpoint for objectstore expiration#3180

Closed
NicoHinderling wants to merge 1 commit intomasterfrom
feat/snapshot-retention
Closed

feat(snapshots): Use retention endpoint for objectstore expiration#3180
NicoHinderling wants to merge 1 commit intomasterfrom
feat/snapshot-retention

Conversation

@NicoHinderling
Copy link
Copy Markdown
Contributor

@NicoHinderling NicoHinderling commented Mar 3, 2026

Related PR: getsentry/sentry#109842

Fetch the snapshot retention value (in days) from
GET /organizations/{org}/preprod/retention/ and use it to construct a
TimeToLive expiration policy for objectstore uploads, instead of using the
expiration_policy string from the upload-options response.

This allows snapshot retention to be configured per-organization via the
backend rather than being hardcoded in the objectstore upload options.

Refs EME-885

…licy

Fetch the snapshot retention value from the organization's preprod
retention endpoint instead of using the expiration policy from the
upload-options response. This allows the retention period to be
configured per-organization via the backend.

Co-Authored-By: Claude <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear bot commented Mar 3, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 3, 2026

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- Use retention endpoint for objectstore expiration ([#3180](https://github.com/getsentry/sentry-cli/pull/3180))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against 23ca864

@NicoHinderling NicoHinderling added the skip-changelog Apply this label to PRs that do not contain any user-facing changes label Mar 3, 2026
@NicoHinderling NicoHinderling marked this pull request as ready for review March 3, 2026 22:20
@NicoHinderling NicoHinderling requested review from a team and szokeasaurusrex as code owners March 3, 2026 22:20
@NicoHinderling
Copy link
Copy Markdown
Contributor Author

synced offline, forgot we have an endpoint dedicated for this case specifically for CLI already

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

Labels

skip-changelog Apply this label to PRs that do not contain any user-facing changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant