Problem/Motivation

With more functional tests being converted to kernel tests, we can have another look at gitlab concurrency and maybe shift things around a bit.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3612651

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

catch created an issue. See original summary.

catch opened merge request !16408

catch’s picture

catch
he/him
Primary language English
commented
Title: Tweak test concurrency again » [PP-*] Tweak test concurrency again
Issue summary: View changes
Related issues: +#3612249: Split update script missing extension test to its own class, +#3612804: Reduce ComposerInspector/ComposerValidator work in package_manager, +#3612656: Split bundle field methods from EntityDefinitionUpdateMultipleTypesTest, +#3612742: Split invalid plugin test methods out of PhpTufValidatorTest, +#3612453: Split a couple of methods off ComposerValidatorTest, +#3612409: Split ComposerPatchesValidatorTest in two, +#3612411: Split a couple of methods out of ManageFieldsFunctionalTest and convert them to kernel tests, +#3612249: Split update script missing extension test to its own class, +#3612172: Merge Umami performance tests, +#3560920: Start unit tests after lint jobs in gitlab CI

There might not be a lot to tweak with concurrency, postponing on various related issues that split up or optimize individual tests.

catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3612828: Composer component tests should skip security audit
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3612922: Optimize ComponentsIsolatedBuildTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3612932: Optimize build tests
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3612939: Optimize ComponentsTaggedReleaseTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3612987: Split up DependencyTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613063: Consolidate methods in WorkspaceTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613162: Split up NonStableModulesTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613188: Optimize DefaultConfigTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613200: Try to reduce calls to composer in package manager
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613246: Split up ckeditor5's MediaTest
catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613250: Split up ConfigTranslationUiTest, +#3613253: Split up MediaAccessControlHandlerTest

Went on a test optimization spree.

With a fast gitlab runner, getting close to the point where no individual test run takes more than 2m30s for any of the test types, was also able to get build tests down from about 3 minutes to more like 90 seconds. The MR currently results in rescanning more than 4000 files in phpstan so the actual CI wall time doesn't look that great,

catch’s picture

catch
he/him
Primary language English
commented
Related issues: +#3613256: Optimize TypedConfigManager::getDefinitionWithReplacements()