fix: fall back to helm error when no pods failed (#115)

airbytehq · Sep 16, 2024 · cf6fc76 · cf6fc76
1 parent 9c63a57
commit cf6fc76
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/internal/cmd/local/local/install.go b/internal/cmd/local/local/install.go
@@ -316,7 +316,7 @@ func (c *Command) diagnoseAirbyteChartFailure(ctx context.Context, chartErr erro
 
 	if podList, err := c.k8s.PodList(ctx, airbyteNamespace); err == nil {
 
-		errors := []string{}
+		var errors []string
 		for _, pod := range podList.Items {
 			c.tel.Attr(fmt.Sprintf("pod-%s-phase", pod.Name), string(pod.Status.Phase))
 
@@ -338,7 +338,9 @@ func (c *Command) diagnoseAirbyteChartFailure(ctx context.Context, chartErr erro
 				errors = append(errors, fmt.Sprintf("pod %s: %s", pod.Name, msg))
 			}
 		}
-		return fmt.Errorf("unable to install airbyte chart:\n%s", strings.Join(errors, "\n"))
+		if errors != nil {
+			return fmt.Errorf("unable to install airbyte chart:\n%s", strings.Join(errors, "\n"))
+		}
 	}
 	return fmt.Errorf("unable to install airbyte chart: %w", chartErr)
 }