| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-17 | |||
| 18:48:55 | gibi | fwiesel: I have a series for oslo.vmware to remove eventlet: https://review.opendev.org/q/project:openstack/oslo.vmware+topic:eventlet-removal but I have no way to test it. The last patch changing the timeout handling of image_transfer can be problematic so that needs extra care | |
| 19:01:55 | sean-k-mooney | gibi: doesnt the ci trigger automaticly on new patchs | |
| 19:02:20 | sean-k-mooney | gibi: given you have now enabeld threadign by default it hsoudl trigger on it no | |
| 19:02:25 | sean-k-mooney | ? | |
| 22:38:27 | tonyb | Are there folks around that can hellp me with some GPU Profile / Resource class questions? | |
| 23:14:35 | opendevreview | melanie witt proposed openstack/nova master: TPM: add RequestContext checks to functional tests https://review.opendev.org/c/openstack/nova/+/966499 | |
| 23:17:04 | opendevreview | melanie witt proposed openstack/nova master: TPM: add RequestContext checks to functional tests https://review.opendev.org/c/openstack/nova/+/966499 | |
| #openstack-nova - 2025-11-18 | |||
| 01:32:18 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 01:32:18 | opendevreview | melanie witt proposed openstack/nova master: TPM: add RequestContext checks to functional tests https://review.opendev.org/c/openstack/nova/+/966499 | |
| 01:32:19 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 01:32:19 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 01:32:20 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 01:32:20 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 01:32:21 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 01:32:22 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 01:32:22 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 01:32:24 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 01:32:24 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 01:32:26 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 01:32:26 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 02:46:03 | melwitt | bauzas: thanks for the reviews :) I have updated the bottom two patches you commented on | |
| 02:54:29 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 03:27:20 | opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | |
| 03:44:47 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 04:23:24 | aravindh8murugesan | :sean-k-mooney I'm passing an nVidia GPU to few of my VMs in passthrough mode. I have noticed for VM with GPU passthrough, openstack nova is not honoring the memory over commitment ratio? Is this expected. Do I need 1:1 memory for passthrough GPU devices? | |
| 04:25:54 | aravindh8murugesan | Maybe I should word it little clearly, nova does honor the overcommitment and scheduled the VM. So for example if my host has 128GB ram and provisioned VMs come out to about 140gb of RAM allocation, trying to boot up all the VMs at once, forces many of these vm to error state. | |
| 07:43:26 | LarsErikP | tobias-urdin: thanks. So, I think I read out of the short discussion that followed, that someone is now thinking about merging it? :P | |
| 08:08:46 | nicolairuckel | sean-k-mooney, I talked to some colleagues who also do operations. They said that it would probably be better if a patch for cold migration would be merged in the same cycle as the other operations that are currently in the NVRAM patch and they were afraid that this won't be possible if those are separate patches. They didn't see any issue with shelving in a separate patch. What do you think? | |
| 08:22:15 | gibi | sean-k-mooney: re vmware ci. No I don't see it triggering on oslo.vmware patches https://review.opendev.org/c/openstack/oslo.vmware/+/967418 | |
| 08:22:39 | gibi | but I can put up nova DNM with depends on to see if that helps | |
| 08:27:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware eventlet removal patches https://review.opendev.org/c/openstack/nova/+/967487 | |
| 08:27:19 | gibi | lets see ^^ | |
| 11:16:06 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix memory_details units in diagnostics API to match documentation https://review.opendev.org/c/openstack/nova/+/967505 | |
| 11:49:48 | opendevreview | sean mooney proposed openstack/nova-specs master: Repropose and update cyborg vGPU (mdev) support https://review.opendev.org/c/openstack/nova-specs/+/967515 | |
| 12:55:15 | nelljerram | Hi all. Could someone take a look at https://review.opendev.org/c/openstack/nova/+/960284? It already has one +2 from Sean but would appreciate a second core review. | |
| 13:02:58 | gibi | nelljerram: have you managed to test this patch with calico succesfully? | |
| 13:03:57 | gibi | nelljerram: the patch looks good to me and it is low risk for us upstream as it only touches a very specific code path. So if it works for calico then I'm fine approving it | |
| 13:05:38 | nelljerram | gibi: Many thanks; I have tested that it works for Calico. | |
| 13:06:58 | gibi | cool. approved | |
| 13:07:03 | gibi | nelljerram: thanks for fixing it | |
| 13:07:19 | nelljerram | gibi: To add a bit more detail, we have a pretty extensive system test - unfortunately closed source at the moment - and it completely fails without this patch, but passes with it. | |
| 13:07:49 | nelljerram | gibi: when tested against Caracal or later, I mean | |
| 13:08:12 | gibi | nelljerram: OK. If you can eventually open up that test env then we would appreciate a 3rd party CI with Calico :) | |
| 13:08:25 | gibi | nelljerram: feel free to ping me with the backports when you propose them | |
| 13:10:27 | nelljerram | gibi: Thanks so much. I would indeed like to open up the testing, and will work towards also connecting that to OpenStack CI. But I expect that will take a little while, so please don't hold your breath for it! | |
| 13:11:28 | nelljerram | gibi: Do I wait until the change is merged to master, before proposing the backports? | |
| 13:13:17 | gibi | I think you can start backporting with `git cherry-pick -x ` It is unlikely that the hash of the commit will change now. But if it does then you just respin the backport to point to the final hash. | |
| 13:13:59 | gibi | nelljerram: no worries I'm not holding breath. :) But I'm happy to see that there is motivated folks trying to open it up | |
| 13:24:09 | sean-k-mooney | gibi: nelljerram the funny thing is we are likely going to end up using managed=no | |
| 13:24:19 | sean-k-mooney | for ovs as well in the future or at least with ovn | |
| 13:24:39 | sean-k-mooney | but before we do that we need to change os-vif to support creating the tap | |
| 13:25:25 | sean-k-mooney | also we chagne form interface=bridge to interface=ethernet precincely because it did not delete the tap | |
| 13:25:32 | sean-k-mooney | so thet libvirt behavior change regressed that again | |
| 13:25:42 | sean-k-mooney | well not quite we didi dit for the ovs prot defintion | |
| 13:25:58 | sean-k-mooney | so im remmeber that wrong but its a similr effect | |
| 13:26:32 | sean-k-mooney | it cause our pre-plugging of the neutron port in pre-livemigrate to be less effective | |
| 13:27:22 | sean-k-mooney | gibi: oh i ment patches to nova | |
| 13:27:29 | sean-k-mooney | gibi: for the vmware changes | |
| 13:27:47 | sean-k-mooney | gibi: do you also need to update oslo.vmware for the eventlet removal | |
| 13:28:19 | sean-k-mooney | libs in general are not ment ot use eventlet directly but i guess they were for some api requests? | |
| 13:32:44 | nelljerram | sean-k-mooney: interesting, thanks. I guess that will also be strong motivation for hooking up some Calico CI. | |
| 13:33:56 | sean-k-mooney | we likely wont be modifying the vif_type=tap code paths but we will be setting the xml atibute form more places | |
| 13:34:24 | sean-k-mooney | we basiclly need neutron to tell us to do it in the vif_details and as i said extened os-vif to be able to create the tap device | |
| 13:34:37 | nelljerram | sean-k-mooney: fwiw I am also currently working on pre-livemigrate for Calico. Currently we ignore that and just flip the port over when migration completes - which is not very seamless :-) | |
| 13:34:40 | sean-k-mooney | im not actully sure how to do that exactly but apprently its a thing we can do | |
| 13:35:02 | sean-k-mooney | nelljerram: ah cool good to know | |
| 13:35:28 | sean-k-mooney | for what its worht we have several jobs that do live migrationin the ci today on nova | |
| 13:35:35 | sean-k-mooney | we can lilkely just add calico to one of them | |
| 13:35:56 | sean-k-mooney | similar to how the nova-alt-config job use ml2/ovs with iptables instead of ovn | |
| 13:36:10 | gibi | sean-k-mooney: oslo_vmware had some home made looping calls, a bit of greenthread.sleep and an eventlet.timeout for image download. So yeah if we want to keep support the vmware virt driver in native threading mode we need changes in the lib | |
| 13:36:18 | sean-k-mooney | we just need to choose one of the jobs, enable the devstack plugin and that will give us some coverage | |
| 13:37:01 | sean-k-mooney | gibi: ack, want might be a strong word for it but if its tested then sure | |
| 13:37:35 | sean-k-mooney | gibi: we dont have visablity into the job defition but i assume depend on does nto work for there third party ci | |
| 13:37:43 | sean-k-mooney | i.e. you cant triger it form nova? | |
| 13:38:24 | nelljerram | thanks, I'll take a look at those. I've found it surprisingly hard to write an effective test for fully seamless live migration. I mean I already have a test that tests continuity of an SSH connection to a guest, across a live migration, and it passes, even though our live plumbing migration is really not properly seamless yet. | |
| 13:39:43 | sean-k-mooney | nelljerram: so we actuly kind of have one | |
| 13:39:58 | sean-k-mooney | nelljerram: there is a neutron test for live migration connectiviy | |
| 13:40:10 | sean-k-mooney | however because our ci provider are quite slow/overloaded | |
| 13:40:24 | sean-k-mooney | we have to set the downtime high in the jobs for stablity | |
| 13:40:38 | gibi | sean-k-mooney: I tried here but it did not triggered (yet) There is also a spec about changing the device attach APIs async. Currently owned by Dan but expected to be taken over by Johannes (outside RH) https://review.opendev.org/c/openstack/nova-specs/+/958900 that I personally care about due to eventlet removal relatedness. If Johannes take it then I would like to get this on the list of | |
| 13:40:44 | gibi | priorities. If Johannes does not take it the I would like to ask if RH can pick it up for G. | |
| 13:40:47 | gibi | ahh sorry | |
| 13:40:50 | gibi | wrong copy paste buffer | |
| 13:40:57 | sean-k-mooney | nelljerram: https://github.com/openstack/tempest/blob/8ccefd4bea48d6b38f7e16c50f97931b3fc2a5e2/tempest/scenario/manager.py#L1062 | |
| 13:41:13 | sean-k-mooney | there are live migration test that use that funciton to verify the conenctivy durign the migration | |
| 13:41:15 | nelljerram | sean-k-mooney: thanks! | |
| 13:41:19 | gibi | sean-k-mooney: so I tried to trigger it from here but no luck yet https://review.opendev.org/c/openstack/nova/+/967487 | |
| 13:41:31 | gibi | I will discuss with fwiesel | |
| 13:41:39 | gibi | when he is arond | |
| 13:42:15 | sean-k-mooney | ack | |
| 13:42:41 | sean-k-mooney | gibi: ya no rush either way i just said i woudl sugget that incase you had not tried it yet | |
| 13:42:56 | gibi | ack | |
| 13:43:12 | sean-k-mooney | if the job is not using zuul or even if it is but is not usign requreed project and devstack it woudl not pull in the patch | |
| 13:44:18 | sean-k-mooney | most job devs dont include libs in required prodject to test with the released verion outside the lib repo so my guess is this wont actully work unless they do some tweaks to test it for us | |
| 13:45:33 | gibi | yeah it is likely wont work | |
| 14:57:19 | opendevreview | Merged openstack/nova master: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/960284 | |
| 15:08:36 | opendevreview | Nell Jerram proposed openstack/nova stable/2025.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967567 | |
| 15:09:30 | opendevreview | Nell Jerram proposed openstack/nova stable/2025.1: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967568 | |
| 15:09:59 | opendevreview | Nell Jerram proposed openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | |
| 15:14:17 | nelljerram | gibi: sean-k-mooney: fyi I've failed the cherry-picks for https://review.opendev.org/c/openstack/nova/+/960284 now; hope I did it right | |