| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-18 | |||
| 15:39:32 | sean-k-mooney | the commit is aviabel in the ui beside where you slect the patch set just under where it say Comments | |
| 15:41:33 | opendevreview | Nell Jerram proposed openstack/nova stable/2025.1: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967568 | |
| 15:44:16 | opendevreview | Nell Jerram proposed openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | |
| 15:49:07 | nelljerram | Ah I see. The commit ID is also available from the gitea link, but the one you pointed to is better. | |
| 15:53:01 | sean-k-mooney | if you use the cli (with or without git review) you can pass a flag to add it automaticly | |
| 15:53:13 | sean-k-mooney | but ya you can do it form the ui too | |
| 15:53:28 | sean-k-mooney | when i do it form the ui i wait for the previous version to merge ot avoid having to update it | |
| 15:53:48 | sean-k-mooney | i approved the first backport ill look at the ohter once its merged | |
| 15:53:56 | nelljerram | many thanks! | |
| 16:13:03 | opendevreview | Merged openstack/osc-placement master: Replace remaining py39 target https://review.opendev.org/c/openstack/osc-placement/+/963809 | |
| 16:36:47 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Add spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | |
| 17:21:40 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Step 1 of Graceful shutodwn spec https://review.opendev.org/c/openstack/nova-specs/+/967585 | |
| 17:23:19 | Uggla | gmaan, question, server_shares policies are PROJECT_MEMBER or PROJECT_READER. Is it normal. Should it be PROJECT_MEMBER_OR_ADMIN or PROJECT_READER_OR_ADMIN respectively ? | |
| 17:24:21 | jrosser | AWS host key signing | |
| 17:24:24 | jrosser | argh | |
| 17:25:04 | sean-k-mooney | Uggla: so project reader allsow anyoen wotih project member to access it | |
| 17:25:28 | gmaan | Uggla: *_OR ADMIN should be there as we do want admin to access those | |
| 17:25:29 | sean-k-mooney | so really the question is shoudl ti be PROJECT_READER or shoudl it be PROJECT_READER_OR_ADMIN | |
| 17:26:04 | gmaan | yeah admin will not able to access PROJECT_READER as project id check is there so it should be PROJECT_READER_OR_ADMIN | |
| 17:26:21 | gmaan | PROJECT_MEMBER can access PROJECT_READER as sean-k-mooney mentioned | |
| 17:26:52 | sean-k-mooney | basicaly there is role inheritnace the member role grants the reader role and the admin role grant member | |
| 17:27:07 | sean-k-mooney | so when we write the policy we specify the minium rule that is required | |
| 17:27:18 | sean-k-mooney | the _OR_ADMIN version | |
| 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 | |