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 docs for resource usage stats #5408

Merged
merged 6 commits into from
Jan 25, 2024

Conversation

bharath-techie
Copy link
Contributor

@bharath-techie bharath-techie commented Oct 27, 2023

Description

Update docs for resource usage stats introduced as part of opensearch-project/OpenSearch#9890

Issues Resolved

Fixes #5409 - Doc issue

Related to opensearch-project/OpenSearch#8910 - Opensearch meta issue

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Bharathwaj G <bharath78910@gmail.com>
Signed-off-by: Bharathwaj G <bharath78910@gmail.com>
Signed-off-by: Bharathwaj G <bharath78910@gmail.com>
@bharath-techie
Copy link
Contributor Author

Hi @Naarcha-AWS @kolchfa-aws ,
Can you please review this PR ?

Copy link
Contributor

@ajaymovva ajaymovva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @bharath-techie! Left some rewrite comments.

_api-reference/nodes-apis/nodes-stats.md Outdated Show resolved Hide resolved
_api-reference/nodes-apis/nodes-stats.md Outdated Show resolved Hide resolved
_api-reference/nodes-apis/nodes-stats.md Outdated Show resolved Hide resolved
_api-reference/nodes-apis/nodes-stats.md Outdated Show resolved Hide resolved
@kolchfa-aws kolchfa-aws self-assigned this Nov 2, 2023
Signed-off-by: Bharathwaj G <bharath78910@gmail.com>
@kolchfa-aws kolchfa-aws added v2.12.0 release-notes PR: Include this PR in the automated release notes labels Nov 3, 2023
Copy link
Contributor

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No edits. Ready for publishing.

@hdhalter hdhalter added the 6 - Done but waiting to merge PR: The work is done and ready to merge label Nov 6, 2023
@bharath-techie
Copy link
Contributor Author

Hi @kolchfa-aws @vagimeli

When will we merge this PR ?

@kolchfa-aws
Copy link
Collaborator

@bharath-techie We merge release PR shortly before the release or on release day.

@kolchfa-aws kolchfa-aws merged commit a3f0646 into opensearch-project:main Jan 25, 2024
3 checks passed
@hdhalter hdhalter added 3 - Done Issue is done/complete and removed 6 - Done but waiting to merge PR: The work is done and ready to merge labels Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete release-notes PR: Include this PR in the automated release notes v2.12.0
Projects
None yet
5 participants