| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-10-15 | |||
| 02:06:38 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 02:06:38 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 07:57:21 | opendevreview | Balazs Gibizer proposed openstack/placement master: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/962776 | |
| 08:48:45 | opendevreview | Kamil Sambor proposed openstack/nova master: [hacking] Improve N373 to catch also other primitives https://review.opendev.org/c/openstack/nova/+/958408 | |
| 13:20:19 | dansmith | gibi: I think I can proxy sean's +2 on that since you just changed some english and added an assert to the test.. agree? | |
| 13:23:45 | gibi | dansmith: I agree. If you are confortable with the patch then I think Sean will not disagree approving it based on Sean's past review rounds | |
| 13:28:36 | gibi | dansmith: thank you for your continued effort on that patch I appreciate it. And sorry again that I forced you to re-review it. | |
| 14:33:33 | dansmith | gibi: eventlet sync? | |
| 14:36:41 | opendevreview | Merged openstack/placement master: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/962776 | |
| 17:59:24 | gmaan | melwitt: +2 on the config option logging fix. I checked in logs (added links in gerrit comment) and it is working fine now https://review.opendev.org/c/openstack/nova/+/963619 | |
| 17:59:49 | gmaan | I added the log flow in comment so that it will be quick for 2nd reviewer to check the result | |
| 17:59:58 | melwitt | gmaan: hey thanks :D /me was not expecting reviews so quick | |
| 19:55:01 | Zhan[m] | hey team, just wanna ask for feedbacks on some ideas on live migrations (using libvirt and QEMU): I saw in this [commit](https://review.opendev.org/c/openstack/nova/+/434870) that we'll be activating port bindings when the VM is paused on the source side, but this change was reverted in this [commit](https://github.com/openstack/nova/commit/aa87b9c288d316b85079e681e0df24354ec1912c) because if the migration fails, the | |
| 19:55:01 | Zhan[m] | `_rollback_live_migration` function cannot properly rollback the changes to port bindings. Does it make sense for me to write some code in the rollback function to basically recreate and reactivate the port bindings on the source side if needed, so that we can have the previous change back (start port binding earlier)? thanks! | |
| 21:43:17 | gmaan | bauzas: gibi: I added some comment in ImagePropertiesWeigher fix, nothing blocker so feel free to +w if those does not need updates or reply and I will +w tomorrow https://review.opendev.org/c/openstack/nova/+/963358 | |
| #openstack-nova - 2025-10-16 | |||
| 07:52:00 | jlejeune | hello bauzas, sean-k-mooney[m], gibi and others :) I would like to backport my changes related to these topics: https://review.opendev.org/q/topic:%22bug-2044235%22 and https://review.opendev.org/q/topic:%22bug/2085135%22, what do you think about them ? thanks | |
| 08:17:30 | gibi | jlejeune: I did some reviews in both series now, and had couple of +2 but both series has some minor issues in later backports so I left -1es later | |
| 08:37:35 | jlejeune | gibi: checking, thanks | |
| 08:38:59 | opendevreview | Masahito Muroi proposed openstack/nova-specs master: Add spec for virtio-blk multiqueue extra_spec https://review.opendev.org/c/openstack/nova-specs/+/951636 | |
| 08:42:42 | masahito | sean-k-mooney[m]: Uggla: gibi: Thanks for the review in the last development cycle for the multiqueue. I resumed the upstream patch update b/c the Gazpacho development cycle was open :) | |
| 08:58:14 | opendevreview | Merged openstack/nova stable/2025.2: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961948 | |
| 09:10:17 | opendevreview | Sylvain Bauza proposed openstack/nova master: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/963358 | |
| 09:10:45 | bauzas | gibi: gmaan had a few good comments on https://review.opendev.org/c/openstack/nova/+/963358 I just rebased it | |
| 09:16:26 | opendevreview | Merged openstack/nova stable/2025.2: Update start_service() function in test https://review.opendev.org/c/openstack/nova/+/963842 | |
| 09:16:38 | opendevreview | Merged openstack/nova stable/2025.2: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963843 | |
| 09:37:25 | opendevreview | Masahito Muroi proposed openstack/nova-specs master: Add spec for virtio-blk multiqueue extra_spec https://review.opendev.org/c/openstack/nova-specs/+/951636 | |
| 11:00:40 | opendevreview | Merged openstack/nova stable/2025.2: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/963844 | |
| 12:24:41 | gibi | bauzas: ack I will check it | |
| 14:19:44 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2025.2: Reproduce GET a_c slowness bug/2126751 https://review.opendev.org/c/openstack/placement/+/964219 | |
| 14:19:45 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2025.2: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/964220 | |
| 14:24:38 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2025.1: Reproduce GET a_c slowness bug/2126751 https://review.opendev.org/c/openstack/placement/+/964221 | |
| 14:24:39 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2025.1: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/964222 | |
| 14:25:56 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.2: Reproduce GET a_c slowness bug/2126751 https://review.opendev.org/c/openstack/placement/+/964223 | |
| 14:25:57 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.2: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/964224 | |
| 14:26:15 | gibi | dansmith: I've pushed the stable backports of the placment fix | |
| 14:26:32 | gibi | when you have time please take look. They are all clean cherry-picks | |
| 14:27:04 | gibi | https://review.opendev.org/q/topic:%22bug/2126438%22 and https://review.opendev.org/q/topic:%22bug/2126751%22+project:openstack/placement | |
| 14:27:29 | dansmith | okay I'm really slammed today and I have a bunch of glance things I just got poked for in the meeting just now | |
| 14:27:46 | gibi | sure, no worries we are not in a rush with these backports | |
| 14:27:48 | dansmith | might be good if some other people could just do the low-impact work of validating the backports | |
| 14:28:26 | gibi | I pinged you as you have the context and Sean is on PTO. But yeah this is easy to review for any cores | |
| 14:28:41 | dansmith | ack | |
| 14:38:10 | jlejeune | gibi:about your remark on my cherry-pick, the change is not the same on other stable branches because you 've introduced that new line in test_service.py in that change https://review.opendev.org/c/openstack/nova/+/947245 and it's not been backported on other stable branches | |
| 14:40:59 | gibi | jlejeune: cool, please note that in the commit message of the patch that first has this difference | |
| 14:41:30 | gibi | this is a common pattern we do to help the reviewers to know that the difference is not a mistake but intentional | |
| 14:42:14 | jlejeune | gibi: ok my bad, doing | |
| 14:44:51 | gibi | no worries. If you never see such backport commit message that needed this then there is no way to know how we handle it | |
| 14:46:05 | jlejeune | do you have an example in mind ? | |
| 14:54:23 | opendevreview | Julien Le Jeune proposed openstack/nova stable/2025.1: Update start_service() function in test https://review.opendev.org/c/openstack/nova/+/963845 | |
| 14:54:24 | opendevreview | Julien Le Jeune proposed openstack/nova stable/2025.1: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963846 | |
| 14:54:25 | opendevreview | Julien Le Jeune proposed openstack/nova stable/2025.1: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/963847 | |
| 14:56:38 | jlejeune | gibi: something like that ? ^ | |
| 15:10:54 | gibi | jlejeune: yepp, look good. thanks | |
| 15:27:34 | gmaan | bauzas: +w now. though i replied for the test comment but that is just comment things actual assert in test is correct so no issue. | |
| 15:41:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]PCI in Placement tuning https://review.opendev.org/c/openstack/nova/+/963281 | |
| 17:03:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func]Test with optimize_for_wide_provider_trees https://review.opendev.org/c/openstack/nova/+/963036 | |
| 23:56:03 | opendevreview | Merged openstack/nova master: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/963358 | |
| #openstack-nova - 2025-10-17 | |||
| 08:48:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: Add a regression test for ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/964287 | |
| 08:48:03 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/964288 | |
| 08:59:20 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Add a regression test for ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/964290 | |
| 08:59:21 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/964291 | |
| 09:09:19 | gibi | melwitt: when you are up I think there is a confusion in the commit message in https://review.opendev.org/c/openstack/nova/+/963619 | |
| 09:14:40 | opendevreview | Moritz Wanzenböck proposed openstack/nova master: Implement extend_volume for local devices https://review.opendev.org/c/openstack/nova/+/878763 | |
| 18:21:32 | melwitt | gibi: oh gosh, I don't know how I made that mistake writing the completely opposite word /facepalm thanks for catching it, I'll fix it | |
| 18:32:15 | opendevreview | melanie witt proposed openstack/nova master: Dump conf to debug log later in the WSGI app pipeline https://review.opendev.org/c/openstack/nova/+/963619 | |
| 21:57:58 | opendevreview | Merged openstack/placement stable/2025.2: Remove excessive logging from GET a_c https://review.opendev.org/c/openstack/placement/+/963382 | |
| 23:27:48 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 23:27:48 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 23:27:49 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 23:27:49 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 23:27:50 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 23:27:50 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 23:27:51 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 23:27:51 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 23:27:52 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| #openstack-nova - 2025-10-18 | |||
| 02:35:05 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 02:35:06 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 02:35:06 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 02:35:07 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 02:35:07 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 02:35:08 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 03:01:23 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 03:01:23 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 03:01:24 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 03:01:24 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 03:01:25 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 03:01:25 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| #openstack-nova - 2025-10-19 | |||
| 10:25:06 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Return 404 on bad project ID for 'os-quota-sets' https://review.opendev.org/c/openstack/nova/+/937125 | |
| 10:29:53 | opendevreview | Stephen Finucane proposed openstack/nova master: rpc: Mark attributes as private https://review.opendev.org/c/openstack/nova/+/792803 | |
| 10:37:39 | opendevreview | Merged openstack/nova master: api-ref: Fix server topology API docs https://review.opendev.org/c/openstack/nova/+/952274 | |
| 10:38:33 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy placement tables from API DB https://review.opendev.org/c/openstack/nova/+/889086 | |
| 10:38:33 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy tables from main DB https://review.opendev.org/c/openstack/nova/+/889087 | |
| 10:38:34 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy columns from main DB https://review.opendev.org/c/openstack/nova/+/889088 | |
| 10:38:47 | opendevreview | sean mooney proposed openstack/nova master: conf: Deprecate AggregateImagePropertiesIsolation opts https://review.opendev.org/c/openstack/nova/+/783396 | |
| 13:23:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unused 'shutdown_terminate' flag https://review.opendev.org/c/openstack/nova/+/773156 | |
| 14:32:03 | opendevreview | Nicolai Ruckel proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/959682 | |
| 14:33:49 | opendevreview | Nicolai Ruckel proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/959682 | |
| #openstack-nova - 2025-10-20 | |||
| 08:15:51 | gibi | hi fellow cores, I have two small and simple patches as follow ups from the placement perf optimization bug | |
| 08:16:15 | gibi | * nova doc patch pointing to the new placement tuning options https://review.opendev.org/c/openstack/nova/+/963281 | |
| 08:17:02 | gibi | * nova func test change that runs our pci in placement tests with the new placement tunings enabled https://review.opendev.org/c/openstack/nova/+/963036 | |