Hello all, it’s time for the weekly migration subsystem meeting. The meeting will take place in slack in various threads
This meeting:
➤ Is for core migrate maintainers and developers and anybody else in the community with an interest in migrations
➤ Usually happens every Thursday and alternates between 1400 and 2100 UTC.
➤ Is done on the #migration channel in Drupal Slack (see www.drupal.org/slack for information).
➤ Happens in threads, which you can follow to be notified of new replies even if you don’t comment in the thread. You may also join the meeting later and participate asynchronously!
➤ Has a public agenda anyone can add to. See the parent issue for an idea of the typical agenda.
➤*Transcript will be exported and posted* to the agenda issue. For anonymous comments, start with a :bust_in_silhouette: emoji. To take a comment or thread off the record, start with a :no_entry_sign: emoji.
Core migration issues
Next video meeting
2021-10-14 - The hope is that most or all of the maintainers will attend. We will try to focus on longer-term goals than in the weekly meeting.
Add agenda items
0️⃣ Who is here today?
| dinarcon | Hello :wave: |
| alison | Hi there! |
| danflanagan8 | Howdy! |
| mortona2k | Hey! |
| quietone | HI. |
1️⃣ What do we need to talk about? We will make threads for issues brought up here as needed.
| dinarcon | I brought up the topic of testing migration results last week. Still wondering what is the best way to detect changes in a site's configuration that would cause a migration to no longer work as expected. Looking for best practices as this might not something that the migrate API necessarily needs to keep track of. For context, this is the thread from last week https://drupal.slack.com/archives/C226VLXBP/p1629382961222700 |
| quietone | #2953111: Only migrate role permissions that exist on the destination |
| quietone | #3226744: Derive statistics module migrations per node type |
| quietone | #3122649: [PP-2] Derive path alias migrations per entity type (and bundle) |
| quietone | Approval of previous minutes |
2️⃣ Action items to be added later
3️⃣ Statistics. I don’t have em. ask Benji next week :stuck_out_tongue:
| Matroskeen | Fixed since last week's meeting: 0 (not counting the issue for the meeting). |
| Matroskeen | RTBC: 9, 1 is Minor and the rest are Normal priority. |
| Matroskeen | NR: 35, including 5 Major and 8 that have not been updated in more than three months. |
| Matroskeen | Google sheet for recording stats: https://docs.google.com/spreadsheets/d/1o0Rjlc1vnnLP5bM5P-SMMyGzqn7258hi... |
4️⃣ migrate mapping & messages table names are truncated, can lead to incorrect mapping lookups
| mikelutz (he/him) | #2845340: migrate mapping & messages table names are truncated, can lead to incorrect mapping lookups |
| quietone | Ooh, @mikelutz (he/him) you rtbc'ed this in the past. Can you review it again? |
5️⃣ Deprecate Migration::set()
| mikelutz (he/him) | #2796755: [PP-1] Deprecate Migration::set() |
| quietone | This needs reviews. And i have time to make patches.Anyone available to review? |
6️⃣ Only migrate role permissions that exist on the destination
| mikelutz (he/him) | Only migrate role permissions that exist on the destination |
| quietone | Do we like the suggestion by alexpott in this comment #2953111: Only migrate role permissions that exist on the destination#comment-14061034 |
| alison | I don't know. Maybe it's pointless clutter for me to reply anyway, but I just want you to know you're not speaking into a void?? :meow_peek: |
| quietone | It isn't clear what happens to the permissions that are "declared as real permissions by the user module under a special migrated permission section" . |
| quietone | I guess I should read the related issue. |
| alison | Ok I didn't understand that either, I thought it was just me. |
| mortona2k | Are the change records on that related ticket worth noting? They were added after alex's comment. |
7️⃣ Derive statistics module migrations per node type
| mikelutz (he/him) | #3226744: Derive statistics module migrations per node type |
| quietone | Just want to be sure how we are handling these issues that add derivers that are not needed by core |
| mikelutz (he/him) | I was quite sure we decide they must live in contrib, unless something has been discussed that I was unaware of. |
| quietone | I believe so too, though I have yet to find the conversations. |
| quietone | Is there a suitable contrib module to move them to? |
| danflanagan8 | There's an empty module called Statistics for drupal 7 to 8 migration. I was briefly excited. |
| quietone | There is conversation in 5️⃣ at #3099135: [meeting] Migrate Meeting 2019-12-05 |
8️⃣ [PP-2] Derive path alias migrations per entity type (and bundle)
| mikelutz (he/him) | #3122649: [PP-2] Derive path alias migrations per entity type (and bundle) |
| danflanagan8 | I love the IS on this one |
| quietone | Same as 7️⃣ this isn't needed by core. |
9️⃣ Wrap-up. Benji does something here, but I’ll be honest, I’ve never looked at it.
🔟 Last Week’s transcript
| mikelutz (he/him) | #3228277: [meeting] Migrate Meeting 2021-08-19 |
Comments
Comment #6
quietone commentedtodo: Need to get drupal user name for mortona2k
Comment #7
matroskeenAdded thread with stats
Comment #9
quietone commentedComment #10
quietone commentedThere were no requests to change these minutes in the following meeting.