[UII] Add retries to starting Kibana for jest preconfiguration integr…

…ation tests (#191318) ## Summary Resolves #172759 Resolves #172760 Resolves #172761 Add retries to starting Kibana for this test suite to hopefully mitigate future flakiness. ### Checklist - ~~[Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed~~ Doesn't seem like I can run Jest integration tests here.
elastic · Aug 27, 2024 · 0a8d6a0 · 0a8d6a0
1 parent 2441e9a
commit 0a8d6a0
Showing 1 changed file with 61 additions and 43 deletions.
diff --git a/x-pack/plugins/fleet/server/integration_tests/cloud_preconfiguration.test.ts b/x-pack/plugins/fleet/server/integration_tests/cloud_preconfiguration.test.ts
@@ -49,56 +49,74 @@ describe('Fleet cloud preconfiguration', () => {
 
     esServer = await startES();
     const startOrRestartKibana = async (kbnConfig: any = defaultKbnConfig) => {
-      if (kbnServer) {
-        await kbnServer.stop();
-      }
-
-      const root = createRootWithCorePlugins(
-        {
-          xpack: {
-            ...kbnConfig.xpack,
-            fleet: {
-              ...kbnConfig.xpack.fleet,
-              registryUrl,
+      const maxTries = 3;
+      let currentTry = 0;
+
+      const startOrRestart = async () => {
+        if (kbnServer) {
+          await kbnServer.stop();
+        }
+
+        const root = createRootWithCorePlugins(
+          {
+            xpack: {
+              ...kbnConfig.xpack,
+              fleet: {
+                ...kbnConfig.xpack.fleet,
+                registryUrl,
+              },
             },
-          },
-          logging: {
-            appenders: {
-              file: {
-                type: 'file',
-                fileName: logFilePath,
-                layout: {
-                  type: 'json',
+            logging: {
+              appenders: {
+                file: {
+                  type: 'file',
+                  fileName: logFilePath,
+                  layout: {
+                    type: 'json',
+                  },
                 },
               },
+              loggers: [
+                {
+                  name: 'root',
+                  appenders: ['file'],
+                },
+                {
+                  name: 'plugins.fleet',
+                  level: 'all',
+                },
+              ],
             },
-            loggers: [
-              {
-                name: 'root',
-                appenders: ['file'],
-              },
-              {
-                name: 'plugins.fleet',
-                level: 'all',
-              },
-            ],
           },
-        },
-        { oss: false }
-      );
-
-      await root.preboot();
-      const coreSetup = await root.setup();
-      const coreStart = await root.start();
-
-      kbnServer = {
-        root,
-        coreSetup,
-        coreStart,
-        stop: async () => await root.shutdown(),
+          { oss: false }
+        );
+
+        await root.preboot();
+        const coreSetup = await root.setup();
+        const coreStart = await root.start();
+
+        kbnServer = {
+          root,
+          coreSetup,
+          coreStart,
+          stop: async () => await root.shutdown(),
+        };
+
+        await waitForFleetSetup(kbnServer.root);
       };
-      await waitForFleetSetup(kbnServer.root);
+
+      try {
+        currentTry++;
+        await startOrRestart();
+      } catch (e) {
+        if (currentTry < maxTries) {
+          await startOrRestart();
+        } else {
+          throw e;
+        }
+      }
     };
+
     await startOrRestartKibana();
 
     return {