| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-05-22 | |||
| 13:54:55 | sean-k-mooney | i.e. if amd wanted to supprot live-migration for SEV-SNP i doubt they coudl do that without first closing the general sev gap | |
| 13:55:14 | zigo | Ok, thanks for the details. | |
| 13:56:39 | sean-k-mooney | zigo: on a related not live migration is not currently supproted with vTPM either that is more upsetting as i expect it will be more widely used becasue of microsfot requiring it for windows server | |
| 13:57:34 | sean-k-mooney | as far as we can tell qemu currently only supprot live migration if you put the vtpm dir on nfs or another shared file system where they do not require the tpm data ot be copied... | |
| 13:58:57 | zigo | sean-k-mooney: But the vTPM files are somewhere in /etc/libvirt, no? | |
| 13:59:34 | sean-k-mooney | by default i think so yes | |
| 13:59:44 | sean-k-mooney | i woudl have to check the code | |
| 13:59:59 | sean-k-mooney | but they are not in /var/lib/nova/** as far as i recal | |
| 14:19:55 | tkajinam | sean-k-mooney, yeah I guess so, because SEV-SNP uses SEV as its core with additional protection features > lm support for SEV-SNP would depend on one for SEV | |
| 17:59:55 | opendevreview | sean mooney proposed openstack/nova master: [WIP] retry write_sys call on device busy https://review.opendev.org/c/openstack/nova/+/920203 | |
| 18:34:56 | opendevreview | Merged openstack/nova-specs master: Remove template files from non-empty directories https://review.opendev.org/c/openstack/nova-specs/+/919597 | |
| 19:03:34 | opendevreview | Merged openstack/nova master: Stop using split UEC image (mostly) https://review.opendev.org/c/openstack/nova/+/919739 | |
| 19:29:35 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose specs for ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/907654 | |
| #openstack-nova - 2024-05-23 | |||
| 07:28:14 | bauzas | good morning | |
| 15:35:55 | opendevreview | Merged openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant https://review.opendev.org/c/openstack/nova/+/896512 | |
| 15:36:44 | sahid | o/ thank you dansmith, bauzas for your time on it :-) | |
| 16:46:12 | melwitt | bauzas: after your questions about unified limits I proposed to try to improve the docs about it if you might be interested https://review.opendev.org/c/openstack/nova/+/919913 | |
| 16:48:41 | bauzas | melwitt: cool thanks | |
| 16:49:15 | bauzas | fwiw, I got some people interested in unified limits testing yesterday on the OpenInfra Day France | |
| 16:52:34 | bauzas | +W | |
| 16:53:10 | bauzas | melwitt: I also +2d your persistent mdev patch just asking you to create a FUP | |
| 16:53:21 | bauzas | for the docs nits and some points I added | |
| 16:54:11 | melwitt | bauzas: that's awesome! hope the presentation went well | |
| 16:54:23 | melwitt | bauzas: yes I just saw, I'm writing a couple of replies and will propose a FUP for the docs stuff | |
| 16:54:26 | bauzas | yup | |
| 16:54:27 | melwitt | thanks :) | |
| 16:54:31 | bauzas | it went very weel | |
| 16:54:46 | bauzas | again the only problem is with the nvidia licensing | |
| 16:55:30 | melwitt | glad to hear (that it went very well) | |
| 16:55:34 | bauzas | your own patch will be very well appreciated :) | |
| 16:55:41 | bauzas | I explained about it | |
| 16:57:18 | bauzas | but this was a weird session, as I spoke French for it :) | |
| 16:57:44 | bauzas | so even if this should have been recorded, you may not be able to understand it :D | |
| 16:58:26 | melwitt | thanks for testing the docs :) helped a lot to know what to add | |
| 16:59:01 | melwitt | heh | |
| 18:40:11 | opendevreview | melanie witt proposed openstack/nova master: docs: Follow up for persistent mdevs https://review.opendev.org/c/openstack/nova/+/920311 | |
| 19:48:44 | opendevreview | Merged openstack/nova master: docs: Add more information about unified limits https://review.opendev.org/c/openstack/nova/+/919913 | |
| 20:41:36 | wncslln | hi nova o/, about this ClientWrapper on neutron client https://github.com/openstack/nova/blob/master/nova/network/neutron.py#L174. what its function? is just to caught the neutron client exceptions and convert to nova exceptions? | |
| 20:42:06 | wncslln | To give some context, I'm solving this bug https://bugs.launchpad.net/nova/+bug/1994139 :) | |
| #openstack-nova - 2024-05-24 | |||
| 08:00:28 | opendevreview | Zhang Hua proposed openstack/nova master: BlockDeviceMapping object has no attribute copy https://review.opendev.org/c/openstack/nova/+/920374 | |
| 11:22:39 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/909611 | |
| #openstack-nova - 2024-05-27 | |||
| 08:43:59 | bauzas | TIL a new query parameter for etherpad :) 'userColor' :) | |
| 08:44:33 | bauzas | after 10 years, I'm eventually able to use the exact same color that the one I was using previous :) | |
| 08:45:08 | bauzas | welcome f8f8d3 :) | |
| 10:05:12 | gibi | :) | |
| 11:01:27 | opendevreview | Merged openstack/nova master: libvirt: Create persistent mdevs https://review.opendev.org/c/openstack/nova/+/910041 | |
| 11:01:34 | opendevreview | Merged openstack/nova master: docs: Follow up for persistent mdevs https://review.opendev.org/c/openstack/nova/+/920311 | |
| 12:24:13 | opendevreview | sean mooney proposed openstack/nova master: retry write_sys call on device busy https://review.opendev.org/c/openstack/nova/+/920203 | |
| 12:24:37 | sean-k-mooney | gibi: ^ that should be more correct and it now has test coverage | |
| 12:28:49 | opendevreview | sean mooney proposed openstack/nova master: retry write_sys call on device busy https://review.opendev.org/c/openstack/nova/+/920203 | |
| 12:29:21 | sean-k-mooney | slightly better log message | |
| 12:29:48 | sean-k-mooney | bauzas: artom: ^ when ye have time i added ye as reviewers also | |
| 12:31:12 | gibi | sean-k-mooney: I will take a look | |
| 12:32:33 | sean-k-mooney | thanks, im going to try stablising the repoducer for the other bug im working on then i can move my attention back to the eventlet or healthcheck series | |
| 13:05:21 | opendevreview | sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322 | |
| 13:05:21 | opendevreview | sean mooney proposed openstack/nova master: add functional repoducer for bug 2048837 https://review.opendev.org/c/openstack/nova/+/919979 | |
| 13:05:45 | sean-k-mooney | ok ^ those should also be reviewable/mergable | |
| 13:12:32 | bauzas | sean-k-mooney: ack, will take a look | |
| 13:12:54 | sean-k-mooney | i added them to https://etherpad.opendev.org/p/nova-dalmatian-status as well to keep track of them | |
| 13:13:39 | sean-k-mooney | before i move back to feature work im going to rebase my ci fixes and stack them so they dont conflict with each other | |
| 13:19:15 | opendevreview | sean mooney proposed openstack/nova master: tweak emulation job to avoid OOM errors https://review.opendev.org/c/openstack/nova/+/918464 | |
| 13:20:58 | opendevreview | sean mooney proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 13:20:59 | opendevreview | sean mooney proposed openstack/nova master: enable numa live migration in the ceph job https://review.opendev.org/c/openstack/nova/+/913842 | |
| 13:40:16 | opendevreview | sean mooney proposed openstack/nova master: enable zswap in nova ci jobs. https://review.opendev.org/c/openstack/nova/+/905791 | |
| 13:43:38 | opendevreview | sean mooney proposed openstack/nova master: enable zswap in nova ci jobs. https://review.opendev.org/c/openstack/nova/+/905791 | |
| 15:21:47 | sean-k-mooney | bauzas: you can baseically see the ci change in the last 5-6 pataches https://github.com/openstack/nova/commits/master/.zuul.yaml | |
| 15:21:57 | bauzas | thanks | |
| 15:54:10 | gibi | sean-k-mooney: I looked at the 1 second sleep in https://review.opendev.org/c/openstack/nova/+/919979 and I have no idea why the test fails without it. So far I was not able to reproduce such failure locally. As this sleep goes away after the fix I'm fine to temporary merge it | |
| 15:55:39 | sean-k-mooney | ack ya i spent like 6 hours gettign that repoducer to work 2 weeks ago and when it finally passed locally it was flaky in the gate. adding the time.sleep makes it stable for me locally and in the gate so its the best i coudl come up with | |
| 17:31:56 | opendevreview | sean mooney proposed openstack/nova master: retry write_sys call on device busy https://review.opendev.org/c/openstack/nova/+/920203 | |
| #openstack-nova - 2024-05-28 | |||
| 02:47:38 | opendevreview | Kien Nguyen proposed openstack/python-novaclient master: support get and list server's metadata https://review.opendev.org/c/openstack/python-novaclient/+/915977 | |
| 08:38:15 | bauzas | tkajinam: I'm doing some spec paperwork for reviews but I don't see any blueprint created for https://specs.openstack.org/openstack/nova-specs/specs/2024.2/approved/libvirt-stateless-firmware.html | |
| 08:38:25 | bauzas | tkajinam: have you created one or no ? | |
| 08:48:53 | tkajinam | bauzas, there is the one https://blueprints.launchpad.net/nova/+spec/libvirt-stateless-firmware | |
| 08:49:01 | bauzas | excellent | |
| 08:49:07 | bauzas | so we only need to update the spec | |
| 08:49:43 | tkajinam | Ugh/. I thought I added the link but didn't. let me push a change so that you can +2 it | |
| 08:50:43 | bauzas | cool, I'll even +W it | |
| 08:52:54 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Fix missing blueprint link https://review.opendev.org/c/openstack/nova-specs/+/920612 | |
| 08:54:57 | bauzas | +Wd | |
| 08:59:13 | tkajinam | thanks ! | |
| 09:07:15 | opendevreview | Merged openstack/nova master: libvirt: Ensure both swtpm and swtpm_setup exist for vTPM support https://review.opendev.org/c/openstack/nova/+/908541 | |
| 09:11:05 | opendevreview | Merged openstack/nova-specs master: Fix missing blueprint link https://review.opendev.org/c/openstack/nova-specs/+/920612 | |
| 09:11:55 | zigo | What's the directive that tells nova-compute to use Ceph as a backend for the system disk of instances? | |
| 09:12:23 | zigo | I see in the logs that nova-compute is reporting the local storage space instead of what's in Ceph, so I'm wondering if it's set properly ... | |
| 10:01:55 | frickler | zigo: "[libvirt] images_type = rbd" iirc | |
| 10:21:09 | sean-k-mooney | yep just images_type=rbd | |
| 10:21:28 | sean-k-mooney | with that config every comptue node will report the total ceph pool stoage as its local storage instead | |
| 11:19:11 | wncslln | morning nova o/, I'm solving this bug https://bugs.launchpad.net/nova/+bug/1994139 | |
| 11:19:19 | wncslln | I've changed the neutron client to openstacksdk client and a bunch of tests are broken with the message: "keystoneauth1.exceptions.auth_plugins.MissingAuthPlugin: An auth plugin is required to determine endpoint URL" | |
| 11:19:44 | wncslln | I need to change the way that the auth plugin is loaded too? | |
| 11:20:35 | sean-k-mooney | yep that is part of changing to the client | |
| 11:20:48 | sean-k-mooney | *form the client to the sdk | |
| 11:21:22 | sean-k-mooney | we already converged form ironic client ot neutron client last cycle | |
| 11:21:35 | JayF | ironic client -> openstacksdk | |
| 11:21:37 | sean-k-mooney | the likely needed to do the same converation ill see if i can find any patches related to it | |
| 11:21:39 | JayF | don't put neutron on me ;) | |
| 11:21:55 | sean-k-mooney | sorry yes ironic to sdk | |
| 11:22:01 | JayF | There are examples of openstacksdk to an external service in Ironic, but I don't think we preserve user context -- you all may need that for Neutron (?) | |
| 11:22:16 | sean-k-mooney | yes in general | |