| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-18 | |||
| 17:27:23 | gmaan | yeah | |
| 17:27:36 | sean-k-mooney | allows admins to bypass the project membership check | |
| 17:27:50 | sean-k-mooney | i..e its for the --all-tenants type commands | |
| 17:28:39 | gmaan | sean-k-mooney: Uggla to avoid using those without OR_ADMIN, i think we should remove these alias and have a note about it there https://github.com/openstack/nova/blob/master/nova/policies/base.py#L41-L43 | |
| 17:29:41 | gmaan | I kept them but forgot to add notes about that we need to support admin to access those project scoped role base permission | |
| 17:30:29 | Uggla | gmaan, so as I suspected, it needs to be corrected on the server_shares, I will open a patch for it. ok ? | |
| 17:30:46 | gmaan | Uggla: yeah | |
| 17:31:03 | Uggla | gmaan 👍 | |
| 17:31:14 | dansmith | melwitt: did you see bauzas' comment on the user patch about coverage? | |
| 17:31:30 | dansmith | I didn't go validate the concern (yet) but thought maybe you missed it | |
| 17:32:04 | melwitt | dansmith: yes, I added coverage. unless I messed up a rebase or upload. I'll check it | |
| 17:32:45 | dansmith | oh, I missed that there was a PS after that comment.. you didn't mark it done so I thought it was fresh | |
| 17:33:13 | melwitt | ah crap | |
| 17:34:46 | dansmith | I guess I'm also not sure the delta between those PSes is what he was asking about | |
| 17:37:10 | melwitt | dansmith: oh really? I thought he was pointing out that use of the image property for tpm_model was not covered? | |
| 17:38:08 | dansmith | I don't know because I don't really know what he's asking.. he said "check the property" and you didn't add any new checks... and the new path excludes the model property | |
| 17:39:05 | dansmith | like you already had a test that passed the tpm_model property | |
| 17:39:34 | dansmith | ohh, as the image prop and not the flavor I guess | |
| 17:39:50 | melwitt | yeah | |
| 17:40:15 | melwitt | I didn't know what else it could mean but I'm open to maybe I misunderstood something :) | |
| 17:40:44 | dansmith | I'm sure you're right I just didn't make sense of it initially | |
| 17:54:18 | opendevreview | Merged openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 17:54:30 | opendevreview | Merged openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | |
| 18:08:38 | opendevreview | Merged openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | |
| 18:08:53 | opendevreview | Merged openstack/placement master: Make sure [cors] allowed_origin accepts a list value https://review.opendev.org/c/openstack/placement/+/966275 | |
| 18:39:03 | dansmith | melwitt: would it be possible to move the deployment mode patch down to just above host before we start into all the live migration related bits? | |
| 18:39:07 | dansmith | or is there some dependency there I'm not seeing? | |
| 18:47:17 | melwitt | dansmith: I don't think there is a dependency, so I think it can be moved. I'll try it and let you know if I run into anything that could prevent it | |
| 18:48:42 | dansmith | okay, seems like it would be a nice organization to get all three of the modes landed and then review the migration stuff with all that sorted in our (my) minds | |
| 18:48:44 | dansmith | if it's not hard.. | |
| 18:50:18 | melwitt | yeah, makes sense | |
| 19:00:45 | opendevreview | Stephen Finucane proposed openstack/placement master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/placement/+/919582 | |
| 19:00:45 | opendevreview | Stephen Finucane proposed openstack/placement master: Correct default oslo.conf file https://review.opendev.org/c/openstack/placement/+/967600 | |
| 19:00:46 | opendevreview | Stephen Finucane proposed openstack/placement master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/placement/+/967601 | |
| 19:00:46 | opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Bump versions https://review.opendev.org/c/openstack/placement/+/967602 | |
| 19:22:40 | opendevreview | Merged openstack/nova stable/2025.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967567 | |
| 19:27:55 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 19:27:55 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 19:27:56 | 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 | |
| 19:27:56 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 19:27:57 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 19:27:57 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 19:27:58 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 19:27:58 | 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 | |
| 19:27:59 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 19:28:00 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 19:31:50 | dansmith | melwitt: was it that easy? | |
| 19:33:22 | melwitt | dansmith: yes, seems so | |
| 19:33:59 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 19:34:04 | dansmith | damn I suck at negotiation.. I should have asked for more :P | |
| 19:34:20 | melwitt | hah | |
| 19:44:21 | opendevreview | Merged openstack/placement master: Replace remaining py39 target https://review.opendev.org/c/openstack/placement/+/963808 | |
| 19:44:23 | opendevreview | Merged openstack/placement master: Remove tags from README https://review.opendev.org/c/openstack/placement/+/941792 | |
| 19:44:26 | opendevreview | Merged openstack/placement master: Remove installation guide for openSUSE/SLES https://review.opendev.org/c/openstack/placement/+/949864 | |
| 21:04:03 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 21:11:24 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Step 1 of Graceful shutodwn spec https://review.opendev.org/c/openstack/nova-specs/+/967585 | |
| 21:23:53 | opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | |
| 21:26:09 | gmaan | dansmith: gibi: graceful shutdown spec is ready for review (a backlog spec to propose complete things and another change to implement the Step1 proposal for this cycle). | |
| 21:26:22 | gmaan | I uploaded my PoC results, PoC and results links are in spec also. https://review.opendev.org/q/topic:%22bp/nova-services-graceful-shutdown%22 | |
| 21:26:35 | gmaan | whenever you get time, no hurry | |
| 21:54:15 | opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | |
| 23:59:13 | opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | |
| #openstack-nova - 2025-11-19 | |||
| 02:31:54 | opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | |
| 06:19:18 | opendevreview | Rajesh Tailor proposed openstack/nova master: Update comment about migrated mypy conf files https://review.opendev.org/c/openstack/nova/+/967633 | |
| 07:00:54 | opendevreview | huanhongda proposed openstack/nova master: live migration: Delete allocation if post_live_migration fails https://review.opendev.org/c/openstack/nova/+/967652 | |
| 08:53:11 | gibi | gmaan: ack. I'm planning to get to it this week | |
| 09:53:53 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix memory_details units in diagnostics API to match documentation https://review.opendev.org/c/openstack/nova/+/967505 | |
| 11:36:19 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova-specs master: [spec] Add Cross-AZ scheduling blueprint https://review.opendev.org/c/openstack/nova-specs/+/900296 | |
| 11:40:18 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova-specs master: [spec] Add Cross-AZ scheduling blueprint https://review.opendev.org/c/openstack/nova-specs/+/900296 | |
| 11:42:20 | mkuroha | bauzas: gibi: sean-k-mooney: Thank you for your previous feedback and for getting the spec for 2026.1 merged. I’d appreciate it if you could review this code as well when you have a moment: https://review.opendev.org/c/openstack/nova/+/948304 | |
| 11:48:04 | noonedeadpunk | I know I'm very late with this spec, and really sorry about not being able to get working on it sooner after ptg: https://review.opendev.org/c/openstack/nova-specs/+/900296 | |
| 11:48:35 | noonedeadpunk | I think I've addressed all comments from etherpad, and ones were left as previous reviews | |
| 12:00:17 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 12:22:12 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute/manager: fix live migration issue due to orphaned attachments https://review.opendev.org/c/openstack/nova/+/967683 | |
| 13:22:47 | opendevreview | Dominik proposed openstack/os-traits master: Add NUMA Topology traits https://review.opendev.org/c/openstack/os-traits/+/967690 | |
| 13:31:12 | sean-k-mooney | noonedeadpunk: dont fret, the soft proposal freeze is tomorrow but spec freze is not for another 2 weeks | |
| 13:31:36 | sean-k-mooney | noonedeadpunk: i saw the email update on it but have not looked at it | |
| 13:32:02 | sean-k-mooney | ill try and review it once i finish writing my last spec for this release :P | |
| 13:33:22 | noonedeadpunk | ok, then I feel a little better about doing everything at last minute :) | |
| 13:33:25 | sean-k-mooney | noonedeadpunk: i gavem most of my feedback/request in the eather pad durign the ptg https://etherpad.opendev.org/p/r.9bac75a686ab0941572757b205c9415c#L1020 | |
| 13:33:48 | noonedeadpunk | I tried to address all your comments in it | |
| 13:33:53 | sean-k-mooney | cool | |
| 13:34:01 | noonedeadpunk | IT could be that I did not elaborate enough on them though... | |
| 13:34:18 | sean-k-mooney | ill try to get to this on friday if i dont feel free to ping me to review | |
| 13:34:29 | noonedeadpunk | But I was reffering to etherpad at least while reviving the spec | |
| 13:34:37 | noonedeadpunk | ++ thanks! | |
| 13:54:50 | opendevreview | Merged openstack/placement master: Replace UPPER_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/placement/+/947483 | |
| 14:34:29 | dansmith | bauzas: why are you not +W on the user patch? I thought we agreed we were going to merge through host, since even without live migration, that massively improves the general situation | |
| 14:34:39 | dansmith | is there something about that point that you're still unsure about? | |
| 14:34:47 | bauzas | because I wanted to look at the other ones, but sure I can +W them | |
| 14:34:57 | bauzas | gimme a second :) | |
| 14:35:41 | dansmith | okay, getting through the host patch means that an operator can have vtpm instances that are restartable after a power outage, so even without live migration stuff, that's a big win | |
| 14:49:48 | bauzas | ditto | |
| 14:55:49 | opendevreview | ribaudr proposed openstack/nova-specs master: Enable memfd support for shared memory backing https://review.opendev.org/c/openstack/nova-specs/+/951689 | |
| 15:42:52 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | |
| 15:43:44 | opendevreview | ribaudr proposed openstack/nova master: Use *_OR_ADMIN policy defaults for server shares https://review.opendev.org/c/openstack/nova/+/967709 | |
| 15:44:11 | Uggla | gmaan, if you can have a look ^ | |
| 15:44:55 | opendevreview | Merged openstack/nova stable/2024.2: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961602 | |
| 15:51:16 | opendevreview | Merged openstack/nova master: TPM: add RequestContext checks to functional tests https://review.opendev.org/c/openstack/nova/+/966499 | |
| 16:00:38 | gmaan | Uggla: ack, will check it | |