[CI] Balance ci groups (#110711) (#113164)

# Conflicts: # test/functional/apps/visualize/index.ts # x-pack/test/functional/config.js
elastic · Sep 27, 2021 · 0a8a334 · 0a8a334
1 parent 7194c68
commit 0a8a334
Show file tree

Hide file tree

Showing 8 changed files with 17 additions and 8 deletions.
diff --git a/test/functional/apps/visualize/index.ts b/test/functional/apps/visualize/index.ts
@@ -94,13 +94,18 @@ export default function ({ getService, loadTestFile }: FtrProviderContext) {
       loadTestFile(require.resolve('./_region_map'));
     });
 
+    describe('visualize ciGroup8', function () {
+      this.tags('ciGroup8');
+
+      loadTestFile(require.resolve('./_tsvb_chart'));
+    });
+
     describe('visualize ciGroup11', function () {
       this.tags('ciGroup11');
 
       loadTestFile(require.resolve('./_tag_cloud'));
       loadTestFile(require.resolve('./_vertical_bar_chart'));
       loadTestFile(require.resolve('./_vertical_bar_chart_nontimeindex'));
-      loadTestFile(require.resolve('./_tsvb_chart'));
       loadTestFile(require.resolve('./_tsvb_time_series'));
       loadTestFile(require.resolve('./_tsvb_markdown'));
       loadTestFile(require.resolve('./_tsvb_table'));

diff --git a/x-pack/test/case_api_integration/security_and_spaces/tests/basic/index.ts b/x-pack/test/case_api_integration/security_and_spaces/tests/basic/index.ts
@@ -12,7 +12,7 @@ import { createSpacesAndUsers, deleteSpacesAndUsers } from '../../../common/lib/
 export default ({ loadTestFile, getService }: FtrProviderContext): void => {
   describe('cases security and spaces enabled: basic', function () {
     // Fastest ciGroup for the moment.
-    this.tags('ciGroup5');
+    this.tags('ciGroup13');
 
     before(async () => {
       await createSpacesAndUsers(getService);

diff --git a/x-pack/test/case_api_integration/security_and_spaces/tests/trial/index.ts b/x-pack/test/case_api_integration/security_and_spaces/tests/trial/index.ts
@@ -12,7 +12,7 @@ import { createSpacesAndUsers, deleteSpacesAndUsers } from '../../../common/lib/
 export default ({ loadTestFile, getService }: FtrProviderContext): void => {
   describe('cases security and spaces enabled: trial', function () {
     // Fastest ciGroup for the moment.
-    this.tags('ciGroup5');
+    this.tags('ciGroup13');
 
     before(async () => {
       await createSpacesAndUsers(getService);

diff --git a/x-pack/test/functional/apps/apm/index.ts b/x-pack/test/functional/apps/apm/index.ts
@@ -9,7 +9,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';
 
 export default function ({ loadTestFile }: FtrProviderContext) {
   describe('APM specs', function () {
-    this.tags('ciGroup6');
+    this.tags('ciGroup10');
     loadTestFile(require.resolve('./feature_controls'));
     loadTestFile(require.resolve('./correlations'));
   });

diff --git a/x-pack/test/functional/apps/lens/index.ts b/x-pack/test/functional/apps/lens/index.ts
@@ -25,10 +25,14 @@ export default function ({ getService, loadTestFile }: FtrProviderContext) {
       await esArchiver.unload('x-pack/test/functional/es_archives/lens/basic');
     });
 
+    describe('', function () {
+      this.tags(['ciGroup3', 'skipFirefox']);
+      loadTestFile(require.resolve('./smokescreen'));
+    });
+
     describe('', function () {
       this.tags(['ciGroup4', 'skipFirefox']);
 
-      loadTestFile(require.resolve('./smokescreen'));
       loadTestFile(require.resolve('./add_to_dashboard'));
       loadTestFile(require.resolve('./table'));
       loadTestFile(require.resolve('./runtime_fields'));

diff --git a/x-pack/test/functional/apps/uptime/index.ts b/x-pack/test/functional/apps/uptime/index.ts
@@ -42,7 +42,7 @@ export default ({ loadTestFile, getService }: FtrProviderContext) => {
   const uptime = getService('uptime');
 
   describe('Uptime app', function () {
-    this.tags('ciGroup6');
+    this.tags('ciGroup10');
 
     beforeEach('delete settings', async () => {
       await deleteUptimeSettingsObject(server);

diff --git a/x-pack/test/functional/apps/uptime/synthetics_integration.ts b/x-pack/test/functional/apps/uptime/synthetics_integration.ts
@@ -78,7 +78,7 @@ export default function ({ getPageObjects, getService }: FtrProviderContext) {
   });
 
   describe('When on the Synthetics Integration Policy Create Page', function () {
-    this.tags(['ciGroup6']);
+    this.tags(['ciGroup10']);
     const basicConfig = {
       name: monitorName,
       apmServiceName: 'Sample APM Service',

diff --git a/x-pack/test/functional/config.js b/x-pack/test/functional/config.js
@@ -33,7 +33,6 @@ export default async function ({ readConfigFile }) {
       resolve(__dirname, './apps/discover'),
       resolve(__dirname, './apps/security'),
       resolve(__dirname, './apps/spaces'),
-      resolve(__dirname, './apps/lens'),
       resolve(__dirname, './apps/logstash'),
       resolve(__dirname, './apps/grok_debugger'),
       resolve(__dirname, './apps/infra'),
@@ -60,6 +59,7 @@ export default async function ({ readConfigFile }) {
       resolve(__dirname, './apps/management'),
       resolve(__dirname, './apps/reporting'),
       resolve(__dirname, './apps/observability'),
+      resolve(__dirname, './apps/lens'), // smokescreen tests cause flakiness in other tests
 
       //This upgrade assistant file needs to be run after the rest of the jobs because of
       //it being destructive.