Turn everything else off in Kubernetes cluster probe

weaveworks · Oct 12, 2018 · fb96fe0 · fb96fe0
1 parent 78eaf93
commit fb96fe0
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/prog/probe.go b/prog/probe.go
@@ -108,6 +108,21 @@ func probeMain(flags probeFlags, targets []appclient.Target) {
 	logCensoredArgs()
 	defer log.Info("probe exiting")
 
+	switch flags.kubernetesRole {
+	case "": // nothing special
+	case kubernetesRoleHost:
+		flags.kubernetesEnabled = true
+	case kubernetesRoleCluster:
+		flags.kubernetesKubeletPort = 0
+		flags.kubernetesEnabled = true
+		flags.spyProcs = false
+		flags.procEnabled = false
+		flags.useConntrack = false
+		flags.useEbpfConn = false
+	default:
+		log.Warnf("unrecognized --probe.kubernetes.role: %s", flags.kubernetesRole)
+	}
+
 	if flags.spyProcs && os.Getegid() != 0 {
 		log.Warn("--probe.proc.spy=true, but that requires root to find everything")
 	}
@@ -233,17 +248,6 @@ func probeMain(flags probeFlags, targets []appclient.Target) {
 	defer endpointReporter.Stop()
 	p.AddReporter(endpointReporter)
 
-	switch flags.kubernetesRole {
-	case "": // nothing special
-	case kubernetesRoleHost:
-		flags.kubernetesEnabled = true
-	case kubernetesRoleCluster:
-		flags.kubernetesKubeletPort = 0
-		flags.kubernetesEnabled = true
-	default:
-		log.Warnf("unrecognized --probe.kubernetes.role: %s", flags.kubernetesRole)
-	}
-
 	if flags.dockerEnabled {
 		// Don't add the bridge in Kubernetes since container IPs are global and
 		// shouldn't be scoped