GitLab CI integration for core

Well, at least it caught some valid typos :)

/var/www/html/.gitlab-ci.yml:28:19 - Unknown word (overriden)
/var/www/html/.gitlab-ci.yml:70:91 - Unknown word (worfklows)
/var/www/html/.gitlab-ci/pipeline.yml:8:9 - Unknown word (Selectig)
/var/www/html/.gitlab-ci/pipeline.yml:10:61 - Unknown word (druaplci)

The others need adding to the dictionary.

We should wrap all comments at 80 characters in the YAML files to match our standards.

This is critical for core test performance and also for Drupal.org gitlab migration, so bumping priority.

Happy to commit this as soon as the comment feedback above is cleaned up and then we can keep going in other issues, and figure out exactly which bits of DrupalCI to switch off.

There has been really good progress here: 📌 [PP-1] Reduce test pipeline times to around 18 minutes and always report back unit tests Closed: duplicate
We are getting total run times of less than 20 minutes now.

I've suggested that those changes be merged here so we have only one source of truth to review for GitlabCI integration.

larowlan → credited el7cosmos → .

larowlan → credited mstrelan → .

larowlan → credited RoSk0 → .

larowlan → credited xurizaemon → .

Crediting those who worked in the sandbox

larowlan → credited nick_schuch → .

And those who helped at Drupal south code sprint

catch → credited bbrala → .

Two more nits on the MR, for me the only thing blocking commit of this is the nightwatch failures which seem pretty consistent.

the only thing blocking commit of this is the nightwatch failures

Fixed.

larowlan → credited alexpott → .

Crediting alex who also worked in the sandbox

Thanks for working on this, this is looking really good. Added some questions and comments, some of which can be dealt with in followups.

Added few very minor nits regarding comments/whitespaces to the MR.

Last commit results in CSpell: Files checked: 1, Issues found: 0 in 0 files which should be correct I think.

I think that is probably all feedback resolved. Opened this follow-up 📌 Reconcile gitlab lint jobs and commit-code-check.sh Needs work . Back to needs review.

Some nits around consistency in the templates, but I think this is almost ready for RTBC and continuing improvements in followups.

Addressed all the feedback.

Thanks! This looks ready to go, I've gone over all changes a couple of times and can find nothing left to nitpick over - see you all in the followups.

Moving this to 10.1.x for backport. We need to tweak the central gitlab templates so that branch tests actually run after a branch commit, so will backport to 10.1.x as soon as this is done.

11.x pipeline: https://git.drupalcode.org/project/drupal/-/pipelines/19594

10.1.x pipeline: https://git.drupalcode.org/project/drupal/-/pipelines/19603

A bit more work to do before we can switch of DrupalCI but this is a massive improvement in so many ways. Marking fixed!

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