Problem/Motivation
Right now #3346707: Add Alpha level Experimental Package Manager module gitlab CI tests run fail. This is at least because we don't have rsync but we don't know 100% there are no issue that will pop up if we solve the rsync issue.
Steps to reproduce
Proposed resolution
To make converting the module for #3253158: Add Alpha level Experimental Update Manager module and to make sure the test pass there:
- Run the conversion script to convert it to the core MR version of the module
- Run the test in the core MR version. If we can just our test groups if not run the whole test suite as it only takes 15 minutes
- Keep drupalci for now to test the contrib, unconverted version of the module. We can rely on this until July at which point hopefully we are in core
This should also help #3346707: Add Alpha level Experimental Package Manager module because this is the same MR but just with Package Manager.
Remaining tasks
If this proves very difficult we could *only* fix this on the core MR but hopefully we won't have to
User interface changes
API changes
Data model changes
Issue fork automatic_updates-3394413
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
Comment #4
tedbowI talked with @phenaproxima about this and I was wondering whether we could try to get tests only running on gitlabci and ask for a core exception to allow skipping our tests run in core drupalci. I thought this would be ok because our test would still run on gitlab ci.
but then I talked to @effulgentsia and he pointed out that some issue are done only via patch in core. In that case gitlab-ci would not run. So we might introduce regressions without knowing it until the gitlab pipelines ran on the branch after commit.
so that is probably not possible until core stops using the patch development workflow together. But effulgentsia thought we should check to see if there is timeframe. We might not have to wait for any infrastructure changes for that since there is limited number of Drupal 10 core committers and they could just agree to all stop using patches or always run a merge request before committing issues.
You could even imagine a core committer script
create-mr-from-patch [PATCH-URL] [ISSUE_NUM]This situation is especially bad for this module
Comment #5
tedbowI am working on #3391715: Change core merge request converter to either produce the Package Manager only MR or with AutoUpdates and running into problems I think this issue could help with.
This effectively keeping the merge request anywhere up-to-date very hard because we can only test 11.x in core merge requestRight now it looks like the problems with gitlab tests in the current MR that @phenaproxima is working on areHere is what I propose we do.
.gitlab-ci.ymlexclude the build tests from running. We are still running them in drupalci for now so we won't introduce any regresssionsskipOnGitlabCI()function. and open up a follow-up to remove this for GitExcluderTest. Again this will still run on drupalciThis obviously is not where we would like to be but will make sure we don't introduce any incompatibilities with 11.x
Also the core merge request the build tests will run in core clone which is what they are currently written assuming. So we could still work on the core merge request changes to core's own gitlabci.yml to get the core MR build tests passing even if they aren't passing yet in contrib module.
Basically lets get what we can passing now on gitlabci and commit that while continue to also run drupalci. Then lets try to get everything running on gitlabci
About to propose a new plan
Comment #7
tedbowunexpected test failures in the new more limited MR.
retesting existing MR to see if it has this new fails too
Comment #8
wim leers100% of test failures on GitLab CI are build tests.
Looks like #3403382: BuildTestBase makes assumptions it should not about the code layout needs to be fixed before they can pass.
Relevant Drupal Slack
#gitlabthread: https://drupal.slack.com/archives/CGKLP028K/p1700658999366429Comment #9
tedbowThis doesn't to be postponed see #5
Comment #10
tedbowI talked with @Wim Leers and @phenaproxima I have updated the summary plan. Basically we just use gitlab to test the version of the module that is converted for the core merge request in #3253158: Add Alpha level Experimental Update Manager module
for we will use drupalci to test the contrib version
Comment #12
tedbowComment #13
tedbowLooks like it creates the codebase correct!
Comment #14
phenaproximaComment #16
phenaproxima