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

quietone created an issue. See original summary.

quietone credited dinarcon.

quietone credited mikelutz.

quietone’s picture

Issue summary: View changes

todo: Need to get drupal user name for mortona2k

matroskeen’s picture

Issue summary: View changes

Added thread with stats

quietone’s picture

quietone’s picture

Status: Needs review » Fixed

There were no requests to change these minutes in the following meeting.

Status: Fixed » Closed (fixed)

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