[Upgrade Assistant] Hide features that don't need to be migrated from…

… flyout (#115535) * Filter out system indices that dont require migration on server side * Rename to attrs to migration * Update flyout snapshot.
elastic · Oct 19, 2021 · 128afbb · 128afbb
1 parent 135b03c
commit 128afbb
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 2 deletions.
diff --git a/x-pack/plugins/upgrade_assistant/server/routes/system_indices_migration.test.ts b/x-pack/plugins/upgrade_assistant/server/routes/system_indices_migration.test.ts
@@ -29,6 +29,17 @@ const mockedResponse = {
         },
       ],
     },
+    {
+      feature_name: 'kibana',
+      minimum_index_version: '7.1.2',
+      upgrade_status: 'MIGRATION_NEEDED',
+      indices: [
+        {
+          index: '.kibana',
+          version: '7.1.2',
+        },
+      ],
+    },
   ],
   migration_status: 'MIGRATION_NEEDED',
 };
@@ -75,7 +86,12 @@ describe('Migrate system indices API', () => {
         method: 'GET',
         path: '/_migration/system_features',
       });
-      expect(resp.payload).toEqual(mockedResponse);
+      expect(resp.payload).toEqual({
+        ...mockedResponse,
+        features: mockedResponse.features.filter(
+          (feature) => feature.migration_status !== 'NO_MIGRATION_NEEDED'
+        ),
+      });
     });
 
     it('returns an error if it throws', async () => {

diff --git a/x-pack/plugins/upgrade_assistant/server/routes/system_indices_migration.ts b/x-pack/plugins/upgrade_assistant/server/routes/system_indices_migration.ts
@@ -34,7 +34,12 @@ export function registerSystemIndicesMigrationRoutes({
           const status = await getESSystemIndicesMigrationStatus(client.asCurrentUser);
 
           return response.ok({
-            body: status,
+            body: {
+              ...status,
+              features: status.features.filter(
+                (feature) => feature.migration_status !== 'NO_MIGRATION_NEEDED'
+              ),
+            },
           });
         } catch (error) {
           return handleEsError({ error, response });