| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-27 | |||
| 17:24:37 | sean-k-mooney | thats kind of a cop out but | |
| 17:24:46 | sean-k-mooney | incremental progress is still better then none | |
| 17:24:51 | dougszu | yeah - I've tested that, it's a path at least | |
| 17:24:59 | dougszu | thanks for the pointers | |
| 17:50:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Adjust validation helpers for a single-method future https://review.opendev.org/c/openstack/nova/+/936365 | |
| 17:50:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 17:50:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only check minimum API version https://review.opendev.org/c/openstack/nova/+/940325 | |
| 17:50:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 17:50:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 17:50:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 17:50:20 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 17:50:20 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 17:50:21 | opendevreview | Stephen Finucane proposed openstack/nova master: api: project/tenant and user IDs are not UUIDs https://review.opendev.org/c/openstack/nova/+/940369 | |
| 17:50:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 17:50:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 17:50:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 17:50:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 17:50:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 17:50:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 17:50:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remote consoles https://review.opendev.org/c/openstack/nova/+/940114 | |
| 17:50:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for keypairs APIs https://review.opendev.org/c/openstack/nova/+/940115 | |
| 17:50:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for image metadata APIs https://review.opendev.org/c/openstack/nova/+/940299 | |
| 17:50:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server topology API https://review.opendev.org/c/openstack/nova/+/940300 | |
| 17:50:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server diagnostics API https://review.opendev.org/c/openstack/nova/+/940506 | |
| 17:50:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server external events API https://review.opendev.org/c/openstack/nova/+/940508 | |
| 18:06:40 | bauzas | gmann: https://review.opendev.org/c/openstack/nova/+/942875 is a bugfix, so it won't be impacted by the FF | |
| 18:09:37 | gmann | bauzas: yeah, bug fix for documentation only and not in the code. At least it is to match the code change (done intentionally or unintentionally ) with documentation unless anyone object not to have any change in update server API response which is not discussed in the spec or original implementation. | |
| 18:10:25 | gmann | bauzas: but we want to change update server response also which is done in the original code change and in my change I am matching the documentation (api-ref, tests, releasenotes etc) accordingly | |
| 18:11:01 | gmann | bauzas: ditto for the https://review.opendev.org/c/openstack/nova/+/942875/2 also | |
| 18:11:26 | gmann | I found it when reviewing the 2.100 microversion https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:11:58 | gmann | also fixing the tempest test/schema also for those microversion https://review.opendev.org/c/openstack/tempest/+/942870 | |
| 18:12:39 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.96 for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942875 | |
| 18:12:52 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.98 doc/tests for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942878 | |
| 18:15:04 | bauzas | nova-cores, I eventually give an extra 3 days (until Thursday) for merging implementation that already got one +2 | |
| 18:15:32 | bauzas | please see https://etherpad.opendev.org/p/nova-2025.1-status for the current state of exceptions | |
| 18:17:06 | gibi | until Tuesday or Thursday? | |
| 18:17:46 | gmann | yeah, I am also holding this to approve in case FFE things here https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:18:04 | gmann | bauzas: can you write the date also along with Tuesday/day so that it will be clear | |
| 18:18:25 | gmann | I saw this and not clear which tuesday https://etherpad.opendev.org/p/nova-2025.1-status#L73 | |
| 18:18:45 | gmann | sorry this one https://etherpad.opendev.org/p/nova-2025.1-status#L84 | |
| 18:18:56 | gmann | 'show-scheduler-hints-in-server-details' | |
| 18:31:33 | gmann | bauzas: added these two microversion doc bug fixes detail in etherpad under 'Feature Complete With Follow Up Pending' https://etherpad.opendev.org/p/nova-2025.1-status | |
| 18:35:28 | sean-k-mooney | gmann: feature freeze is today | |
| 18:35:32 | sean-k-mooney | not tuesday | |
| 18:36:12 | sean-k-mooney | which is why i was hopign we could proceed with https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:36:20 | gmann | sean-k-mooney ok its today right not next thursday? | |
| 18:36:46 | sean-k-mooney | ill check again | |
| 18:36:50 | sean-k-mooney | maybe im off by a week | |
| 18:36:58 | gmann | days are confusing there, having date will be more clear | |
| 18:37:05 | sean-k-mooney | https://releases.openstack.org/epoxy/schedule.html#epoxy-3-milestone | |
| 18:37:12 | sean-k-mooney | February 27, 2025 i | |
| 18:37:15 | sean-k-mooney | so today | |
| 18:37:16 | gmann | I asked bauzas just to be more clear | |
| 18:37:23 | stephenfin | Uggla: fyi, you probably want to respin https://review.opendev.org/c/openstack/openstacksdk/+/880056 soon enough if that's intended for this release | |
| 18:37:59 | sean-k-mooney | gmann: nova observes the unifed feature freeze with is milesotone 3 which is today February 27, 2025 | |
| 18:38:15 | gmann | sean-k-mooney: ohk, great then we are on perfect time for 938604 | |
| 18:38:52 | sean-k-mooney | ya so we normally allow things that are +w to merge up until the next irc meeting | |
| 18:39:00 | Uggla | Hi stephenfin, I'd like to. But I'm unsure I'll find time to do it. :( | |
| 18:39:12 | sean-k-mooney | so if its approved by close of buisness PST | |
| 18:39:19 | sean-k-mooney | then we consider it in for FF | |
| 18:39:32 | sean-k-mooney | even if it only merges at the weekend | |
| 18:39:46 | gmann | yeah, just +W on that | |
| 18:39:53 | stephenfin | Guess it's for next cycle so. You'll just have to remember to backport it if you need it downstream | |
| 18:42:48 | Uggla | stephenfin, unfortunately yes. But my current prio is on VFIO PCI live migration. | |
| 18:43:58 | gmann | sean-k-mooney: these two microversion doc bug fixes are also up, it pass the CI but I fix one thing/rebased so you can wait until gate is green https://review.opendev.org/c/openstack/nova/+/942875 https://review.opendev.org/c/openstack/nova/+/942878 | |
| 18:44:24 | gmann | I *fixed* one thing | |
| 18:45:41 | sean-k-mooney | i saw the first one this morning but didnt review it yet | |
| 18:46:02 | gmann | k] | |
| 18:46:02 | sean-k-mooney | those are not subject to FF and can merge up to RC1 although the first we should backprot to stable | |
| 18:46:08 | gmann | cool | |
| 18:46:13 | sean-k-mooney | i think 2.96 was last cycle | |
| 18:47:06 | sean-k-mooney | oh it was carical | |
| 18:47:11 | sean-k-mooney | so 2024.1 | |
| 18:47:40 | gmann | yeah, that we need to backport https://github.com/openstack/nova/blob/stable/2024.2/nova/api/openstack/compute/rest_api_version_history.rst#296-maximum-in-20241-caracal-and-20242-dalmatian | |
| 18:48:01 | gmann | will do that once master change is merge | |
| 18:48:59 | sean-k-mooney | if i try to review it now i think ill jsut miss things, im sure its correct but ill try an look at this tomorrow | |
| 18:50:27 | gmann | thanks | |
| 19:44:00 | opendevreview | ribaudr proposed openstack/nova master: Update manager to allow vfio pci device live migration https://review.opendev.org/c/openstack/nova/+/942145 | |
| 19:44:00 | opendevreview | ribaudr proposed openstack/nova master: Update conductor and filters allowing migration with SR-IOV devices https://review.opendev.org/c/openstack/nova/+/942146 | |
| 19:44:01 | opendevreview | ribaudr proposed openstack/nova master: Update driver to map the targeted address for SR-IOV PCI devices https://review.opendev.org/c/openstack/nova/+/942147 | |
| 19:53:13 | sean-k-mooney | Uggla: i reviewed some of the series and left comments. im not sure what persecnetage fo those have been adressed in ^ if any but even with the extention bauzas is proposing im not sure we should merge the live migration part this cycle | |
| 19:53:31 | sean-k-mooney | ill try an take a look at this again tomorrow | |
| 19:53:59 | sean-k-mooney | but since we do not have tempest testing for this the funcitonal test weill need to be pretty exaustive | |
| 19:54:47 | sean-k-mooney | and i dont want ot merge any of the migration series until we are +2 on all the patches | |
| 20:23:54 | opendevreview | Merged openstack/nova master: Drop environment for Python 3.8 https://review.opendev.org/c/openstack/nova/+/939027 | |
| 20:44:30 | Callum027 | Hi there, yesterday I published a blueprint (https://blueprints.launchpad.net/nova/+spec/xml-image-meta) and patch (https://review.opendev.org/c/openstack/nova/+/942766) for implementing adding image_meta metadata to the libvirt XML for downstream services (primarily Ceilometer) to consume. I'd appreciate some feedback on the design of the XML and | |
| 20:44:31 | Callum027 | the patch itself if a core reviewer has free time to do so. Thanks! | |
| 22:53:29 | sean-k-mooney | Callum027: so what your propsoing is someitng i started workign on a year or two ago and then never had time to get back too | |
| 22:53:51 | sean-k-mooney | Callum027: i have some idea of what i would like to include if we extend the metadta futher | |
| 22:54:25 | sean-k-mooney | notable we currenly store some metadata about the image and falvor but we dont express that consitently in the xml | |
| 22:55:13 | sean-k-mooney | so if we do enchnce this i woudl liek to try and encode both consistently i.e. provide both the name and uuid for the flavor and image | |
| 22:55:31 | sean-k-mooney | provdrive the extra specs and properties for both | |
| 22:56:42 | Callum027 | Hi sean-k-mooney, interesting, got any links to what you proposed earlier? | |
| 22:57:35 | sean-k-mooney | kind of so we had a converation with the ironic folk about the fact that they wanted simialr metadat for ther own usage. | |
| 22:58:18 | sean-k-mooney | so i did a poc of factoring out the metadat functions https://review.opendev.org/c/openstack/nova/+/905406 which we combiend with there work | |
| 22:58:20 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/923910/5/nova/virt/driver.py#49 | |
| 22:58:27 | sean-k-mooney | and ^ were the calsess we ended up with | |
| 22:59:09 | sean-k-mooney | what i didnt end of pocign was basiclly the changes to print all fo the files in the xml | |
| 22:59:38 | sean-k-mooney | so for image you see it has id name and propereis | |
| 23:00:12 | sean-k-mooney | in my orginal poc i also had id name and "extra_specs" for flavor https://review.opendev.org/c/openstack/nova/+/905406/2/nova/virt/driver.py | |