fengxsong commented on code in PR #382:
URL: https://github.com/apache/apisix-helm-chart/pull/382#discussion_r1008968957


##########
charts/apisix-dashboard/README.md:
##########
@@ -67,17 +67,22 @@ The following tables lists the configurable parameters of 
the apisix-dashboard c
 | Name                                 | Description                           
                                                    | Value           |
 | ------------------------------------ | 
-----------------------------------------------------------------------------------------
 | --------------- |
 | `config.conf.listen.host`                       | The address on which the 
`Manager API` should listen. The default value is 0.0.0.0, if want to specify, 
please enable it. This value accepts IPv4, IPv6, and hostname.                  
                                                 | `0.0.0.0`             |
-| `config.conf.listen.port`                       | The port on which the 
`Manager API` should listen.                                                    
              | `9000`             |
+| `config.conf.listen.port`                       | The port on which the 
`Manager API` should listen.                                                    
                | `9000`          |
 | `config.authentication.secert`                  | Secret for jwt token 
generation | `secert` |
-| `config.authentication.expireTime`                  | JWT token expire time, 
in second | `3600` |
-| `config.authentication.users`                  | Specifies username and 
password for login `manager api`. | `[{username: admin, password: admin}]` |
-| `config.conf.etcd.endpoints`                       | Supports defining 
multiple etcd host addresses for an etcd cluster                                
                                  | `apisix-etcd:2379`             |
-| `config.conf.etcd.prefix`                       | Apache APISIX config's 
prefix in etcd, /apisix by default                                              
                    | `/apisix`             |
-| `config.conf.etcd.username`                       | Specifies etcd basic 
auth username if  enable etcd auth                                              
                  | `~`             |
-| `config.conf.etcd.password`                       | Specifies etcd basic 
auth password  if  enable etcd auth                                             
                 | `~`             |
-| `config.conf.log.accessLog.filePath`                  | Access log path | 
`/dev/stdout` |
-| `config.conf.log.errorLog.filePath`                  | Error log path | 
`/dev/stderr` |
-| `config.conf.log.errorLog.level`                  | Error log level. 
Supports levels, lower to higher: debug, info, warn, error, panic, fatal | 
`warn` |
+| `config.authentication.expireTime`              | JWT token expire time, in 
second | `3600` |
+| `config.authentication.users`                   | Specifies username and 
password for login `manager api`. | `[{username: admin, password: admin}]` |
+| `config.conf.etcd.endpoints`                    | Supports defining multiple 
etcd host addresses for an etcd cluster                                         
           | `apisix-etcd:2379` |
+| `config.conf.etcd.prefix`                       | Apache APISIX config's 
prefix in etcd, /apisix by default                                              
               | `/apisix`       |
+| `config.conf.etcd.username`                     | Specifies etcd basic auth 
username if  enable etcd auth                                                   
            | `~`             |
+| `config.conf.etcd.password`                     | Specifies etcd basic auth 
password  if  enable etcd auth                                                  
            | `~`             |
+| `config.conf.etcd.mtls.enabled`                 | Enable MTLS option when 
connecting to an etcd cluster with TLS enabled                                  
              | `false`         |
+| `config.conf.etcd.mtls.existingSecret`          | Name of existing secret 
that contains certs for TLS authentication                                      
              | `""`            |
+| `config.conf.etcd.mtls.caFile`                  | Key of CA file in secret   
                                                                                
           | `ca.crt`        |

Review Comment:
   LGTM. then should this 
[line](https://github.com/apache/apisix-dashboard/blob/master/api/internal/conf/conf.go#L86)
 be modified as well? 



-- 
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