| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-24 | |||
| 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: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 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: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: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: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/ | |
| 14:22:45 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Move regex helpers to utils https://review.opendev.org/c/openstack/nova/+/958745 | |
| 14:22:46 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify API version check for flavor description https://review.opendev.org/c/openstack/nova/+/958747 | |
| 14:22:46 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Clean up flavors tests https://review.opendev.org/c/openstack/nova/+/958746 | |
| 14:22:47 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove dead fields from flavors response https://review.opendev.org/c/openstack/nova/+/958749 | |
| 14:22:47 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add ability to filter flavors by name https://review.opendev.org/c/openstack/nova/+/958748 | |
| 14:22:48 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 14:22:48 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Restrict additional query string arguments https://review.opendev.org/c/openstack/nova/+/958750 | |
| 14:22:49 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 14:22:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 14:55:47 | opendevreview | Nope proposed openstack/nova master: scheduler: re-query Placement after stale-snapshot claim failures https://review.opendev.org/c/openstack/nova/+/977908 | |
| 15:40:34 | bauzas | tkajinam: done now :) | |
| 15:48:57 | bauzas | gmaan: gibi: there is a potential merge conflict in between https://review.opendev.org/c/openstack/nova/+/975588/26 (2nd RPC service) and n-cpu threading support so I won't accept the former until there is a rebase | |
| 15:49:20 | bauzas | threading support is more important for the cycle, I hope you get it | |
| 15:49:44 | sean-k-mooney | yep that is why we are merging the eventlet patches first | |
| 15:50:31 | gibi | sorry folks I totally missed the eventlet call today | |
| 15:50:32 | gibi | :/ | |
| 15:51:21 | sean-k-mooney | ^ cordiating with graceful shutdown was the only topic i was going to raise | |
| 15:51:47 | sean-k-mooney | im review that serice now but honestly i was expect gmann ot have creted fucntionla test to test graceful shutdown | |
| 15:51:48 | gibi | bauzas: yeah the eventlet series is approved but gate is slow and unstable so it will take time. | |
| 15:52:05 | gibi | I'm monitoring it... | |
| 15:52:17 | sean-k-mooney | so im not sure that is ready i am also not really ok with callign it anything more then expreimetnal until we comple phase 2 and 3 | |
| 15:58:59 | bauzas | I guess I'll give another swing to vtpm live-migration series | |
| 16:41:42 | tkajinam | bauzas, thanks ! | |
| 16:42:20 | melwitt | bauzas, dansmith: for vtpm the patches targeted for FF are the fixups patch https://review.opendev.org/c/openstack/nova/+/976316 and the service version bump https://review.opendev.org/c/openstack/nova/+/975724 (since we got 'host' secret security mode available at this time). the rest will have to wait until H | |
| 16:45:47 | dansmith | melwitt: ack, I'll try to circle back soon, sorry for being pulled in multiple directions | |
| 16:46:11 | melwitt | thanks 🙏 | |
| 17:08:04 | bauzas | melwitt: noted, I'll look at those before ending my day | |
| 17:09:08 | melwitt | thanks 🙏 | |
| 17:20:56 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Update REST API version history for 2.101 https://review.opendev.org/c/openstack/nova/+/978002 | |
| 17:36:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused gabbi https://review.opendev.org/c/openstack/nova/+/978006 | |
| 17:47:48 | opendevreview | Merged openstack/nova master: libvirt: Add capability to load smm feature from existing xml https://review.opendev.org/c/openstack/nova/+/969131 | |
| 18:13:11 | opendevreview | Merged openstack/nova master: libvirt: Use firmware auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| 19:47:38 | gmaan | sean-k-mooney: thanks for review on graceful shutdown, i am ok to callout the experimental in releasenotes and once we finish the all phases we will have exact list/things which finish or move to safe state in shitdown | |
| 19:48:26 | gmaan | sean-k-mooney: bauzas gibi seeing the gate eventlet series might take time to merge and rebasing the graceful shutdown can miss the FF date. | |
| 19:48:57 | gmaan | as eventlet changes are with +w let me rebase in advance so that graceful shutdown can be in gate in parallel | |
| 19:49:35 | sean-k-mooney | ack | |
| 19:50:17 | sean-k-mooney | i really jsut dont want to get a custoemr case in the future whre the go "you said shutdown was graceful" and the interupted a live migration and got into an underined state | |
| 19:50:55 | sean-k-mooney | so if we call it experimetal untill pahse 2 or 3 is done i think its fine to merge although i woudl really like to see functional tests added next cycle to buidl conficance in this | |
| 19:51:08 | gmaan | for me interrupted is fine as it can be solved with the configured timeout but an underined state i agree that we should make sure | |
| 19:51:10 | sean-k-mooney | the ansible/bash tests help | |
| 19:51:56 | gmaan | sean-k-mooney: functional test for phase 2 and 3 will cover the things but for phase1 which is all about RPC redesign cannot be cover by functional tests as they use fake driver | |
| 19:52:01 | sean-k-mooney | oh ya i have said its totally fine with me if we just abort the ongoing live migration or interupt them in a fixable state. | |