| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-03-13 | |||
| 14:53:44 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move Caracal implemented specs https://review.opendev.org/c/openstack/nova-specs/+/912783 | |
| 15:05:45 | bauzas | cores, did a bunch of new patches for RC1, like this one ^ | |
| 15:05:49 | bauzas | please review them today | |
| 15:06:16 | bauzas | https://etherpad.opendev.org/p/nova-caracal-rc-potential#L12 | |
| 15:12:09 | sean-k-mooney | bauzas: -1 on https://review.opendev.org/c/openstack/nova-specs/+/912783 | |
| 15:12:18 | sean-k-mooney | you delete the specs instead of moving them | |
| 15:12:50 | bauzas | oh shit, I just forgot to add the new files | |
| 15:12:54 | bauzas | git add * | |
| 15:13:17 | sean-k-mooney | yep | |
| 15:13:21 | sean-k-mooney | otherwise it looks fine | |
| 15:13:37 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move Caracal implemented specs https://review.opendev.org/c/openstack/nova-specs/+/912783 | |
| 15:13:40 | bauzas | done ^ | |
| 15:14:02 | sean-k-mooney | did you use the script to generate this by the way | |
| 15:14:07 | sean-k-mooney | i assume yes | |
| 15:14:16 | bauzas | I use the specific tox target | |
| 15:14:24 | sean-k-mooney | ya ok so just forgot the add | |
| 15:14:27 | bauzas | move-implemented-specs | |
| 15:14:32 | bauzas | indeed | |
| 15:14:35 | sean-k-mooney | i just wanted to check that the script was not broken | |
| 15:14:45 | bauzas | (16:12:50) bauzas: oh shit, I just forgot to add the new files | |
| 15:14:45 | bauzas | (16:12:53) bauzas: git add * | |
| 15:15:04 | bauzas | indeed, the script works correctly | |
| 15:20:46 | gibi | bauzas: left +2s for those patches | |
| 15:20:54 | bauzas | thanks | |
| 15:21:04 | bauzas | I'll now create the prelude | |
| 16:03:27 | opendevreview | Merged openstack/nova-specs master: Move Caracal implemented specs https://review.opendev.org/c/openstack/nova-specs/+/912783 | |
| 19:03:57 | sean-k-mooney | bauzas: for https://review.opendev.org/c/openstack/nova/+/912617 you have decied not to include https://review.opendev.org/c/openstack/nova/+/891289 then in caracal | |
| 19:04:50 | sean-k-mooney | bauzas: https://review.opendev.org/c/openstack/nova/+/891289 has been waiting on yoru direct input for 8 days now and i have pinged you before | |
| 19:05:06 | sean-k-mooney | so im makeing sure you didnt miss it | |
| 19:05:50 | sean-k-mooney | it could just move the ailias but this shoudl either be in RC 1 or in dalmation | |
| 19:06:01 | sean-k-mooney | a compute service bump and rpc bump shoudl not be in rc2 | |
| 19:24:16 | opendevreview | Merged openstack/nova master: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/909795 | |
| 23:15:29 | opendevreview | Merged openstack/nova master: Add service version for Caracal https://review.opendev.org/c/openstack/nova/+/912625 | |
| 23:15:36 | opendevreview | Merged openstack/nova master: doc: mark the maximum microversion for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/912616 | |
| 23:48:31 | opendevreview | Merged openstack/nova master: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/912320 | |
| 23:48:41 | opendevreview | Merged openstack/nova master: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/910022 | |
| #openstack-nova - 2024-03-14 | |||
| 00:06:51 | opendevreview | Merged openstack/nova master: pwr mgmt: handle live migrations correctly https://review.opendev.org/c/openstack/nova/+/909806 | |
| 07:00:26 | frickler | I see nova is still busy, but is placement ready for rc1? https://review.opendev.org/c/openstack/releases/+/912427 | |
| 08:32:56 | bauzas | sean-k-mooney: morning, I absolutely forgot about https://review.opendev.org/c/openstack/nova/+/891289 | |
| 08:34:23 | bauzas | sean-k-mooney: so if you want, we can merge it this morning and then I'll update https://review.opendev.org/c/openstack/nova/+/912617 | |
| 09:06:22 | bauzas | sean-k-mooney: gibi: the SRIOV fix for mdevs is still unapproved, could you please give it a shot ? | |
| 09:06:24 | bauzas | https://review.opendev.org/c/openstack/nova/+/899625 | |
| 09:42:07 | gibi | bauzas: I can be the second core there but I need Sean's or Dan's +2 first as I don't have the full context about the discussion here https://review.opendev.org/c/openstack/nova/+/899625/11#message-c3edab5a61f5dfcfc1297e2891ace5d742185f6a | |
| 09:42:35 | bauzas | this was resolved | |
| 09:44:10 | gibi | I haven't seen Dan's review since that discussion. | |
| 09:47:59 | bauzas | cool enough | |
| 10:47:59 | opendevreview | Merged openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 11:26:09 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add a Caracal prelude section https://review.opendev.org/c/openstack/nova/+/912973 | |
| 11:39:40 | sean-k-mooney | bauzas: ack i see https://review.opendev.org/c/openstack/nova/+/891289 is now merged are you going to update https://review.opendev.org/c/openstack/nova/+/912617 | |
| 11:47:14 | sean-k-mooney | bauzas: gibi i readded my +2 but if you want to wait for dan to review that fine too | |
| 11:48:30 | sean-k-mooney | bauzas: ignoring release stuff for a sec i assume your next priority is using the mtty devices to add basic testing in ci of creating instnaces with mdevs | |
| 11:48:42 | sean-k-mooney | by basic i mean without live migration | |
| 11:51:23 | sean-k-mooney | out side of perhaps the mdev persitence feature melwitt has in flight and those two bug fixes i really dont want to do anything else realted to mdevs until we have the ci job in place | |
| 11:58:21 | gibi | I +Ad | |
| 11:58:24 | gibi | https://review.opendev.org/c/openstack/nova/+/899625 | |
| 12:00:33 | sean-k-mooney | https://www.youtube.com/watch?v=ixZpBOxMopc spacex are launching in an hour just an fyi | |
| 12:00:47 | sean-k-mooney | take 3 at starship | |
| 12:01:42 | gibi | yepp, I following it :) | |
| 12:19:35 | bauzas | sean-k-mooney: could you please stop to ask me something when I'm off about 12:30pm ? :) | |
| 12:20:28 | sean-k-mooney | ask you stuff only when your away aroudn 12:30 sure | |
| 12:20:28 | bauzas | anyway thanks for the reviews :) | |
| 12:21:44 | bauzas | (for going with her to her school) | |
| 14:29:23 | artom | Huh, https://review.opendev.org/c/openstack/nova/+/877773 somehow avoided the merge conflict | |
| 14:29:33 | artom | Weird, but yey for less work for Uggla | |
| 14:30:45 | Uggla | artom, cool, I was about to do it, so it is useless. | |
| 14:34:45 | sean-k-mooney | artom: it the non backporable one that was in merge conflcit | |
| 14:35:02 | sean-k-mooney | so ya i guess that my queue to go review the backportable one | |
| 14:35:04 | artom | They're both apparently clear | |
| 14:35:11 | artom | Not sure how | |
| 14:35:33 | sean-k-mooney | oh its says same topic not merge conflict | |
| 14:35:51 | sean-k-mooney | artom: are your patches emged now | |
| 14:36:00 | sean-k-mooney | i tought several of them were still pending | |
| 14:36:07 | artom | Yeah, they all went through last night | |
| 14:37:18 | sean-k-mooney | in that case would you mind starting the backport of your patches | |
| 14:44:42 | artom | Yep | |
| 14:44:52 | artom | To Antelope, right? When did we actually add power management? | |
| 14:45:17 | artom | Yeah https://specs.openstack.org/openstack/nova-specs/specs/2023.1/implemented/libvirt-cpu-state-mgmt.html | |
| 14:46:34 | sean-k-mooney | yes | |
| 14:47:40 | sean-k-mooney | Uggla: change choudl go back futher but antelope would be ok for that too | |
| 14:47:57 | sean-k-mooney | zed will move to unmainteined in a few weeks | |
| 14:48:19 | artom | We'd like it downstream in 17.1 though | |
| 14:48:21 | sean-k-mooney | basically when the offial release of caracal happens in april | |
| 14:48:43 | sean-k-mooney | we can optionally propose it to the unmained branches if we like | |
| 14:49:01 | sean-k-mooney | assuming that helps with the downstream backprot withc it likely will | |
| 14:49:22 | sean-k-mooney | just saying that by default the stable/core team are not expected to do backports to unmaintained/* | |
| 14:57:13 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: Add cpuset_reserved helper to instance NUMA topology https://review.opendev.org/c/openstack/nova/+/913193 | |
| 14:57:14 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: Reproducer for not powering on isolated emulator threads cores https://review.opendev.org/c/openstack/nova/+/913194 | |
| 14:57:15 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/913195 | |
| 14:57:16 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/913196 | |
| 14:57:17 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/913197 | |
| 14:57:20 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.2: pwr mgmt: handle live migrations correctly https://review.opendev.org/c/openstack/nova/+/913198 | |
| 15:08:12 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: fup for power management series https://review.opendev.org/c/openstack/nova/+/913221 | |
| 15:08:13 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: Add cpuset_reserved helper to instance NUMA topology https://review.opendev.org/c/openstack/nova/+/913222 | |
| 15:08:14 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: Reproducer for not powering on isolated emulator threads cores https://review.opendev.org/c/openstack/nova/+/913223 | |
| 15:08:15 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/913224 | |
| 15:08:16 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/913225 | |
| 15:08:17 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/913226 | |
| 15:08:19 | opendevreview | Artom Lifshitz proposed openstack/nova stable/2023.1: pwr mgmt: handle live migrations correctly https://review.opendev.org/c/openstack/nova/+/913227 | |
| 15:09:48 | bauzas | artom: sean-k-mooney: yeah, I'll look at the backports after GA | |
| 15:09:52 | bauzas | (Apr 03) | |