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

[DOC] Add missing weighted_routing nodes stats #7244

Closed
1 of 4 tasks
lukas-vlcek opened this issue May 24, 2024 · 0 comments · Fixed by #7245
Closed
1 of 4 tasks

[DOC] Add missing weighted_routing nodes stats #7244

lukas-vlcek opened this issue May 24, 2024 · 0 comments · Fixed by #7245
Assignees
Labels
2 - In progress Issue/PR: The issue or PR is in progress.

Comments

@lukas-vlcek
Copy link
Contributor

lukas-vlcek commented May 24, 2024

What do you want to do?

  • Request a change to existing documentation
  • Add new documentation
  • Report a technical problem with the documentation
  • Other

Tell us about your request. Provide a summary of the request and all versions that are affected.

The Nodes Stats documentation is missing weighted_routing metric group and its details.

I think the feature has been added in opensearch-project/OpenSearch#6041 and a high-level explanation can be found also here opensearch-project/OpenSearch#4735 (comment). Unfortunately, the documentation was not included.

What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

# Do a clean checkout of OpenSearch from git and run a single node:
./gradlew clean run

# After it is ready open a web browser of your choice and point it to nodes stats API
# using `weighted_routing` as a metric group filter.

open -a "Google Chrome"  "http://localhost:9200/_nodes/stats/weighted_routing?pretty&filter_path=nodes.*.weighted_routing"

We can see the following:

{
  "nodes" : {
    "usAVD2H5Sn6SaRaN-Z67Mg" : {
      "weighted_routing" : {
        "stats" : {
          "fail_open_count" : 0
        }
      }
    }
  }
}

PR link: #7245

lukas-vlcek added a commit to lukas-vlcek/documentation-website that referenced this issue May 24, 2024
@hdhalter hdhalter added 2 - In progress Issue/PR: The issue or PR is in progress. and removed untriaged labels May 24, 2024
lukas-vlcek added a commit to lukas-vlcek/documentation-website that referenced this issue May 27, 2024
lukas-vlcek added a commit to lukas-vlcek/documentation-website that referenced this issue May 29, 2024
lukas-vlcek added a commit to lukas-vlcek/documentation-website that referenced this issue May 29, 2024
lukas-vlcek added a commit to lukas-vlcek/documentation-website that referenced this issue May 29, 2024
lukas-vlcek added a commit to lukas-vlcek/documentation-website that referenced this issue May 30, 2024
opensearch-trigger-bot bot pushed a commit that referenced this issue Jun 4, 2024
Closes #7244

Signed-off-by: Lukáš Vlček <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
(cherry picked from commit ceab04c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 - In progress Issue/PR: The issue or PR is in progress.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants