Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@idsulik
Copy link
Collaborator

@idsulik idsulik commented Oct 28, 2025

What I did
Fixed a bug where services with pull_policy: build were built twice during docker compose run when they were dependencies of the target service. The issue occurred because ensureImagesExists was called twice - once for dependencies in startDependencies and again for the full project in prepareRun. Modified prepareRun to restrict the second ensureImagesExists call to only build the target service by filtering BuildOptions.Services.

Related issue
#13323

image

@idsulik idsulik requested review from a team as code owners October 28, 2025 14:15
@idsulik idsulik requested review from glours and ndeloof October 28, 2025 14:15
@idsulik
Copy link
Collaborator Author

idsulik commented Oct 28, 2025

It's strange... the tests work fine locally

@idsulik
Copy link
Collaborator Author

idsulik commented Oct 28, 2025

Looks like it was a parallel issue. Should I return the parallel execution and try to fix the issue, or is it ok to leave non-parallel execution?

@glours
Copy link
Contributor

glours commented Oct 28, 2025

Yes fix it please, use c := NewParallelCLI(t) instead in your test

This reverts commit e4f4a5a.

Signed-off-by: Suleiman Dibirov <[email protected]>
…es for Docker Compose commands.

Signed-off-by: Suleiman Dibirov <[email protected]>
@idsulik
Copy link
Collaborator Author

idsulik commented Oct 28, 2025

done

Copy link
Contributor

@glours glours left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @idsulik 🙏

@glours glours merged commit fc74c78 into docker:main Oct 29, 2025
24 checks passed
@idsulik idsulik deleted the issue-13323 branch October 29, 2025 10:26
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Nov 3, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [docker/compose](https://github.com/docker/compose) | patch | `v2.40.2` -> `v2.40.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.40.3`](https://github.com/docker/compose/releases/tag/v2.40.3)

[Compare Source](docker/compose@v2.40.2...v2.40.3)

#### What's Changed

##### 🐛 Fixes

- Fix OCI compose override support by [@&#8203;ndeloof](https://github.com/ndeloof) [#&#8203;13311](docker/compose#13311)
- Fix help output for "exec --no-tty" option by [@&#8203;tonyo](https://github.com/tonyo) [#&#8203;13314](docker/compose#13314)
- Prompt default implementation to prevent a panic by [@&#8203;ndeloof](https://github.com/ndeloof) [#&#8203;13317](docker/compose#13317)
- Run hooks on restart by [@&#8203;ndeloof](https://github.com/ndeloof) [#&#8203;13321](docker/compose#13321)
- Fix(run): Ensure images exist only for the target service in run command by [@&#8203;idsulik](https://github.com/idsulik) [#&#8203;13325](docker/compose#13325)
- Fix(git): Fix path traversal vulnerability in git remote loader  by [@&#8203;idsulik](https://github.com/idsulik) [#&#8203;13331](docker/compose#13331)

##### 🔧  Internal

- Test to check writeComposeFile detects invalid OCI artifact by [@&#8203;ndeloof](https://github.com/ndeloof) [#&#8203;13309](docker/compose#13309)
- Code Cleanup by [@&#8203;ndeloof](https://github.com/ndeloof) [#&#8203;13315](docker/compose#13315)

##### ⚙️ Dependencies

- Bump compose-go to version v2.9.1 by [@&#8203;glours](https://github.com/glours) [#&#8203;13332](docker/compose#13332)

**Full Changelog**: <docker/compose@v2.40.2...v2.40.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:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNjkuMSIsInVwZGF0ZWRJblZlciI6IjQxLjE2OS4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
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.

3 participants