Skip to content

[9.1](backport #46332) x-pack/filebeat/input/httpjson: improve/fix logic for health status updates#46401

Merged
efd6 merged 2 commits into9.1from
mergify/bp/9.1/pr-46332
Sep 5, 2025
Merged

[9.1](backport #46332) x-pack/filebeat/input/httpjson: improve/fix logic for health status updates#46401
efd6 merged 2 commits into9.1from
mergify/bp/9.1/pr-46332

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Sep 5, 2025

Proposed commit message

x-pack/filebeat/input/httpjson: improve/fix logic for health status updates

There was no way to ask the input to not report degraded health in the
case that a template error was required to signal that some aspect of
the input's operation should terminate, outside of the template itself.
This change fixes that.

Also fix buggy cursor template tests that were identified by this
improvement.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

This is an automatic backport of pull request #46332 done by [Mergify](https://mergify.com).

@efd6 @mergify
x-pack/filebeat/input/httpjson: improve/fix logic for health status u…
0174908 
…pdates (#46332)

There was no way to ask the input to not report degraded health in the
case that a template error was required to signal that some aspect of
the input's operation should terminate, outside of the template itself.
This change fixes that.

Also fix buggy cursor template tests that were identified by this
improvement.

(cherry picked from commit 64617f5)
@mergify mergify bot added the backport label Sep 5, 2025
@mergify mergify bot requested review from a team as code owners September 5, 2025 03:34
@mergify mergify bot added the backport label Sep 5, 2025
@mergify mergify bot assigned efd6 Sep 5, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 5, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 5, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Sep 5, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Sep 5, 2025

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 5, 2025 •
edited
Loading

🔍 Preview links for changed docs

@efd6 efd6 enabled auto-merge (squash) September 5, 2025 03:59
@efd6 efd6 merged commit 94d7c44 into 9.1 Sep 5, 2025
35 of 38 checks passed
@efd6 efd6 deleted the mergify/bp/9.1/pr-46332 branch September 5, 2025 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@efd6 efd6 efd6 approved these changes

Assignees

@efd6 efd6

Labels

backport bugfix enhancement Filebeat Filebeat Team:Security-Service Integrations Security Service Integrations Team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@elasticmachine @efd6