This is an automated email from the ASF dual-hosted git repository.
czy006 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/amoro.git
The following commit(s) were added to refs/heads/master by this push:
new 0ffdec22b [Improvement][AMORO-3178]: Change the default configuration
of values.yaml to improve user experience (#3179)
0ffdec22b is described below
commit 0ffdec22b3834172115094d866f6d0ae0ef47575
Author: berylzsh <[email protected]>
AuthorDate: Sat Oct 12 14:22:45 2024 +0800
[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]>
---
charts/amoro/templates/_pod.tpl | 2 +-
charts/amoro/values.yaml | 8 ++++----
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/charts/amoro/templates/_pod.tpl b/charts/amoro/templates/_pod.tpl
index 7fe844be3..721860306 100644
--- 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
index 74e71e6fe..acb4fc114 100644
--- 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
