Skip to content

Fix flaky S3 multipart PUT test#6631

Merged
davidh44 merged 2 commits into
masterfrom
hdavidh/s3-multipart-put-flaky-test
Dec 17, 2025
Merged

Fix flaky S3 multipart PUT test#6631
davidh44 merged 2 commits into
masterfrom
hdavidh/s3-multipart-put-flaky-test

Conversation

@davidh44

@davidh44 davidh44 commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

Fix flaky S3 Multipart PUT tests by increasing invocation count caps

Similar to previous fixes #6399 && #6540


com.github.tomakehurst.wiremock.client.VerificationException: 
Expected exactly 1 requests matching the following pattern but received 2:
{
  "method" : "PUT",
  "queryParameters" : {
    "partNumber" : {
      "matches" : "1"
    }
  }
}
    at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:650)
    at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:630)
    at com.github.tomakehurst.wiremock.client.WireMock.verify(WireMock.java:671)
    at software.amazon.awssdk.services.s3.internal.multipart.S3MultipartClientPutObjectWiremockTest.mpuDefaultSplitImpl_partsFailOfRetryableError_shouldFail(S3MultipartClientPutObjectWiremockTest.java:201)
com.github.tomakehurst.wiremock.client.VerificationException: 
Expected exactly 2 requests matching the following pattern but received 3:
{
  "method" : "PUT",
  "queryParameters" : {
    "partNumber" : {
      "matches" : "2"
    }
  }
}
    at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:650)
    at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:630)
    at com.github.tomakehurst.wiremock.client.WireMock.verify(WireMock.java:671)
    at software.amazon.awssdk.services.s3.internal.multipart.S3MultipartClientPutObjectWiremockTest.mpuWithBufferedSplittableAsyncRequestBody_partsFailOfRetryableError_shouldRetry(S3MultipartClientPutObjectWiremockTest.java:172)

@davidh44 davidh44 requested a review from a team as a code owner December 16, 2025 23:38
@davidh44 davidh44 added the changelog-not-required Indicate changelog entry is not required for a specific PR label Dec 16, 2025
@sonarqubecloud

Copy link
Copy Markdown

@RanVaknin RanVaknin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you please just add the failure stack trace to the PR?

@davidh44 davidh44 added this pull request to the merge queue Dec 17, 2025
Merged via the queue into master with commit fa9db76 Dec 17, 2025
48 of 51 checks passed
@github-actions

Copy link
Copy Markdown

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 17, 2025
@davidh44 davidh44 deleted the hdavidh/s3-multipart-put-flaky-test branch December 17, 2025 21:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

changelog-not-required Indicate changelog entry is not required for a specific PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants