| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-11 | |||
| 16:27:11 | sean-k-mooney | so i dont wnat to hold it up | |
| 16:27:14 | bauzas | I just provided the existing master SHA1s for each of the projects | |
| 16:27:31 | bauzas | that means that any new merged patch wouldn't be in the eol tag | |
| 16:27:47 | sean-k-mooney | thats ok we have not merged anything since then | |
| 16:27:50 | sean-k-mooney | and i dont expect too | |
| 16:28:10 | bauzas | elodilles: btw. do we have a CI job that tests this and say "eh, look, you want to tag with this sha1, but I see some other patch above, so -1" | |
| 16:28:10 | elodilles | (sean-k-mooney: i propsed a gate fix patch 2 yrs ago: https://review.opendev.org/c/openstack/os-vif/+/807839 :D ) | |
| 16:28:22 | sean-k-mooney | ... | |
| 16:28:35 | sean-k-mooney | ok well then we coudl fix it or just not | |
| 16:29:10 | elodilles | bauzas: no, there's no such check, but i ussually check it myself | |
| 16:29:31 | elodilles | bauzas: though it could be added to relmgmt tools, yes | |
| 16:29:34 | sean-k-mooney | i dont really want to delay the eol process but we can discuss this after the meeting | |
| 16:29:37 | bauzas | okay | |
| 16:30:02 | bauzas | sean-k-mooney: I need to drop quickly after the meeting, but sure I let you to discuss this with elodilles | |
| 16:30:08 | sean-k-mooney | ack | |
| 16:30:14 | elodilles | bauzas: and the branch removal script checks if we have patches on top of the EOL tag (but anyway, we cannot modify the tag) | |
| 16:30:45 | bauzas | elodilles: okay, so I'll probably double-check the train branches before we eventually +W | |
| 16:30:47 | elodilles | bauzas: no need, we can continue after the meeting with sean-k-mooney | |
| 16:30:52 | bauzas | cool | |
| 16:30:54 | bauzas | moving on | |
| 16:30:57 | bauzas | #topic Open discussion | |
| 16:31:04 | bauzas | (agenda empty) | |
| 16:31:16 | bauzas | oh, actually, there is one | |
| 16:31:47 | bauzas | I'll be off between Tues Aug 1 and Friday Aug 25 | |
| 16:32:15 | bauzas | so someone needs to be leading the nova meeting in between Aug 1 and Aug 22 | |
| 16:32:23 | bauzas | who wants to do it ? :) | |
| 16:32:35 | gibi | I can do the meeting on 15 and 22 | |
| 16:32:47 | bauzas | gibi: appreciated | |
| 16:32:53 | gibi | I'm also off on 1 and 8 | |
| 16:33:27 | bauzas | ack so we'll see next week | |
| 16:33:37 | bauzas | either someone proposes or we drop Aug 1 and 8 meetings | |
| 16:33:48 | gibi | ack | |
| 16:33:50 | bauzas | that was it for me | |
| 16:34:12 | bauzas | as every week, anything anyone ? | |
| 16:34:17 | gibi | - | |
| 16:34:35 | elodilles | - | |
| 16:35:13 | bauzas | cool | |
| 16:35:20 | bauzas | I give you all 25 mins back of your time ! | |
| 16:35:24 | gibi | thanks! | |
| 16:35:25 | bauzas | thanks all | |
| 16:35:28 | bauzas | #endmeeting | |
| 16:35:28 | opendevmeet | Meeting ended Tue Jul 11 16:35:28 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:35:28 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-07-11-16.00.html | |
| 16:35:28 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-07-11-16.00.txt | |
| 16:35:28 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-07-11-16.00.log.html | |
| 16:35:44 | elodilles | thanks bauzas & all o/ | |
| 16:37:42 | sean-k-mooney | elodilles: back to os-vif i personally dont have an issue with signel core approving yoru gate fix for train given noone else has objected | |
| 16:38:01 | sean-k-mooney | in almost 2 years | |
| 16:38:25 | elodilles | sean-k-mooney: note that since it's 2 yrs old, it probably won't merge | |
| 16:38:53 | sean-k-mooney | probably but lookign at https://review.opendev.org/q/project:openstack%252Fos-vif+branch:stable%252Ftrain+status:open | |
| 16:39:18 | sean-k-mooney | thre are some patches that might be worht landing but i dont rellly want the risk fo breakign the eol tag by rushing | |
| 16:39:40 | sean-k-mooney | so really im wondiering if its worth fixign those issue vs just taging what we have | |
| 16:39:46 | elodilles | yes, don't break things before the EOL :S | |
| 16:40:12 | elodilles | sean-k-mooney: i'm OK with leaving the branch as it is | |
| 16:40:48 | sean-k-mooney | ok that also works for me. the patchs are aviable and people could apply them to the eol tag if they wanted too | |
| 16:40:55 | sean-k-mooney | so im fine with leave it as is | |
| 16:41:51 | elodilles | the patches will be abandoned, yes, but who else needs it can cherry pick them downstream later | |
| 16:43:35 | sean-k-mooney | actully we have the switchdev ones backported downstream already and i was updating the other patches because i was pulling it downstream too | |
| 16:45:17 | sean-k-mooney | so im not actully that worreied that they will break thigns but just dont want to force in 5 patches right before eol | |
| 16:47:40 | sean-k-mooney | we dont patch things in our downstream without it landing in master first but we do backport thing that gerenally cant be backported upstream due to policy so just because it happend to work in our downstream buidl does not mean it wont break somethign upstream | |
| 16:53:05 | elodilles | sean-k-mooney: yeah, i guess upstream testing can be a bit different... | |
| 22:51:44 | opendevreview | melanie witt proposed openstack/nova stable/2023.1: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/888160 | |
| 22:54:54 | opendevreview | melanie witt proposed openstack/nova stable/zed: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/888161 | |
| 22:55:35 | opendevreview | melanie witt proposed openstack/nova stable/yoga: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/888162 | |
| 22:56:09 | opendevreview | melanie witt proposed openstack/nova stable/xena: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/888163 | |
| 22:56:44 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/888164 | |
| #openstack-nova - 2023-07-12 | |||
| 02:38:50 | opendevreview | melanie witt proposed openstack/nova stable/victoria: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888203 | |
| 02:38:51 | opendevreview | melanie witt proposed openstack/nova stable/victoria: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888204 | |
| 02:41:04 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888205 | |
| 02:41:05 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888206 | |
| 09:01:49 | kashyap | gibi (or anyone): When you get a min, I feel _something_ is off with the code removal of MIN_LIBVIRT_AARCH64_CPU_COMPARE: https://review.opendev.org/c/openstack/nova/+/887255/2/nova/virt/libvirt/driver.py#b9978 | |
| 09:01:55 | kashyap | I can't quite tell what, though | |
| 09:42:02 | opendevreview | alecorps proposed openstack/nova master: Workaround for issues with ephemeral disk named disk.local during resize https://review.opendev.org/c/openstack/nova/+/888220 | |
| 09:42:02 | gibi | kashyap: left feedback in the patch | |
| 09:42:03 | opendevreview | alecorps proposed openstack/nova master: Workaround to fix the issue in resize with ephemeral disk named disk.local. Add a condition to ensure the backing file is present on target host. https://review.opendev.org/c/openstack/nova/+/888221 | |
| 09:49:51 | kashyap | gibi: Thank you; looking | |
| 10:45:05 | gibi | kashyap: added one more comment to that review now | |
| 10:49:24 | frickler | sean-k-mooney: gibi: may I turn your attention to https://review.opendev.org/c/openstack/nova/+/871968 once more? there's a comment about a missing backport and I'm not sure whether the CI failure is maybe related or not | |
| 10:51:07 | sean-k-mooney | i belive that is blocked by my fixes for the gate blocker | |
| 10:51:19 | gibi | so one of the error: | |
| 10:51:20 | gibi | tempest.lib.exceptions.InvalidParam: Invalid Parameter passed: When validation.connect_method equals floating, validation_resources cannot be None | |
| 10:51:25 | sean-k-mooney | frickler: basically a test was merged in tempest that required validation to be enabled | |
| 10:51:26 | gibi | this is a known issue | |
| 10:51:44 | sean-k-mooney | i have not backported that fix for nova-lvm to zed yet | |
| 10:51:47 | gibi | ahh both job hit the same | |
| 10:52:13 | sean-k-mooney | only master and the 2023.1 patch is pending a second backport in the devstack ceph plugin | |
| 10:53:54 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/887674 is the current backport on the nova side | |
| 10:54:25 | sean-k-mooney | but we might need https://review.opendev.org/c/openstack/tempest/+/886991 https://review.opendev.org/c/openstack/devstack/+/887660 or https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887879 as well | |
| 10:55:42 | opendevreview | sean mooney proposed openstack/nova stable/2023.1: enable validations in nova-lvm https://review.opendev.org/c/openstack/nova/+/887674 | |
| 10:56:24 | sean-k-mooney | i have also added the depends-on for the change elodilles suggested in devstack and we can see if that helps | |
| 10:56:59 | sean-k-mooney | if it fails again ill try also pulling in the second devstack-plugin-ceph patch but im not actully sure that is requried | |
| 11:00:26 | frickler | so what about backporting https://review.opendev.org/c/openstack/nova/+/869950 , should that still also happen? at least to zed? | |
| 11:00:46 | sean-k-mooney | it needs to go to yoga im currently creatign one for zed | |
| 11:01:14 | sean-k-mooney | i was waiting for the antelope patch to actully be working before creating it but i can push one up qickly | |
| 11:04:33 | sean-k-mooney | i need to cherry pick 3 patchs total across 3 repos i think so im goign to do the other two first and then the nova patch which depend on them | |
| 11:11:32 | opendevreview | sean mooney proposed openstack/nova stable/zed: enable validations in nova-lvm https://review.opendev.org/c/openstack/nova/+/888230 | |
| 11:11:55 | sean-k-mooney | frickler: so i think ^ will work if the antelope one is now correct | |
| 11:49:25 | kashyap | gibi: Thanks! That return was the thing I was not quite sure of :) | |
| 11:49:34 | kashyap | But your reason makes sense | |
| 11:52:27 | kashyap | gibi: Just nuking that return should be fine there, I think. | |