tokers commented on a change in pull request #414:
URL:
https://github.com/apache/apisix-ingress-controller/pull/414#discussion_r623724885
##########
File path: pkg/config/config.go
##########
@@ -78,8 +78,18 @@ type KubernetesConfig struct {
// APISIXConfig contains all APISIX related config items.
type APISIXConfig struct {
- BaseURL string `json:"base_url" yaml:"base_url"`
+ // DefaultClusterName is the name of default cluster.
+ DefaultClusterName string `json:"default_cluster_name"`
+ // DefaultClusterBaseURL is the base url configuration for the default
cluster.
+ DefaultClusterBaseURL string `json:"default_cluster_base_url"
yaml:"default_cluster_base_url"`
+ // DefaultClusterAdminKey is the admin key for the default cluster.
// TODO: Obsolete the plain way to specify admin_key, which is insecure.
+ DefaultClusterAdminKey string `json:"default_cluster_admin_key"
yaml:"default_cluster_admin_key"`
+ // BaseURL is same to DefaultClusterBaseURL.
+ // Deprecated: use DefaultClusterBaseURL instead.
+ BaseURL string `json:"base_url" yaml:"base_url"`
Review comment:
What about:
```
// Deprecated: use DefaultClusterBaseURL instead, BaseURL will be removed
once v1.0.0 is released.
```
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
