Skip to content

set path.home for beat receivers to be components dir#11726

Merged
leehinman merged 1 commit intoelastic:mainfrom
leehinman:48010_set_path_home
Dec 10, 2025
Merged

set path.home for beat receivers to be components dir#11726
leehinman merged 1 commit intoelastic:mainfrom
leehinman:48010_set_path_home

Conversation

@leehinman
Copy link
Contributor

@leehinman leehinman 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?
  • ...
@mergify
Copy link
Contributor

mergify bot commented Dec 10, 2025

This pull request does not have a backport label. Could you fix it @leehinman? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-./d./d is the label that automatically backports to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.
@leehinman leehinman added the backport-8.19 Automated backport to the 8.19 branch label Dec 10, 2025
@leehinman leehinman marked this pull request as ready for review December 10, 2025 20:38
@leehinman leehinman requested a review from a team as a code owner December 10, 2025 20:38
@elasticmachine
Copy link
Contributor

@leehinman leehinman merged commit 7096b95 into elastic:main Dec 10, 2025
24 checks passed
mergify bot pushed a commit that referenced this pull request Dec 10, 2025
leehinman added a commit that referenced this pull request Dec 11, 2025
(cherry picked from commit 7096b95)

Co-authored-by: Lee E Hinman <57081003+leehinman@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-8.19 Automated backport to the 8.19 branch

3 participants