| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-03-14 | |||
| 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 | |
| 15:14:06 | sean-k-mooney | i prefer when our backprots upstream or downstream are made avaiabel to as wide an audinace as possibel so i prefer upstrema first to be our default | |
| 15:19:24 | bauzas | for sure | |
| 15:19:48 | bauzas | I was just saying to make sure we don't need to discuss about our downstreack backports here | |
| 15:20:28 | bauzas | not because of security, but rather Red Hat is not the only company working upstream :) | |
| 15:20:28 | sean-k-mooney | yep i just wanted artom to start on the upstream one and we can figure out downsteam when wever that becomes relevent | |
| 15:21:02 | bauzas | (and it was just a reminder that we log everything here : https://meetings.opendev.org/irclogs/%23openstack-nova/ ) | |
| 15:21:23 | artom | sean-k-mooney, they're all posted anyways, even the downstream ones :) | |
| 15:22:24 | artom | bauzas, I know this is a public channel - again, I don't think it's a surprise to anyone, or is secret information, that Red Hat has a downstream product with cherry-picks :) And that's there the mention of it ends, nothing further will be discussed | |
| 15:23:03 | bauzas | no worries, but maybe not all of us know about the logs :) ) | |
| 15:24:07 | sean-k-mooney | i do and im sure artom does so is it new info to you? | |
| 15:25:11 | artom | I was totally about to post by credit card PIN | |
| 15:25:55 | sean-k-mooney | password: | |
| 15:25:58 | sean-k-mooney | :) | |
| 15:26:45 | sean-k-mooney | anyway i got distracted by thing ill go back and review Uggla patch now | |
| 15:32:59 | artom | See, then you typed your password, I saw *****, but you saw hunter2, because IRC is smart like that | |
| 15:59:57 | opendevreview | Sylvain Bauza proposed openstack/nova master: Update compute rpc alias for caracal https://review.opendev.org/c/openstack/nova/+/912617 | |
| 16:00:21 | bauzas | dansmith: gibi: sean-k-mooney: yet another paperwork change for RC1 ^ | |
| 16:00:36 | bauzas | (just updated it b/c of the new RPC version we just added) | |
| 16:03:56 | sean-k-mooney | ack im +2 on https://review.opendev.org/c/openstack/nova/+/877773 by the way | |
| 16:20:13 | gibi | I'm +2 the RPC alias | |
| 17:04:09 | bauzas | elodilles: sean-k-mooney: I just figured out (my bad) that we still have a few open patches in placement that could be merged before RC1 https://review.opendev.org/q/project:openstack/placement+is:open | |
| 17:04:22 | bauzas | https://review.opendev.org/c/openstack/placement/+/907742 seems at least nice to have | |
| 17:23:30 | sean-k-mooney | the slurp job can merge at any point | |
| 17:23:40 | sean-k-mooney | we can trivially just backport that to the stable branch | |
| 17:23:49 | sean-k-mooney | but i agree it woudl be nice to have | |
| 17:49:09 | elodilles | bauzas: ACK, thanks for signaling it on the release patch! | |
| 17:49:25 | elodilles | (though you linked a nova patch there o:)) | |
| 18:37:27 | melwitt | sean-k-mooney: speaking of persistent mdevs, I think it's ready for review if you were wondering https://review.opendev.org/c/openstack/nova/+/910041 | |
| 18:38:38 | sean-k-mooney | how did testing host reboot go. were you able to do it with the sytemd service file? | |
| 18:40:01 | sean-k-mooney | melwitt: this is for dalmation ya so i can try and review it but we need to wait for RC1 to ship before merging i guess | |
| 18:41:09 | sean-k-mooney | this is pretty localised within the libvirt code so im not really concerned with this havign issues with other RCs so i would be ok ewith merging this once we reapprove the blueprint from a paperwork perpwective | |
| 18:41:14 | melwitt | sean-k-mooney: yeah, no rush obvs. I've been intentionally not bringing it up but just so you're aware. I don't expect review until later on | |
| 18:41:28 | sean-k-mooney | cool | |
| 18:42:11 | melwitt | sean-k-mooney: I didn't think to try the systemd way for some reason. I only did it manually. I'll try systemd next | |
| 18:42:43 | melwitt | I did remember to put the systemd mention into docs/ in that patch 😛 | |
| 18:42:53 | sean-k-mooney | ack if your including the doc update with the systemd script then ya it would be good to test that locally | |
| 18:43:20 | sean-k-mooney | yep its teh second ting i read after the commit message | |
| 18:43:31 | melwitt | yeah. agree | |
| 18:45:34 | sean-k-mooney | oh i see your supproting eithe rway based on the libvirt version | |
| 18:46:14 | melwitt | yeah.. initially I didn't do that but realized it later. we would need a min version bump to not have to do it I think | |
| 18:46:28 | sean-k-mooney | am that will not strictly be required as we can move to libvirt 8.0.0 next cycle | |
| 18:46:38 | sean-k-mooney | but since you have it working we can keep it | |
| 18:46:45 | melwitt | which we might be doing for dalamation, so I can take that out if so | |
| 18:47:31 | melwitt | it would make the change much prettier if we bumped to 8.0.0 heh | |
| 18:47:55 | sean-k-mooney | ya although its not terrible as is | |
| 18:47:56 | melwitt | persistent mdevs in 7.3.0 and autostart in 7.8.0 | |
| 18:48:46 | sean-k-mooney | autostart in this context being create them automatically on host start right | |
| 18:48:51 | sean-k-mooney | rather then domain start | |
| 18:49:30 | sean-k-mooney | may be worth adding follow up patches that do the min bump | |
| 18:49:55 | sean-k-mooney | and we can then defer that choice until we decied if we bump to 8.0.0 | |
| 18:50:54 | sean-k-mooney | it looks like there would not actully be much clean up to do fo 8.0.0 | |
| 18:51:57 | sean-k-mooney | i mean for other feature | |