Skip to content

Commit

Permalink
feat(helm): add RollingUpdate parameters (#20931)
Browse files Browse the repository at this point in the history
Co-authored-by: gforien <gforien.externe@bedrockstreaming.com>
  • Loading branch information
gforien and gforien authored Sep 22, 2022
1 parent e671d80 commit 3f8e9a5
Show file tree
Hide file tree
Showing 5 changed files with 27 additions and 1 deletion.
2 changes: 1 addition & 1 deletion helm/superset/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ maintainers:
- name: craig-rueda
email: craig@craigrueda.com
url: https://github.com/craig-rueda
version: 0.7.3
version: 0.7.4
dependencies:
- name: postgresql
version: 11.1.22
Expand Down
4 changes: 4 additions & 0 deletions helm/superset/templates/deployment-worker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,10 @@ spec:
matchLabels:
app: {{ template "superset.name" . }}-worker
release: {{ .Release.Name }}
{{- if .Values.supersetWorker.strategy }}
strategy:
{{- toYaml .Values.supersetWorker.strategy | nindent 4 }}
{{- end }}
template:
metadata:
annotations:
Expand Down
4 changes: 4 additions & 0 deletions helm/superset/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ metadata:
{{- end }}
spec:
replicas: {{ .Values.supersetNode.replicaCount }}
{{- if .Values.supersetNode.strategy }}
strategy:
{{- toYaml .Values.supersetNode.strategy | nindent 4 }}
{{- end }}
selector:
matchLabels:
app: {{ template "superset.name" . }}
Expand Down
6 changes: 6 additions & 0 deletions helm/superset/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -343,6 +343,9 @@
},
"containerSecurityContext": {
"type": "object"
},
"strategy": {
"$ref": "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.23.0/_definitions.json##/definitions/io.k8s.api.apps.v1.DeploymentStrategy"
}
},
"required": [
Expand Down Expand Up @@ -386,6 +389,9 @@
},
"containerSecurityContext": {
"type": "object"
},
"strategy": {
"$ref": "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.23.0/_definitions.json##/definitions/io.k8s.api.apps.v1.DeploymentStrategy"
}
},
"required": [
Expand Down
12 changes: 12 additions & 0 deletions helm/superset/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,12 @@ supersetNode:
# memory: 128Mi
podSecurityContext: {}
containerSecurityContext: {}
strategy: {}
# type: RollingUpdate
# rollingUpdate:
# maxSurge: 25%
# maxUnavailable: 25%

##
## Superset worker configuration
supersetWorker:
Expand Down Expand Up @@ -322,6 +328,12 @@ supersetWorker:
# memory: 128Mi
podSecurityContext: {}
containerSecurityContext: {}
strategy: {}
# type: RollingUpdate
# rollingUpdate:
# maxSurge: 25%
# maxUnavailable: 25%

##
## Superset beat configuration (to trigger scheduled jobs like reports)
supersetCeleryBeat:
Expand Down

0 comments on commit 3f8e9a5

Please sign in to comment.