Customize resource requests per job

Follows the pattern of https://www.drupal.org/project/drupal/issues/3398321 📌 Optimize GitLab resource requests phase 1 Fixed (drupal core)

I would start with 2 for phpstan also. Modules are generally very small. Other than that, 2 sounds fine.

This should be fine. If I understand the infra correctly this will speed up jobs and use way les resources.

There might be some modules that see increase in testtimes. But I'd rather let that happen and either specifically up resources there or tweak based on feedback.

Rtbc

The new variable should be added in include.drupalci.variables.yml in my opinion. Core also uses this file but it is a good-enough default value and also core overrides this value per job.

Once it's moved I think it's good to go.

Great cleanup, that looks a lot better :)

Thoroughly testen by fjgarlin. Looks great. Let's save some resources <3

Merged! Less waiting time and less resources, it's a total win-win.
Thanks!

Appearantly global vars do not play nice with child pipelines and such. We need to EXPLICITLY set them on the jobs instead so we dont break core.

See https://drupal.slack.com/archives/CGKLP028K/p1700658667684839 for report

Weirdness explained in gitlab issue: https://gitlab.com/gitlab-org/gitlab/-/issues/357275

So, basically, we need to set the vars on the jobs themselves.

I'll go with the approach that Moshe did initially, which was to include it in the "includes/include.drupalci.main.yml" file instead. That affects only "contrib" space. I'll also add it to the D7 file.

I moved the code to the files that are only used by contrib.
MR: https://git.drupalcode.org/project/gitlab_templates/-/merge_requests/88

Yeah, the yaml will not be included in core. And since we do not use the weird child pipeline setup in contrib this should actually be fine.

Sorry for missing the weirdness in overwriting in child pipelines. ;x

Ok, merged.
Thanks again for patience, feedback, understanding... hopefully, we will start to see the difference in contrib now after this change.

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