[Improvement][AMORO-3178]: Change the default configuration of values…

….yaml to improve user experience (#3179) * [Doc]Fix typo in managing-optimizers.md * [AMORO-3178]Change the default configuration of values.yaml to improve user experience --------- Co-authored-by: zhaoshuaihua <[email protected]> Co-authored-by: ZhouJinsong <[email protected]>
apache · Oct 12, 2024 · 0ffdec2 · 0ffdec2
1 parent 3e17628
commit 0ffdec2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/charts/amoro/templates/_pod.tpl b/charts/amoro/templates/_pod.tpl
@@ -120,7 +120,7 @@ spark distribution package will be installed to here*/ -}}
 {{- if .Values.plugin.metricReporters }}
 {{- if .Values.plugin.metricReporters.prometheusExporter.enabled }}
 - name: prometheus
-  containerPort: {{ .Values.plugin.metricReporters.prometheusExporter.port }}
+  containerPort: {{ .Values.plugin.metricReporters.prometheusExporter.properties.port }}
 {{- end -}}
 {{- end -}}
 {{- end -}}

diff --git a/charts/amoro/values.yaml b/charts/amoro/values.yaml
@@ -282,12 +282,12 @@ plugin:
 #      ## @param Enable or disable the Prometheus exporter
 #      ##
 #      enabled: true
-#      ## @param Prometheus port
-#      ##
-#      port: 7001
 #      ## @param Properties for the Prometheus exporter
 #      ##
-#      properties: ~
+#      properties:
+#      ## @param Prometheus port
+#      ##
+#        port: 7001
 
 ## Configure the ingress resource that allows you to access the
 ## Amoro installation. Set up the URL