16:01:12 <Uggla> #startmeeting nova
16:01:12 <opendevmeet> Meeting started Tue Jun 10 16:01:12 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:13 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:01:13 <opendevmeet> The meeting name has been set to 'nova'
16:01:25 <Uggla> Hello everyone
16:02:09 <fwiesel> o/
16:02:32 <bauzas> o/
16:03:32 <dansmith> o/
16:04:05 <tkajinam> o/
16:04:38 <Uggla> Let's start
16:04:47 <Uggla> #topic Bugs (stuck/critical)
16:04:56 <Uggla> #info No Critical bug
16:05:06 <gmaan> o/
16:05:35 * gibi lurks from pto
16:05:50 <Uggla> #info https://review.opendev.org/c/openstack/nova/+/922140 bauzas working on a patch to enable mdev test again.
16:06:08 <bauzas> had no time to look at it :(
16:06:42 <Uggla> bauzas no worries.
16:07:06 <Uggla> If you wish I can remove it from this topic.
16:07:50 <Uggla> gibi, are you crazy. ;)
16:08:14 <Uggla> #topic Gate status
16:08:27 <Uggla> #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs
16:08:33 <Uggla> #link https://etherpad.opendev.org/p/nova-ci-failures-minimal
16:08:40 <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 status
16:08:47 <Uggla> #info Please look at the gate failures and file a bug report with the gate-failure tag.
16:08:53 <Uggla> #info Please try to provide meaningful comment when you recheck
16:09:34 <Uggla> Note: I have not checked periodics this week.
16:09:50 <Uggla> #topic tempest-with-latest-microversion job status
16:09:57 <Uggla> #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=0
16:10:17 <Uggla> gmaan do you want to add something abut ^
16:10:18 <gmaan> did not get chance to do any progress on it.
16:10:33 <gmaan> nothing to share from me
16:10:44 <Uggla> ok, no pb, moving on.
16:10:54 <Uggla> #topic Release Planning
16:11:02 <Uggla> #link https://releases.openstack.org/flamingo/schedule.html
16:11:16 <Uggla> #info Nova Spec review day will be next week.
16:11:49 <Uggla> #info IMHO, we should start reviewing specs.
16:12:07 <Uggla> #topic Review priorities
16:12:17 <Uggla> #link https://etherpad.opendev.org/p/nova-2025.2-status
16:12:48 <Uggla> I will make sure it is up2date with current reviews for specs.
16:13:26 <Uggla> But that should be not too bad. I think almost every spec is part of the document.
16:14:34 <Uggla> #topic OpenAPI
16:14:41 <Uggla> #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned
16:14:47 <Uggla> #info 18 patches remaining lots of them in the pipe ready to be merged.
16:15:02 <Uggla> Thanks sean-k-mooney and gmann. Let's continue this way.
16:15:26 <Uggla> gmaan*
16:15:36 <sean-k-mooney> i think stephen has ~ anouther 10 or so after the current set
16:15:52 <sean-k-mooney> but they are holding them until the current batch are in
16:16:01 <sean-k-mooney> but ya we are makeing progress
16:16:05 <Uggla> Yep there are follow up patches.
16:16:47 <Uggla> btw, I'm counting only the ones for nova. I know there are many for cinder, etc...
16:17:09 <gmaan> I did not check the new one but I was planning to finish the series today
16:17:11 <sean-k-mooney> so was i :)
16:17:37 <Uggla> 👍
16:17:49 <sean-k-mooney> gmaan: ack i proably wont get to them until next week but i may have time to look on friday
16:18:15 <gmaan> ack
16:18:53 <Uggla> I'm gonna skip stable branch topic because Elod is on pto.
16:19:06 <Uggla> #topic vmwareapi 3rd-party CI efforts Highlights
16:19:15 <Uggla> fwiesel, something on your side ?
16:19:15 <fwiesel> Hi, no updates from my side
16:19:24 <Uggla> 👍
16:19:48 <Uggla> #topic Gibi's news about eventlet removal.
16:19:50 <gibi> o/
16:19:55 <Uggla> #link Blog: https://gibizer.github.io/categories/eventlet/
16:20:01 <Uggla> #link nova-scheduler series is ready for core review, starting at https://review.opendev.org/c/openstack/nova/+/947966
16:20:10 <gibi> I'v booked a weekly video call to sync up
16:20:18 <gibi> Eventlet weekly sync Wednesday 14:30 UTC https://meet.google.com/bcy-uqoz-hje
16:20:23 <gibi> I might or might not be able to host it tomorrow due to PTO, but if I cannot you can still try to sync there.
16:21:02 <Uggla> ok.
16:21:14 <gibi> that is all from me
16:21:28 <sean-k-mooney> gibi: i tought you were on pto today as well
16:21:50 <Uggla> sean-k-mooney, he is.
16:21:51 <gibi> I'm :)
16:22:00 <sean-k-mooney> gibi: ok then go enjoy :)
16:22:09 <gibi> sure. thanks
16:22:18 <Uggla> yep gibi time to disconnect. :)
16:22:45 <Uggla> #topic Open discussion
16:23:00 <Uggla> there is nothing in the agenda.
16:23:15 * gibi retreats to the shadows..
16:23:24 <Uggla> Someone wants to discuss about something ?
16:23:43 <opendevreview> Merged openstack/nova master: update pre-commit version pins  https://review.opendev.org/c/openstack/nova/+/949291
16:23:54 <opendevreview> Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec  https://review.opendev.org/c/openstack/nova-specs/+/940440
16:23:57 <tkajinam> just a quick question but is anyone working on cleaning up some code after libvirt min version bump ?
16:24:19 <Uggla> tkajinam yep I have a WIP patch for this.
16:24:33 <tkajinam> Uggla, ah, ok. I'll search it
16:25:07 <tkajinam> I have a few items in my local TODO (mostly related to TPM feature detection) so will look into it but I didn't want to duplicate the effort if the core part is covered
16:25:43 <Uggla> I left some TODO's for removal in the code, so this patch will be to remove what is not useful anymore.
16:26:29 <tkajinam> yeah I saw these comments
16:26:41 <sean-k-mooney> tkajinam: i should put that back on my review list
16:26:54 <sean-k-mooney> tkajinam: if i recall it was pretty close before FF
16:27:10 <sean-k-mooney> tkajinam: is the tpm seriese rebased and passing?
16:27:25 <tkajinam> sean-k-mooney, are you talking about sev-es series, right /
16:27:26 <tkajinam> ?
16:27:39 <tkajinam> I guess it has been kept for some time without rebase. I'll rebase it later after the meeting
16:27:44 <sean-k-mooney> the detection one. but the sev serise is also intersting
16:27:56 <sean-k-mooney> ack
16:28:06 <tkajinam> hm ok. I believe the detection feature was merged in the past but I'll recheck if there is anything remaining
16:28:26 <sean-k-mooney> i know we merged the swtpm and version detection
16:28:41 <sean-k-mooney> i dont recall if we also merged the patch to auto detect the capsity
16:28:44 <sean-k-mooney> although maybe we did
16:29:03 <tkajinam> I believe so but I'll double-check it
16:29:27 <opendevreview> Stephen Finucane proposed openstack/nova master: api: Address issues with keypairs APIs  https://review.opendev.org/c/openstack/nova/+/952214
16:29:29 <sean-k-mooney> ack, so your next priority would be the patches to extend that to SEV-ES right
16:29:52 <tkajinam> these are not really related but yeah sev-es would be the next one I may want to have.
16:30:15 <sean-k-mooney> ack
16:30:18 <tkajinam> that helps pending sev-snp as well as arm-cca which some folks from fujitsu have been working on
16:30:38 <tkajinam> I was supposed to submit spec for sev-snp but I'm not very sure if I can push one quite soon
16:31:48 <sean-k-mooney> no worries just askign for context
16:31:54 <tkajinam> :-)
16:32:33 <Uggla> I guess nobody expects SEV-SNP this cycle.
16:33:08 <tkajinam> yeah, though I have got a few local pings about my plan about it.
16:33:48 <tkajinam> anyway we can follow up separately about sev thing and can wrap up the meeting unless anyone else have their topics. (assuming not many people may be interested in such nich feature :-P )
16:34:15 <Uggla> yep, something else before moving to bug scrubbing.
16:34:31 <tkajinam> ahh yes
16:35:17 <Uggla> #topic Bug scrubbing
16:35:26 <Uggla> #link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L4
16:35:37 <Uggla> #link: https://truc.uggla.fr/ I kicked off this draft page to follow open bugs.
16:35:45 <opendevreview> Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server group APIs  https://review.opendev.org/c/openstack/nova/+/952281
16:35:45 <opendevreview> Stephen Finucane proposed openstack/nova master: api: Add response body schemas for removed APIs  https://review.opendev.org/c/openstack/nova/+/952282
16:35:46 <opendevreview> Stephen Finucane proposed openstack/nova master: api: Add response body schemas for networks API  https://review.opendev.org/c/openstack/nova/+/952283
16:35:46 <opendevreview> Stephen Finucane proposed openstack/nova master: api: Add response body schemas for images APIs  https://review.opendev.org/c/openstack/nova/+/952284
16:36:47 <Uggla> First:
16:36:47 <Uggla> https://bugs.launchpad.net/nova/+bug/2112808 - nova-compute restart breaks queued live migration objects (artom)
16:37:50 <Uggla> I think this is probably valid. sean-k-mooney do you know about it ?
16:38:54 <sean-k-mooney> so this came form a 16.2 bug report i.e. train
16:39:24 <sean-k-mooney> i know we did some work to improve this since then so before considering it valid we likely need to see if it can happen in master
16:39:48 <Uggla> yes it seems to be the case. Artom mentioned that it is probably applicable on master.
16:39:53 <sean-k-mooney> with that said there probely is a bug in the init_host logic that is ment to heal migrations on start up
16:40:25 <sean-k-mooney> so it may be but the bug does nto really meet or minium stnadard for reporting a bug :)
16:40:55 <Uggla> I know. :)
16:41:00 <sean-k-mooney> https://issues.redhat.com/browse/OSPRH-13122 is public but you should not have to click into downstream tackers to asses the bugs valididity
16:41:42 <sean-k-mooney> anyway the downstream bug suggest the bug is here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1040-L1057
16:41:42 <Uggla> Is it possible to create a reproducer with fn tests in that case ?
16:42:08 <sean-k-mooney> and the logic that its only catherign for runnign mighrations
16:42:08 <sean-k-mooney> looks correct
16:42:22 <sean-k-mooney> so this proably is a bug
16:42:38 <sean-k-mooney> that affect master but ya we need a proper repoducer fo this to fix it
16:43:06 <sean-k-mooney> and yes our funcitonal tests shoudl be able to repoduce this
16:43:35 <sean-k-mooney> there is one slight complciation with how restarts work in the functional enve but we have some exampels of this already
16:44:13 <sean-k-mooney> lets mark it as triaged and add a permalink to the relevnet code https://github.com/openstack/nova/blob/517415b6cbdd1730b5179b714229cfc0a65b1024/nova/compute/manager.py#L1043-L1060
16:44:49 <Uggla> ok good. That works for me. I'll do that later. Thanks
16:45:14 <Uggla> Next one:
16:45:14 <Uggla> ttps://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot
16:45:27 <Uggla> https://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot
16:47:24 <Uggla> That looks valid too. I may encourage the reporter to submit a patch because he proposed something in the report.
16:48:32 <sean-k-mooney> that is valid, i fixed that for non BFV instnaces years ago
16:48:44 <sean-k-mooney> but i never fixed BFV
16:48:55 <sean-k-mooney> the porblem is with the way we do mappings
16:49:08 <sean-k-mooney> basiclly we mapp sata and scsi to sd*
16:49:24 <sean-k-mooney> and as a result we end up with the the wrong type or something like that
16:49:58 <sean-k-mooney> this is a old know issue
16:50:12 <sean-k-mooney> we have a downstream tracker for this somewhere i think
16:50:27 <sean-k-mooney> oh
16:50:35 <sean-k-mooney> actully this is sligly diffent maybe
16:50:48 <sean-k-mooney> hw:scsi_model i think is only an imave property
16:51:16 <sean-k-mooney> what i was descibing was the bug when you have that image peropery on the volume in the volume_image_propties filed
16:51:34 <sean-k-mooney> i dont recally if we ever added  hw:scsi_model and hw:disk_bus
16:51:43 <sean-k-mooney> as flavor extra specs
16:51:54 <sean-k-mooney> the were orginaly only image properties
16:52:09 <sean-k-mooney> that woudl make this a feature request not a bug
16:52:29 <sean-k-mooney> let me check the valdiation code
16:52:57 <sean-k-mooney> https://github.com/openstack/nova/blob/master/nova/api/validation/extra_specs/hw.py
16:53:04 <sean-k-mooney> so ya this is not soemthing we supprot today
16:54:18 <Uggla> What do you think about reporter fallback ?
16:54:25 <sean-k-mooney> we coudl add the rfe tag and make it a wishlest bug
16:54:40 <sean-k-mooney> but it is a new feature and soemthign we intentiaonly did not suprpot in the past
16:55:12 <sean-k-mooney> Uggla: historically flavor extra specs were for capstyisty and imave proprites were used for things liek device model or busses
16:55:29 <sean-k-mooney> the logic is that only the image know what type of virutal device it needs
16:56:11 <sean-k-mooney> and since some clodus do not show flavor extra specs to user allowing it to be set in the flavor coudl break enduses expcations.
16:56:33 <sean-k-mooney> with that said we have relaxed that over the years and now allow many thing to be set in either
16:57:44 <sean-k-mooney> i can write somethign up but really this shoudl be a small specless blueprint or as i said we coudl consider it a wishlist bug with the rfe tag
16:58:34 <Uggla> or we can close the bug and ask reporter to propose something in next PTG ?
16:59:46 <sean-k-mooney> we could
16:59:54 <sean-k-mooney> i added a comment to the bug
17:00:05 <sean-k-mooney> do we want to revisit it next week after they have time to respond
17:00:26 <Uggla> sure.
17:00:34 <Uggla> thanks
17:01:22 <Uggla> We are on the top of the hour. So closing the meeting.
17:01:51 <Uggla> Thanks all, see you next week.
17:01:52 <Uggla> #endmeeting