Skip to content
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

use NamedContext so buildkit is aware of cross service build dependencies #9324

Closed
wants to merge 1 commit into from

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Mar 30, 2022

What I did
Used NamedContexts so buildkit is aware of base image for a service being the actual result of another service build.

Related issue
closes #9232
closes #8538

…cies

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
milas added a commit to milas/compose that referenced this pull request Jun 22, 2022
When using the "classic" (non-BuildKit) builder, ensure that
services are iterated in dependency order for a build so that
it's possible to guarantee the presence of a base image that's
been added as a dependency with `depends_on`. This is a very
common pattern when using base images with Compose.

A fix for BuildKit is blocked currently until we can rely on a
newer version of the engine (see docker#9324)[^1].

[^1]: docker#9232 (comment)

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
debdutdeb pushed a commit to debdutdeb/compose that referenced this pull request Jun 30, 2022
When using the "classic" (non-BuildKit) builder, ensure that
services are iterated in dependency order for a build so that
it's possible to guarantee the presence of a base image that's
been added as a dependency with `depends_on`. This is a very
common pattern when using base images with Compose.

A fix for BuildKit is blocked currently until we can rely on a
newer version of the engine (see docker#9324)[^1].

[^1]: docker#9232 (comment)

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
@ndeloof
Copy link
Contributor Author

ndeloof commented Apr 17, 2023

closing as obsolete

@ndeloof ndeloof closed this Apr 17, 2023
@ndeloof ndeloof deleted the namedcontext branch December 20, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant