2024-11-07 Meeting notes

All public Working Group meetings follow the Recording Policy for Open edX Meetings

 Date

Nov 7, 2024

 Participants

  • @Feanil Patel

Previous TODOs

 Discussion topics

Item

Presenter

Notes

Item

Presenter

Notes

Moving the meeting time?

 

  • Should we move the meeting to 1hr later during US Daylight Savings Time?

    • Kyle and Jeremy can’t make the new time.

    • No objections to moving.

Codejail/Ubuntu 24.04 news



  • Code jail 24.04 will be fixed imminently

  • Suggestion, let this land on master and don’t backport.

    • Both will have support for multiple years so no major issue.

Django 5 Upgrade

 

  • We need someone to coordinate this work.

  • Should be pretty low stakes, we have 12 months to land this.

Potential setup.py to pyproject.toml migration

 

  • We may need to inject this maintenance in at some point, still need to investigate further.

edx-platform topics start here

edx-platform roadmap

Kyle

[Draft] edx-platform Architectural Roadmap · Issue #35144 · openedx/edx-platform

Inform - I’m coming back to this now that Sumac is cut.

Looking to wrap up what was in flight earlier in Sumac, and to establish goals for Teak.

 

Feanil

MFE Rewrite Tracker

Feanil will try to work on filling this out and working through dropping some of the old frontends

Auth PR

Feanil

feat: Allow trusted apps to perform cookie login. by xitij2000 · Pull Request #35221 · openedx/edx-platform

Django Upgrade

 

Probably start in earnest at the beginning of next year.

Preview moved to Learning MFE

 

Related PRs:

Example URLs

 Action items

@Feanil Patel move the meeting to 1hr later.
@Jeremy Ristau to forward us info on Kristin’s work on the Studio Preview page

Recording and Transcripts

Maintenance WG Recording: Maintenance Working Group Meeting – 2024/11/07 07:59 EST – Recording

Maintenance Working Group Meeting – 2024/11/07 07:59 EST – Transcript

Attendees

Adolfo Brandes, Awais Qureshi, Feanil Patel, Feanil Patel's Presentation, Sarina Canelake

Transcript

Adolfo Brandes: 

Adolfo Brandes: Hey, so the meeting time stayed the same. I figured it might have whichever for me.

Feanil Patel: Yeah. Yeah,…

Feanil Patel: it was set to UTC, so it stayed the same. It sounds like we might have enough people that want to move it. I don't know. Let's see who else is. Yeah, I figure you don't care, but I'll put it on the agenda.

Adolfo Brandes: I don't care.

Feanil Patel: share my screen. All right, I don't think we have a lot on the agenda. the list of previous to-dos is mostly things that I need to get around to doing. So, I will try to do that probably next week at the earliest. none of it's urgent. I do have some code jail news.

Feanil Patel: your chair and all.

Feanil Patel: 

Feanil Patel: Does anybody else have any other items to add to the agenda? I'll also drop a link here for first up is moving the meeting time to be an hour later. both Jeremy and Kyle can't make this time a waste. I feel like this would affect you the most because it would be an hour later. Is that something that is easy for you to make?

Awais Qureshi: Yes, no problem.

Feanil Patel: Okay, Serena, same. I assume it's fine with you. It's good.

Sarina Canelake: Yeah, I'm like feeding my cats right now. So, an hour later would mean they'd be fed.

Feanil Patel: Okay, that sounds so no objections. I'll move it enough. the code maintainer. so this is a good news on the Ubuntu front.

Feanil Patel: the code maintainer Moyesus was able to fi figure out the last thing that was preventing us from upgrading code to 2404. which is so I'm in the middle of reviewing that now, but I think we'll be able to land that and…

Adolfo Brandes: Nice.

Feanil Patel: move platform to running on 2404 on master. but I guess the question I have is I don't think it's worth backporting this into sumac and we'll just catch it on teik, but I wanted to talk that through if anybody else thought it was worthwhile.

00:05:00

Feanil Patel: We're already on 2204 for sumac and…

Adolfo Brandes: 

Feanil Patel: so going to 2404 for te seems fine.

Adolfo Brandes: I'd guess so.

Adolfo Brandes: I'm not the most heavy user of cold jail at all.

