Skip to content

Commit aadce87

Browse files
ndeloofglours
authored andcommitted
inject secrets/config just before container is started
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
1 parent b3207c4 commit aadce87

2 files changed

Lines changed: 20 additions & 9 deletions

File tree

pkg/compose/convergence.go

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -756,14 +756,7 @@ func (s *composeService) createMobyContainer(ctx context.Context,
756756
}
757757
}
758758
}
759-
760-
err = s.injectSecrets(ctx, project, service, created.ID)
761-
if err != nil {
762-
return created, err
763-
}
764-
765-
err = s.injectConfigs(ctx, project, service, created.ID)
766-
return created, err
759+
return created, nil
767760
}
768761

769762
// getLinks mimics V1 compose/service.py::Service::_get_links()
@@ -897,6 +890,17 @@ func (s *composeService) startService(ctx context.Context,
897890
if ctr.State == ContainerRunning {
898891
continue
899892
}
893+
894+
err = s.injectSecrets(ctx, project, service, ctr.ID)
895+
if err != nil {
896+
return err
897+
}
898+
899+
err = s.injectConfigs(ctx, project, service, ctr.ID)
900+
if err != nil {
901+
return err
902+
}
903+
900904
eventName := getContainerProgressName(ctr)
901905
w.Event(progress.StartingEvent(eventName))
902906
err = s.apiClient().ContainerStart(ctx, ctr.ID, containerType.StartOptions{})

pkg/compose/run.go

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,14 @@ func (s *composeService) prepareRun(ctx context.Context, project *types.Project,
127127
if err != nil {
128128
return "", err
129129
}
130-
return created.ID, nil
130+
131+
err = s.injectSecrets(ctx, project, service, created.ID)
132+
if err != nil {
133+
return created.ID, err
134+
}
135+
136+
err = s.injectConfigs(ctx, project, service, created.ID)
137+
return created.ID, err
131138
}
132139

133140
func applyRunOptions(project *types.Project, service *types.ServiceConfig, opts api.RunOptions) {

0 commit comments

Comments
 (0)