Monday, 2025-11-03

*** mhen_ is now known as mhen02:54
opendevreviewChungWon Lee proposed openstack/nova master: Reduce node build fail value by time  https://review.opendev.org/c/openstack/nova/+/93992906:17
opendevreviewChungWon Lee proposed openstack/nova master: Reduce node build fail value by time  https://review.opendev.org/c/openstack/nova/+/93992906:26
opendevreviewChungWon Lee proposed openstack/nova master: Reduce node build fail value by time  https://review.opendev.org/c/openstack/nova/+/93992909:44
* Uggla would like to remind to update nova upstream meeting timeslot: https://meetings.opendev.org/#Nova_Team_Meeting09:50
opendevreviewBalazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional  https://review.opendev.org/c/openstack/nova/+/95609009:53
opendevreviewBalazs Gibizer proposed openstack/nova master: Make guestfs' Tpool usage optional  https://review.opendev.org/c/openstack/nova/+/96582109:53
opendevreviewBalazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock  https://review.opendev.org/c/openstack/nova/+/95609109:53
opendevreviewBalazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests  https://review.opendev.org/c/openstack/nova/+/95609209:53
opendevreviewBalazs Gibizer proposed openstack/nova master: Do not fork compute workers in native threading mode  https://review.opendev.org/c/openstack/nova/+/96546609:53
opendevreviewBalazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode  https://review.opendev.org/c/openstack/nova/+/96546709:53
opendevreviewBalazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312  https://review.opendev.org/c/openstack/nova/+/95591509:56
sean-k-mooneyUggla: oh ya thats today have you updated the wiki?09:56
Ugglasean-k-mooney, not yet gonna update it.09:57
sean-k-mooneyhttps://meetings.opendev.org/#Nova_Team_Meeting09:57
sean-k-mooneyit looks correct09:57
gibisambork_: I moved https://review.opendev.org/c/openstack/nova/+/955915 out from my series and put it top of master as it belongs to the vnc series you volunteered to. Feel free to build top of it09:57
sean-k-mooneyUggla: oh i didnt mean the adgenda09:58
sean-k-mooneybut that would be good to update as well09:58
sambork_gibi, ack thanks!09:58
sean-k-mooneyUggla: i was asking becaue the meetign page has ical files so i could just update it in my calandar09:59
jkuliksean-k-mooney: (since I got from the PTG that pinging people is _the way_) if you find some time, could you take another look at https://review.opendev.org/c/openstack/nova/+/959696 ?09:59
sean-k-mooneysure10:00
sean-k-mooneyoh i remember looking at this before10:00
sean-k-mooneyjkulik: you mention there is a second change ye plan to propose? is that also pushed for reveiw or still somethign ye are working on10:03
jkulikI haven't pushed it, yet. I've got the code downstream already and I think I described it in the bug-report.10:03
sean-k-mooney```Another reason is, that nova-api queries all server-groups from the DB - fetching all the data and creating all the objects - and afterwards applies the limits/offset. This is highly inefficient if the number of server-groups is larger than the pagination limit.```10:04
sean-k-mooneyit that part10:04
jkulikyes10:05
sean-k-mooneyack10:05
jkulikI didn't want to overwhelm upstream with too many parallel patches, so I thought pushing one patch at a time would be better.10:06
Ugglasean-k-mooney, yep fyi the ics here should be ok to import https://meetings.opendev.org/#Nova_Team_Meeting, and now the wiki page https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting should be ok too regarding team meetings. 10:06
sean-k-mooneyjkulik: my only real feedback is https://review.opendev.org/c/openstack/nova/+/959696/2/nova/tests/unit/api/openstack/compute/test_server_groups.py#393 is not a unit test it sa functional test but its not discimilar to the other "unit" tests in the file so i think its ok10:08
sean-k-mooneyits defiently testing more then a single fucntion in isolation10:08
gibijkulik: I have a request inline to improve  condition and I would like to ask you to provide a releasenotes10:09
gibiotherwise it looks OK to me10:09
jkulikthank you both10:10
sean-k-mooneygibi: ah using is none thats fair10:10
sean-k-mooneyan empty set woudl resutl in member bing an empty list otherwiws when then intersection is doen10:11
sean-k-mooneyjkulik: adding a release note also makes sense but i woudl suggest updating the release not in your second patch so we have one release note for the bug assuming we merge both this cycle and backport both together10:13
jkulikok.10:13
sean-k-mooneyjkulik: is that the only nova review you have open currently. i feel like there was another but looking at your gerrit list i dont see it10:16
sean-k-mooneyi may be miss remembering as i knwo there were a few patches that were dicussed at the ptg that i intened to loop back to this week10:17
jkulikI don't have another open PR myself, but was wondering over in #openstack-eventlet-removal how to best handle olso.vmware's use of a Timeout in https://review.opendev.org/c/openstack/oslo.vmware/+/94210910:21
opendevreviewPablo Colson proposed openstack/nova stable/2025.1: wip  https://review.opendev.org/c/openstack/nova/+/96590210:21
jkuliksean-k-mooney: maybe you're remembering the NVRAM topic, because I joined in on the discussion? it was nicolairuckel's though.10:21
*** sambork_ is now known as sambork10:43
opendevreviewJohannes Kulik proposed openstack/nova master: api: Pre-query not deleted members in server groups  https://review.opendev.org/c/openstack/nova/+/95969610:46
sean-k-mooneyjkulik: i think it was either that or some of noonedeadpunk topic where we said we shoudl just merge the existing patchs for iothread and live migration concurrency11:09
opendevreviewJohannes Beisiegel proposed openstack/nova master: fix cinderclient _SESSION http_retries  https://review.opendev.org/c/openstack/nova/+/96591411:58
*** bogdando_ is now known as bogdando12:38
opendevreviewPablo Colson proposed openstack/nova stable/2025.1: wip  https://review.opendev.org/c/openstack/nova/+/96590212:42
opendevreviewBalazs Gibizer proposed openstack/nova master: [CI]nova-alternative-configuration tests eventlet  https://review.opendev.org/c/openstack/nova/+/96592213:07
opendevreviewBalazs Gibizer proposed openstack/nova master: Move monkey_patch call to each cmd from init  https://review.opendev.org/c/openstack/nova/+/96592313:07
opendevreviewBalazs Gibizer proposed openstack/nova master: Default native threading for sch, api and metadata  https://review.opendev.org/c/openstack/nova/+/96592413:07
opendevreviewBalazs Gibizer proposed openstack/nova master: Default native threading for sch, api and metadata  https://review.opendev.org/c/openstack/nova/+/96592413:11
gibisean-k-mooney: dansmith: ^^ as agreed on the PTG, lets switch the default early in G ^^13:13
sean-k-mooneyoh for the threading model ack13:13
opendevreviewMax proposed openstack/nova master: fix: leftover volume_attachment on instance delete  https://review.opendev.org/c/openstack/nova/+/93498413:17
opendevreviewMax proposed openstack/nova master: Add regression test for bug #2088066  https://review.opendev.org/c/openstack/nova/+/96592713:17
opendevreviewBalazs Gibizer proposed openstack/nova master: Default native threading for sch, api and metadata  https://review.opendev.org/c/openstack/nova/+/96592414:22
Ugglafungi, I might need your help. My ptg summary is blocked (too large) waiting for your validation. Can you please allow this message on the mailing list.15:12
fungiUggla: no problem, i was just starting to do my daily check of the moderation queues, just a sec15:13
UgglaTo all: nova upstream meeting in ~45mn.15:14
Ugglafungi, thx.15:14
fungiUggla: done, should be in everyone's inbox momentarily. let me know if you need anything else15:16
Ugglafungi, cool thanks15:19
opendevreviewMax proposed openstack/nova master: Add regression test for bug #2088066  https://review.opendev.org/c/openstack/nova/+/96592715:24
opendevreviewMax proposed openstack/nova master: fix: leftover volume_attachment on instance delete  https://review.opendev.org/c/openstack/nova/+/93498415:24
opendevreviewRajesh Tailor proposed openstack/nova master: Update the api-ref for unshelve  https://review.opendev.org/c/openstack/nova/+/93805415:32
clarkbUggla: I think your email was sent as both html and plain text. So you can cut the size in ~half by only sending text fwiw15:54
Ugglaclarkb, yes, but I'd like to provide both. I thin html is a bit more friendly for graphical clients.15:59
Uggla*think15:59
clarkbI use graphical clients and the first thing I do is disable html everywhere I can...16:00
sean-k-mooneysame16:01
sean-k-mooneyi only render html if there i sno plain text version in the email16:01
Ugglaclarkb, I agree, but not sure everybody is doing the same.16:01
* dansmith is too16:02
Uggla#startmeeting nova16:02
opendevmeetMeeting started Mon Nov  3 16:02:44 2025 UTC and is due to finish in 60 minutes.  The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot.16:02
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:02
opendevmeetThe meeting name has been set to 'nova'16:02
fwieselo/16:02
dansmitho/16:03
sean-k-mooneyo/16:03
Uggla Hello everyone16:03
gibio/16:03
UgglaFirst meeting with the new timing.16:04
UgglaI hope you are all good after this quite long PTG.16:05
UgglaNot the notes on the wiki agenda are not updated, I lacked time to do that.16:06
UgglaAnyway let's start.16:06
Uggla#topic Bugs (stuck/critical) 16:06
Uggla#info No Critical bug16:06
Uggla#topic Gate status 16:06
Uggla#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:06
Uggla#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:06
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:07
Uggla#info Please look at the gate failures and file a bug report with the gate-failure tag.16:07
Uggla#info Please try to provide a meaningful comment when you recheck16:07
UgglaI just have a quick glance at the gate jobs, I have not seen something frightening.16:07
UgglaBut I could be wrong. 16:08
UgglaDoes someone want to report something ?16:08
gibinope. Gate seems to be OK16:09
gibito me16:09
Ugglaok so moving on16:09
Uggla#topic tempest-with-latest-microversion job status16:09
Ugglagmaan, are you around ? Something to report ?16:09
UgglaI guess gmaan is not available so skiping this topic.16:11
Uggla#topic Release Planning 16:11
Uggla#link https://releases.openstack.org/gazpacho/schedule.html16:11
Uggla^ this is not yet updated with the PTG info, I will do that either later or tomorrow.16:12
Uggla#info PTG etherpad for 2026.1 is available: https://etherpad.opendev.org/p/nova-2026.1-ptg16:13
UgglaHe have just sent the summary, https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/7RD26DHKJV5DBXHXLRZIRFKJNMHZFCUB/16:13
UgglaI hope it is ok and I don't miss too many things.16:13
sean-k-mooneyits on my too read list but i have not got to it yet16:15
sean-k-mooneyif i think of anything ill reply to the email16:15
Ugglasean-k-mooney, sure 16:16
Ugglalet's say that sometime I may have summarized a bit too much. 16:17
UgglaI also use a bit of AI, but not too much. Lazy I am.16:18
sean-k-mooneywell you included the readonly link to the full ptg etherpad16:18
sean-k-mooneyso all the info is aviabel if peopel want to dig deeper16:18
Ugglayep, I was also lazy not to provide the summary on x session that Nova had not run.16:19
UgglaSo Ironic, Neutron, Glance.16:19
UgglaI think we can move on, unless someone wants to add something else.16:20
bauzaso/16:20
Ugglabauzas o/16:20
Uggla#topic OpenAPI 16:21
Uggla#link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned16:21
Uggla#info still 28 remaining atm.16:22
UgglaAs Elod seems not available skipping Stable branch point16:22
UgglaJust worth to mention that Caracal will move to unmaintained.16:24
Uggla#topic vmwareapi 3rd-party CI efforts Highlights16:24
fwieselHi, no updates from my side.16:24
Ugglafwiesel, thx, as about to ask, but you were faster than me.16:25
Uggla#topic Gibi's news about eventlet removal.16:25
Ugglagibi, something you'd like to say.16:25
sean-k-mooneyfwiesel: are there still patches for vmware that are pednign reveiw16:25
gibisure16:25
sean-k-mooneyi fell like we never got around to merging all of them but i coudl be wrgon16:25
gibinova-sch, -api, -metadata is now proposed to be switched to native threading by default as agreed on the PTG. Patch series https://review.opendev.org/c/openstack/nova/+/96592416:27
gibinova-compute can boot VMs now in native threading mode with https://review.opendev.org/c/openstack/nova/+/965467/3 . Libvirt event handling does not work yet so live migration and device attach is failing.16:27
gibiUggla: this is the two news I had16:28
dansmithgibi: nice16:28
Ugglagibi 👍16:28
fwieselsean-k-mooney: To be honest, I am not sure I submitted it to review yet. It was depending on the bug in cinder getting fixed, 16:28
sean-k-mooneyfwiesel: ack16:29
sean-k-mooneyfwiesel: just wanted to make sure you were not blocked on reviwe on the nova side16:29
sean-k-mooneygibi: cool i briefly looked last week but not in any detail16:29
fwieselsean-k-mooney: Thanks16:30
Uggla#topic Open discussion 16:31
UgglaNo agenda topic today.16:32
UgglaDoes someone want to share about something ?16:32
gibi-16:32
bauzas.16:33
sean-k-mooney`16:33
gibi:)16:33
sean-k-mooneyi think that is a no16:33
gibi16:33
sean-k-mooneydo we want to do some bug triage or give everyone back some time16:33
UgglaI think we gonna keep this one short. Especially after last long week.16:34
sean-k-mooney+116:34
UgglaI will restart the rooster stuff for triage, along the week. it should be ready next week.16:35
fwieselThen, thanks everyone16:35
Ugglaanyway, time to close. Thanks for joining this meeting. Have a nice day/evening.16:35
Uggla#endmeeting16:36
opendevmeetMeeting ended Mon Nov  3 16:36:03 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:36
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-03-16.02.html16:36
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-03-16.02.txt16:36
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-03-16.02.log.html16:36
gibithakmks16:36
gibithanks16:36
opendevreviewBalazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet  https://review.opendev.org/c/openstack/nova/+/96594916:51
dansmithmelwitt: so... I redeployed the vTPM series at the host patch this morning18:36
dansmithand I'm able to create an instance in host mode as a user, stop it, and then restart it as the admin18:37
dansmithwhich shows that we've actually improved things before we even get to the deployment mode and migration part of the series18:37
dansmiththat's expected right?18:38
melwittdansmith: yes that is expected18:38
melwittit was a small change in the 'host' support patch to make restart as admin work18:38
dansmithconfirming because I'm not really testing the tpm part, but the instance does have a tpm and it's not complaining when the admin does it18:38
dansmithyeah, cool18:38
dansmithso I'm thinking it would be good, since we have several +2s (some of which are stale) to just go ahead and merge up to the host patch to reduce the size of the series, claim some success and focus on the later bits18:39
dansmithany problem with that? 18:39
dansmithI wish bauzas was here but I think he's already EOD18:39
melwittsounds cool to me18:40
dansmithokay I'll make sure my +2s are good down the line from there and then we can pounce on him tomorrow18:41
melwitt++18:43

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