Reduce CPU requirement and concurrency for unit tests

1 CPU and this still finishes in under a minute. Needed to pull in 🐛 CoreRecipesTest is slow RTBC and 📌 Show test run time by class in run-tests.sh output Fixed to see what's going on, but other changes here should be ready for review.

Added @group #slow to a handful of tests to maximize the concurrency towards the end of the test runs, and this one came out at 39 seconds https://git.drupalcode.org/project/drupal/-/jobs/2412264/viewer

Current test runs take over a minute, but this is because of 🐛 CoreRecipesTest is slow RTBC which is causing functional tests to be run in the phpunit test job. However can't really beat 39 seconds so shows going down to one CPU is fine.

There are unit tests reading files, sleep()ing etc. so the tests aren't entirely CPU constrainted.

Reverted the changes that aren't specific to this issue.

Should the changes to the pipeline be reverted?

No the pipeline changes are the main thing in this issue. The @group #slow just ensures that with reduced concurrency we won't end up with long-running tests starting at the end of the run.

Ah gotcha, well based on your findings #5 say lets give it a shot

Committed 52dc1d7 and pushed to 11.x. Thanks!

doesn't cherry pick cleanly on 11.0.x

nod_ → closed merge request !9168

had to revert:

---- Drupal\Tests\PhpCs\SortTest ----
Status    Group      Filename          Line Function                            
--------------------------------------------------------------------------------
Fail      Other      phpunit-447.xml      0 Drupal\Tests\PhpCs\SortTest        
    PHPUnit Test failed to complete; Error: PHPUnit 10.5.29 by Sebastian
    Bergmann and contributors.
    Runtime:       PHP 8.3.10
    Configuration: /builds/project/drupal/core/phpunit.xml.dist
    .F                                                                  2 / 2
    (100%)
    Time: 00:00.024, Memory: 8.00 MB
    There was 1 failure:
    1) Drupal\Tests\PhpCs\SortTest::testSorted
    Failed asserting that two arrays are equal.
    --- Expected
    +++ Actual
    @@ @@
         1 => '*/node_modules/*'
         2 => './assets/vendor/*'
         3 => './core/.phpstan-baseline.php'
    -    4 => './core/phpstan-tmp/*'
    -    5 => './core/lib/Drupal/Component/Diff/'
    +    4 => './core/lib/Drupal/Component/Diff/'
    +    5 => './core/phpstan-tmp/*'
         6 => './core/tests/Drupal/Tests/Com...trine/'
         7 => './modules/system/tests/fixtur...ssTest'
     )
    /builds/project/drupal/core/tests/Drupal/Tests/PhpCs/SortTest.php:98
    /builds/project/drupal/core/tests/Drupal/Tests/PhpCs/SortTest.php:56
    FAILURES!
    Tests: 2, Assertions: 7, Failures: 1.

catch → reopened merge request !9168

catch → changed the visibility of the branch 3467391-reduce-cpu-requirement to active.

The commit here accidentally included 📌 Ignore phpstan-tmp in phpcs.xml.dist RTBC , hence the test failure since the MR over there had the same one. Moving back to RTBC.

nod_ → closed merge request !9168

Committed bfce456 and pushed to 11.x. Thanks!

Committed f8d6615 and pushed to 11.0.x. Thanks!
Committed 7d43854 and pushed to 10.4.x. Thanks!
Committed c0eda49 and pushed to 10.3.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.