Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.15] fix: Add wrapper around libbeat instrumentation to propagate missing tracer config (backport #13653) #13675

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jul 11, 2024

Motivation/summary

This PR addresses the issue by adding a thin wrapper around libbeat instrumentation to propagate missing tracer config from elastic-agent via environmental variables.

Checklist

For functional changes, consider:

  • Is it observable through the addition of either logging or metrics?
  • Is its use being published in telemetry to enable product improvement?
  • Have system tests been added to avoid regression?

How to test these changes

Follow instructions in Building an Elastic Agent container image with a locally built APM Server and Running an Elastic Agent container with a locally built APM Server section in TESTING.md, update the relevant agent.monitoring sections in elastic-agent.yml to see the config are propagated down to tracing correctly.

Related issues

#13597
#11381
#13514


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

…tracer config (#13653)

Add wrapper to libbeat instrumentation to propagate missing tracer
config via apm environmental variables.

(cherry picked from commit 0a57bbe)
@mergify mergify bot requested a review from a team as a code owner July 11, 2024 17:15
@mergify mergify bot added the backport label Jul 11, 2024
@mergify mergify bot assigned 1pkg Jul 11, 2024
@mergify mergify bot merged commit 67fc25e into 8.15 Jul 11, 2024
16 checks passed
@mergify mergify bot deleted the mergify/bp/8.15/pr-13653 branch July 11, 2024 17:36
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.

1 participant