Skip to content

Commit

Permalink
deploy: fix bug of ceph-csi-cephfs helm chart
Browse files Browse the repository at this point in the history
fix bug that make ceph-csi-cephfs-provisioner get dup affinities when deploy helm chart ceph-csi-cephfs.
  • Loading branch information
dashjay committed May 10, 2023
1 parent 1f5ad24 commit ebc0f0c
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions charts/ceph-csi-cephfs/templates/provisioner-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,14 @@ spec:
values:
- {{ .Values.provisioner.name }}
topologyKey: "kubernetes.io/hostname"
{{- if .Values.provisioner.affinity }}
{{ toYaml .Values.provisioner.affinity | indent 8 -}}
{{- end -}}
{{- else -}}
{{- if .Values.provisioner.affinity }}
affinity:
{{ toYaml .Values.provisioner.affinity | indent 8 -}}
{{- end -}}
{{- end }}
serviceAccountName: {{ include "ceph-csi-cephfs.serviceAccountName.provisioner" . }}
hostNetwork: {{ .Values.provisioner.enableHostNetwork }}
Expand Down Expand Up @@ -228,10 +236,6 @@ spec:
emptyDir: {
medium: "Memory"
}
{{- if .Values.provisioner.affinity }}
affinity:
{{ toYaml .Values.provisioner.affinity | indent 8 -}}
{{- end -}}
{{- if .Values.provisioner.nodeSelector }}
nodeSelector:
{{ toYaml .Values.provisioner.nodeSelector | indent 8 -}}
Expand Down

0 comments on commit ebc0f0c

Please sign in to comment.