Skip to content

Commit 0de0bd0

Browse files
authored
Added node selector for helm chart (#185)
Issue #, if available: Description of changes: Added nodSelector label to allow options in case cluster has window and linux nodes. Set default value to linux node. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 3242121 commit 0de0bd0

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

templates/helm/templates/deployment.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,3 +77,4 @@ spec:
7777
- name: ACK_RESOURCE_TAGS
7878
value: {{ join "," .Values.resourceTags | quote }}
7979
terminationGracePeriodSeconds: 10
80+
nodeSelector: {{ toYaml .Values.deployment.nodeSelector | nindent 8 }}

templates/helm/values.yaml.tpl

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ deployment:
1515
annotations: {}
1616
labels: {}
1717
containerPort: 8080
18+
nodeSelector:
19+
kubernetes.io/os: linux
1820

1921
metrics:
2022
service:

0 commit comments

Comments
 (0)