[Uptime] Update show apm data links (#137974) (#137558)

elastic · Aug 9, 2022 · 5faf880 · 5faf880
1 parent 0bc8cf7
commit 5faf880
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 10 deletions.
diff --git a/...s/overview/monitor_list/monitor_list_drawer/__snapshots__/integration_group.test.tsx.snap b/...s/overview/monitor_list/monitor_list_drawer/__snapshots__/integration_group.test.tsx.snap
diff --git a/...synthetics/public/legacy_uptime/lib/helper/observability_integration/get_apm_href.test.ts b/...synthetics/public/legacy_uptime/lib/helper/observability_integration/get_apm_href.test.ts
@@ -39,14 +39,14 @@ describe('getLegacyApmHref', () => {
   it('creates href with base path when present', () => {
     const result = getLegacyApmHref(summary, 'foo', 'now-15m', 'now');
     expect(result).toMatchInlineSnapshot(
-      `"foo/app/apm#/services?kuery=url.domain:%20%22www.elastic.co%22&rangeFrom=now-15m&rangeTo=now"`
+      `"foo/app/apm/services?kuery=url.domain:%20%22www.elastic.co%22&rangeFrom=now-15m&rangeTo=now"`
     );
   });
 
   it('does not add a base path or extra slash when base path is empty string', () => {
     const result = getLegacyApmHref(summary, '', 'now-15m', 'now');
     expect(result).toMatchInlineSnapshot(
-      `"/app/apm#/services?kuery=url.domain:%20%22www.elastic.co%22&rangeFrom=now-15m&rangeTo=now"`
+      `"/app/apm/services?kuery=url.domain:%20%22www.elastic.co%22&rangeFrom=now-15m&rangeTo=now"`
     );
   });
 
@@ -59,7 +59,7 @@ describe('getLegacyApmHref', () => {
     it('links to the named service', () => {
       const result = getLegacyApmHref(summary, 'foo', 'now-15m', 'now');
       expect(result).toMatchInlineSnapshot(
-        `"foo/app/apm#/services?kuery=service.name:%20%22${serviceName}%22&rangeFrom=now-15m&rangeTo=now"`
+        `"foo/app/apm/services/MyServiceName/overview/?rangeFrom=now-15m&rangeTo=now"`
       );
     });
   });

diff --git a/...gins/synthetics/public/legacy_uptime/lib/helper/observability_integration/get_apm_href.ts b/...gins/synthetics/public/legacy_uptime/lib/helper/observability_integration/get_apm_href.ts
@@ -14,13 +14,20 @@ export const getLegacyApmHref = (
   dateRangeStart: string,
   dateRangeEnd: string
 ) => {
-  const clause = summary?.state?.service?.name
-    ? `service.name: "${summary.state.service.name}"`
-    : `url.domain: "${summary.state.url?.domain}"`;
+  const serviceName = summary?.state?.service?.name;
+
+  if (serviceName) {
+    return addBasePath(
+      basePath,
+      `/app/apm/services/${serviceName}/overview/?rangeFrom=${dateRangeStart}&rangeTo=${dateRangeEnd}`
+    );
+  }
+
+  const clause = `url.domain: "${summary.state.url?.domain}"`;
 
   return addBasePath(
     basePath,
-    `/app/apm#/services?kuery=${encodeURI(
+    `/app/apm/services?kuery=${encodeURI(
       clause
     )}&rangeFrom=${dateRangeStart}&rangeTo=${dateRangeEnd}`
   );