fix: don't read outputs for site without components

mach-composer · Jan 21, 2025 · 9279aa1 · 9279aa1
1 parent d3705ac
commit 9279aa1
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/.changes/unreleased/Fixed-20250121-122320.yaml b/.changes/unreleased/Fixed-20250121-122320.yaml
@@ -0,0 +1,3 @@
+kind: Fixed
+body: Don't read outputs for site without components
+time: 2025-01-21T12:23:20.045132931+01:00
diff --git a/internal/runner/utils.go b/internal/runner/utils.go
@@ -2,6 +2,7 @@ package runner
 
 import (
 	"context"
+	"fmt"
 	"github.com/mach-composer/mach-composer-cli/internal/graph"
 	"github.com/mach-composer/mach-composer-cli/internal/utils"
 	"github.com/rs/zerolog/log"
@@ -32,6 +33,18 @@ func terraformCanPlan(ctx context.Context, n graph.Node) (bool, error) {
 	}
 
 	for _, parent := range parents {
+		if parent.Type() == graph.SiteType {
+			sp, ok := parent.(*graph.Site)
+			if !ok {
+				return false, fmt.Errorf("failed to cast parent to site")
+			}
+
+			if len(sp.NestedNodes) == 0 {
+				log.Debug().Msgf("site node does not contain components, so no output is available")
+				continue
+			}
+		}
+
 		v, err := utils.GetTerraformOutputs(ctx, parent.Path())
 		if err != nil {
 			return false, nil