| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-24 | |||
| 19:34:20 | mikal | Also, I have way more random tangents than I will ever have time to persue. | |
| 19:34:28 | sean-k-mooney | ya so ironic were not supper ok with the idea of alpine because they worried about he use of musle libc | |
| 19:34:49 | sean-k-mooney | nova does not really care about that which is why i was prposin git | |
| 19:35:04 | sean-k-mooney | mikal: for tempest we need a small usbset of utils | |
| 19:35:16 | sean-k-mooney | toybox/busybox mostly covers it | |
| 19:35:34 | sean-k-mooney | although i belive we use gnu core utils version in cirros today | |
| 19:36:26 | sean-k-mooney | mikal: i think frickler has been somehwat invovled in cirros mantaince lately but im not reallly sure | |
| 19:37:15 | sean-k-mooney | frickler: do you know if they are still working on a 0.7.x release with 24.04 kernel and package rebase? | |
| 19:37:18 | mikal | sean-k-mooney: they certainly did the most recent release. | |
| 19:39:17 | sean-k-mooney | mikal: i know we are hoping to do a release with https://github.com/cirros-dev/cirros/pull/122 after epoxy is released | |
| 19:39:47 | sean-k-mooney | before would also be fine but as of expoxy nova will be able to create vms with igb nic which supprot emulated sriov in the guest | |
| 19:40:10 | sean-k-mooney | mikal: we are hopping to be able to do some sriov tesing in ci once our providers are capabliy fo supproting that | |
| 19:40:43 | sean-k-mooney | it will be a while before thats realsitc however | |
| 20:31:38 | Uggla | sean-k-mooney, elodilles, https://review.opendev.org/c/openstack/releases/+/941862 +1 again. | |
| 20:36:40 | frickler | sean-k-mooney: I'm not aware of anyone actively working on cirros, but I could do a bit if needed. that PR should be in 0.6.3 though? | |
| 20:39:25 | opendevreview | Luis Enrique Morales Mendez proposed openstack/nova master: Add 'LOG' to archive_deleted_rows function https://review.opendev.org/c/openstack/nova/+/942441 | |
| 21:25:49 | opendevreview | Merged openstack/nova master: Reproducer for bug 2098892 https://review.opendev.org/c/openstack/nova/+/942402 | |
| 21:57:26 | sean-k-mooney | frickler: 0.6.3 is using hte latest (at the time) 22.04 hwe kernel | |
| 21:57:36 | sean-k-mooney | frickler: but not 24.04 | |
| 21:57:56 | sean-k-mooney | frickler: unless you ment the igb change which might be in 0.6.3 | |
| 23:25:55 | opendevreview | Merged openstack/nova master: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942334 | |
| 23:36:24 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: Reproducer for bug 2098892 https://review.opendev.org/c/openstack/nova/+/942651 | |
| 23:36:25 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942652 | |
| 23:37:40 | opendevreview | melanie witt proposed openstack/nova stable/2024.1: Reproducer for bug 2098892 https://review.opendev.org/c/openstack/nova/+/942653 | |
| 23:37:41 | opendevreview | melanie witt proposed openstack/nova stable/2024.1: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942654 | |
| 23:38:49 | opendevreview | melanie witt proposed openstack/nova stable/2023.2: Reproducer for bug 2098892 https://review.opendev.org/c/openstack/nova/+/942655 | |
| 23:38:50 | opendevreview | melanie witt proposed openstack/nova stable/2023.2: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942656 | |
| #openstack-nova - 2025-02-25 | |||
| 00:24:15 | melwitt | fyi elodilles these ^ are fixes for an not-yet-released regression so it would be best to not make releases on these branches until they are merged | |
| 00:35:06 | opendevreview | Merged openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 01:26:14 | opendevreview | Merged openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 03:23:02 | opendevreview | Merged openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 03:24:04 | sean-k-mooney[m] | mikal: ^ congrats | |
| 03:43:21 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/nova/+/942605 | |
| 03:48:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 03:48:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 03:48:46 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 03:48:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 03:50:40 | tkajinam | I know people are busy these days but I really hope I can get feedback about this series ^^^ | |
| 03:50:44 | tkajinam | and https://review.opendev.org/c/openstack/nova/+/845660 | |
| 03:51:53 | tkajinam | https://review.opendev.org/c/openstack/nova/+/940619 is also easy-win to get rid of deprecation warnings | |
| 05:10:08 | sshmlbrg | sean-k-mooney: Are you around? Is nova_context temporary mutation context manager? | |
| 06:29:36 | frickler | sean-k-mooney: I was talking about the igb change that you referenced before, yes. 0.6.3 was done 2 days after it merged, so it should be included. also 22.04 hwe and 24.04 kernels should be kind of synced? | |
| 08:48:14 | mikal | sean-k-mooney: I saw that, but had to cook dinner so the kids didn't resort to cannibalism. Thanks again for all your help. | |
| 08:48:29 | mikal | sean-k-mooney: now to get the client patches merged I suppose. I wonder if I've missed the deadline for that in epoxy... | |
| 09:09:23 | opendevreview | ribaudr proposed openstack/nova master: Add managed flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/937649 | |
| 09:09:24 | opendevreview | ribaudr proposed openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 11:43:48 | sean-k-mooney | mikal: client freeze is gerneally the saem as FF | |
| 11:45:09 | sean-k-mooney | frickler: the 22.04 hwe kernel is basiclly the kernels form the 3 non lts release until the next lts. 24.04 baseline might be similar but 24.04 hwe will be newer | |
| 11:46:23 | sean-k-mooney | frickler: its not critical but i was hoping that it might help with some of the kernel panic we see although that genearly seam to be related to swtiching form the initramfs so it might not actuly be a "kernel" bug so much as an issue with how we are booting | |
| 11:57:38 | sean-k-mooney | frickler: i like cirros and dfisk image builder (i know cirros does not use that) but on think i hav ebeen meening to look into is bootc images i.e. bootable iamge build with podman and https://github.com/osbuild/bootc-image-builder fedora-minimal is 48MB i wonder how much bigger that woudl be if i added the kernel via the bootc build process | |
| 11:58:50 | sean-k-mooney | i got pretty far with my alpine poc a few years ago but never had time to complete it | |
| 11:59:46 | sean-k-mooney | friday is a hack/hussle day technailly at redhat so i might actuly use it to hack on somethign related to cirros or another minimal image | |
| 12:00:13 | sean-k-mooney | that or ill get distracted and work on feature freeze stuff :) | |
| 12:01:32 | sean-k-mooney | the normal fedora bootc image is like 900 MB which defaute the goal fo a small test image because there are plent fo cloud image out there that are smaller or the same size | |
| 12:48:59 | opendevreview | Douglas Viroel proposed openstack/nova master: Add support for showing scheduler_hints in server details https://review.opendev.org/c/openstack/nova/+/938604 | |
| 14:34:08 | opendevreview | ribaudr proposed openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 14:34:08 | opendevreview | ribaudr proposed openstack/nova master: Add managed flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/937649 | |
| 14:54:43 | opendevreview | ribaudr proposed openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 14:54:43 | opendevreview | ribaudr proposed openstack/nova master: Add managed flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/937649 | |
| 15:25:48 | sean-k-mooney | Uggla: can you confirm the tags are proply updated on the comptue node tabel too | |
| 15:25:59 | sean-k-mooney | for https://review.opendev.org/c/openstack/nova/+/937649 | |
| 15:29:13 | sean-k-mooney | Uggla: we do not require it for the manage feature but the tags are includied in the pci_stats column in the ComputeNode table https://github.com/openstack/nova/blob/master/nova/db/main/models.py#L227-L229 | |
| 15:29:29 | sean-k-mooney | we will need that for the live migratable tag | |
| 15:30:01 | sean-k-mooney | i expect the existing logig to proprly update that without you needing to do anything | |
| 15:30:05 | sean-k-mooney | for managed | |
| 15:30:12 | sean-k-mooney | but it woudl be good to confirm | |
| 15:31:03 | gibi | sean-k-mooney: do you mean if the live-migratable flag is removed from the dev_spec then the pci pool containing the matching device should also be updated? | |
| 15:31:08 | Uggla | sean-k-mooney, you mean if this is register in the DB table ? --> https://paste.opendev.org/show/ba4BTb1bDKjCU052R3UP/ | |
| 15:31:28 | gibi | I think that is fairly implict as the pools are memory only and the tag is removed with a nova-compute service restart | |
| 15:31:35 | sean-k-mooney | the schduler does nto use the pci_devices table | |
| 15:31:45 | sean-k-mooney | it uses the pci_stats form the compute node table | |
| 15:31:53 | gibi | ahh | |
| 15:31:57 | gibi | then one more table to look at | |
| 15:32:01 | sean-k-mooney | so the tag for "live migratable" will be stroed in the pci_stats | |
| 15:32:18 | gibi | I assumed pci stats in memory only and not persisted, but then I was wrong | |
| 15:32:21 | sean-k-mooney | gibi: think of pools as placement inventoies | |
| 15:32:21 | gibi | TIL | |
| 15:32:36 | sean-k-mooney | they just have the count and tags instead of traits | |
| 15:32:58 | sean-k-mooney | ya no this is needed for multi cell | |
| 15:33:14 | sean-k-mooney | we cant pass the statc via the scheduler update | |
| 15:33:18 | sean-k-mooney | rpc | |
| 15:33:31 | sean-k-mooney | the scdhluer need to be able to load them form the cell db when creatign the host state object | |
| 15:35:22 | sean-k-mooney | gibi: we dont need to look at managed in the schduer so this is really only relevnet ot the second series but the behvior shoudld be the same. the pools shoudl be in sync with the config | |
| 15:35:45 | gibi | ack | |
| 15:36:30 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/pci/stats.py#L66-L79 | |
| 15:36:41 | sean-k-mooney | mangage may be something we should add to the ignored tags | |
| 15:38:16 | sean-k-mooney | i can leave a comment to that effect on the reviews but i just notice the patches have been updated | |
| 15:38:55 | sean-k-mooney | i have not reviewd the most recent version yet. wehn i saw the paste only had the pci device table in it i was wondering if this had been missed | |
| 15:39:02 | sean-k-mooney | but i have not looked at the code to confirm | |
| 15:39:51 | gibi | sean-k-mooney: I think (weakly) that having extra tags on the device / pool is not an issue as the pool matching logic should be looking at the requested tags from the request side and ignore any extra on the pool side | |
| 15:40:34 | gibi | this is the a response for the ignored tags comment | |
| 15:40:39 | sean-k-mooney | correct and the ovo defines the tags as a dict of nullable strings | |
| 15:40:41 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/pci_device_pool.py#L36 | |
| 15:40:54 | sean-k-mooney | so its not an object change | |
| 15:41:21 | sean-k-mooney | however we should keep the summery in the pools small in general to keep the memory overhead in the schduler and rpc size down | |
| 15:42:08 | sean-k-mooney | i think this could be a follow up patch but we likely shoudl filter out managed since we do not have a usecase for it in the schduler currently since managed is not in the alias | |
| 15:42:08 | gibi | wait do we send the Pool across the wire? you just said above scheduler loads it from the DB | |
| 15:42:39 | sean-k-mooney | i belive we send it across the wire in a few cases. | |
| 15:42:54 | sean-k-mooney | we have the schduler resouce view update perodic | |
| 15:43:18 | sean-k-mooney | where the compute manger updates the schduler with info about the instance on the host ectra | |