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

Video meeting 2022-10-13 2100Z

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.

0️⃣ Who is here today?

quietone Hi
benjifisher Hello! :wave:
mikelutz (he/him) Jeez, I haven’t even sent out pings yet.. you two are on it today..
srjosh :wave::skin-tone-2:
bsnodgrass (he/him) Hello
danflanagan8 Dan here

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.

quietone Upgrading Drupal wiki guide, https://www.drupal.org/docs/upgrading-drupal
benjifisher #3266491: Add content in migrate UI functional tests only when needed

Was RTBC, now NR after reroll.
mikelutz (he/him) Seems more of an action item than a thread, but I’ll bite.
benjifisher As you see fit.
benjifisher I guess I am hoping that someone will comment "mine" if we make a thread and then
  1. It will get reviewed.
  2. We will avoid duplicated effort.
benjifisher Action item from last week: #3063856: Add ability to view migrate_message table data. I am about to open a new issue and mark the current one as postponed. If I do not mention it here, then @quietone might do the same thing at the same time.
danflanagan8 I’m interested in getting a maintainer’s thoughts on this issue: #2960204: Make it possible for optional log messages from *skip* process plugins to contain context

2️⃣ Action items. To be added later.

3️⃣ Statistics

mikelutz (he/him) Fixed: 0, RTBC: 5, NR: 14

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

5️⃣ Previous minutes.

mikelutz (he/him) #3311216: [meeting] Migrate Meeting 2022-09-22 1400Z
benjifisher I was about to generate a transcript for 2022-09-15. Let's not duplicate the effort.
quietone I haven't made the transcript in a while, seems like my turn.
benjifisher NR: (2022-09-15) #3308616: [meeting] Migrate Meeting 2022-09-15 1400Z
benjifisher Have you updated the Slack scraper yet?
quietone I have but I haven't used it. I'll check it now.
quietone Yes, it worked and I have posted the transcript.

6️⃣ Announcements

benjifisher The next video meeting will be in two weeks, 2022-10-13. Same time as today's meeting. We will post a link in the channel to join. (edited)
Matroskeen The issue with file source plugin documentation is finally in core - #3189481: [Meta] Add source plugin documentation to the codebase

It was the last task of the meta I created almost 2 years ago: https://www.drupal.org/project/drupal/issues/3189481

Thanks to everyone who helped with reviews, especially @quietone and @benjifisher :bow:
mikelutz (he/him) 2 weeks is October 13th.
benjifisher Oops. I edited my earlier comment in this thread.
mikelutz (he/him) Well, now my comment makes me look like I’m not paying attention.

7️⃣ Drupal Upgrade manual (edited) 

quietone Wanted  to check that the recent restructure is moving in the right direction and what other changes should be made.
mikelutz (he/him) https://www.drupal.org/docs/upgrading-drupal
quietone There are to 'preparation pages', https://www.drupal.org/docs/upgrading-drupal/prepare-major-upgrade/prepa... and https://www.drupal.org/docs/upgrading-drupal/upgrading-from-drupal-6-or-.... Can we get that into one?
srjosh Is this about upgrading migration modules/code to compatibility with D10, or about upgrading sites using migration?
dinarcon Or about about updating the documentation itself?
bsnodgrass (he/him) Good topic, mobile ATM but wanted to check in!
quietone This is about the changes to the documentation itself.
quietone And more about structure than the details. Although, details need to be considered when there are two pages for 'preparing'.
quietone What are your thoughts about making the 'Upgrading Drupal' guide easier to navigate?
bsnodgrass (he/him) I'll be glad to review the detail on this... it's a topic that is very timely for us, and I've been getting deep into details on a number of projects currently
bsnodgrass (he/him) I have a reminder set for tomorrow sometime if my day stays under control

8️⃣ BADCamp is next week, in person and online!