Feanil Patel: Yeah. Yeah. I mean I think as long as you're using it's not like both of those will be under security support for the next six months. So I don't see a huge value other than to be sort of I don't know officially on 2404 as a marketing thing,…

Adolfo Brandes: or the only question I'd have is around the Python version that actually runs it. I know that for 22 there was already an upgrade or…

Feanil Patel: right? No,…

Adolfo Brandes: or was it independent? I forget. Anyway,

Adolfo Brandes: 

Feanil Patel: is it? We've been upgrading Python independently. So, we're now Yeah.

Adolfo Brandes: Yeah, that's the thing with the most impact,…

Adolfo Brandes: right? The Python version…

Feanil Patel: Yeah. Exactly.

Adolfo Brandes: because the actual things that the authors wrote needs touchups, right?

Feanil Patel: Yeah. Yeah. So, yeah, the Python version is independently upgraded. We're on 311 officially for edex platform, but, I think most of the libraries now run 312. That's not an emergency. So, I think that's like I need to go see which libraries still need the 312 upgrade and…

Adolfo Brandes: 

Feanil Patel: we can probably just ticket that out and let maintainers take care of it. we'd have, I think, three years to do that. So 311 is going to stay in support for a while and…

Adolfo Brandes: Okay.

Adolfo Brandes: Okay, that's all right.

Feanil Patel: 3 yeah 312 makes other changes…

Adolfo Brandes: Not agenda.

Feanil Patel: which I'm going to talk about in a little bit with the potential move from setup to apply project. yeah. Yeah.

Sarina Canelake: 

Sarina Canelake: I think I have a question just around realistic adoption. if 2202 only has six more months of support and realistically people who adopt to the next named release do so a few months after it's actually released and so they won't be going on to TE for a few months after TEK is released. Does it? And if the backboard is easy, I kind of don't see why we wouldn't just do it.

Adolfo Brandes: Yeah, it's April 2027.

Feanil Patel: Yeah, 2204 doesn't continues to get security support for another two years.

Sarina Canelake: I thought you said six months. Okay, never mind then. Seems fine.

Feanil Patel: No, no, I was saying in the next six months nothing is going to happen they're both going to continue to get support and in six months we'll have moved to 2404.

Sarina Canelake: All right.

Feanil Patel: Yeah. Yeah.

Sarina Canelake: My concern is unfounded. Thank you.

Feanil Patel: Yeah, you got it. No. Yeah, that would be a very good reason to move it. but I think there's nothing. So, we'll land it on master. We'll announce it, but it's not an emergency for operators…

Adolfo Brandes: 

Feanil Patel: who run on releases to adopt. Right.

Adolfo Brandes: And just to keep this in the back of our minds,…

Adolfo Brandes: if somebody asks, Ubuntu also offers paid support for this stuff for a lot longer.

Feanil Patel: For longer,…

Adolfo Brandes: So yeah,…

Feanil Patel: So they Yeah.

Adolfo Brandes: if somebody's stuck on an old version of Abuntu,…

Feanil Patel: They want business.

Adolfo Brandes: they have the option of paying for it, So they'd have to maintain stuff manually.

Feanil Patel: For Yeah. All right. The next major upgrade is going to be the Django 52 which their LTS is not available for I think another couple of months.

Adolfo Brandes: 

Feanil Patel: But 50 and 51 are already available. so we can start fixing these things are dropped in Django 50 or…

Adolfo Brandes: Do we still get a lot of those?

Feanil Patel: these things are like the deprecation warnings are for 50 and so we can start fixing those now and I'm trying to there's still a lot…

Adolfo Brandes: So I remember it was much worse in the past. Now it's kind of over.

Feanil Patel: if you start up at xplatform there's one major one which is around how the indexing works which is in a bunch of our old models other than that. there's one for that and there's one around localization where a flag is being set to default true and being removed from Dango.

00:10:00

Adolfo Brandes: Yeah, there's some work there.

Feanil Patel: So use L10 Z.

Adolfo Brandes: 

Sarina Canelake: Do you know what flag that rings a bell, but I don't remember.

