[Metrics UI] Remove APM Hard Dependency

elastic · Apr 30, 2020 · e4cd3dd · e4cd3dd
1 parent 5887c97
commit e4cd3dd
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 96 deletions.
diff --git a/x-pack/plugins/infra/kibana.json b/x-pack/plugins/infra/kibana.json
@@ -4,7 +4,6 @@
   "kibanaVersion": "kibana",
   "requiredPlugins": [
     "features",
-    "apm",
     "usageCollection",
     "spaces",
     "home",

diff --git a/x-pack/plugins/infra/server/routes/metadata/index.ts b/x-pack/plugins/infra/server/routes/metadata/index.ts
@@ -18,7 +18,6 @@ import {
 import { InfraBackendLibs } from '../../lib/infra_types';
 import { getMetricMetadata } from './lib/get_metric_metadata';
 import { pickFeatureName } from './lib/pick_feature_name';
-import { hasAPMData } from './lib/has_apm_data';
 import { getCloudMetricsMetadata } from './lib/get_cloud_metric_metadata';
 import { getNodeInfo } from './lib/get_node_info';
 import { throwErrors } from '../../../common/runtime_types';
@@ -74,16 +73,13 @@ export const initMetadataRoute = (libs: InfraBackendLibs) => {
         const cloudMetricsFeatures = pickFeatureName(cloudMetricsMetadata.buckets).map(
           nameToFeature('metrics')
         );
-        const hasAPM = await hasAPMData(framework, requestContext, configuration, nodeId, nodeType);
-        const apmMetricFeatures = hasAPM ? [{ name: 'apm.transaction', source: 'apm' }] : [];
-
         const id = metricsMetadata.id;
         const name = metricsMetadata.name || id;
         return response.ok({
           body: InfraMetadataRT.encode({
             id,
             name,
-            features: [...metricFeatures, ...cloudMetricsFeatures, ...apmMetricFeatures],
+            features: [...metricFeatures, ...cloudMetricsFeatures],
             info,
           }),
         });

diff --git a/x-pack/plugins/infra/server/routes/metadata/lib/has_apm_data.ts b/x-pack/plugins/infra/server/routes/metadata/lib/has_apm_data.ts
diff --git a/x-pack/test/api_integration/apis/infra/metadata.ts b/x-pack/test/api_integration/apis/infra/metadata.ts
@@ -269,42 +269,6 @@ export default function({ getService }: FtrProviderContext) {
           }
         });
       });
-      describe('APM metrics', () => {
-        const archiveName = 'infra/8.0.0/metrics_and_apm';
-        before(() => esArchiver.load(archiveName));
-        after(() => esArchiver.unload(archiveName));
-
-        it('host without APM data', async () => {
-          const metadata = await fetchMetadata({
-            sourceId: 'default',
-            nodeId: 'gke-observability-8--observability-8--bc1afd95-f0zc',
-            nodeType: 'host',
-            timeRange: timeRange800,
-          });
-          if (metadata) {
-            expect(
-              metadata.features.some(f => f.name === 'apm.transaction' && f.source === 'apm')
-            ).to.be(false);
-          } else {
-            throw new Error('Metadata should never be empty');
-          }
-        });
-        it('pod with APM data', async () => {
-          const metadata = await fetchMetadata({
-            sourceId: 'default',
-            nodeId: 'c1031331-9ae0-11e9-9a96-42010a84004d',
-            nodeType: 'pod',
-            timeRange: timeRange800,
-          });
-          if (metadata) {
-            expect(
-              metadata.features.some(f => f.name === 'apm.transaction' && f.source === 'apm')
-            ).to.be(true);
-          } else {
-            throw new Error('Metadata should never be empty');
-          }
-        });
-      });
     });
   });
 }