-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
docs: update /v1/agent/self #14987
docs: update /v1/agent/self #14987
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need a little clarification
Thank you for updating the docs to more fully describe current behavior! Given reloading might be used more often going forward (e.g., auto-reload), I recommend we create a separate Github issue to track making this API endpoint return the current config, not just the config on startup. |
Co-authored-by: trujillo-adam <[email protected]>
Hi Jared, I have created an internal bug report with reference number Cheers, |
This pull request has been automatically flagged for inactivity because it has not been acted upon in the last 60 days. It will be closed if no new activity occurs in the next 30 days. Please feel free to re-open to resurrect the change if you feel this has happened by mistake. Thank you for your contributions. |
closing due to the bug is fixed with #18681 |
Description
/v1/agent/self
endpoint returns the configuration and member information of the local agent. This endpoint, however, does not reflect the runtime configurations after Consul reloadsTesting & Reproduction steps
make
in/website
directoryLinks
PR Checklist