Better error check for no collectors

cmd/troubleshoot/cli/run.go

@@ -74,11 +74,6 @@ func runTroubleshoot(v *viper.Viper, args []string) error {
 		return nil
 	}
 
-	// Check if we have any collectors to run in the support bundle specs
-	if len(mainBundle.Spec.Collectors) == 0 && len(mainBundle.Spec.HostCollectors) == 0 {
-		return errors.New("no collectors specified to run")
-	}
-
 	interactive := v.GetBool("interactive") && isatty.IsTerminal(os.Stdout.Fd())
 
 	if interactive {
@@ -273,6 +268,14 @@ func loadSpecs(ctx context.Context, args []string, client kubernetes.Interface)
 		kinds.Add(moreKinds)
 	}
 
+	// Check if we have any collectors to run in the troubleshoot specs
+	// TODO: Do we use the RemoteCollectors anymore?
+	if len(kinds.CollectorsV1Beta2) == 0 &&
+		len(kinds.HostCollectorsV1Beta2) == 0 &&
+		len(kinds.SupportBundlesV1Beta2) == 0 {
+		return nil, nil, errors.New("no collectors specified to run")
+	}
+
 	// Merge specs
 	// We need to add the default type information to the support bundle spec
 	// since by default these fields would be empty