- πΈπ°Slovakia kaszarobert
We also encountered this issue.
- set up domains on Drupal site's master branch.
- create new feature branch where you alter the site title, export the new config and commit.
- now go back to the master branch, and import config (here we tested something else manually that got merged to master)
- go back to the previous feature branch and try to import config, you get:pc@pc:/var/www/html/ks-graphic.site (master =)$ lando drush cim +------------+------------------------------------------------------------+-----------+ | Collection | Config | Operation | +------------+------------------------------------------------------------+-----------+ | | system.site::domain.config.dogbanner_localhost.system.site | Rename | +------------+------------------------------------------------------------+-----------+ Import the listed configuration changes? (yes/no) [yes]: > y [error] Drupal\Core\Config\ConfigImporterException: There were errors validating the config synchronization. Rename operation for simple configuration. Existing configuration system.site and staged configuration domain.config.dogbanner_localhost.system.site. in Drupal\Core\Config\ConfigImporter->validate() (line 794 of /app/web/core/lib/Drupal/Core/Config/ConfigImporter.php). In ConfigImportCommands.php line 324: The import failed due to the following reasons: Rename operation for simple configuration. Existing configuration system.site and staged configuration domain.config.dogbanner_localhost.system.site.
This meant we could not import config anymore, so we had to remove this domain-altered config manually. Therefore, I'm raising priority because it could break the config deployment to QA and production sites.
- πΈπ°Slovakia kaszarobert
Again, the same error I experienced when collegues changed the site name per domain and I tried to import config on a completely different site:
| | domain.config.mezotrade_eu.system.site::domain.config.age_attraction_sk.system.site | Rename | | | system.site::domain.config.age_attraction_sk.system.site | Rename | +------------+-------------------------------------------------------------------------------------+-----------+ // Import the listed configuration changes?: yes. [error] Drupal\Core\Config\ConfigImporterException: There were errors validating the config synchronization. Rename operation for simple configuration. Existing configuration domain.config.mezotrade_eu.system.site and staged configuration domain.config.age_attraction_sk.system.site. Rename operation for simple configuration. Existing configuration system.site and staged configuration domain.config.age_attraction_sk.system.site. in Drupal\Core\Config\ConfigImporter->validate() (line 794 of /app/web/core/lib/Drupal/Core/Config/ConfigImporter.php). In ConfigImportCommands.php line 324: The import failed due to the following reasons: Rename operation for simple configuration. Existing configuration domain.config.mezotrade_eu.system.site and staged configuration domain.config.age_attraction_sk.system.site. Rename operation for simple configuration. Existing configuration system.site and staged configuration domain.config.age_attraction_sk.system.site.
system.site.yml:
_core: default_config_hash: 5fiO5Le0rk-Y8-wpD231_eVHzGxPd2HtLK4cHmzZ2hw langcode: sk uuid: f4d6487a-1aec-48b6-9ddd-15a1c7595ded name: 'Age attraction' mail: mail@example.com slogan: '' page: 403: '' 404: '' front: /node/21 admin_compact_mode: false weight_select_max: 100 default_langcode: sk mail_notification: ''
domain.config.age_attraction_sk.system.site.yml:
_core: default_config_hash: 5fiO5Le0rk-Y8-wpD231_eVHzGxPd2HtLK4cHmzZ2hw langcode: sk uuid: f4d6487a-1aec-48b6-9ddd-15a1c7595ded name: 'Age attraction' mail: mail@example.com slogan: '' page: 403: '' 404: '' front: /node/21 admin_compact_mode: false weight_select_max: 100 default_langcode: sk mail_notification: ''
domain.config.mezotrade_eu.system.site.yml
_core: default_config_hash: 5fiO5Le0rk-Y8-wpD231_eVHzGxPd2HtLK4cHmzZ2hw langcode: sk uuid: f4d6487a-1aec-48b6-9ddd-15a1c7595ded name: 'Mezo trade' mail: mail@example.com slogan: '' page: 403: '' 404: '' front: /node/28 admin_compact_mode: false weight_select_max: 100 default_langcode: sk mail_notification: ''
- π¨π¦Canada arakwar
We're using the Drupal 9.4.14 version with Domain 2.0.0-beta1, and have the same issue.
- First commit to issue fork.
- Open on Drupal.org βCore: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
about 1 year ago Not currently mergeable. - last update
about 1 year ago 88 pass, 10 fail - Status changed to Needs review
about 1 year ago 8:13am 1 September 2023 - πΊπ¦Ukraine HitchShock Ukraine
In general, I would say that the better practice is to add all 'domain.config.*' to the config_ignore.settings.yml and this also will resolve the issue.
However, I made a quick solution to resolve UUID duplicates.
It will be nice to test it. - Status changed to Needs work
about 1 year ago 8:14am 1 September 2023 - πΊπ¦Ukraine HitchShock Ukraine
Back to work cuz tests must be reviewed and fixed too.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Quoting configuration system maintainer @alexpott from #3405800-20: Config collections do not trigger configuration events consistently β :
Domain is not using its own StorableConfigBase object - see https://git.drupalcode.org/project/domain/-/blob/8.x-1.x/domain_config/s... - so it will trigger regular config events for it's overrides. Which is incorrect because as with config translations there are only partial representations of configuration and meant to be merged via the override system with active config in the default collection.
- πΊπΈUnited States brianbrarian
Are there prospects for a patch anytime soon?
- π§πͺBelgium tim-diels Belgium π§πͺ
I came here with the issue that on a site install with existing config and rename validation was thrown as described in the issue summary.
The steps are quiet easy to reproduce.- Install a website
- Enable domain
- Add 2 domains
- Enable domain_config_ui
- Add language and domain context to system site page for domains
- Set config per domain per language
- Export config
- Try to reinstall site from config and it should fail
@HitchShock Thank you for the work in your MR. This looks like the way forward on fixing this issue.
Not sure how to write tests for site install with existing config.
But for everyone having this issue:
- Use the patch from the MR
- Delete the UUID from the config files and re-install
- Save the config pages again and export config
- πΊπΈUnited States agentrickard Georgia (US)
MR is out of date and needs a refactor.
- πΊπΈUnited States agentrickard Georgia (US)
Marking as a fearure. Domain Config UI is a *convenience* module and experimental. Developers can work around this issue as noted or by using Domain Config directly.
- First commit to issue fork.
- ππΊHungary denes.szabo Hungary
Re-rolled the patch against the latest dev 2.0.x.
- π«π·France dqd London | N.Y.C | Paris | Hamburg | Berlin
#14 has described a workaround already (thanks Steffen!) and this is not breaking a site so decreasing issue prio...
Descriptions of the Priority β and Status β values can be found in the Drupal project issues β documentation.
- Status changed to Needs review
2 months ago 12:02pm 3 September 2024