Skip to content

Conversation

@meet2mky
Copy link
Collaborator

@meet2mky meet2mky commented Aug 20, 2025

Description

Upgrade go storage from 1.55.0 to 1.56.1 (which includes resumable API fix and race condition fixes).
Perf:
8eQ2GgGcccwKpSM

https://screenshot.googleplex.com/8eQ2GgGcccwKpSM

Link to the issue in case of a bug fix.

b/435057951

Testing details

  1. Manual - NA
  2. Unit tests - NA
  3. Integration tests - NA

Any backward incompatible change? If so, please explain.

@meet2mky meet2mky requested a review from a team as a code owner August 20, 2025 05:40
@meet2mky meet2mky requested a review from kislaykishore August 20, 2025 05:40
@github-actions github-actions bot added the remind-reviewers Auto remind reviewers in attention set for review post 24hrs of inactivity on PR. label Aug 20, 2025
@meet2mky meet2mky added execute-perf-test Execute performance test in PR and removed remind-reviewers Auto remind reviewers in attention set for review post 24hrs of inactivity on PR. labels Aug 20, 2025
@meet2mky meet2mky self-assigned this Aug 20, 2025
@codecov
Copy link

codecov bot commented Aug 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.22%. Comparing base (2617eec) to head (f8086e9).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #3710       +/-   ##
===========================================
+ Coverage        0   82.22%   +82.22%     
===========================================
  Files           0      144      +144     
  Lines           0    22821    +22821     
===========================================
+ Hits            0    18765    +18765     
- Misses          0     3512     +3512     
- Partials        0      544      +544     
Flag Coverage Δ
unittests 82.22% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@meet2mky meet2mky added the execute-integration-tests Run only integration tests label Aug 20, 2025
@meet2mky meet2mky merged commit 69f7d72 into master Aug 20, 2025
17 checks passed
@meet2mky meet2mky deleted the meet2mky/go-sdk-upgrade-1.56.1 branch August 20, 2025 07:10
charith87 pushed a commit that referenced this pull request Aug 20, 2025
alleaditya pushed a commit that referenced this pull request Aug 25, 2025
Tulsishah pushed a commit that referenced this pull request Aug 25, 2025
Copy link

@wavybaby5280 wavybaby5280 left a comment

Choose a reason for hiding this comment

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

👍🏿

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

Labels

execute-integration-tests Run only integration tests execute-perf-test Execute performance test in PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants