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.

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

Add /v1/agent/self and return local agent config #173

Merged
merged 2 commits into from
May 29, 2014
Merged

Conversation

tiwilliam
Copy link
Contributor

This is a proposal for introducing a new API endpoint called /v1/agent/self. It returns the local agent's configuration as requested in #120.

This is an easier way to get the local state without trying to guess node using /v1/agent/members.

@armon
Copy link
Member

armon commented May 27, 2014

This is a good start, but I think we want to include some additional information about the node's configuration. For example, people have asked about the configured DNS suffix (".consul"), and other configuration values. Maybe we make the response a larger JSON object of which the existing output is nested under "Member"?

@tiwilliam
Copy link
Contributor Author

I see, thought this would be useful enough for most use cases. But might be nice to have the config as well. Would this be good:

{
    "Member": {
        "Name": "foobar",
        ...
    },
    "Config": {
        "Bootstrap": true,
        ...
    }
}

@armon
Copy link
Member

armon commented May 27, 2014

Yeah, I think that should cover most of our basis for now!

armon added a commit that referenced this pull request May 29, 2014
Add `/v1/agent/self` and return local agent config
@armon armon merged commit 3874498 into master May 29, 2014
@armon
Copy link
Member

armon commented May 29, 2014

Thanks!

@armon armon deleted the f-agent-self branch May 29, 2014 18:18
@tiwilliam tiwilliam restored the f-agent-self branch July 16, 2014 18:23
@tiwilliam tiwilliam deleted the f-agent-self branch July 16, 2014 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants