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

[APM] Show JVM tab for otel java agent #100253

Closed
simitt opened this issue May 18, 2021 · 9 comments · Fixed by #102346
Closed

[APM] Show JVM tab for otel java agent #100253

simitt opened this issue May 18, 2021 · 9 comments · Fixed by #102346
Labels
bug Fixes for quality problems that affect the customer experience good first issue low hanging fruit low hanging fruit DO NOT USE. Use `good first issue` instead Team:APM All issues that need APM UI Team support v7.14.0

Comments

@simitt
Copy link
Contributor

simitt commented May 18, 2021

Related: elastic/apm-server#4986 (comment)

When using a java opentelemetry agent to ingest data, the JVM tab should appear in the APM UI. Currently the ui only shows JVM tab when agent.name is java.

@simitt simitt added the Team:APM All issues that need APM UI Team support label May 18, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@simitt simitt added the v7.14.0 label May 18, 2021
@sorenlouv sorenlouv added [zube]: 7.14 bug Fixes for quality problems that affect the customer experience labels May 27, 2021
@sorenlouv
Copy link
Member

JVM tab is conditionally displayed here:

if (isJavaAgentName(agentName)) {
tabs.push(nodesListTab);
} else if (agentName && !isRumAgentName(agentName)) {
tabs.push(metricsTab);
}

It looks like we can simply update isJavaAgentName here:

export function isJavaAgentName(
agentName: string | undefined
): agentName is 'java' {
return agentName === 'java';
}

To this:

export function isJavaAgentName(
  agentName: string | undefined
): agentName is 'java' {
  return agentName === 'java' || agentName === 'opentelemetry/java';
}

@sorenlouv
Copy link
Member

sorenlouv commented May 27, 2021

@simitt The title of the issue is currently "Show JVM metrics tab independent of agent.name". My suggestion is to still use the agent.name to decide whether to show the JVM tab or not, but include the otel agent name opentelemetry/java when doing so. Sounds ok or do you have better alternatives?

@sorenlouv sorenlouv changed the title [APM] Show JVM metrics tab independend of agent.name [APM] Show JVM tab for otel java agent May 27, 2021
@simitt
Copy link
Contributor Author

simitt commented May 27, 2021

@sqren sounds good.

@hansh0801
Copy link

@sqren when this issue is going to fix?

@sorenlouv
Copy link
Member

@hansh0801 Currently aiming for 7.14 but can't give any guarantees. If trivial we will also backport the fix as far back as possible.

You can subscribe to this issue to stay updated with the progress.

@sorenlouv sorenlouv added low hanging fruit DO NOT USE. Use `good first issue` instead good first issue low hanging fruit labels Jun 7, 2021
@cyrille-leclerc
Copy link
Contributor

Thanks @sqren . CC @michaelhyatt & @mikktoki

@hansh0801
Copy link

thanks @sqren !

@michaelhyatt
Copy link

I am using AWS Distro for OTel with Java lamdas and pointing it at our APM server's OTel intake. In addition to 1 transaction and 1 span I am getting about 30 metrics documents (as per the attached file) that contain various JVM and GC metrics. Can be a great addition to populate the APM UI with.
lambda+metrics.json.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience good first issue low hanging fruit low hanging fruit DO NOT USE. Use `good first issue` instead Team:APM All issues that need APM UI Team support v7.14.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants