Skip to content

Conversation

@jalaziz
Copy link
Contributor

@jalaziz jalaziz commented Apr 16, 2025

Allow the default wait strategy to be overriden for docker compose environments. The default "listening ports" strategy doesn't work for distroless images and in complex setups, having to explicitly override the wait strategy for each service is error-prone.

@netlify
Copy link

netlify bot commented Apr 16, 2025

Deploy Preview for testcontainers-node ready!

Name Link
🔨 Latest commit 8a1cba7
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-node/deploys/6806daaf675d7c0008f3a08e
😎 Deploy Preview https://deploy-preview-990--testcontainers-node.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cristianrgreco cristianrgreco added enhancement New feature or request minor Backward compatible functionality labels Apr 17, 2025
Allow the default wait strategy to be overriden for docker compose
environments. The default "listening ports" strategy doesn't work for
distroless images and in complex setups, having to explicitly override
the wait strategy for each service is error-prone.
@jalaziz jalaziz force-pushed the compose-default-wait-strategy branch from f952c2d to ed0fbbf Compare April 21, 2025 02:46
@jalaziz
Copy link
Contributor Author

jalaziz commented Apr 21, 2025

pushed a change to fix the linting failure.

@jalaziz
Copy link
Contributor Author

jalaziz commented Apr 21, 2025

All tests passed with the previous commit. I just added docs to hopefully get this over the line 🙏🏼

@anapsix
Copy link

anapsix commented Apr 22, 2025

This change is extremely helpful, especially while working with distroless containers. Hoping to see it merged soon.

Copy link
Member

@kiview kiview left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @jalaziz, I think it is a nice convenient feature and we should consider porting it to other languages.

@cristianrgreco cristianrgreco merged commit 1a6102d into testcontainers:main Apr 29, 2025
256 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request minor Backward compatible functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants