| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-30 | |||
| 12:12:37 | bauzas | oh, strangely, you aren't | |
| 12:12:39 | bauzas | wtf | |
| 12:12:57 | gibi | the multicell job as two build timeout | |
| 12:13:13 | gibi | the grenade failed on a live migration test I need to look deeper | |
| 12:13:29 | gibi | probably unrelated but I don't have a matcher for that failure yet | |
| 12:13:51 | gibi | teh multicell build timeout is due to vif plug timeout | |
| 12:13:59 | gibi | that is unrelated for sure | |
| 12:16:08 | gibi | the grenade live migration failure is due to https://bugs.launchpad.net/nova/+bug/1912310 | |
| 12:16:36 | gibi | I hit recheck | |
| 12:59:36 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/811947 | |
| 13:05:42 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/811947 | |
| 13:06:51 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/811947 | |
| 13:35:06 | bauzas | gibi: https://review.opendev.org/c/openstack/nova/+/810909/2/nova/compute/manager.py#5642 shouldn't we also revert to the original flavor in revert_resize ? | |
| 13:35:42 | gibi | bauzas: is that missing? if yes then that is a separate bug | |
| 13:35:52 | bauzas | I dunno | |
| 13:36:06 | bauzas | I'm afraid you set a field on the instance before confirming | |
| 13:36:19 | bauzas | which means a revert needs to get back | |
| 13:36:34 | gibi | I set it earlier than before, but before it was still set during resize not resize confirm | |
| 13:37:10 | bauzas | gibi: I can't see it in the left side | |
| 13:37:11 | gibi | I'm not moving this field change from resize_confirm to resize. I move it from resize_finish (which is part of resize) to resize | |
| 13:37:20 | bauzas | oh fuck | |
| 13:37:23 | bauzas | you're right | |
| 13:37:25 | bauzas | stupid me | |
| 13:37:39 | bauzas | I confused myself with the resize methods naming | |
| 13:37:40 | gibi | bauzas: sorry, yeah, first I moved, but that breaks upgrade, now I just double set | |
| 13:37:51 | gibi | check the first PS | |
| 13:38:00 | bauzas | resize_confirm != resize_finish | |
| 13:38:00 | gibi | that has the move implemented | |
| 13:38:11 | gibi | bauzas: yepp that naming is hard :D | |
| 13:38:26 | bauzas | yeah you finish a resize before you confirm it :p | |
| 13:38:56 | gibi | you finish the resize_instance RPC call on the destination :D | |
| 13:39:10 | gibi | there can be finsh_confirm_resize too | |
| 13:39:45 | gibi | but I think we dont call it like that | |
| 13:47:20 | opendevreview | Merged openstack/nova master: Reproduce bug 1944759 https://review.opendev.org/c/openstack/nova/+/810763 | |
| 13:47:51 | bauzas | gibi: yeah I know | |
| 13:48:00 | bauzas | it was my brain which fscked | |
| 13:50:59 | opendevreview | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1943431 https://review.opendev.org/c/openstack/nova/+/810755 | |
| 13:51:00 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Move _wait_for_volume_{attach,detach} to os-volume_attachments https://review.opendev.org/c/openstack/nova/+/810775 | |
| 13:51:00 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Update volume_id within connection_info during swap_volume https://review.opendev.org/c/openstack/nova/+/807025 | |
| 13:51:01 | opendevreview | Lee Yarwood proposed openstack/nova master: fup: Refactor and simplify Cinder fixture GET volume mock https://review.opendev.org/c/openstack/nova/+/810776 | |
| 13:52:07 | lyarwood | sorry are we holding +Ws until we are out of rc? | |
| 13:52:31 | gibi | lyarwood: I'm hesitant. bauzas? | |
| 13:52:51 | bauzas | holding stable patches ? | |
| 13:53:00 | bauzas | or master ones ? | |
| 13:53:05 | gibi | I think lyarwood is asking for master | |
| 13:53:06 | bauzas | former, yes | |
| 13:53:09 | bauzas | latter, nope | |
| 13:53:26 | gibi | ack, then I'm addin +A | |
| 13:53:33 | bauzas | like, I litterally sent gibi's patch to the gate | |
| 13:53:35 | gibi | to https://review.opendev.org/c/openstack/nova/+/810755 | |
| 13:53:54 | lyarwood | cool yeah I just did for the regression test that landed above | |
| 13:53:56 | bauzas | gibi: we branched Xena on RC1, right? | |
| 13:54:01 | gibi | bauzas: right | |
| 13:54:14 | bauzas | so we're officially working on Yoga with master | |
| 13:54:33 | gibi | in my head RC period is when we want to keep master and stable close for easy backport of last minute issues | |
| 13:54:43 | bauzas | that said, any backported change to stable/xena can't be approved until we deliver GA or it's a regression bugfix | |
| 13:55:11 | bauzas | gibi: today is the last RC day | |
| 13:55:17 | gibi | yepp I know | |
| 13:55:21 | bauzas | I don't think we're taking an absolute risk | |
| 13:55:28 | gibi | OK | |
| 13:55:31 | bauzas | this is tho an interesting thought | |
| 13:56:14 | bauzas | honestly, I think we should make more room for bugfixing during our release cadence | |
| 13:56:51 | bauzas | but let's discuss this at the PTG | |
| 14:02:11 | lyarwood | bug fixing never stops ;) | |
| 14:03:53 | gibi | bauzas: you mean feature freezing at F2? ;) | |
| 14:04:13 | bauzas | I don't know | |
| 14:04:21 | bauzas | I'm not opiniated about the solution | |
| 14:04:59 | bauzas | if we could somehow prioritize bugs over features during a period of time that would be explicitely and properly communicated, this would get my interest | |
| 14:05:42 | bauzas | but that also means that people thinking about new features need to understand in advance our release cadence (ideally drafting before the PTG) | |
| 14:10:11 | opendevreview | Merged openstack/nova master: Add section for 'nova-manage placement audit' tool https://review.opendev.org/c/openstack/nova/+/809479 | |
| 15:19:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Enable min pps tempest testing in nova-next https://review.opendev.org/c/openstack/nova/+/811748 | |
| 15:49:47 | bauzas | gibi: sorry was unclear with my comment on your fix, as I really didn't wanted you to invest time into functesting different compute versions, but this is possible https://github.com/openstack/nova/blob/d64edd3da2336a5c7c8f69cced45272cbaf638a9/nova/tests/functional/test_cold_migrate.py#L74 | |
| 15:50:28 | bauzas | this is just a fyk | |
| 15:51:53 | gibi | bauzas: that probably works if the old behavior is tight to an old service version. In my case there is no service version bump | |
| 15:52:21 | bauzas | then I can't remember where but I'm also sure I wrote some upgrade checks | |
| 15:52:30 | bauzas | anyway | |
| 15:53:05 | bauzas | iirc, the idea is to pin rpc versions to make the compute appear "old" | |
| 15:57:41 | gibi | yepp, if there is such old version | |
| 15:57:58 | gibi | but in my case there is no version difference between the unpatches and the patches compute | |
| 15:58:11 | gibi | unpatched and patched | |
| 15:59:46 | melwitt | elodilles: hi, what is the other patch you think might be better for fixing l-c on stable/train? re: https://review.opendev.org/c/openstack/nova/+/811762/1#message-d28da6263de9241f21a437daf6a0b7273eec47e4 | |
| 16:00:38 | melwitt | I'm not opinionated how to fix it, so if there is another way, I'm happy to rebase the patches onto that | |
| 16:01:58 | opendevreview | Merged openstack/nova master: Update contributor guide for Yoga https://review.opendev.org/c/openstack/nova/+/809936 | |
| 16:12:31 | opendevreview | Merged openstack/nova master: Add regression test for bug #1943431 https://review.opendev.org/c/openstack/nova/+/810755 | |
| 16:40:50 | elodilles | melwitt: I though about this one: https://review.opendev.org/c/openstack/nova/+/810461 | |
| 16:41:25 | elodilles | I missed that it is not yet merged o:) | |
| 16:41:48 | elodilles | so anyway, both ways work for me | |
| 17:01:18 | melwitt | elodilles: ah ok, thanks for the link. does it need an update based on the latest comments about min version? or would that be a separate patch? | |
| 17:14:13 | melwitt | elodilles: also meant to tell you I saw grenade jobs on stable/train also failing with "AttributeError: module 'jsonschema' has no attribute 'compat'" https://zuul.opendev.org/t/openstack/build/08e374906e2844ef865d4c698b3b78e5/log/job-output.txt#28564 is this also related to the setuptools issue? | |
| 17:34:40 | elodilles | melwitt: oh, I haven't seen that yet :( that most probably the very same jsonschema 4.0.0 release from yesterday that we faced in openstack-releases' validator :( | |
| 17:37:56 | elodilles | though jsonschema should be upper constrained in stable O.o | |
| 17:39:18 | melwitt | hm yeah | |
| 17:41:02 | melwitt | elodilles: oh but the error was raised from tempest, which would be coming from master | |
| 17:43:43 | elodilles | actually tempest should be used from train-last tag and still, should be installed with upper constraints. or am I wrong? | |
| 17:44:52 | elodilles | seemingly I am wrong at some point otherwise it wouldn't fail :) | |
| 17:46:34 | melwitt | well.. not sure. because if this fails here it seems like it would fail in the tempest repo itself but it's not | |
| 17:48:22 | melwitt | I see that the job began collecting jsonschema==2.6.0 but ended up installing 4.0.0 | |
| 17:48:54 | melwitt | but why... | |
| 17:49:37 | melwitt | or how. it's pinned to 3.0.2 as you said earlier https://github.com/openstack/requirements/blob/stable/train/upper-constraints.txt#L634 | |
| 17:49:53 | melwitt | collecting: https://zuul.opendev.org/t/openstack/build/08e374906e2844ef865d4c698b3b78e5/log/job-output.txt#38834 | |