Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

TaggedAddresses should support 'lan' #261

Closed
kgignatyev-inspur opened this issue Aug 31, 2017 · 1 comment
Closed

TaggedAddresses should support 'lan' #261

kgignatyev-inspur opened this issue Aug 31, 2017 · 1 comment

Comments

@kgignatyev-inspur
Copy link

com.orbitz.consul.model.catalog.TaggedAddresses should support 'lan' attribute as a minimum, better yet make it to support any number of addresses per
https://www.consul.io/api/catalog.html#taggedaddresses
here is example registration request

{
"Datacenter": "dc1",
"ID": "40e4a748-2192-161a-0510-9bf59fe950b5",
"Node": "foobar",
"Address": "192.168.10.10",
"TaggedAddresses": {
"lan": "192.168.10.10",
"wan": "10.0.10.10"
},
"NodeMeta": {
"somekey": "somevalue"
},
"Service": {
"ID": "redis1",
"Service": "redis",
"Tags": [
"primary",
"v1"
],
"Address": "127.0.0.1",
"Port": 8000
},
"Check": {
"Node": "foobar",
"CheckID": "service:redis1",
"Name": "Redis health check",
"Notes": "Script based health check",
"Status": "passing",
"ServiceID": "redis1"
}
}
»

@rickfast
Copy link
Owner

rickfast commented Oct 2, 2017

fixed 2220fe4

@rickfast rickfast closed this as completed Oct 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants