Skip to content

Failure store clarifications#7378

Open
gmarouli wants to merge 5 commits into
mainfrom
failure-store-clarifications
Open

Failure store clarifications#7378
gmarouli wants to merge 5 commits into
mainfrom
failure-store-clarifications

Conversation

@gmarouli

@gmarouli gmarouli commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

It has come to our attention that it was not clear to our users that the failure store does not store security errors and that the reroute processor can change the target failure store too.

Generative AI disclosure

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

@gmarouli
gmarouli requested review from a team as code owners July 15, 2026 08:55
@gmarouli
gmarouli requested a review from masseyke July 15, 2026 08:55
@github-actions

github-actions Bot commented Jul 15, 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 15, 2026

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

@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 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Docs review summary

Focus areas

  • Style and clarity: One issue in the new sentence — comma splice, "Note that" filler, and future tense instead of present tense. The important admonition update (adding "security issues") is clear and accurate.
  • Jargon: No new unexplained jargon introduced.
  • Frontmatter and applies_to: Frontmatter is intact; Vale flagged mapped_pages as a pre-existing warning (not changed by this PR).
  • Content type fit: How-to page structure unchanged; the two added lines fit naturally into the existing intro section.
  • Parent issue satisfaction: Satisfied. The PR adds the reroute processor caveat and the security-issues exclusion, which directly address the two gaps called out in the PR description.

Generated by Docs review agent for issue #7378 · 24.9 AIC · ⌖ 4.66 AIC · ⊞ 10.7K

Comment thread manage-data/data-store/data-streams/failure-store.md Outdated

@ankikuma ankikuma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Mary!

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