| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-08 | |||
| 17:38:58 | sean-k-mooney | thats a horbile hack but on the first execution of the filter for a given request spec object you can abuse the fact that python will allow you to assign to filed that dont exist at any time to do the lookup form the buildRequest onece and then pass the same data to all other hosts | |
| 17:40:59 | sean-k-mooney | for everyone else watching done ever do that in a out of tree filters that just asking for pain | |
| 17:41:19 | sean-k-mooney | s/done/don't/ | |
| 18:20:15 | opendevreview | Merged openstack/nova stable/2023.1: Drop Fedora support https://review.opendev.org/c/openstack/nova/+/925940 | |
| 18:32:09 | opendevreview | Merged openstack/nova master: libvirt: Remove node device XML validate flags https://review.opendev.org/c/openstack/nova/+/925826 | |
| 20:38:21 | opendevreview | Merged openstack/nova stable/2024.1: [CI] Replace deprecated regex https://review.opendev.org/c/openstack/nova/+/925522 | |
| 23:45:39 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: DNM: test dependency on devstack-plugin-ceph changes https://review.opendev.org/c/openstack/nova/+/918957 | |
| #openstack-nova - 2024-08-09 | |||
| 00:29:01 | opendevreview | Merged openstack/nova stable/2023.2: [CI] Replace deprecated regex https://review.opendev.org/c/openstack/nova/+/925523 | |
| 04:15:49 | opendevreview | Merged openstack/nova stable/2023.1: [CI] Replace deprecated regex https://review.opendev.org/c/openstack/nova/+/925524 | |
| 05:11:35 | opendevreview | Merged openstack/nova master: libvirt: Ensure swtpm_ioctl is available for vTPM support https://review.opendev.org/c/openstack/nova/+/908546 | |
| 05:11:44 | opendevreview | Merged openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 05:11:52 | opendevreview | Merged openstack/nova master: Fix guard for NVMeOF volumes https://review.opendev.org/c/openstack/nova/+/895192 | |
| 09:29:47 | elodilles | sean-k-mooney: thanks for the +2+W's on stable regex fixing patches o:) | |
| 09:38:17 | sean-k-mooney | i could not sleep so i tought it would be nice to wake up and have those and a few other small patches merged :) | |
| 10:28:00 | opendevreview | Maxime Lubin proposed openstack/nova-specs master: USB over IP https://review.opendev.org/c/openstack/nova-specs/+/920687 | |
| 15:04:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Report available TPM models https://review.opendev.org/c/openstack/nova/+/909183 | |
| 15:07:13 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Report available TPM models https://review.opendev.org/c/openstack/nova/+/909183 | |
| 15:21:11 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 15:21:12 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 16:14:24 | opendevreview | Piotr Czarnik proposed openstack/nova master: Bump websockify version to 0.10.0 due to methods removal in Python3.9 https://review.opendev.org/c/openstack/nova/+/926077 | |
| 18:00:23 | opendevreview | Artom Lifshitz proposed openstack/nova master: Fix s/addtional/additional/ typo https://review.opendev.org/c/openstack/nova/+/926083 | |
| 18:48:44 | opendevreview | Ghanshyam proposed openstack/placement master: DNM: testing latest oslo.policy https://review.opendev.org/c/openstack/placement/+/926088 | |
| 21:27:34 | obrest | Would it be possible for nova to add traits for Intel-AMX for hypervisors where this is available? Similar to how AVX512/AVX2 etc are implemented already? | |
| 21:30:11 | obrest | I guess a patch similar to this is whats needed? https://github.com/openstack/nova/commit/bc784a1c1fa6ae0fa332cfd4dfa35283d3ee2869 | |
| #openstack-nova - 2024-08-10 | |||
| 03:40:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Report available TPM models https://review.opendev.org/c/openstack/nova/+/909183 | |
| 03:44:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 04:21:34 | opendevreview | melanie witt proposed openstack/nova master: DNM testing cont'd https://review.opendev.org/c/openstack/nova/+/925838 | |
| 04:21:34 | opendevreview | melanie witt proposed openstack/nova master: DNM testing https://review.opendev.org/c/openstack/nova/+/925635 | |
| 06:10:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 06:21:11 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 06:21:11 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 13:54:58 | opendevreview | Artom Lifshitz proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| 13:54:58 | opendevreview | Artom Lifshitz proposed openstack/nova master: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/760354 | |
| #openstack-nova - 2024-08-12 | |||
| 03:17:11 | zhhuabj | hi team, can anyone help review this patch? thanks - https://review.opendev.org/c/openstack/nova/+/920374 | |
| 08:31:48 | songwenping_ | hi, team, create vm with specify compute nodes raises the exception:No nodes matched due to not matching 'force_nodes' value, any advice? | |
| 08:34:48 | songwenping_ | migrate vm to this node is success. | |
| 10:56:39 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 10:56:40 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 10:56:40 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 11:02:57 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer test for image property hw_architecture https://review.opendev.org/c/openstack/nova/+/926128 | |
| 13:03:16 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer test for image property hw_architecture https://review.opendev.org/c/openstack/nova/+/926128 | |
| 14:34:06 | opendevreview | Dan Smith proposed openstack/nova master: Deprecate AMI image formats https://review.opendev.org/c/openstack/nova/+/926144 | |
| 14:34:07 | dansmith | bauzas: ^ | |
| 15:06:26 | sean-k-mooney | dansmith: i think bauzas is on pto for a another week | |
| 15:06:33 | sean-k-mooney | maybe too | |
| 15:06:39 | sean-k-mooney | but ill also look at that soon | |
| 15:06:39 | dansmith | yep, realize now | |
| 15:28:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 15:28:59 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 15:28:59 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 16:09:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 16:09:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 16:09:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 23:31:14 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 23:31:15 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 23:31:15 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| #openstack-nova - 2024-08-13 | |||
| 02:39:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 02:39:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 14:23:32 | opendevreview | sean mooney proposed openstack/nova master: Skip new image format tests https://review.opendev.org/c/openstack/nova/+/926215 | |
| 14:26:50 | dansmith | gibi_: can you +W this trivial skip patch to unblock the gate (for us and others) ? ^ | |
| 14:30:29 | sean-k-mooney | im not sure that job runs elsewhere | |
| 14:30:36 | sean-k-mooney | but it might i could check | |
| 14:30:38 | dansmith | it runs in lots of places | |
| 14:30:42 | dansmith | glance for one | |
| 14:30:49 | dansmith | also in the ceph plugin | |
| 14:31:01 | sean-k-mooney | oh ok then yes for others too | |
| 14:31:22 | dansmith | I think cinder's ceph import job might be based on it too, I forget | |
| 14:49:29 | gibi_ | dansmith: +A thanks for the tempest tests | |
| 14:49:36 | dansmith | thanks! | |
| 15:44:36 | opendevreview | Artom Lifshitz proposed openstack/nova master: POC: Work around lbvirt reporting offline cores as being on socket 0 https://review.opendev.org/c/openstack/nova/+/926218 | |
| 16:02:42 | sean-k-mooney | @artom that not what we discussed doing to fix that | |
| 16:03:19 | sean-k-mooney | artom_: we dicussed makeign usre we first online all the cores at startup. then gather the info and then offline them when using the cpu state policy | |
| 16:04:26 | gibi_ | hehe, just commented that to the review:) | |
| 16:04:26 | artom | I don't think I was there during those discussions :) | |
| 16:04:36 | artom | But... the effect is the same, no? | |
| 16:05:00 | gibi_ | artom: nova-compute cannot assume that the pcpus are online after a restart | |
| 16:05:14 | gibi_ | especially as nova-compute offline them | |
| 16:05:14 | artom | Ah, good point. | |
| 16:06:35 | sean-k-mooney | artom: please also decorrage the get capablities with https://docs.python.org/3/library/functools.html#functools.lru_cache | |
| 16:06:52 | sean-k-mooney | the content of get_capabilities shoudl never change while the agent is running | |
| 16:07:02 | sean-k-mooney | we done actully need to call libvirt every time | |
| 16:07:16 | artom | Well that's a different thing | |
| 16:07:27 | artom | An improvement, but not related to fixing the bug | |
| 16:07:29 | sean-k-mooney | artom: we need to make this change to fix the bug | |
| 16:07:33 | sean-k-mooney | no its required | |
| 16:07:40 | artom | Explain yourself :) | |
| 16:07:47 | sean-k-mooney | because depenign on the livbirt verion once we turn off the cores the output will change | |
| 16:08:15 | sean-k-mooney | in some cases it will return the cached values form the first invocation in other version it will update and the info will be wrong | |
| 16:08:28 | artom | Oh, so you're saying memoize the entire return value instead of manually preserving the socket IDs? | |
| 16:08:34 | sean-k-mooney | yep | |
| 16:08:45 | artom | OK... can any other caps change while the agent is running? | |
| 16:08:56 | sean-k-mooney | not without breaking nova :) | |
| 16:09:37 | sean-k-mooney | we do not supprot cpu/memory hotplug or chaning the amount of hugepages while nova-comptue is running or chaning the cpu flags your cpu reprot at runtime | |
| 16:10:08 | sean-k-mooney | which is what this funciton is basicaly providing | |
| 16:10:33 | sean-k-mooney | if you want to do any of the above you shoudl restart nova-compute to pick up the change | |
| 16:10:50 | gibi_ | we basically already memoizing caps as there is a self._caps to save it and an early return | |
| 16:10:57 | sean-k-mooney | you proably shoudl not do any of the above on a host with instnaces either | |
| 16:11:14 | sean-k-mooney | oh true | |
| 16:11:22 | artom | There's a cache size limit, though. What happens when we hit it, and end up actually calling libvirt's caps and potentially getting the wrong socket? | |
| 16:11:53 | sean-k-mooney | artom: that now how that work but gibi is write | |