| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-26 | |||
| 17:47:46 | sean-k-mooney | so ya im jsut clarifyign are you asking for a 410 to be returend or the spec to be updated to 404 | |
| 17:47:52 | gmaan | yeah, gate check our patient level and gives blessing | |
| 17:48:02 | gmaan | gibi: heh yeah | |
| 17:48:40 | gmaan | sean-k-mooney: ohk, I am saying code is good i am +2 there. i am asking spec to update once we land the code | |
| 17:48:46 | gmaan | just paper work nothing else | |
| 17:48:59 | sean-k-mooney | ack stephenfin ^ you might want to read back | |
| 17:49:45 | gmaan | which can be done later but I am holding +w due to microversion conflict only | |
| 17:52:35 | sean-k-mooney | stephen wont be around for the next 2 hours but they will be back on line later | |
| 17:57:24 | gmaan | ack, no issue, I will check as soon as it is updated | |
| 18:14:22 | opendevreview | Sebastian Krott proposed openstack/nova master: network: Add host filtering to `cleanup_instance_network_on_host` https://review.opendev.org/c/openstack/nova/+/978122 | |
| 18:14:22 | opendevreview | Sebastian Krott proposed openstack/nova master: conductor: Clean up port-bindings for live migrations https://review.opendev.org/c/openstack/nova/+/978121 | |
| 18:19:07 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version to enable live migration https://review.opendev.org/c/openstack/nova/+/975724 | |
| 18:19:07 | opendevreview | melanie witt proposed openstack/nova master: TPM: fixups for live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/976316 | |
| 18:19:08 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 18:19:08 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 18:19:09 | opendevreview | melanie witt proposed openstack/nova master: TPM: enable conversion of secret security modes via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 18:19:09 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 18:19:10 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 18:21:13 | opendevreview | melanie witt proposed openstack/nova master: TPM: fixups for live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/976316 | |
| 18:21:14 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 18:21:14 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version to enable live migration https://review.opendev.org/c/openstack/nova/+/975724 | |
| 18:21:15 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 18:21:15 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 18:21:16 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 18:21:16 | opendevreview | melanie witt proposed openstack/nova master: TPM: enable conversion of secret security modes via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 18:24:52 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version to enable live migration https://review.opendev.org/c/openstack/nova/+/975724 | |
| 18:24:53 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 18:24:53 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 18:24:54 | opendevreview | melanie witt proposed openstack/nova master: TPM: enable conversion of secret security modes via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 18:24:54 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 18:24:55 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 18:27:33 | melwitt | bauzas, dansmith: fyi I had to manually rebase and change service version 71 => 72 due to merge conflict in objects/service.py after https://review.opendev.org/c/openstack/nova/+/975588 merged | |
| 18:28:08 | dansmith | ack | |
| 18:30:56 | gmaan | melwitt: ah, sorry for that, I did not realise that vtpm also bumping service version | |
| 18:31:38 | melwitt | not your fault gmaan :) | |
| 19:22:30 | stephenfin | sean-k-mooney: gmaan: I can't rebase my series because that one in the gate is based off an old master. I'll have to do so once it lands (or rebase everything if it fails the gate) | |
| 19:28:46 | gmaan | stephenfin: if they are not conflict on files then it should work as zuul will pick the master and merge the things while testing. | |
| 19:29:19 | gmaan | it was same for eventlet and grafeul shutdown series, I rebased gs one on eventlet which did not have the latest master | |
| 19:29:26 | gmaan | try, I think that should work | |
| 19:30:50 | stephenfin | gmaan: unfortunately there's conflicts on files due to missing openapi patches :( | |
| 19:32:07 | gmaan | stephenfin: I mean this one does not need openapi things right? https://review.opendev.org/c/openstack/nova/+/958864 | |
| 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 | |