Skip to content

fix: CI docker testcontainers failing#27139

Closed
czentgr wants to merge 1 commit intoprestodb:masterfrom
czentgr:cz_fix_test_container
Closed

fix: CI docker testcontainers failing#27139
czentgr wants to merge 1 commit intoprestodb:masterfrom
czentgr:cz_fix_test_container

Conversation

@czentgr
Copy link
Copy Markdown
Contributor

@czentgr czentgr commented Feb 13, 2026

The CI started failing with errors related to the docker that is installed with the github runner image.

For example,

2026-02-13T05:02:16.962-0600 INFO Testcontainers version: 1.20.5
2026-02-13T05:02:17.345-0600 INFO docker-machine executable was not found on PATH ([/opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.15-6/x64/bin, /snap/bin, /home/runner/.local/bin, /opt/pipx_bin, /home/runner/.cargo/bin, /home/runner/.config/composer/vendor/bin, /usr/local/.ghcup/bin, /home/runner/.dotnet/tools, /usr/local/sbin, /usr/local/bin, /usr/sbin, /usr/bin, /sbin, /bin, /usr/games, /usr/local/games, /snap/bin])
2026-02-13T05:02:17.346-0600 SEVERE Could not find a valid Docker environment. Please check configuration. Attempted configurations were:
Error:  
Cannot instantiate class com.facebook.presto.plugin.singlestore.TestSingleStoreTypeMapping
	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"}
)

As a result we upgrade to the latest testcontainer version available for version 1.x. We assume something in the github runner image changed with respect to docker and it is addressed in the latest available version applicable.

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.
  • If adding new dependencies, verified they have an OpenSSF Scorecard score of 5.0 or higher (or obtained explicit TSC approval for lower scores).

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== NO RELEASE NOTE ==

@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Feb 13, 2026
@czentgr czentgr marked this pull request as ready for review February 13, 2026 21:30
@czentgr czentgr requested a review from a team as a code owner February 13, 2026 21:30
@prestodb-ci prestodb-ci requested review from a team, agrawalreetika and infvg and removed request for a team February 13, 2026 21:30
@czentgr
Copy link
Copy Markdown
Contributor Author

czentgr commented Feb 13, 2026

@amitkdutta FYI

Copy link
Copy Markdown
Contributor

@amitkdutta amitkdutta left a comment

Choose a reason for hiding this comment

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

Thanks a lot @czentgr for the quick fix!

@czentgr
Copy link
Copy Markdown
Contributor Author

czentgr commented Feb 13, 2026

@tdcmeehan Tim can you please review?

@imjalpreet
Copy link
Copy Markdown
Member

Looks like another PR upgraded it to 2.0.3: #27140

@czentgr
Copy link
Copy Markdown
Contributor Author

czentgr commented Feb 16, 2026

Yes. Thanks. Closing this PR.

@czentgr czentgr closed this Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:IBM PR from IBM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants