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

Profiling Integration on APM services uses incorrect data source #183654

Open
JonasKunz opened this issue May 16, 2024 · 1 comment
Open

Profiling Integration on APM services uses incorrect data source #183654

JonasKunz opened this issue May 16, 2024 · 1 comment
Labels
apm bug Fixes for quality problems that affect the customer experience Feature:UniversalProfiling Universal Profiling observability feature Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.14.0

Comments

@JonasKunz
Copy link
Contributor

While playing around with the Universal Profiling <-> APM integration on staging, I uncovered a bug in the Universal Profiling tab on the APM service UI.

I'm talking about this page:
image

When an APM-agent is running with the universal profiling correlation enabled, this page is supposed to show a flamegraph for all samples recorded for the process of the service.

However, what it actually shows is only the samples which have occurred within transactions of said service. This is visible by following the Go to Universal Profiling Flamegraph link:

image

Here the flamegraph shows much more data, consisting of samples which occurred within transaction AND samples of background tasks which are not covered by transactions.

Note that I manually had to add the service.name: "$serviceName" filter after following the link, this also seems like a bug to me.

I assume the root cause for this problem is that the Universal Profiling tab on the APM-service page uses the wrong data set for rendering the flamegraph:

  • It appears to be using the transaction.profiler_stack_trace_ids field from APM-transaction documents
  • It should be using the profiling indices instead filtered on the service.name (like shown in the second screenshot)
@JonasKunz JonasKunz added bug Fixes for quality problems that affect the customer experience apm v8.14.0 labels May 16, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label May 16, 2024
@smith smith added Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team Feature:UniversalProfiling Universal Profiling observability feature labels May 16, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@botelastic botelastic bot removed the needs-team Issues missing a team label label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm bug Fixes for quality problems that affect the customer experience Feature:UniversalProfiling Universal Profiling observability feature Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.14.0
Projects
None yet
Development

No branches or pull requests

3 participants