Use composer and yarn artifacts from parent pipeline

Not sure what I am doing wrong here. I think I have followed the docs in https://docs.gitlab.com/ee/ci/pipelines/downstream_pipelines.html?tab=Pa... but the child pipeline immediately fails with "This job could not start because it could not retrieve the needed artifacts."

Seems to be working! The error I made previously was that the default variables is overwritten by the job-specific variables.

This still needs work to tidy up the variables and also the use of the caches, some of which is redundant now.

I just opened 📌 Move lint jobs to the pipeline Active (moving everything to the child pipeline) which would conflict with this. The goal is very similar so we should go ahead here and then the other issue will have a better baseline to deal with.

Cleaned up the use of cache - we only need this in the build steps, after this we are passing around artifacts instead and so the lint and test jobs don't need caches.

Also moved stage in the child pipeline to the defaults, given that we only have one stage now.

Also ensured that build (and lint) run on scheduled pipelines, because the test jobs now need the parent build job artifacts.

This covers 90% of what I wanted to get from 📌 Move lint jobs to the pipeline Active and the last 10% is quite marginal, so closed as duplicate.

MR looks great to me.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Rebased.

This is excellent and low risk. Will commit in ~24 hours if no objections, or sooner if there's a +1.

+1 from my side for what is worth.

I love the ability of referencing parent pipelines jobs in the “needs” section. I think it’sa great refactoring.

lauriii → closed merge request !4838

Nice work @longwave! 👏

Committed 4a70e44 and pushed to 11.x. Thanks!

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