Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable s3 transport transparent compression/decompression #11604

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

brandond
Copy link
Member

Proposed Changes

Disable s3 transport transparent compression/decompression

Types of Changes

bugfix

Verification

Cannot really verify this one as the issue only occurs on a specific vendor's S3 implementation, and there is no free version available. We can verify that it doesn't regress other users though.

Testing

Linked Issues

#11603

User-Facing Change

Further Comments

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@brandond brandond requested a review from a team as a code owner January 14, 2025 21:57
Copy link

codecov bot commented Jan 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.77%. Comparing base (646e313) to head (4e69997).
Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11604      +/-   ##
==========================================
- Coverage   50.04%   47.77%   -2.27%     
==========================================
  Files         185      185              
  Lines       19265    19271       +6     
==========================================
- Hits         9641     9207     -434     
- Misses       8236     8733     +497     
+ Partials     1388     1331      -57     
Flag Coverage Δ
e2etests 40.53% <0.00%> (-3.76%) ⬇️
inttests 35.20% <0.00%> (+0.06%) ⬆️
unittests 17.05% <100.00%> (+0.02%) ⬆️

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants