- Issue created by @drumm
- πΊπΈUnited States drumm NY, US
This mostly works now, but this error appears 281 times
[error] All the id fields are required for a table migration. (/var/www/html/web/modules/contrib/migrate_plus/src/Plugin/migrate/destination/Table.php:180) [notice] Processed 64153 items (63872 created, 0 updated, 281 failed, 0 ignored) - done with 'drupalorg_migrate_project_release_supported_versions'
- πΊπΈUnited States drumm NY, US
Those look like projects that didnβt migrate for me locally for whatever reason, I didnβt pay attention to the console output there, but there was some.
So the
plugin: migration_lookup
section might not be right for the compound primary key in this table. We might not care since production should be in a better state. - πͺπΈSpain fjgarlin
I think that the release references should use the
migration_lookup
plugin to double check that they belong to migrated releases. - πͺπΈSpain fjgarlin
Aren't those references to release nodes? I think the migration IDs need to be different in that case. I made some suggestions in the MR.
- πΊπΈUnited States drumm NY, US
Yep, they are release nodes, I was on autopilot with copy/paste. Runs okay with the changes applied locally for me.
-
fjgarlin β
committed b218979d on 1.0.x authored by
drumm β
Issue #3517014 by drumm, fjgarlin: Migrate...
-
fjgarlin β
committed b218979d on 1.0.x authored by
drumm β
- πͺπΈSpain fjgarlin
The MR looks good to me. Merging and it will be deployed in the next site deployment.
Automatically closed - issue fixed for 2 weeks with no activity.