| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-10 | |||
| 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 | fwiesel | Hi, no updates from my side | |
| 16:19:15 | Uggla | fwiesel, something on your 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: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 | |