Skip to content

Allow overriding default compose wait strategy#990

Merged
cristianrgreco merged 2 commits intotestcontainers:mainfrom
jalaziz:compose-default-wait-strategy
Apr 29, 2025
Merged

Allow overriding default compose wait strategy#990
cristianrgreco merged 2 commits intotestcontainers:mainfrom
jalaziz:compose-default-wait-strategy

Conversation

@jalaziz
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Contributor Author

jalaziz commented Apr 21, 2025

pushed a change to fix the linting failure.

@jalaziz
Copy link
Copy Markdown
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
Copy Markdown

anapsix commented Apr 22, 2025

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

Copy link
Copy Markdown
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
vaernion pushed a commit to Arbeidstilsynet/brevgen2 that referenced this pull request Dec 3, 2025
….25.0

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [testcontainers](https://github.com/testcontainers/testcontainers-node) | [`10.24.2` -> `10.25.0`](https://renovatebot.com/diffs/npm/testcontainers/10.24.2/10.25.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/testcontainers/10.25.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/testcontainers/10.25.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/testcontainers/10.24.2/10.25.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/testcontainers/10.24.2/10.25.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>testcontainers/testcontainers-node (testcontainers)</summary>

### [`v10.25.0`](https://github.com/testcontainers/testcontainers-node/releases/tag/v10.25.0)

[Compare Source](testcontainers/testcontainers-node@v10.24.2...v10.25.0)

#### Changes

#### 🚀 Features

-   Allow overriding default compose wait strategy [@&#8203;jalaziz](https://github.com/jalaziz) ([#&#8203;990](testcontainers/testcontainers-node#990))
-   Add ClickHouse module [@&#8203;Victorlouisdg](https://github.com/Victorlouisdg) ([#&#8203;981](testcontainers/testcontainers-node#981))

#### 📦 Dependency Updates

-   Bump vite from 6.2.5 to 6.3.4 [@&#8203;dependabot](https://github.com/dependabot) ([#&#8203;992](testcontainers/testcontainers-node#992))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or PR is renamed to start with "rebase!".

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).

Related work items: #30372
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