Skip to content

CopyObject: add test for encrypted objects #595

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

CopyObject: add test for encrypted objects #595

wants to merge 7 commits into from

Conversation

@clwluvw
Copy link
Member

@clwluvw clwluvw commented Oct 13, 2024

Add test for copy on sse-s3 and sse-c encrypted objects.

@clwluvw clwluvw mentioned this pull request Oct 13, 2024
Open
@clwluvw clwluvw mentioned this pull request Jun 7, 2025
Open
14 tasks
@clwluvw clwluvw force-pushed the enc-copy branch 2 times, most recently from c84d5f4 to aef70d8 Compare June 8, 2025 12:47
@cbodley
Copy link
Contributor

cbodley commented Jun 12, 2025

got some initial results for ceph/ceph#54543 in https://pulpito.ceph.com/cbodley-2025-06-11_21:56:11-rgw-mdw-main-coe-20-distro-default-smithi/ and see lots of failures here. could you help to diagnose these?

for example in https://qa-proxy.ceph.com/teuthology/cbodley-2025-06-11_21:56:11-rgw-mdw-main-coe-20-distro-default-smithi/8323384/teuthology.log

=========================== short test summary info ============================
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1b[unencrypted-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1b[unencrypted-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1b[sse-c-sse-c] - K...
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1b[sse-c-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1b[sse-kms-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1b[sse-kms-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1kb[unencrypted-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1kb[unencrypted-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1kb[sse-c-sse-c] - ...
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1kb[sse-c-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1kb[sse-kms-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1kb[sse-kms-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1mb[unencrypted-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1mb[unencrypted-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1mb[sse-c-sse-c] - ...
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1mb[sse-c-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1mb[sse-kms-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_1mb[sse-kms-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_8mb[unencrypted-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_8mb[unencrypted-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_8mb[sse-c-sse-c] - ...
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_8mb[sse-c-sse-kms]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_8mb[sse-kms-sse-c]
FAILED s3tests_boto3/functional/test_s3.py::test_copy_enc_8mb[sse-kms-sse-kms]

@clwluvw
Copy link
Member Author

clwluvw commented Jun 12, 2025 •
edited
Loading

might be related to: ceph/ceph#54543 (comment)

@clwluvw clwluvw force-pushed the enc-copy branch 2 times, most recently from cabb5c6 to be09573 Compare June 21, 2025 17:12
@cbodley
Copy link
Contributor

cbodley commented Sep 24, 2025

@clwluvw could you please rebase?

@clwluvw
CopyObject: add test for encrypted objects
8f9d461 
Add test for copy on sse-s3 and sse-c encrypted objects.

Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
CopyObject: add storage class copy tests
6b486d7 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
lc-transition: add test with encrypted objects
bfb5c8e 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
UploadPart: add copy part with encryption
1e4780f 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
CompleteMultipartUpload: test to verify sse-c headers
6c16b0e 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
UploadPart: test to verify sse-c headers
e0e68ca 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
refactor: merge test_copy_enc_ with test_copy_enc_storage_class
3129648 
Signed-off-by: Seena Fallah <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@clwluvw @cbodley