Skip to content

Commit

Permalink
[APM] Added improved JVM runtime metrics dashboard. (elastic#162460)
Browse files Browse the repository at this point in the history
## Summary

Replaces the current JVM runtime metrics dashboard with a more advanced
dashboard (based on the portable dashboards feature).

Additional information (that has not been shown in the previous view):
- memory allocation rate
- Heap usage split by memory pools

<img width="1428" alt="image"
src="https://github.com/elastic/kibana/assets/866830/c147f9c4-b115-455e-a86e-c288a94e25cb">
<img width="1424" alt="image"
src="https://github.com/elastic/kibana/assets/866830/3ff8978f-917f-4b5e-bd82-fe3a52fe78c7">

---------

Signed-off-by: Alexander Wert <alexander.wert@elastic.co>
  • Loading branch information
AlexanderWert authored and Devon Thomson committed Aug 1, 2023
1 parent b786005 commit 60f71e3
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 7 deletions.
15 changes: 8 additions & 7 deletions x-pack/plugins/apm/public/components/app/metrics/index.tsx
Expand Up @@ -22,13 +22,6 @@ export function Metrics() {
const { agentName, runtimeName, serverlessType } = useApmServiceContext();
const isAWSLambda = isAWSLambdaAgent(serverlessType);

if (
!isAWSLambda &&
(isJavaAgentName(agentName) || isJRubyAgent(agentName, runtimeName))
) {
return <JvmMetricsOverview />;
}

if (isAWSLambda) {
return <ServerlessMetrics />;
}
Expand All @@ -38,6 +31,7 @@ export function Metrics() {
runtimeName,
serverlessType,
});

if (hasStaticDashboard) {
return (
<JsonMetricsDashboard
Expand All @@ -48,5 +42,12 @@ export function Metrics() {
);
}

if (
!isAWSLambda &&
(isJavaAgentName(agentName) || isJRubyAgent(agentName, runtimeName))
) {
return <JvmMetricsOverview />;
}

return <ServiceMetrics />;
}
Expand Up @@ -7,6 +7,7 @@

export const AGENT_NAME_DASHBOARD_FILE_MAPPING: Record<string, string> = {
nodejs: 'nodejs',
java: 'java',
};

/**
Expand All @@ -21,6 +22,12 @@ export async function loadDashboardFile(filename: string): Promise<any> {
'./nodejs.json'
);
}
case 'java': {
return import(
/* webpackChunkName: "lazyJavaDashboard" */
'./java.json'
);
}
default: {
break;
}
Expand Down

Large diffs are not rendered by default.

0 comments on commit 60f71e3

Please sign in to comment.