Friday, 2025-08-22

*** clarkb is now known as Guest2471807:46
*** Guest24718 is now known as clarkb13:47
opendevreviewGhanshyam proposed openstack/governance master: Show inactive project status in project.yaml  https://review.opendev.org/c/openstack/governance/+/95822917:01
JayFI just read the eventlet deadline move change that landed during my sabbatical; I'm going to note specifically: GR-OSS will not be dedicating resources to keeping the lights on for eventlet that long. It's actively harmful to the ecosystem to wait until 2027.2 to completely remove it.18:27
opendevreviewJay Faulkner proposed openstack/governance master: Retire metalsmith project from bare metal program  https://review.opendev.org/c/openstack/governance/+/95829718:33
opendevreviewJay Faulkner proposed openstack/governance master: Retire metalsmith project from bare metal program  https://review.opendev.org/c/openstack/governance/+/95829718:47
sean-k-mooneyJayF: you realise we actuly have not even deprecated supprot for eventlet in most of the project right. we are racying ot see if we can get to the point of having expermintal support for runnign in threaded mode in 2026.1 so that we can deprecated it there and be able to remove it in 2027.1 the over all time frame has not really changed the doc was just updated to refelct the19:35
sean-k-mooneyfact that we are treatign 2026.1 as  streach goal to be able to run without it and 2027.1 is the last release we expect to supprot using eventlet19:35
JayFsean-k-mooney: I think those arguments ignore an impending technical train coming down the tracks with every new python version. We can't just say things will work for a period of time when it's extremely likely we don't control all the pieces to make that true.19:36
JayFsean-k-mooney: my experts have been telling me that already python 3.13+eventlet is dubious19:36
sean-k-mooneywe have it fuly working on 3.13 now19:36
JayFthe subset of eventlet+openstack we test in CI works, yes19:37
JayFI do not think that can be implied to say "eventlet always works in python 3.13" 19:37
sean-k-mooneywell it works enouch for debain right now19:37
sean-k-mooneyto be clear the document is seting the hard deadlien for when we need to be done19:38
sean-k-mooneyits not the dates we are targeting19:38
sean-k-mooneynova might and i stress might be able to run wihtout devstack next cycle19:38
sean-k-mooneywater very likely will19:39
JayFI've been in the community long enough to not trust us to push that deadline out yet again, consequences be damned. I'm trying to do the best I can to ensure that doesn't happen again.19:39
sean-k-mooneywell we cannot deprecate supprot for it until we ahve a working repalcement so i dont know what documenting a timelien that does not match realtiy would have done19:40
JayFif we have eventlet deliverables in 2027.2, that means until end of of 2028, more or less, we'll have eventlet software to maintain19:40
sean-k-mooneyother then sete bad expectation for everyone invovled19:40
JayFwhich is absurd from my pov19:40
sean-k-mooneyJayF: that not wha tthat says19:40
sean-k-mooney2027.2 is the first release with no eventlet19:41
JayFah, so I'm 6 months off19:41
JayFthe point still is valid19:41
sean-k-mooneythat the release where we are deleteing any remaining code19:41
JayFessentially two more years of supporting a broken model19:41
JayFbecause we have to support what we release for 18 months19:41
JayFthat's the real "eol" for eventlet openstack19:41
sean-k-mooneyeffectivly but keep in mind that supproting it for 18 months does not mean on all python version19:42
sean-k-mooneyJayF: one of the thigns wew have been talking about as we have been doign this si that we may only supprot the non eventlet mode in 3.1419:42
sean-k-mooneyand to keep running in the legacy mode you will need to use an older python if eventelt is not fucntional on the new one19:43
JayFThat would reduce my concern a nontrivial amount, but I still am not "ok" with the plan we've created.19:43
JayFI'm clearly in the minority there, and that's fine, but I didn't want to let the topic lie because, as I said above, we have a tendency as a community to let the tail of these kinds of migrations last indefinitely long.19:43
sean-k-mooneywell nova was ask by operator to have at least 1 slurp release that supprot both19:43
sean-k-mooneywe are trying to make that 2026.119:44
JayFI don't think operators are informed well enough to make that decision tbh19:44
sean-k-mooneyif we dont get ther we either need to say no to the operator that wnated it or make that 2027.119:44
JayFI'm *thrilled* Ironic is not doing that path.19:44
sean-k-mooneywe want our default to swpa before that19:44
JayFwell, I guess "doing" is probably the wrong verb; we're done.19:44
sean-k-mooneyim not sure either. we are about 50% maybe more done with nova this cycle19:45
sean-k-mooneywe might also get the conductor workign without eventlet19:46
sean-k-mooneythat will just leave nova-compute and the console proxies19:46
JayFWe have some already-written stuff up on ironicbaremetal.org in the blog about performance changes19:46
JayFwhich could be gleaned for other projects, I suspect, to help with transition19:46
JayFI suspect the song may be different but it'll all be the same key19:46
JayFhttps://ironicbaremetal.org/blog/coming-soon-threading/19:47
sean-k-mooneyso far we are not seeign any performnce concrnes from the move but we are hoping to figure out how to do some larger testing19:47
JayFwe have no *concerns*, but the shape of the performance is changing from a "what shows up in your graphs" and/or "how do I tune my containers" standpoint19:47
JayFalthough I guess Ironic may just have more threaded workloads because of the nature of what we do19:48
sean-k-mooneyso in nova we used eventle tmainly as a threading lib not as a networkign layer19:54
sean-k-mooneyso moving from greenthreads to a real thread pool is a very good fit because we alrady used it as one19:54
JayFyeah19:55
JayFwe just need to be sure folks like kolla-ansible or similar with container-based deployments19:55
JayFloosen up the restrictions19:55
JayFsince we'll likely be using more ram to get more performance19:55
JayFa good tradeoff, but one that changes the shape of performance19:55
sean-k-mooneyeven then while we will have a litte more usage we didnt see much sofar19:55
sean-k-mooneybut we have a very small data set19:56
JayFIronic gets a lot of free bonus data19:56
sean-k-mooneyat least in the ci the memory usage for the nova schduler has not really increased much19:56
JayFfrom metal3.io CI and ironic-image CI 19:56
JayFover in k8s land19:56
JayFthey run in configurations that sometimes are unique to our ci configs which is nice for getting new data19:57
JayFthat's how the performance regression surfaced19:57
sean-k-mooneyso on the watcher side im really not worred about the meory usage because it used greenthread with a default limit of 4 https://docs.openstack.org/watcher/latest/configuration/watcher.html#watcher_decision_engine.max_general_workers19:58
JayFI think our threading cap is 300 by default, pre and post eventlet19:58
* JayF -> AFK for a while19:58
sean-k-mooneywhat i woudl be reallin interested in is if cern or a similar operator that updates to flamingo could try the treaded schduler for a day, a week, perhaps even on only 1 of n scchulder 20:00
sean-k-mooneyand they could let us know how it works at scale. if they notice no real delta that would be awsome.20:01
sean-k-mooneythe nova api already was scale by process not thread and also had very littel eventlet usage. im considering proposing making the threaded mode the default for it next cycle but that a ptg dicussion20:02
JayFkubajj in #openstack-ironic is who I'd ask, or email Arne, if you wanted some info. I know they run a couple of patches but nothing that'd be that impactful (they needed a way to pass a kickstart file thru openstack server create into the ironic anaconda driver)20:04
clarkbfwiw I don't think we need to deprecate eventlet its an implementation detail and not user facing20:04
clarkbif the software gets refactored but continues to function the same way as observed by the humans using it you can just change things20:04
JayFclarkb: Ironic didn't.20:04
clarkbsome of the library code might have to as those implementation details matter there. But they do not matter to someone deploying nova20:05
clarkbwhat matters to someone deploying nova is that the service continues to ingest the same configuration and behave consistently with that configuration20:05
sean-k-mooneywell wether we deprecate or not we do have to deal with the fact that its likely that some functionaltiy will be lost20:05
sean-k-mooneywe are groing to try and prot the in tree virt driver liek vmware but we really dont have a way to test it or zvm20:06
clarkbya so you'd deprecate the vmware driver in that case20:06
sean-k-mooneyim not sure if every cinder voluem driver "will just work"20:06
clarkbnot eventlet20:06
sean-k-mooneyor if they will also have similar problems20:06
sean-k-mooneyya perhaps20:07
sean-k-mooneyclarkb: to be entrilly transparent i didnt wnat to offer the option of swtichign between backend20:07
sean-k-mooneywhen i was orginally POCing the removal in nova i epxlcitly didnt20:07
JayFIronic'ers are used enough to hardware vendors breaking us and having to do quick fixes, so we might be a little less afraid of us breaking us :D20:08
sean-k-mooneyso the nova core team is really only doing because we were directly asked too20:08
clarkbyes I think that should never have been a toggle too. Once you believed it works without eventlet make the switch and move on. But as you say people asked for it and the decision was made20:09
sean-k-mooneyfor watcher we have one compoent called the applier that we need to port everything else was started and complete this cycle20:09
clarkba good lesson for the future if making changes that are largely transparent to operators or users.20:09
sean-k-mooneyso next cycle we will proably defatl to treading and in 2026.2 i expect to start deleteing the eventl code20:09
clarkbkinda like not asking end users about type annotations or which sql libraries are supported. We do what makes sense for the project and users roll with them20:10
gmaanwell, deadlines are modified considering the community bandwidth and current progress. we cannot always force things on upstream to do with hard deadlines. If it is not working on future python (at least python 3.13 is not issue now), then I am all ok to stay us on pythin version where things work fine20:42
gmaanthere are lot of best we can/should do but that is not reality always20:42
gmaanthe best will be if we can complete things even as per new deadlines but I will not be surprise if we do not 20:44
gmaanand I am talking about all the OpenStack projects (~30 where it is applicable) and not just a few one20:45
JayFYeah, you are sorta pointing at my real concern: if we already pushed the deadline back once, I'm very concerned we'd push it back again. Our community tends to be last minute at times with deadlines.20:45
clarkbright its less about trying to stick to a strict deadline and more that if we give ourselves 2 years then that is the fastest it will happen20:46
clarkbif we aim for 1 then maybe it gets done sooner20:46
gmaanmore matters here is to find more volunteer to finish the work. if we leave on projects then it will not be that fast20:48
gmaanI can bet ~40% projects even not aware of old or new deadlines20:49
JayFTo put it plainly: if you're a PTL or DPL liason for an OpenStack project, and you're not aware there is an urgent need to migrate from eventlet, are you doing your job? 20:50
gmaantrue, that is the challenges we are facing since many years, project activeness and the maintainers bandwidth. we can easily say and retire them but i do not think that is solution to any of the existing problems.  20:53
gmaanone day, oslo, keystone might be in same plate. 20:54
fungii clearly picked the wrong time to go find food20:54

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!