Merge pull request #164 from consideRatio/pr/style-non-running-pods

Make non-running pods panels consistent and style them intuitively

Author: consideRatio

dashboards/cluster.jsonnet

@@ -403,22 +403,23 @@ local nodeOOMKills =
   ]);
 
 local nonRunningPods =
-  common.barChartOptions
-  + barChart.new('Pods not in Running state')
-  + barChart.panelOptions.withDescription(
+  common.tsOptions
+  + common.tsPodStateStylingOverrides
+  + ts.new('Non Running Pods')
+  + ts.panelOptions.withDescription(
     |||
-      Pods in states other than 'Running'.
+      Pods in a non-running state in the hub's namespace.
 
-      In a functional clusters, pods should not be in non-Running states for long.
-    |||,
+      Pods stuck in non-running states often indicate an error condition
+    |||
   )
-  + ts.fieldConfig.defaults.custom.stacking.withMode('normal')
-  + barChart.standardOptions.withDecimals(0)
-  + barChart.queryOptions.withTargets([
+  + ts.queryOptions.withTargets([
     prometheus.new(
       '$PROMETHEUS_DS',
       |||
-        sum(kube_pod_status_phase{phase!="Running"}) by (phase)
+        sum(
+          kube_pod_status_phase{phase!="Running"}
+        ) by (phase)
       |||
     )
     + prometheus.withLegendFormat('{{phase}}'),

dashboards/common.libsonnet

@@ -98,6 +98,61 @@ local _getDashedLineOverride(pattern, color) = {
     ])
   ,
 
+  tsPodStateStylingOverrides:
+    ts.standardOptions.withOverrides([
+      {
+        matcher: { id: 'byName', options: 'Pending' },
+        properties: [{
+          id: 'color',
+          value: {
+            fixedColor: 'yellow',
+            mode: 'fixed',
+          },
+        }],
+      },
+      {
+        matcher: { id: 'byName', options: 'Running' },
+        properties: [{
+          id: 'color',
+          value: {
+            fixedColor: 'blue',
+            mode: 'fixed',
+          },
+        }],
+      },
+      {
+        matcher: { id: 'byName', options: 'Succeeded' },
+        properties: [{
+          id: 'color',
+          value: {
+            fixedColor: 'green',
+            mode: 'fixed',
+          },
+        }],
+      },
+      {
+        matcher: { id: 'byName', options: 'Unknown' },
+        properties: [{
+          id: 'color',
+          value: {
+            fixedColor: 'orange',
+            mode: 'fixed',
+          },
+        }],
+      },
+      {
+        matcher: { id: 'byName', options: 'Failed' },
+        properties: [{
+          id: 'color',
+          value: {
+            fixedColor: 'red',
+            mode: 'fixed',
+          },
+        }],
+      },
+    ])
+  ,
+
   // grafonnet ref: https://grafana.github.io/grafonnet/API/dashboard/variable.html
   variables: {
     prometheus:

dashboards/jupyterhub.jsonnet

@@ -230,6 +230,7 @@ local usersPerNode =
 
 local nonRunningPods =
   common.tsOptions
+  + common.tsPodStateStylingOverrides
   + ts.new('Non Running Pods')
   + ts.panelOptions.withDescription(
     |||
@@ -238,8 +239,6 @@ local nonRunningPods =
       Pods stuck in non-running states often indicate an error condition
     |||
   )
-  // decimalsY1=0,
-  + ts.fieldConfig.defaults.custom.stacking.withMode('normal')
   + ts.queryOptions.withTargets([
     prometheus.new(
       '$PROMETHEUS_DS',
@@ -265,7 +264,6 @@ local sharedVolumeFreeSpace =
       what extra deployment is needed.
     |||
   )
-  // decimalsY1=0,
   + ts.standardOptions.withMax(1)
   + ts.standardOptions.withUnit('percentunit')
   + ts.queryOptions.withTargets([