Problem/Motivation

It's very slow.

Adding a path repo to composer takes 2-500ms each time, and we do that for components * components.

Steps to reproduce

Proposed resolution

Run the composer commands async in batches by component, on my local this reduces the test run time from 4 minutes to 2m20s however whether we get similar savings on CI is a different question.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3612922

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’s picture

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

Just got a 47s run locally before pushing which is even more encouraging - although again doesn't mean it'll do as much on CI.

catch opened merge request !16418

catch’s picture

catch
he/him
Primary language English
commented

28s on CI now 28.692s Drupal\BuildTests\Composer\Component\ComponentsIsolatedBuildTest 1 passed

https://git.drupalcode.org/project/drupal/-/jobs/11132183

smustgrave’s picture

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

Seems good to me.

larowlan made their first commit to this issue’s fork.

larowlan opened merge request !16467

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Version: main » 11.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed ae35ab423bc to main and a1157ed4cee to 11.x. 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.

  • larowlan committed a1157ed4 on 11.x 
    task: #3612922 Optimize ComponentsIsolatedBuildTest

By: catch
By:...

  • larowlan committed ae35ab42 on main 
    task: #3612922 Optimize ComponentsIsolatedBuildTest

By: catch
By:...
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

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.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented

Thanks @longwave

Status: Fixed » Closed (fixed)

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