Tuesday, 2025-06-10

opendevreviewRajesh Tailor proposed openstack/nova-specs master: Show finish_time field in instance action show  https://review.opendev.org/c/openstack/nova-specs/+/92978007:10
opendevreviewRajesh Tailor proposed openstack/nova master: Add support for showing instance-action finish_time  https://review.opendev.org/c/openstack/nova/+/92893307:47
opendevreviewFriedrich Hiekel proposed openstack/nova master: Fix nova-scheduler placement error  https://review.opendev.org/c/openstack/nova/+/95193607:55
stephenfinsean-k-mooney: any chance you could grab https://review.opendev.org/c/openstack/nova/+/950867 again?09:21
opendevreviewMerged openstack/nova master: Fix small documentation issue  https://review.opendev.org/c/openstack/nova/+/95215109:22
opendevreviewStephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs  https://review.opendev.org/c/openstack/nova/+/95221309:48
opendevreviewStephen Finucane proposed openstack/nova master: api: Address issues with keypairs APIs  https://review.opendev.org/c/openstack/nova/+/95221409:53
stephenfinUggla: Follow-ups for your comments on the openapi series are there, if and when you have time ^10:17
opendevreviewStephen Finucane proposed openstack/nova master: api: Address issues with instance actions API  https://review.opendev.org/c/openstack/nova/+/95194110:51
sean-k-mooneystephenfin: yep added +212:10
sean-k-mooneyby the way im still.... seeing random kernel panics related to cinder volume detach. i know we never fixed that but im wondering if we shoudl consider creating a decorator in tempest to detect that and retry the test. i.e. if the console output form tempest has "end Kernel panic" in it just retry the test once more and only fail if that second try also fails12:14
UgglaHi stephenfin, I will review it thx.12:21
sean-k-mooneystephenfin: replied on https://review.opendev.org/c/openstack/openstacksdk/+/948584 thansk for the review, i might not get back to it for a few days but ill let you knwo when i have a new revision up12:33
stephenfinsean-k-mooney: ack12:36
*** haleyb|out is now known as haleyb12:50
UgglaNova meeting in ~1h.15:03
cardoeI wanted to follow up on https://review.opendev.org/c/openstack/nova-specs/+/471815 and ask what my next steps are to advance that.15:04
opendevreviewMerged openstack/os-vif master: OVS Trunk: Add bridge_name to external_ids  https://review.opendev.org/c/openstack/os-vif/+/94973615:08
opendevreviewStephen Finucane proposed openstack/nova master: db: Resolve alembic deprecation warning  https://review.opendev.org/c/openstack/nova/+/95226615:21
opendevreviewStephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID  https://review.opendev.org/c/openstack/nova/+/95226715:21
opendevreviewStephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides  https://review.opendev.org/c/openstack/nova/+/95226815:21
opendevreviewStephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture  https://review.opendev.org/c/openstack/nova/+/95226915:21
opendevreviewStephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id'  https://review.opendev.org/c/openstack/nova/+/95227015:21
opendevreviewStephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory  https://review.opendev.org/c/openstack/nova/+/95227115:21
opendevreviewStephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware  https://review.opendev.org/c/openstack/nova/+/95227215:21
opendevreviewStephen Finucane proposed openstack/nova master: api-ref: Fix server topology API docs  https://review.opendev.org/c/openstack/nova/+/95227415:41
stephenfinUggla: That last one is for you15:42
Ugglastephenfin, thanks for fixing this.15:50
UgglaNova meeting in ~10mn15:50
*** ChanServ sets mode: +o dansmith15:52
*** maeda was kicked by dansmith (Your behavior is not conducive to the desired environment.)15:53
*** dansmith sets mode: +b *!~maeda@240a:61:1196:9753:b065:99e5:8c7:29a215:56
*** maeda was kicked by dansmith (Your behavior is not conducive to the desired environment.)15:56
bauzastick tock16:00
Uggla#startmeeting nova16:01
opendevmeetMeeting 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
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
fwieselo/16:02
bauzaso/16:02
dansmitho/16:03
*** ChanServ sets mode: -o dansmith16:03
tkajinamo/16:04
UgglaLet's start16:04
Uggla#topic Bugs (stuck/critical) 16:04
Uggla#info No Critical bug 16:04
gmaano/16:05
* gibi lurks from pto16:05
Uggla#info https://review.opendev.org/c/openstack/nova/+/922140 bauzas working on a patch to enable mdev test again.16:05
bauzashad no time to look at it :(16:06
Ugglabauzas no worries. 16:06
UgglaIf you wish I can remove it from this topic.16:07
Ugglagibi, are you crazy. ;)16:07
Uggla#topic Gate status 16:08
Uggla#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:08
Uggla#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:08
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:08
Uggla#info Please look at the gate failures and file a bug report with the gate-failure tag.16:08
Uggla#info Please try to provide meaningful comment when you recheck16:08
UgglaNote: I have not checked periodics this week.16:09
Uggla#topic tempest-with-latest-microversion job status 16:09
Uggla#link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=016:09
Ugglagmaan do you want to add something abut ^16:10
gmaandid not get chance to do any progress on it. 16:10
gmaannothing to share from me16:10
Ugglaok, no pb, moving on.16:10
Uggla#topic Release Planning 16:10
Uggla #link https://releases.openstack.org/flamingo/schedule.html16:11
Uggla#info Nova Spec review day will be next week.16:11
Uggla#info IMHO, we should start reviewing specs.16:11
Uggla#topic Review priorities 16:12
Uggla#link https://etherpad.opendev.org/p/nova-2025.2-status16:12
UgglaI will make sure it is up2date with current reviews for specs.16:12
UgglaBut that should be not too bad. I think almost every spec is part of the document.16:13
Uggla#topic OpenAPI16:14
Uggla#link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned16:14
Uggla#info 18 patches remaining lots of them in the pipe ready to be merged.16:14
UgglaThanks sean-k-mooney and gmann. Let's continue this way.16:15
Ugglagmaan*16:15
sean-k-mooneyi think stephen has ~ anouther 10 or so after the current set16:15
sean-k-mooneybut they are holding them until the current batch are in16:15
sean-k-mooneybut ya we are makeing progress16:16
UgglaYep there are follow up patches.16:16
Ugglabtw, I'm counting only the ones for nova. I know there are many for cinder, etc...16:16
gmaanI did not check the new one but I was planning to finish the series today 16:17
sean-k-mooneyso was i :)16:17
Uggla👍16:17
sean-k-mooneygmaan: ack i proably wont get to them until next week but i may have time to look on friday16:17
gmaanack16:18
UgglaI'm gonna skip stable branch topic because Elod is on pto.16:18
Uggla#topic vmwareapi 3rd-party CI efforts Highlights16:19
Ugglafwiesel, something on your side ?16:19
fwieselHi, no updates from my side16:19
Uggla👍16:19
Uggla#topic Gibi's news about eventlet removal.16:19
gibio/16:19
Uggla#link Blog: https://gibizer.github.io/categories/eventlet/16:19
Uggla#link nova-scheduler series is ready for core review, starting at https://review.opendev.org/c/openstack/nova/+/94796616:20
gibiI'v booked a weekly video call to sync up16:20
gibiEventlet weekly sync Wednesday 14:30 UTC https://meet.google.com/bcy-uqoz-hje16:20
gibiI 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:20
Ugglaok.16:21
gibithat is all from me16:21
sean-k-mooneygibi: i tought you were on pto today as well16:21
Ugglasean-k-mooney, he is. 16:21
gibiI'm :)16:21
sean-k-mooneygibi: ok then go enjoy :)16:22
gibisure. thanks16:22
Ugglayep gibi time to disconnect. :)16:22
Uggla#topic Open discussion16:22
Ugglathere is nothing in the agenda.16:23
* gibi retreats to the shadows..16:23
UgglaSomeone wants to discuss about something ?16:23
opendevreviewMerged openstack/nova master: update pre-commit version pins  https://review.opendev.org/c/openstack/nova/+/94929116:23
opendevreviewStephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec  https://review.opendev.org/c/openstack/nova-specs/+/94044016:23
tkajinamjust a quick question but is anyone working on cleaning up some code after libvirt min version bump ?16:23
Ugglatkajinam yep I have a WIP patch for this.16:24
tkajinamUggla, ah, ok. I'll search it16:24
tkajinamI 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 covered16:25
UgglaI left some TODO's for removal in the code, so this patch will be to remove what is not useful anymore.16:25
tkajinamyeah I saw these comments16:26
sean-k-mooneytkajinam: i should put that back on my review list16:26
sean-k-mooneytkajinam: if i recall it was pretty close before FF16:26
sean-k-mooneytkajinam: is the tpm seriese rebased and passing?16:27
tkajinamsean-k-mooney, are you talking about sev-es series, right /16:27
tkajinam?16:27
tkajinamI guess it has been kept for some time without rebase. I'll rebase it later after the meeting16:27
sean-k-mooneythe detection one. but the sev serise is also intersting16:27
sean-k-mooneyack16:27
tkajinamhm ok. I believe the detection feature was merged in the past but I'll recheck if there is anything remaining16:28
sean-k-mooneyi know we merged the swtpm and version detection16:28
sean-k-mooneyi dont recall if we also merged the patch to auto detect the capsity16:28
sean-k-mooneyalthough maybe we did16:28
tkajinamI believe so but I'll double-check it16:29
opendevreviewStephen Finucane proposed openstack/nova master: api: Address issues with keypairs APIs  https://review.opendev.org/c/openstack/nova/+/95221416:29
sean-k-mooneyack, so your next priority would be the patches to extend that to SEV-ES right16:29
tkajinamthese are not really related but yeah sev-es would be the next one I may want to have.16:29
sean-k-mooneyack16:30
tkajinamthat helps pending sev-snp as well as arm-cca which some folks from fujitsu have been working on16:30
tkajinamI was supposed to submit spec for sev-snp but I'm not very sure if I can push one quite soon16:30
sean-k-mooneyno worries just askign for context16:31
tkajinam:-)16:31
UgglaI guess nobody expects SEV-SNP this cycle.16:32
tkajinamyeah, though I have got a few local pings about my plan about it.16:33
tkajinamanyway 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:33
Ugglayep, something else before moving to bug scrubbing.16:34
tkajinamahh yes16:34
Uggla#topic Bug scrubbing 16:35
Uggla#link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L416:35
Uggla#link: https://truc.uggla.fr/ I kicked off this draft page to follow open bugs.16:35
opendevreviewStephen Finucane proposed openstack/nova master: api: Add response body schemas for server group APIs  https://review.opendev.org/c/openstack/nova/+/95228116:35
opendevreviewStephen Finucane proposed openstack/nova master: api: Add response body schemas for removed APIs  https://review.opendev.org/c/openstack/nova/+/95228216:35
opendevreviewStephen Finucane proposed openstack/nova master: api: Add response body schemas for networks API  https://review.opendev.org/c/openstack/nova/+/95228316:35
opendevreviewStephen Finucane proposed openstack/nova master: api: Add response body schemas for images APIs  https://review.opendev.org/c/openstack/nova/+/95228416:35
UgglaFirst:16:36
Ugglahttps://bugs.launchpad.net/nova/+bug/2112808 - nova-compute restart breaks queued live migration objects (artom)16:36
UgglaI think this is probably valid. sean-k-mooney do you know about it ?16:37
sean-k-mooneyso this came form a 16.2 bug report i.e. train16:38
sean-k-mooneyi 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 master16:39
Ugglayes it seems to be the case. Artom mentioned that it is probably applicable on master.16:39
sean-k-mooneywith that said there probely is a bug in the init_host logic that is ment to heal migrations on start up16:39
sean-k-mooneyso it may be but the bug does nto really meet or minium stnadard for reporting a bug :)16:40
UgglaI know. :)16:40
sean-k-mooneyhttps://issues.redhat.com/browse/OSPRH-13122 is public but you should not have to click into downstream tackers to asses the bugs valididity16:41
sean-k-mooneyanyway the downstream bug suggest the bug is here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1040-L105716:41
UgglaIs it possible to create a reproducer with fn tests in that case ?16:41
sean-k-mooneyand the logic that its only catherign for runnign mighrations16:42
sean-k-mooneylooks correct16:42
sean-k-mooneyso this proably is a bug16:42
sean-k-mooneythat affect master but ya we need a proper repoducer fo this to fix it16:42
sean-k-mooneyand yes our funcitonal tests shoudl be able to repoduce this16:43
sean-k-mooneythere is one slight complciation with how restarts work in the functional enve but we have some exampels of this already16:43
sean-k-mooneylets mark it as triaged and add a permalink to the relevnet code https://github.com/openstack/nova/blob/517415b6cbdd1730b5179b714229cfc0a65b1024/nova/compute/manager.py#L1043-L106016:44
Ugglaok good. That works for me. I'll do that later. Thanks16:44
UgglaNext one:16:45
Ugglattps://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot16:45
Uggla    https://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot16:45
UgglaThat looks valid too. I may encourage the reporter to submit a patch because he proposed something in the report.16:47
sean-k-mooneythat is valid, i fixed that for non BFV instnaces years ago16:48
sean-k-mooneybut i never fixed BFV16:48
sean-k-mooneythe porblem is with the way we do mappings 16:48
sean-k-mooneybasiclly we mapp sata and scsi to sd*16:49
sean-k-mooneyand as a result we end up with the the wrong type or something like that16:49
sean-k-mooneythis is a old know issue16:49
sean-k-mooneywe have a downstream tracker for this somewhere i think16:50
sean-k-mooneyoh16:50
sean-k-mooneyactully this is sligly diffent maybe16:50
sean-k-mooneyhw:scsi_model i think is only an imave property16:50
sean-k-mooneywhat i was descibing was the bug when you have that image peropery on the volume in the volume_image_propties filed16:51
sean-k-mooneyi dont recally if we ever added  hw:scsi_model and hw:disk_bus 16:51
sean-k-mooneyas flavor extra specs16:51
sean-k-mooneythe were orginaly only image properties16:51
sean-k-mooneythat woudl make this a feature request not a bug16:52
sean-k-mooneylet me check the valdiation code16:52
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/api/validation/extra_specs/hw.py16:52
sean-k-mooneyso ya this is not soemthing we supprot today16:53
UgglaWhat do you think about reporter fallback ?16:54
sean-k-mooneywe coudl add the rfe tag and make it a wishlest bug16:54
sean-k-mooneybut it is a new feature and soemthign we intentiaonly did not suprpot in the past16:54
sean-k-mooneyUggla: historically flavor extra specs were for capstyisty and imave proprites were used for things liek device model or busses16:55
sean-k-mooneythe logic is that only the image know what type of virutal device it needs16:55
sean-k-mooneyand 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
sean-k-mooneywith that said we have relaxed that over the years and now allow many thing to be set in either16:56
sean-k-mooneyi 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 tag16:57
Ugglaor we can close the bug and ask reporter to propose something in next PTG ?16:58
sean-k-mooneywe could16:59
sean-k-mooneyi added a comment to the bug16:59
sean-k-mooneydo we want to revisit it next week after they have time to respond17:00
Ugglasure.17:00
Ugglathanks17:00
UgglaWe are on the top of the hour. So closing the meeting.17:01
UgglaThanks all, see you next week.17:01
Uggla#endmeeting17:01
opendevmeetMeeting ended Tue Jun 10 17:01:52 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.html17:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.txt17:01
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.log.html17:01
sean-k-mooneyUggla: that other bug i initally descibe is a real bug with bfv or addtional data volumes by the way i just dont have a link to it17:02
Ugglaok17:02
Ugglasean-k-mooney, I will look at your comment and update the bugs, probably tomorrow morning.17:03
Ugglasean-k-mooney, thanks for your help.17:03
sean-k-mooneyoh its https://bugs.launchpad.net/nova/+bug/202209317:03
sean-k-mooneynova is only ment ot read the prories form the root disk so setting  hw_disk_bus=sata  is a redherring there but17:04
sean-k-mooneyits still incorrect beahvior17:04
sean-k-mooneyall disks shoudl bave been added with sata if the disk type was not cdrom 17:05
Ugglaok I'll read this bug with a fresh mind.17:07
tkajinamsean-k-mooney, fyi. full series of tpm feature detection was merged https://review.opendev.org/q/topic:%22bp/libvirt-detect-vtpm-support%22 and now nova detects (1) tpm support (2) tpm models (3) tpm versions via libvirt API17:07
tkajinam(1) and (2) requires libvirt 8.0.0 so now I can clean up fallback for these, while (3) requires 8.6.0 so slight logic may be kept17:08
* tkajinam runs away17:08
sean-k-mooneytkajinam: ack sound good17:10
opendevreviewTakashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider  https://review.opendev.org/c/openstack/nova/+/92181417:33
opendevreviewTakashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support  https://review.opendev.org/c/openstack/nova/+/92568517:33
opendevreviewTakashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property  https://review.opendev.org/c/openstack/nova/+/92770617:34
opendevreviewTakashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption  https://review.opendev.org/c/openstack/nova/+/92610617:34
*** ChanServ sets mode: +o dansmith17:47
opendevreviewStephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID  https://review.opendev.org/c/openstack/nova/+/95226717:48
opendevreviewStephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides  https://review.opendev.org/c/openstack/nova/+/95226817:48
opendevreviewStephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture  https://review.opendev.org/c/openstack/nova/+/95226917:48
opendevreviewStephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id'  https://review.opendev.org/c/openstack/nova/+/95227017:48
opendevreviewStephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory  https://review.opendev.org/c/openstack/nova/+/95227117:48
opendevreviewStephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware  https://review.opendev.org/c/openstack/nova/+/95227217:48
opendevreviewMerged openstack/nova master: api: Add response body schemas for hosts APIs  https://review.opendev.org/c/openstack/nova/+/93704722:39
opendevreviewMerged openstack/nova master: api: Add response body schemas for instance actions  https://review.opendev.org/c/openstack/nova/+/93704822:47
opendevreviewsean mooney proposed openstack/os-vif master: remove linux bridge plugin  https://review.opendev.org/c/openstack/os-vif/+/94158623:48

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