feat: Add possibility to apply priorityClassName to all deployments (…

…#485)
FlowFuse · Oct 28, 2024 · 2482fb4 · 2482fb4
2 parents 469c821 + d858276
commit 2482fb4
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 0 deletions.
diff --git a/helm/flowforge/README.md b/helm/flowforge/README.md
@@ -51,6 +51,7 @@ For other values please refer to the documentation below.
  - `forge.podLabels` allows to add custom labels to the core application pod (default `{}`)
  - `forge.replicas` allows the number of instances of the FlowFuse App to be set. Scaling only supported with ingress-nginx controller (default `1`)
  - `forge.tolerations` allows to configure [tolerations](https://kubernetes.io/docs/concepts/scheduling-eviction/taint-and-toleration/) for the core application deployment (default `[]`)
+ - `forge.priorityClassName` allows to set [priorityClassName](https://kubernetes.io/docs/concepts/configuration/pod-priority-preemption/) for all deployments created by this Helm chart (default not set)
 
 
 note: `forge.projectSelector` and `forge.managementSelector` defaults mean that you must have at least 2 nodes in your cluster and they need to be labeled before installing.

diff --git a/helm/flowforge/templates/broker.yaml b/helm/flowforge/templates/broker.yaml
@@ -28,6 +28,9 @@ spec:
         {{- toYaml . | nindent 8 }}
         {{- end }}
     spec:
+      {{- if .Values.forge.priorityClassName }}
+      priorityClassName: "{{ .Values.forge.priorityClassName}}"
+      {{- end }}
       automountServiceAccountToken: false
       securityContext:
         {{- toYaml .Values.forge.broker.podSecurityContext | nindent 8 }}

diff --git a/helm/flowforge/templates/deployment.yaml b/helm/flowforge/templates/deployment.yaml
@@ -35,6 +35,9 @@ spec:
         prometheus.io/path: "/metrics"
         {{- end  }}
     spec:
+      {{- if .Values.forge.priorityClassName }}
+      priorityClassName: "{{ .Values.forge.priorityClassName}}"
+      {{- end }}
       serviceAccountName: flowforge
       automountServiceAccountToken: true 
       securityContext:

diff --git a/helm/flowforge/templates/file-storage.yml b/helm/flowforge/templates/file-storage.yml
@@ -49,6 +49,9 @@ spec:
         prometheus.io/path: "/metrics"
         {{- end  }}
     spec:
+      {{- if .Values.forge.priorityClassName }}
+      priorityClassName: "{{ .Values.forge.priorityClassName}}"
+      {{- end }}
       automountServiceAccountToken: false
       securityContext:
         {{- toYaml .Values.forge.fileStore.podSecurityContext | nindent 8 }}

diff --git a/helm/flowforge/values.schema.json b/helm/flowforge/values.schema.json
@@ -528,6 +528,9 @@
                                 }
                             }
                         },
+                        "priorityClassName": {
+                            "type": "string"
+                        },
                         "livenessProbe": {
                             "type": "object",
                             "properties": {

diff --git a/helm/flowforge/values.yaml b/helm/flowforge/values.yaml
@@ -127,6 +127,8 @@ forge:
 
   tolerations: []
 
+  priorityClassName: ""
+
   logPassthrough: false
   customHostname:
     enabled: false