| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-10 | |||
| 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 | |
| 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 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.html | |
| 17:01:52 | opendevmeet | Meeting ended Tue Jun 10 17:01:52 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:01:52 | Uggla | #endmeeting | |
| 17:01:53 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.log.html | |
| 17:01:53 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.txt | |
| 17:02:22 | sean-k-mooney | Uggla: 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 it | |
| 17:02:51 | Uggla | ok | |
| 17:03:17 | Uggla | sean-k-mooney, I will look at your comment and update the bugs, probably tomorrow morning. | |
| 17:03:38 | Uggla | sean-k-mooney, thanks for your help. | |
| 17:03:42 | sean-k-mooney | oh its https://bugs.launchpad.net/nova/+bug/2022093 | |
| 17:04:50 | sean-k-mooney | nova is only ment ot read the prories form the root disk so setting hw_disk_bus=sata is a redherring there but | |
| 17:04:56 | sean-k-mooney | its still incorrect beahvior | |
| 17:05:30 | sean-k-mooney | all disks shoudl bave been added with sata if the disk type was not cdrom | |
| 17:07:45 | Uggla | ok I'll read this bug with a fresh mind. | |
| 17:07:59 | tkajinam | sean-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 API | |
| 17:08:27 | 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 kept | |
| 17:10:21 | sean-k-mooney | tkajinam: ack sound good | |
| 17:33:43 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 17:33:51 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 17:34:00 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 17:34:09 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 17:48:16 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | |
| 17:48:16 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | |
| 17:48:17 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | |
| 17:48:17 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | |
| 17:48:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| 17:48:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | |
| 22:39:23 | opendevreview | Merged openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 22:47:35 | opendevreview | Merged openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 23:48:16 | opendevreview | sean mooney proposed openstack/os-vif master: remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | |
| #openstack-nova - 2025-06-11 | |||
| 00:14:17 | gmaan | stephenfin: I finish the openapi series review except the image one which is WIP. | |
| 00:14:40 | gmaan | *finished | |
| 01:04:43 | opendevreview | sean mooney proposed openstack/nova master: libvirt: Enable autodeflate and freePageReporting for memballoon https://review.opendev.org/c/openstack/nova/+/952306 | |
| 01:08:27 | sean-k-mooney | Uggla: ^ is the small memballoon feature that was requested on the mailing list. let me know if you want me to file a specless blueprint for that or if your happy with just the release note. | |
| 02:35:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 02:41:27 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 03:36:40 | opendevreview | Merged openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 04:01:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 06:02:00 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove logic for unsupported old libvirt/qemu https://review.opendev.org/c/openstack/nova/+/952318 | |