Skip to content

feat: add image hook to run commands before ready#766

Merged
DDtKey merged 1 commit intotestcontainers:mainfrom
yuri-becker:main
Feb 6, 2025
Merged

feat: add image hook to run commands before ready#766
DDtKey merged 1 commit intotestcontainers:mainfrom
yuri-becker:main

Conversation

@yuri-becker
Copy link
Contributor

@yuri-becker yuri-becker commented Feb 5, 2025

Hey there, i was implementing a module for Dex and needed something like containerIsStarting in the Java implementation.
I wrote a test and confirmed it works with my module implementation – see testcontainers/testcontainers-rs-modules-community#286 for that PR.

Please let me know what i can improve to get this merged. I hope it is okay to implement this without an issue :)

@netlify
Copy link

netlify bot commented Feb 5, 2025

Deploy Preview for testcontainers-rust ready!

Name Link
🔨 Latest commit 1979a44
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-rust/deploys/67a497562699470008d6aa94
😎 Deploy Preview https://deploy-preview-766--testcontainers-rust.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.

Copy link
Contributor

@DDtKey DDtKey left a comment

Choose a reason for hiding this comment

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

That looks pretty clear too me and aligned with the current interface 👍

Thanks a lot for your contribution and well explained rationale 🚀

LGTM (just one comment to consider)

@DDtKey DDtKey changed the title feat: Add image hook to run commands before ready feat: add image hook to run commands before ready Feb 5, 2025
@DDtKey DDtKey enabled auto-merge (squash) February 6, 2025 00:10
auto-merge was automatically disabled February 6, 2025 11:04

Head branch was pushed to by a user without write access

@DDtKey DDtKey enabled auto-merge (squash) February 6, 2025 14:15
@DDtKey DDtKey merged commit b979cde into testcontainers:main Feb 6, 2025
12 checks passed
@github-actions github-actions bot mentioned this pull request Feb 6, 2025
@yuri-becker
Copy link
Contributor Author

Thanks for merging :)

DDtKey pushed a commit that referenced this pull request Feb 7, 2025
## 🤖 New release

* `testcontainers`: 0.23.2

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

## [0.23.2] - 2025-02-06

### Details
#### Documentation
- Add examples of `ImageExt` usage and cover `GenericImage` (#747)

#### Features
- Add `cap_add` and `cap_drop` support (#726)
- Add support for user-configured labels (#756)
- Reusable containers (#757)
- Add image hook to run commands before ready (#766)

#### Miscellaneous Tasks
- Update thiserror requirement from 1.0.60 to 2.0.3 (#754)
- Bump the bollard dep to latest (#759)

#### Testing
- Deflake the test `start_containers_in_parallel` (#748)
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
DDtKey added a commit to testcontainers/testcontainers-rs-modules-community that referenced this pull request Feb 10, 2025
Module for [Dex](https://dexidp.io/). A community module for Java exists
as well: https://github.com/Kehrlann/testcontainers-dex/tree/main

Dependent on this PR in the core crate:
testcontainers/testcontainers-rs#766

---------

Co-authored-by: Artem Medvedev <i@ddtkey.com>
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.

2 participants

Comments