[AMORO-3178]Change the default configuration of values.yaml to improv…

…e user experience
apache · Sep 6, 2024 · 78a4b8a · 78a4b8a
1 parent 48e3645
commit 78a4b8a
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