Feanil Patel: Yeah. We do need somebody to coordinate this work. I would love if it was not me, but once I've done a couple of other things that are on my plate right now, I may take this up just because I want what I want to do for this one is a little bit different, which get the tickets out. But then since we have two releases to land this stuff, hopefully this can just be background maintenance work that people can do schedule into their sprints over the next six months. So this should be really low stakes in some way. Yeah.

Adolfo Brandes: You're talking about the full jangle upgrade, right? Not just the

Feanil Patel: the whole Django upgrade. But for most of our libraries, they could be testing with Django 50 and 51 pretty easily. and if something is passing all three of those without warnings, then verifi we've removed a bunch of the warnings already. And when 52 lands, that jump and upgrade becomes a lot shorter and simpler. All still looking for somebody to help with that. I would love to have other people be able to coordinate large maintenance work, but that is a call I'll put out on the forums as well.

Feanil Patel: the last thing I have is this is a potential piece of maintenance that might come in over the next 12 months. There's changes happening to PIP and not to setup tools, but to pip to change how it works in because that the way you define a Python project has changed over the last couple years. pipro.tl TML is the official way to define a Python project and py and a bunch of its capabilities are being deprecated. so we may need to make this move sooner rather than later. I think most of our stuff is published so I think it's okay. but there needs to be some more investigation here.

Feanil Patel: 

Sarina Canelake: I know nothing about this. I'm curious for a shot in the dark level of effort.

Feanil Patel: Okay. …

Feanil Patel: I think for most of our libraries it will be a pretty easy migration. You just take a bunch of the fields that are in y and you move them into a py project. There's helpers for pi project.l to read the version out of a python file and things like that. So it's not super complex. I suspect for things like edex platform and some of the other big services there might be some figuring out to do where things live in a combination of files that include cfg piprotoml and setup.py. but like that that's those big ones are the ones that we need to figure out basically.

Feanil Patel: 

Feanil Patel: But I need to understand what's I have a couple of tickets I need to read to figure out what features are actually breaking in the next six months to a year. because one of the features that's breaking is editable installs of things via y and that is a change that we should be fairly resilient to because everything is published. but if you install directly from GitHub into an editable install, that's a thing that would break. So pip install- e and…

Adolfo Brandes: 

Feanil Patel: then a GitHub URL. So I think we've fixed a lot of that already. So we might be all right. But I like there's further investigation and understanding that needs to happen here.

Adolfo Brandes: The only thing I can think of that does that are or…

Adolfo Brandes: recommends doing that are the open edextor plugins repository.

Feanil Patel: Right. Right. the plug-in installs might need to but if it also depends like that is okay as long as those plugins are not using pay Pi if they're using comtotl that should still work.

Adolfo Brandes: then it's Yeah.

Feanil Patel: Yeah. Yeah.

00:15:00

Adolfo Brandes: So, just upgrade the plug. Yeah.

Feanil Patel: So we may just need to upgrade but I've done a deep dive on this stuff yet so I don't understand exactly what is the thing that's going to break. so I've just seen enough things flying through the Python packaging news feeds that I need to look at it further. and that's all we have for today. Does anybody else have anything?

Adolfo Brandes: All right.

Feanil Patel: All right.

Sarina Canelake: Thank you, Fel.

Feanil Patel: I'll let you guys go then. Yeah. Thank you.

Meeting ended after 00:15:57 👋

This editable transcript was computer generated and might contain errors. People can also change the text after it was created.

 

edx-platform Sub-Group Recording: edx-platform Maintenance Sub-Group – 2024/11/07 09:27 EST – Recording

edx-platform Maintenance  Sub-Group – 2024/11/07 09:27 EST – Transcript

Attendees

Feanil Patel, Feanil Patel's Presentation, Jeremy Ristau, Kyle McCormick, Robert Raposa

Transcript

Feanil Patel: 

Feanil Patel: You go.

Kyle McCormick: There we go.

Robert Raposa: 

Robert Raposa: Hello. …

Feanil Patel: Share my screen. Yeah. Yeah.

Robert Raposa: I'll be watching your screen, but if you don't mind sharing that link. Thanks.

Robert Raposa: 

Robert Raposa: So those were the topics from earlier.

Robert Raposa: I'm in the update.

Feanil Patel: Yeah. Yeah.

Feanil Patel: