Ensure eager variable replacement happens for dependencies too

Author: mcastellini

cmd/print.go

@@ -88,13 +88,10 @@ func (cmd *PrintCmd) Run(f factory.Factory) error {
 		log.Warnf("Unable to create new kubectl client: %v", err)
 	}
 
-	localCache, err := configLoader.LoadLocalCache()
-	if err != nil {
-		return err
-	}
+	parser := loader.NewEagerParser()
 
 	// load config
-	config, err := configLoader.LoadWithParser(context.Background(), localCache, client, loader.NewEagerParser(), configOptions, log)
+	config, err := configLoader.LoadWithParser(context.Background(), nil, client, parser, configOptions, log)
 	if err != nil {
 		return err
 	}
@@ -105,7 +102,7 @@ func (cmd *PrintCmd) Run(f factory.Factory) error {
 		WithKubeClient(client)
 
 	// resolve dependencies
-	dependencies, err := dependency.NewManager(ctx, configOptions).ResolveAll(ctx, dependency.ResolveOptions{})
+	dependencies, err := dependency.NewManagerWithParser(ctx, configOptions, parser).ResolveAll(ctx, dependency.ResolveOptions{})
 	if err != nil {
 		log.Warnf("Error resolving dependencies: %v", err)
 	}

pkg/devspace/dependency/manager.go

@@ -31,6 +31,12 @@ func NewManager(ctx devspacecontext.Context, configOptions *loader.ConfigOptions
 	}
 }
 
+func NewManagerWithParser(ctx devspacecontext.Context, configOptions *loader.ConfigOptions, parser loader.Parser) Manager {
+	return &manager{
+		resolver: NewResolver(ctx, configOptions).WithParser(parser),
+	}
+}
+
 type ResolveOptions struct {
 	SkipDependencies []string
 	Dependencies     []string

pkg/devspace/dependency/resolver.go

@@ -24,6 +24,7 @@ import (
 // ResolverInterface defines the resolver interface that takes dependency configs and resolves them
 type ResolverInterface interface {
 	Resolve(ctx devspacecontext.Context) ([]types.Dependency, error)
+	WithParser(parser loader.Parser) ResolverInterface
 }
 
 // Resolver implements the resolver interface
@@ -32,6 +33,7 @@ type resolver struct {
 
 	BaseCache  localcache.Cache
 	BaseConfig *latest.Config
+	BaseParser loader.Parser
 
 	ConfigOptions *loader.ConfigOptions
 }
@@ -90,6 +92,16 @@ func (r *resolver) Resolve(ctx devspacecontext.Context) ([]types.Dependency, err
 	return children, nil
 }
 
+func (r *resolver) WithParser(parser loader.Parser) ResolverInterface {
+	if r == nil {
+		return nil
+	}
+
+	n := *r
+	n.BaseParser = parser
+	return &n
+}
+
 func (r *resolver) resolveRecursive(ctx devspacecontext.Context, basePath, parentConfigName string, currentDependency *Dependency, dependencies []*latest.DependencyConfig) error {
 	if currentDependency != nil {
 		currentDependency.children = []types.Dependency{}
@@ -206,7 +218,11 @@ func (r *resolver) resolveDependency(ctx devspacecontext.Context, dependencyConf
 			return err
 		}
 
-		dConfigWrapper, err = configLoader.Load(ctx.Context(), client, cloned, ctx.Log())
+		if r.BaseParser == nil {
+			dConfigWrapper, err = configLoader.Load(ctx.Context(), client, cloned, ctx.Log())
+		} else {
+			dConfigWrapper, err = configLoader.LoadWithParser(ctx.Context(), nil, client, r.BaseParser, cloned, ctx.Log())
+		}
 		if err != nil {
 			return errors.Wrap(err, fmt.Sprintf("loading config for dependency %s", dependencyName))
 		}