| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-26 | |||
| 19:32:43 | stephenfin | good point. I was only thinking of stacking the flavor one on top | |
| 19:33:09 | stephenfin | ...though if I stack that one and it goes into the gate, I'd have to wait for that to merge before I can stack the flavor one 😅 | |
| 19:33:12 | gmaan | yeah that one need, but at least can get os-volumes_boot in | |
| 19:33:37 | stephenfin | and the flavor one is more important (we want it for gopherclouds and openstack-resource-controller) | |
| 19:33:37 | gmaan | ohk | |
| 19:34:15 | sean-k-mooney | yep the flavor one is more imporant | |
| 19:34:27 | sean-k-mooney | and the follow up clean up patches | |
| 19:34:29 | gmaan | it seems async vol is a bout to merge if nova-next goes fine, <1 min remaining | |
| 19:34:37 | stephenfin | it's okay though it's about to merge https://zuul.opendev.org/t/openstack/status | |
| 19:34:42 | stephenfin | gmaan: jinx, you owe me a coke | |
| 19:34:45 | gmaan | yeah | |
| 19:34:51 | stephenfin | :) | |
| 19:35:11 | stephenfin | okay that makes my life easier | |
| 20:04:38 | opendevreview | Merged openstack/nova master: Attaching a volume returns HTTP 202 https://review.opendev.org/c/openstack/nova/+/971068 | |
| 20:06:13 | gmaan | stephenfin: you need to remove rxtx_factor from sort key also https://review.opendev.org/c/openstack/nova/+/958749 | |
| 20:07:34 | stephenfin | ack | |
| 20:08:08 | gmaan | a few doc comment, I think you are checking those while doping microvesion number change | |
| 20:08:24 | gmaan | I am on 958856, will be finishing soon | |
| 20:18:57 | gmaan | stephenfin: done | |
| 20:19:38 | opendevreview | Merged openstack/nova master: api: Remove errant field https://review.opendev.org/c/openstack/nova/+/976765 | |
| 20:27:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add ability to filter flavors by name https://review.opendev.org/c/openstack/nova/+/958748 | |
| 20:27:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Restrict additional query string arguments https://review.opendev.org/c/openstack/nova/+/958750 | |
| 20:27:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove dead fields from flavors response https://review.opendev.org/c/openstack/nova/+/958749 | |
| 20:27:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 20:27:26 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Update REST API version history for 2.102 https://review.opendev.org/c/openstack/nova/+/978002 | |
| 20:27:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 20:27:27 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 20:27:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 20:27:28 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 20:27:55 | stephenfin | gmaan: Should be good. Just running through unit/pep8 tests now in case I missed something | |
| 20:28:07 | gmaan | ack | |
| 20:34:04 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove dead fields from flavors response https://review.opendev.org/c/openstack/nova/+/958749 | |
| 20:34:05 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Update REST API version history for 2.102 https://review.opendev.org/c/openstack/nova/+/978002 | |
| 20:34:05 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Restrict additional query string arguments https://review.opendev.org/c/openstack/nova/+/958750 | |
| 20:34:06 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 20:34:06 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 20:34:07 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 20:34:07 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 20:34:08 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 20:46:37 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 20:46:38 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 20:46:38 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 20:46:39 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 20:46:39 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 21:16:57 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 21:16:57 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 21:16:58 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 21:16:58 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 21:16:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 21:17:39 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 21:17:39 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 21:17:40 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 21:17:40 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 21:17:41 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 21:18:03 | stephenfin | gmaan: Okay, I believe all unit tests and pep8 tests should pass now. I had to update some tests to remove use of os-volumes_boot since it's now deprecated | |
| 21:18:16 | stephenfin | I'm running functional tests but those take _forever_ | |
| 21:19:01 | gmaan | stephenfin: ack, I am going for lunch, will check after that | |
| 21:28:13 | stephenfin | nw. I also have to include https://review.opendev.org/c/openstack/nova/+/958863/ before the remove os-volumes_boot since I rely on the renamed methods in that patch. It's trivial enough, thankfully, and the other test-related patches are shoved to the end | |
| 21:30:35 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Update remove-os-volumes_boot-api to reflect changes https://review.opendev.org/c/openstack/nova-specs/+/978144 | |
| 21:39:42 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Restrict additional query string arguments https://review.opendev.org/c/openstack/nova/+/958750 | |
| 21:39:42 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove dead fields from flavors response https://review.opendev.org/c/openstack/nova/+/958749 | |
| 21:39:43 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 21:39:43 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Update REST API version history for 2.102 https://review.opendev.org/c/openstack/nova/+/978002 | |
| 21:39:44 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 21:39:44 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 21:39:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 21:39:46 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 21:39:57 | stephenfin | one failing functional test :( | |
| #openstack-nova - 2026-02-27 | |||
| 00:28:04 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: refactor memory encryption trait logic for extensiblity https://review.opendev.org/c/openstack/nova/+/967971 | |
| 00:28:04 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: stop using _get_mem_encryption_config() for SEV checks https://review.opendev.org/c/openstack/nova/+/967970 | |
| 00:28:05 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: refactor _guest_configure_mem_encryption() for extensibility https://review.opendev.org/c/openstack/nova/+/967973 | |
| 00:28:05 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: make RP creation independent of specific encryption models https://review.opendev.org/c/openstack/nova/+/967972 | |
| 00:28:06 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: introduce a check between mem_encryption and locked_memory https://review.opendev.org/c/openstack/nova/+/971300 | |
| 00:28:06 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: adjust requirement checks for mem_encryption guests https://review.opendev.org/c/openstack/nova/+/967974 | |
| 01:03:54 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: make RP creation independent of specific encryption models https://review.opendev.org/c/openstack/nova/+/967972 | |
| 01:03:54 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: refactor memory encryption trait logic for extensibility https://review.opendev.org/c/openstack/nova/+/967971 | |
| 01:03:55 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: adjust requirement checks for mem_encryption guests https://review.opendev.org/c/openstack/nova/+/967974 | |
| 01:03:55 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: refactor _guest_configure_mem_encryption() for extensibility https://review.opendev.org/c/openstack/nova/+/967973 | |
| 01:03:56 | opendevreview | Taketani Ryo proposed openstack/nova master: mem-enc: introduce a check between mem_encryption and locked_memory https://review.opendev.org/c/openstack/nova/+/971300 | |
| 05:39:55 | opendevreview | Merged openstack/nova master: Accept an empty key for addresses https://review.opendev.org/c/openstack/nova/+/978089 | |
| 10:59:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet from CacheConcurrencyTestCase https://review.opendev.org/c/openstack/nova/+/970069 | |
| 10:59:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet from libvirt/volume/test_mount https://review.opendev.org/c/openstack/nova/+/970071 | |
| 10:59:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet from libvirt/test_driver https://review.opendev.org/c/openstack/nova/+/970070 | |
| 10:59:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | |
| 11:12:01 | r-taketn | I have updated sev refactoring series. However, is merging these changes currently restricted by Feature Freeze? | |
| 11:33:37 | gibi | r-taketn: we reached FF for Gazpacho. One can argure that your series is more of a refactor than a feauture. But I defer to Uggla to decide if we need to wait with your series until master is open for the H cycle | |
| 12:04:09 | r-taketn | Except for the latest patch, I believe this series falls within refactoring. It would be helpful for me to continue the review, but I’ll wait for a decision from Uggla. Thank you for a clarification. | |
| 12:04:35 | opendevreview | Merged openstack/nova master: TPM: fixups for live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/976316 | |
| 13:28:03 | opendevreview | Merged openstack/nova master: Destroy scatter_gather in conductor https://review.opendev.org/c/openstack/nova/+/977055 | |
| 13:37:00 | nicolairuckel | I'd like to backport my NVRAM patch. Is there any documentation on how to do that? | |
| 13:47:12 | gibi | nicolairuckel: in general backporting bugs are simply. You need to git cherry-pick -x to the youngest stable branch first then when that is done you can cherry-pick from that branch to the next one. Not all patches are accepted as backports on stable. In general we don't allow feature backports, or backports that has external interface impacts like API impacts or DB schema changes. | |
| 13:47:30 | gibi | elodilles: ^^ | |
| 13:55:05 | elodilles | nicolairuckel: what gibi said :) feel free to ping me if you have any further question or need a review for your backport | |
| 14:41:03 | Uggla | So from my side, I’m fine to proceed and not wait for master to reopen for the H cycle. | |
| 14:41:03 | Uggla | Given that, I’d prefer we take advantage of the current momentum and continue the review with the goal of merging it during Gazpacho, if possible. This refactoring will also make the upcoming confidential computing work (SEV-SNP and TDX) easier to integrate and reason about. | |
| 14:41:03 | Uggla | gibi, r-taketn, I agree that this series is primarily a refactoring rather than a feature change. | |
| 14:45:03 | LarsErikP | sean-k-mooney: any more traction here? :P https://review.opendev.org/c/openstack/nova/+/916409 | |
| 14:49:29 | gibi | Uggla: thanks. When r-taketn is back we should clarify with them that then we have an extra two weeks for the refactor | |
| 14:49:52 | opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Object changes https://review.opendev.org/c/openstack/nova/+/978209 | |
| 14:49:59 | sean-k-mooney | LarsErikP: nope | |