jenkins-bot has submitted this change and it was merged. Change subject: mediawiki: Merge parsertests job back into main phpunit job ......................................................................
mediawiki: Merge parsertests job back into main phpunit job This was originally separated because the parser tests were too slow. Since recent refactoring they are much much faster. By accident, the refactor also broke --exclude-group=ParserTests (fixed in I8131c3b13). This meant that for the past week we've already been running parser tests in the main phpunit jobs as well. And still, it wasn't the slowest job. Change-Id: Ic2934746be68c06c1fc747e93f4fa73200eba652 --- M jjb/mediawiki.yaml M zuul/layout.yaml 2 files changed, 4 insertions(+), 81 deletions(-) Approvals: Krinkle: Looks good to me, approved jenkins-bot: Verified diff --git a/jjb/mediawiki.yaml b/jjb/mediawiki.yaml index 1342e29..033b930 100644 --- a/jjb/mediawiki.yaml +++ b/jjb/mediawiki.yaml @@ -105,7 +105,7 @@ publishers: - mw-teardown -# Runs all MediaWiki tests under hhvm on Nodepool instances +# Runs all MediaWiki PHPUnit tests on Nodepool - job-template: name: 'mediawiki-phpunit-{phpflavor}-{image}' node: ci-{image}-wikimedia @@ -124,43 +124,13 @@ - mw-install-mysql - mw-apply-settings - mw-fetch-composer-dev - # ParserTests tests are run via 'mediawiki-phpunit-parsertests-phpflavor' - - shell: "PHPUNIT_EXCLUDE_GROUP=ParserTests /srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh" + - shell: "/srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh" publishers: - phpunit-junit-2 - archive-log-dir - castor-save -# Only runs the 'parsertests' MediaWiki test suite on Nodepool instances -# Complements 'mediawiki-phpunit-{phpflavor}-{image}' -- job-template: - name: 'mediawiki-parsertests-{phpflavor}-{image}' - node: ci-{image}-wikimedia - concurrent: true - triggers: - - zuul - builders: - - castor-load - - assert-phpflavor: - phpflavor: '{phpflavor}' - - hhvm-clear-hhbc - - zuul-cloner: - projects: > - mediawiki/core - mediawiki/vendor - - mw-install-mysql - - mw-apply-settings - - mw-fetch-composer-dev - - shell: "PHPUNIT_TESTSUITE=parsertests /srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh" - publishers: - - phpunit-junit-2 - - archive-log-dir - - castor-save - -# Runs subset of MediaWiki tests -# -# Excludes '@group ParserTests' tests which are run in parallel via -# mediawiki-phpunit-parsertests-{phpflavor} +# Runs all MediaWiki PHPUnit tests on a CI slave - job-template: name: 'mediawiki-phpunit-{phpflavor}' node: 'contintLabsSlave && phpflavor-{phpflavor}' @@ -180,37 +150,7 @@ - mw-install-mysql - mw-apply-settings - mw-fetch-composer-dev - # ParserTests tests are run via 'mediawiki-phpunit-parsertests-phpflavor' - - shell: "PHPUNIT_EXCLUDE_GROUP=ParserTests /srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh" - publishers: - - phpunit-junit-2 - - mw-teardown-mysql - - archive-log-dir - -# mediawiki-phpunit-parsertests-{phpflavor} -# -# Only runs the 'parsertests' MediaWiki testsuite. -# Complements 'mediawiki-phpunit-{phpflavor}'. -- job-template: - name: 'mediawiki-phpunit-parsertests-{phpflavor}' - node: 'contintLabsSlave && phpflavor-{phpflavor}' - concurrent: true - properties: - - throttle-one-per-node - triggers: - - zuul - builders: - - assert-phpflavor: - phpflavor: '{phpflavor}' - - hhvm-clear-hhbc - - zuul-cloner: - projects: > - mediawiki/core - mediawiki/vendor - - mw-install-mysql - - mw-apply-settings - - mw-fetch-composer-dev - - shell: "PHPUNIT_TESTSUITE=parsertests /srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh" + - shell: "/srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh" publishers: - phpunit-junit-2 - mw-teardown-mysql @@ -361,9 +301,6 @@ - 'mediawiki-phpunit-{phpflavor}': phpflavor: - php53 - - 'mediawiki-phpunit-parsertests-{phpflavor}': - phpflavor: - - php53 - 'mediawiki-phpunit-{phpflavor}-{image}': &phpflavor_images phpflavor: - php55: @@ -373,7 +310,6 @@ image: trusty - hhvm: image: jessie - - 'mediawiki-parsertests-{phpflavor}-{image}': *phpflavor_images - 'mediawiki-phpunit-{phpflavor}-composer' - 'mediawiki-extensions-{phpflavor}': diff --git a/zuul/layout.yaml b/zuul/layout.yaml index e522fca..40da7e5 100644 --- a/zuul/layout.yaml +++ b/zuul/layout.yaml @@ -1339,15 +1339,12 @@ - mediawiki-core-php53lint - mediawiki-core-php55lint - mediawiki-phpunit-hhvm-trusty - - mediawiki-parsertests-hhvm-trusty - mediawiki-core-qunit-jessie - composer-package-validate php5: # Comment "check php5" to manually run this - mediawiki-phpunit-php55-trusty - - mediawiki-parsertests-php55-trusty - mediawiki-phpunit-php53 - - mediawiki-phpunit-parsertests-php53 gate-and-submit: - mediawiki-core-jsduck - mediawiki-core-npm-node-4-jessie @@ -1355,12 +1352,9 @@ - mediawiki-core-php55lint - mediawiki-core-phpcs-trusty - mediawiki-phpunit-hhvm-trusty - - mediawiki-parsertests-hhvm-trusty - mediawiki-phpunit-hhvm-composer - mediawiki-phpunit-php53 - - mediawiki-phpunit-parsertests-php53 - mediawiki-phpunit-php55-trusty - - mediawiki-parsertests-php55-trusty - mediawiki-core-qunit-jessie - composer-package-validate postmerge: @@ -1375,7 +1369,6 @@ - mediawiki-phpunit-php53-composer # For when we switch WMF app servers to Jessie - mediawiki-phpunit-hhvm-jessie - - mediawiki-parsertests-hhvm-jessie - name: mediawiki/debian test: @@ -1398,21 +1391,15 @@ - composer-validate - mediawiki-core-qunit-jessie - mediawiki-phpunit-hhvm-trusty - - mediawiki-parsertests-hhvm-trusty php5: - mediawiki-phpunit-php55-trusty - - mediawiki-parsertests-php55-trusty - mediawiki-phpunit-php53 - - mediawiki-phpunit-parsertests-php53 gate-and-submit: - php53lint - php55lint - mediawiki-phpunit-hhvm-trusty - - mediawiki-parsertests-hhvm-trusty - mediawiki-phpunit-php55-trusty - - mediawiki-parsertests-php55-trusty - mediawiki-phpunit-php53 - - mediawiki-phpunit-parsertests-php53 - mediawiki-core-qunit-jessie - composer-validate -- To view, visit https://gerrit.wikimedia.org/r/310701 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2934746be68c06c1fc747e93f4fa73200eba652 Gerrit-PatchSet: 2 Gerrit-Project: integration/config Gerrit-Branch: master Gerrit-Owner: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: Hashar <has...@free.fr> Gerrit-Reviewer: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com> Gerrit-Reviewer: Tim Starling <tstarl...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits