This is an automated email from the ASF dual-hosted git repository.
mbalassi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/flink-kubernetes-operator.git
The following commit(s) were added to refs/heads/main by this push:
new dc173cbc [FLINK-30329] Mount flink-operator-config-volume at
/opt/flink/conf without subPath
dc173cbc is described below
commit dc173cbccd5f9a90fe573ce01790684253dcc0a9
Author: Ottomata <ao...@wikimedia.org>
AuthorDate: Wed Dec 7 14:48:15 2022 -0500
[FLINK-30329] Mount flink-operator-config-volume at /opt/flink/conf without
subPath
When mounted volumes use subPath, the volume that uses them is not updated
automatically.
So, even you do update the ConfigMap in k8s, the flink-conf.yaml file will
never see those updates.
All of the entires in the flink-operator-config-volume configMap are
mounted in each
container anyway, so we might as well just mount the
flink-operator-config-volume
directly at /opt/flink/conf witihout specifying the subPath of each file.
This allows for Dynamic Operator Configuration to work when deployed with
these helm charts.
https://nightlies.apache.org/flink/flink-kubernetes-operator-docs-main/docs/operations/configuration/#dynamic-operator-configuration
---
.../templates/flink-operator.yaml | 18 ++----------------
1 file changed, 2 insertions(+), 16 deletions(-)
diff --git a/helm/flink-kubernetes-operator/templates/flink-operator.yaml
b/helm/flink-kubernetes-operator/templates/flink-operator.yaml
index f94c2198..f8da1543 100644
--- a/helm/flink-kubernetes-operator/templates/flink-operator.yaml
+++ b/helm/flink-kubernetes-operator/templates/flink-operator.yaml
@@ -100,14 +100,7 @@ spec:
{{- toYaml .Values.operatorSecurityContext | nindent 12 }}
volumeMounts:
- name: flink-operator-config-volume
- mountPath: /opt/flink/conf/flink-conf.yaml
- subPath: flink-conf.yaml
- - name: flink-operator-config-volume
- mountPath: /opt/flink/conf/log4j-operator.properties
- subPath: log4j-operator.properties
- - name: flink-operator-config-volume
- mountPath: /opt/flink/conf/log4j-console.properties
- subPath: log4j-console.properties
+ mountPath: /opt/flink/conf
{{- if .Values.operatorVolumeMounts.create }}
{{- toYaml .Values.operatorVolumeMounts.data | nindent 12 }}
{{- end }}
@@ -167,14 +160,7 @@ spec:
mountPath: "/certs"
readOnly: true
- name: flink-operator-config-volume
- mountPath: /opt/flink/conf/flink-conf.yaml
- subPath: flink-conf.yaml
- - name: flink-operator-config-volume
- mountPath: /opt/flink/conf/log4j-operator.properties
- subPath: log4j-operator.properties
- - name: flink-operator-config-volume
- mountPath: /opt/flink/conf/log4j-console.properties
- subPath: log4j-console.properties
+ mountPath: /opt/flink/conf
{{- end }}
{{- if index (.Values.operatorPod) "dnsPolicy" }}
dnsPolicy: {{ .Values.operatorPod.dnsPolicy | quote }}
