Skip to content

CopyObject: add test for encrypted objects#595

Merged
cbodley merged 8 commits intoceph:masterfrom
clwluvw:enc-copy
Dec 1, 2025
Merged

CopyObject: add test for encrypted objects#595
cbodley merged 8 commits intoceph:masterfrom
clwluvw:enc-copy

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
Closed
@clwluvw clwluvw mentioned this pull request Jun 7, 2025
Merged
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
8a91eb2 
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
eed0551 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
lc-transition: add test with encrypted objects
db3d28b 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
UploadPart: add copy part with encryption
9732871 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
CompleteMultipartUpload: test to verify sse-c headers
0086db1 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
UploadPart: test to verify sse-c headers
9ca6d04 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
refactor: merge test_copy_enc_ with test_copy_enc_storage_class
1edf7e5 
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
test_multipart_sse_c_get_part: pass enc attrs on mp complete
5982e64 
Based on the new fixes around copy enc, those headers are required
when completing mp upload.

Signed-off-by: Seena Fallah <[email protected]>
@cbodley cbodley merged commit 21e08b5 into ceph:master Dec 1, 2025
@cbodley
Copy link
Contributor

cbodley commented Dec 2, 2025

cherry-picked to ceph-master

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

Reviewers

@cbodley cbodley cbodley approved these changes

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