| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-03-13 | |||
| 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) | |
| 15:10:08 | artom | bauzas, sean-k-mooney, I'll cherry-pick them downstream anyways | |
| 15:10:38 | bauzas | sure, but please don't discuss downstream usage here :) | |
| 15:10:42 | bauzas | it's a public channel | |
| 15:10:54 | bauzas | (and we log everything :) ) | |
| 15:11:05 | sean-k-mooney | this is entirly approate to disucss here | |
| 15:11:17 | bauzas | sean-k-mooney: gibi: dansmith: https://review.opendev.org/c/openstack/nova/+/912973 prelude is open for reviews | |
| 15:11:18 | sean-k-mooney | artom was askign if we shoudl consider backportign to unmaintend branhces | |
| 15:11:28 | sean-k-mooney | and i say we can do that if we want to for other reasons | |
| 15:11:41 | sean-k-mooney | but form a core or stable core perspective we do not maintian those | |
| 15:11:48 | sean-k-mooney | and are not expected to do backport to them | |
| 15:11:57 | artom | I don't think it's a surprise that Red Hat has a downstream product, and that we do cherry-picks :) | |
| 15:12:07 | artom | It's not like we're discussing downstream-only code changes here | |
| 15:12:32 | sean-k-mooney | right so i was just trying to provide guidence that we shoudl default to backporting to stable/* | |
| 15:12:39 | artom | Oh, totally | |
| 15:13:00 | sean-k-mooney | and unmaintained/* is optional but not expected | |
| 15:13:04 | artom | As far back as is reasonably possible. | |
| 15:13:05 | artom | Yep. | |
| 15:13:36 | sean-k-mooney | if we are backporting downstream past stable/* it does not hurt to provide the patches publicly but if peopel choose not to it that ok too | |