| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-26 | |||
| 17:53:28 | sean-k-mooney | its apprently an arry of uuids | |
| 17:53:45 | sean-k-mooney | at least based on your tempets change | |
| 17:55:25 | sean-k-mooney | oh | |
| 17:55:36 | sean-k-mooney | it can be a single uuid as a stirng or an away or them | |
| 17:55:59 | sean-k-mooney | the 2.90 create exmaple has | |
| 17:56:04 | sean-k-mooney | "OS-SCH-HNT:scheduler_hints": { | |
| 17:56:06 | sean-k-mooney | "same_host": "48e6a9f6-30af-47e0-bc04-acaed113bb4e" | |
| 17:56:08 | sean-k-mooney | } | |
| 17:56:43 | sean-k-mooney | as an example in the request | |
| 17:56:58 | sean-k-mooney | so you can just do the same in the responce example | |
| 17:57:18 | sean-k-mooney | well without the OS-SCH-HNT prefix | |
| 17:57:52 | sean-k-mooney | everything else looked good excpti the example value | |
| 18:01:10 | dviroel | yeah, they are adding this in the request sample, but not in the template, but the test does not validate that | |
| 18:03:48 | melwitt | huh, yeah, this should not have been a regex 😝 https://github.com/openstack/nova/blob/master/doc/api_samples/servers/v2.63/server-create-req.json someone missed that in review | |
| 18:04:08 | sean-k-mooney | ya.. i noticed it was broken there | |
| 18:04:25 | sean-k-mooney | but given that out of scop of dviroel's patch i decied not to mention that | |
| 18:04:36 | sean-k-mooney | im sure we have other cases where that has happen | |
| 18:05:12 | melwitt | well he mentioned it above, that's how I found it | |
| 18:06:33 | sean-k-mooney | melwitt: since your about would you have tim eto take a look at dviroel patch since you have recently look at all this for the other api changes | |
| 18:07:15 | dviroel | i used the 'uuid' - and it the end, it was replace in the post server: https://opendev.org/openstack/nova/src/commit/8f57fa7359d57e186fc6b75424b4900dc63c564f/nova/tests/functional/api_sample_tests/test_servers.py#L70 | |
| 18:07:18 | melwitt | yeah, I can look at it | |
| 18:07:39 | sean-k-mooney | cool https://review.opendev.org/c/openstack/nova/+/938604/11 is the nova change and https://review.opendev.org/c/openstack/tempest/+/939375 is the tempest update, finall https://review.opendev.org/c/openstack/nova/+/942728 pulls those together | |
| 18:08:09 | sean-k-mooney | dviroel: i will try and look at the sdk and clinet changes also but nova and tempest are the priorities | |
| 18:09:52 | dviroel | ack, thanks sean-k-mooney - yeah, not so sure if my client change is correct, ptal when you have some time | |
| 18:21:22 | opendevreview | Douglas Viroel proposed openstack/nova master: Add support for showing scheduler_hints in server details https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:30:41 | gmann | sean-k-mooney: sure, checking | |
| 18:50:39 | opendevreview | ribaudr proposed openstack/nova master: Add live_migratable flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/942144 | |
| 18:50:39 | opendevreview | ribaudr proposed openstack/nova master: Augment the LiveMigrateData object https://review.opendev.org/c/openstack/nova/+/942143 | |
| 18:50:40 | opendevreview | ribaudr proposed openstack/nova master: Update conductor and filters allowing migration with SR-IOV devices https://review.opendev.org/c/openstack/nova/+/942146 | |
| 18:50:40 | opendevreview | ribaudr proposed openstack/nova master: Update manager to allow vfio pci device live migration https://review.opendev.org/c/openstack/nova/+/942145 | |
| 18:50:41 | 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:04:47 | 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:04:48 | 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:14:58 | mikal | Morning | |
| 19:19:22 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 19:21:52 | opendevreview | Ryota, Ochiai proposed openstack/nova master: doc: Delete deprecated settings of the installation guide https://review.opendev.org/c/openstack/nova/+/896074 | |
| 20:12:31 | gmann | sean-k-mooney: dviroel: reviewed the nova and tempest change, we are missing the update server APIs changes which is also impacted when we update the views/servers.py | |
| 20:13:04 | gmann | in nova side it is just documentation and test and in tempest tests we need to add schems and test for update server | |
| 20:16:28 | dviroel | gmann: oh thanks, I missed the update server. going to propose an update, thanks | |
| 20:20:48 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 20:29:59 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 20:53:24 | opendevreview | Douglas Viroel proposed openstack/nova master: Add support for showing scheduler_hints in server details https://review.opendev.org/c/openstack/nova/+/938604 | |
| 21:27:41 | opendevreview | Max proposed openstack/nova master: feat: nova-manage db instance events cleanup https://review.opendev.org/c/openstack/nova/+/916927 | |
| 22:03:36 | gmann | 34 | |
| 22:05:02 | gmann | please ignore | |
| 22:09:34 | gmann | dviroel_bbl: ack | |
| 22:11:34 | sean-k-mooney[m] | gmann: so dviroel dint modify server update in this api microverions | |
| 22:12:04 | sean-k-mooney[m] | if we missed that in this change its also missing from 2.98 | |
| 22:12:17 | sean-k-mooney[m] | for the image properies in server show | |
| 23:30:41 | dviroel | sean-k-mooney: and pinned_availability_zone in 2.96 too | |
| 23:31:24 | sean-k-mooney[m] | yes more then likely but we should fix that seperatly | |
| 23:33:48 | dviroel | my tempest change is not going to pass without theses fixes in tempest side, since now I am testing update server too, and thy are missing | |
| 23:38:38 | gmann | sean-k-mooney[m]: it does. this line change the update server API also https://review.opendev.org/c/openstack/nova/+/938604/12/nova/api/openstack/compute/views/servers.py#335 | |
| 23:39:42 | sean-k-mooney[m] | in directly via reusing show | |
| 23:40:03 | gmann | basically we have to keep these 4 APIs (GET, GET details, Rebuild, update server) response consistent (we fixed it in 2.74). I commented in the spec also to mention all these 4 APIs response change together | |
| 23:41:19 | gmann | I did not check earlier microversion 2.98 or 2.96 but we should keep all these 4 APIs response consistently and they get changed together when we change the views/servers.py | |
| #openstack-nova - 2025-02-27 | |||
| 00:20:45 | opendevreview | ribaudr proposed openstack/nova master: Bump MIN_{LIBVIRT,QEMU} for "Epoxy" https://review.opendev.org/c/openstack/nova/+/942871 | |
| 00:20:46 | opendevreview | ribaudr proposed openstack/nova master: WIP Below version constants removal for MIN_{LIBVIRT,QEMU} https://review.opendev.org/c/openstack/nova/+/942872 | |
| 00:24:55 | dviroel | gmann: thanks for proposing the fix to 2.96 and 2.98 in tempest | |
| 00:28:52 | gmann | dviroel: those should fix and nova documentation for 2.96, 2.98 we can fix after that | |
| 00:29:42 | dviroel | +1 | |
| 02:09:12 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.96 for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942875 | |
| 03:39:44 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.96 for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942875 | |
| 04:53:16 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.98 doc/tests for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942878 | |
| 09:17:05 | jkulik | anybody up for a review on https://review.opendev.org/c/openstack/nova/+/942371 ? it's a 2-line fix for a bug in libvirt's maxphysaddr support | |
| 09:22:14 | gibi | jkulik: thanks for fixing that bug, if you can fix my comment in the unit test then I'm happy to +2 it | |
| 09:22:53 | jkulik | gibi: thanks. will do | |
| 09:33:12 | opendevreview | Johannes Kulik proposed openstack/nova master: libvirt: fix maxphysaddr passthrough dom parsing https://review.opendev.org/c/openstack/nova/+/942371 | |
| 09:55:43 | gibi | jkulik: thanks, +2 | |
| 10:17:32 | frickler | looks like tempest.api.compute.servers.test_create_server.ServersV294TestFqdnHostnames.test_verify_hostname_allows_fqdn is unstable, haven't seen this before https://zuul.opendev.org/t/openstack/build/4cb5f1127e464ab3b2b5cf651fd5ac9e | |
| 10:54:17 | opendevreview | ribaudr proposed openstack/nova master: WIP Below version constants removal for MIN_{LIBVIRT,QEMU} https://review.opendev.org/c/openstack/nova/+/942872 | |
| 10:54:17 | opendevreview | ribaudr proposed openstack/nova master: Bump MIN_{LIBVIRT,QEMU} for "Epoxy" https://review.opendev.org/c/openstack/nova/+/942871 | |
| 11:02:04 | Uggla | sean-k-mooney[m], it you have time, can you review migration serie --> 942143: Augment the LiveMigrateData object | https://review.opendev.org/c/openstack/nova/+/942143 | |
| 11:24:51 | sean-k-mooney | Uggla: i can see if i can fidn time i reviewd one patch breifly last night | |
| 11:50:35 | sean-k-mooney | bauzas: can i get your advice on something. one of the things i have said i woudl do for watcher as part of the release liason role is prepare the cycle highlights and release note prelude. | |
| 11:50:43 | sean-k-mooney | bauzas: im lookign at https://docs.openstack.org/nova/latest/contributor/ptl-guide.html#milestone-3 | |
| 11:50:56 | sean-k-mooney | and the thing that need to be doen between m3 and rc1 | |
| 11:51:35 | sean-k-mooney | i dont see the release note prelude in that list but do you normally base one on teh toher i.e. the prelude on the cycle highlight and vise versa | |
| 11:52:40 | sean-k-mooney | is there anything that is easy to miss doing that is time sensive that you would call out | |
| 11:59:50 | sean-k-mooney | watcher does not have a equivalent doc so im probely goitn go steal nova and updte it a lltie with a new title and add it to watcher after the release | |
| 12:00:20 | sean-k-mooney | "Chronological Release Milestone guide" or soemthing like that | |
| 12:14:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add new option to enforce multipath volume connections https://review.opendev.org/c/openstack/nova/+/845660 | |
| 12:16:14 | tkajinam | Does anyone have time to review this tiny cleanup https://review.opendev.org/c/openstack/nova/+/939027 ? | |
| 12:20:17 | tkajinam | stephenfin, thanks ! | |
| 12:22:17 | sean-k-mooney | yep looks good | |
| 12:22:33 | sean-k-mooney | we will want to add 3.13 early next cycle | |
| 12:24:31 | stephenfin | and hopefully remove 3.9 | |
| 12:25:04 | tkajinam | hopefully though you may have to make sure that c9s is fully replaced by c10s in RDO | |
| 12:25:21 | stephenfin | indeed | |
| 12:25:31 | tkajinam | IIRC python 3.9 is the system python in c9s | |
| 12:26:04 | sean-k-mooney | 3.9 cant be removed until 2026.1 | |
| 12:26:22 | sean-k-mooney | we need to support both c9s and c10s in 2025.2 for upgrades | |
| 12:27:03 | tkajinam | you can make 2025.1 as the pivot between c9s and c10s I guess | |
| 12:27:28 | tkajinam | as we did for train | |
| 12:27:28 | sean-k-mooney | no becuase it was not aviable at the start of the 2025.1 cycle | |
| 12:27:29 | tkajinam | which may be better given some people may try direct upgrade from 2025.1 to 2026.1 | |
| 12:27:36 | sean-k-mooney | so its not in the officl testing runtimes for this release | |
| 12:27:44 | tkajinam | yeah that's the problem | |
| 12:27:50 | sean-k-mooney | the first tiem c10s can be added is 2025.2 | |
| 12:28:03 | tkajinam | there may be number of things we have to address such as redis->valkey | |