Contrib modules can now use GitLab CI for testing.

Instructions for setting up and opt-in are on #3261803: Using GitLab CI instead of Drupal CI

Issue fork scheduler-3356800

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:

Comments

jonathan1055 created an issue. See original summary.

  • jonathan1055 committed 00e271ec on 2.x
    Issue #3356800 by jonathan1055: Add .gitlab-ci.yml from project/...

  • jonathan1055 committed 2410739c on 8.x-1.x
    Issue #3356800 by jonathan1055: Add .gitlab-ci.yml from gitlab-ci/...
jonathan1055’s picture

StatusFileSize
new153.59 KB

Added the .gitlab-ci.yml from the template and we have our The first pipeline run. It was not very sucessful, as six of the seven jobs failed :-) The only one that passed was variable-checks which just echos some system variables.

The Composer job failed with a surprising error that I don't understand:

$ touch $_WEB_ROOT/core/.env
Uploading artifacts for successful job 
Uploading artifacts...
/builds/project/scheduler: found 68462 matching files and directories 
.git: excluded 1 files 
.git/**/*: excluded 105 files 
ERROR: Uploading artifacts as "archive" to coordinator... 413 Request Entity Too Large id=44659 responseStatus=413 Request Entity Too Large status=413 token=64_A21Dw
FATAL: too large 
Cleaning up project directory and file based variables 00:00 
ERROR: Job failed: command terminated with exit code 1

Everything else failed for various reasons, for example vendor/bin/phpcs: No such file or directory and error Directory "/builds/project/scheduler/web/core" doesn't exist ... most likely due to this first step failing.

Also there appears to be only one dependency link. I'm not sure what that means though. If the composer job is critical, then maybe all the other jobs should depend on it?
gitlab-ci job dependencies

jonathan1055’s picture

StatusFileSize
new130.06 KB

That's an improvement. The subsequent jobs are skipped if the Composer job fails
skip jobs

jonathan1055’s picture

https://git.drupalcode.org/project/gitlab_templates/-/merge_requests/11 has been committed, so Scheduler's modification is now removed.

jonathan1055’s picture

Would be useful to be able to test the GitLab-CI pipelines without having to push to a MR and have all the noise above.

#3358806: Rename phpcs.xml.dist to phpcs.xml in preparation for GitLabCI fixes the PHPCS problem.

  • jonathan1055 committed 80473d3f on 2.x
    Issue #3356800 by jonathan1055: Fix DrushCronMessages test to run with...

  • jonathan1055 committed d75e699c on 2.x
    Issue #3356800 by jonathan1055: Update .gitlab-ci.ym with new $...

  • jonathan1055 committed 576eb243 on 8.x-1.x
    Issue #3356800 by jonathan1055: Update .gitlab-ci.ym with new $...

  • jonathan1055 committed 061253ba on 2.x
    git commit -m 'Issue #3356800 by jonathan1055: Add _PHPUNIT_CONCURRENT...

  • jonathan1055 committed b23c2daa on 2.x
    Issue #3356800 by jonathan1055: Gitlab testing - Redefine matrix
    

  • jonathan1055 committed 677cf590 on 2.x
    Issue #3356800 by jonathan1055: Add javascript tests to GitLab CI matrix
    

  • jonathan1055 committed d1fb4180 on 2.x
    Issue #3356800 by jonathan1055: Run pipelines without concurrency
    
jonathan1055’s picture

StatusFileSize
new1.93 KB

Test via patch

jonathan1055’s picture

Status: Active » Fixed

Gitlab testing is now fully implemented for Scheduler 2.x and 8.x-1.x so this issue can be closed.

#3387331: Add GitLab CI testing on 7.x is also complete.

  • jonathan1055 committed 0cd4c679 on 2.x
    Issue #3356800 by jonathan1055: Include default rules: within customised...

Status: Fixed » Closed (fixed)

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

jonathan1055 changed the visibility of the branch 3356800-drupal8 to hidden.

jonathan1055 changed the visibility of the branch 2.x to hidden.

jonathan1055 changed the visibility of the branch 3356800-spelling to hidden.

jonathan1055 changed the visibility of the branch 3356800-general to hidden.

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

mondrake’s picture

@jonathan1055 I made a commit to MR!194 that gets it at what you were looking for (I assume)

jonathan1055 changed the visibility of the branch 3356800-pipeline-rules-in-mr-or-on-commit to hidden.

jonathan1055 changed the visibility of the branch 3356800-test-only-changes to hidden.

jonathan1055 changed the visibility of the branch 3356800-pipeline-rules-in-mr-or-on-commit to active.