Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MWS: extraVolumeMounts and extraVolumes implementation added #111

Merged
merged 3 commits into from
Sep 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion mywebmethodsserver/helm/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ helm install wm-mws mywebmethodsserver
| `1.0.2` | `storage.enabled` added in `values.yaml`. For backward reason is this value set to `true` as default. `false` doesn't create and mount any PV. (You can test this feature if you don't want to deploy any own MWS application. MWS is more stateless and has only the database as stateful dependency.) |
| `1.0.2` | `containerName` added in `values.yaml`. Default is the Chart name. (Use `helm repo update` to get latest Helm Chart version.) |
| `1.0.3` | `tpl` function support in `affinity` value added. `topologySpreadConstraints` support added. |
| `1.0.4` | `priorityClassName` support added. Implementation of `extraConfigMaps` added. |
| `1.0.4` | `priorityClassName` support added. Implementation of `extraConfigMaps`, `extraVolumeMounts` and `extraVolumes` added. |

## Values

Expand Down
2 changes: 1 addition & 1 deletion mywebmethodsserver/helm/README.md.gotmpl
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,6 @@ helm install wm-mws mywebmethodsserver
| `1.0.2` | `storage.enabled` added in `values.yaml`. For backward reason is this value set to `true` as default. `false` doesn't create and mount any PV. (You can test this feature if you don't want to deploy any own MWS application. MWS is more stateless and has only the database as stateful dependency.) |
| `1.0.2` | `containerName` added in `values.yaml`. Default is the Chart name. (Use `helm repo update` to get latest Helm Chart version.) |
| `1.0.3` | `tpl` function support in `affinity` value added. `topologySpreadConstraints` support added. |
| `1.0.4` | `priorityClassName` support added. Implementation of `extraConfigMaps` added. |
| `1.0.4` | `priorityClassName` support added. Implementation of `extraConfigMaps`, `extraVolumeMounts` and `extraVolumes` added. |

{{ template "chart.valuesSection" . }}
24 changes: 16 additions & 8 deletions mywebmethodsserver/helm/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -145,11 +145,26 @@ spec:
mountPath: {{ get $value "path" }}
{{- end }}
{{- end }}
{{- if .Values.extraVolumeMounts }}
{{- if eq "string" (printf "%T" .Values.extraVolumeMounts) }}
{{- tpl .Values.extraVolumeMounts . | nindent 10 }}
{{- else }}
{{- toYaml .Values.extraVolumeMounts | nindent 10 }}
{{- end }}
{{- end }}
resources:
{{- toYaml .Values.resources | nindent 12 }}
{{ if .Values.extraContainers }}
{{- if .Values.extraContainers }}
{{- toYaml .Values.extraContainers | nindent 8 }}
{{- end }}
volumes:
{{- if .Values.extraVolumes }}
{{- if eq "string" (printf "%T" .Values.extraVolumes) }}
{{- tpl .Values.extraVolumes . | nindent 8 }}
{{- else }}
{{- toYaml .Values.extraVolumes | nindent 8 }}
{{- end }}
{{- end }}
{{- with .Values.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
Expand All @@ -169,13 +184,6 @@ spec:
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- if .Values.extraVolumeMounts }}
{{- if eq "string" (printf "%T" .Values.extraVolumeMounts) }}
{{- tpl .Values.extraVolumeMounts . | nindent 6 }}
{{- else }}
{{- toYaml .Values.extraVolumeMounts | nindent 6 }}
{{- end }}
{{- end }}
volumeClaimTemplates:
{{- if .Values.storage.enabled }}
{{ $defaultVolumeNotMapped := true }}
Expand Down
Loading