Skip to content

Update Nodes view descriptions#6919

Merged
wajihaparvez merged 3 commits into
elastic:mainfrom
wajihaparvez:autoops-nodes-view-updates
Jun 16, 2026
Merged

Update Nodes view descriptions#6919
wajihaparvez merged 3 commits into
elastic:mainfrom
wajihaparvez:autoops-nodes-view-updates

Conversation

@wajihaparvez

Copy link
Copy Markdown
Contributor

Summary

This PR updates some metric descriptions on the AutoOps Nodes view page to match the UI.
Closes: https://github.com/elastic/opex-product/issues/680

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

@wajihaparvez wajihaparvez requested a review from arnonst June 12, 2026 21:51
@wajihaparvez wajihaparvez self-assigned this Jun 12, 2026
@wajihaparvez wajihaparvez requested a review from a team as a code owner June 12, 2026 21:51
@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

⏳ Building and deploying preview... View progress

This comment will be updated with preview links when the build is complete.

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 3 warnings found

⚠️ Warnings (3): Fix when the suggestion improves clarity or correctness.
File Line Rule Message
deploy-manage/monitor/autoops/ec-autoops-nodes-view.md 62 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'for example' instead of 'e.g'.
deploy-manage/monitor/autoops/ec-autoops-nodes-view.md 68 Elastic.Spelling 'tx' is a possible misspelling.
deploy-manage/monitor/autoops/ec-autoops-nodes-view.md 69 Elastic.Spelling 'tx' is a possible misspelling.

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@arnonst arnonst left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@shainaraskas shainaraskas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

some suggestions for style mostly :)

Comment thread deploy-manage/monitor/autoops/ec-autoops-nodes-view.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-nodes-view.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-nodes-view.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-nodes-view.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-nodes-view.md Outdated
Comment thread deploy-manage/monitor/autoops/ec-autoops-nodes-view.md Outdated
@wajihaparvez wajihaparvez merged commit 5eccb61 into elastic:main Jun 16, 2026
7 of 8 checks passed
@wajihaparvez wajihaparvez deleted the autoops-nodes-view-updates branch June 16, 2026 22:44
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.

3 participants