Update dependency @elastic/elasticsearch to ^8.16.0 (main) (#200275)

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@elastic/elasticsearch](http://www.elastic.co/guide/en/elasticsearch/client/javascript-api/current/index.html) ([source](https://togithub.com/elastic/elasticsearch-js)) | dependencies | minor | [`^8.15.2` -> `^8.16.0`](https://renovatebot.com/diffs/npm/@elastic%2felasticsearch/8.15.2/8.16.0) | --- ### Release Notes <details> <summary>elastic/elasticsearch-js (@elastic/elasticsearch)</summary> ### [`v8.16.0`](https://togithub.com/elastic/elasticsearch-js/releases/tag/v8.16.0) [Compare Source](https://togithub.com/elastic/elasticsearch-js/compare/v8.15.2...v8.16.0) [Changelog](https://www.elastic.co/guide/en/elasticsearch/client/javascript-api/8.16/changelog-client.html) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate).  Co-authored-by: elastic-renovate-prod[bot] <174716857+elastic-renovate-prod[bot]@users.noreply.github.com>
elastic · Dec 13, 2024 · e061b4c · e061b4c
1 parent e287528
commit e061b4c
Show file tree

Hide file tree

Showing 21 changed files with 164 additions and 43 deletions.
diff --git a/package.json b/package.json
@@ -115,7 +115,7 @@
     "@elastic/datemath": "5.0.3",
     "@elastic/ebt": "^1.1.1",
     "@elastic/ecs": "^8.11.1",
-    "@elastic/elasticsearch": "^8.15.2",
+    "@elastic/elasticsearch": "^8.16.0",
     "@elastic/ems-client": "8.5.3",
     "@elastic/eui": "98.1.0-borealis.0",
     "@elastic/eui-theme-borealis": "0.0.4",

diff --git a/...ed-objects/core-saved-objects-api-server-internal/src/lib/apis/open_point_in_time.test.ts b/...ed-objects/core-saved-objects-api-server-internal/src/lib/apis/open_point_in_time.test.ts
@@ -36,6 +36,7 @@ import {
   createGenericNotFoundErrorPayload,
 } from '../../test_helpers/repository.test.common';
 import { PointInTimeFinder } from '../point_in_time_finder';
+import { OpenPointInTimeResponse } from '@elastic/elasticsearch/lib/api/types';
 
 describe('SavedObjectsRepository', () => {
   let client: ReturnType<typeof elasticsearchClientMock.createElasticsearchClient>;
@@ -81,7 +82,7 @@ describe('SavedObjectsRepository', () => {
   describe('#openPointInTimeForType', () => {
     const type = 'index-pattern';
 
-    const generateResults = (id?: string) => ({ id: id || 'id' });
+    const generateResults = (id?: string) => ({ id: id || 'id' } as OpenPointInTimeResponse);
     const successResponse = async (type: string, options?: SavedObjectsOpenPointInTimeOptions) => {
       client.openPointInTime.mockResponseOnce(generateResults());
       const result = await repository.openPointInTimeForType(type, options);
@@ -136,7 +137,7 @@ describe('SavedObjectsRepository', () => {
       it(`throws when ES is unable to find the index`, async () => {
         client.openPointInTime.mockResolvedValueOnce(
           elasticsearchClientMock.createSuccessTransportRequestPromise(
-            { id: 'error' },
+            { id: 'error' } as OpenPointInTimeResponse,
             { statusCode: 404 }
           )
         );

diff --git a/...ects/core-saved-objects-api-server-internal/src/lib/repository.security_extension.test.ts b/...ects/core-saved-objects-api-server-internal/src/lib/repository.security_extension.test.ts
@@ -66,6 +66,7 @@ import {
 import { savedObjectsExtensionsMock } from '../mocks/saved_objects_extensions.mock';
 import { arrayMapsAreEqual } from '@kbn/core-saved-objects-utils-server';
 import { mockAuthenticatedUser } from '@kbn/core-security-common/mocks';
+import { OpenPointInTimeResponse } from '@elastic/elasticsearch/lib/api/types';
 
 describe('SavedObjectsRepository Security Extension', () => {
   let client: ReturnType<typeof elasticsearchClientMock.createElasticsearchClient>;
@@ -676,7 +677,7 @@ describe('SavedObjectsRepository Security Extension', () => {
     test(`returns result when partially authorized`, async () => {
       setupAuthorizeFunc(mockSecurityExt.authorizeOpenPointInTime, 'partially_authorized');
 
-      client.openPointInTime.mockResponseOnce({ id });
+      client.openPointInTime.mockResponseOnce({ id } as OpenPointInTimeResponse);
       const result = await repository.openPointInTimeForType(type);
 
       expect(mockSecurityExt.authorizeOpenPointInTime).toHaveBeenCalledTimes(1);
@@ -687,7 +688,7 @@ describe('SavedObjectsRepository Security Extension', () => {
     test(`returns result when fully authorized`, async () => {
       setupAuthorizeFunc(mockSecurityExt.authorizeOpenPointInTime, 'fully_authorized');
 
-      client.openPointInTime.mockResponseOnce({ id });
+      client.openPointInTime.mockResponseOnce({ id } as OpenPointInTimeResponse);
       const result = await repository.openPointInTimeForType(type);
 
       expect(mockSecurityExt.authorizeOpenPointInTime).toHaveBeenCalledTimes(1);
@@ -702,7 +703,7 @@ describe('SavedObjectsRepository Security Extension', () => {
 
     test(`calls authorizeOpenPointInTime with correct parameters`, async () => {
       setupAuthorizeFunc(mockSecurityExt.authorizeOpenPointInTime, 'fully_authorized');
-      client.openPointInTime.mockResponseOnce({ id });
+      client.openPointInTime.mockResponseOnce({ id } as OpenPointInTimeResponse);
       const namespaces = [namespace, 'x', 'y', 'z'];
       await repository.openPointInTimeForType(type, { namespaces });
       expect(mockSecurityExt.authorizeOpenPointInTime).toHaveBeenCalledTimes(1);

diff --git a/packages/kbn-generate-csv/src/lib/search_cursor_pit.test.ts b/packages/kbn-generate-csv/src/lib/search_cursor_pit.test.ts
@@ -17,6 +17,7 @@ import { createSearchSourceMock } from '@kbn/data-plugin/common/search/search_so
 import { createSearchRequestHandlerContext } from '@kbn/data-plugin/server/search/mocks';
 import type { SearchCursorSettings } from './search_cursor';
 import { SearchCursorPit } from './search_cursor_pit';
+import { OpenPointInTimeResponse } from '@elastic/elasticsearch/lib/api/types';
 
 class TestSearchCursorPit extends SearchCursorPit {
   constructor(...args: ConstructorParameters<typeof SearchCursorPit>) {
@@ -69,7 +70,7 @@ describe('CSV Export Search Cursor', () => {
 
     openPointInTimeSpy = jest
       .spyOn(es.asCurrentUser, 'openPointInTime')
-      .mockResolvedValue({ id: 'somewhat-pit-id' });
+      .mockResolvedValue({ id: 'somewhat-pit-id' } as OpenPointInTimeResponse);
 
     logger = loggingSystemMock.createLogger();
   });

diff --git a/packages/kbn-generate-csv/src/lib/search_cursor_scroll.test.ts b/packages/kbn-generate-csv/src/lib/search_cursor_scroll.test.ts
@@ -16,6 +16,7 @@ import { createSearchSourceMock } from '@kbn/data-plugin/common/search/search_so
 import { createSearchRequestHandlerContext } from '@kbn/data-plugin/server/search/mocks';
 import type { SearchCursorSettings } from './search_cursor';
 import { SearchCursorScroll } from './search_cursor_scroll';
+import type { OpenPointInTimeResponse } from '@elastic/elasticsearch/lib/api/types';
 
 class TestSearchCursorScroll extends SearchCursorScroll {
   constructor(...args: ConstructorParameters<typeof SearchCursorScroll>) {
@@ -47,7 +48,9 @@ describe('CSV Export Search Cursor', () => {
 
     es = elasticsearchServiceMock.createScopedClusterClient();
     data = createSearchRequestHandlerContext();
-    jest.spyOn(es.asCurrentUser, 'openPointInTime').mockResolvedValue({ id: 'simply-scroll-id' });
+    jest
+      .spyOn(es.asCurrentUser, 'openPointInTime')
+      .mockResolvedValue({ id: 'simply-scroll-id' } as OpenPointInTimeResponse);
 
     logger = loggingSystemMock.createLogger();
   });

diff --git a/src/plugins/data/server/search/strategies/sql_search/sql_search_strategy.ts b/src/plugins/data/server/search/strategies/sql_search/sql_search_strategy.ts
@@ -11,7 +11,7 @@ import type { IncomingHttpHeaders } from 'http';
 import type { IScopedClusterClient, Logger } from '@kbn/core/server';
 import { catchError, tap } from 'rxjs';
 import type { DiagnosticResult } from '@elastic/transport';
-import { SqlQueryResponse } from '@elastic/elasticsearch/lib/api/types';
+import { SqlQueryResponse, type SqlQuerySqlFormat } from '@elastic/elasticsearch/lib/api/types';
 import { getKbnServerError } from '@kbn/kibana-utils-plugin/server';
 import { getKbnSearchError } from '../../report_search_error';
 import type { ISearchStrategy, SearchStrategyDependencies } from '../../types';
@@ -61,9 +61,9 @@ export const sqlSearchStrategyProvider = (
       } else {
         ({ headers, body, meta } = await client.sql.query(
           {
-            format: params.format ?? 'json',
             ...getDefaultAsyncSubmitParams(searchConfig, options),
             ...params,
+            format: (params.format ?? 'json') as SqlQuerySqlFormat,
           },
           { ...options.transport, signal: options.abortSignal, meta: true }
         ));

diff --git a/src/plugins/data_view_field_editor/server/routes/field_preview.ts b/src/plugins/data_view_field_editor/server/routes/field_preview.ts
@@ -81,9 +81,8 @@ export const registerFieldPreviewRoute = ({ router }: RouteDependencies): void =
       };
 
       try {
-        // client types need to be update to support this request format
+        // client types need to be updated to support this request format
         // when it does, supply response types
-        // @ts-expect-error
         const { result } = await client.asCurrentUser.scriptsPainlessExecute(body);
 
         return res.ok({ body: { values: result } });

diff --git a/src/plugins/telemetry/server/telemetry_collection/get_cluster_stats.ts b/src/plugins/telemetry/server/telemetry_collection/get_cluster_stats.ts
@@ -20,8 +20,6 @@ export async function getClusterStats(esClient: ElasticsearchClient) {
   return await esClient.cluster.stats(
     {
       timeout: CLUSTER_STAT_TIMEOUT,
-
-      // @ts-expect-error
       include_remotes: true,
     },
     {

diff --git a/x-pack/platform/plugins/shared/ml/common/types/management.ts b/x-pack/platform/plugins/shared/ml/common/types/management.ts
@@ -29,7 +29,7 @@ export interface AnalyticsManagementItems {
 export interface TrainedModelsManagementItems {
   id: string;
   description: string;
-  state: estypes.MlDeploymentState | '';
+  state: estypes.MlDeploymentAssignmentState | '';
   type: Array<string | undefined>;
   spaces: string[];
 }

diff --git a/...lugins/shared/ml/public/application/model_management/test_models/models/inference_base.ts b/...lugins/shared/ml/public/application/model_management/test_models/models/inference_base.ts
@@ -376,10 +376,10 @@ export abstract class InferenceBase<TInferResponse> {
     };
   }
 
-  // @ts-expect-error error does not exist in type
   protected getDocFromResponse({ doc, error }: estypes.IngestSimulatePipelineSimulation) {
     if (doc === undefined) {
       if (error) {
+        // @ts-expect-error Error is now typed in estypes. However, I doubt that it doesn't get the HTTP wrapper expected.
         this.setFinishedWithErrors(error);
         throw Error(error.reason);
       }

diff --git a/...d/observability_solution/observability_ai_assistant/server/routes/knowledge_base/route.ts b/...d/observability_solution/observability_ai_assistant/server/routes/knowledge_base/route.ts
@@ -12,7 +12,7 @@ import * as t from 'io-ts';
 import {
   InferenceInferenceEndpointInfo,
   MlDeploymentAllocationState,
-  MlDeploymentState,
+  MlDeploymentAssignmentState,
 } from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
 import moment from 'moment';
 import { createObservabilityAIAssistantServerRoute } from '../create_observability_ai_assistant_server_route';
@@ -34,7 +34,7 @@ const getKnowledgeBaseStatus = createObservabilityAIAssistantServerRoute({
     enabled: boolean;
     endpoint?: Partial<InferenceInferenceEndpointInfo>;
     model_stats?: {
-      deployment_state: MlDeploymentState | undefined;
+      deployment_state: MlDeploymentAssignmentState | undefined;
       allocation_state: MlDeploymentAllocationState | undefined;
     };
   }> => {

diff --git a/x-pack/plugins/alerting/server/alerts_service/alerts_service.test.ts b/x-pack/plugins/alerting/server/alerts_service/alerts_service.test.ts
@@ -102,6 +102,7 @@ const GetDataStreamResponse: IndicesGetDataStreamResponse = {
       template: 'ignored',
       next_generation_managed_by: 'Index Lifecycle Management',
       prefer_ilm: false,
+      rollover_on_write: false,
     },
   ],
 };

diff --git a/x-pack/plugins/elastic_assistant/server/ai_assistant_service/index.test.ts b/x-pack/plugins/elastic_assistant/server/ai_assistant_service/index.test.ts
@@ -79,6 +79,7 @@ const GetDataStreamResponse: IndicesGetDataStreamResponse = {
       template: 'ignored',
       next_generation_managed_by: 'Data stream lifecycle',
       prefer_ilm: false,
+      rollover_on_write: false,
     },
   ],
 };

diff --git a/x-pack/plugins/event_log/server/es/cluster_client_adapter.test.ts b/x-pack/plugins/event_log/server/es/cluster_client_adapter.test.ts
@@ -569,6 +569,7 @@ export const GetDataStreamsResponse: estypes.IndicesGetDataStreamResponse = {
       template: '',
       hidden: true,
       prefer_ilm: false,
+      rollover_on_write: true,
       next_generation_managed_by: 'Index Lifecycle Management',
     },
   ],

diff --git a/x-pack/plugins/index_management/common/types/data_streams.ts b/x-pack/plugins/index_management/common/types/data_streams.ts
@@ -81,7 +81,7 @@ export interface DataStreamIndex {
   name: string;
   uuid: string;
   preferILM: boolean;
-  managedBy: string;
+  managedBy?: string;
 }
 
 export interface DataRetention {

diff --git a/x-pack/plugins/index_management/public/application/lib/data_streams.tsx b/x-pack/plugins/index_management/public/application/lib/data_streams.tsx
@@ -87,28 +87,30 @@ export const isDataStreamFullyManagedByILM = (dataStream?: DataStream | null) =>
   return (
     dataStream?.nextGenerationManagedBy?.toLowerCase() === 'index lifecycle management' &&
     dataStream?.indices?.every(
-      (index) => index.managedBy.toLowerCase() === 'index lifecycle management'
+      (index) => index.managedBy?.toLowerCase() === 'index lifecycle management'
     )
   );
 };
 
 export const isDataStreamFullyManagedByDSL = (dataStream?: DataStream | null) => {
   return (
     dataStream?.nextGenerationManagedBy?.toLowerCase() === 'data stream lifecycle' &&
-    dataStream?.indices?.every((index) => index.managedBy.toLowerCase() === 'data stream lifecycle')
+    dataStream?.indices?.every(
+      (index) => index.managedBy?.toLowerCase() === 'data stream lifecycle'
+    )
   );
 };
 
 export const isDSLWithILMIndices = (dataStream?: DataStream | null) => {
   if (dataStream?.nextGenerationManagedBy?.toLowerCase() === 'data stream lifecycle') {
     const ilmIndices = dataStream?.indices?.filter(
-      (index) => index.managedBy.toLowerCase() === 'index lifecycle management'
+      (index) => index.managedBy?.toLowerCase() === 'index lifecycle management'
     );
     const dslIndices = dataStream?.indices?.filter(
-      (index) => index.managedBy.toLowerCase() === 'data stream lifecycle'
+      (index) => index.managedBy?.toLowerCase() === 'data stream lifecycle'
     );
 
-    // When there arent any ILM indices, there's no need to show anything.
+    // When there aren't any ILM indices, there's no need to show anything.
     if (!ilmIndices?.length) {
       return;
     }

diff --git a/x-pack/plugins/index_management/server/lib/data_stream_serialization.ts b/x-pack/plugins/index_management/server/lib/data_stream_serialization.ts
@@ -43,13 +43,13 @@ export function deserializeDataStream(dataStreamFromEs: EnhancedDataStreamFromEs
       ({
         index_name: indexName,
         index_uuid: indexUuid,
-        prefer_ilm: preferILM,
+        prefer_ilm: preferILM = false,
         managed_by: managedBy,
       }: {
         index_name: string;
         index_uuid: string;
-        prefer_ilm: boolean;
-        managed_by: string;
+        prefer_ilm?: boolean;
+        managed_by?: string;
       }) => ({
         name: indexName,
         uuid: indexUuid,

diff --git a/x-pack/plugins/observability_solution/dataset_quality/server/services/data_stream.ts b/x-pack/plugins/observability_solution/dataset_quality/server/services/data_stream.ts
@@ -20,7 +20,6 @@ class DataStreamService {
     try {
       const { data_streams: dataStreamsInfo } = await esClient.indices.getDataStream({
         name: datasetName,
-        // @ts-expect-error
         verbose: true,
       });
 

diff --git a/x-pack/plugins/security_solution/server/endpoint/mocks/utils.mock.ts b/x-pack/plugins/security_solution/server/endpoint/mocks/utils.mock.ts
@@ -6,7 +6,7 @@
  */
 
 import type { ElasticsearchClientMock } from '@kbn/core-elasticsearch-client-server-mocks';
-import type { SearchResponse } from '@elastic/elasticsearch/lib/api/types';
+import type { OpenPointInTimeResponse, SearchResponse } from '@elastic/elasticsearch/lib/api/types';
 import { v4 as uuidV4 } from 'uuid';
 import { BaseDataGenerator } from '../../../common/endpoint/data_generators/base_data_generator';
 
@@ -45,14 +45,14 @@ export const applyEsClientSearchMock = <TDocument = unknown>({
       const pitResponse = { id: `mock:pit:${index}:${uuidV4()}` };
       openedPitIds.add(pitResponse.id);
 
-      return pitResponse;
+      return pitResponse as OpenPointInTimeResponse;
     }
 
     if (priorOpenPointInTimeImplementation) {
       return priorOpenPointInTimeImplementation(...args);
     }
 
-    return { id: 'mock' };
+    return { id: 'mock' } as OpenPointInTimeResponse;
   });
 
   esClientMock.closePointInTime.mockImplementation(async (...args) => {

diff --git a/...curity_solution_api_integration/test_suites/entity_analytics/entity_store/utils/ingest.ts b/...curity_solution_api_integration/test_suites/entity_analytics/entity_store/utils/ingest.ts
@@ -31,7 +31,6 @@ export const applyIngestProcessorToDoc = async (
 
   const firstDoc = res.docs?.[0];
 
-  // @ts-expect-error error is not in the types
   const error = firstDoc?.error;
   if (error) {
     log.error('Full painless error below: ');