Skip to content

Conversation

@lcian
Copy link
Member

@lcian lcian commented Oct 29, 2025

Partially revert #102080.
In particular the usage of the options.
It was causing sentry-options-automator to fail for a currently unknown reason, see INC-1478.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 29, 2025
@lcian lcian changed the title inc fix(objectstore): Partially revert #102080 Oct 29, 2025
cursor[bot]

This comment was marked as outdated.

@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

❌ Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/objectstore/service.py 25.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #102354      +/-   ##
===========================================
+ Coverage   80.16%    80.95%   +0.78%     
===========================================
  Files        8800      8799       -1     
  Lines      389979    389938      -41     
  Branches    24817     24817              
===========================================
+ Hits       312646    315682    +3036     
+ Misses      76959     73882    -3077     
  Partials      374       374              

@lcian lcian requested a review from a team as a code owner October 29, 2025 21:29
@lcian lcian merged commit aa6037f into master Oct 29, 2025
68 checks passed
@lcian lcian deleted the lcian/partial-revert-objectstore-options branch October 29, 2025 22:30
armenzg added a commit that referenced this pull request Oct 31, 2025
This was removed in #102354 during an incident but Dan's change has been merged since then.
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.

4 participants