Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

@DaveCTurner DaveCTurner commented Apr 3, 2025

S3BlobContainerRetriesTests uses MD5DigestCalculatingInputStream
from the AWS v1 SDK to compute a MD5 checksum, but this feature is not
available in the v2 SDK. With this commit we remove this dependency and
compute the MD5 checksums directly instead.

Backport of #126099 to 8.x

`S3BlobContainerRetriesTests` uses `MD5DigestCalculatingInputStream`
from the AWS v1 SDK to compute a MD5 checksum, but this feature is not
available in the v2 SDK. With this commit we remove this dependency and
compute the MD5 checksums directly instead.

Backport of elastic#126099 to `8.x`
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v8.19.0 labels Apr 3, 2025
@DaveCTurner DaveCTurner requested review from a team as code owners April 3, 2025 13:16
@DaveCTurner DaveCTurner changed the base branch from main to 8.x April 3, 2025 13:16
@DaveCTurner DaveCTurner changed the title 2025/04/03/backport 126099 md5 digest calculating input stream Migrate tests away from S3 SDK MD5DigestCalculatingInputStream Apr 3, 2025
@DaveCTurner DaveCTurner removed request for a team April 3, 2025 13:17
@elasticsearchmachine elasticsearchmachine merged commit b6a6c84 into elastic:8.x Apr 3, 2025
15 checks passed
@DaveCTurner DaveCTurner deleted the 2025/04/03/backport-126099-MD5DigestCalculatingInputStream branch April 3, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test Issues or PRs that are addressing/adding tests v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants