| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-05 | |||
| 11:16:08 | gibi | ohh wait. I only found your version somehow not bauzas' | |
| 11:16:25 | gibi | so I will look at bauzas' too | |
| 11:16:29 | sean-k-mooney | i added you to mine :) | |
| 11:16:44 | sean-k-mooney | its similar without the test update and no fall back | |
| 11:23:20 | gibi | now I added my feedback to both :) | |
| 11:29:13 | sean-k-mooney | gibi: thanks just read them and i geneally agree with them | |
| 11:30:03 | sean-k-mooney | ill make those changes shortly and ill also update teh api ref to not that this is a driver depenent api respocne and that its cached, in addtion ot the mention in the release note. | |
| 11:30:46 | gibi | cool thanks. Please sean-k-mooney bauzas agree on which patch is the "official" one :) | |
| 11:30:47 | sean-k-mooney | gibi: what did you think of the release note and commit message in general. those were the only thing i got ai to do | |
| 11:31:04 | gibi | I had no problem with those indeed | |
| 11:36:53 | jkulik | as said yesterday, I'd like to get our server-group listing performance improvement patches upstream if possible. I opened https://bugs.launchpad.net/nova/+bug/2122109 for that and already pushed the first one: https://review.opendev.org/c/openstack/nova/+/959696 - feedback appreciated | |
| 11:37:07 | gibi | bauzas: if you have a chance then you can take a look at https://review.opendev.org/q/topic:%22bug/2115905%22 . I respun it to fix sean-k-mooney's comments and sean-k-mooney is +2 on it | |
| 12:17:27 | bauzas | I have another thing to do now | |
| 12:17:57 | bauzas | honestly I don't care, I created my patch first, but if you prefer Sean's one, meh | |
| 12:18:14 | bauzas | I just want it for RC1 | |
| 12:18:47 | gibi | bauzas: I did not expressed preference :) | |
| 12:20:26 | opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/os-vif/+/959741 | |
| 12:20:27 | opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/os-vif/+/959742 | |
| 12:20:29 | opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/os-vif/+/959743 | |
| 12:20:35 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/osc-placement/+/959744 | |
| 12:20:36 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/osc-placement/+/959745 | |
| 12:20:38 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/osc-placement/+/959746 | |
| 12:20:44 | gibi | my only preference is not having two competing patches at the end | |
| 12:20:46 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/python-novaclient/+/959747 | |
| 12:20:47 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/python-novaclient/+/959748 | |
| 12:20:48 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/python-novaclient/+/959749 | |
| 13:05:41 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 13:06:06 | opendevreview | Merged openstack/nova stable/2025.1: Reproduce that only half of the PCI devs are removed https://review.opendev.org/c/openstack/nova/+/959308 | |
| 13:10:20 | opendevreview | Merged openstack/os-vif stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/os-vif/+/959741 | |
| 13:14:56 | opendevreview | Merged openstack/python-novaclient stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/python-novaclient/+/959747 | |
| 13:14:58 | opendevreview | Merged openstack/python-novaclient stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/python-novaclient/+/959748 | |
| 13:55:56 | opendevreview | Merged openstack/os-vif master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/os-vif/+/959743 | |
| 14:02:42 | tkajinam | o/ there is a bug with SEV encrypted instances recently reported. I've proposed a fix for it which was validated by a reporter. Can we merge it before rc (this may not conflict with the feature freeze policy). I'm ok to merge it after rc and then backport it later, though | |
| 14:02:44 | tkajinam | https://review.opendev.org/c/openstack/nova/+/958868 | |
| 14:04:04 | opendevreview | Merged openstack/os-vif stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/os-vif/+/959742 | |
| 14:06:23 | opendevreview | Takashi Kajinami proposed openstack/os-vif stable/2025.2: Remove remaining job with Ubuntu Jammy (22.04) https://review.opendev.org/c/openstack/os-vif/+/959855 | |
| 14:15:29 | gibi | tkajinam: thanks! it is approved now | |
| 14:15:43 | tkajinam | gibi, thx ! | |
| 14:40:48 | opendevreview | Merged openstack/nova stable/2025.1: Fix pci_tracker.save to delete all removed devs https://review.opendev.org/c/openstack/nova/+/959309 | |
| 14:40:59 | opendevreview | Merged openstack/nova master: Reproduce bug/2115905 https://review.opendev.org/c/openstack/nova/+/954336 | |
| 14:47:52 | opendevreview | Sylvain Bauza proposed openstack/nova master: Change uptime in hypervisor to be provided async using stats https://review.opendev.org/c/openstack/nova/+/959571 | |
| 14:49:07 | bauzas | sean-k-mooney: gibi: I offer to use my patch as the fix (as mine was a bit older than sean's one and given I have capacity for that), but I added Sean as co-author to reflect his effort too | |
| 14:49:40 | bauzas | sean-k-mooney: if you prefer having yours as main, fine, tell me and I'll abandon mine | |
| 14:49:55 | bauzas | as I said, my main concern is to make sure we have that by RC1 | |
| 14:50:26 | sean-k-mooney | well i do | |
| 14:50:37 | sean-k-mooney | becuase i said i would work on it before you wrote your patch | |
| 14:50:56 | sean-k-mooney | so i was annoy wen you pushed it while i was fixing uniti tests and docs ectra | |
| 14:51:41 | dansmith | sean-k-mooney: to me it sounded like you'd work on it if you had time, and weren't particularly concerned about getting it any time soon | |
| 14:51:51 | dansmith | so I was glad bauzas just went ahead with it | |
| 14:52:20 | sean-k-mooney | i only did push it early because of the sprint/jira disucssion downstream | |
| 14:52:40 | sean-k-mooney | like i had it open the entire tiem and was workign on it on an doff since i said i would on irc | |
| 14:52:56 | sean-k-mooney | i kept stopping to explian how to fix it to bauzas | |
| 14:53:20 | dansmith | sean-k-mooney: that seems like a rather uncharitable characterization of the conversation as I remember it | |
| 14:54:18 | bauzas | okay, I don't want to create a situation, lemme abandon my change | |
| 14:54:30 | bauzas | this is Friday afternoon for me and I'm done arguing for the week :) | |
| 14:54:45 | sean-k-mooney | if your change is done and has all the test and what gibi asked for i litrally only have one test left ot finsh | |
| 14:55:01 | sean-k-mooney | so i can drop it but i was almost done before my team call earier today | |
| 14:55:25 | bauzas | I don't f... care :) | |
| 14:55:33 | bauzas | if you have time to work on it, do it | |
| 14:56:09 | bauzas | anyway, patch abandoned, floor is yours but please make sure that's on a good shape for rc1 | |
| 14:56:23 | bauzas | I can't just argue, again I'm done | |
| 14:56:25 | dansmith | I don't want to go dig up actual quotes, but it surely seemed to me we left the conversation with sean-k-mooney indicating not a lot of time to work on it and bauzas saying he did | |
| 14:56:51 | dansmith | so I reviewed bauzas' immediately and then sean-k-mooney pushed a competing one up after even that first round of reviews | |
| 14:56:53 | bauzas | fwiw, we have a tracker somewhere downstream that also needs to reflect the who does what | |
| 14:57:01 | dansmith | so now I've got to go review the other one? | |
| 14:57:04 | bauzas | so please update it as well | |
| 14:57:39 | bauzas | dansmith: the two are basically identical, except an object modification (which hopefully is just an helper) | |
| 14:58:00 | bauzas | tests diverge a little bit | |
| 14:58:07 | dansmith | so even less necessary to have pushed up a competing patch after the first one was the same solution? | |
| 14:58:31 | bauzas | that's 5pm here on a Friday anyway and I started at 8am, I call it a wrap | |
| 15:01:28 | sean-k-mooney | so i honestly done know what to say | |
| 15:01:56 | sean-k-mooney | bauzas: almost made me rage quite since it feld liek tehy were inteally not reading anythin i wrote on irc | |
| 15:01:58 | bauzas | well, my patch is abandoned, the problem is over | |
| 15:02:23 | sean-k-mooney | so i stop what i was workign on and said i woudl owrk on it | |
| 15:02:53 | sean-k-mooney | i spemnd my evning na dmoning workign on it between interups and while i was waiting for tox runs you pushed a pathc | |
| 15:03:36 | bauzas | that's my point, I had capacity, you didn't had, which is why I offered to push it | |
| 15:03:42 | sean-k-mooney | that patch didnt have hte fallback or tests which i had in my locally so i contineus working on it | |
| 15:03:53 | bauzas | now you say that you have capacity, which is cool, then I abandon mine | |
| 15:04:25 | bauzas | if that sounds a matter of IP, then surely I don't give a single cent about it | |
| 15:04:31 | sean-k-mooney | no i made time to work on it because it felt like you were ignoring me | |
| 15:04:44 | sean-k-mooney | you almost made me rage quite irc yesterday | |
| 15:05:09 | bauzas | the good news is that irc logs are public | |
| 15:05:43 | bauzas | I don't think I had the intent to make you rage, I just expressed one thing about capacity and again I offered a solution | |
| 15:06:06 | bauzas | now, again, fact is that my patch is ABANDONED, so we anyway need to review your patch | |
| 15:06:07 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix num_instances_weight_multiplier option example https://review.opendev.org/c/openstack/nova/+/959866 | |
| 15:06:30 | sean-k-mooney | bauzas: im currently reviewing your | |
| 15:06:37 | bauzas | please then update yours based on gibi's feedback and then I'll review it | |
| 15:06:38 | sean-k-mooney | to see what the delta is | |
| 15:07:29 | sean-k-mooney | i have, im litrally just workign ont hte unit test to show the case where the rpc is called and hwen it not based on if the stats have uptime | |
| 15:08:08 | sean-k-mooney | that was the only bit of feedback i didnt have completed 2 hours ago | |
| 15:12:48 | bauzas | sean-k-mooney: I really have to drop soon, again, fact is, I had time today to work on a PS3, no other bits than that | |
| 15:13:42 | sean-k-mooney | bauzas: enjoy your weekend sylvain | |
| 15:13:43 | bauzas | I don't have a particular opinion of my contributions fwiw, so again, feel free to update your patch so I can review it | |
| 15:14:30 | sean-k-mooney | and yes ill finish updating it but i feel like we need to reset | |
| 15:15:05 | sean-k-mooney | we really need ot just chil and hang out some time | |
| 15:15:14 | dansmith | I must say, I feel like something pretty uncool has happened here | |
| 15:19:12 | sean-k-mooney | we both obviously were not comunicating well and both feel like we have wated each other time. syvalisn patch does nto have hte optimisation to prevent the slow down for dirver that dont supprot uptime, that the only delta between the two in general at this point | |
| 15:19:46 | dansmith | sean-k-mooney: I had already commented on that on his patch before you even posted yours, so that's not a reason | |
| 15:20:10 | dansmith | sean-k-mooney: also, he marked you as co-author so if you had tweaks you could have pushed them directly there after it was up, or written tests for his patch instead of writing your own and then tests for that | |
| 15:20:20 | dansmith | so it really feels a lot like a snipe to me, sorry to say | |