Skip to content

Add ECE+ECK diag troubleshooting video links - #7572

Merged
stefnestor merged 2 commits into
elastic:mainfrom
stefnestor:stef_EcxDiags
Jul 28, 2026
Merged

Add ECE+ECK diag troubleshooting video links#7572
stefnestor merged 2 commits into
elastic:mainfrom
stefnestor:stef_EcxDiags

Conversation

@stefnestor

Copy link
Copy Markdown
Member

Summary

Links to ECE diagnostic-to-video and ECK diagnostic-to-video troubleshooting pages to Youtube walkthrough videos. (Same sentence formatting as Kibana diag, Elasticsearch diag, and Logstash diag pages.)

Generative AI disclosure

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

@stefnestor
stefnestor requested a review from a team as a code owner July 24, 2026 18:03
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

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

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

@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


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.

@github-actions github-actions Bot mentioned this pull request Jul 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Docs review summary

Reviewed both eligible changed files: troubleshoot/deployments/cloud-enterprise/run-ece-diagnostics-tool.md and troubleshoot/deployments/cloud-on-k8s/run-eck-diagnostics.md.

Focus areas

  • Style and clarity: The added sentence ("Watch this video for a walkthrough of capturing an {{ece/eck}} diagnostic.") matches the exact wording pattern already used on the sibling Elasticsearch diagnostics page (troubleshoot/elasticsearch/diagnostic.md), so it's consistent with established convention. No clarity or style issues.
  • Jargon: None introduced. {{ece}}/{{eck}} are already-established substitutions used throughout each page.
  • Frontmatter and applies_to: Unchanged by this PR.
  • Content type fit: No change to page structure or type; the added sentence fits naturally after the intro paragraph, consistent with the same placement pattern on sibling diagnostic pages.
  • Parent issue satisfaction: Not applicable (no linked issue).

Notes

  • The pre-fetched Vale finding (Elastic.MappedPages warning on line 3 of both files) refers to the pre-existing mapped_pages frontmatter key, which this PR does not modify — not applicable to this diff.

No actionable issues found. This is a clean, minimal, well-formed change.

Generated by Docs review agent for #7572 · sonnet50 24.5 AIC · ⌖ 6.39 AIC · ⊞ 15.6K ·

@yetanothertw yetanothertw 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.

LGTM, thanks, Stef! 🚀

@stefnestor
stefnestor enabled auto-merge (squash) July 28, 2026 19:01
@stefnestor
stefnestor merged commit 816685c into elastic:main Jul 28, 2026
7 of 8 checks passed
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.

2 participants