| opendevreview | Steve Baker proposed openstack/nova master: Add VNC console support for the Ironic driver https://review.opendev.org/c/openstack/nova/+/942528 | 01:30 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/nova master: Add VNC console support for the Ironic driver https://review.opendev.org/c/openstack/nova/+/942528 | 01:47 |
| *** mhen_ is now known as mhen | 01:58 | |
| opendevreview | melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | 03:32 |
| 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 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | 03:32 |
| 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 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | 03:32 |
| opendevreview | melanie witt proposed openstack/nova master: Add handling for vTPM secret permission error https://review.opendev.org/c/openstack/nova/+/963648 | 03:32 |
| bauzas | gibi: sean-k-mooney: dansmith: sorry for the late ping, but I'd appreciate reviews for the image props weigher fix https://review.opendev.org/q/topic:%22bug/2125935%22 | 07:20 |
| jlejeune | hello, I'd also appreciate some reviews on my topic https://review.opendev.org/q/topic:%22bug/2085135%22 :) | 07:53 |
| jlejeune | and also on these backports please: https://review.opendev.org/q/topic:%22bug-2044235%22 :) | 07:53 |
| opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | 08:39 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/962776 | 10:03 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: trying a proposal from the parent's review https://review.opendev.org/c/openstack/placement/+/963673 | 10:03 |
| LarsErikP | hi there! I'm so excited that we've no got persistent mdevs for vgpus with mdevctl. But.. with the vendor-specific vfio framework, we're not really using mdevs anymore. Is there any documentation on you can persist the vgpu type of VF over a reboot? | 10:07 |
| sean-k-mooney | gibi: geminis respocne was easer to parse wehen it render the notation in markdown/latex and i asks for graphs :) but i could noe put those in a gerrit comment. | 10:09 |
| sean-k-mooney | gibi: https://imgur.com/a/S6e658j | 10:12 |
| sean-k-mooney | i got that by copying your entire test function with the prompt "why 30?" feels like a good return on the investment | 10:14 |
| sean-k-mooney | modulo bugs the recursive check is basically jsut winning | 10:15 |
| sean-k-mooney | where it looses it a negligable regressions | 10:16 |
| sean-k-mooney | since the prune spacess is so small at 2 and 3 providers | 10:16 |
| opendevreview | Merged openstack/nova master: Fix string format specifier https://review.opendev.org/c/openstack/nova/+/961211 | 10:21 |
| opendevreview | Merged openstack/nova stable/2024.1: Fix 'nova-manage image_property set' command https://review.opendev.org/c/openstack/nova/+/963348 | 10:21 |
| opendevreview | Merged openstack/nova stable/2024.1: Fix case-sensitivity for metadata keys https://review.opendev.org/c/openstack/nova/+/961352 | 10:22 |
| opendevreview | Merged openstack/nova master: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963368 | 10:22 |
| LarsErikP | adding to my question.. There is no way I can tell nova which VGPU type I want to create with this new framework, like we can with mdev, no? | 10:56 |
| gibi | sean-k-mooney: yeah I agree we don't loose much on small spaces | 11:03 |
| sean-k-mooney | LarsErikP: not really you woudl have to use a tool like nvidia https://github.com/NVIDIA/mig-parted or simialr | 11:06 |
| sean-k-mooney | LarsErikP: this is not really in socpe of openstack to automate for the most part | 11:06 |
| sean-k-mooney | LarsErikP: nova in particalr is a static allcoation model where you as the oepratior need to prepare the devices before deploying nova and it just manages allcoting them form teh set you have prepared | 11:07 |
| sean-k-mooney | LarsErikP: i have been asked to look at cyborg a little more closely this cycle to see if it coudl have some sort of role to play there | 11:08 |
| sean-k-mooney | but i have not actully started doing that really but i belive the anse is we woudl have to build that if we wanted it i.e. it cant today but could | 11:09 |
| sean-k-mooney | LarsErikP: my best recomendation today as i said woudl be https://github.com/NVIDIA/mig-parted | 11:09 |
| LarsErikP | sean-k-mooney: okok. thanks. For the first question, that's kind of what I expected =) | 11:32 |
| LarsErikP | and tbh I'm fine with that not being openstacks problem. It would be nice if there was some sort of way to make nova set "current_vgpu_type" though. Much like we could/can with mdevs | 11:33 |
| LarsErikP | In the meantime I guess I'll have to make someting myself. mig-parted seems nice, but we're not really using MIG atm.. hehe | 11:34 |
| jlejeune | gibi: hello, I've answered to your question on https://review.opendev.org/c/openstack/nova/+/933022 | 11:52 |
| sean-k-mooney | LarsErikP: well i think it does non MIG thing too but it was orginally created for mig | 11:55 |
| sean-k-mooney | i have nto looked at it since i forst found it a few years ago | 11:55 |
| sean-k-mooney | but it seam to still have new commit so i assume it has evovled over time | 11:56 |
| LarsErikP | ah. I'll have a closer looke then :P Thanks for letting me know of its existence =) | 11:56 |
| sean-k-mooney | i like the idea of having a simple yaml file that just details this and havign a single agent that just configures it on boot | 11:56 |
| LarsErikP | much like mdevctl ;) | 11:56 |
| LarsErikP | I'm currently writing puppet for dynamically creating a oneshot systemd-unit for this, based on info from hiera | 11:57 |
| LarsErikP | :p | 11:57 |
| sean-k-mooney | kind of but really i feell ike this would be beter as some kidn fo simple systemd service extention | 11:57 |
| sean-k-mooney | or udev rule | 11:57 |
| LarsErikP | absolutley | 11:58 |
| sean-k-mooney | kindof like https://www.redhat.com/en/blog/quadlet-podman | 11:58 |
| LarsErikP | oh. yeah. that would be nice | 11:58 |
| sean-k-mooney | on the one hand i dont like having to ram alot of thing into systemd on the other hand it can be better then installing 50 billion one off tools form diffent venders for this kind of config | 12:00 |
| LarsErikP | mm | 12:00 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test with placement fix https://review.opendev.org/c/openstack/nova/+/963035 | 12:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with optimize_for_wide_provider_trees https://review.opendev.org/c/openstack/nova/+/963036 | 12:06 |
| opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | 13:01 |
| opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | 13:11 |
| opendevreview | Bogdan Dobrelya proposed openstack/nova master: Determine the device type from configuration first https://review.opendev.org/c/openstack/nova/+/963702 | 13:11 |
| opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | 13:26 |
| LarsErikP | sean-k-mooney: hehe https://paste.openstack.org/show/bFie9CvEeQyFLHNsO6Tw/ | 13:57 |
| LarsErikP | works :P | 13:57 |
| LarsErikP | feels.. bodgy. but hey, if it works.. :P | 13:58 |
| sean-k-mooney | LarsErikP: that actully a clearner implemation of tha tthen i have seen other doo | 14:03 |
| sean-k-mooney | you could add it to our docs if you like | 14:03 |
| sean-k-mooney | LarsErikP: we have similar docs like this https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#enable-gpu-types-compute | 14:04 |
| sean-k-mooney | LarsErikP: that is how we document you shoudl enable the vfs in teh first place you caould add a secont for configurign the vgpus type on those vfs | 14:06 |
| sean-k-mooney | the current example is mianly for the mdev based approch | 14:06 |
| LarsErikP | sean-k-mooney: oh, thanks <3 | 14:06 |
| sean-k-mooney | so you woul proably want to add a new section to the docs for the new managed mode passthough | 14:07 |
| LarsErikP | that page should probably contain a section for the vendor-specific vfio type as well | 14:07 |
| LarsErikP | I can look into it on monday. How do i contribute? :P | 14:07 |
| sean-k-mooney | https://docs.openstack.org/nova/latest/admin/pci-passthrough.html#support-for-multiple-types-of-vfs | 14:08 |
| sean-k-mooney | this is the ohter place you may want to coder | 14:08 |
| sean-k-mooney | LarsErikP: well the normal way woudl be to submit a patch via gerrit to update the rst file | 14:08 |
| sean-k-mooney | you coudl also file a bug for the lack fo docs for this and put the suggestins in the launchpad bug | 14:09 |
| sean-k-mooney | LarsErikP: since your more intersted in the new way of doign it with vfio variant dirvers https://docs.openstack.org/nova/latest/admin/pci-passthrough.html#support-for-multiple-types-of-vfs is more correct | 14:09 |
| sean-k-mooney | LarsErikP: for the mdev based approch we do have some docs on how to discove rthe relvent types https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#how-to-discover-a-gpu-type | 14:11 |
| sean-k-mooney | so we could add somethign similar fo the new vfio-variant driver way too if you knwo how | 14:11 |
| *** amorin_ is now known as amorin | 14:15 | |
| LarsErikP | I've learned quite a bit about this the last few days, so I'll try to contribute to the docs next week =) | 14:36 |
| LarsErikP | one way or another ;) | 14:36 |
| LarsErikP | imho, it would be helpful with some kind of "suggestion" about the migration path here as well.. I don't know if that is something that belongs in the docs | 14:38 |
| LarsErikP | our story here is going from Caracal on 22.04 to Epoxy on 24.04. And when you have everything setup for VGPUs with mdev on 22.04, the migration and upgrade is getting quite involved with these changes | 14:40 |
| LarsErikP | We had to leave our gpu-nodes on 22.04 because of these changes as well, since the combination of 24.04/Caracal/GPUs with SR-IOV simply is not supported | 14:41 |
| LarsErikP | So our migration plan would be: Resize all GPU-VMs to non-gpu flavors. Reinstall gpu-nodes and reconfigure everything that's needed. Resize the VMs back to their original (modified) GPU-flavors | 14:43 |
| bauzas | LarsErikP: replying late but fwiw, Epoxy supports 22.04 https://governance.openstack.org/tc/reference/runtimes/2025.1.html | 15:44 |
| sean-k-mooney | LarsErikP: the migration path is unfortunatly to do a resize | 16:14 |
| sean-k-mooney | so ya you kind of need to drain 1 or two hosts and move vms between then via resize | 16:15 |
| sean-k-mooney | instead of modifyign the flavor however i would stongly advice that you just keep the new and old flaovrs and eventually retire the old | 16:16 |
| sean-k-mooney | that gets a littel tricky if you have heat inovled | 16:16 |
| LarsErikP | bauzas: but there is no UCA for epoxy in 22.04? | 17:14 |
| LarsErikP | sean-k-mooney: yeah, but we can live with that in our env =) | 17:18 |
| opendevreview | Merged openstack/nova stable/2025.1: [PCI tracker]Remove non configured devs when freed https://review.opendev.org/c/openstack/nova/+/960242 | 17:54 |
| opendevreview | Merged openstack/nova stable/2025.1: [pci]Keep used dev in Placement regardless of dev_spec https://review.opendev.org/c/openstack/nova/+/960243 | 18:29 |
| opendevreview | Merged openstack/nova master: Add a regression test for ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/963357 | 18:29 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!