| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-26 | |||
| 17:20:49 | dansmith | I just approved the async volume attach one that wants 101 | |
| 17:20:51 | gmaan | this one is in gate, which should get it https://review.opendev.org/c/openstack/nova/+/971068 | |
| 17:20:56 | gmaan | dansmith: yeah | |
| 17:21:44 | gmaan | I am reviewing the remove-os-volumes_boot which is almost ready but remember the async volume one | |
| 17:22:11 | gmaan | stephenfin: do you want to update remove-os-volumes_boot on top of async vol one? | |
| 17:31:34 | gmaan | stephenfin: I am holding the +w and one comment to update the spec whihc mentioned to return 410 instead of 404 | |
| 17:32:01 | gmaan | sean-k-mooney: also, you tagged this cleanup as part of this BP but that is not related, it will be good to keep that separate https://review.opendev.org/c/openstack/nova/+/958863 | |
| 17:32:30 | gmaan | stephenfin: ^^ | |
| 17:33:01 | gmaan | sean-k-mooney: please ignore, i need to start my auto tab after typing two letter :) | |
| 17:41:17 | sean-k-mooney | ya i mented that seperatly to stephenfin, that it woudl be better to flip the order of that and teh one on top | |
| 17:42:07 | sean-k-mooney | gmaan: i was debeaitn if i shoudl rasie the 404 vs 410 point | |
| 17:42:36 | sean-k-mooney | i decied not to becasue i think the pattern we have is we only return 410 if we remove it form all micoverions | |
| 17:43:07 | sean-k-mooney | but use a 404 if its only remove in a specific microverison | |
| 17:43:59 | gmaan | yes, because we have same patter for other APIs also for example proxy one. and the one which are gone for all microversion they return 410 | |
| 17:44:46 | opendevreview | Merged openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 17:45:02 | opendevreview | Merged openstack/nova master: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | |
| 17:45:15 | gibi | \○/ | |
| 17:45:16 | gmaan | \o/ gibi grab a beer | |
| 17:45:30 | gibi | I'm done for today :) | |
| 17:45:46 | opendevreview | Merged openstack/nova master: Prepare resize/cold migration for graceful shutdown https://review.opendev.org/c/openstack/nova/+/977182 | |
| 17:45:48 | gmaan | o/ | |
| 17:45:52 | sean-k-mooney | gmaan: so this doesnt remvoe it outright right | |
| 17:45:56 | gibi | o/ | |
| 17:46:28 | opendevreview | Merged openstack/nova master: Add more test for graceful shutdown https://review.opendev.org/c/openstack/nova/+/977405 | |
| 17:46:45 | gibi | when things start moving... | |
| 17:46:47 | opendevreview | Merged openstack/nova master: Add operator document for graceful shutdown https://review.opendev.org/c/openstack/nova/+/977590 | |
| 17:46:48 | gmaan | sean-k-mooney: did not get, you mean for spec update? | |
| 17:47:05 | sean-k-mooney | i ment https://review.opendev.org/c/openstack/nova/+/958864 | |
| 17:47:35 | gibi | gmaan: you reached early beer state today :) | |
| 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: conductor: Clean up port-bindings for live migrations https://review.opendev.org/c/openstack/nova/+/978121 | |
| 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: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: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: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: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: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: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: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: bump service version to enable live migration https://review.opendev.org/c/openstack/nova/+/975724 | |
| 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: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: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: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:21:16 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 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: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 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: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: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: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 | gmaan | ohk | |
| 19:33:37 | stephenfin | and the flavor one is more important (we want it for gopherclouds and openstack-resource-controller) | |
| 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: Remove dead fields from flavors response https://review.opendev.org/c/openstack/nova/+/958749 | |
| 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: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:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate os-volumes_boot API https://review.opendev.org/c/openstack/nova/+/958864 | |
| 20:27:27 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 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:28 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 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:55 | stephenfin | gmaan: Should be good. Just running through unit/pep8 tests now in case I missed something | |