add container mapping

rmyz · Sep 30, 2024 · e4fac40 · e4fac40
1 parent 9d040cf
commit e4fac40
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/...plugins/observability_solution/apm/server/routes/services/get_service_metadata_details.ts b/...plugins/observability_solution/apm/server/routes/services/get_service_metadata_details.ts
@@ -245,7 +245,6 @@ export async function getServiceMetadataDetails({
   const response = await apmEventClient.search('get_service_metadata_details', params);
 
   const fields = response.hits.hits[0]?.fields;
-  // todo: missing `fields` property?
   const fieldsNorm = (fields ? serviceMetadataDetailsMapping(fields) : undefined) as
     | ServiceMetadataDetailsRaw
     | undefined;

diff --git a/x-pack/plugins/observability_solution/apm/server/utils/es_fields_mappings.ts b/x-pack/plugins/observability_solution/apm/server/utils/es_fields_mappings.ts
@@ -744,7 +744,7 @@ export const serviceVersionMapping = (
   };
 };
 
-// todo: missing `container` and `could` mappings
+// todo: missing `could` mapping
 export const serviceInstanceMetadataDetailsMapping = (
   fields: Partial<Record<string, unknown[]>> = {}
 ) => {
@@ -796,6 +796,9 @@ export const serviceInstanceMetadataDetailsMapping = (
         uid: normalizeValue<string>(fields[KUBERNETES_POD_UID]),
       },
     },
+    container: {
+      id: normalizeValue<string>(fields[CONTAINER_ID]),
+    },
   };
 };