docs: fulfill discovery_type description #4984
Conversation
previously "discovery_type" only showed support for "eureka", now added "consul_kv", "eureka", "dns", "nacos".
| |nodes |required, can't be used with `service_name` |Hash table or array. If it is a hash table, the key of the internal element is the upstream machine address list, the format is `Address + (optional) Port`, where the address part can be IP or domain name, such as `192.168.1.100:80`, `foo.com:80`, etc. The value is the weight of node. If it is an array, each item is a hash table with key `host`/`weight` and optional `port`/`priority`. The `nodes` can be empty, which means it is a placeholder and will be filled later. Clients use such an upstream will get 502 response. |`192.168.1.100:80`| | ||
| |service_name |required, can't be used with `nodes` |the name of service used in the service discovery, see [discovery](discovery.md) for more details|`a-bootiful-client`| | ||
| |discovery_type |required, if `service_name` is used | the type of service discovery, see [discovery](discovery.md) for more details|`eureka`| | ||
| |discovery_type |required, if `service_name` is used | the type of service discovery, see [discovery](discovery.md) for more details|`consul_kv`, `eureka`, `dns`, `nacos`| |
There was a problem hiding this comment.
This cell is just an example. You need to add the types after "the type of service discovery, see discovery for more details".
There was a problem hiding this comment.
This cell is just an example. You need to add the types after "the type of service discovery, see discovery for more details".
ok, my bad.
spacewander
changed the title
|
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@apisix.apache.org list. Thank you for your contributions. |
|
This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
2 participants
Fix #4981
previously "discovery_type" only showed support for "eureka", now added "consul_kv", "eureka", "dns", "nacos".