| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-28 | |||
| 17:57:05 | gmann | parent job | |
| 17:57:31 | dansmith | ah, voting on the job def and not the queue? but why is the regular job voting? | |
| 17:57:53 | gmann | dansmith: regular job ? | |
| 17:58:11 | dansmith | the regular ceph job | |
| 17:58:23 | dansmith | maybe it doesn't inherit from multinode? | |
| 17:58:29 | gmann | yeah, multinode | |
| 17:58:34 | dansmith | ack | |
| 17:58:46 | dansmith | I think we could take that out of the gate queue probably | |
| 17:59:02 | gmann | yeah we should do that until that is voting | |
| 17:59:10 | dansmith | ack | |
| 17:59:22 | gmann | and if no plan to fix in nearby may be move to periodic ? | |
| 17:59:37 | gmann | running known failing non voting in check pipeline also costly | |
| 18:00:41 | opendevreview | Dan Smith proposed openstack/nova master: Remove n-v ceph live migration job from gate https://review.opendev.org/c/openstack/nova/+/889992 | |
| 18:00:45 | dansmith | it doesn't fail that I know of | |
| 18:01:12 | dansmith | even if it's spotty, it's good to have it in check, but I think it's more just because it has been unstable in the past, and is not yet converted to cephadm, etc | |
| 18:01:31 | dansmith | I think it's also reasonable to just not gate on it anyway, even if it's voting in check | |
| 18:02:18 | dansmith | given that we require clean rechecks, I think we could probably trim down our gate jobs a bit anyway and reduce the number of fails we have there | |
| 18:02:41 | dansmith | the chances of threading the needle between check and gate for something that works on regular lm and not with ceph, that changed between check and gate is pretty dang unlikely, IMHO | |
| 18:05:50 | gmann | ack | |
| 18:19:04 | dansmith | melwitt: I can haz? https://review.opendev.org/c/openstack/nova/+/889992 | |
| 18:20:03 | melwitt | sure why not | |
| 18:22:58 | dansmith | thanks | |
| 18:34:30 | dansmith | damn | |
| 18:34:37 | dansmith | I thought we were going to get three patches to merge | |
| 18:34:52 | dansmith | but no.. one failure in one job at the last minute, of course | |
| 18:37:00 | opendevreview | Merged openstack/nova master: Online migrate missing Instance.compute_id fields https://review.opendev.org/c/openstack/nova/+/879905 | |
| 18:37:04 | dansmith | whatever, I'll take it | |
| #openstack-nova - 2023-07-29 | |||
| 23:25:20 | gmann | btw, skipping of get console tests failing on lvm is merged | |
| #openstack-nova - 2023-07-31 | |||
| 10:17:57 | opendevreview | Merged openstack/nova stable/2023.1: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888091 | |
| 12:03:11 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 13:09:34 | opendevreview | Amit Uniyal proposed openstack/nova master: Allow swap resize from non-zero to zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 13:12:27 | opendevreview | Merged openstack/nova stable/2023.1: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888092 | |
| 13:53:48 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 15:47:36 | opendevreview | Merged openstack/nova stable/2023.1: Fix failed count for anti-affinity check https://review.opendev.org/c/openstack/nova/+/885343 | |
| 16:13:14 | opendevreview | Dan Smith proposed openstack/nova master: Remove n-v ceph live migration job from gate https://review.opendev.org/c/openstack/nova/+/889992 | |
| 19:00:56 | opendevreview | Merged openstack/nova stable/zed: Have host look for CPU controller of cgroupsv2 location. https://review.opendev.org/c/openstack/nova/+/882914 | |
| 19:13:04 | opendevreview | Merged openstack/nova stable/2023.1: Unbind port when offloading a shelved instance https://review.opendev.org/c/openstack/nova/+/887183 | |
| 19:17:10 | opendevreview | Felipe Reyes proposed openstack/nova master: Add RISCV64 QEMU emulation support. https://review.opendev.org/c/openstack/nova/+/889137 | |
| #openstack-nova - 2023-08-01 | |||
| 03:00:46 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 03:00:47 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 03:00:47 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 03:00:48 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 03:00:48 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 03:00:49 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 03:00:49 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 03:00:50 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 03:00:51 | opendevreview | melanie witt proposed openstack/nova master: WIP Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | |
| 03:00:51 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild and unshelve with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870939 | |
| 03:00:53 | opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 03:00:53 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 03:00:55 | opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 03:10:15 | opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd https://review.opendev.org/c/openstack/nova/+/862416 | |
| 06:09:14 | opendevreview | Amit Uniyal proposed openstack/nova master: [WIP] add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015 | |
| 06:09:15 | opendevreview | Amit Uniyal proposed openstack/nova master: [WIP] add healthcheck tracker to nova context https://review.opendev.org/c/openstack/nova/+/829468 | |
| 06:09:15 | opendevreview | Amit Uniyal proposed openstack/nova master: [WIP] add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | |
| 06:09:16 | opendevreview | Amit Uniyal proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | |
| 06:09:16 | opendevreview | Amit Uniyal proposed openstack/nova master: [WIP] add healthcheck utils and constants https://review.opendev.org/c/openstack/nova/+/829469 | |
| 08:19:58 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 10:40:13 | dvo-plv | sean-k-mooney, Uggla Hello. Could you please make addotional review after merge conflict fix https://review.opendev.org/c/openstack/nova/+/876075 | |
| 10:57:48 | sean-k-mooney | dvo-plv: still looks ok to me | |
| 10:58:54 | dvo-plv | Do i have to get one more vote for merge ? | |
| 11:00:50 | sean-k-mooney | yes perhasp bauzas can review it before they go on pto. otherwise dansmith or melwitt will be on later or gmann. | |
| 11:01:37 | dvo-plv | bauzas, dansmith melwitt Maybe you will have a chance to review patchset ? https://review.opendev.org/c/openstack/nova/+/876075 | |
| 11:16:07 | frickler | I have a follow-up question to last weeks console log discussion: how can I check whether libvirtd is actually using virtlogd? so far the only test I've found is to write lots of data to the log and see if it gets cycled at 2M size | |
| 11:16:47 | frickler | (and I found that it doesn't work in kolla because virtlogd isn't running there, but that's a deployment issue) | |
| 11:30:06 | sean-k-mooney | you can configure it in qemu.conf | |
| 11:31:13 | sean-k-mooney | stdio_handler = "file" or stdio_handler = "logd" | |
| 11:31:40 | sean-k-mooney | # The backend to use for handling stdout/stderr output from | |
| 11:31:42 | sean-k-mooney | # QEMU processes. | |
| 11:31:52 | sean-k-mooney | although that may not be entirly correct | |
| 11:32:30 | sean-k-mooney | that might just handel the instance log al /var/log/libvirt/qemu/instnace-* | |
| 11:43:46 | sean-k-mooney | frickler: looking at the libvirt code i think stdio_handler in qemu.conf is what contols this | |
| 11:43:56 | sean-k-mooney | i have not found anything else that appears relevent | |
| 11:50:11 | opendevreview | Merged openstack/nova master: Add nova-tox-functional-py310 to gate jobs https://review.opendev.org/c/openstack/nova/+/881339 | |
| 11:50:46 | sean-k-mooney | odd somethign merged. | |
| 15:54:54 | auniyal | HI sean-k-mooney, can you please review and merge stable branch patches from here https://etherpad.opendev.org/p/release-liaison-PatchesToReview | |
| 15:57:12 | auniyal | elodilles- can we merge this https://review.opendev.org/c/openstack/nova/+/877500 ? | |
| 16:03:25 | sean-k-mooney | i have limited review time so i can but i wont be able to prioritsie it. ill review as i get time | |
| 16:07:41 | auniyal | ack, | |
| 17:49:54 | opendevreview | Merged openstack/nova stable/zed: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882315 | |
| 18:32:46 | frickler | melwitt: is there anything missing except reviews on https://review.opendev.org/c/openstack/nova/+/888513? got a customer that seems affected by this, too | |
| 18:37:18 | melwitt | frickler: nope, just needs reviews. it doesn't have test coverage but that's just bc I couldn't think of a meaningful test for it. I'll add a comment on the review to that effect | |
| 18:43:23 | frickler | yes, I don't see how you could test that, either. fwiw we saw this after upgrading y->z | |
| 18:48:23 | sean-k-mooney | without poisoning the decorator to detech nesting or somehting like that im not sure iether | |
| 18:49:04 | sean-k-mooney | that said i feel like this si reversing our normal pattern | |
| 18:50:01 | sean-k-mooney | i.e. we normally decorat the generic inner function and then call it form the less geneeric singel use functions | |
| 18:50:45 | melwitt | the generic function isn't executing the query though. it's just returning a query object. you need the session manager to wrap the actual query execution | |
| 18:51:06 | sean-k-mooney | ya which makes this diffent then our normal pattern | |
| 18:51:20 | sean-k-mooney | so i think the pathc makes sense | |
| 18:52:00 | sean-k-mooney | your moving the reader decorator to where the execute happens | |
| 18:54:22 | melwitt | oh, yeah. I agree the query building helper function is different than the usual pattern (which is probably part of why this happened) | |
| 18:55:38 | melwitt | that said, it's been like this for a very long time so I'm not sure what more recent thing could have made the problem occur more often, or otherwise got people to report it | |
| 18:56:54 | sean-k-mooney | the only way i cna think to test this is to make _flavor_get_query_from_db a spy and assert that the read context is heald before and after callign it | |
| 18:58:51 | sean-k-mooney | we are usign this context manager as a decorrator correct https://github.com/openstack/nova/blob/a6126881244da66edc59028ea0ef21741c6fd492/nova/db/api/api.py#L23 | |
| 18:58:59 | melwitt | yeah.. I was kinda thinking if there's a way we could test for this generically like for everything. something like a spy | |
| 18:59:39 | frickler | the only change on that file from yoga to zed is https://review.opendev.org/c/openstack/nova/+/837156/1/nova/objects/flavor.py , but I don't understand enough alchemy to see how that might affect things | |
| 19:00:10 | frickler | might of course also be something happening somewhere else | |
| 19:00:26 | sean-k-mooney | frickler this is more likely to be realted to sqlachemy version i think | |
| 19:17:59 | sean-k-mooney | melwitt: frickler maybe im too tired to think of a clear test case but i dont see anything wrong with the patch so +2 | |
| 19:19:15 | melwitt | thanks sean-k-mooney | |