Skip to content

Support REACTIVE pipeline recovery with config.reload manager (#18930)#18967

Merged
yaauie merged 1 commit intoelastic:9.4from
yaauie:backport-pipeline-recovery-reactive
Apr 8, 2026
Merged

Support REACTIVE pipeline recovery with config.reload manager (#18930)#18967
yaauie merged 1 commit intoelastic:9.4from
yaauie:backport-pipeline-recovery-reactive

Conversation

@yaauie
Copy link
Copy Markdown
Member

@yaauie yaauie commented Apr 8, 2026

This is a clean cherry-pick backport of #18930.

…c#18930)

* reload automatic: recover crashed pipelines during convergence

* recovery: add health report probes

* derp: fix invocation of failure_injector filter

* PR feedback:

 - do not resolve recovery action until pipeline has settled into
   its crash state
 - capture a successful recovery as a successful reload in metrics

* health tests: back out local branch changes

* update recovery test assertions to use new $match helper

* Apply suggestion from @yaauie

* add logging, examples for `pipeline.recoverable` setting

* recovery: clean up old pipeline

* recovery: keep only last 5min of recovery log

* Apply suggestions from code review

Co-authored-by: Cas Donoghue <cas.donoghue@gmail.com>

---------

Co-authored-by: Cas Donoghue <cas.donoghue@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • run exhaustive tests : Run the exhaustive tests Buildkite pipeline.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

🔍 Preview links for changed docs

Copy link
Copy Markdown
Member

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

looks clean!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Vale Linting Results

Summary: 1 suggestion found

💡 Suggestions (1)
File Line Rule Message
docs/reference/logstash-settings-file.md 60 Elastic.WordChoice Consider using 'can, might' instead of 'may', unless the term is in the UI.

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.

@yaauie yaauie merged commit 796ae46 into elastic:9.4 Apr 8, 2026
12 of 13 checks passed
@elasticmachine
Copy link
Copy Markdown

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