Skip to content

feat(preprod): Add snapshots subcommand#3049

Draft
noahsmartin wants to merge 1 commit intomasterfrom
addSnapshotsSubcommand
Draft

feat(preprod): Add snapshots subcommand#3049
noahsmartin wants to merge 1 commit intomasterfrom
addSnapshotsSubcommand

Conversation

@noahsmartin
Copy link
Contributor

Just sharing this early version of a snapshot upload command to collaborate on it. A lot is still TBD such as objectstore public endpoints and connection pooling

@github-actions
Copy link
Contributor

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

- Add snapshots subcommand ([#3049](https://github.com/getsentry/sentry-cli/pull/3049))

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 95876c4

lcian added a commit that referenced this pull request Feb 27, 2026
Updated version of #3049 to
discuss and iterate on things.

Notable changes:
- Removed `shard_index` parameter from the command. I'm not sure what
the purpose of that was originally.
- This uses the new `many` (batch) API from `objectstore_client`. All
uploads are executed as batch requests, reducing network overhead.
Unfortunately, with they way things are implemented now, we will still
have to buffer all files in memory before sending the request, as we
need to hash their contents to determine the filename. If we could just
use the filename as the key in objectstore, it would be much better
because that way we could stream the files over.

Note that the objectstore endpoint is currently gated by a feature flag,
and only enabled for internal orgs/teams.

Ref FS-233

---------

Co-authored-by: Noah Martin <noahm444@gmail.com>
Co-authored-by: Ryan Brooks <ryanjeffrey.brooks112@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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