Skip to content

Commit

Permalink
Merge pull request #30 from grzesuav/ffffff
Browse files Browse the repository at this point in the history
fix: Fix Daemonset schema, update strategy placed incorrectly
  • Loading branch information
apetruhin committed Sep 6, 2024
2 parents c67312e + 7928e3a commit e20fcdd
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
11 changes: 6 additions & 5 deletions charts/node-agent/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ metadata:
labels:
{{- include "node-agent.labels" . | nindent 4 }}
spec:
{{- with .Values.updateStrategy }}
updateStrategy:
{{- toYaml . | nindent 4 }}
{{- end }}
selector:
matchLabels:
{{- include "node-agent.selectorLabels" . | nindent 6 }}
Expand All @@ -25,10 +29,6 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- include "node-agent.affinity" . | nindent 6 }}
{{- with .Values.updateStrategy }}
updateStrategy:
{{- toYaml . | nindent 8 }}
{{- end }}
tolerations:
- operator: Exists
priorityClassName: {{ .Values.priorityClassName }}
Expand Down Expand Up @@ -101,4 +101,5 @@ spec:
path: /sys/kernel/debug
name: debugfs
- emptyDir: {}
name: tmp
name: tmp

2 changes: 1 addition & 1 deletion charts/node-agent/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ fullnameOverride: ""
updateStrategy:
type: RollingUpdate
rollingUpdate:
maxUnavailable: 25%
maxUnavailable: 1
podAnnotations: {}

priorityClassName: ""
Expand Down

0 comments on commit e20fcdd

Please sign in to comment.