fix linux node exporter

Author: MikeZappa87

clusterloader2/pkg/prometheus/manifests/exporters/node_exporter/linux-exporter-podmonitor.yaml

@@ -0,0 +1,19 @@
+apiVersion: monitoring.coreos.com/v1
+kind: PodMonitor
+metadata:
+  name: node-exporter
+  namespace: monitoring
+  labels:
+    app: node-exporter  # must match your Prometheus CR's `podMonitorSelector`
+spec:
+  selector:
+    matchLabels:
+      app: node-exporter
+  namespaceSelector:
+    matchNames:
+      - monitoring
+  podMetricsEndpoints:
+    - port: metrics
+      path: /metrics
+      interval: 5s
+

clusterloader2/pkg/prometheus/manifests/exporters/node_exporter/node-exporter-daemonset.yaml

@@ -0,0 +1,94 @@
+apiVersion: v1
+kind: ServiceAccount
+metadata:
+  name: node-exporter
+  namespace: monitoring
+---
+apiVersion: rbac.authorization.k8s.io/v1
+kind: ClusterRole
+metadata:
+  name: node-exporter
+rules:
+  - apiGroups: [""]
+    resources:
+      - nodes
+      - nodes/proxy
+      - services
+      - endpoints
+      - pods
+    verbs: ["get", "list", "watch"]
+  - apiGroups: [""]
+    resources:
+      - nodes/metrics
+    verbs: ["get"]
+---
+apiVersion: rbac.authorization.k8s.io/v1
+kind: ClusterRoleBinding
+metadata:
+  name: node-exporter
+roleRef:
+  apiGroup: rbac.authorization.k8s.io
+  kind: ClusterRole
+  name: node-exporter
+subjects:
+  - kind: ServiceAccount
+    name: node-exporter
+    namespace: monitoring
+---
+apiVersion: apps/v1
+kind: DaemonSet
+metadata:
+  name: node-exporter
+  namespace: monitoring
+  labels:
+    app: node-exporter
+spec:
+  selector:
+    matchLabels:
+      app: node-exporter
+  template:
+    metadata:
+      labels:
+        app: node-exporter
+    spec:
+      serviceAccountName: node-exporter
+      hostNetwork: true
+      hostPID: true
+      containers:
+        - name: node-exporter
+          image: quay.io/prometheus/node-exporter:latest
+          args:
+            - "--path.procfs=/host/proc"
+            - "--path.sysfs=/host/sys"
+            - "--collector.disable-defaults"
+            - "--collector.cpu"
+            - "--collector.meminfo"
+            - "--collector.loadavg"
+            - "--collector.stat"
+          ports:
+            - containerPort: 9100
+              hostPort: 9100
+              name: metrics
+          resources:
+            limits:
+              memory: 100Mi
+            requests:
+              cpu: 100m
+              memory: 50Mi
+          volumeMounts:
+            - name: proc
+              mountPath: /host/proc
+              readOnly: true
+            - name: sys
+              mountPath: /host/sys
+              readOnly: true
+      nodeSelector:
+        kubernetes.io/os: linux
+      volumes:
+        - name: proc
+          hostPath:
+            path: /proc
+        - name: sys
+          hostPath:
+            path: /sys
+      terminationGracePeriodSeconds: 30

clusterloader2/pkg/prometheus/manifests/exporters/node_exporter/node-exporter.yaml

@@ -27,7 +27,6 @@ import (
 	"sync"
 	"time"
 
-	"golang.org/x/sync/errgroup"
 	corev1 "k8s.io/api/core/v1"
 	rbacv1 "k8s.io/api/rbac/v1"
 	apierrs "k8s.io/apimachinery/pkg/api/errors"
@@ -56,7 +55,7 @@ const (
 	kubeStateMetricsManifests    = "exporters/kube-state-metrics/*.yaml"
 	masterIPServiceMonitors      = "master-ip/*.yaml"
 	metricsServerManifests       = "exporters/metrics-server/*.yaml"
-	nodeExporterPod              = "exporters/node_exporter/node-exporter.yaml"
+	nodeExporterManifests        = "exporters/node_exporter/*.yaml"
 	windowsNodeExporterManifests = "exporters/windows_node_exporter/*.yaml"
 	pushgatewayManifests         = "pushgateway/*.yaml"
 )
@@ -236,7 +235,7 @@ func (pc *Controller) SetUpPrometheusStack() error {
 		return err
 	}
 	if pc.clusterLoaderConfig.PrometheusConfig.ScrapeNodeExporter {
-		if err := pc.runNodeExporter(); err != nil {
+		if err := pc.applyDefaultManifests(nodeExporterManifests); err != nil {
 			return err
 		}
 	}
@@ -395,46 +394,6 @@ func (pc *Controller) exposeAPIServerMetrics() error {
 	return nil
 }
 
-// runNodeExporter adds node-exporter as master's static manifest pod.
-// TODO(mborsz): Consider migrating to something less ugly, e.g. daemonset-based approach,
-// when master nodes have configured networking.
-func (pc *Controller) runNodeExporter() error {
-	klog.V(2).Infof("Starting node-exporter on master nodes.")
-	kubemarkFramework, err := framework.NewFramework(&pc.clusterLoaderConfig.ClusterConfig, numK8sClients)
-	if err != nil {
-		return err
-	}
-
-	// Validate masters first
-	nodes, err := client.ListNodes(kubemarkFramework.GetClientSets().GetClient())
-	if err != nil {
-		return err
-	}
-
-	var g errgroup.Group
-	numMasters := 0
-	for _, node := range nodes {
-		node := node
-		if util.LegacyIsMasterNode(&node) || util.IsControlPlaneNode(&node) {
-			numMasters++
-			g.Go(func() error {
-				f, err := manifestsFS.Open(nodeExporterPod)
-				if err != nil {
-					return fmt.Errorf("unable to open manifest file: %v", err)
-				}
-				defer f.Close()
-				return pc.ssh.Exec("sudo tee /etc/kubernetes/manifests/node-exporter.yaml > /dev/null", &node, f)
-			})
-		}
-	}
-
-	if numMasters == 0 {
-		return fmt.Errorf("node-exporter requires master to be registered nodes")
-	}
-
-	return g.Wait()
-}
-
 func (pc *Controller) waitForPrometheusToBeHealthy() error {
 	klog.V(2).Info("Waiting for Prometheus stack to become healthy...")
 	return wait.PollImmediate(