Merge pull request #7197 from GeorgianaElena/rm-alert

Rm alert

Author: GeorgianaElena

helm-charts/support/values.jsonnet

@@ -38,32 +38,6 @@ function(VARS_2I2C_AWS_ACCOUNT_ID=null)
     },
   };
 
-  local makeServerStartupFailureAlert = function(
-    summary,
-    severity,
-    labels={},
-                                        ) {
-    // Structure is documented in https://prometheus.io/docs/prometheus/latest/configuration/alerting_rules/
-    alert: 'One server failed to start',
-    expr: |||
-      # We trigger any time there is a server startup failure, for any reason.
-      # The 'max' is to reduce the labels being passed to only the necessary ones
-      max by (namespace) (
-          (jupyterhub_server_spawn_duration_seconds_count{status="failure"} > 0)
-        -
-          ((jupyterhub_server_spawn_duration_seconds_count{status="failure"} offset 2m) > 0)
-      ) > 0
-    |||,
-    'for': '1m',
-    labels: {
-      cluster: cluster_name,
-      severity: severity,
-    } + labels,
-    annotations: {
-      summary: summary,
-    },
-  };
-
   local makeTwoServersStartupFailureAlert = function(
     summary,
     severity,
@@ -241,10 +215,6 @@ function(VARS_2I2C_AWS_ACCOUNT_ID=null)
             {
               name: 'Server Startup Failure',
               rules: [
-                makeServerStartupFailureAlert(
-                  'A server failed to start: cluster %s hub:{{ $labels.namespace }}' % [cluster_name],
-                  'same day action needed'
-                ),
                 makeTwoServersStartupFailureAlert(
                   'At least two servers have failed to start in the last hour: cluster %s hub:{{ $labels.namespace }}' % [cluster_name],
                   'immediate action needed'