| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-13 | |||
| 16:56:12 | elodilles | the usual things ^^^ | |
| 16:56:16 | elodilles | that's it | |
| 16:56:39 | bauzas | elodilles: fancy writing an email to the world explaining the gate blocker ? | |
| 16:56:53 | elodilles | bauzas: yepp, i can do that | |
| 16:57:03 | bauzas | I just hope someone will just magically wake up and say he'll look | |
| 16:57:27 | bauzas | elodilles: all projects being impacted by the sdk, you said ? | |
| 16:57:42 | elodilles | not all | |
| 16:57:49 | elodilles | but i've seen some others | |
| 16:57:51 | sean-k-mooney | is that the storage cleanpu failure | |
| 16:57:57 | sean-k-mooney | in the functional job | |
| 16:58:03 | sean-k-mooney | where its failing to delete the volume | |
| 16:58:09 | sean-k-mooney | becuase the snapshot is not deleted | |
| 16:58:29 | sean-k-mooney | ah it is test_block_storage_cleanup | |
| 16:58:30 | elodilles | yepp, test_block_storage_cleanup | |
| 16:58:42 | sean-k-mooney | ya so they trided to fix it by doing it in a loop twice | |
| 16:58:55 | sean-k-mooney | but that obviouly does not work | |
| 16:59:08 | sean-k-mooney | so the test is flaky | |
| 16:59:14 | elodilles | oh, so it's a known issue :-o | |
| 16:59:18 | bauzas | => remove the flakey test then | |
| 16:59:27 | sean-k-mooney | ill see if i can get the patch | |
| 16:59:32 | bauzas | and unblock the gate | |
| 16:59:44 | sean-k-mooney | we cant currently filter those | |
| 16:59:48 | sean-k-mooney | via the job config | |
| 17:00:25 | bauzas | anyway, we're on time | |
| 17:00:36 | sean-k-mooney | efoley and i https://review.opendev.org/c/openstack/openstacksdk/+/856509 tried to fix it by backporting that | |
| 17:00:47 | sean-k-mooney | but then we realised it always uses master | |
| 17:00:49 | bauzas | let's wrap this meeting and continue the stable/yoga discussion right after if you want | |
| 17:00:59 | bauzas | #topic Open discussion | |
| 17:01:01 | bauzas | nothing | |
| 17:01:02 | sean-k-mooney | so https://review.opendev.org/c/openstack/openstacksdk/+/852992 was ment to fix it but does not | |
| 17:01:14 | JayF | Might be worth noting for meeting notes purposes that folks running Ironic back as far as train should update their Nova checkout | |
| 17:01:22 | JayF | as all known fixes for the Ironic driver have been backported | |
| 17:01:24 | opendevreview | Merged openstack/osc-placement stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856784 | |
| 17:01:26 | opendevreview | Merged openstack/osc-placement stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856785 | |
| 17:01:45 | JayF | (and I know for em branches we don't do releases) | |
| 17:01:58 | bauzas | JayF: want me to offically set an #info ? | |
| 17:01:59 | bauzas | :) | |
| 17:02:08 | JayF | bauzas: if you think anyone who cares would read it :D | |
| 17:02:29 | sean-k-mooney | elodilles: i just opened a revert https://review.opendev.org/c/openstack/openstacksdk/+/857471 | |
| 17:02:52 | sean-k-mooney | the https://review.opendev.org/c/openstack/openstacksdk/+/852992 fixes an issue and intoduces the flaky test | |
| 17:02:58 | sean-k-mooney | so maybe a partial revert | |
| 17:03:11 | sean-k-mooney | keep the fix but revert the new test caes | |
| 17:03:15 | bauzas | #info Ironic operators running train or later are more than welcome to upgrade their nova checkout with latest stable releases since bugfixes are released now | |
| 17:03:24 | bauzas | JayF: ^ happy ? :) | |
| 17:03:28 | JayF | thank you :D | |
| 17:03:31 | elodilles | sean-k-mooney: ack, thanks for the info! i'll look at them then | |
| 17:03:51 | gibi | (it feels like we have to parallel meeting both overrun its time) | |
| 17:03:53 | bauzas | JayF: people reading our notes are beasts I don't know | |
| 17:03:57 | gibi | *two | |
| 17:04:12 | bauzas | gibi: well, I'll offload you some task | |
| 17:04:16 | bauzas | thanks all | |
| 17:04:20 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-13-16.00.log.html | |
| 17:04:20 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-13-16.00.txt | |
| 17:04:20 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-13-16.00.html | |
| 17:04:20 | opendevmeet | Meeting ended Tue Sep 13 17:04:20 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:04:20 | bauzas | #endmeeting | |
| 17:04:27 | elodilles | thanks bauzas o/ | |
| 17:04:41 | gibi | bauzas, sean-k-mooney: fyi I -1 on https://review.opendev.org/c/openstack/nova/+/856895 It think that patch does more what we need | |
| 17:04:53 | gibi | but I have to drop now | |
| 17:04:59 | gibi | ping me tomorrow if the comment is not clear | |
| 17:05:04 | gibi | o/ | |
| 17:05:38 | sean-k-mooney | afck | |
| 17:05:40 | sean-k-mooney | ack | |
| 17:06:11 | elodilles | sean-k-mooney: if the original fix did not merge into yoga, then how is it causing failing tests? :-o or is that only some part of the fix? | |
| 17:07:04 | elodilles | sean-k-mooney: sorry, it seems i'm a bit lost there o:) | |
| 17:07:42 | sean-k-mooney | we proposed it in yoga but then realised that there is a branch overried | |
| 17:07:47 | sean-k-mooney | so emmas patch is not needed | |
| 17:07:58 | sean-k-mooney | even though the sdk is branched | |
| 17:08:08 | elodilles | sean-k-mooney: oh, i see! as sdk is taken from master branch | |
| 17:08:11 | sean-k-mooney | the sdk jobs always use master sdk on any stable branch | |
| 17:08:15 | sean-k-mooney | yep | |
| 17:08:22 | sean-k-mooney | so that is why a mager change broke stable | |
| 17:08:24 | elodilles | sean-k-mooney: sorry :S now i get it o:) | |
| 17:11:58 | sean-k-mooney | so we might want to drop https://review.opendev.org/c/openstack/openstacksdk/+/857471/1/openstack/block_storage/v3/_proxy.py but https://review.opendev.org/c/openstack/openstacksdk/+/857471/1/openstack/tests/functional/cloud/test_project_cleanup.py is the flaky test that we should revert out | |
| 17:12:16 | sean-k-mooney | ill do that and rebase it to the tip of master | |
| 17:14:52 | elodilles | sean-k-mooney: thanks! | |
| 17:20:07 | sean-k-mooney | https://review.opendev.org/c/openstack/openstacksdk/+/857471 | |
| 17:20:15 | sean-k-mooney | ok so that is just removing the test cases now | |
| 17:20:45 | sean-k-mooney | the retry they added is preserved to not regress https://storyboard.openstack.org/#!/story/2010217 | |
| 17:23:10 | elodilles | sean-k-mooney: ++ | |
| 17:30:24 | sean-k-mooney | bauzas: by the way while im going to attent the ptg i dont plan on addign any topics this time to the adgenda | |
| 17:32:38 | sean-k-mooney | dansmith: by the way does your n-2 greade job still work | |
| 17:33:13 | sean-k-mooney | if so we should likely add that to the check pipline or the perodic pipelien one RC1 is out using yoga as a base | |
| 17:33:43 | sean-k-mooney | so weekly at a minium but we could run it on each patch if we wanted too | |
| 17:34:09 | sean-k-mooney | thats proably a ptg topic i guess | |
| 17:34:55 | sean-k-mooney | i.e. how and what level of testing we wil do in A as part of the dress rehersal for C | |
| 17:36:08 | sean-k-mooney | is it the grenade-skip-level: jobs | |
| 17:36:17 | sean-k-mooney | https://github.com/openstack/nova/blob/master/.zuul.yaml#L705-L706 | |
| 17:37:38 | sean-k-mooney | ya it is https://opendev.org/openstack/grenade/src/branch/master/.zuul.yaml#L377 | |
| 17:38:19 | sean-k-mooney | so we wil need a variant for tha tthat is based on yoga as the base and master as the target for A | |
| 17:38:42 | sean-k-mooney | its proably best to do that in hte grenade repo | |
| 17:42:20 | dansmith | sean-k-mooney: we won't know until we start running it again, but it's supposed to | |
| 17:42:38 | dansmith | but we shouldn't be running it on zed-rc right? just on master (antelope) right? | |
| 17:42:46 | dansmith | oh, "once rc1 is out" yeah | |
| 17:42:47 | sean-k-mooney | yep | |
| 17:42:57 | sean-k-mooney | so from friday or next week | |
| 17:43:03 | sean-k-mooney | we can trun it back on on master | |
| 17:43:08 | sean-k-mooney | and pin to yoga | |
| 17:43:12 | sean-k-mooney | as teh from branch | |
| 17:43:24 | sean-k-mooney | its currently wallaby | |