Skip to content

docs: update /v1/agent/self#14987

Closed
nvanthao wants to merge 2 commits into
hashicorp:mainfrom
nvanthao:gerard/api-docs
Closed

docs: update /v1/agent/self#14987
nvanthao wants to merge 2 commits into
hashicorp:mainfrom
nvanthao:gerard/api-docs

Conversation

@nvanthao
Copy link
Copy Markdown
Contributor

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 reloads

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • not a security concern

@nvanthao nvanthao requested a review from a team as a code owner October 14, 2022 01:23
@github-actions github-actions Bot added the type/docs Documentation needs to be created/updated/clarified label Oct 14, 2022
Copy link
Copy Markdown
Contributor

@trujillo-adam trujillo-adam left a 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

Comment thread website/content/api-docs/agent/index.mdx Outdated
@jkirschner-hashicorp
Copy link
Copy Markdown
Contributor

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.

@jkirschner-hashicorp jkirschner-hashicorp added type/docs-cherrypick pr/no-changelog PR does not need a corresponding .changelog entry pr/no-metrics-test labels Oct 14, 2022
Co-authored-by: trujillo-adam <47586768+trujillo-adam@users.noreply.github.com>
@nvanthao
Copy link
Copy Markdown
Contributor Author

Hi Jared,

I have created an internal bug report with reference number 1203169189302122 prior to this doc PR. Please let me know if a Github Issue is still required.

Cheers,

@github-actions
Copy link
Copy Markdown

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.

@github-actions github-actions Bot added the meta/stale Automatically flagged for inactivity by stalebot label Dec 18, 2022
@nvanthao
Copy link
Copy Markdown
Contributor Author

closing due to the bug is fixed with #18681

@nvanthao nvanthao closed this Sep 14, 2023
@nvanthao nvanthao deleted the gerard/api-docs branch September 14, 2023 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta/stale Automatically flagged for inactivity by stalebot pr/no-changelog PR does not need a corresponding .changelog entry pr/no-metrics-test type/docs Documentation needs to be created/updated/clarified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants