Problem/Motivation

https://git.drupalcode.org/project/drupal/-/pipelines/815573/builds indicates that functional test jobs are taking longer purely due to the number of tests rather than individual long tests. We might be able to get test run times down with higher concurrency.

Once we've converted more functional tests to kernel tests, we might be able to reduce concurrency again or hopefully the number of total jobs. Not there yet though.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3589125

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 !15703

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review

https://git.drupalcode.org/project/drupal/-/pipelines/815595/builds looks pretty good. There are two tests that take around 3 minutes each (on that run at least), so that's the current floor for how quickly functional tests can finish overall

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

LGTM, trust @catch with these changes.

catch’s picture

catch
he/him
Primary language English
commented
Issue summary: View changes
catch’s picture

catch
he/him
Primary language English
commented
Title: Add some more functional test concurrency » Rebalance tests again
Status: Reviewed & tested by the community » Needs review

Spotted some more things, widening scope slightly to include adding #slow to some additional tests.

mondrake’s picture

mondrake
Location 🇮🇹
commented
Related issues: +#3589417: Move long-running unit tests to kernel

Could we combine with #3589417: Move long-running unit tests to kernel?

smustgrave’s picture

smustgrave commented

So is this being combined?

mondrake changed the visibility of the branch 3589125-combined-with-3589417 to hidden.

mondrake’s picture

mondrake
Location 🇮🇹
commented

#8 no, let’s not for now, see #3589417-6: Move long-running unit tests to kernel.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Going to mark this one then. Thanks for the follow up!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 1f48cce and pushed to main. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • alexpott committed 1f48cce9 on main 
    task: #3589125 Rebalance tests again

By: catch