| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-24 | |||
| 15:40:48 | gibi | gmaan: I'm +2 on the gracefull series, execept the doc patch at the top | |
| 15:40:59 | gibi | good progress! | |
| 15:58:12 | sean-k-mooney | jkulik: generally we need to test the new and old behvior so while you want have to duplciate all the test you woudl want to add new ones for the 2.101 bevhior although im hoping that micorversion will go to stephenfin's get flavor by name serise | |
| 15:59:11 | sean-k-mooney | the old test shoudl alreday be pinning there microversion to the previous change too the atchment api | |
| 15:59:17 | sean-k-mooney | they should not be using latest | |
| 16:00:46 | sean-k-mooney | jkulik: althoug hi dont see that so maybe they are gmaan would be the best person ot ask on how to proceed | |
| 16:03:57 | sean-k-mooney | i guess some of them do | |
| 16:03:59 | sean-k-mooney | https://github.com/openstack/tempest/blob/master/tempest/api/compute/volumes/test_attach_volume.py#L195-L196 | |
| 16:27:54 | gmaan | stephenfin: ack | |
| 16:28:28 | gmaan | gibi: thanks., i will check on eventlet one after dropping my son to school. and thanks for graceful shutdown review, will fix the doc change soon | |
| 16:43:22 | opendevreview | ribaudr proposed openstack/nova master: FUP Add HW_PCI_LIVE_MIGRATABLE trait to PCI resource providers https://review.opendev.org/c/openstack/nova/+/977310 | |
| 17:29:12 | gibi | tkajinam: I'm +2 all the way up on the fw autoselect series. Thanks for the work. I added +As to the bottom patches so it is starting to land soon | |
| 17:46:01 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add operator document for graceful shutdown https://review.opendev.org/c/openstack/nova/+/977590 | |
| 17:46:22 | gmaan | gibi: ^^ updated the doc one | |
| 17:49:05 | gibi | gmaan: thanks +2 there as well | |
| 17:49:18 | gmaan | thanks | |
| 17:58:29 | opendevreview | Merged openstack/nova master: Add manager graceful shutdown, timeout, and wait https://review.opendev.org/c/openstack/nova/+/975586 | |
| 18:23:52 | opendevreview | Merged openstack/nova master: Fix for bug 2140537 https://review.opendev.org/c/openstack/nova/+/975934 | |
| 19:14:08 | opendevreview | Merged openstack/nova master: Fix the negative sleep value in graceful_shutdown() https://review.opendev.org/c/openstack/nova/+/977708 | |
| 19:15:01 | opendevreview | Merged openstack/nova master: libvirt: Extend functional test coverage of UEFI boot guests https://review.opendev.org/c/openstack/nova/+/969263 | |
| 19:32:01 | gmaan | bauzas: thanks for review on first 3, these 4 are also ready (need 2nd +2), hoping we can land those before FF. https://review.opendev.org/q/topic:%22bp/nova-services-graceful-shutdown-part1%22+status:open | |
| 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 | |