First (untested) cut at DrupalCI integration for contrib testing. Should be compatible with the current 'dev' branch of the DrupalCI test runner.

Deployment

Staging only, remove all non-core jobs:
SELECT * FROM pift_ci_job j INNER JOIN field_data_field_release_project rp ON rp.entity_id = j.release_nid AND rp.field_release_project_target_id <> 3060;

Remove queued contrib issue tests:
SELECT * FROM pift_ci_job j INNER JOIN field_data_field_release_project rp ON rp.entity_id = j.release_nid AND rp.field_release_project_target_id <> 3060 WHERE j.target_type = 'file';

Remove duplicate queued contrib branch tests:
SELECT * FROM pift_ci_job j INNER JOIN field_data_field_release_project rp ON rp.entity_id = j.release_nid AND rp.field_release_project_target_id <> 3060 WHERE j.target_type = 'branch' AND j.job_id NOT IN (SELECT max(job_id) FROM pift_ci_job j INNER JOIN field_data_field_release_project rp ON rp.entity_id = j.release_nid AND rp.field_release_project_target_id <> 3060 WHERE j.target_type = 'branch' GROUP BY j.release_nid);

CommentFileSizeAuthor
pift-ci.patch4.51 KBjthorson

Comments

jthorson’s picture

Updating attribution

drumm’s picture

Status: Active » Needs review

I've added the DCI_TestItem and DCI_AdditionalRepositories parameters to https://dispatcher.drupalci.org/job/default/. Build 1391 will be the first with them.

drumm’s picture

Issue summary: View changes

  • drumm committed 66443fd on 2535786-contrib authored by jthorson
    Issue #2535786 by jthorson: [DrupalCI Integration] Enable contrib...

  • drumm committed 4b4ec28 on 2535786-contrib
    Issue #2535786: Enable contrib testing, use more entity_metadata_wrapper
    
  • drumm committed 98afa8b on 2535786-contrib
    Issue #2535786: Enable contrib testing, use machine name for directory...
drumm’s picture

Issue summary: View changes

I pushed a couple commits.

  • The first is taking advantage of EntityMetadataWrapper.
  • The second uses the machine name, instead of repo name, for the checkout directory. Not really a visible change, since they are almost always the same, but more consistent with what people will expect.
drumm’s picture

Status: Needs review » Fixed
Issue tags: +needs drupal.org deployment

  • drumm committed 4b4ec28 on 7.x-3.x
    Issue #2535786: Enable contrib testing, use more entity_metadata_wrapper
    
  • drumm committed 66443fd on 7.x-3.x authored by jthorson
    Issue #2535786 by jthorson: [DrupalCI Integration] Enable contrib...
  • drumm committed 98afa8b on 7.x-3.x
    Issue #2535786: Enable contrib testing, use machine name for directory...
drumm’s picture

Looks ready to deploy.

drumm’s picture

Now deployed.

  • drumm committed 110dea0 on 7.x-3.x
    Issue #2535786: Enable contrib testing, fix dependency directories
    
drumm’s picture

I managed to break this with my commits. Deploying the last commit now to fix it.

Status: Fixed » Closed (fixed)

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

drumm’s picture

Issue tags: -needs drupal.org deployment