Skip to content

Commit

Permalink
chore(NA): rebalance x-pack cigroups (#85797) (#86146)
Browse files Browse the repository at this point in the history
# Conflicts:
#	vars/kibanaCoverage.groovy

Co-authored-by: Tiago Costa <[email protected]>
  • Loading branch information
Tyler Smalley and mistic authored Dec 16, 2020
1 parent 046ac36 commit c24da96
Show file tree
Hide file tree
Showing 20 changed files with 78 additions and 55 deletions.
2 changes: 2 additions & 0 deletions .ci/es-snapshots/Jenkinsfile_verify_es
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,8 @@ kibanaPipeline(timeoutMinutes: 150) {
'xpack-ciGroup9': kibanaPipeline.xpackCiGroupProcess(9),
'xpack-ciGroup10': kibanaPipeline.xpackCiGroupProcess(10),
'xpack-ciGroup11': kibanaPipeline.xpackCiGroupProcess(11),
'xpack-ciGroup12': kibanaPipeline.xpackCiGroupProcess(12),
'xpack-ciGroup13': kibanaPipeline.xpackCiGroupProcess(13),
]),
])
}
Expand Down
2 changes: 2 additions & 0 deletions .ci/jobs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@ JOB:
- x-pack-ciGroup9
- x-pack-ciGroup10
- x-pack-ciGroup11
- x-pack-ciGroup12
- x-pack-ciGroup13
- x-pack-accessibility
- x-pack-visualRegression

Expand Down
2 changes: 1 addition & 1 deletion test/functional/apps/visualize/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ export default function ({ getService, loadTestFile }: FtrProviderContext) {
});

