Skip to content

[9.4](backport #50980) Fix filebeat test_shutdown_wait_ok#50985

Open
mergify[bot] wants to merge 1 commit into
9.4from
mergify/bp/9.4/pr-50980
Open

[9.4](backport #50980) Fix filebeat test_shutdown_wait_ok#50985
mergify[bot] wants to merge 1 commit into
9.4from
mergify/bp/9.4/pr-50980

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify Bot commented May 28, 2026

Proposed commit message

With the fix merged from #50625 it results in all events being removed from the pipeline on shutdown. Then this test didn't set the correct settings to prevent file rotation that would cause the test to fail. With both the change in this PR and the race condition fix this test can be re-enabled.

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. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • [ ] I have added an entry in ./changelog/fragments using the changelog tool.

Disruptive User Impact

None

How to test this PR locally

Run unit tests.

Related issues

(cherry picked from commit 73c48e4)
@mergify mergify Bot requested a review from a team as a code owner May 28, 2026 18:44
@mergify mergify Bot added the backport label May 28, 2026
@mergify mergify Bot requested review from leehinman and mauri870 and removed request for a team May 28, 2026 18:44
@botelastic botelastic Bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 28, 2026
@botelastic
Copy link
Copy Markdown

botelastic Bot commented May 28, 2026

This pull request doesn't have a Team:<team> label.

@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport needs_team Indicates that the issue/PR needs a Team:* label skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant