Merge pull request #22 from commandemy/fix-issues

added patch and updated service monitor
infralovers · Jun 22, 2021 · bf51134 · bf51134
2 parents 40589ea + af37aeb
commit bf51134
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 3 deletions.
diff --git a/examples/12-factor-app-v2/k8s/patch-cluster-role.yaml b/examples/12-factor-app-v2/k8s/patch-cluster-role.yaml
@@ -0,0 +1,20 @@
+- apiGroups:
+  - ""
+  resources:
+  - nodes/metrics
+  verbs:
+  - get
+- nonResourceURLs:
+  - /metrics
+  verbs:
+  - get
+- apiGroups:
+  - ""
+  resources:
+  - services
+  - pods
+  - endpoints
+  verbs:
+  - get
+  - list
+  - watch
diff --git a/examples/12-factor-app-v2/k8s/service-monitor-traefik.yaml b/examples/12-factor-app-v2/k8s/service-monitor-traefik.yaml
@@ -8,6 +8,7 @@ spec:
     matchLabels:
       app: traefik
   namespaceSelector:
-    name: 12-factor-app      
+    matchNames:
+      - 12-factor-app
   endpoints:
   - port: metrics
diff --git a/examples/12-factor-app-v2/k8s/service-monitor.yaml b/examples/12-factor-app-v2/k8s/service-monitor.yaml
@@ -8,6 +8,7 @@ spec:
     matchLabels:
       app: api
   namespaceSelector:
-    name: 12-factor-app      
+    matchNames:
+      - 12-factor-app
   endpoints:
-  - port: metrics # Here we use the name of the port defined earlier
+  - port: metrics # Here we use the name of the port defined earlier