Skip to content

Comments

fix: upgraded testcontainers test dependency#359

Merged
cpswan merged 3 commits intotrunkfrom
fix_docker_api_issue
Feb 16, 2026
Merged

fix: upgraded testcontainers test dependency#359
cpswan merged 3 commits intotrunkfrom
fix_docker_api_issue

Conversation

@akafredperry
Copy link
Collaborator

@akafredperry akafredperry commented Feb 16, 2026

closes #358

- What I did
The integration tests which operate against the virtual env docker container use the project testcontainers
The version we were using was unable to work with the updated version of docker in the github workflow
I upgrade the version of testcontainers

- How I did it

  • Updated the version property in the parent pom
  • Added echo of docker version to ci workflow (useful to always have this)
  • Added debug logging to simpleLogger.properties (commented out but useful for any future issues)
  • Added comment to VirtualEnv.java with regards to windows dev (the upgraded testcontainers will NOT work with windows desktop docker and neither can I get it to work with dockerd running in wsl. This is not critical just annoying).

- How to verify it
This is verified by the ci workflow for the PR

- Description for the changelog

upgraded testcontainers version to fix github docker inter-operability

@akafredperry akafredperry self-assigned this Feb 16, 2026
akafredperry added 2 commits February 16, 2026 11:14
@akafredperry akafredperry changed the title fix: echo DOCKER_API_VERSION fix: upgraded testcontainers test dependency Feb 16, 2026
@akafredperry akafredperry marked this pull request as ready for review February 16, 2026 13:32
@akafredperry akafredperry requested a review from cpswan February 16, 2026 13:32
@cpswan cpswan merged commit 4568477 into trunk Feb 16, 2026
4 checks passed
@cpswan cpswan deleted the fix_docker_api_issue branch February 16, 2026 14:04
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.

Integration test workflow failing

2 participants