Fix Preflight CheckRBAC (#776)

* return collect result instead of nil

Author: diamonwiggins

pkg/preflight/collect.go

@@ -170,7 +170,8 @@ func Collect(opts CollectOpts, p *troubleshootv1beta2.Preflight) (CollectResult,
 	}
 
 	if foundForbidden && !opts.IgnorePermissionErrors {
-		return nil, errors.New("insufficient permissions to run all collectors")
+		collectResult.isRBACAllowed = false
+		return collectResult, errors.New("insufficient permissions to run all collectors")
 	}
 
 	// generate a map of all collectors for atomic status messages

preflight_default.yaml

@@ -4,8 +4,13 @@ metadata:
   name: modules
 spec:
   collectors:
-    - cpu: {}
-    # - kernelModules: {}
+    - logs:
+        selector:
+          - app=svclb-traefik
+        namespace: kube-system
+        name: backend/container/logs
+        limits:
+          maxLines: 1000
   analyzers:
     - clusterVersion:
         outcomes: