| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-10 | |||
| 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: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 removed APIs https://review.opendev.org/c/openstack/nova/+/952282 | |
| 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: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: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:36:47 | Uggla | https://bugs.launchpad.net/nova/+bug/2112808 - nova-compute restart breaks queued live migration objects (artom) | |
| 16:36:47 | Uggla | First: | |
| 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 | Uggla | Is it possible to create a reproducer with fn tests in that case ? | |
| 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:42:08 | sean-k-mooney | looks correct | |
| 16:42:08 | sean-k-mooney | and the logic that its only catherign for runnign mighrations | |
| 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 | ttps://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot | |
| 16:45:14 | Uggla | Next one: | |
| 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 | |