tokers commented on code in PR #285:
URL: https://github.com/apache/apisix-helm-chart/pull/285#discussion_r874276587


##########
charts/apisix-dashboard/values.yaml:
##########
@@ -76,6 +76,7 @@ config:
   authentication:
     secert: secert
     expireTime: 3600
+    userCreate: true

Review Comment:
   > I have no good idea how to consistent the admin key between apisix and 
apisix-ingress-controller, because it is different when quote the variable 
twice, so we must get it from configmap when apisix installed.
   Maybe we can tell users to install apisix-ingress-controller with --set 
config.apisix.adminKey filed which can execute the command kubectl get cm {{ 
include "apisix.fullname" . }} -n {{ .Release.Namespace }} -o=jsonpath={.data} 
| sed -nr '/- name: "admin"/,+2 p' | awk -F": " '/key/{print $2}' to get.
   
   But the case is we install APISIX and Ingress Controller **at the same 
time**.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to