describe('', function () {
this.tags('ciGroup11');
this.tags('ciGroup4');

loadTestFile(require.resolve('./_line_chart'));
loadTestFile(require.resolve('./_pie_chart'));
Expand Down
4 changes: 3 additions & 1 deletion test/scripts/jenkins_xpack_build_kibana.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,9 @@ node scripts/functional_tests --assert-none-excluded \
--include-tag ciGroup8 \
--include-tag ciGroup9 \
--include-tag ciGroup10 \
--include-tag ciGroup11
--include-tag ciGroup11 \
--include-tag ciGroup12 \
--include-tag ciGroup13

echo " -> building and extracting default Kibana distributable for use in functional tests"
cd "$KIBANA_DIR"
Expand Down
2 changes: 1 addition & 1 deletion vars/tasks.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ def functionalXpack(Map params = [:]) {
kibanaPipeline.buildXpack(10)

if (config.ciGroups) {
def ciGroups = 1..11
def ciGroups = 1..13
tasks(ciGroups.collect { kibanaPipeline.xpackCiGroupProcess(it) })
}

Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/api_integration_basic/apis/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('apis', function () {
this.tags('ciGroup2');
this.tags('ciGroup11');

loadTestFile(require.resolve('./ml'));
loadTestFile(require.resolve('./transform'));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,30 @@ import { FtrProviderContext } from '../../../common/ftr_provider_context';
// eslint-disable-next-line import/no-default-export
export default ({ loadTestFile }: FtrProviderContext): void => {
describe('Detection exceptions data types and operators', function () {
this.tags('ciGroup11');
describe('', function () {
this.tags('ciGroup11');

loadTestFile(require.resolve('./date'));
loadTestFile(require.resolve('./double'));
loadTestFile(require.resolve('./float'));
loadTestFile(require.resolve('./integer'));
loadTestFile(require.resolve('./ip'));
loadTestFile(require.resolve('./ip_array'));
loadTestFile(require.resolve('./keyword'));
loadTestFile(require.resolve('./keyword_array'));
loadTestFile(require.resolve('./long'));
loadTestFile(require.resolve('./text'));
loadTestFile(require.resolve('./text_array'));
loadTestFile(require.resolve('./date'));
loadTestFile(require.resolve('./double'));
loadTestFile(require.resolve('./float'));
loadTestFile(require.resolve('./integer'));
});

describe('', function () {
this.tags('ciGroup12');

loadTestFile(require.resolve('./ip'));
loadTestFile(require.resolve('./ip_array'));
loadTestFile(require.resolve('./keyword'));
loadTestFile(require.resolve('./keyword_array'));
loadTestFile(require.resolve('./long'));
});

describe('', function () {
this.tags('ciGroup13');

loadTestFile(require.resolve('./text'));
loadTestFile(require.resolve('./text_array'));
});
});
};
Original file line number Diff line number Diff line change
Expand Up @@ -9,33 +9,38 @@ import { FtrProviderContext } from '../../common/ftr_provider_context';
// eslint-disable-next-line import/no-default-export
export default ({ loadTestFile }: FtrProviderContext): void => {
describe('detection engine api security and spaces enabled', function () {
this.tags('ciGroup11');
describe('', function () {
this.tags('ciGroup11');

loadTestFile(require.resolve('./add_actions'));
loadTestFile(require.resolve('./add_prepackaged_rules'));
loadTestFile(require.resolve('./create_rules'));
loadTestFile(require.resolve('./create_rules_bulk'));
loadTestFile(require.resolve('./create_threat_matching'));
loadTestFile(require.resolve('./create_exceptions'));
loadTestFile(require.resolve('./delete_rules'));
loadTestFile(require.resolve('./delete_rules_bulk'));
loadTestFile(require.resolve('./exception_operators_data_types/index'));
loadTestFile(require.resolve('./export_rules'));
loadTestFile(require.resolve('./find_rules'));
loadTestFile(require.resolve('./find_statuses'));
loadTestFile(require.resolve('./generating_signals'));
loadTestFile(require.resolve('./get_prepackaged_rules_status'));
loadTestFile(require.resolve('./import_rules'));
loadTestFile(require.resolve('./read_rules'));
loadTestFile(require.resolve('./update_rules'));
loadTestFile(require.resolve('./update_rules_bulk'));
loadTestFile(require.resolve('./patch_rules_bulk'));
loadTestFile(require.resolve('./patch_rules'));
loadTestFile(require.resolve('./query_signals'));
loadTestFile(require.resolve('./open_close_signals'));
loadTestFile(require.resolve('./get_signals_migration_status'));
loadTestFile(require.resolve('./create_signals_migrations'));
loadTestFile(require.resolve('./finalize_signals_migrations'));
loadTestFile(require.resolve('./delete_signals_migrations'));
loadTestFile(require.resolve('./add_actions'));
loadTestFile(require.resolve('./add_prepackaged_rules'));
loadTestFile(require.resolve('./create_rules'));
loadTestFile(require.resolve('./create_rules_bulk'));
loadTestFile(require.resolve('./create_threat_matching'));
loadTestFile(require.resolve('./create_exceptions'));
loadTestFile(require.resolve('./delete_rules'));
loadTestFile(require.resolve('./delete_rules_bulk'));
loadTestFile(require.resolve('./export_rules'));
loadTestFile(require.resolve('./find_rules'));
loadTestFile(require.resolve('./find_statuses'));
loadTestFile(require.resolve('./generating_signals'));
loadTestFile(require.resolve('./get_prepackaged_rules_status'));
loadTestFile(require.resolve('./import_rules'));
loadTestFile(require.resolve('./read_rules'));
loadTestFile(require.resolve('./update_rules'));
loadTestFile(require.resolve('./update_rules_bulk'));
loadTestFile(require.resolve('./patch_rules_bulk'));
loadTestFile(require.resolve('./patch_rules'));
loadTestFile(require.resolve('./query_signals'));
loadTestFile(require.resolve('./open_close_signals'));
loadTestFile(require.resolve('./get_signals_migration_status'));
loadTestFile(require.resolve('./create_signals_migrations'));
loadTestFile(require.resolve('./finalize_signals_migrations'));
loadTestFile(require.resolve('./delete_signals_migrations'));
});

describe('', function () {
loadTestFile(require.resolve('./exception_operators_data_types/index'));
});
});
};
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('encryptedSavedObjects', function encryptedSavedObjectsSuite() {
this.tags('ciGroup2');
this.tags('ciGroup13');
loadTestFile(require.resolve('./encrypted_saved_objects_api'));
});
}
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('feature controls', function () {
this.tags(['ciGroup2']);
this.tags(['ciGroup8']);

loadTestFile(require.resolve('./api_keys_security'));
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('feature controls', function () {
this.tags(['ciGroup2']);
this.tags(['ciGroup8']);

loadTestFile(require.resolve('./ccr_security'));
});
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/dev_tools/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('console', function () {
this.tags('ciGroup10');
this.tags('ciGroup13');

loadTestFile(require.resolve('./feature_controls'));
loadTestFile(require.resolve('./searchprofiler_editor'));
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/grok_debugger/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('logstash', function () {
this.tags('ciGroup2');
this.tags('ciGroup13');

loadTestFile(require.resolve('./grok_debugger'));
});
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/index_management/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';

export default ({ loadTestFile }: FtrProviderContext) => {
describe('Index Management app', function () {
this.tags('ciGroup3');
this.tags('ciGroup13');
loadTestFile(require.resolve('./feature_controls'));
loadTestFile(require.resolve('./home_page'));
});
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional_cors/tests/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('Kibana cors', function () {
this.tags('ciGroup2');
this.tags('ciGroup12');
loadTestFile(require.resolve('./cors'));
});
}
2 changes: 1 addition & 1 deletion x-pack/test/licensing_plugin/public/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { FtrProviderContext } from '../services';
// eslint-disable-next-line import/no-default-export
export default function ({ loadTestFile }: FtrProviderContext) {
describe('Licensing plugin public client', function () {
this.tags('ciGroup2');
this.tags('ciGroup5');
loadTestFile(require.resolve('./feature_usage'));
// MUST BE LAST! CHANGES LICENSE TYPE!
loadTestFile(require.resolve('./updates'));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('event_log', function taskManagerSuite() {
this.tags('ciGroup2');
this.tags('ciGroup6');
loadTestFile(require.resolve('./public_api_integration'));
loadTestFile(require.resolve('./service_api_integration'));
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('task_manager', function taskManagerSuite() {
this.tags('ciGroup2');
this.tags('ciGroup12');
loadTestFile(require.resolve('./health_route'));
loadTestFile(require.resolve('./task_management'));
loadTestFile(require.resolve('./task_management_removed_types'));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export default function ({ loadTestFile }: { loadTestFile: (file: string) => voi
* worth keeping around for future use, rather than being rewritten time and time again.
*/
describe.skip('task_manager_perf', function taskManagerSuite() {
this.tags('ciGroup2');
this.tags('ciGroup12');
loadTestFile(require.resolve('./task_manager_perf_integration'));
});
}
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';

export default function ({ loadTestFile }: FtrProviderContext) {
describe('GlobalSearch API', function () {
this.tags('ciGroup10');
this.tags('ciGroup7');
loadTestFile(require.resolve('./global_search_providers'));
loadTestFile(require.resolve('./global_search_bar'));
});
Expand Down

0 comments on commit c24da96

Please sign in to comment.