Tuesday, 2025-04-22

opendevreviewTakashi Kajinami proposed openstack/nova master: Use consistent program name for wsgi scripts and entry points  https://review.opendev.org/c/openstack/nova/+/94260503:20
opendevreviewChungWon Lee proposed openstack/nova master: Remove condition for kvm serial condition  https://review.opendev.org/c/openstack/nova/+/94778005:28
opendevreviewChungWon Lee proposed openstack/nova master: Remove condition for kvm serial console  https://review.opendev.org/c/openstack/nova/+/94778005:29
opendevreviewArnaud Morin proposed openstack/nova master: Fix limit when instances are stuck in build_requests  https://review.opendev.org/c/openstack/nova/+/94780408:48
opendevreviewLajos Katona proposed openstack/nova master: blueprint: iothreads-for-instances  https://review.opendev.org/c/openstack/nova/+/93925409:26
opendevreviewKamil Sambor proposed openstack/nova master: Replace eventlet sleep with time.sleep  https://review.opendev.org/c/openstack/nova/+/94721210:06
*** ralonsoh_ is now known as ralonsoh10:19
*** haleyb|out is now known as haleyb12:57
gibidansmith: welcome back! We made progress with the eventlet removal. You can read up on it here https://gibizer.github.io/categories/eventlet/ or look at the patches here https://review.opendev.org/q/topic:%22eventlet-removal%22+project:openstack/nova13:20
dansmithack!13:23
bauzasgibi: nice blog writing on eventlet effort ! If I can help by reviewing patches, lemme know because now I better have context :)14:07
opendevreviewThomas Rotsaert proposed openstack/nova master: feat(affinity) - Create Affinity/Anti-Affinity based on a new hypervisor property: hypervisor_rack  https://review.opendev.org/c/openstack/nova/+/94783314:16
opendevreviewThomas Rotsaert proposed openstack/nova master: feat(affinity) - Create Affinity/Anti-Affinity based on a new hypervisor property: hypervisor_rack  https://review.opendev.org/c/openstack/nova/+/94783314:20
andrewbonneyIf any cores have a second could I request a review of the small bug fix in https://review.opendev.org/c/openstack/nova/+/919961? Sean has added +2 already15:15
UgglaNova meeting in around 20mn15:41
kevkoHi folks, I would like to ask ... if i have auto_converge to false , complete_action to force_complete , timeouts set , post_copy disabled ...why my instance is in infinity live-migrate action ? I would say that if there is instance under heavy LOAD and there are too many dirty pages to transfet which can't be handled ..it should fail because of15:43
kevkotimeout and then PAUSE trigerred ... I can imagine that auto_converge and post copy can help ..but i can't turn on because of customer ... But i am curious ..why nova is not pausing instance and instead of this ...still trying to copy ram ...of course only last 5 percent which is permanently changing because of heavy load to instanced15:43
sean-k-mooneykevko: simple answer is because nova does not contole live migration15:59
sean-k-mooneykevko: basically once we call libvirt or any other hypervior to start the migration we are hands off until it completes15:59
sean-k-mooneythe timeout and timeout action are the excpetion 15:59
kevkosean-k-mooney: it's normally working for normal instances16:00
sean-k-mooneyif you set ti to force compelte we can ask libvirt to force compelte the migration which involves pausign the guest on the dest and then compleinting the copy16:00
kevkosean-k-mooney: yes, of course 16:00
Ugglahi guys, sorry to interrupt, we gonna start the meeting.16:00
kevkosean-k-mooney: so why it is not paused ? and during that time the RAM copied ? 16:00
Uggla#startmeeting nova16:01
opendevmeetMeeting started Tue Apr 22 16:01:07 2025 UTC and is due to finish in 60 minutes.  The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
opendevmeetThe meeting name has been set to 'nova'16:01
UgglaHello everyone16:01
sean-k-mooneykevko: because it would not be a lilve migration any more, but we can talk about that after the meeting16:01
Ugglaawaiting a moment for people to join.16:01
kevkosean-k-mooney: okay, i will wait ... 16:02
gibio/16:02
bauzaso/16:02
elodilleso/16:02
sean-k-mooneyo/16:04
Uggla#topic Bugs (stuck/critical) 16:04
Uggla#info No Critical bug16:04
Uggla#topic Gate status16:04
Uggla#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs16:04
Uggla#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:04
Uggla#link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status16:05
bauzasnothing to say 16:05
Uggla#info Please look at the gate failures and file a bug report with the gate-failure tag.16:05
Uggla#info Please try to provide meaningful comment when you recheck16:05
Ugglatbh, I have not looked at the jobs this time. Shame on me.16:05
Ugglathanks bauzas for the info.16:06
Uggla#topic tempest-with-latest-microversion job status 16:06
Uggla         #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=016:06
Uggla#link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=016:07
UgglaStatus is the same as I have not checked with Gmann about it.16:07
Uggla#topic PTG summary 16:08
Uggla#link https://etherpad.opendev.org/p/r.bf5f1185e201e31ed8c3adeb45e3cf6d16:08
Ugglaas a reminder16:08
Uggla#topic Release Planning 16:08
Uggla#link https://releases.openstack.org/flamingo/schedule.html16:08
Uggla#info Nova deadlines are set in the above schedule16:08
Uggla#topic Review priorities16:09
Uggla#link https://etherpad.opendev.org/p/nova-2025.2-status16:09
UgglaStill draft.16:09
Uggla#topic Stable Branches16:10
Ugglaelodilles, please go ahead.16:10
elodillesthanks o/16:10
elodilles#info stable gates seems to be in good health16:10
elodilles#info stable/2023.2 (bobcat) will move to End of Life next week16:10
Uggla\o/16:10
elodilles#info final 2023.2 bobcat releases are out: nova 2023.2 (28.3.1), placement 2024.2, 2024.1, 2023.2 (12.0.1, 11.0.1, 10.0.1)16:10
elodillesthanks melwitt and sean-k-mooney for the tons of stable reviews o/16:11
Ugglamelwitt++16:11
gibiindeed, thanks for making these releases full of backports16:11
Ugglasean-k-mooney++16:11
elodillesi think we just have to wait until the deadline and hope we did not introduce regressions o:)16:12
gibinah, what regressions, those are upgrade oportunities to caracal :)16:12
elodilles:]16:12
elodillesyepp :)16:12
Ugglagibi, I like your mindset ! :)16:12
elodillesbtw, i've prepared nova stable releases for dalmatian and caracal as well16:13
elodillesto not leave everything till the end o:)16:13
elodillesnova: Release 2024.1 Caracal 29.2.1  https://review.opendev.org/c/openstack/releases/+/94784616:13
gibiearly upgrade opportunities :)16:13
elodillesnova: Release 2024.2 Dalmatian 30.0.1  https://review.opendev.org/c/openstack/releases/+/94784716:13
elodilles#info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci16:14
elodillesplease add here if you find any gate issue ^^^16:14
elodillesand that's all about stable from me for now16:14
elodillesUggla: back to you16:14
Ugglathx elodilles !16:14
elodillesnp16:14
Uggla#topic Gibi's news about eventlet removal16:15
Uggla#link Series: https://gibizer.github.io/categories/eventlet/16:15
Uggla#link reminder patches to review: https://review.opendev.org/q/topic:%22eventlet-removal%22+project:openstack/nova16:15
Ugglagibi, anything new you want to share ?16:15
gibinope, you shared all what I had :)16:16
bauzasas I said, I can help for reviewing the series16:16
gibiany help is appreciated :)16:16
bauzascool16:16
UgglaI know sean-k-mooney already reviewed some of them.16:16
UgglaHaving someone else would be great.16:17
sean-k-mooneysome yes melwitt  looked at a couple too16:17
sean-k-mooneybut ya more the merrier16:17
Ugglaok moving on to next topic16:19
Uggla#topic Bug scrubbing16:19
UgglaSo I wanted to clean up a bit the new bugs. But I didn't have a chance to to it yet. Anyway I have selected 5 new bugs that looks properly defined.16:20
UgglaSo first one : https://bugs.launchpad.net/nova/+bug/210425516:20
UgglaSo my expected outcome would be to know if we think it is a valid bug or not. If it is incomplete what to ask.16:22
sean-k-mooneyits valid. i looked at it breifly and chated about it on irc16:22
bauzasSeems a caching issue16:23
bauzasso yeah valid16:23
sean-k-mooneytl;dr if the nova-comptue agent is started when the vf is attached to the vm 16:23
sean-k-mooneywe cant get the network capablities16:23
sean-k-mooneyand we cache the vf info on start up in both nova and libvirt16:23
sean-k-mooneylibvirt invalidates its cache based on udev events16:23
sean-k-mooneyi.e. when the vf is returned to the host16:24
sean-k-mooneynova does nto invlaidate its cache so that part of the problem16:24
Ugglaok cool do not worry about settings the bug fields I'll do that later after the call16:24
sean-k-mooneylets mark it valid and medium16:24
Ugglagreat16:24
sean-k-mooneyand we can decied how to fix it later16:24
Ugglanext one:16:24
Ugglahttps://bugs.launchpad.net/nova/+bug/210598716:25
sean-k-mooneyso melwitt has a patch that might fix this indirectly16:26
sean-k-mooneygibi: do you know if we merged the one to auto discover the config options and use the sdk to get the limits?16:26
gibisean-k-mooney: not top of my head16:27
sean-k-mooneyack16:27
gibibut the bug above links to a separate oslo.limit fix16:27
sean-k-mooneyso if this config option is not being repected in nova today16:27
sean-k-mooneythen its valid but we need to confirm that16:27
gibiso I'm wondering if just using the latest oslo.limits fixes it for nova16:27
sean-k-mooneyya i suspect that might be hte case16:27
sean-k-mooneyalthough im a bit confused16:28
sean-k-mooneybecause the also have endpoint_interface16:28
sean-k-mooneywhich i tought was ment to do the same thing as the new filed they just added16:28
sean-k-mooneyhttps://review.opendev.org/c/openstack/oslo.limit/+/946128/4/oslo_limit/opts.py16:28
sean-k-mooneymaybe we should ask if they set endpoint_interface=internal16:29
sean-k-mooneyin the bug and come back to this16:29
sean-k-mooneyill go ask that16:29
Ugglaok so we can set it to incomplete and then review it when it will be back in new.16:30
sean-k-mooneytkajinam: sicne you wrote https://review.opendev.org/c/openstack/oslo.limit/+/946128 you might be able to add some context to https://bugs.launchpad.net/nova/+bug/210598716:31
sean-k-mooneyUggla: for what its worht we configure our dwonstream installer to use internal too16:31
sean-k-mooneyso im not entirly sure why the oslo change is requried16:32
sean-k-mooneybut we can move on16:32
Ugglaok next one: https://bugs.launchpad.net/nova/+bug/2106073   this one was opened by Mohammed Naser, corfirmed on Ironic. And I'm pretty sure Mohammed proposed a fix for it.16:33
Ugglabtw I have just noticed sean-k-mooney commented on it.16:34
sean-k-mooneyya so this is unfixable in nova without rewriting parts of the ironic deriver16:35
sean-k-mooneytl;dr ironic is modifying port bidinges, vnic typs and other atributes out of band of nova16:36
sean-k-mooneythat means that the info we provide in teh configdrive string can never be correct becuase ironic changes things in neutron that its not allowed too after we call provision16:36
Ugglaok but that sounds a valid bug despite we rely on Ironic to fix it ?16:37
sean-k-mooneyso if itgs going to do that it also need ot update the config drive info it has16:37
sean-k-mooneyya its somewhat valid to fix in ironic16:37
sean-k-mooneyagain they are modifying neutron ports in a way that only valid wehn using ironic without nova16:37
sean-k-mooneythat the root of the issue.16:37
sean-k-mooneyironic port groups only exixits in the ironic api16:38
sean-k-mooneythey are not modled in nova or neutron16:38
sean-k-mooneyso there is no portable way to express nic bonds ectra today16:38
Ugglashould we set it as "won't fix" on our side ?16:39
sean-k-mooneyi would proably mark the nova sid as cant fix 16:39
sean-k-mooneywont fix is fine too16:39
sean-k-mooneyi think we have invalid?16:39
sean-k-mooneynot sure which is better16:39
Ugglayes invalid is an option too16:39
sean-k-mooneyanywya the tl;dr is because ironic and the ironic virt driver are doing thign virt drivers are not allowed too it can only be "fixed" in ironic16:40
UgglaI vote for invalid on our side. Explaing ^ any objection with that ?16:40
Uggla3...16:41
Uggla2...16:42
Uggla1...16:42
Ugglaok let's do that.16:42
Ugglanext one: https://bugs.launchpad.net/nova/+bug/210638016:43
sean-k-mooneythat seam valid ish16:44
sean-k-mooneyif we delete a vm we should remove it form instance_group_member in some form16:44
sean-k-mooneyi suspect that wont happen until we archive it however16:44
sean-k-mooneyalthough we dont have shadow tables in the api db16:45
sean-k-mooneyso perhaps we should just delete it when the vm is deleted16:45
gibiyeah it seems like we should remove the vm from instance_group_member during delete, but it is hard as it is in the api db while the vm is in the cell db16:46
sean-k-mooneythere is a seperate bug for the slownes due to the request spec16:46
sean-k-mooneyloading16:46
sean-k-mooneyso we can keep that out of scope16:46
sean-k-mooneyso dansmith and i were wondering about a related topic. if you do not have api access from teh cell condutor and do local delete beacuse the conenction to the cell is down16:47
sean-k-mooneywill anything clean up the instnace mapping?16:47
sean-k-mooneylike to me this feels similar i feel like the instnace group membership is somethign we shoudl eb cleaning up once the instnace goes to delete in the cell db16:48
gibiI think it would have the same problem that the VM delete happens in the cell and we don't have an up call16:48
sean-k-mooneybut i dont know if anythign will actully do it today16:48
sean-k-mooneygibi: this kind of feels like something the supercondutor could do as a perodic16:48
gibiso the only reliable way would be a periodic16:49
gibiyepp16:49
sean-k-mooneybut i belive that would be the first such perodic in teh super conductor16:49
gibiyeah, I think so16:49
gibinot impossible to do just not a low hanging bugfiz16:49
gibix16:49
sean-k-mooneyUggla: to summerise, this is valid, it proably need a new periodic and there are likely other cases like this that should be fixed16:49
Ugglaperfect.16:50
sean-k-mooneyi would also agree not a low hanging fuirt but perhaps low to medium imporantce16:50
UgglaLatest one for this week: https://bugs.launchpad.net/nova/+bug/210681716:52
gibino stack trace so it is hard to tell16:55
sean-k-mooneyso neutron supproted adressless port long before nova did16:56
sean-k-mooneysylvain added supprot for adressless port but i dont know if that in antelope?16:56
bauzasbobcat iirc16:56
sean-k-mooneyhttps://specs.openstack.org/openstack/nova-specs/specs/yoga/implemented/boot-vm-with-unaddressed-port.html16:57
bauzasor maybe antelope, can't remember this16:57
sean-k-mooneyit was actully yoga16:57
bauzasohoho16:57
sean-k-mooneybut i do know there are still some issue with metadta16:57
sean-k-mooneyso they are aslo using trunk ports16:58
sean-k-mooneywhic i dont know if that was tested as part of this16:58
sean-k-mooneythey are also doing this via heat16:59
sean-k-mooneywhich is fun...16:59
Ugglamaybe we can ask for a reproducer. At least the procedure to reproduce it16:59
Ugglawithout heat16:59
sean-k-mooneyi knwo that heat uses the same micorverion for all nova calls16:59
sean-k-mooneyand it default to a pretty old one16:59
sean-k-mooneyUggla: yes i think we shoudl ask for a simple repoducer without heat17:00
sean-k-mooneywhat im unsure about is is this a nova or neutron issue17:00
sean-k-mooneybauzas: i dotn think we added a an api microversion for this correct17:00
sean-k-mooneyUggla: supproting adress port in combiniation with turnk ports was not in socpe fo the spec17:01
sean-k-mooneyso if this is repoducable without heat17:01
sean-k-mooneywe have two choices 17:02
sean-k-mooneytreat it as a new feature to actully supprot using both feature togetehr17:02
sean-k-mooneyor consider it a bug in the existing implimation17:02
UgglaOk, that works for me — incomplete plus a reproducer. That way, we’ll be able to narrow down the possible issue.17:03
sean-k-mooneyi think to do it properly in nova nova woudl have to check the port bidning of all the sub ports instead fo teh trunk port17:03
sean-k-mooneyso im inclidne to say this is a feature request but sure lest see wht they come back with17:03
Ugglayep I was thinking the same. Maybe that's not a "bug". But it will end up in a feature request.17:04
Ugglacool ok thanks all for this triage session. Latest topic 17:05
Uggla#topic Open discussion17:05
Ugglaone point from my side17:05
UgglaNext week's meeting: I’ll be on vacation and unable to run it. Would someone be available to run it on my behalf?17:05
UgglaI'll propably prepare a list of bug to triage. (but that's not mandatory)17:06
bauzasSorry, I can only help for the first 30 mins17:07
bauzas(double meeting at the same time with TC one started)17:07
Ugglabauzas, ok better than nothing. IMHO.17:08
bauzasokay, I'll try to help then17:09
bauzaslet's then do a short one17:09
Ugglaok17:10
UgglaTime to close.17:10
gibio/17:10
Ugglasorry for the 10mn overtime.17:10
sean-k-mooneyo/17:11
UgglaThanks all17:11
Uggla#endmeeting17:11
opendevmeetMeeting ended Tue Apr 22 17:11:14 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:11
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2025/nova.2025-04-22-16.01.html17:11
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-04-22-16.01.txt17:11
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2025/nova.2025-04-22-16.01.log.html17:11
UgglaThat's in the box.17:11
UgglaI hope the bug triage was ok. Feel free to let me know if something is not appreciated.17:12
sean-k-mooneyUggla:i think it was effeective17:12
sean-k-mooneythats all triage is really ment to be, look at the issue, see if there is enough info to say if its valid or not, ask for it if its missing and mark it triaged if there is17:13
sean-k-mooneyif we have an idea of how to fix it even better but we dont need to actully come up with the solution but its good to take notes in the bugs if we know of any workarounds17:14
sean-k-mooneygibi: i know you less of a fan of doing this on the fly, how did you find it?17:14
Ugglayep, but that's the first time I'm doing this with IRC. So I wonder about the timing.17:14
UgglaOf course if you are uncomfortable to do it on the fly, I could send the bugs upfront (e.g Friday). Just let me know.17:15
sean-k-mooneykevko: so looping back to your live migration quetion17:18
sean-k-mooneylooking at the code im not sure why you are not seeing the guest pause as expected17:18
sean-k-mooneywaht i will say is we likely shoudl add some extra logging17:18
sean-k-mooneybecuase there isnt a debug level log showitn that we took the pause codepath unless there is an excption when we call libvirt17:19
sean-k-mooneykevko: it might be worht checkign your libvirt logs for the domain to see if there is a pause request or any error on teh libvirt side17:19
sean-k-mooneykevko: it would be also good to confirm thate you do not see teh "failed to pause" warning https://github.com/openstack/nova/blob/master/nova/virt/libvirt/migration.py#L755-L76917:20
opendevreviewMerged openstack/nova master: Remove eventlet based WSGI server entry points  https://review.opendev.org/c/openstack/nova/+/94724017:22
sean-k-mooney:) 17:24
sean-k-mooneykevko: at the very minium we should likely imporve debug loging to that its similar to understand why you are not seeing the behvior we expect17:25
sean-k-mooneykevko: it may be good to report a bug for this but likely it will take effort to try and repoduce17:25
sean-k-mooneykevko: hopefully the issue you are seeign is not disto/version sepciifc. 17:26
opendevreviewMerged openstack/nova master: Remove WSGIService and WSGIServer classes  https://review.opendev.org/c/openstack/nova/+/94724517:38
sean-k-mooneyUggla: we triaged some bugs in the meeting but i dont see that reflected in launchpad18:37
sean-k-mooneyUggla: are you going to update them async based on what we discussed18:37
opendevreviewMerged openstack/nova master: Fix description of [pci] alias  https://review.opendev.org/c/openstack/nova/+/94461122:28

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