Add priorityClassName values to operator and connect

1Password · Aug 13, 2024 · 0dd27cc · 0dd27cc
1 parent 282df93
commit 0dd27cc
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 2 deletions.
diff --git a/charts/connect/README.md b/charts/connect/README.md
@@ -83,6 +83,7 @@ helm install --set connect.applicationName=connect connect ./connect
 | connect.ingress.pathType            | string     | `"Prefix"`                         | Ingress PathType see [docs](https://kubernetes.io/docs/concepts/services-networking/ingress/#path-types)                                                                                           |
 | connect.ingress.tls                 | list       | `[]`                               | Ingress TLS see [docs](https://kubernetes.io/docs/concepts/services-networking/ingress/#tls)                                                                                                       |
 | connect.nodeSelector                | object     | `{}`                               | [Node selector](https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#nodeselector) stanza for the Connect pod                                                                  |
+| connect.priorityClassName           | string | `""` | priorityClassName to apply to the Connect API deployment resource. |
 | connect.probes.readiness            | boolean    | `true`                             | Denotes whether the 1Password Connect API readiness probe will operate and ensure the pod is ready before serving traffic                                                                          |
 | connect.probes.liveness             | boolean    | `true`                             | Denotes whether the 1Password Connect API will be continually checked by Kubernetes for liveness and restarted if the pod becomes unresponsive                                                     |
 | connect.annotations                 | object     | `{}`                               | Additional annotations to be added to the Connect API deployment resource.                                                                                                                         |
@@ -112,6 +113,7 @@ helm install --set connect.applicationName=connect connect ./connect
 | operator.podAnnotations             | object     | `{}`                               | Additional annotations to be added to the Operator pods.                                                                                                                                           |
 | operator.podLabels                  | object     | `{}`                               | Additional labels to be added to the Operator pods.                                                                                                                                                |
 | operator.pollingInterval            | integer    | `600`                              | How often the 1Password Operator will poll for secrets updates.                                                                                                                                    |
+| operator.priorityClassName          | string | `""` | priorityClassName to apply to the Operator pods. |
 | operator.clusterRole.create         | boolean    | `{{.Values.operator.create}}`      | Denotes whether or not a cluster role will be created for each for the 1Password Operator                                                                                                          |
 | operator.clusterRole.name           | string     | `"onepassword-connect-operator"`   | The name of the 1Password Operator Cluster Role                                                                                                                                                    |
 | operator.clusterRoleBinding.create  | boolean    | `{{.Values.operator.create}}`      | Denotes whether or not a Cluster role binding will be created for the 1Password Operator Service Account                                                                                           |

diff --git a/charts/connect/templates/connect-deployment.yaml b/charts/connect/templates/connect-deployment.yaml
@@ -37,6 +37,9 @@ spec:
       nodeSelector:
 {{ toYaml . | indent 8 }}
 {{- end }}
+      {{- if .Values.connect.priorityClassName }}
+      priorityClassName: {{ .Values.connect.priorityClassName }}
+      {{- end }}
       volumes:
         - name: {{ .Values.connect.dataVolume.name }}
           {{ .Values.connect.dataVolume.type }}: {{- toYaml .Values.connect.dataVolume.values | nindent 12 }}
@@ -87,7 +90,7 @@ spec:
             {{- range .Values.connect.customEnvVars}}
             - name: {{ .name }}
               value: {{ .value }}
-            {{- end }} 
+            {{- end }}
           {{- if .Values.connect.probes.readiness }}
           readinessProbe:
             httpGet:

diff --git a/charts/connect/templates/operator-deployment.yaml b/charts/connect/templates/operator-deployment.yaml
@@ -36,6 +36,9 @@ spec:
       nodeSelector:
 {{ toYaml . | indent 8 }}
 {{- end }}
+      {{- if .Values.operator.priorityClassName }}
+      priorityClassName: {{ .Values.operator.priorityClassName }}
+      {{- end }}
       tolerations:
 {{ toYaml .Values.operator.tolerations | indent 8 }}
       serviceAccountName: {{ .Values.operator.serviceAccount.name }}
@@ -75,7 +78,7 @@ spec:
             {{- range .Values.operator.customEnvVars}}
             - name: {{ .name }}
               value: {{ .value }}
-            {{- end }} 
+            {{- end }}
           resources:
             {{- toYaml .Values.operator.resources | nindent 12 }}
 {{- end }}
diff --git a/charts/connect/values.yaml b/charts/connect/values.yaml
@@ -107,6 +107,9 @@ connect:
     # by Kubernetes for liveness and restarted if the pod becomes unresponsive
     readiness: true
 
+  # priorityClassName to apply to the Connect API deployment resource.
+  priorityClassName: ''
+
   # Additional annotations to be added to the Connect API deployment resource.
   annotations: {}
 
@@ -234,6 +237,9 @@ operator:
   # Additional labels to be added to the Operator pods.
   podLabels: {}
 
+  # priorityClassName to apply to the Operator pods.
+  priorityClassName: ''
+
   # List of tolerations to be added to the Operator pods.
   tolerations: []