srjosh Oh, I wish I could make it this year!
irinaz JOin us for online sprint !!!! https://docs.google.com/document/d/1GruXFHmdSVj7H_mGZdUQGrZQDsPVIPMRlhuu...
irinaz https://www.badcamp.org/community/contribution-lounge
irinaz and for sessions https://www.badcamp.org/events/schedule
srjosh interesting!
mikelutz (he/him) @irinaz If you want me to make threads in the meeting, please comment in 1️⃣ . Please don’t post threads with number icons in the middle of a moderated meeting.
irinaz @mikelutz (he/him) my apologies.    I can delete this thread and send you content directly
mikelutz (he/him) In the future.. It’s done now, and a worthwhile thread to have in the meeting.
bsnodgrass (he/him) I set a reminder for myself on this one as well.
bsnodgrass (he/him) @irinaz per for contrib sprints, those are online? Is there a need to register or just show up? and are there Sessions that will be available online? I couldn't find anything that looked like registrations for remote attendees
irinaz yes, today and tomorrow
irinaz sprint is online, camp is in person
irinaz no need to register for sprint 🙂
bsnodgrass (he/him) @irinaz thanks!
bsnodgrass (he/him) Tight schedule, but I will see what I can do.

9️⃣ Add content in tests only when needed

mikelutz (he/him) #3266491: Add content in migrate UI functional tests only when needed
mikelutz (he/him) Needs a reroll, any volunteers?
benjifisher Correction: the latest reroll needs a review. Any volunteers?
benjifisher It was RTBC before the reroll.
quietone I can reroll it, probably today.
quietone Ugh, read the thread! It does not need a reroll, it needs a review.
quietone I can look at it but I can't RTBC.
benjifisher I did the reroll, so I cannot RTBC.
mikelutz (he/him) You can RTBC your own reroll...
mikelutz (he/him) As long as you didn't write the code in the original patch.
benjifisher OK, I am on it.
mikelutz (he/him) And even then you are fine, as long as someone else RTBCed the original.
benjifisher I think I owe you a snarky comment ...
benjifisher ... #3266491: Add content in migrate UI functional tests only when needed#comment-14713566

🔟 Make it possible for optional log messages from *skip* process plugins to contain context

mikelutz (he/him) #2960204: Make it possible for optional log messages from *skip* process plugins to contain context
danflanagan8 I have an approach that works well for this, but it also seems like something that maybe doesn’t provide much value anymore.
mikelutz (he/him) I'm not a huge fan of adding more code around this. I think it's of limited usefulness given modern debugging tools. This is turning into the migration equivalent of adding print statements throughout your code, and I'm not a huge fan of enabling that or having to maintain the code and functionality around it when we want to make real improvements to the system.
danflanagan8 What do you think about closing as “will not fix” then? And justifying it by saying that it’s no longer of significant value given the related issue I added in my most recent comment?
mikelutz (he/him) I love closed won't fix...
mikelutz (he/him) Just in general..
mikelutz (he/him) But here too.
danflanagan8 Closing anything always feels good
benjifisher I do not think we give issue credit for Closed (won't fix). It is a shame.

Less is more.
danflanagan8 It’s ok, most of the patches on this one were unnecessary credit-farming things anyway
mikelutz (he/him) I don't think we should give issue credit for closed won't fix...
danflanagan8 We get credit for talking about it in the meeting 🙂
benjifisher :rolling_on_the_floor_laughing:
danflanagan8 Is one of you going to close it? Feels like a maintainer should flip the switch on a closed won’t fix
mikelutz (he/him) Done, and commented on my reasons.
danflanagan8 thanks!
danflanagan8 ha ha, I just read it
mikelutz (he/him) Feel free to elaborate regarding the discussion here. ;-)
danflanagan8 will do

1️⃣1️⃣ Wrap up

quietone Thanks mike!

Comments

benjifisher created an issue. See original summary.

benjifisher credited danflanagan8.

benjifisher credited dinarcon.

benjifisher credited irinaz.

benjifisher credited Matroskeen.

benjifisher credited quietone.

benjifisher credited srjosh.

benjifisher’s picture

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

benjifisher credited bsnodgrass.

benjifisher credited mikelutz.

benjifisher’s picture

benjifisher
he/him or they/them
Primary language English
Location Boston area
commented
quietone’s picture

quietone commented
Issue summary: View changes

I reviewed the minutes and the number of replies is correct for each discussion. But I did find a missing sentence in the reply from Matroskeen in Announcements and have added that. And made formatting corrections. I agree with the credit.

Version: 9.5.x-dev » 10.1.x-dev

Drupal 9.5.0-beta2 and Drupal 10.0.0-beta2 were released on September 29, 2022, which means new developments and disruptive changes should now be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

quietone’s picture

quietone commented
Status: Needs review » Fixed

There have been no comments here that there are errors in the minutes. I am going to close this now.

Status: Fixed » Closed (fixed)

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