| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-10-16 | |||
| 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: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 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: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: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: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: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: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: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: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: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 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: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: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: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: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 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: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: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:25 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 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 | |
| #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 tables from main DB https://review.opendev.org/c/openstack/nova/+/889087 | |
| 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: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 | |
| 09:19:33 | nicolairuckel | could not convert string to float: b'8.5e-'" | |
| 09:19:33 | nicolairuckel | Can someone explain the py3 tests to me? Tox says they fail but I can't make sense of the error message: "py3: commands[0]> stestr run | |
| 09:27:53 | gibi | nicolairuckel: could you post a bit more context to https://paste.opendev.org/ or point to a gerrit review where you have this issue? | |
| 09:30:18 | ierdem__ | hi, database.connection parameter points to cell0 by default in nova.conf when deployed with kolla-ansible. Should not it point to nova db instead of cell0? I use single cell architecture (cell v2). Should I remove cell0 and set it to nova db? | |
| 09:40:43 | nicolairuckel | gibi: This is the Gerrit review although there another pipeline fails: https://review.opendev.org/c/openstack/nova/+/959682 | |
| 11:03:48 | gibi | nicolairuckel: I only see nova-multi-cell failing there. Which job you get the above error from? | |
| 11:11:22 | nicolairuckel | The error above was from locally running `tox -e py3` (or just `tox` of course). | |
| 11:24:49 | gibi | that feels like something in your local env as the unit test CI job passed on that review. | |
| 11:44:46 | nicolairuckel | I guess then I have to look into nova-multi-cell instead. :D | |
| 11:44:48 | nicolairuckel | thanks | |
| 11:54:23 | frickler | nicolairuckel: for the local failure, I would guess at some localization issue. can you try running with LANG=C.utf8? otherwise please show more context of the error on paste | |
| 12:01:22 | nicolairuckel | frickler: this is the full output https://paste.opendev.org/show/bbTfG5OFaJ2XvXSLyFIN/ | |
| 12:18:46 | frickler | nicolairuckel: hmm, that looks like a previous run may have produced broken data? try removing (or move away at first only) the .stestr directory | |
| 12:55:19 | gibi | yeah that looks like the stestr repository on disk is corrupt | |
| 13:45:31 | nicolairuckel | frickler: that soled it, thanks | |
| 13:46:28 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2025.1: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961774 | |
| 14:28:39 | opendevreview | Merged openstack/nova stable/2025.2: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961951 | |
| 15:01:10 | jlejeune | gibi: hello, as requested I've rebased my change over parent change: https://review.opendev.org/c/openstack/nova/+/961774 Can you have a look please ? | |
| 15:03:23 | gibi | jlejeune: done :) | |
| 15:14:49 | jlejeune | thanks :) | |
| 17:53:06 | opendevreview | Nicolai Ruckel proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/959682 | |
| 20:06:26 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 20:06:26 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |