Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Oct 14, 2025

What does this PR do?

Fixes tests which implicitly assume beats are running as processes as opposed to otel receivers. More specifically:

  • 2952a12 makes the runtime leak test only look at beats running as processes.
  • c0d8385 makes integration tests check for running agent processes by only looking at processes without parents which are also elastic agents.
  • 72855e5 explicitly sets the runtime in tests where it matters.
  • 9d8cc7a checks for beats metrics in a more generic way.
  • 1c1a0c3 checks for the component build hash in a way that covers beats receivers as well.

Why is it important?

We want to enable beats receivers for self-monitoring by default. This PR makes the test adjustments separately to show they all pass for beats processes as well.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • 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/fragments using the changelog tool
  • I have added an integration test or an E2E test

Related issues


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

)

* Fix leak integration test

* Improve detection of agent processes in integration tests

* Explicitly set the runtime in monitoring integration test

* Make the check for component metrics more generic

* Check the right build hash in integration tests

(cherry picked from commit 3ccf28d)
@mergify mergify bot requested a review from a team as a code owner October 14, 2025 18:38
@mergify mergify bot added the backport label Oct 14, 2025
@mergify mergify bot requested review from michalpristas and pkoutsovasilis and removed request for a team October 14, 2025 18:38
@mergify mergify bot added the backport label Oct 14, 2025
@github-actions github-actions bot added skip-changelog chore Tasks that just need to be done, they are neither bug, nor enhancements labels Oct 14, 2025
@swiatekm swiatekm enabled auto-merge (squash) October 14, 2025 20:47
@swiatekm swiatekm merged commit 61ab378 into 9.2 Oct 14, 2025
24 checks passed
@swiatekm swiatekm deleted the mergify/bp/9.2/pr-10537 branch October 14, 2025 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport chore Tasks that just need to be done, they are neither bug, nor enhancements skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants