Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add error translation from S3ResponseError to Botocore errors #284

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

nateprewitt
Copy link
Contributor

This PR adds functionality to the CRT S3 Transfer Manager to handle error translation from the S3 service. This will ensure Boto3 integration retains existing error hierarchy to provide a seamless experience between the two transfer methods.

Merge of this PR is pending boto/botocore#3073 to provide the operation_name off the CRT shape.

@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2023

Codecov Report

Attention: 39 lines in your changes are missing coverage. Please review.

Comparison is base (7b13b56) 84.23% compared to head (7450ccb) 83.01%.

Files Patch % Lines
s3transfer/crt.py 0.00% 39 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                 Coverage Diff                  @@
##           crt-enhancements     #284      +/-   ##
====================================================
- Coverage             84.23%   83.01%   -1.23%     
====================================================
  Files                    16       16              
  Lines                  2760     2796      +36     
====================================================
- Hits                   2325     2321       -4     
- Misses                  435      475      +40     

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

Copy link
Member

@kyleknap kyleknap left a comment

Choose a reason for hiding this comment

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

Looks good. Just had a couple of comments.

tests/unit/test_crt.py Show resolved Hide resolved
s3transfer/crt.py Show resolved Hide resolved
Co-authored-by: Michael Graeb <graebm@amazon.com>
Copy link
Member

@kyleknap kyleknap left a comment

Choose a reason for hiding this comment

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

🚢

@nateprewitt nateprewitt merged commit 92e7749 into crt-enhancements Nov 24, 2023
26 checks passed
@nateprewitt nateprewitt deleted the crt-error-mapping branch November 24, 2023 21:34
aws-sdk-python-automation added a commit that referenced this pull request Nov 27, 2023
* release-0.8.0:
  Bumping version to 0.8.0
  Remove CRT translation integ test (#287)
  Move minimum botocore version to 1.32.7 (#286)
  Update minimum version for Botocore
  Add error translation from S3ResponseError to Botocore errors (#284)
  Update interface for providing credentials to CRT
  Update throughput target defaults
  Add CRT process lock utility
  Turn on checksum validation for CRT S3 transfer manager
  Support file-like objects in CRT transfer manager
  update ci to use GA python 3.12 (#278)
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.

None yet

3 participants