Skip to content

Commit 3851249

Browse files
committed
fix: move ensureNamespace to pipeline's dependency start
Signed-off-by: Luca Di Maio <[email protected]>
1 parent e22e3f8 commit 3851249

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

pkg/devspace/pipeline/pipeline.go

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -223,10 +223,6 @@ func (p *pipeline) StartNewDependencies(ctx devspacecontext.Context, dependencie
223223
continue
224224
}
225225

226-
err := ensureNamespace(ctx, dependency.DependencyConfig().Namespace)
227-
if err != nil {
228-
return errors.Wrapf(err, "cannot run dependency %s", dependency.Name())
229-
}
230226
deployDependencies = append(deployDependencies, dependency)
231227
}
232228

@@ -388,6 +384,13 @@ func (p *pipeline) startNewDependency(ctx devspacecontext.Context, dependency ty
388384
pipelineConfig *latest.Pipeline
389385
err error
390386
)
387+
388+
// Ensure dependency namespace exists
389+
err = ensureNamespace(ctx, dependency.DependencyConfig().Namespace)
390+
if err != nil {
391+
return errors.Wrapf(err, "cannot run dependency %s", dependency.Name())
392+
}
393+
391394
if dependency.Config().Config().Pipelines == nil || dependency.Config().Config().Pipelines[executePipeline] == nil {
392395
pipelineConfig, err = types.GetDefaultPipeline(executePipeline)
393396
if err != nil {

0 commit comments

Comments
 (0)