| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-10 | |||
| 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 | |
| 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 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.html | |
| 17:01:53 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.txt | |
| 17:01:53 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.log.html | |
| 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: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | |
| 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: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: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:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | |
| 17:48:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| 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 | |
| 08:17:06 | opendevreview | Merged openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 08:32:23 | Uggla | sean-k-mooney, if you don't mind I'd like to have a specless BP for https://review.opendev.org/c/openstack/nova/+/952306, it will be easier to track new features included into flamingo. | |
| 09:02:23 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 09:02:24 | opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 09:02:24 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 09:02:25 | opendevreview | Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| 09:13:11 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213 | |
| 09:17:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with server diagnostics APIs https://review.opendev.org/c/openstack/nova/+/952341 | |
| 09:47:36 | 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 | |
| 09:48:17 | tkajinam | Uggla, fyi. I pushed https://review.opendev.org/c/openstack/nova/+/952318 which may replace your WIP one. | |
| 09:48:55 | Uggla | tkajinam, ok I'll have a look. Thanks | |
| 09:49:29 | tkajinam | that does not cover cleanup about tpm detection logic but 952308 does | |
| 11:13:12 | opendevreview | sean mooney proposed openstack/nova master: libvirt: Enable autodeflate and freePageReporting for memballoon https://review.opendev.org/c/openstack/nova/+/952306 | |
| 11:13:54 | sean-k-mooney | Uggla: done https://blueprints.launchpad.net/nova/+spec/automatic-memballoon-freeing | |
| 11:47:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for removed APIs https://review.opendev.org/c/openstack/nova/+/952282 | |
| 11:47:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for networks API https://review.opendev.org/c/openstack/nova/+/952283 | |
| 11:47:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for images APIs https://review.opendev.org/c/openstack/nova/+/952284 | |
| 11:49:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347 | |
| 11:49:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | |
| 11:49:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 11:49:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | |
| 11:50:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 11:51:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 11:52:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 11:52:56 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 12:08:05 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with hypervisors APIs https://review.opendev.org/c/openstack/nova/+/950867 | |