| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-06 | |||
| 19:08:24 | noonedeadpunk | or well. Since Liberty: https://review.opendev.org/c/openstack/openstack-ansible/+/227839 | |
| 19:17:21 | gmaan | melwitt: about project-admin comment in manager role spec, sean-k-mooney and I replied but I agree to add some explanation in spec. While I am doing that, I wanted to check if it is ok to add that explanation in follow up and we merge current change or you would like to see that in this change itself. https://review.opendev.org/c/openstack/nova-specs/+/937650/9/specs/2025.2/approved/policy-manager-role-default.rst#29 | |
| 19:17:25 | gmaan | either is ok for me | |
| 20:17:58 | opendevreview | Merged openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 20:42:23 | melwitt | gmaan: cool thanks, I'm reading through. I don't think it's necessary to follow up or change it, it was just a bit of context I was missing or didn't already know | |
| 20:43:14 | melwitt | this whole time I thought project-admin meant admin but only for a given project. if it's global, then I don't understand what's the difference between system-admin and project-admin then 😛 | |
| 20:44:40 | melwitt | but the important point is that if project-admin is global then that makes it obvious the usefulness of project-manager | |
| 21:49:37 | gmaan | melwitt: yeah, by doing back and forth on those admin, we end up keeping only one admin (legacy admin) who can do things on system level | |
| 21:51:37 | gmaan | if we could have system admin then it could make admin at different level more clear/safe to use but that confused operator instead of helpful :) | |
| 22:38:10 | opendevreview | Merged openstack/nova-specs master: Propose API policy manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| #openstack-nova - 2025-06-07 | |||
| 01:45:26 | opendevreview | melanie witt proposed openstack/nova stable/2025.1: libvirt: Use common naming convention for ephemeral disk labels https://review.opendev.org/c/openstack/nova/+/952003 | |
| 01:46:41 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: libvirt: Use common naming convention for ephemeral disk labels https://review.opendev.org/c/openstack/nova/+/952004 | |
| 01:47:25 | opendevreview | melanie witt proposed openstack/nova stable/2024.2: libvirt: Use common naming convention for ephemeral disk labels https://review.opendev.org/c/openstack/nova/+/952004 | |
| 01:48:47 | opendevreview | melanie witt proposed openstack/nova stable/2024.1: libvirt: Use common naming convention for ephemeral disk labels https://review.opendev.org/c/openstack/nova/+/952005 | |
| 02:53:05 | opendevreview | Merged openstack/nova master: api: Only check minimum API version https://review.opendev.org/c/openstack/nova/+/940325 | |
| 07:14:41 | opendevreview | Merged openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 16:20:53 | opendevreview | Merged openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| #openstack-nova - 2025-06-08 | |||
| 16:38:35 | opendevreview | Merged openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 22:51:31 | opendevreview | Merged openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 23:14:09 | opendevreview | Merged openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| #openstack-nova - 2025-06-09 | |||
| 11:22:50 | stephenfin | gmaan: When you're around, I think I've addressed your comments (via follow-ups) on the following: https://review.opendev.org/c/openstack/nova/+/937047/ https://review.opendev.org/c/openstack/nova/+/937048/ https://review.opendev.org/c/openstack/nova/+/937246/ | |
| 11:23:13 | stephenfin | and sean-k-mooney: you can probably re-add your +2 on https://review.opendev.org/c/openstack/nova/+/937048/ also since I have a follow-up now | |
| 11:26:36 | sean-k-mooney | ack so it was just in the test code | |
| 11:27:23 | sean-k-mooney | and only 5 tests at that | |
| 11:28:37 | sean-k-mooney | stephenfin: done | |
| 14:21:18 | stephenfin | Uggla: On https://review.opendev.org/c/openstack/nova/+/940114, I had intentionally included since-removed values in the enum ('xvpvnc') since the schemas need to work for all versions of Nova. Does that make sense? | |
| 14:22:41 | stephenfin | IMO, the api-ref should probably list those also since that's unversioned. I removed them I2f7f2379d0cd54e4d0a91008ddb44858cfc5a4cf. Perhaps I should add them back but with a warning that they won't work on modern (>= Ussuri) nova | |
| 15:05:55 | gmaan | stephenfin: ack, could not finish on friday but I will continue on those today. | |
| 16:15:36 | opendevreview | Arnaud Morin proposed openstack/nova master: Fix small documentation issue https://review.opendev.org/c/openstack/nova/+/952151 | |
| #openstack-nova - 2025-06-10 | |||
| 07:10:06 | opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Show finish_time field in instance action show https://review.opendev.org/c/openstack/nova-specs/+/929780 | |
| 07:47:32 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing instance-action finish_time https://review.opendev.org/c/openstack/nova/+/928933 | |
| 07:55:12 | opendevreview | Friedrich Hiekel proposed openstack/nova master: Fix nova-scheduler placement error https://review.opendev.org/c/openstack/nova/+/951936 | |
| 09:21:32 | stephenfin | sean-k-mooney: any chance you could grab https://review.opendev.org/c/openstack/nova/+/950867 again? | |
| 09:22:50 | opendevreview | Merged openstack/nova master: Fix small documentation issue https://review.opendev.org/c/openstack/nova/+/952151 | |
| 09:48:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213 | |
| 09:53:06 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with keypairs APIs https://review.opendev.org/c/openstack/nova/+/952214 | |
| 10:17:43 | stephenfin | Uggla: Follow-ups for your comments on the openapi series are there, if and when you have time ^ | |
| 10:51:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with instance actions API https://review.opendev.org/c/openstack/nova/+/951941 | |
| 12:10:55 | sean-k-mooney | stephenfin: yep added +2 | |
| 12:14:43 | sean-k-mooney | by the way im still.... seeing random kernel panics related to cinder volume detach. i know we never fixed that but im wondering if we shoudl consider creating a decorator in tempest to detect that and retry the test. i.e. if the console output form tempest has "end Kernel panic" in it just retry the test once more and only fail if that second try also fails | |
| 12:21:53 | Uggla | Hi stephenfin, I will review it thx. | |
| 12:33:53 | sean-k-mooney | stephenfin: replied on https://review.opendev.org/c/openstack/openstacksdk/+/948584 thansk for the review, i might not get back to it for a few days but ill let you knwo when i have a new revision up | |
| 12:36:33 | stephenfin | sean-k-mooney: ack | |
| 15:03:32 | Uggla | Nova meeting in ~1h. | |
| 15:04:29 | cardoe | I wanted to follow up on https://review.opendev.org/c/openstack/nova-specs/+/471815 and ask what my next steps are to advance that. | |
| 15:08:30 | opendevreview | Merged openstack/os-vif master: OVS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/949736 | |
| 15:21:23 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Resolve alembic deprecation warning https://review.opendev.org/c/openstack/nova/+/952266 | |
| 15:21:24 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | |
| 15:21:24 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | |
| 15:21:25 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | |
| 15:21:25 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | |
| 15:21:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | |
| 15:21:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| 15:41:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api-ref: Fix server topology API docs https://review.opendev.org/c/openstack/nova/+/952274 | |
| 15:42:22 | stephenfin | Uggla: That last one is for you | |
| 15:50:35 | Uggla | stephenfin, thanks for fixing this. | |
| 15:50:46 | Uggla | Nova meeting in ~10mn | |
| 16:00:54 | bauzas | tick tock | |
| 16:01:12 | Uggla | #startmeeting nova | |
| 16:01:12 | opendevmeet | Meeting started Tue Jun 10 16:01:12 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:13 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:13 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:25 | Uggla | Hello everyone | |
| 16:02:09 | fwiesel | o/ | |
| 16:02:32 | bauzas | o/ | |
| 16:03:32 | dansmith | o/ | |
| 16:04:05 | tkajinam | o/ | |
| 16:04:38 | Uggla | Let's start | |
| 16:04:47 | Uggla | #topic Bugs (stuck/critical) | |
| 16:04:56 | Uggla | #info No Critical bug | |
| 16:05:06 | gmaan | o/ | |
| 16:05:50 | Uggla | #info https://review.opendev.org/c/openstack/nova/+/922140 bauzas working on a patch to enable mdev test again. | |
| 16:06:08 | bauzas | had no time to look at it :( | |
| 16:06:42 | Uggla | bauzas no worries. | |
| 16:07:06 | Uggla | If you wish I can remove it from this topic. | |
| 16:07:50 | Uggla | gibi, are you crazy. ;) | |
| 16:08:14 | Uggla | #topic Gate status | |
| 16:08:27 | Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:08:33 | Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:08:40 | Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | |
| 16:08:47 | Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:08:53 | Uggla | #info Please try to provide meaningful comment when you recheck | |
| 16:09:34 | Uggla | Note: I have not checked periodics this week. | |
| 16:09:50 | Uggla | #topic tempest-with-latest-microversion job status | |
| 16:09:57 | Uggla | #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=0 | |
| 16:10:17 | Uggla | gmaan do you want to add something abut ^ | |
| 16:10:18 | gmaan | did not get chance to do any progress on it. | |
| 16:10:33 | gmaan | nothing to share from me | |
| 16:10:44 | Uggla | ok, no pb, moving on. | |
| 16:10:54 | Uggla | #topic Release Planning | |
| 16:11:02 | Uggla | #link https://releases.openstack.org/flamingo/schedule.html | |
| 16:11:16 | Uggla | #info Nova Spec review day will be next week. | |
| 16:11:49 | Uggla | #info IMHO, we should start reviewing specs. | |
| 16:12:07 | Uggla | #topic Review priorities | |
| 16:12:17 | Uggla | #link https://etherpad.opendev.org/p/nova-2025.2-status | |
| 16:12:48 | Uggla | I will make sure it is up2date with current reviews for specs. | |
| 16:13:26 | Uggla | But that should be not too bad. I think almost every spec is part of the document. | |
| 16:14:34 | Uggla | #topic OpenAPI | |
| 16:14:41 | Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | |
| 16:14:47 | Uggla | #info 18 patches remaining lots of them in the pipe ready to be merged. | |