update README/values; Add test for schedulerName

Signed-off-by: Sathya Balakrishnan <sathya@portworx.com>
elastic · Jun 3, 2019 · e2c5b6b · e2c5b6b
1 parent 1b6baf9
commit e2c5b6b
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/elasticsearch/README.md b/elasticsearch/README.md
@@ -100,7 +100,7 @@ helm install --name elasticsearch elastic/elasticsearch --version 7.1.0 --set im
 | `nodeSelector`             | Configurable [nodeSelector](https://kubernetes.io/docs/concepts/configuration/assign-pod-node/#nodeselector) so that you can target specific nodes for your Elasticsearch cluster                                                                                                                                          | `{}`                                                                                                                      |
 | `tolerations`              | Configurable [tolerations](https://kubernetes.io/docs/concepts/configuration/taint-and-toleration/)                                                                                                                                                                                                                        | `[]`                                                                                                                      |
 | `ingress`                  | Configurable [ingress](https://kubernetes.io/docs/concepts/services-networking/ingress/) to expose the Elasticsearch service. See [`values.yaml`](./values.yaml) for an example                                                                                                                                            | `enabled: false`                                                                                                          |
-| `schedulerName`            | Name of the alternate scheduler (e.g. stork)                                                                                                | `nil`  |
+| `schedulerName`            | Name of the [alternate scheduler](https://kubernetes.io/docs/tasks/administer-cluster/configure-multiple-schedulers/#specify-schedulers-for-pods)  | `nil`  |
 
 ## Try it out
 

diff --git a/elasticsearch/tests/elasticsearch_test.py b/elasticsearch/tests/elasticsearch_test.py
@@ -569,3 +569,15 @@ def test_priority_class_name():
     r = helm_template(config)
     priority_class_name = r['statefulset'][uname]['spec']['template']['spec']['priorityClassName']
     assert priority_class_name == "highest"
+
+
+def test_scheduler_name():
+    r = helm_template('')
+    spec = r['statefulset'][uname]['spec']['template']['spec']
+    assert 'schedulerName' not in spec
+
+    config = '''
+schedulerName: "stork"
+'''
+    r = helm_template(config)
+    assert r['statefulset'][uname]['spec']['template']['spec']['schedulerName'] == "stork"
diff --git a/elasticsearch/values.yaml b/elasticsearch/values.yaml
@@ -142,7 +142,7 @@ readinessProbe:
 # https://www.elastic.co/guide/en/elasticsearch/reference/current/cluster-health.html#request-params wait_for_status
 clusterHealthCheckParams: "wait_for_status=green&timeout=1s"
 
-## Use an alternate scheduler, e.g. "stork".
+## Use an alternate scheduler.
 ## ref: https://kubernetes.io/docs/tasks/administer-cluster/configure-multiple-schedulers/
 ##
 # schedulerName: