| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-10 | |||
| 07:15:34 | zigo | As I attempted to say during the PTG: the main issue for us is resize, where placement mandates the *SOURCE* host to have enough space for the resize, even though the VM will go somewhere else. That is a real bug, triggered *not* by an operator mistake. | |
| 09:04:46 | gibi | zigo: definitely an interesting case and sounds like a real bug. I'm not sure if dansmith_pto's fix will cover this, but we need to check via reproducing the issue and the applying the placement fix. | |
| 09:38:28 | sean-k-mooney | zigo: so the thing for resize the only interaction we have with the sourc node is moving the allocation form the instance uuid to the migration uuid | |
| 09:39:13 | gibi | sean-k-mooney: yeah and an at-capacity source node that should still succeed. So something else is in play | |
| 09:39:36 | sean-k-mooney | well that was one of the thigns dan was suggesting might be the issue | |
| 09:39:49 | sean-k-mooney | we are not sure if its <= or == that cause the issue | |
| 09:40:13 | sean-k-mooney | i.e. we are unsure if you get the overcapasity issue if your exactly at capasity | |
| 09:40:44 | sean-k-mooney | so dan was suggesting that it might be an edgcase of the over capsity issue | |
| 09:41:10 | sean-k-mooney | which woudl hopefully mean there fix would fix both | |
| 09:42:35 | sean-k-mooney | i dont know if we have a reproducer for the non oversubsriped resize case | |
| 09:43:11 | sean-k-mooney | but if we do we can perhaps test it with dan's placement change and see if it helps | |
| 12:34:52 | Uggla | To all at 13:00 UTC Cross team neutron/cinder/nova | |
| 12:47:56 | Uggla | MengyangZhang[m], ping I'm in the room, if you want to join me and check for screen sharing. | |
| 13:44:09 | opendevreview | Merged openstack/nova master: Remove tags from README https://review.opendev.org/c/openstack/nova/+/944994 | |
| 14:12:39 | tobias-urdin | Uggla: sorry i totally missed the dates for my topic yesterday as i were on pto, i didn't have anything special to bring up other than checking if somebody was going to work on the open patches that gmann has | |
| 14:12:56 | tobias-urdin | i can take that offline in upcoming weeks, since i missed my time slot | |
| 14:13:21 | Uggla | tobias-urdin, no worries it can happen. | |
| 14:13:37 | Uggla | Are you around today ? | |
| 14:14:07 | Uggla | because if we have a free time slot we could try to address your topic. | |
| 14:14:23 | tobias-urdin | Uggla: yes, i have my neutron topic now but after that im free | |
| 14:15:13 | tobias-urdin | (now = 14:20 UTC) | |
| 14:15:18 | Uggla | ok I'll keep you posted if it would be possible. However it could be on a short notice. | |
| 14:17:31 | tobias-urdin | ack, thanks! not an issue if it cannot be done i will be available for the next hour | |
| 14:21:11 | gmann | tobias-urdin: yes, I will continue working on those and target is to finish those in this cycle. | |
| 14:23:03 | tobias-urdin | gmann: thanks! | |
| 14:23:33 | tobias-urdin | Uggla: then i dont really have anything to bring up on that topic except for some priority on the device_id patch and on gmann's patches when they are completed :) | |
| 14:23:57 | Uggla | ok | |
| 14:47:22 | artom | sean-k-mooney, before I forget, one (I think) important capability is instance resume on host reboot | |
| 14:47:47 | artom | I probably won't be around for the spec review, so I wanted to get that in writing :P | |
| 15:06:53 | gibi_ | artom: added to the etherpad at least :) | |
| 15:07:20 | artom | Oh yeah | |
| 15:07:50 | artom | Oh you did that already, thanks! | |
| 15:11:35 | opendevreview | Andre Aranha proposed openstack/nova master: Replace paramiko with ssh-python https://review.opendev.org/c/openstack/nova/+/946922 | |
| 15:13:13 | Uggla | stephenfin, gtema ping I think we will start the OpenAPI series topic soon. | |
| 15:13:47 | stephenfin | Uggla: Okay. I'm in the Cinder room for the next 30 or so, so if you could have one more topic first that would suit me (cc gtema) | |
| 15:22:14 | Uggla | stephenfin, fyi we move to the power management topic. | |
| 15:22:49 | Uggla | so It should give you enough time to finish your session and join | |
| 15:43:04 | Uggla | stephenfin, we restart with your topic at 15:50 UTC I hope you will be able to join. However gtema is in the call I guess I may speak on your behalf. | |
| 15:50:20 | Uggla | s/I/he/ | |
| 17:15:12 | opendevreview | sean mooney proposed openstack/nova master: Fix creating virtual servers with multiple security groups https://review.opendev.org/c/openstack/nova/+/946079 | |
| 17:15:13 | opendevreview | sean mooney proposed openstack/nova master: add repoducer test for bug 2105896 https://review.opendev.org/c/openstack/nova/+/946945 | |
| 17:17:30 | opendevreview | sean mooney proposed openstack/nova master: Fix creating virtual servers with multiple security groups https://review.opendev.org/c/openstack/nova/+/946079 | |
| 17:18:38 | opendevreview | sean mooney proposed openstack/nova master: Fix creating virtual servers with multiple security groups https://review.opendev.org/c/openstack/nova/+/946079 | |
| 17:38:23 | opendevreview | sean mooney proposed openstack/nova master: [DNM] test if ovn send plug time event when live migrating https://review.opendev.org/c/openstack/nova/+/946950 | |
| 18:04:23 | opendevreview | Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to lock API https://review.opendev.org/c/openstack/nova/+/946223 | |
| 18:35:40 | sean-k-mooney | supamatt: by the way i updated https://review.opendev.org/c/openstack/nova/+/946079 with a repoducer. assuming that passes ci we can likely proceed with that. | |
| 18:36:06 | sean-k-mooney | supamatt: we perhaps should consider adding a release note but i think it would be ok as is too | |
| 18:37:25 | opendevreview | Merged openstack/placement stable/2024.1: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938942 | |
| #openstack-nova - 2025-04-11 | |||
| 02:36:06 | opendevreview | Merged openstack/nova stable/2024.1: ironic: fix logging of validation errors https://review.opendev.org/c/openstack/nova/+/946802 | |
| 02:57:32 | opendevreview | Merged openstack/nova stable/2024.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944009 | |
| 03:09:26 | opendevreview | Merged openstack/nova stable/2024.2: Fix parameter order in add_instance_info_to_node https://review.opendev.org/c/openstack/nova/+/943994 | |
| 03:09:41 | opendevreview | Merged openstack/nova stable/2024.2: Revert "Test live migration between hosts with differnet cpu_shared_sets" https://review.opendev.org/c/openstack/nova/+/932379 | |
| 03:09:47 | opendevreview | Merged openstack/nova stable/2024.2: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/934049 | |
| 03:09:49 | opendevreview | Merged openstack/nova stable/2024.2: Revert "[libvirt] Live migration fails when config_drive_format=iso9660" https://review.opendev.org/c/openstack/nova/+/936645 | |
| 03:09:55 | opendevreview | Merged openstack/nova stable/2024.1: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934303 | |
| 03:10:01 | opendevreview | Merged openstack/nova stable/2024.1: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934304 | |
| 07:30:44 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/946974 | |
| 07:59:09 | opendevreview | Rodolfo Alonso proposed openstack/os-vif master: Adapt unit tests to pyroute2 0.9.1 https://review.opendev.org/c/openstack/os-vif/+/946977 | |
| 08:00:20 | ralonsoh | gibi, sean-k-mooney ^ this is for the next pyroute2 release | |
| 08:18:30 | gibi | ralonsoh: I've left feedback inline | |
| 08:28:16 | ralonsoh | gibi, thankks | |
| 08:30:15 | opendevreview | Merged openstack/placement stable/2024.1: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938943 | |
| 08:30:29 | opendevreview | Rodolfo Alonso proposed openstack/os-vif master: Adapt unit tests to pyroute2 0.9.1 https://review.opendev.org/c/openstack/os-vif/+/946977 | |
| 08:35:00 | gibi | ralonsoh: looks good. thanks | |
| 09:19:11 | opendevreview | Merged openstack/placement stable/2024.1: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938944 | |
| 10:53:20 | sean-k-mooney | ralonsoh: gibi before i merge that i have one reservation | |
| 10:53:52 | sean-k-mooney | when we make this change it will make older version fo pyrout2 not work with os-vif in the unit tests | |
| 10:54:03 | sean-k-mooney | honestly i think thats fine btu just calling that out | |
| 10:57:43 | sean-k-mooney | gibi: are you busy, i have been thinking about maintance mode a liitle since johnthetubaguy | |
| 10:57:53 | sean-k-mooney | put that seed in my brain | |
| 11:03:52 | ralonsoh | sean-k-mooney, this patch works with both versions | |
| 11:04:07 | sean-k-mooney | oh is create jsut ignored | |
| 11:04:14 | sean-k-mooney | on older verisons | |
| 11:05:48 | sean-k-mooney | ah | |
| 11:05:50 | sean-k-mooney | If you pass in create=True, and the attribute doesn’t exist, patch will create the attribute for you when the patched function is called, and delete it again after the patched function has exited. | |
| 11:06:14 | ralonsoh | right | |
| 11:06:40 | sean-k-mooney | i guess the impoarnt part is actuly "This is useful for writing tests against attributes that your production code creates at runtime." | |
| 11:06:47 | sean-k-mooney | which is what happening in this case | |
| 11:07:13 | ralonsoh | the problem, IMO, is how the API of these objects have changed | |
| 11:07:29 | ralonsoh | in a way that should have never happen | |
| 11:07:39 | sean-k-mooney | ya thats unfortunetly out of our contol | |
| 11:07:55 | sean-k-mooney | i approved the fix so it should merge soon thanks for adressing it | |
| 11:08:01 | ralonsoh | cool thanks! | |
| 11:08:07 | sean-k-mooney | since your here | |
| 11:08:26 | sean-k-mooney | ill see if i can find time to rip out the ovn hack properly | |
| 11:08:46 | sean-k-mooney | since it appear not to be needed and i might even backport that to epoxy | |
| 11:08:55 | sean-k-mooney | im hesitnat to bring it back any futher | |
| 11:09:02 | sean-k-mooney | because i dont really know when it was fixed | |
| 11:09:20 | sean-k-mooney | i.e. what version of ovn/neutron | |
| 11:09:38 | ralonsoh | let me check that | |
| 11:09:41 | sean-k-mooney | i appares to not be needed on ubuntu noble with master (which is basicaly epoxy at this point) | |
| 11:11:24 | ralonsoh | the problem is that 2025.1 is still testing in ubuntu 22.04 | |
| 11:11:41 | ralonsoh | and the ovn version provided is 22.03, that doesn't have this support | |
| 11:13:14 | ralonsoh | OVN v23.09.0 - 15 Sep 2023 | |
| 11:13:22 | ralonsoh | requested-chassis option for a remote logical switch port. | |
| 11:13:22 | ralonsoh | - Introduce support for binding remote ports in ovn-northd if the CMS sets | |
| 11:14:02 | sean-k-mooney | ack that what i was afarid of | |
| 11:14:19 | sean-k-mooney | 2025.2 is the first release to be noble+ | |
| 11:14:38 | sean-k-mooney | ok so i can fix this on master but not older branhces | |
| 11:15:20 | sean-k-mooney | ralonsoh: so for the release note ovn 23.09.0 is the first release to have the functionatlity we need | |