Skip to content

Commit

Permalink
Make worker annotations configurable (#122)
Browse files Browse the repository at this point in the history
* Make worker annotations configurable

* Annotations are a dict not a list

* Rerun frigate
  • Loading branch information
jacobtomlinson authored Nov 6, 2020
1 parent b3c6b39 commit 7a20cf4
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions dask/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ The following table lists the configurable parameters of the Dask chart and thei
| `worker.extraArgs` | Extra cli arguments to be passed to the worker | `[]` |
| `worker.resources` | Worker pod resources. see `values.yaml` for example values. | `{}` |
| `worker.mounts` | Worker pod volumes and volume mounts, mounts.volumes follows kuberentes api v1 volumes spec. mounts.volumemounts follows kubernetesapi v1 volumemount spec | `{}` |
| `worker.annotations` | Annotations | `{}` |
| `worker.tolerations` | Tolerations. | `[]` |
| `worker.affinity` | Container affinity. | `{}` |
| `worker.nodeSelector` | Node selector. | `{}` |
Expand Down
4 changes: 4 additions & 0 deletions dask/templates/dask-worker-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ spec:
app: {{ template "dask.name" . }}
release: {{ .Release.Name | quote }}
component: worker
{{- if .Values.worker.annotations }}
annotations:
{{- toYaml .Values.worker.annotations | nindent 8 }}
{{- end }}
spec:
imagePullSecrets:
{{- toYaml .Values.worker.image.pullSecrets | nindent 8 }}
Expand Down
1 change: 1 addition & 0 deletions dask/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@ worker:
# volumeMounts:
# - name: data
# mountPath: /data
annotations: {} # Annotations
tolerations: [] # Tolerations.
affinity: {} # Container affinity.
nodeSelector: {} # Node Selector.
Expand Down

0 comments on commit 7a20cf4

Please sign in to comment.