Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add datacenter to catalog node API #2713

Closed
sethvargo opened this issue Feb 6, 2017 · 0 comments
Closed

Add datacenter to catalog node API #2713

sethvargo opened this issue Feb 6, 2017 · 0 comments
Assignees
@magiconair
Milestone
0.8.2

Comments

@sethvargo
Copy link
Contributor

consul version for both Client and Server

Client: 0.7.2
Server: 0.7.2

Description of the Issue (and unexpected/desired result)

Add support for the datacenter a node is running in.

Refs
@slackpad slackpad added this to the 0.8.0 milestone Feb 6, 2017
@slackpad slackpad modified the milestones: 0.8.1, 0.8.0 Mar 24, 2017
@slackpad slackpad modified the milestones: 0.8.1, 0.8.2 Apr 12, 2017
@slackpad slackpad mentioned this issue Apr 17, 2017
Closed
@magiconair magiconair self-assigned this Apr 17, 2017
magiconair added a commit that referenced this issue Apr 17, 2017
@magiconair
Add datacenter to catalog node API
b6d0e0a 
This patch stores the datacenter of a node in the memdb
and exposes it via the API in all places where a Node
structure is returned.

 * /catalog/nodes
 * /catalog/node/:node
 * /catalog/service/:service

Fixes #2713
@magiconair magiconair mentioned this issue Apr 17, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@magiconair magiconair

Labels
None yet
Projects
None yet
Milestone
0.8.2
Development

No branches or pull requests
3 participants
@magiconair @sethvargo @slackpad