| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-26 | |||
| 15:38:05 | tkajinam | https://bugs.launchpad.net/nova/+bug/2142767 | |
| 15:38:11 | tkajinam | https://bugs.launchpad.net/nova/+bug/2142765 | |
| 15:38:24 | tkajinam | I have to dig into the latter because it is blocking instance creation by heat, it seems | |
| 15:40:07 | tkajinam | hmm I guess these two are connected because the failure might be detected during returning a response | |
| 15:41:33 | bauzas | I reviewed the first patch and I'm looking into the second, but I still have a bit of concerns, so I guess we gonna have to make a draw | |
| 15:49:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: Accept an empty key for addresses https://review.opendev.org/c/openstack/nova/+/978089 | |
| 15:49:32 | gibi | both seems to complaim that addresses are empty | |
| 15:49:42 | Uggla | gibi, bauzas ok | |
| 15:50:25 | tkajinam | gibi, yeah. I think the problem is the top key under the addresses are empty and IIUC these keys are network names in neutron | |
| 15:50:49 | tkajinam | and ... name in neutron networks is optional. | |
| 15:51:44 | tkajinam | testing the above patch by https://review.opendev.org/c/openstack/ceilometermiddleware/+/978090 ... | |
| 15:54:13 | opendevreview | Merged openstack/nova master: db: Move regex helpers to utils https://review.opendev.org/c/openstack/nova/+/958745 | |
| 15:54:32 | opendevreview | Merged openstack/nova master: tests: Clean up flavors tests https://review.opendev.org/c/openstack/nova/+/958746 | |
| 15:56:29 | sean-k-mooney | cardoe: we only allow exepciton that have no hostspecif info to get to the user | |
| 15:56:55 | sean-k-mooney | or ratehr we often only show the name of the exctpion to non admins | |
| 15:57:27 | cardoe | yeah this makes the behavior identical to KVM/libvirt driver for this error | |
| 15:58:08 | cardoe | because the ironic driver has prepare_networks_before_block_device_mapping, it sets up the networks earlier than the KVM/libvirt does. | |
| 15:58:34 | sean-k-mooney | on the nova side we treat any exposure of the hypervior/compute node name as a security hardenign opertunity | |
| 15:59:14 | sean-k-mooney | not nessiarly a cve but we try to treat that as privldage so we are a bit carfeul with the raw tracebacks as a result | |
| 16:00:08 | sean-k-mooney | cardoe: ya idely the behvior shoudl be the same give me a sec to finsih up something now im off a call and ill take a look again | |
| 16:01:58 | gibi | tkajinam: make sense +2 | |
| 16:20:56 | opendevreview | Merged openstack/nova master: api: Simplify API version check for flavor description https://review.opendev.org/c/openstack/nova/+/958747 | |
| 16:59:33 | opendevreview | Merged openstack/nova master: Deprecate unlimited compute actions https://review.opendev.org/c/openstack/nova/+/975833 | |
| 16:59:50 | opendevreview | Merged openstack/nova master: [compute]Use single long task executor https://review.opendev.org/c/openstack/nova/+/977251 | |
| 17:02:54 | sean-k-mooney | gibi: ^ :) | |
| 17:06:58 | gibi | \○/ | |
| 17:07:10 | gibi | one more to go | |
| 17:07:25 | sean-k-mooney | im watching nova-next finish its tempest run | |
| 17:07:28 | gibi | then we can root for the graceful series on top | |
| 17:08:42 | sean-k-mooney | yep also https://review.opendev.org/c/openstack/nova/+/958748/6 if we can land that | |
| 17:14:37 | gmaan | finally :) | |
| 17:20:31 | gmaan | I think there are 3 changes competing for microversion 2.101 | |
| 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: 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 | |