Skip to content

Commit

Permalink
Revert "[ci] compress jobs for CI stability" (#44584) (#44674)
Browse files Browse the repository at this point in the history
This reverts commit debf8c6.
  • Loading branch information
Spencer authored Sep 3, 2019
1 parent 9c2ee43 commit 0850e52
Show file tree
Hide file tree
Showing 34 changed files with 68 additions and 53 deletions.
21 changes: 17 additions & 4 deletions .ci/jobs.yml
Original file line number Diff line number Diff line change
@@ -1,21 +1,34 @@
JOB:
- intake
- firefoxSmoke
- kibana-intake
- x-pack-intake
- kibana-firefoxSmoke
- kibana-ciGroup1
- kibana-ciGroup2
- kibana-ciGroup3
- kibana-ciGroup4
- kibana-ciGroup5
- kibana-ciGroup6
# - kibana-visualRegression
- kibana-ciGroup7
- kibana-ciGroup8
- kibana-ciGroup9
- kibana-ciGroup10
- kibana-ciGroup11
- kibana-ciGroup12
- kibana-visualRegression

# make sure all x-pack-ciGroups are listed in test/scripts/jenkins_xpack_ci_group.sh
- x-pack-firefoxSmoke
- x-pack-ciGroup1
- x-pack-ciGroup2
- x-pack-ciGroup3
- x-pack-ciGroup4
- x-pack-ciGroup5
# - x-pack-visualRegression
- x-pack-ciGroup6
- x-pack-ciGroup7
- x-pack-ciGroup8
- x-pack-ciGroup9
- x-pack-ciGroup10
- x-pack-visualRegression

# `~` is yaml for `null`
exclude: ~
10 changes: 8 additions & 2 deletions .ci/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ source src/dev/ci_setup/setup.sh
source src/dev/ci_setup/checkout_sibling_es.sh

case "$JOB" in
intake)
kibana-intake)
./test/scripts/jenkins_unit.sh
;;
kibana-ciGroup*)
Expand All @@ -21,16 +21,22 @@ kibana-ciGroup*)
kibana-visualRegression*)
./test/scripts/jenkins_visual_regression.sh
;;
firefoxSmoke*)
kibana-firefoxSmoke*)
./test/scripts/jenkins_firefox_smoke.sh
;;
x-pack-intake)
./test/scripts/jenkins_xpack.sh
;;
x-pack-ciGroup*)
export CI_GROUP="${JOB##x-pack-ciGroup}"
./test/scripts/jenkins_xpack_ci_group.sh
;;
x-pack-visualRegression*)
./test/scripts/jenkins_xpack_visual_regression.sh
;;
x-pack-firefoxSmoke*)
./test/scripts/jenkins_xpack_firefox_smoke.sh
;;
*)
echo "JOB '$JOB' is not implemented."
exit 1
Expand Down
4 changes: 2 additions & 2 deletions test/functional/apps/management/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ export default function ({ getService, loadTestFile }) {
});

