This is an automated email from the ASF dual-hosted git repository.

jinsongzhou 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 ccda63475 [Improvement] expose 'http-server.rest-auth-type' variable 
in Helm chart (#3345)
ccda63475 is described below

commit ccda63475bfc7568934ede0ce414dbe3f01f29c2
Author: JiHyun Jenny Song <[email protected]>
AuthorDate: Fri Dec 6 16:11:39 2024 +0900

    [Improvement] expose 'http-server.rest-auth-type' variable in Helm chart 
(#3345)
    
    * [Improvement] expose 'http-server.rest-auth-type' variable in Helm chart
    
    * fix by comment
    
    ---------
    
    Co-authored-by: Xavier Bai <[email protected]>
---
 charts/amoro/templates/amoro-configmap.yaml | 1 +
 charts/amoro/values.yaml                    | 1 +
 2 files changed, 2 insertions(+)

diff --git a/charts/amoro/templates/amoro-configmap.yaml 
b/charts/amoro/templates/amoro-configmap.yaml
index 24d3a6744..bf06c4038 100644
--- a/charts/amoro/templates/amoro-configmap.yaml
+++ b/charts/amoro/templates/amoro-configmap.yaml
@@ -52,6 +52,7 @@ data:
           bind-port: {{ .Values.server.optimizing.port }}
 
       http-server:
+        rest-auth-type: {{ .Values.server.rest.restAuthType }}
         bind-port: {{ .Values.server.rest.port }}
 
       refresh-external-catalogs:
diff --git a/charts/amoro/values.yaml b/charts/amoro/values.yaml
index a7b937b8f..bcb058565 100644
--- a/charts/amoro/values.yaml
+++ b/charts/amoro/values.yaml
@@ -86,6 +86,7 @@ server:
   rest:
     enabled: true
     port: 1630
+    restAuthType: token
     service:
       ## @param type Can set as "ClusterIP" or "NodePort". If set to 
"NodePort", @param nodePort below should set a value
       ##

Reply via email to