cleans up nodes usage API call

elastic · Sep 24, 2020 · 06e3b79 · 06e3b79
1 parent a572d75
commit 06e3b79
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/plugins/telemetry/server/telemetry_collection/get_nodes_usage.test.ts b/src/plugins/telemetry/server/telemetry_collection/get_nodes_usage.test.ts
@@ -52,12 +52,12 @@ describe('get_nodes_usage', () => {
     const esClient = elasticsearchServiceMock.createClusterClient().asInternalUser;
     esClient.nodes.usage.mockImplementationOnce(
       // @ts-ignore
-      async (_params = { metric: '_all', timeout: TIMEOUT }) => {
+      async (_params = { timeout: TIMEOUT }) => {
         return response;
       }
     );
     const item = await getNodesUsage(esClient);
-    expect(esClient.nodes.usage).toHaveBeenCalledWith({ metric: '_all', timeout: TIMEOUT });
+    expect(esClient.nodes.usage).toHaveBeenCalledWith({ timeout: TIMEOUT });
     expect(item).toStrictEqual({
       nodes: [
         {

diff --git a/src/plugins/telemetry/server/telemetry_collection/get_nodes_usage.ts b/src/plugins/telemetry/server/telemetry_collection/get_nodes_usage.ts
@@ -57,7 +57,6 @@ export async function fetchNodesUsage(
   esClient: ElasticsearchClient
 ): Promise<NodesFeatureUsageResponse> {
   const { body } = await esClient.nodes.usage<NodesFeatureUsageResponse>({
-    metric: '_all',
     timeout: TIMEOUT,
   });
   return body;