bauzas | Uggla: looks to me we're having some problem with your cpu_shared_set series https://review.opendev.org/c/openstack/nova/+/877773 | 08:22 |
---|---|---|
bauzas | elodilles: I approved a lot of placement changes this morning, so I'll provide a new revision for the placement RC1 release once all of them are merged | 08:22 |
Uggla | bauzas, looking at it | 08:23 |
bauzas | elodilles: for nova, we need to await a bit more, at least the prelude change is not yet approved https://review.opendev.org/c/openstack/nova/+/912973 + I'd like https://review.opendev.org/c/openstack/nova/+/899625/ to be merged | 08:23 |
bauzas | I know we're late :( | 08:23 |
elodilles | bauzas: ACK, thanks for the heads up | 08:24 |
elodilles | sean-k-mooney: LGTM, +2+W'd the patch | 09:01 |
Uggla | bauzas, found the issue this is _disconnect_volume which is called with 2 more parameters than expected destroy_secrets and force. What is funny that it is the case with the [backport] patch and not with the other. | 09:20 |
bauzas | cool to know | 09:22 |
opendevreview | Merged openstack/placement master: Add upgrade job from 2023.1 https://review.opendev.org/c/openstack/placement/+/907742 | 09:58 |
opendevreview | Merged openstack/placement master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/placement/+/911286 | 09:59 |
opendevreview | Merged openstack/placement master: reno: Update master for unmaintained/victoria https://review.opendev.org/c/openstack/placement/+/911276 | 09:59 |
opendevreview | Merged openstack/placement master: reno: Update master for unmaintained/xena https://review.opendev.org/c/openstack/placement/+/911296 | 09:59 |
bauzas | \o/ | 09:59 |
elodilles | ~o~ | 10:00 |
bauzas | one last to go, and then I'll update the placement RC1 patch | 10:00 |
elodilles | +1 | 10:00 |
elodilles | thx | 10:01 |
opendevreview | Merged openstack/placement master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/placement/+/904618 | 10:01 |
opendevreview | Merged openstack/nova master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/nova/+/901587 | 10:01 |
bauzas | perfect on time | 10:01 |
* bauzas needs to remember on the releases tooling :) | 10:02 | |
bauzas | https://releases.openstack.org/reference/using.html#using-new-release-command | 10:03 |
bauzas | cool, found ^ | 10:03 |
elodilles | well, to tell you the truth, sometimes it's easier to just edit the hash o:) | 10:04 |
elodilles | though you have to look for the correct hash, so new-release command can be easier :X | 10:05 |
bauzas | elodilles: I prefer the tooling in order to ensure that the git repo is all good | 10:08 |
bauzas | for example, github mirror doesn't have yet the last patch | 10:08 |
bauzas | anyway, updated | 10:09 |
elodilles | bauzas: true. thanks for the update o/ | 10:16 |
Uggla | bauzas, progressing it seems the do_cleanup flag is set with the [backport] patch and not with the other patch | 10:32 |
opendevreview | Merged openstack/nova stable/2023.1: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/911070 | 10:54 |
Uggla | bauzas, found the issue this is linked with power_management. I need to crosscheck with artom. | 10:55 |
opendevreview | Steven Blatzheim proposed openstack/nova stable/zed: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/912755 | 11:03 |
stblatzheim | sean-k-mooney, elodilles: thanks for the reviews for the backport of the metadata bug. Here's the last cherrypick for zed: https://review.opendev.org/c/openstack/nova/+/912755 | 11:05 |
elodilles | stblatzheim: thanks too for the backports! | 11:17 |
opendevreview | OpenStack Release Bot proposed openstack/placement stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/placement/+/913348 | 14:17 |
opendevreview | OpenStack Release Bot proposed openstack/placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/placement/+/913349 | 14:17 |
opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/placement/+/913350 | 14:17 |
bauzas | Uggla: artom: okay, so please tell me if we need to get a new change before we provide the RC1 | 14:21 |
bauzas | elodilles: f*** the gate failed again for https://review.opendev.org/c/openstack/nova/+/899625 | 14:21 |
elodilles | :S | 14:29 |
elodilles | it would be good to have the nova release today, but it's not the end of the world if we can merge the release patch only Monday :/ | 14:30 |
elodilles | it's good that you signaled on the patch that we should wait | 14:30 |
dansmith | melwitt: do you know anything about why we're hitting this for BFV? https://zuul.opendev.org/t/openstack/build/05d3e04d9fac460c9e7011e0b40a7089/log/controller/logs/screen-n-cpu.txt#9547 | 15:00 |
dansmith | I know that's very old code, but it surely seems like we shouldn't be hitting that as a regular event, but it's all over the logs | 15:00 |
sean-k-mooney | 15:28 | |
sean-k-mooney | dansmith: is that form the nfs job | 15:29 |
sean-k-mooney | dansmith: i know melwitt was trying to fix the snapshot code at somepoint for nfs but i dont know what the current state of tha tis | 15:29 |
sean-k-mooney | it migh be related too https://bugs.launchpad.net/nova/+bug/1989514 | 15:31 |
sean-k-mooney | whic is adressed by https://review.opendev.org/c/openstack/cinder/+/857528 | 15:32 |
bauzas | dansmith: sean-k-mooney: please review the prelude patch when you can https://review.opendev.org/c/openstack/nova/+/912973 | 16:33 |
bauzas | elodilles: yet again it failed :( | 16:33 |
elodilles | :/ | 16:58 |
sean-k-mooney | bauzas: did you see my comments on the release highlights patch | 17:24 |
sean-k-mooney | bauzas: -1 on the prelude https://review.opendev.org/c/openstack/nova/+/912973 i have called out mostly the same issues that were in the highlight | 17:44 |
sean-k-mooney | i would prefer if we could fix both but i dont want to duplicate them in the prelude so at a minium i would ike to fix them there sicne that is not merged | 17:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!