| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-04-25 | |||
| 16:06:07 | bauzas | we may want to only use one :) | |
| 16:06:19 | dansmith | yeah | |
| 16:06:51 | sean-k-mooney | elodilles: your missing the depend on agains the devstack patch but those need to be squashed first | |
| 16:06:51 | bauzas | anyway, we'll sort that out of the meeting | |
| 16:06:53 | sean-k-mooney | so its fine for now | |
| 16:07:21 | elodilles | ++ | |
| 16:07:55 | bauzas | yup and thanks to dansmith and sean-k-mooney for working hardly on the ceph patches | |
| 16:08:04 | sean-k-mooney | mainly dansmith | |
| 16:08:05 | bauzas | -EOLDGREEK to me | |
| 16:08:18 | sean-k-mooney | i have been busy with other things other then checking back every now and then | |
| 16:08:27 | bauzas | (I mean, I know what ceph does and all the things, but that is a different story) | |
| 16:08:42 | sean-k-mooney | anywya we can move on | |
| 16:08:46 | bauzas | cool | |
| 16:08:51 | bauzas | https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly | |
| 16:09:01 | bauzas | unsurprisingly they are all green | |
| 16:09:15 | bauzas | but by their runs we didn't had the py39 updates :)Ã | |
| 16:09:15 | bauzas | but by their runs we didn't had the py39 updates :)à | |
| 16:09:31 | sean-k-mooney | good timing i guess | |
| 16:10:31 | bauzas | well, tooz upgraded to 4.0 the week before | |
| 16:10:32 | opendevreview | Merged openstack/nova stable/xena: Fix rescue volume-based instance https://review.opendev.org/c/openstack/nova/+/875343 | |
| 16:10:46 | bauzas | so that's fortunate we haven't merged the u-c patch during the weekend | |
| 16:10:58 | bauzas | anyway, moving on | |
| 16:11:04 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:11:09 | bauzas | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | |
| 16:11:14 | bauzas | #topic Release Planning | |
| 16:11:18 | bauzas | #link https://releases.openstack.org/bobcat/schedule.html | |
| 16:11:23 | bauzas | #info Nova deadlines are set in the above schedule | |
| 16:11:28 | bauzas | #info Bobcat-1 is in 2 weeks | |
| 16:11:51 | bauzas | (we'll have a stable branch review day on Bobcat-1 but I'll explain this the next week) | |
| 16:12:02 | bauzas | #topic Review priorities | |
| 16:12:09 | bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | |
| 16:12:14 | bauzas | #info As a reminder, cores eager to review changes can +1 to indicate their interest, +2 for committing to the review | |
| 16:12:19 | bauzas | #topic Stable Branches | |
| 16:12:28 | bauzas | elodilles: take the mic | |
| 16:12:43 | elodilles | yepp | |
| 16:12:53 | elodilles | beyond the usual stuff, | |
| 16:13:04 | elodilles | (unblocked gate & many rechecks) | |
| 16:13:19 | elodilles | auniyal prepared release patches for yoga and zed | |
| 16:13:30 | elodilles | they haven't merged yet | |
| 16:13:53 | elodilles | and meanwhile 1-1 patches merged to stable/yoga and zed | |
| 16:14:07 | auniyal | yes, got 1 +2 yet :), thanks for review elodilles | |
| 16:14:12 | elodilles | otherwise they are good as they are | |
| 16:14:26 | elodilles | auniyal: thanks for proposing the patches :) | |
| 16:14:41 | bauzas | cool | |
| 16:15:54 | bauzas | anything else ? | |
| 16:16:04 | elodilles | nope, i think that was all | |
| 16:16:07 | elodilles | from my side | |
| 16:16:15 | elodilles | sorry :) | |
| 16:17:10 | bauzas | cool, moving on | |
| 16:17:19 | bauzas | #topic Open discussion | |
| 16:17:29 | bauzas | (ykarel) Allow to add tb-cache size libvirt option for qemu, context https://bugs.launchpad.net/nova/+bug/1949606 | |
| 16:17:33 | bauzas | ykarel: go for it | |
| 16:19:20 | ykarel | hi | |
| 16:19:41 | ykarel | so qemu-5.0.0(included in ubuntu jammy) update default tb cache size to 1 GiB(from 32 MiB) for system-emulated guest vms and with that each guest VM is using much more system memory(1 GB+ memory for guest), resulting into oom-kill issues when creating multiple guest vms concurrently in neutron scenario jobs using ubuntu guest vms | |
| 16:20:17 | ykarel | libvirt-8.0.0 added an option to configure it per guest vm | |
| 16:20:25 | ykarel | Currently testing WIP nova patch https://review.opendev.org/c/openstack/nova/+/868419 in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/881391 | |
| 16:20:46 | ykarel | should this be a specless RFE or can continue as a bug? | |
| 16:20:54 | sean-k-mooney | ykarel: did you file the specless pluprint i asked for | |
| 16:20:56 | sean-k-mooney | ah | |
| 16:20:58 | sean-k-mooney | its not a bug | |
| 16:21:19 | ykarel | sean-k-mooney, i just added to meeting agenda to get clearity | |
| 16:21:29 | ykarel | but if it's a way to go then i will file it | |
| 16:21:30 | sean-k-mooney | we might conider it to be a small enough workaround that we might want to backport it upstream | |
| 16:21:33 | sean-k-mooney | to enable ci testing | |
| 16:22:27 | ykarel | hmm mainly would be needed for releases running on jammy | |
| 16:22:30 | sean-k-mooney | the issue is that i dont think 22.04 has libvirt 8.0.0 | |
| 16:22:37 | ykarel | it has | |
| 16:22:44 | ykarel | that's what we testing | |
| 16:22:54 | sean-k-mooney | ah ok | |
| 16:23:07 | bauzas | we damn need update of https://docs.openstack.org/nova/latest/reference/libvirt-distro-support-matrix.html | |
| 16:23:19 | sean-k-mooney | yes | |
| 16:23:30 | sean-k-mooney | so your patch is mising a min libvirt version check | |
| 16:23:33 | bauzas | we're still having libvirt 6.0.0 as a min, right? | |
| 16:23:35 | sean-k-mooney | so that will need to be added | |
| 16:23:40 | ykarel | from job logs libvirt-daemon 8.0.0-1ubuntu7.4 | |
| 16:23:55 | sean-k-mooney | bauzas: yes and im hoping to bump that to 7.0.0 this release | |
| 16:23:58 | ykarel | sean-k-mooney, yes will include that in next update | |
| 16:24:10 | sean-k-mooney | but we will sitll need to check 8.0.0 supprot | |
| 16:24:15 | sean-k-mooney | ack | |
| 16:24:16 | bauzas | sean-k-mooney: but we would still need to only set it if libvirt >=8 | |
| 16:24:32 | bauzas | do we want to schedule on it, or is it just a performance fix ? | |
| 16:24:34 | sean-k-mooney | ya either only set it or preferable check this in init-host | |
| 16:24:43 | sean-k-mooney | and raise an error if you set the config option on an old libvirt | |
| 16:24:58 | bauzas | will there be any config knob ? | |
| 16:25:07 | sean-k-mooney | yes there shoudl be | |
| 16:25:12 | ykarel | yes that's second question | |
| 16:25:14 | sean-k-mooney | to set the cache size | |
| 16:25:15 | ykarel | Default setting for this new option, unconfigured or set to defaults like 32MB or 128 MB etc? | |
| 16:26:07 | bauzas | ykarel: if that's a config option, that indeed needs to be an hard fail if the operators sets this config value | |
| 16:26:18 | bauzas | and libvirt isn't recent enough | |
| 16:26:23 | sean-k-mooney | its not somethign we should hardcode | |
| 16:26:28 | ykarel | bauzas, sure will take care that in the patch | |
| 16:26:29 | bauzas | for that reason, I'm not happy with a default value except none | |
| 16:26:47 | sean-k-mooney | so it shoudl eb a config option im ok with a low default or leaving it unset | |
| 16:27:02 | bauzas | I'd prefer it unset for upgrade reasons | |
| 16:27:04 | ykarel | okk Thanks will keep it like that no default and let user configure it | |
| 16:27:05 | sean-k-mooney | bauzas: to your poitn it woudl be nice to have a triat for this but not required | |
| 16:27:28 | bauzas | ykarel: s/user/operator but I think I get your point | |
| 16:27:50 | ykarel | so will propose the blueprint and update the patch with as per all the suggestions, Thanks | |
| 16:27:50 | sean-k-mooney | in this case really it will be the zuul job | |