Problem/Motivation

Eventually, d.o upstream will allow for different types of build triggers, which result in different build targets for the testbot.

This will mostly be transparent to the testbot, because the d.o layer will generate a full spec as a build.yml file.

However it also means that patched drupalci.yml files which need to be parsed at runtime could also specify build targets. We load drupalci.yml files if they're patched, and we'll need to know which target to execute.

This issue is mostly a placeholder for figuring out what this means.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

Mile23 created an issue. See original summary.

mile23’s picture

Issue summary: View changes
mile23’s picture

Mixologic’s picture

Current upstream target is the same for every test, which we're calling the development target.

The next target to come down the line will likely be regression. The main difference is that we'd skip some steps in regression testing (like phpcs/stylelint/phplint etc) because they've already been run as part of the development phase.

For now, a drupalci.yml would have the main root build key.

When we add additional ones, we'll add them as build.regression, and always assume that build, all by itself, means build.development

The other assumption that we'll make, for now, is that even if a developer patches the build.regression section of their drupalci.yml, we're still only going to replace the build.development section for that particular patch test.

They'll have to manually run a regression test after they commit to see if the patch to their drupalci.yml build.regression section was effective. (We could probably, in the future, offer an ad-hoc testing button to run the other targets in the issue)

Mixologic’s picture

So, when pift creates a build, should it set a variable somewhere in the build.yml that is the 'build target' ? or should it send the build target as a label to begin with?

i.e build.development at the top level?

mile23’s picture

Priority: Normal » Critical
Issue tags: +Drupal 9

If #2976407-64: Use drupalci.yml for composer dependency min/max testing - DO NOT COMMIT is critical and essentially a dupe/closed in favor of this issue, then this issue should be critical as well.