| opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | 01:49 |
|---|---|---|
| opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Add 'COMPUTE_ADDRESS_SPACE_*' traits support https://review.opendev.org/c/openstack/nova/+/873221 | 02:33 |
| *** blarnath is now known as d34dh0r53 | 02:59 | |
| opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | 04:12 |
| ihti[m] | Hi, question: Does anyone know why we are using warnings module instead of LOG.warning() e.g for deprecations in oslo_policy. It breaks logs parsing for us, as the warnings emitted deviates from the standard logging formatting. | 07:58 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Remove "see nova-manage.log" string from console https://review.opendev.org/c/openstack/nova/+/873496 | 08:09 |
| *** ralonsoh_ is now known as ralonsoh | 08:37 | |
| bauzas | morning Nova | 08:38 |
| gibi | bauzas: o/ | 08:53 |
| bauzas | currently writing some PTL nomination email, but will be back in 20 mins | 08:54 |
| gibi | ack | 09:07 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Look for cpu controller on cgroups v2 https://review.opendev.org/c/openstack/nova/+/873127 | 09:15 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Remove "see nova-manage.log" string from console https://review.opendev.org/c/openstack/nova/+/873496 | 10:33 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Look for cpu controller on cgroups v2 https://review.opendev.org/c/openstack/nova/+/873127 | 10:38 |
| bauzas | gibi: can't imagine https://review.opendev.org/c/openstack/tempest/+/873300 is still not merged | 10:58 |
| * bauzas is having his mind blown | 10:58 | |
| kashyap | There is an emoji for that in Slack! | 11:05 |
| kashyap | 🤯 | 11:06 |
| bauzas | gibi: https://zuul.openstack.org/status#873300 will fuckingly fuckering fucker fail again | 11:17 |
| bauzas | gibi: we may need to skip the ceph-multistore job by today if we wanna just merge some stuff in nova | 11:24 |
| sean-k-mooney | bauzas: push a patch to make it non voting and add those test to the exclude regex and you can ping me to help merge it if you like | 11:41 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Look for cpu controller on cgroups v2 https://review.opendev.org/c/openstack/nova/+/873127 | 11:43 |
| *** priteau_ is now known as priteau | 12:25 | |
| opendevreview | Edward Hope-Morley proposed openstack/nova stable/yoga: ignore deleted server groups in validation https://review.opendev.org/c/openstack/nova/+/867989 | 12:34 |
| opendevreview | Konrad Gube proposed openstack/nova master: Use Cinder's os-extend_volume_completion volume action. https://review.opendev.org/c/openstack/nova/+/873560 | 13:11 |
| gibi | bauzas: we can make ceph non voting for now, but it does not solve the issue that it seems to be impossible to merge anything in tempest now | 13:34 |
| bauzas | gibi: let's see what's the latest result then | 13:34 |
| *** dasm|off is now known as dasm | 13:35 | |
| bauzas | yet again a f.f.f. failure https://review.opendev.org/c/openstack/tempest/+/873300 | 13:35 |
| * bauzas needs to hold for a sec, be back in 10 mins | 13:36 | |
| *** haleyb_ is now known as haleyb | 13:36 | |
| gibi | bauzas: it seems you commented recheck at 12:23 and zuul voted at 12:37 so I guess you rechecked before the job finished | 13:36 |
| gibi | I queued up again | 13:42 |
| opendevreview | Amit Uniyal proposed openstack/nova master: nova-manage volume_attachment issues https://review.opendev.org/c/openstack/nova/+/873561 | 13:53 |
| opendevreview | Amit Uniyal proposed openstack/nova master: nova-manage volume_attachment issues https://review.opendev.org/c/openstack/nova/+/873561 | 14:00 |
| dansmith | bauzas: I wonder if we're getting to the point where we should make the ceph job n-v | 14:11 |
| dansmith | I really hate to do that, but this is kinda nuts | 14:11 |
| bauzas | dansmith: yeah, I dunno what to say | 14:11 |
| dansmith | there are other such fixes in the queue and we've been unable to land them over the weekend as well | 14:11 |
| bauzas | yup | 14:12 |
| bauzas | so, how could we merge the changes that were accepted ? that's only my concern :) | 14:13 |
| dansmith | the nova changes you mean? making the ceph job n-v at least gives you a chance to pass the other jobs | 14:14 |
| dansmith | it'll still be a struggle, but.. | 14:14 |
| gibi | dansmith, bauzas: can we ask infra to promot the tempest fix by skiping the gate? Or is it too dangerous? | 14:43 |
| dansmith | we could, but I think given that it's not landing due to all the other broken, it's not really the emergent situation that normally demands that sort of thing | 14:44 |
| 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:44 |
| bauzas | https://zuul.openstack.org/status#873300 looks like tempest-full-py3 can try twice | 14:47 |
| bauzas | gibi: dansmith: so, given the time we have before FF, what should we be doing ? | 14:49 |
| bauzas | punting the FF ? | 14:49 |
| bauzas | or awaiting for the change to be merged ? | 14:49 |
| 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:49 |
| bauzas | or non-voting the job ? | 14:50 |
| dansmith | but you'll still suffer all the other issues | 14:50 |
| dansmith | how much is still needing to land for FF? | 14:50 |
| bauzas | https://etherpad.opendev.org/p/nova-antelope-blueprint-status at least two series | 14:50 |
| bauzas | + https://review.opendev.org/c/openstack/nova/+/872413 | 14:51 |
| gibi | I'm OK to make the ceph job non voting to land the pending nova series | 14:53 |
| bauzas | ok, then I'll create the change | 14:54 |
| bauzas | and I'll also create a revert one | 14:54 |
| dansmith | just make the revert one depends-on the fix and we can go ahead and have it pre-approved | 14:54 |
| gibi | yeah and you can make a depends on in the revert to point to the tempest fix | 14:54 |
| bauzas | cool | 14:54 |
| bauzas | working on now | 14:55 |
| bauzas | gibi: I found another series that eventually has a change https://review.opendev.org/c/openstack/nova/+/873560 | 14:55 |
| bauzas | but given it needs some cinder change... | 14:55 |
| bauzas | Uggla is off this week, so I guess we won't be able to merge some changes for the virtiofs series | 14:56 |
| 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:58 |
| bauzas | instead of making the job n-v ? | 14:59 |
| 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 |
| dansmith | but up to you | 14:59 |
| bauzas | ok, lemme try then | 15:00 |
| dansmith | bauzas: honestly, | 15:01 |
| dansmith | as broken as things are, | 15:01 |
| dansmith | making it n-v reduces the chance of it going timed_out or hitting other neutron, cinder, etc failures | 15:01 |
| dansmith | so n-v is more of a gain than just skipping that test | 15:01 |
| bauzas | ok, then non-voting it is | 15:01 |
| 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:03 |
| bauzas | gibi: I just wanted to explain you I found a new series to review when we have time | 15:04 |
| bauzas | before today, it was said "no changes found" in the etherpad | 15:04 |
| 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 |
| 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 |
| bauzas | gibi: dansmith: ^ | 15:08 |
| dansmith | Swip? | 15:09 |
| bauzas | swipe* maybe ? | 15:09 |
| dansmith | swap? | 15:09 |
| bauzas | or swap | 15:09 |
| bauzas | man lemme explain it in French | 15:09 |
| dansmith | can you just fix it right quick? | 15:10 |
| * dansmith steadies his +2 finger | 15:10 | |
| bauzas | 'Décaler nova-ceph-multistore comme non-votant" | 15:10 |
| bauzas | :p | 15:10 |
| bauzas | ok, can do | 15:10 |
| dansmith | let me explain in english | 15:10 |
| dansmith | no parlezvous francais | 15:10 |
| 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 |
| 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:11 |
| dansmith | got it | 15:12 |
| bauzas | dansmith: that's sad Louis the 14th didn't sent more French people in Quebec in 1750 :p | 15:13 |
| dansmith | maybe not the best time to be talking about violating sovereignty eh? :D | 15:13 |
| dansmith | wait, I have a better one | 15:13 |
| dansmith | bauzas: did he send them in balloons? | 15:14 |
| bauzas | lol | 15:14 |
| bauzas | he could have send his head in 1792 | 15:15 |
| bauzas | well, it was his grand-child but meh | 15:15 |
| bauzas | (and this was Louis the 15th, sorry I mistyped) | 15:16 |
| *** jamesdenton_ is now known as jamesdenton | 16:56 | |
| *** jgwentworth is now known as melwitt | 18:33 | |
| 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 |
| opendevreview | Edward Hope-Morley proposed openstack/nova stable/yoga: ignore deleted server groups in validation https://review.opendev.org/c/openstack/nova/+/867989 | 20:06 |
| *** JohnnyW0 is now known as JohnnyW | 20:19 | |
| opendevreview | Amit Uniyal proposed openstack/nova master: Added a lock_unlock dcorator for instance https://review.opendev.org/c/openstack/nova/+/873648 | 21:09 |
| *** iurygregory_ is now known as iurygregory | 21:22 | |
| opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | 21:52 |
| *** dasm is now known as dasm|off | 22:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!