Skip to content

[FLINK-39099] Update testcontainers version to 1.21.4#27618

Closed
ferenc-csaky wants to merge 1 commit intoapache:masterfrom
ferenc-csaky:FLINK-39099
Closed

[FLINK-39099] Update testcontainers version to 1.21.4#27618
ferenc-csaky wants to merge 1 commit intoapache:masterfrom
ferenc-csaky:FLINK-39099

Conversation

@ferenc-csaky
Copy link
Contributor

Trivial dependency bump to avoid Docker client too old errors:

org.testcontainers.dockerclient.DockerClientProviderStrategy [] -     UnixSocketClientProviderStrategy: failed with exception BadRequestException (Status 400: {"message":"client version 1.32 is too old. Minimum supported API version is 1.44, please upgrade your client to a newer version"}

This can affect CI (GitHub) and downstream connector projects.

@flinkbot
Copy link
Collaborator

flinkbot commented Feb 16, 2026

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

Copy link
Contributor

@gaborgsomogyi gaborgsomogyi left a comment

Choose a reason for hiding this comment

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

Assuming you can kick it to green ci :)

@davidradl
Copy link
Contributor

@ferenc-csaky fyi I see a duplicate #27606

@davidradl
Copy link
Contributor

@flinkbot run azure

@ferenc-csaky
Copy link
Contributor Author

The CI failure is unrelated and there is already a PR to cover that: #27619

Hence, I'll merge this if no objections until EOD today.

@pnowojski
Copy link
Contributor

Hey @ferenc-csaky a friendly reminder for the future. Next time please do not create duplicate PRs and do not supersede identical PRs that were created earlier. Especially when this was pointed out to you.

@ferenc-csaky
Copy link
Contributor Author

Now that @MartijnVisser confirmed on #27619 the cause of the CI failure (funny enough, same cause, but it's cause we bumped Azure runner images as well), I'm merging this.

@ferenc-csaky
Copy link
Contributor Author

@pnowojski I'm happy to merge the other one, there was just more context here and also on the Jira, so I opted for trackability. But it's a fair point.

@ferenc-csaky ferenc-csaky deleted the FLINK-39099 branch February 17, 2026 10:43
@pnowojski
Copy link
Contributor

pnowojski commented Feb 17, 2026

Yeah, it's a minor thing in this case, and I agree that I would be on the fence myself given that this PR has a jira ticket attached and the other doesn't, so it's neither a big nor a clear cut issue in this case. But we still need to be very careful when it comes to preserving and akcnolweding the authorship. Thank you for your understanding.

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.

5 participants