describe('', function () {
this.tags('ciGroup1');
this.tags('ciGroup7');

loadTestFile(require.resolve('./_create_index_pattern_wizard'));
loadTestFile(require.resolve('./_index_pattern_create_delete'));
Expand All @@ -45,7 +45,7 @@ export default function ({ getService, loadTestFile }) {
});

describe('', function () {
this.tags('ciGroup2');
this.tags('ciGroup8');

loadTestFile(require.resolve('./_index_pattern_filter'));
loadTestFile(require.resolve('./_scripted_fields_filter'));
Expand Down
8 changes: 4 additions & 4 deletions test/functional/apps/visualize/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export default function({ getService, loadTestFile }: FtrProviderContext) {
});

describe('', function() {
this.tags('ciGroup3');
this.tags('ciGroup9');

loadTestFile(require.resolve('./_embedding_chart'));
loadTestFile(require.resolve('./_chart_types'));
Expand All @@ -50,7 +50,7 @@ export default function({ getService, loadTestFile }: FtrProviderContext) {
});

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

loadTestFile(require.resolve('./_inspector'));
loadTestFile(require.resolve('./_experimental_vis'));
Expand All @@ -62,7 +62,7 @@ export default function({ getService, loadTestFile }: FtrProviderContext) {
});

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

loadTestFile(require.resolve('./_line_chart'));
loadTestFile(require.resolve('./_pie_chart'));
Expand All @@ -76,7 +76,7 @@ export default function({ getService, loadTestFile }: FtrProviderContext) {
});

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

loadTestFile(require.resolve('./_tag_cloud'));
loadTestFile(require.resolve('./_tile_map'));
Expand Down
12 changes: 6 additions & 6 deletions test/mocha_decorations.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,12 @@ type Tags =
| 'ciGroup4'
| 'ciGroup5'
| 'ciGroup6'
| 'ciGroup1'
| 'ciGroup2'
| 'ciGroup3'
| 'ciGroup4'
| 'ciGroup5'
| 'ciGroup6';
| 'ciGroup7'
| 'ciGroup8'
| 'ciGroup9'
| 'ciGroup10'
| 'ciGroup11'
| 'ciGroup12';

// We need to use the namespace here to match the Mocha definition
// eslint-disable-next-line @typescript-eslint/no-namespace
Expand Down
2 changes: 0 additions & 2 deletions test/scripts/jenkins_firefox_smoke.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,5 +17,3 @@ checks-reporter-with-killswitch "Firefox smoke test" \
--kibana-install-dir "$installDir" \
--include-tag "smoke" \
--config test/functional/config.firefox.js;

source "$KIBANA_DIR/test/scripts/jenkins_xpack_firefox_smoke.sh"
2 changes: 0 additions & 2 deletions test/scripts/jenkins_unit.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,3 @@ trap 'node "$KIBANA_DIR/src/dev/failed_tests/cli"' EXIT
export TEST_BROWSER_HEADLESS=1

"$(FORCE_COLOR=0 yarn bin)/grunt" jenkins:unit --dev;

source "$KIBANA_DIR/test/scripts/jenkins_xpack.sh"
8 changes: 4 additions & 4 deletions test/scripts/jenkins_xpack_ci_group.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,10 @@ node scripts/functional_tests --assert-none-excluded \
--include-tag ciGroup4 \
--include-tag ciGroup5 \
--include-tag ciGroup6 \
--include-tag ciGroup1 \
--include-tag ciGroup2 \
--include-tag ciGroup3 \
--include-tag ciGroup4
--include-tag ciGroup7 \
--include-tag ciGroup8 \
--include-tag ciGroup9 \
--include-tag ciGroup10

echo " -> building and extracting default Kibana distributable for use in functional tests"
cd "$KIBANA_DIR"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export default function alertingApiIntegrationTests({
const esArchiver = getService('esArchiver');

describe('alerting api integration security and spaces enabled', function() {
this.tags('ciGroup3');
this.tags('ciGroup8');

before(async () => {
for (const space of Spaces) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default function alertingApiIntegrationTests({
const esArchiver = getService('esArchiver');

describe('alerting api integration spaces only', function() {
this.tags('ciGroup3');
this.tags('ciGroup8');

before(async () => {
for (const space of Object.values(Spaces)) {
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/api_integration/apis/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('apis', function () {
this.tags('ciGroup1');
this.tags('ciGroup6');

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

export default function ({ loadTestFile }) {
describe('security', function () {
this.tags('ciGroup1');
this.tags('ciGroup6');

loadTestFile(require.resolve('./basic_login'));
loadTestFile(require.resolve('./builtin_es_privileges'));
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/api_integration/apis/spaces/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('spaces', function() {
this.tags('ciGroup1');
this.tags('ciGroup6');

loadTestFile(require.resolve('./saved_objects'));
loadTestFile(require.resolve('./space_attributes'));
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/apm/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('APM', function() {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./feature_controls'));
});
}
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/dashboard_mode/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('dashboard mode', function () {
this.tags('ciGroup2');
this.tags('ciGroup7');

loadTestFile(require.resolve('./dashboard_view_mode'));
});
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 ({ loadTestFile }: FtrProviderContext) => {
describe('Index Lifecycle Management app', function() {
this.tags('ciGroup2');
this.tags('ciGroup7');
loadTestFile(require.resolve('./home_page'));
});
};
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/infra/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('InfraOps app', function() {
this.tags('ciGroup2');
this.tags('ciGroup7');

loadTestFile(require.resolve('./home_page'));
loadTestFile(require.resolve('./feature_controls'));
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/license_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('License app', function() {
this.tags('ciGroup2');
this.tags('ciGroup7');
loadTestFile(require.resolve('./home_page'));
});
};
4 changes: 2 additions & 2 deletions x-pack/test/functional/apps/maps/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export default function ({ loadTestFile, getService }) {
});

describe('', function () {
this.tags('ciGroup2');
this.tags('ciGroup7');
loadTestFile(require.resolve('./documents_source'));
loadTestFile(require.resolve('./saved_object_management'));
loadTestFile(require.resolve('./sample_data'));
Expand All @@ -38,7 +38,7 @@ export default function ({ loadTestFile, getService }) {
});

describe('', function () {
this.tags('ciGroup5');
this.tags('ciGroup10');
loadTestFile(require.resolve('./es_geo_grid_source'));
loadTestFile(require.resolve('./joins'));
loadTestFile(require.resolve('./add_layer_panel'));
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/functional/apps/uptime/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export default ({ loadTestFile, getService }: FtrProviderContext) => {
await kibanaServer.uiSettings.replace({ 'dateFormat:tz': 'UTC' });
});
after(async () => await esArchiver.unload(ARCHIVE));
this.tags('ciGroup1');
this.tags('ciGroup6');

loadTestFile(require.resolve('./feature_controls'));
loadTestFile(require.resolve('./overview'));
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/kerberos_api_integration/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 Kerberos', function() {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./security'));
});
}
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('apis', function () {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./oidc_auth'));
});
}
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { FtrProviderContext } from '../../ftr_provider_context';
// eslint-disable-next-line import/no-default-export
export default function({ loadTestFile }: FtrProviderContext) {
describe('apis', function() {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./oidc_auth'));
});
}
2 changes: 1 addition & 1 deletion x-pack/test/pki_api_integration/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 PKI', function() {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./security'));
});
}
2 changes: 1 addition & 1 deletion x-pack/test/reporting/functional/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('reporting app', function () {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./reporting'));
});
}
2 changes: 1 addition & 1 deletion x-pack/test/saml_api_integration/apis/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('apis SAML', function () {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./security'));
});
}
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ export default function({ getService, loadTestFile }: FtrProviderContext) {
const supertest = getService('supertest');

describe('saved objects security only enabled', function() {
this.tags('ciGroup4');
this.tags('ciGroup9');

before(async () => {
await createUsersAndRoles(es, supertest);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export default function({ loadTestFile, getService }: TestInvoker) {
const supertest = getService('supertest');

describe('spaces api with security', function() {
this.tags('ciGroup3');
this.tags('ciGroup8');

before(async () => {
await createUsersAndRoles(es, supertest);
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/token_api_integration/auth/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

export default function ({ loadTestFile }) {
describe('token-based auth', function () {
this.tags('ciGroup1');
this.tags('ciGroup6');
loadTestFile(require.resolve('./login'));
loadTestFile(require.resolve('./logout'));
loadTestFile(require.resolve('./header'));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default function uiCapabilitiesTests({ loadTestFile, getService }: FtrPro
const featuresService: FeaturesService = getService('features');

describe('ui capabilities', function() {
this.tags('ciGroup4');
this.tags('ciGroup9');

before(async () => {
const features = await featuresService.get();
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/ui_capabilities/security_only/tests/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export default function uiCapabilitesTests({ loadTestFile, getService }: FtrProv
const securityService: SecurityService = getService('security');

describe('ui capabilities', function() {
this.tags('ciGroup4');
this.tags('ciGroup9');

before(async () => {
for (const user of UserScenarios) {
Expand Down
2 changes: 1 addition & 1 deletion x-pack/test/ui_capabilities/spaces_only/tests/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export default function uiCapabilitesTests({ loadTestFile, getService }: FtrProv
const featuresService: FeaturesService = getService('features');

describe('ui capabilities', function() {
this.tags('ciGroup4');
this.tags('ciGroup9');

before(async () => {
const features = await featuresService.get();
Expand Down
Loading

0 comments on commit 0850e52

Please sign in to comment.