| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-03-26 | |||
| 19:06:45 | sean-k-mooney | not that really talkign about the affect of enabling the viommu on a pci device that is passhth though to the guest | |
| 19:06:59 | sean-k-mooney | be it an ssd or a nic | |
| 19:07:32 | sean-k-mooney | and we can see the overhead is a 60-80% performance loss | |
| 19:07:57 | sean-k-mooney | but i dont know if we will see any performance hit for virual device like virtio-net or virito-blk | |
| 19:08:10 | sean-k-mooney | as such in ci it might not matter | |
| 19:09:15 | sean-k-mooney | that also with kernel 5.12, 6.14 just came out so this has proably impvoed over the last 4 years | |
| 19:10:37 | sean-k-mooney | the qemu they were using to test does not evne meet our current min requirements for nova and we are not exactly agressive with moving that forward | |
| 19:11:49 | sean-k-mooney | i think it would be wroth just trying it locally with some basic test with iperf and fio | |
| 19:12:29 | sean-k-mooney | and if you dont see any horible impact we could talk to the infra folks about trying it on one image in say the vexhost cloud or ovh adn see how it goes | |
| 19:19:45 | dansmith | ack, I should be able to do that.. maybe just tweaking the pool we run nova-next on or something and seeing if it looks to be performing the same for a week would be enough | |
| 19:21:09 | sean-k-mooney | yep | |
| 19:28:45 | dansmith | sean-k-mooney: so, I thought the iommu changes I was making in my first-level guest were to share/expose the actual iommu to the guest, | |
| 19:29:07 | sean-k-mooney | no | |
| 19:29:08 | dansmith | but reading danpb's post I'm thinking what I've been using is actually just the viommu stuff that nova would be enabling | |
| 19:29:17 | sean-k-mooney | yes | |
| 19:29:34 | dansmith | don't mind me, first day on the job people! | |
| 19:29:35 | sean-k-mooney | nova and qemu does not care if the host has an iommu or even if its enabeld | |
| 19:29:43 | dansmith | ack | |
| 19:30:08 | dansmith | so, also from reading danpb's post it seems like the non-passthrough devices should be unchanged with it enabled, but I'll do some testing to make sure | |
| 19:30:37 | sean-k-mooney | ack, that what i was expecting/hoping but i was unsure | |
| 19:31:43 | sean-k-mooney | the iommu can levage some host hardware accalration if its supproted but like vtpm its a purly vritual impelmetion by default | |
| 19:32:42 | sean-k-mooney | if i rememebr correctly we do enable caching by defualt https://libvirt.org/formatdomain.html#iommu-devices | |
| 19:34:57 | sean-k-mooney | ya we do https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7987-L8019 | |
| 19:35:33 | sean-k-mooney | so if the host support intel vt-d it can use it | |
| #openstack-nova - 2025-03-27 | |||
| 08:24:51 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix disable memballoon device https://review.opendev.org/c/openstack/nova/+/945621 | |
| 08:30:24 | opendevreview | Stefan K proposed openstack/nova-specs master: Add Cloud Hypervisor support spec https://review.opendev.org/c/openstack/nova-specs/+/945549 | |
| 10:34:11 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix disable memballoon device https://review.opendev.org/c/openstack/nova/+/945621 | |
| 11:14:53 | opendevreview | Merged openstack/nova stable/2023.2: Reproducer for bug 2098892 https://review.opendev.org/c/openstack/nova/+/942655 | |
| 11:19:04 | opendevreview | Merged openstack/nova stable/2023.2: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942656 | |
| 12:12:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Adjust validation helpers for a single-method future https://review.opendev.org/c/openstack/nova/+/936365 | |
| 12:12:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 12:12:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only check minimum API version https://review.opendev.org/c/openstack/nova/+/940325 | |
| 12:12:55 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 12:12:55 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 12:12:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 12:12:56 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 12:12:57 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 12:12:57 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 12:12:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 12:12:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 12:13:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 12:13:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 12:13:03 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 12:13:03 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remote consoles https://review.opendev.org/c/openstack/nova/+/940114 | |
| 12:13:05 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for keypairs APIs https://review.opendev.org/c/openstack/nova/+/940115 | |
| 12:13:05 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for image metadata APIs https://review.opendev.org/c/openstack/nova/+/940299 | |
| 12:13:07 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server topology API https://review.opendev.org/c/openstack/nova/+/940300 | |
| 12:13:07 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server diagnostics API https://review.opendev.org/c/openstack/nova/+/940506 | |
| 12:13:09 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server external events API https://review.opendev.org/c/openstack/nova/+/940508 | |
| 12:18:40 | opendevreview | Merged openstack/nova master: FUP Remove unnecessary PCI check https://review.opendev.org/c/openstack/nova/+/944105 | |
| 12:24:02 | opendevreview | Merged openstack/nova master: FUP improve comment accuracy and variable naming for tag removal https://review.opendev.org/c/openstack/nova/+/943124 | |
| 12:35:18 | opendevreview | Merged openstack/nova master: FUP Add a warning to make non-explicit live migration request debugging easier https://review.opendev.org/c/openstack/nova/+/944133 | |
| 12:35:24 | opendevreview | Merged openstack/nova master: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/944153 | |
| 12:35:30 | opendevreview | Merged openstack/nova master: FUP: Improve libvirt fixture for hostdevs https://review.opendev.org/c/openstack/nova/+/945232 | |
| 14:19:33 | Vii | Hello. Openstack 2024.2, I shelved the instance (I don't know if it's important on the host that was "compute service set --disable [hostname] nova-compute"). Now I can't unshelve this instance. There's an error in the nova logs: https://paste.openstack.org/show/bshRPNzpjA1ckFkGoa21/ does anyone have any ideas where the error might be? | |
| 14:20:46 | dansmith | Vii: it's timing out talking to neutron | |
| 15:09:22 | opendevreview | ribaudr proposed openstack/nova master: FUP improve and add integration tests for PCI SR-IOV servers https://review.opendev.org/c/openstack/nova/+/944106 | |
| 15:55:57 | opendevreview | Dan Smith proposed openstack/nova master: Extend invalidate_rp to only invalidate cache https://review.opendev.org/c/openstack/nova/+/944148 | |
| 15:55:57 | opendevreview | Dan Smith proposed openstack/nova master: Invalidate PCI-in-placement cached RPs during claim https://review.opendev.org/c/openstack/nova/+/944149 | |
| 15:55:58 | opendevreview | Dan Smith proposed openstack/nova master: Support "one-time-use" PCI devices https://review.opendev.org/c/openstack/nova/+/943816 | |
| 15:55:58 | opendevreview | Dan Smith proposed openstack/nova master: Add one-time-use devices docs and reno https://review.opendev.org/c/openstack/nova/+/944262 | |
| 15:55:59 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Test nova with placement alloc fix https://review.opendev.org/c/openstack/nova/+/945626 | |
| 16:16:34 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add [libvirt]default_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940194 | |
| 16:16:35 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add [libvirt]supported_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940195 | |
| 16:16:35 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 16:16:36 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 16:16:36 | opendevreview | Artom Lifshitz proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501 | |
| 16:16:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: TPM: support booting instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 16:16:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 16:16:38 | opendevreview | Artom Lifshitz proposed openstack/nova master: [WIP] TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 16:16:39 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 16:16:39 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migration for `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 17:31:54 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add [libvirt]default_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940194 | |
| 17:31:55 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add [libvirt]supported_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940195 | |
| 17:31:55 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 17:31:56 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 17:31:56 | opendevreview | Artom Lifshitz proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501 | |
| 17:31:57 | opendevreview | Artom Lifshitz proposed openstack/nova master: TPM: support booting instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 17:31:57 | opendevreview | Artom Lifshitz proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 17:31:58 | opendevreview | Artom Lifshitz proposed openstack/nova master: [WIP] TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 17:31:59 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 17:31:59 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migration for `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 18:38:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server password APIs https://review.opendev.org/c/openstack/nova/+/945736 | |
| 21:47:05 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.2: WIP - api version https://review.opendev.org/c/openstack/nova/+/945777 | |
| 21:49:22 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.2: WIP - api version https://review.opendev.org/c/openstack/nova/+/945777 | |
| 21:51:12 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.2: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945777 | |
| 21:51:49 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2025.1: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945778 | |
| 21:55:18 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2025.1: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945778 | |
| 21:56:56 | opendevreview | Ivan Anfimov proposed openstack/nova master: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945781 | |
| 21:59:50 | opendevreview | Ivan Anfimov proposed openstack/nova master: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945781 | |
| 22:02:28 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.1: WIP https://review.opendev.org/c/openstack/nova/+/945782 | |
| 22:06:09 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.1: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945782 | |
| 22:08:28 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2023.2: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945783 | |
| 22:11:16 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2023.2: docs: fix api version https://review.opendev.org/c/openstack/nova/+/945783 | |
| 22:16:26 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2025.1: doc: fix api version https://review.opendev.org/c/openstack/nova/+/945778 | |
| 22:16:33 | opendevreview | Ivan Anfimov proposed openstack/nova master: doc: fix api version https://review.opendev.org/c/openstack/nova/+/945781 | |
| 22:16:38 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.1: doc: fix api version https://review.opendev.org/c/openstack/nova/+/945782 | |
| 22:16:43 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2023.2: doc: fix api version https://review.opendev.org/c/openstack/nova/+/945783 | |
| 22:16:48 | opendevreview | Ivan Anfimov proposed openstack/nova stable/2024.2: doc: fix api version https://review.opendev.org/c/openstack/nova/+/945777 | |
| #openstack-nova - 2025-03-28 | |||
| 10:57:40 | opendevreview | Stefan K proposed openstack/nova-specs master: Add Cloud Hypervisor support spec https://review.opendev.org/c/openstack/nova-specs/+/945549 | |