| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-13 | |||
| 14:44:34 | dansmith | I think we reserve that for situations where we've got two things wedged against each other and one has to merge without tests | |
| 14:47:40 | bauzas | https://zuul.openstack.org/status#873300 looks like tempest-full-py3 can try twice | |
| 14:49:27 | bauzas | gibi: dansmith: so, given the time we have before FF, what should we be doing ? | |
| 14:49:33 | bauzas | punting the FF ? | |
| 14:49:45 | bauzas | or awaiting for the change to be merged ? | |
| 14:49:59 | dansmith | the 100% fail is only in the ceph job, so you can just make that job n-v for the moment and watch it carefully | |
| 14:50:00 | bauzas | or non-voting the job ? | |
| 14:50:05 | dansmith | but you'll still suffer all the other issues | |
| 14:50:16 | dansmith | how much is still needing to land for FF? | |
| 14:50:55 | bauzas | https://etherpad.opendev.org/p/nova-antelope-blueprint-status at least two series | |
| 14:51:47 | bauzas | + https://review.opendev.org/c/openstack/nova/+/872413 | |
| 14:53:30 | gibi | I'm OK to make the ceph job non voting to land the pending nova series | |
| 14:54:04 | bauzas | ok, then I'll create the change | |
| 14:54:17 | bauzas | and I'll also create a revert one | |
| 14:54:40 | dansmith | just make the revert one depends-on the fix and we can go ahead and have it pre-approved | |
| 14:54:45 | gibi | yeah and you can make a depends on in the revert to point to the tempest fix | |
| 14:54:57 | bauzas | cool | |
| 14:55:00 | bauzas | working on now | |
| 14:55:20 | bauzas | gibi: I found another series that eventually has a change https://review.opendev.org/c/openstack/nova/+/873560 | |
| 14:55:32 | bauzas | but given it needs some cinder change... | |
| 14:56:08 | bauzas | Uggla is off this week, so I guess we won't be able to merge some changes for the virtiofs series | |
| 14:58:52 | bauzas | dansmith: gibi: I'm actually wondering, shouldn't we rather exclude the ImageLocationsTest tests in https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4ad/872413/3/check/nova-ceph-multistore/4adf1ea/testr_results.html ? | |
| 14:59:04 | bauzas | instead of making the job n-v ? | |
| 14:59:42 | dansmith | seems less invasive to just make it n-v, and it doesn't look like we have anything that is likely to fail that job anyway, from the list you posted | |
| 14:59:50 | dansmith | but up to you | |
| 15:00:59 | bauzas | ok, lemme try then | |
| 15:01:03 | dansmith | bauzas: honestly, | |
| 15:01:06 | dansmith | as broken as things are, | |
| 15:01:24 | dansmith | making it n-v reduces the chance of it going timed_out or hitting other neutron, cinder, etc failures | |
| 15:01:38 | dansmith | so n-v is more of a gain than just skipping that test | |
| 15:01:39 | bauzas | ok, then non-voting it is | |
| 15:03:35 | gibi | bauzas: so what do you suggest on https://review.opendev.org/c/openstack/nova/+/873560 ? Should I start review it or is it anyhow blocked on the cinder side? | |
| 15:04:03 | bauzas | gibi: I just wanted to explain you I found a new series to review when we have time | |
| 15:04:21 | bauzas | before today, it was said "no changes found" in the etherpad | |
| 15:08:43 | opendevreview | Sylvain Bauza proposed openstack/nova master: Swip nova-ceph-multistore job to n-v https://review.opendev.org/c/openstack/nova/+/873569 | |
| 15:08:43 | opendevreview | Sylvain Bauza proposed openstack/nova master: Revert "Swip nova-ceph-multistore job to n-v" https://review.opendev.org/c/openstack/nova/+/873590 | |
| 15:08:51 | bauzas | gibi: dansmith: ^ | |
| 15:09:02 | dansmith | Swip? | |
| 15:09:19 | bauzas | swipe* maybe ? | |
| 15:09:27 | dansmith | swap? | |
| 15:09:34 | bauzas | or swap | |
| 15:09:43 | bauzas | man lemme explain it in French | |
| 15:10:01 | dansmith | can you just fix it right quick? | |
| 15:10:12 | bauzas | 'Décaler nova-ceph-multistore comme non-votant" | |
| 15:10:12 | bauzas | 'Décaler nova-ceph-multistore comme non-votant" | |
| 15:10:14 | bauzas | :p | |
| 15:10:17 | bauzas | ok, can do | |
| 15:10:27 | dansmith | let me explain in english | |
| 15:10:35 | dansmith | no parlezvous francais | |
| 15:11:58 | opendevreview | Sylvain Bauza proposed openstack/nova master: Swap nova-ceph-multistore job to n-v https://review.opendev.org/c/openstack/nova/+/873569 | |
| 15:11:58 | opendevreview | Sylvain Bauza proposed openstack/nova master: Revert "Swap nova-ceph-multistore job to n-v" https://review.opendev.org/c/openstack/nova/+/873590 | |
| 15:12:46 | dansmith | got it | |
| 15:13:15 | bauzas | dansmith: that's sad Louis the 14th didn't sent more French people in Quebec in 1750 :p | |
| 15:13:46 | dansmith | maybe not the best time to be talking about violating sovereignty eh? :D | |
| 15:13:55 | dansmith | wait, I have a better one | |
| 15:14:00 | dansmith | bauzas: did he send them in balloons? | |
| 15:14:52 | bauzas | lol | |
| 15:15:09 | bauzas | he could have send his head in 1792 | |
| 15:15:27 | bauzas | well, it was his grand-child but meh | |
| 15:16:07 | bauzas | (and this was Louis the 15th, sorry I mistyped) | |
| 20:06:35 | opendevreview | Edward Hope-Morley proposed openstack/nova stable/yoga: add repoducer test for bug 1890244 https://review.opendev.org/c/openstack/nova/+/872663 | |
| 20:06:36 | opendevreview | Edward Hope-Morley proposed openstack/nova stable/yoga: ignore deleted server groups in validation https://review.opendev.org/c/openstack/nova/+/867989 | |
| 21:09:48 | opendevreview | Amit Uniyal proposed openstack/nova master: Added a lock_unlock dcorator for instance https://review.opendev.org/c/openstack/nova/+/873648 | |
| 21:52:18 | opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | |
| #openstack-nova - 2023-02-14 | |||
| 05:26:58 | opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | |
| 05:42:03 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 05:42:03 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 05:42:04 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 05:42:04 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 05:42:05 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 05:42:05 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 05:42:06 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 05:42:06 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 05:42:07 | opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | |
| 05:42:07 | opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | |
| 05:42:09 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 05:42:09 | opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | |
| 05:56:40 | opendevreview | Amit Uniyal proposed openstack/nova master: Added a lock_unlock dcorator for instance https://review.opendev.org/c/openstack/nova/+/873648 | |
| 08:57:10 | gibi_ | bauzas: the tempest fix laneded so I'm inclined to fast approve the revert of the ceph non voting patch https://review.opendev.org/c/openstack/nova/+/873590 | |
| 08:57:23 | bauzas | gibi_: do it fer sur | |
| 08:58:14 | gibi_ | me too | |
| 08:59:47 | bauzas | gibi_: I'm telling 'recheck Ia72b50f7f7bf64fe0ddd3f1a415b1807ff264b66 merged.' | |
| 09:16:22 | gibi | bauzas: fyi I put a -1 on https://review.opendev.org/c/openstack/nova/+/873560 as it needs the cinder deps land first but they are not in a landing state | |
| 09:17:06 | bauzas | gibi: yup, I had open comments that I didn't uploaded in Gerrit | |
| 09:18:06 | gibi | I think I did what I could now so I'm switching to my other duty for a while. feel free to ping me if my eyes are needed on something here | |
| 09:18:24 | bauzas | ++ | |
| 12:34:07 | gibi | bauzas: ahh gmann caught that the tempest fix merged before we was able to merge the ceph non voting job. So I'm abandoning that non-voting and voting again series | |
| 12:34:57 | bauzas | gibi: sec, what ? | |
| 12:46:40 | gibi | bauzas: we never merged the ceph non voting patch, as the tempest fix merged first, then gmann stopped the merging of the non voting patch | |
| 13:02:43 | bauzas | Ack ok then no worries | |
| 13:10:38 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/845748 | |
| 13:10:39 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/845753 | |
| 13:11:30 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/845754 | |
| 13:13:43 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/873575 | |
| 13:14:39 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/845748 | |
| 13:14:51 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/845753 | |
| 13:14:57 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/845754 | |
| 13:24:28 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/873576 | |
| 13:25:46 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/873577 | |
| 13:27:10 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/873578 | |