tyler.durden/compose
1
0
Fork 0
mirror of https://github.com/docker/compose.git synced 2025-07-23 21:54:40 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix service: additional_contexts running internal buildkit client

Browse Source 
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
This commit is contained in:
Nicolas De Loof 2025-02-25 09:16:14 +01:00 committed by Nicolas De loof
parent 145bb8466d
commit e38b729a30
2 changed files with 28 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
pkg/compose/build.go
View File

@ -79,28 +79,7 @@ func (s *composeService) build(ctx context.Context, project *types.Project, opti
policy = types.IncludeDependencies policy = types.IncludeDependencies
} }
serviceDeps := false err := project.ForEachService(options.Services, func(serviceName string, service *types.ServiceConfig) error {
project, err := project.WithServicesTransform(func(serviceName string, service types.ServiceConfig) (types.ServiceConfig, error) {
if service.Build != nil {
for _, c := range service.Build.AdditionalContexts {
if t, found := strings.CutPrefix(c, types.ServicePrefix); found {
serviceDeps = true
if service.DependsOn == nil {
service.DependsOn = map[string]types.ServiceDependency{}
}
service.DependsOn[t] = types.ServiceDependency{
Condition: "build", // non-canonical, but will force dependency graph ordering
}
}
}
}
return service, nil
})
if err != nil {
return imageIDs, err
}
err = project.ForEachService(options.Services, func(serviceName string, service *types.ServiceConfig) error {
if service.Build == nil { if service.Build == nil {
return nil return nil
} }
@ -124,8 +103,24 @@ func (s *composeService) build(ctx context.Context, project *types.Project, opti
return s.doBuildBake(ctx, project, serviceToBuild, options) return s.doBuildBake(ctx, project, serviceToBuild, options)
} }
if serviceDeps { // Not using bake, additional_context: service:xx is implemented by building images in dependency order
logrus.Infof(`additional_context with "service:"" is better supported when delegating build go bake. Set COMPOSE_BAKE=true`) project, err = project.WithServicesTransform(func(serviceName string, service types.ServiceConfig) (types.ServiceConfig, error) {
if service.Build != nil {
for _, c := range service.Build.AdditionalContexts {
if t, found := strings.CutPrefix(c, types.ServicePrefix); found {
if service.DependsOn == nil {
service.DependsOn = map[string]types.ServiceDependency{}
}
service.DependsOn[t] = types.ServiceDependency{
Condition: "build", // non-canonical, but will force dependency graph ordering
}
}
}
}
return service, nil
})
if err != nil {
return imageIDs, err
} }
// Initialize buildkit nodes // Initialize buildkit nodes
@ -468,7 +463,7 @@ func (s *composeService) toBuildOptions(project *types.Project, service types.Se
ContextPath: service.Build.Context, ContextPath: service.Build.Context,
DockerfileInline: service.Build.DockerfileInline, DockerfileInline: service.Build.DockerfileInline,
DockerfilePath: dockerFilePath(service.Build.Context, service.Build.Dockerfile), DockerfilePath: dockerFilePath(service.Build.Context, service.Build.Dockerfile),
NamedContexts: toBuildContexts(service.Build.AdditionalContexts), NamedContexts: toBuildContexts(service, project),
}, },
CacheFrom: pb.CreateCaches(cacheFrom.ToPB()), CacheFrom: pb.CreateCaches(cacheFrom.ToPB()),
CacheTo: pb.CreateCaches(cacheTo.ToPB()), CacheTo: pb.CreateCaches(cacheTo.ToPB()),
@ -573,13 +568,15 @@ func getImageBuildLabels(project *types.Project, service types.ServiceConfig) ty
return ret return ret
} }
func toBuildContexts(additionalContexts types.Mapping) map[string]build.NamedContext { func toBuildContexts(service types.ServiceConfig, project *types.Project) map[string]build.NamedContext {
namedContexts := map[string]build.NamedContext{} namedContexts := map[string]build.NamedContext{}
for name, contextPath := range additionalContexts { for name, contextPath := range service.Build.AdditionalContexts {
if _, found := strings.CutPrefix(contextPath, types.ServicePrefix); found { if strings.HasPrefix(contextPath, types.ServicePrefix) {
// image we depend on has been build previously, as we run in dependency order. // image we depend on has been built previously, as we run in dependency order.
// this assumes use of docker engine builder, so that build can access local images // so we convert the service reference into an image reference
continue target := contextPath[len(types.ServicePrefix):]
image := api.GetImageNameOrDefault(project.Services[target], project.Name)
contextPath = "docker-image://" + image
} }
namedContexts[name] = build.NamedContext{Path: contextPath} namedContexts[name] = build.NamedContext{Path: contextPath}
} }

1
pkg/e2e/fixtures/build-dependencies/compose.yaml
View File

@ -1,6 +1,5 @@
services: services:
base: base:
image: base
init: true init: true
build: build:
context: . context: .