Closed (fixed)
Project:
Experience Builder
Version:
0.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
23 Dec 2024 at 12:22 UTC
Updated:
21 Jan 2025 at 23:39 UTC
Jump to comment: Most recent
Comments
Comment #3
longwaveComment #4
longwaveThis is showing some interesting fails in 11.x that need further investigation.
Comment #5
longwaveAt least some of the fails appear to be related to #3416522: Add the ability to install multiple modules and only do a single container rebuild to ModuleInstaller. Experience Builder creates config entities in
hook_rebuild(), this means that we can no longer install XB and other config that depends on XB's generated config entities at the same time.Even setting
container_rebuild_required: truedoesn't help because config validity is checked for the entire set of modules, not per group:Comment #6
longwaveOpened #3497173: Modules that create config cannot be installed alongside modules that depend on that config for #5.
Unsure if we actually want to hard fail on errors in 11.x or let them slide for now, as long as we fix them before 11.2 is out we are OK, and chasing HEAD is difficult, especially if core makes a change and MR tests start failing for reasons unrelated to the MR itself. Maybe we should only fail "next minor" on branch tests and not MRs?
Comment #7
longwaveThe fails in ComponentTreeItemTest are also solved by adding
container_rebuild_required: trueto xb_test_config_node_article.info.yml.Comment #8
longwaveReverted the fail-on-next-minor change, let's fix that over in #3497866: CI: also test against next minor: 11.2.x aka Drupal 11.x HEAD
Comment #9
wim leers+1 — but then the MR needs a small update :)
Comment #10
longwaveComment #11
longwaveComment #12
longwaveComment #13
wim leersComment #15
wim leers