Friday, 2024-03-15

bauzasUggla: looks to me we're having some problem with your cpu_shared_set series https://review.opendev.org/c/openstack/nova/+/877773 08:22
bauzaselodilles: 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 merged08:22
Ugglabauzas, looking at it08:23
bauzaselodilles: 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 merged08:23
bauzasI know we're late :(08:23
elodillesbauzas: ACK, thanks for the heads up08:24
elodillessean-k-mooney: LGTM, +2+W'd the patch09:01
Ugglabauzas, 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
bauzascool to know09:22
opendevreviewMerged openstack/placement master: Add upgrade job from 2023.1  https://review.opendev.org/c/openstack/placement/+/90774209:58
opendevreviewMerged openstack/placement master: reno: Update master for unmaintained/wallaby  https://review.opendev.org/c/openstack/placement/+/91128609:59
opendevreviewMerged openstack/placement master: reno: Update master for unmaintained/victoria  https://review.opendev.org/c/openstack/placement/+/91127609:59
opendevreviewMerged openstack/placement master: reno: Update master for unmaintained/xena  https://review.opendev.org/c/openstack/placement/+/91129609:59
bauzas\o/09:59
elodilles~o~10:00
bauzasone last to go, and then I'll update the placement RC1 patch10:00
elodilles+110:00
elodillesthx10:01
opendevreviewMerged openstack/placement master: Update python classifier in setup.cfg  https://review.opendev.org/c/openstack/placement/+/90461810:01
opendevreviewMerged openstack/nova master: Update python classifier in setup.cfg  https://review.opendev.org/c/openstack/nova/+/90158710:01
bauzasperfect on time10:01
* bauzas needs to remember on the releases tooling :)10:02
bauzashttps://releases.openstack.org/reference/using.html#using-new-release-command10:03
bauzascool, found ^10:03
elodilleswell, to tell you the truth, sometimes it's easier to just edit the hash o:)10:04
elodillesthough you have to look for the correct hash, so new-release command can be easier :X10:05
bauzaselodilles: I prefer the tooling in order to ensure that the git repo is all good10:08
bauzasfor example, github mirror doesn't have yet the last patch10:08
bauzasanyway, updated10:09
elodillesbauzas: true. thanks for the update o/10:16
Ugglabauzas, progressing it seems the do_cleanup flag is set with the [backport] patch and not with the other patch10:32
opendevreviewMerged openstack/nova stable/2023.1: Fix nova-metadata-api for ovn dhcp native networks  https://review.opendev.org/c/openstack/nova/+/91107010:54
Ugglabauzas, found the issue this is linked with power_management. I need to crosscheck with artom.10:55
opendevreviewSteven Blatzheim proposed openstack/nova stable/zed: Fix nova-metadata-api for ovn dhcp native networks  https://review.opendev.org/c/openstack/nova/+/91275511:03
stblatzheimsean-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/+/91275511:05
elodillesstblatzheim: thanks too for the backports!11:17
opendevreviewOpenStack Release Bot proposed openstack/placement stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/placement/+/91334814:17
opendevreviewOpenStack Release Bot proposed openstack/placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/placement/+/91334914:17
opendevreviewOpenStack Release Bot proposed openstack/placement master: Update master for stable/2024.1  https://review.opendev.org/c/openstack/placement/+/91335014:17
bauzasUggla: artom: okay, so please tell me if we need to get a new change before we provide the RC114:21
bauzaselodilles: f*** the gate failed again for https://review.opendev.org/c/openstack/nova/+/89962514:21
elodilles:S14:29
elodillesit 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
elodillesit's good that you signaled on the patch that we should wait14:30
dansmithmelwitt: 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#954715:00
dansmithI 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 logs15:00
sean-k-mooney 15:28
sean-k-mooneydansmith: is that form the nfs job15:29
sean-k-mooneydansmith: i know melwitt was trying to fix the snapshot code at somepoint for nfs but i dont know what the current state of tha tis15:29
sean-k-mooneyit migh be related too https://bugs.launchpad.net/nova/+bug/198951415:31
sean-k-mooneywhic is adressed by https://review.opendev.org/c/openstack/cinder/+/85752815:32
bauzasdansmith: sean-k-mooney: please review the prelude patch when you can https://review.opendev.org/c/openstack/nova/+/91297316:33
bauzaselodilles: yet again it failed :(16:33
elodilles:/16:58
sean-k-mooneybauzas: did you see my comments on the release highlights patch 17:24
sean-k-mooneybauzas: -1 on the prelude https://review.opendev.org/c/openstack/nova/+/912973 i have called out mostly the same issues that were in the highlight17:44
sean-k-mooneyi 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 merged17:45

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!