| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-03-12 | |||
| 18:06:31 | opendevreview | Sylvain Bauza proposed openstack/nova master: doc: mark the maximum microversion for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/912616 | |
| 18:09:10 | opendevreview | Sylvain Bauza proposed openstack/nova master: Update compute rpc alias for caracal https://review.opendev.org/c/openstack/nova/+/912617 | |
| 18:14:41 | opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912618 | |
| 18:14:45 | opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912619 | |
| 18:14:48 | opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912620 | |
| 18:15:05 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912621 | |
| 18:15:08 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912622 | |
| 18:15:11 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912623 | |
| 18:15:29 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912624 | |
| 18:15:31 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add service version for Caracal https://review.opendev.org/c/openstack/nova/+/912625 | |
| 18:15:33 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912626 | |
| 18:15:36 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912627 | |
| 19:50:12 | opendevreview | Merged openstack/python-novaclient stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912624 | |
| 19:53:52 | opendevreview | Merged openstack/osc-placement stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912621 | |
| 19:53:53 | opendevreview | Merged openstack/osc-placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912622 | |
| 19:56:56 | opendevreview | Merged openstack/python-novaclient stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912626 | |
| 21:40:56 | opendevreview | Merged openstack/os-vif stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912618 | |
| 21:56:31 | opendevreview | Merged openstack/os-vif stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912619 | |
| 22:23:03 | opendevreview | norman shen proposed openstack/nova master: Remove an unncessary LOG https://review.opendev.org/c/openstack/nova/+/912661 | |
| #openstack-nova - 2024-03-13 | |||
| 00:34:22 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Create persistent mdevs https://review.opendev.org/c/openstack/nova/+/910041 | |
| 05:17:01 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Create persistent mdevs https://review.opendev.org/c/openstack/nova/+/910041 | |
| 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:53) bauzas: git add * | |
| 15:14:45 | bauzas | (16:12:50) bauzas: oh shit, I just forgot to add the new files | |
| 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 | bauzas | anyway thanks for the reviews :) | |
| 12:20:28 | sean-k-mooney | ask you stuff only when your away aroudn 12:30 sure | |
| 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 | |