| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-24 | |||
| 19:34:22 | sean-k-mooney | those seam to be listed as merconfict with some of the eventlet serise. is tehre a planned merge order betweeen teh two? | |
| 19:34:46 | sean-k-mooney | or a plan to stack them to avoid that? | |
| 19:35:06 | gmaan | oh, I did not see in gerrit. | |
| 19:35:25 | gmaan | where did you see merge conflict | |
| 19:36:03 | sean-k-mooney | there isnt one yet but if you look at the merge conflicts listed on https://review.opendev.org/c/openstack/nova/+/975588/26 | |
| 19:36:26 | sean-k-mooney | it will conflict with https://review.opendev.org/c/openstack/nova/+/965467 and the two patches below it | |
| 19:36:56 | gmaan | I think that is only for https://review.opendev.org/c/openstack/nova/+/977582 | |
| 19:37:32 | gmaan | but I need to check with gibi if we need to plan that as part of this cycle of can we do in next cycle | |
| 19:37:51 | gmaan | other one should be fine | |
| 19:37:59 | sean-k-mooney | well i think its form https://review.opendev.org/c/openstack/nova/+/975833 | |
| 19:38:04 | gmaan | I do not see any other changing same file | |
| 19:38:08 | sean-k-mooney | and the other two just depend on that | |
| 19:38:52 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/975833 and https://review.opendev.org/c/openstack/nova/+/975588 are both changing the comptue manger | |
| 19:39:24 | gmaan | yes but i do not think there will be merge conflict as both changing different lines | |
| 19:39:36 | gmaan | not sure if gerrit showing correctly but if it happens we can resolve those | |
| 19:39:45 | gmaan | let's see how it goes in order | |
| 19:39:52 | sean-k-mooney | ok well i could just review https://review.opendev.org/c/openstack/nova/+/975833 now and appove it i have reviewd the other two and we could finish that work for the cycle | |
| 19:40:09 | sean-k-mooney | and the i can try and look at your serise tomorrow if you remined me | |
| 19:40:12 | gmaan | sean-k-mooney: yeah, i was typing the same :) | |
| 19:40:42 | gmaan | if we can merge 975833, it will be more clear | |
| 19:41:26 | sean-k-mooney | cool ill look at that now. im around for the next 20 mins | |
| 19:41:36 | gmaan | perfect, thanks | |
| 19:41:51 | gmaan | now time for openapi series, let see how much i can finish | |
| 19:45:17 | sean-k-mooney | ok assuming the recheck for the kernel panic goes through those 3 eventelt patche should be ready to merge | |
| 19:45:58 | gmaan | cool | |
| 19:46:07 | sean-k-mooney | ill try and do a pass on https://review.opendev.org/q/topic:%22bp/nova-services-graceful-shutdown-part1%22+is:open tomorrow | |
| 19:46:15 | gmaan | perfect, thanks | |
| 19:47:05 | gmaan | i see ceph error "Could not find the requested service virtlockd.service: host" | |
| 19:47:10 | gmaan | in other change | |
| 19:47:19 | gmaan | did not we merge your fix? | |
| 19:47:46 | sean-k-mooney | we did | |
| 19:47:59 | sean-k-mooney | in what job? | |
| 19:48:19 | gmaan | ceph job but that is 7 days old https://zuul.opendev.org/t/openstack/build/89f3ebb0ca94482082afe504069ed2a8/console | |
| 19:48:56 | gmaan | yeah, it merged after this test run https://review.opendev.org/c/openstack/nova/+/977150 | |
| 19:49:00 | gmaan | so recheck should work | |
| 19:49:30 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/977150/1 | |
| 19:49:49 | sean-k-mooney | that merged 6 days ago so ya recheck should pass now | |
| 19:49:53 | sean-k-mooney | that or a rebase | |
| 19:49:55 | gmaan | yeah, all good. i recheked the failure | |
| 19:50:01 | sean-k-mooney | cool | |
| 19:50:06 | gmaan | it should pick up latest master in recheck | |
| 20:11:43 | opendevreview | Merged openstack/nova master: libvirt: Add basic xml generation for firmware auto selection https://review.opendev.org/c/openstack/nova/+/969085 | |
| 20:12:07 | opendevreview | Merged openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | |
| 21:56:00 | opendevreview | Doug Goldstein proposed openstack/nova master: return error about external network to the user on build failure https://review.opendev.org/c/openstack/nova/+/972601 | |
| 22:30:45 | gmaan | stephenfin: one comment for this, rest all lgtm https://review.opendev.org/c/openstack/nova/+/976765 | |
| 23:44:58 | opendevreview | Merged openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | |
| 23:45:13 | opendevreview | Merged openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | |
| 23:57:06 | opendevreview | Merged openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | |
| 23:57:19 | opendevreview | Merged openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | |
| #openstack-nova - 2026-02-25 | |||
| 00:17:44 | opendevreview | Merged openstack/nova master: tests: Fix bound https://review.opendev.org/c/openstack/nova/+/971038 | |
| 01:34:20 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version to enable live migration https://review.opendev.org/c/openstack/nova/+/975724 | |
| 01:34:21 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 01:34:21 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 01:34:22 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 01:34:22 | opendevreview | melanie witt proposed openstack/nova master: TPM: enable conversion of secret security modes via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 01:39:43 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 01:44:35 | opendevreview | Doug Goldstein proposed openstack/nova master: return error about external network to the user on build failure https://review.opendev.org/c/openstack/nova/+/972601 | |
| 02:11:58 | gmaan | gibi: fyi, due to dentist appointment, i will skipping the eventlet call tomorrow | |
| 05:00:43 | opendevreview | Nope proposed openstack/nova master: scheduler: re-query Placement after stale-snapshot claim failures https://review.opendev.org/c/openstack/nova/+/977908 | |
| 07:00:30 | opendevreview | Nope proposed openstack/nova master: scheduler: re-query Placement after stale-snapshot claim failures https://review.opendev.org/c/openstack/nova/+/977908 | |
| 08:39:23 | gibi | gmaan: ack, good luck with the dentist | |
| 08:42:59 | gibi | it seems nova-next is blocked by | |
| 08:43:01 | gibi | Details: {'code': 400, 'message': 'Port 7896377a-52fa-43b1-94d6-c97eb5e4ac79 requires a FixedIP in order to be used.'} | |
| 08:43:10 | gibi | e.g. https://9d9eac1385c9916c71e7-8b0899cb6c8c0582fa25b52fb6031f3e.ssl.cf2.rackcdn.com/openstack/e293ec7c475544fab51185c08ef61814/testr_results.html | |
| 08:43:18 | gibi | https://zuul.opendev.org/t/openstack/builds?job_name=nova-next&project=openstack/nova | |
| 08:43:29 | gibi | since this morning | |
| 08:45:48 | frickler | gibi: that looks like https://review.opendev.org/c/openstack/tempest/+/959187 ? | |
| 08:46:25 | gibi | frickler: ahh thanks it very much like that | |
| 08:46:58 | gibi | I'm going to propose revert to save the progress towards FF | |
| 08:47:40 | frickler | ack, I think I could override if you don't want to wait for gmaan's dentist | |
| 08:50:06 | gibi | frickler: yes please Signed-off-by: Balazs Gibizer <gibi@redhat.com> | |
| 08:50:10 | gibi | https://review.opendev.org/c/openstack/tempest/+/977922 | |
| 08:50:18 | gibi | (sorry wrong buffer) | |
| 08:52:52 | gibi | cc Uggla gmaan ^^ | |
| 08:54:03 | Uggla | gibi :+1 | |
| 08:59:38 | frickler | and done. gibi can you please also create a revert of the revert and add a nova testing patch this time? | |
| 09:01:22 | gibi | sure I can | |
| 09:01:27 | gibi | thanks for the help | |
| 09:21:35 | gibi | frickler: this is the reapply patch with nova-next testing now https://review.opendev.org/c/openstack/tempest/+/977930 | |
| 09:22:26 | frickler | thx | |
| 11:37:50 | sean-k-mooney | gibi: i have not looks but we could also just skip that test in nova-next quickly | |
| 11:39:54 | sean-k-mooney | ah i see the revert is merged already | |
| 12:13:09 | gibi | yep | |
| 12:27:17 | r-taketn | gibi: Uggla: Hello. Thank you for reviewing the SEV refactoring patches despite your busy schedule. I'd appreciate it if you could also take a look at the remaining patches. | |
| 12:27:58 | sean-k-mooney | gibi: im going to review gmaan graceful shutdown patches now but ill hold +w on the bottom patch until we hopefully see your 3 eventlet removal patchs merge | |
| 12:28:15 | sean-k-mooney | gibi: but we can check in on that in the call later today | |
| 12:30:19 | gibi | sean-k-mooney: ack thanks. I saw the merge conflict. Hopefully we land the eventlet patches soon. They are approved but needed a recheck due to the tempest issue | |
| 12:30:52 | opendevreview | Bence Romsics proposed openstack/nova master: Functional reproducer for #2051685 https://review.opendev.org/c/openstack/nova/+/977331 | |
| 12:32:20 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove errant field https://review.opendev.org/c/openstack/nova/+/976765 | |
| 12:32:43 | stephenfin | gmaan: sean-k-mooney: Addressed gmaan's comment on ☝️ if you could revisit 🙏 | |
| 12:32:46 | sean-k-mooney | gibi: yep i checked them when i logged on this morning | |
| 12:33:51 | sean-k-mooney | stephenfin: done | |
| 12:33:55 | stephenfin | ty | |
| 13:00:35 | tkajinam | gibi, thanks ! | |
| 13:06:21 | tkajinam | bauzas, I'm wondering if you have chance to re-review https://review.opendev.org/c/openstack/nova/+/969132 ? I think I addressed your previous comments blocking +2 (If I took these right, hopefully | |
| 13:06:29 | tkajinam | the others are being merged | |
| 14:12:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for general additionalProperties https://review.opendev.org/c/openstack/nova/+/958858 | |
| 14:12:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Fix issue with instance usage audit log schema https://review.opendev.org/c/openstack/nova/+/977967 | |
| 14:12:55 | stephenfin | gmaan: sean-k-mooney: I had to rework that second patch slightly to handle a few more corner cases. Also found a bug in one of the more complicated schemas. Would appreciate a once-over ☝️ | |
| 14:14:09 | stephenfin | s/second/first/ | |