Skip to content

Conversation

@lcian
Copy link
Member

@lcian lcian commented Nov 14, 2025

  • Bumps objectstore-client to 0.0.10 and makes the necessary changes to adapt to the new API.
  • Removes the leftover tests in tests/sentry/objectstore/test_objectstore.py as those e2e tests are now part of the client.

@lcian lcian requested review from a team as code owners November 14, 2025 14:51
@lcian lcian requested review from Swatinem and removed request for a team November 14, 2025 14:51
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 14, 2025
@codecov
Copy link

codecov bot commented Nov 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #103365      +/-   ##
===========================================
+ Coverage   78.01%    80.67%   +2.66%     
===========================================
  Files        9235      9237       +2     
  Lines      394520    394510      -10     
  Branches    25146     25146              
===========================================
+ Hits       307790    318286   +10496     
+ Misses      86283     75777   -10506     
  Partials      447       447              

@lcian lcian changed the title ref(objectstore): Upgrade client to 0.0.9 ref(objectstore): Upgrade client to 0.0.10 Nov 14, 2025
@lcian lcian merged commit 2bda6a9 into master Nov 17, 2025
71 checks passed
@lcian lcian deleted the lcian/ref/objectstore-upgrade branch November 17, 2025 10:52
@sentry
Copy link

sentry bot commented Nov 17, 2025

Issues attributed to commits in this pull request

This pull request was merged and Sentry observed the following issues:

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