Merge pull request #30961 from alex-hunt-materialize/support_alternative_schedulers

orchestratord support alternative schedulers

Author: alex-hunt-materialize

misc/helm-charts/operator/README.md

@@ -143,6 +143,7 @@ The following table lists the configurable parameters of the Materialize operato
 | `operator.resources.requests` | Resources requested by the operator for CPU and memory | ``{"cpu":"100m","memory":"512Mi"}`` |
 | `operator.secretsController` | Which secrets controller to use for storing secrets. Valid values are 'kubernetes' and 'aws-secrets-manager'. Setting 'aws-secrets-manager' requires a configured AWS cloud provider and IAM role for the environment with Secrets Manager permissions. | ``"kubernetes"`` |
 | `rbac.create` | Whether to create necessary RBAC roles and bindings | ``true`` |
+| `schedulerName` | Optionally use a non-default kubernetes scheduler. | ``nil`` |
 | `serviceAccount.create` | Whether to create a new service account for the operator | ``true`` |
 | `serviceAccount.name` | The name of the service account to be created | ``"orchestratord"`` |
 | `storage.storageClass.allowVolumeExpansion` |  | ``false`` |

misc/helm-charts/operator/templates/deployment.yaml

@@ -150,5 +150,8 @@ spec:
         {{- end }}
         {{- end }}
         - "--enable-security-context"
+        {{- if .Values.schedulerName }}
+        - "--scheduler-name={{ .Values.schedulerName }}"
+        {{- end }}
         resources:
           {{- toYaml .Values.operator.resources | nindent 10 }}

misc/helm-charts/operator/tests/deployment_test.yaml

@@ -178,3 +178,17 @@ tests:
   - contains:
       path: spec.template.spec.containers[0].args
       content: "--secrets-controller=aws-secrets-manager"
+
+- it: should not pass the scheduler when not configured
+  asserts:
+  - notContains:
+      path: spec.template.spec.containers[0].args
+      content: "--scheduler-name"
+
+- it: should pass the scheduler when configured
+  set:
+    schedulerName: my-scheduler
+  asserts:
+  - contains:
+      path: spec.template.spec.containers[0].args
+      content: "--scheduler-name=my-scheduler"

misc/helm-charts/operator/values.yaml

@@ -230,6 +230,9 @@ rbac:
   # -- Whether to create necessary RBAC roles and bindings
   create: true
 
+# -- Optionally use a non-default kubernetes scheduler.
+schedulerName: null
+
 # Service account settings
 serviceAccount:
   # -- Whether to create a new service account for the operator