fix backport

x-pack/plugins/upgrade_assistant/public/application/components/tabs/checkup/deprecations/reindex/flyout/checklist_step.tsx

@@ -16,13 +16,12 @@ import {
   EuiFlexItem,
   EuiFlyoutBody,
   EuiFlyoutFooter,
-  EuiLink,
   EuiSpacer,
   EuiTitle,
 } from '@elastic/eui';
 import { FormattedMessage } from '@kbn/i18n/react';
 
-import { ReindexStatus, ReindexWarning } from '../../../../../../../../common/types';
+import { ReindexStatus } from '../../../../../../../../common/types';
 import { LoadingState } from '../../../../../types';
 import { ReindexState } from '../polling_service';
 import { ReindexProgress } from './progress';
@@ -78,7 +77,7 @@ export const ChecklistFlyoutStep: React.FunctionComponent<{
   cancelReindex: () => void;
   http: HttpSetup;
 }> = ({ closeFlyout, reindexState, startReindex, cancelReindex, http, renderGlobalCallouts }) => {
-  const { loadingState, status, hasRequiredPrivileges, reindexWarnings } = reindexState;
+  const { loadingState, status, hasRequiredPrivileges } = reindexState;
   const loading = loadingState === LoadingState.Loading || status === ReindexStatus.inProgress;
 
   return (

x-pack/plugins/upgrade_assistant/server/lib/es_migration_apis.test.ts

@@ -37,12 +37,12 @@ describe('getUpgradeAssistantStatus', () => {
   esClient.asCurrentUser.indices.resolveIndex.mockResolvedValue(asApiResponse(resolvedIndices));
 
   it('calls /_migration/deprecations', async () => {
-    await getUpgradeAssistantStatus(esClient, false, []);
+    await getUpgradeAssistantStatus(esClient, false);
     expect(esClient.asCurrentUser.migration.deprecations).toHaveBeenCalled();
   });
 
   it('returns the correct shape of data', async () => {
-    const resp = await getUpgradeAssistantStatus(esClient, false, []);
+    const resp = await getUpgradeAssistantStatus(esClient, false);
     expect(resp).toMatchSnapshot();
   });
 
@@ -56,7 +56,7 @@ describe('getUpgradeAssistantStatus', () => {
       })
     );
 
-    await expect(getUpgradeAssistantStatus(esClient, false, [])).resolves.toHaveProperty(
+    await expect(getUpgradeAssistantStatus(esClient, false)).resolves.toHaveProperty(
       'readyForUpgrade',
       false
     );
@@ -72,7 +72,7 @@ describe('getUpgradeAssistantStatus', () => {
       })
     );
 
-    await expect(getUpgradeAssistantStatus(esClient, false, [])).resolves.toHaveProperty(
+    await expect(getUpgradeAssistantStatus(esClient, false)).resolves.toHaveProperty(
       'readyForUpgrade',
       true
     );
@@ -94,7 +94,7 @@ describe('getUpgradeAssistantStatus', () => {
       })
     );
 
-    const result = await getUpgradeAssistantStatus(esClient, true, []);
+    const result = await getUpgradeAssistantStatus(esClient, true);
 
     expect(result).toHaveProperty('readyForUpgrade', true);
     expect(result).toHaveProperty('cluster', []);

x-pack/plugins/upgrade_assistant/server/lib/es_migration_apis.ts

@@ -12,17 +12,15 @@ import {
   UpgradeAssistantStatus,
 } from '../../common/types';
 
-import { isSystemIndex } from './reindexing';
-
 import { esIndicesStateCheck } from './es_indices_state_check';
 
 export async function getUpgradeAssistantStatus(
   dataClient: IScopedClusterClient,
-  isCloudEnabled: boolean,
+  isCloudEnabled: boolean
 ): Promise<UpgradeAssistantStatus> {
-  const [{ body: deprecations }] = await Promise.all([
-    dataClient.asCurrentUser.migration.deprecations<DeprecationAPIResponse>(),
-  ]);
+  const {
+    body: deprecations,
+  } = await dataClient.asCurrentUser.migration.deprecations<DeprecationAPIResponse>();
 
   const cluster = getClusterDeprecations(deprecations, isCloudEnabled);
   const indices = getCombinedIndexInfos(deprecations);
@@ -50,32 +48,19 @@ export async function getUpgradeAssistantStatus(
 }
 
 // Reformats the index deprecations to an array of deprecation warnings extended with an index field.
-const getCombinedIndexInfos = (
-  deprecations: DeprecationAPIResponse,
-) => {
-
-  return (
-    Object.keys(deprecations.index_settings)
-      .reduce((indexDeprecations, indexName) => {
-        return indexDeprecations.concat(
-          deprecations.index_settings[indexName].map(
-            (d) =>
-              ({
-                ...d,
-                index: indexName,
-                reindex: /Index created before/.test(d.message),
-                needsDefaultFields: /Number of fields exceeds automatic field expansion limit/.test(
-                  d.message
-                ),
-              } as EnrichedDeprecationInfo)
-          )
-        );
-      }, [] as EnrichedDeprecationInfo[])
-      // Filter out warnings for system indices until we know more about what changes are required for the
-      // next upgrade in a future minor version.
-      .filter((deprecation) => !isSystemIndex(deprecation.index!))
-  );
-};
+const getCombinedIndexInfos = (deprecations: DeprecationAPIResponse) =>
+  Object.keys(deprecations.index_settings).reduce((indexDeprecations, indexName) => {
+    return indexDeprecations.concat(
+      deprecations.index_settings[indexName].map(
+        (d) =>
+          ({
+            ...d,
+            index: indexName,
+            reindex: /Index created before/.test(d.message),
+          } as EnrichedDeprecationInfo)
+      )
+    );
+  }, [] as EnrichedDeprecationInfo[]);
 
 const getClusterDeprecations = (deprecations: DeprecationAPIResponse, isCloudEnabled: boolean) => {
   const combined = deprecations.cluster_settings