Skip to content

Commit

Permalink
[stable/kured] Allow setting imagePullSecrets and nodeSelector from v…
Browse files Browse the repository at this point in the history
…alues.yaml (helm#9590)

* [stable/kured] Allow setting `image.pullSecrets` and `nodeSelector` from values.yaml

Signed-off-by: William Hughes <[email protected]>

* Update Chart.yaml

Signed-off-by: Paul Czarkowski <[email protected]>
  • Loading branch information
insertjokehere authored and k8s-ci-robot committed Jan 9, 2019
1 parent bb47f58 commit f841959
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 1 deletion.
2 changes: 1 addition & 1 deletion stable/kured/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
appVersion: "1.1.0"
description: A Helm chart for kured
name: kured
version: 1.0.0
version: 1.1.0
home: https://github.com/weaveworks/kured
maintainers:
- name: plumdog
Expand Down
2 changes: 2 additions & 0 deletions stable/kured/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,14 @@ See https://github.com/weaveworks/kured
| `image.repository` | Image repository | `quay.io/weaveworks/kured` |
| `image.tag` | Image tag | `master-c42fff3` |
| `image.pullPolicy` | Image pull policy | `IfNotPresent` |
| `image.pullSecrets` | Image pull secrets | `[]` |
| `extraArgs` | Extra arguments to pass to `/usr/bin/kured`. See below. | `{}` |
| `rbac.create` | Create RBAC roles | `true` |
| `serviceAccount.create` | Create service account roles | `true` |
| `serviceAccount.name` | Service account name to create (or use if `serviceAccount.create` is false) | (chart fullname) |
| `updateStrategy` | Daemonset update strategy | `OnDelete` |
| `tolerations` | Tolerations to apply to the daemonset (eg to allow running on master) | `[{"key": "node-role.kubernetes.io/master", "effect": "NoSchedule"}]`|
| `nodeSelector` | Node Selector for the daemonset (ie, restrict which nodes kured runs on) | `{}` |
| `podAnnotations` | Annotations to apply to pods (eg to add Prometheus annotations) | `{}` |

See https://github.com/weaveworks/kured#configuration for values for `extraArgs`. Note that
Expand Down
8 changes: 8 additions & 0 deletions stable/kured/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,14 @@ spec:
serviceAccountName: {{ template "kured.serviceAccountName" . }}
hostPID: true
restartPolicy: Always
{{- with .Values.image.pullSecrets }}
imagePullSecrets:
{{ toYaml . | indent 8 }}
{{- end }}
{{- with .Values.nodeSelector }}
nodeSelector:
{{ toYaml . | indent 8 }}
{{- end }}
containers:
- name: {{ .Chart.Name }}
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
Expand Down
3 changes: 3 additions & 0 deletions stable/kured/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ image:
# Appears to be without numbered numbered tags, so using this instead
tag: 1.1.0
pullPolicy: IfNotPresent
pullSecrets: []

extraArgs: {}

Expand All @@ -19,4 +20,6 @@ tolerations:
- key: node-role.kubernetes.io/master
effect: NoSchedule

nodeSelector: {}

podAnnotations: {}

0 comments on commit f841959

Please sign in to comment.