| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-13 | |||
| 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 | |
| 17:43:38 | sean-k-mooney | for xena | |
| 17:43:49 | sean-k-mooney | sorry for yoga | |
| 17:44:10 | sean-k-mooney | letters are hard hehe | |
| 17:44:20 | dansmith | yeah | |
| 18:27:42 | opendevreview | Merged openstack/python-novaclient master: Update master for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856790 | |
| #openstack-nova - 2022-09-14 | |||
| 07:38:47 | obre | Hi all! I wonder if it is possible for me to propse a small change to nova? In short I would like to add a config-option allowing us to specify the value nova-compute reports to placement for VCPU:max_unit. The use-case is to avoid having instances consuming all CPU's of a compute-node (ex: I do not want instances using 24 cores to end up on my hypervisor with 24 cores; Id | |
| 07:38:50 | obre | rather want them on larger hypervisors). Does it make sense? | |
| 07:40:13 | obre | I am able to write the changes needed in nova/virt/libvirt/driver.py and nova/conf/compute.py, I just wonder if this is something that makes sense for me to try push upstream; or if there are obvious blockers I dont see? | |
| 07:40:56 | gibi | obre: I'm wondering. it might be already possible with a provider.yaml file https://docs.openstack.org/nova/latest/admin/managing-resource-providers.html | |
| 07:41:16 | gibi | but if not, then I would enhance that facility to configure max_unit | |