2024-12-05 Meeting notes
- Feanil Patel
All public Working Group meetings follow the Recording Policy for Open edX Meetings
Date
Dec 5, 2024
Participants
@Feanil Patel
Previous TODOs
Discussion topics
Item | Presenter | Notes |
---|
Item | Presenter | Notes |
---|---|---|
Sumac.1 will happen on Monday |
|
|
Maintainers-at-large |
| |
edx-platform | ||
Paver update |
|
|
Action items
Recording and Transcript
Recording: https://drive.google.com/file/d/12YLzVSpxzODskPN-Xxh-7k_Ubj7xHko_/view?usp=sharing
Maintenance Working Group Meeting - 2024/12/05 09:00 EST - Transcript
Attendees
Adolfo Brandes, Awais Qureshi, Feanil Patel, Kyle McCormick, Michelle Philbrick, Robert Raposa
Transcript
Robert Raposa: Good
Adolfo Brandes: Hey, Robert. I don't know.
Robert Raposa: You're recording now.
Adolfo Brandes: I think it's automatic.
Robert Raposa: I see. hello everyone out there. Morning.
Adolfo Brandes: Yeah. Hey, Kyle.
Kyle McCormick: Good morning.
Adolfo Brandes: So, Fil was sick yesterday. I don't know if he's better. Maybe.?
Kyle McCormick: He was in a meeting 30 minutes ago, but he sounded pretty rough, you might need to lay down. So, you might want to move on without him.
Adolfo Brandes: Okay.
Michelle Philbrick: I don't have any agenda items.
Adolfo Brandes: Let me try and find the
Adolfo Brandes: I don't think there's an agenda for today.
Kyle McCormick: Yeah, same.
Adolfo Brandes: All I can see are the last meeting.
Robert Raposa: Are there any next time to do items?
Adolfo Brandes: Actually, the one I'm seeing is even …
Feanil Patel: Everybody, sorry I'm one second.
Adolfo Brandes: there's one. Okay.
Adolfo Brandes: Doesn't seem like it.
Feanil Patel: Do people have a lot of content for today they want to discuss?
Feanil Patel: Yeah. Yeah.
Adolfo Brandes: We ran a quick poll just now and nobody had anything to add.
Feanil Patel: I think we're kind of in waiting until we start picking up some of the next big maintenance things which is I think the Django related work which we have a little bit of time for. so I'm starting the meeting notes here.
00:05:00
Feanil Patel: discuss. We don't have to stick around.
Adolfo Brandes: Yeah. Only thing I want to mention is we're looking to release sumac on Monday.
Feanil Patel: I think maintainers at large it looks like is one of the things we need to talk about for next time.
Kyle McCormick: Mhm.
Feanil Patel: But This is sumac.1.
Adolfo Brandes: So if you have any last minute things you want to get into sumac, this would be the time. Otherwise, Come on up to my top one.
Feanil Patel: All right.
Kyle McCormick: On the edex platform side, I just wanted to mention that we are still in the process of getting paver out of edex platform and a lot of that work has been slowed by places we've realized where the CIA suite wasn't checking the things it said it was going in and removing checks that are just way too far gone to care about anymore and then fixing up the checks that we are keeping and then also rewriting them in paper. So that's been ongoing. it's been good to jump in and really find things and we're trying to do better printing so that it's easier to tell if this ever happens in the future.
Kyle McCormick: But just wanted to highlight that I imagine that that's specific to edex platform because of favor. But I don't think any test suite is immune to false positives. So something to keep in mind. don't just blindly trust the green check mark.
Feanil Patel: And I think in particular the more complexity we add to our test suite the harder it will be to tell that test got run.
Kyle McCormick: Yes. Yeah. Yeah. Keep it simple, Big time for Just run the freaking shell command.
Feanil Patel: Yeah. Yeah,…
Kyle McCormick: Put it in a make file if you need to. That's it. Ireland out.
Feanil Patel: seven layers of Python decorators weren't the right answer you're saying. okay, that sounds good.
Feanil Patel: in the maintainers at large role. The only thing that I wanted to check in about is whether that's one big role or a couple of domain specific roles. I could imagine seeing a front-end maintainers at large and a backend maintainers at large at least because I could imagine different people wanting to sort of step up for one or the other of those. but other than that split, I couldn't think of any other ways that we might want to significantly reduce or restrict people's access. But I'm also not sure if that's valuable.
Feanil Patel: If maintainers are people we trust to sort of restrain themselves and only do maintenance where they know what they're doing, then having just the singular maintainers at large that has right access to all of the repos isn't crazy to me.
Adolfo Brandes: So for the front end, we pretty much already have something like maintainers at large. yeah,…
Feanil Patel: And yeah, the maintainers front end.
Adolfo Brandes: we have a I don't know what the group's called currently. It used to be front end-all.
Feanil Patel: Yeah. Yeah.
Adolfo Brandes: Now it's committers front end. Okay, there you go. I don't know if we want to count that as precedent or as we want to fix that. I'm fine which whatever decision is
Feanil Patel: Yeah. Maybe…
Adolfo Brandes: All right.
Feanil Patel: then adding just a committers back end and saying that anybody can get nominated but that group is not de facto maintainer. One of the things about committers front end that's interesting is that it's a smaller group and that group is the maintainer of certain repos. which is different from the maintainers at large because that group would never become the official maintainer of a repo.
Kyle McCormick: My impression of committer's front end is that it's native front end experts.
00:10:00
Feanil Patel: Right. Right.
Kyle McCormick: Whereas maintainers at large I could imagine having people who we thrust to work with CI and tooling and small tasks but we don't necessarily considered backend experts. So in my head they're somewhat different,…
Feanil Patel: They can resolve build and lint issues but not necessarily like module store shenanigans.
Kyle McCormick: right? Yeah.
Feanil Patel: Yeah, Maybe that makes sense to have just a single use at large.
Kyle McCormick: Yeah. I think a lot of this stuff will transfer between front end and back end.
Feanil Patel: Yeah. Yeah.
Kyle McCormick: Particularly anything GitHub actions will just apply everywhere. And the differences between package.json and setup.py Pi is something a maintainer at large should probably learn pretty quickly if they know one or the other.
Feanil Patel: Yeah, I think so. A lot of it is honestly knowing more about GitHub actions than it is knowing about the specific technology stack.
Feanil Patel: And we'll add people to those with the understanding that they're only working on tasks related to keeping sort of CI working and other maintenance tasks driven by this group.
Kyle McCormick: Sounds great to me.
Feanil Patel: I think that's it then, unless anybody has anything else. Cool. All right.
Robert Raposa: Just Are we down to no to-do items or are we just missing the section?
Feanil Patel: We're just missing the section right now. I can add it back in. the two items for next were the ones we just discussed.
Robert Raposa: Is there anything else to check in or no? Yeah,…
Feanil Patel: there's a couple of things that I'm following up on.
Feanil Patel: But that's been delayed because everybody's been so focused on the named release. and then there's a couple of things outstanding for Jeremy who's not here.
Robert Raposa: we did reach out about the foreign performance testing, but I don't think
Robert Raposa: Did anyone see anything happen? I can double check on that one.
Feanil Patel: Yeah, it would be good to know about that one. And I assume that we haven't had any movements on the front end deppers.
Feanil Patel: Although I think I'm going to start trying to pick some of those up in the new year and landing them as best as I can so that we can reduce the number of things we're tracking on the rewrite tracker because I think those ready to go.
Robert Raposa: Yeah, I think on the forum performance,…
Robert Raposa: the person most responsible is on leave, parental leave,…
Feanil Patel: Okay.
Robert Raposa: which we found out because of other forum stuff that is happening.
Robert Raposa: And I guess the forum v2 is either blocked or was reverted. I don't know because there were other bugs that needed to get fixed, but we're getting close to releasing that.
Feanil Patel: That sounds good. And those bugs either have been reported and are being worked on or…
Robert Raposa: Yeah, I think Rita's worked on the first one happened to be in parallel but I guess it was in forums itself. So maybe Dave didn't know about it. and then now there's something else, but it might just be config changes on our side that need to happen.
Adolfo Brandes: All right.
Robert Raposa: So I don't know if just better docs might be needed.
Feanil Patel: Yeah. Yeah,…
Robert Raposa: I'm not sure.
Feanil Patel: those might sounds like maybe there's updates to the forums repo read me which has all the setup instructions.
00:15:00
Robert Raposa: Thank you.
Feanil Patel: All In that case, I think I can let you all go. Thank you. Bye.
Kyle McCormick: Folks.
Adolfo Brandes: All right.
Awais Qureshi: My
Meeting ended after 00:15:28 👋
This editable transcript was computer generated and might contain errors. People can also change the text after it was created.