Skip to content

Commit

Permalink
Update pods on config change (#17)
Browse files Browse the repository at this point in the history
**Description** <Describe what changed.>
This PR ensures that Orchestrators are restarted when the config
changes. This is different from config reloading when using an external
bucket for config.

Note: This could eventually be configurable, so it can be toggled off
when `MAVERICS_RELOAD_CONFIG: true`; however, this setting is commonly
used with remote config, so this toggle was omitted for the near-term
fix.

**Testing** <Describe how you tested the change.>
Tested against a cluster.

**Documentation** <Describe any documentation that was added.>
N/A
  • Loading branch information
gramidt committed Aug 22, 2023
1 parent e84f249 commit 68405fd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/orchestrator/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ type: application
# This is the chart version. This version number should be incremented each time we
# make changes to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.4
version: 0.1.5

# This is the version number of the application being deployed. This version number
# should be incremented each time we make changes to the application. Versions are
Expand Down
3 changes: 2 additions & 1 deletion charts/orchestrator/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,11 @@ spec:
type: {{ .Values.updateStrategy }}
template:
metadata:
{{- with .Values.podAnnotations }}
annotations:
{{- with .Values.podAnnotations }}
{{- toYaml . | nindent 8 }}
{{- end }}
checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
labels:
{{- include "orchestrator.selectorLabels" . | nindent 8 }}
spec:
Expand Down

0 comments on commit 68405fd

Please sign in to comment.