While working on a migration I ran into a errors when with URL_alias. Looking back and comparing previous url_alias.yml configuration-only files I found that the adding back an optional dependency (upgrade_d7_node_translation) solved the errors.

So previously --configuration-only output files included the upgrade_d7_node_translation dependency where as a recent --configuration-only output omitted it and caused the url_alias.yml migration to fail.

url_alias.yml...

destination:
plugin: url_alias
migration_dependencies:
required: { }
optional:
- upgrade_d7_node_translation

CommentFileSizeAuthor
#9 3023504-9.patch9.81 KBheddn
#9 interdiff_6-9.txt3.41 KBheddn
#6 3023504-6.patch10.04 KBheddn
#4 3023504-4.patch9.68 KBheddn
#2 3023504.patch9.66 KBheddn

Comments

jeffwpetersen created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Active » Needs review
StatusFileSize
new 3023504.patch9.66 KB

Here's a test that shows the migration is exported. Are you sure this isn't working?

Status: Needs review » Needs work

The last submitted patch, 2: 3023504.patch, failed testing. View results

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs work » Needs review
StatusFileSize
new 3023504-4.patch9.68 KB

The profile field migration stuff is being addressed in #2823414: User Profile field values not migrated from Drupal 6

Status: Needs review » Needs work

The last submitted patch, 4: 3023504-4.patch, failed testing. View results

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs work » Needs review
StatusFileSize
new 3023504-6.patch10.04 KB

  • heddn committed c92c89e on 8.x-3.x 
    Issue #3023504 by heddn: url_alias.yml potential bug.

Status: Needs review » Needs work

The last submitted patch, 6: 3023504-6.patch, failed testing. View results

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs work » Needs review
StatusFileSize
new interdiff_6-9.txt3.41 KB
new 3023504-9.patch9.81 KB

This will fail until upstream is fixed in #3023747: D6 profile migrations assume stubs, which fail

Status: Needs review » Needs work

The last submitted patch, 9: 3023504-9.patch, failed testing. View results

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs work » Needs review

Upstream fixed. Re-queueing.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Title: url_alias.yml potential bug. » Add test coverage for drush config export

Re-titling.

  • heddn committed f001e56 on 8.x-3.x 
    Issue #3023504 by heddn: Add test coverage for drush config export
heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.