Needs review
Project:
DrupalCI: Test Runner
Component:
Testrunner Codebase
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Jan 2023 at 15:44 UTC
Updated:
15 Aug 2023 at 21:01 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
alexpottHopefully this will fix DrupalCI.
Comment #3
alexpottComment #4
alexpottAnother idea...
Comment #5
alexpottComment #6
alexpottOkay... I think this will work but it will need to be committed to prove it because of the way that DrupalCI works with custom drupalci.yml files...
Comment #8
berdirCommitted, but doesn't seem to work either, both the on-commit test and a manual retest failed. I'm not seeing a second composer step happening.
A found a bunch of exsamples that do something similar, but they all do it later, which is too late for us I think. For example http://grep.xnddx.ru/node/33571933.
Comment #9
alexpottMaybe because we have a drupalci.yml file now we can test changing it...
Comment #10
alexpottTurns out you can only customise the assessment stage - see https://www.drupal.org/drupalorg/docs/drupal-ci/customizing-drupalci-tes...
So this is not going to work and we're going to need to fix DrupalCI :(
Comment #11
alexpottMoving to the DrupalCI: Test Runner queue as this can only be fixed there. Not sure at all how to fix this - maybe:
Comment #12
alexpottAlternatively maybe we move simpleSAMLphp Authentication to gitlab ci and see if we can fix this there.
Comment #13
wim leersRan into this problem too over at #3334243-12: CDN 4.x requires PHP >=8.1 but composer does not respect it due to bug in d.o composer facade. Pity that tweaking
drupalci.ymldoes not work at all per #10, but at least I don't have to experiment with that now thanks to @alexpott in #10 😔Apparently nobody reported the need for
allow-pluginssupport at: https://www.drupal.org/project/issues/project_issue_file_test?text=allow... 🤯Then discovered that #3265092: [META] Define a default .gitlab-ci.yml template that projects can inherit has been silent for >11 months … so not very hopeful about the GitLab CI approach either. But trying it anyway: #3334243-14: CDN 4.x requires PHP >=8.1 but composer does not respect it due to bug in d.o composer facade.
Comment #14
berdirFWIW, the drupalspoons template which seems to be pretty much the only thing out there at the moment only just hardcodes a bunch of specific plugins: https://gitlab.com/drupalspoons/composer-plugin/-/raw/master/bin/setup. To customize that, you would need to copy that whole file and add your own?
I think the only reason that "works" for you Wim is that it doesn't include any of the magic that DrupalCI does and does not actually automatically include require-dev dependencies.
Changing to a feature request, as commented in #3265092: [META] Define a default .gitlab-ci.yml template that projects can inherit, can't we just allow all plugins and move on? The security aspect of that seems irrelevant, as we're running literally any PHP code that's in the repository that's being tested.
Comment #16
berdirIf I understand this correctly, we'd need to add the allow-all plugins to https://git.drupalcode.org/project/drupalci_testbot/-/blob/dev/src/Drupa..., before the composer require of the project.
so, run this command there:
composer config allow-plugins true
Created that as a merge request, but the integration doesn't seem to fully work here, so I did that manually, directly in Gitlab and its Web IDE. Obviously entirely untested :)
https://git.drupalcode.org/project/drupalci_testbot/-/merge_requests/9
Comment #17
moshe weitzman commentedYou an allow whatever plugins you want in your project's composer.json and those will be honored/used by DrupalSpoons. The nones you saw there are auto-allowed so that projects dont *have* to explicitly allow them.
Comment #19
hestenetMerged into drupalci_testbot:production - sorry for such a long delay on a straightforward change. Many different priorities swirling around and I simply lost track. Thanks to those who pinged in Slack for an update.
Before marking fixed - would someone please test this on an affected issue and confirm it is resolved?
Comment #20
joseph.olstadThanks for the fix
Comment #22
cmlaraNote it appears this might have broken testing on PHP 7.4 images, due to a bug in composer, see #3381247: Composer config failure when running functional tests