Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Dec 10, 2025

What does this PR do?

sets path.home for beat receivers to be the components dir.

Why is it important?

This is necessary so metricbeat receiver gets the correct directory to
find light modules

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

Disruptive User Impact

How to test this PR locally

Unit test

cd internal/pkg/otel/translate
go test .

By hand

  1. build agent
  2. enroll with policy that has activemq metrics
  3. override so all metrics are collected with metricbeat receiver
PUT kbn:/api/fleet/agent_policies/<policy_id>
{
    "name": "test",
    "namespace": "default",
    "overrides": {
        "agent": {
            "internal": {
                "runtime": {
                    "metricbeat": {
                        "default": "otel"
                    }
                }
            }
        }
    }
}
4. verify activemq metrics starts

Related issues

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

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

@mergify mergify bot requested a review from a team as a code owner December 10, 2025 23:45
@mergify mergify bot added the backport label Dec 10, 2025
@mergify mergify bot requested review from swiatekm and ycombinator and removed request for a team December 10, 2025 23:45
@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 11, 2025

💔 Build Failed

Failed CI Steps

History

cc @leehinman

@leehinman leehinman merged commit a38d38f into 8.19 Dec 11, 2025
23 checks passed
@leehinman leehinman deleted the mergify/bp/8.19/pr-11726 branch December 11, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants