Skip to content

inject secrets/config just before container is started #12970

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Jun 24, 2025

What I did

delay injection of secrets/config after dependencies have been resolved. This allows to avoid issue when an init container is used to setup a volume, and container mounts this volume with subpath. As CopyToContainer require the container's filesystem to be all set, this can't run on create but must wait for init container completion

Related issue

(not mandatory) A picture of a cute animal, if possible in relation to what you did

@ndeloof ndeloof requested a review from a team as a code owner June 24, 2025 09:59
@ndeloof ndeloof requested a review from glours June 24, 2025 09:59
@glours glours enabled auto-merge (rebase) June 24, 2025 10:00
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@glours glours merged commit aadce87 into docker:main Jun 24, 2025
25 checks passed
@ndeloof ndeloof deleted the late_inject branch June 24, 2025 10:26
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jun 25, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [docker/compose](https://github.com/docker/compose) | patch | `v2.37.2` -> `v2.37.3` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>docker/compose (docker/compose)</summary>

### [`v2.37.3`](https://github.com/docker/compose/releases/tag/v2.37.3)

[Compare Source](docker/compose@v2.37.2...v2.37.3)

#### What's Changed

##### ✨ Improvements

- add support for cache\_to with bake by [@&#8203;ndeloof](https://github.com/ndeloof) in docker/compose#12959

##### 🐛 Fixes

- fix bake intergation by [@&#8203;ndeloof](https://github.com/ndeloof) in docker/compose#12960
- don't create services passed as parameters of run command during dependencies creation process by [@&#8203;glours](https://github.com/glours) in docker/compose#12968
- inject secrets/config just before container is started by [@&#8203;ndeloof](https://github.com/ndeloof) in docker/compose#12970
- propagate target docker host set by --host to Bake by [@&#8203;ndeloof](https://github.com/ndeloof) in docker/compose#12961

##### 🔧  Internal

- pkg/compose: remove uses of ExecOptions.Detach by [@&#8203;thaJeztah](https://github.com/thaJeztah) in docker/compose#12950

##### ⚙️ Dependencies

- build(deps): bump github.com/moby/buildkit from 0.23.0 to 0.23.1 by [@&#8203;dependabot](https://github.com/dependabot) in docker/compose#12964

**Full Changelog**: docker/compose@v2.37.2...v2.37.3

</details>

---

### Configuration

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

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever MR is behind base branch, or you tick the rebase/retry checkbox.

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

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MC42Mi4xIiwidXBkYXRlZEluVmVyIjoiNDAuNjIuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90Il19-->
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