Skip to content

Commit

Permalink
Merge pull request #97 from juanjocop/master
Browse files Browse the repository at this point in the history
Adding nodeSelector option to execute only in nodes you need
  • Loading branch information
cfis authored Feb 26, 2024
2 parents 707c266 + a34fbe0 commit 3af72fa
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/docker-mailserver/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
appVersion: "13.3.1"
description: A fullstack but simple mailserver (smtp, imap, antispam, antivirus, ssl...) using Docker.
name: docker-mailserver
version: 3.0.3
version: 3.0.4
sources:
- https://github.com/docker-mailserver/docker-mailserver-helm
maintainers:
Expand Down
1 change: 1 addition & 0 deletions charts/docker-mailserver/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ spec:
{{ toYaml .Values.deployment.annotations | indent 8 }}
{{ end }}
spec:
nodeSelector: {{- toYaml .Values.deployment.nodeSelector | nindent 8 }}
runtimeClassName: {{ .Values.deployment.runtimeClassName }}
priorityClassName: {{ .Values.deployment.priorityClassName }}
serviceAccountName: {{ template "dockermailserver.serviceAccountName" . }}
Expand Down
3 changes: 3 additions & 0 deletions charts/docker-mailserver/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ metadata:
chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
heritage: "{{ .Release.Service }}"
release: "{{ .Release.Name }}"
{{- if .Values.service.labels }}
{{ toYaml .Values.service.labels | indent 2 }}
{{ end }}
name: {{ template "dockermailserver.fullname" . }}
spec:
## If a load balancer is being used, ensure that the newer type of LB that passes along IP information is used
Expand Down
4 changes: 4 additions & 0 deletions charts/docker-mailserver/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ deployment:
## Optionally specify a priorityClassName for the deployment
priorityClassName:

## Optionally specify a nodeSelector for the deployment
nodeSelector: {}

## Use the host’s network namespace. If this option is set, the ports that
## will be used must be specified.
## Ref: https://kubernetes.io/docs/api-reference/v1/definitions/#_v1_podspec
Expand Down Expand Up @@ -283,6 +286,7 @@ service:
## If there is a Hostname associated with this site, add it here and it will be rendered in the documentation.
# hostName:
annotations: {}
labels: {}

# Note this is a dictionary and not a list so invidual keys can be overriden by --set or --value helm parameters
persistence:
Expand Down

0 comments on commit 3af72fa

Please sign in to comment.