Skip to content

Comments

Wip crc64nvme#626

Merged
cbodley merged 2 commits intoceph:masterfrom
linuxbox2:wip-crc64nvme
Mar 31, 2025
Merged

Wip crc64nvme#626
cbodley merged 2 commits intoceph:masterfrom
linuxbox2:wip-crc64nvme

Conversation

@mattbenjamin
Copy link
Contributor

No description provided.

includes tests for CRC64NVME, tests for selecting
COMPOSITE and FULL_OBJECT checksums

a decomposed matrix of tests for all checksum types

also removes the mixed checksum upload case that no longer
works in recent boto3

cleanups, add sha1

checksum validation failure (mismatch) returns BadDigest

multipart checksum matrix helper now validates checksum and checksum type
for all operations which can return them (complete-multipart, head-object,
and get-object-attributes)

Signed-off-by: Matt Benjamin <[email protected]>
@cbodley cbodley merged commit 4d29588 into ceph:master Mar 31, 2025
@cbodley
Copy link
Contributor

cbodley commented Mar 31, 2025

cherry-picked to ceph-master

@cbodley
Copy link
Contributor

cbodley commented Jun 19, 2025

cherry-picked to ceph-squid for ceph/ceph#62312

minor conflicts because ceph-squid doesn't have GetObjectAttributes tests from #571

@cbodley
Copy link
Contributor

cbodley commented Jun 20, 2025

reverting on ceph-squid until the ceph/ceph#62312 backport has everything

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.

2 participants