Problem/Motivation

Now that I have enabled the GitHub actions for the latest 6.2.x branch, we are getting a failure during the install of the wxt_ext_* modules:

Notice: ]
Performed install task: install_finish_translations
Notice: ] Performed install task: install_finished
Notice: ] Performed install task: wxt_extension_configure_form
Notice: ] views.view.block_content rewritten by wxt_ext_blocks

In EntityType.php line 924:
Missing bundle entity, entity type media_type, entity id image.

Failing test: https://github.com/drupalwxt/site-wxt/actions/runs/19401147529/job/55508806004

It appears that changes in ModuleInstaller for core 11.2.x fundamentally changes the way modules are installed and config dependencies are calculated. @millnut in the related ticket tracked this down to the following commit from the parent issue - https://github.com/drupal/core/commit/3c297956866bd5f003643770febdbd16a92e69c9

Steps to reproduce

Using 6.2.x branch, install WxT and enable all optional modules during install, see the error. Optionally if you are using site-wxt 11.2.x, run make drupal_install and see failures for the builds as reported above.

There are issues for conditional config, as well as config rewrite.

Proposed resolution

Not sure, maybe we have to move to an "all or nothing" profile install, where we list all sub-module dependencies directly on the profile and don't provide the option to install one-at-a-time. Or maybe this will push us to a recipe-based approach, not 100% sure the best way forward.

Remaining tasks

Identify best course of action.

CommentFileSizeAuthor
#2 wxt-6-2-x-failure.jpg277.17 KBsmulvih2

Comments

smulvih2 created an issue. See original summary.

smulvih2’s picture

StatusFileSize
new277.17 KB

I get the same error when installing via the UI:

WxT 6.2.x GitHub test failure

smulvih2’s picture

Looks like LocalGov Drupal folks are also running into this issue, which might be caused by differences between 11.1.x and 11.2.x, see related issue.

smulvih2’s picture

Adding parent issue where this regression originated.

Here is my summary of the changes and how they impact WxT:

It looks like Drupal 11.2.x introduced major changes to how module configuration is validated and installed:

  1. Modules are now validated and installed in batches, not individually. This means all default config across a group of modules is validated together before any of them install. If one module references a bundle/config provided by another module, and that config is in /config/optional or not present in the install profile's exported config, the entire batch now fails validation.
  2. Config install is now split into phases: InstallSimple -> InstallEntities -> Optional. Bundles placed in /config/optional don’t get created until the "Optional" phase, which now runs after entity display and field config that depends on those bundles. This exposes hidden ordering problems that used to work by accident (and were always a bit finicky).
  3. Profile-based installs no longer see a module's /config/install as part of the "initial config set". If a distribution rewrites config from contrib modules (e.g., Metatag) but the profile does not export that base config, config_rewrite now reports "initial config missing".

For Drupal WxT, we have several profile extensions that are optionally enabled (e.g., wxt_ext_metatag adjusts Metatag defaults to align with Government of Canada standards). With these installer changes, it appears we can no longer rely on optional profile modules to safely provide or rewrite configuration. We may now need to declare more dependencies directly on the profile itself, making the configuration more "all or nothing".

Might be time to consider other options like recipes, or remove "optional" modules and just install everything. I think to get this working now, we would need to add all sub-module dependencies on the profile itself.

smulvih2’s picture

Title: [6.2.x] Fix failing GitHub build action » [6.2.x] Drupal 11.2.x ModuleInstaller changes break WxT sub-module installs
Issue summary: View changes
smulvih2’s picture

Issue summary: View changes
smulvih2’s picture

Issue summary: View changes
joseph.olstad’s picture

Suggest having a look at the related change records for core.

This might lead to the most pragmatic solution.

New container_rebuild_required key in .info.yml files

Change \Drupal\Core\Config\ConfigInstaller to support installing multiple modules with a single container rebuild

smulvih2’s picture

Issue summary: View changes
joseph.olstad’s picture

I pinged Longwave in Slack about this asking for suggestions, no answer yet. Longwave is the one that merged this change into core or committed it.

There's a slack thread here , not sure how useful this is.

https://drupal.slack.com/archives/C1BMUQ9U6/p1763991961822269

web247’s picture

I just saw this issue when trying to test 6.2.x, I would suggest the recipe approach in the near future, in the meantime, as mentioned by @smulvih2 above, don't give the option to install individual modules and just install all modules at once.

joseph.olstad’s picture

Assigned: Unassigned » smulvih2
joseph.olstad’s picture

Unrelated Drupal 11.2.10 upgrade fixes included below:
#3557188: 11.2.x upgrade

  1. layout_builder_st and the status report page patch is required
  2. Another for wb-panel.js compatibility with the updated ck5 library included with D11.2.10 (you know about this one already and likely already have a similar fix)
  3. Fix for codemirror #3531472-24: Drupal 10.5 / 11.2 incompatibility