Hello all, it’s time for the biweekly 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 second 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 2023-10-12

0️⃣ Who is here today?

smustgrave fly on the wall
mikelutz (he/him) o/
Stephanie lurking...
cosmicdreams Present
Dan Davis Here
quietone Hi

1️⃣ What should we talk about today? Suggest topics here and I will add threads. I will also check for comments on the issue for today's meeting.

smustgrave #2713327: Document ways to remove migration tables (ID map etc.) is the 2nd oldest ticket in review
benjifisher ... and it has only 158 comments.

2️⃣ Action items. To be added later.

3️⃣ Statistics

benjifisher All open: 347.
benjifisher Fixed in the last 2 weeks: 4 (not counting issues for meetings) including 1 Critical.
benjifisher RTBC: 9, 2 of which are Major and 4 have not been updated in more than 1 month.
benjifisher NR: 2, both Normal priority.
benjifisher Needs subsystem maintainer review: 6
benjifisher Google sheet for recording stats: https://docs.google.com/spreadsheets/d/1o0Rjlc1vnnLP5bM5P-SMMyGzqn7258hi...

4️⃣ Comment in this thread if you are looking for ways to contribute. Give us some idea of what you would like to do: documentation, code review, testing, project management, ...

5️⃣ Previous minutes.

benjifisher We need to add a transcript to the issue for the 2023-09-14 meeting: #3384564: [meeting] Migrate Meeting 2023-09-14 2100Z
benjifisher Also the one before that, 2023-08-31: #3381772: [meeting] Migrate Meeting 2023-08-31 1400Z
smustgrave If ya ping me I can review those for you
benjifisher I am working on 2023-08-31.
benjifisher @smustgrave: that one is now NR.
Dan Davis @benjifisher I can post 9-14 if you’d like?
benjifisher Please do.
Dan Davis @benjifisher I have uploaded a transcript for 2023-09-14. Can you please give credit to the participants? I am not currently a maintainer.

6️⃣ Announcements

benjifisher The next meeting (2023-10-12 at 21:00 UTC) will be the quarterly video meeting.
cosmicdreams What qualifies as an announcement?  Is this where I post wild ideas, or I'm sorry, announce I'm considering a wild idea?
benjifisher We have not formalized any rules, but I think announcements
  1. have general interest
  2. are not meant to start discussions.

7️⃣ Consolidate code in the migrate_drupal module. (edited) 

8️⃣ Provide a way to remove migration tables (ID map etc.)

benjifisher #2713327: Document ways to remove migration tables (ID map etc.)
mikelutz (he/him) I suppose we can at least run tests on the latest patch.
benjifisher quietone (#156) reminded me that I promised (#144, not #44) to do some wordsmithing:

For example, replace "This is a process that is not reversible" with "This action cannot be undone", which is used pretty consistently in Drupal.
benjifisher Is that the only thing holding up this issue?
smustgrave I haven’t reviewed it as quietone seemed to specifically want you or assuming another submaintainer to review. (edited)
smustgrave Seemed pretty advanced for someone like me to make the call that it was the correct approach
mikelutz (he/him) I bumped it back to NW with some complaints about the approach.
mikelutz (he/him) I don’t want to add a UX to the api module.
mikelutz (he/him) the api should provide apis, and it does provide an api to remove tables through MigrateIdMapInterface::destroy()
mikelutz (he/him) It should be up to the drush commands and UX modules like migrate tools and migrate_drupal_ui to provide a UX for that api.
benjifisher I think we are planning to add UI to the API module in #3063856: Add ability to view migrate_message table data
benjifisher But this issue is much more likely to be needed for sites that have used migrate_drupal, so I think there is an argument for putting the changes in migrate_drupal_ui.
mikelutz (he/him) YEah.. Now that you mention it, I’m not a fan of that being in the api module either.
benjifisher Unfortunately, we do not have a separate migrate_ui module (and we will not add one at this point).
mikelutz (he/him) That should also go in migrate tools.
mikelutz (he/him) No, we won’t.  I think of migrate tools as the migrate_ui module, but I admit its messy.
benjifisher But migrate_tools is designed to work with config migrations, based on migrate_plus. I think it already exposes migration messages from those.
mikelutz (he/him) I won’t stand in the way of adding table deletion UI in the api module, but I’m not a fan.
mikelutz (he/him) Yes, I would prefer a migrate_ui module in core, and no, that’s not happening.
mikelutz (he/him) but I feel like we are opening a can of worms with these.
mikelutz (he/him) The migrate module should be a developers api, and shouldn’t provide UX.  It should provide tools geared towards developers.
benjifisher Yes, but then there should be a migrate_ui module. As long as we are wishing for things that will not happen, I would like to make migrate_plus an API module (process plugins, the uri source plugin, and the fetcher and parser plugins related to that) and move the config migrations to migrate_tools, which also provides UI for them. (edited)

9️⃣ Wrap up

benjifisher Thanks for participating! Please continue to add comments in the threads. In 1-7 days, we will post a transcript for today's meeting.

Comments

benjifisher created an issue. See original summary.

benjifisher credited cosmicdreams.

benjifisher credited ddavisboxleitner.

benjifisher credited mikelutz.

benjifisher credited smustgrave.

benjifisher’s picture

benjifisher
he/him or they/them
Primary language English
Location Boston area
commented
Issue summary: View changes
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Fixed

Went into slack and #migration channel to 9/28/2023 and all the conversations threads have been captured.

Status: Fixed » Closed (fixed)

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