| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-11 | |||
| 12:44:43 | Uggla | sean-k-mooneythx | |
| 12:44:47 | Uggla | sean-k-mooney, thx | |
| 13:26:07 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing instance-action finish_time https://review.opendev.org/c/openstack/nova/+/928933 | |
| 14:18:21 | dansmith | gibi: I'm about to push up a couple typo fixes to your pci alias verification series, as noted in review comments | |
| 14:18:23 | dansmith | and then I will fast approve after that | |
| 14:18:25 | dansmith | hope that's okay | |
| 14:21:40 | gibi | dansmith: sure, thanks. I trust your English :) | |
| 14:21:59 | dansmith | I've been practicing it for longer than I'd like to admit :) | |
| 14:22:40 | gibi | :) | |
| 14:23:21 | opendevreview | Dan Smith proposed openstack/nova master: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/944062 | |
| 14:23:21 | opendevreview | Dan Smith proposed openstack/nova master: Return HTTP400 for multi spec pci alias if PCI in Placement https://review.opendev.org/c/openstack/nova/+/951007 | |
| 14:23:22 | opendevreview | Dan Smith proposed openstack/nova master: Validate [pci]alias at service startup https://review.opendev.org/c/openstack/nova/+/951143 | |
| 14:23:22 | opendevreview | Dan Smith proposed openstack/nova master: Validated that PCI alias has proper ids https://review.opendev.org/c/openstack/nova/+/951129 | |
| 14:23:23 | opendevreview | Dan Smith proposed openstack/nova master: Cache [pci]alias parsing https://review.opendev.org/c/openstack/nova/+/951151 | |
| 14:24:43 | dansmith | bauzas: not sure why you didn't, but can you upgrade to +2+W here? https://review.opendev.org/c/openstack/nova/+/944062/6 | |
| 14:25:56 | bauzas | done | |
| 14:26:12 | gibi | dansmith: bauzas: thanks for the reviews there | |
| 14:26:12 | dansmith | bauzas: thanks, there's one other in that series you didn't seem to hit that forms a hole | |
| 14:26:24 | dansmith | bauzas: https://review.opendev.org/c/openstack/nova/+/951129/4 | |
| 14:26:40 | dansmith | an unapproved sandwiched between approveds | |
| 14:27:04 | bauzas | I'll review the series when I'm done with J*RA | |
| 14:32:36 | opendevreview | Merged openstack/nova master: Add functional reproducer for bug 2102038 https://review.opendev.org/c/openstack/nova/+/944061 | |
| 14:33:14 | Uggla | FYI now: [public]Nova Eventlet removal sync --> meet.google.com/bcy-uqoz-hje | |
| 22:36:11 | melwitt | Uggla, bauzas: when I was looking at service versions I noticed the oldest version needs to be bumped and found this patch https://review.opendev.org/c/openstack/nova/+/944018 it should be to merge now right? | |
| 22:43:03 | opendevreview | Merged openstack/nova master: Remove unused config options https://review.opendev.org/c/openstack/nova/+/951945 | |
| #openstack-nova - 2025-06-12 | |||
| 03:13:24 | opendevreview | Merged openstack/nova master: Allow autopep8 to fix more things https://review.opendev.org/c/openstack/nova/+/949292 | |
| 03:13:36 | opendevreview | Merged openstack/nova master: reorder and extend pre-commit hooks https://review.opendev.org/c/openstack/nova/+/949299 | |
| 03:28:07 | opendevreview | Merged openstack/nova master: Return HTTP400 for multi spec pci alias if PCI in Placement https://review.opendev.org/c/openstack/nova/+/951007 | |
| 03:28:19 | opendevreview | Merged openstack/nova master: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/944062 | |
| 03:28:56 | opendevreview | Merged openstack/nova master: Validated that PCI alias has proper ids https://review.opendev.org/c/openstack/nova/+/951129 | |
| 03:31:13 | opendevreview | Merged openstack/nova master: Validate [pci]alias at service startup https://review.opendev.org/c/openstack/nova/+/951143 | |
| 03:31:25 | opendevreview | Merged openstack/nova master: Cache [pci]alias parsing https://review.opendev.org/c/openstack/nova/+/951151 | |
| 06:38:34 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing instance-action finish_time https://review.opendev.org/c/openstack/nova/+/928933 | |
| 06:46:29 | Uggla | melwitt, I think you are right, we forget about this patch. | |
| 06:58:09 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix instance vm_state during shelve https://review.opendev.org/c/openstack/nova/+/934294 | |
| 10:22:07 | 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 | |
| 11:08:03 | 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 | |
| 11:47:41 | 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 | |
| 12:02:31 | 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 | |
| 12:58:10 | opendevreview | Johannes Beisiegel proposed openstack/nova master: Adds packing_host_pci_numa_cells_allocation_strategy option https://review.opendev.org/c/openstack/nova/+/952451 | |
| 13:25:58 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Add functional reproducer for bug 2102038 https://review.opendev.org/c/openstack/nova/+/952452 | |
| 13:25:59 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Return HTTP400 for multi spec pci alias if PCI in Placement https://review.opendev.org/c/openstack/nova/+/952453 | |
| 13:26:00 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/952454 | |
| 13:26:01 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Validated that PCI alias has proper ids https://review.opendev.org/c/openstack/nova/+/952455 | |
| 13:26:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Validate [pci]alias at service startup https://review.opendev.org/c/openstack/nova/+/952456 | |
| 13:26:03 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Cache [pci]alias parsing https://review.opendev.org/c/openstack/nova/+/952457 | |
| 13:57:13 | opendevreview | Merged openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 14:16:32 | opendevreview | Merged openstack/nova master: api: Address issues with hypervisors APIs https://review.opendev.org/c/openstack/nova/+/950867 | |
| 14:29:47 | stephenfin | sean-k-mooney: Every +W'd openapi patch is now merged and gmaan has most of the outstanding patches reviewed. The ball's in your court now if you're happy/eager for us to keep moving on these 🙏 | |
| 14:38:22 | opendevreview | Moritz Wanzenböck proposed openstack/nova master: Implement extend_volume for local devices https://review.opendev.org/c/openstack/nova/+/878763 | |
| 14:38:58 | sean-k-mooney | stephenfin: ack. not today but ill try and do a pass on some/all them tomorrow | |
| 14:39:06 | sean-k-mooney | thats really good progress | |
| 14:39:10 | stephenfin | thanks | |
| 14:39:13 | stephenfin | and yes, great progress | |
| 14:40:25 | sean-k-mooney | so there are 16 patches i think for nova outstanding. do you stil have another batch on your github once this next bactch is done or is this the full set - any followups needed | |
| 14:41:58 | sean-k-mooney | stephenfin: context im tryign to decied if i think we can get this compleded by m2 or not | |
| 14:42:23 | sean-k-mooney | it woudl be nice if we can but im not sure how much review time i can dedicate between now and then | |
| 14:43:29 | stephenfin | Ah, there's quite a few patches still outstanding. 20 in total, targeting 14 APIs (I've split the servers one up into 6 patches, as you've already seen) | |
| 14:43:55 | stephenfin | When is m2? Assuming it's less than ~6 weeks away, that sounds optimistic | |
| 14:44:30 | sean-k-mooney | so the 16 that are open and ~20 after. ok well i hope we can land the first batch of 16 before m2 and we can then continut with the rest. its july 3rd i think | |
| 14:44:48 | sean-k-mooney | so 3 weeks time | |
| 14:45:29 | sean-k-mooney | that not a deadline by any means i just wanted to knwo were we were since i knew you proviosly had more patches that were pending | |
| 14:46:52 | sean-k-mooney | i dont knwo what gmaan PTO plans are like and im also aware that they have the policy work to do this cycle so im just trying to guage it we are likely to get this compelte this cyle. i think we are still on track to do that | |
| 14:47:06 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing rdo experimental jobs https://review.opendev.org/c/openstack/nova/+/927762 | |
| 14:48:02 | sean-k-mooney | gibi: as an fyi whiel that "works" we get some volume related test failures if i recall corectly | |
| 14:48:12 | sean-k-mooney | im not really sure why | |
| 14:49:19 | sean-k-mooney | gibi: ^ we also need to clean up the fr2 branch of the operator to remvoe the duplicate definiton of the pragm that may or may not cause that job to fail | |
| 14:49:52 | sean-k-mooney | gibi: if it does nto work as exepcted let me knwo an ill take a quick look | |
| 14:53:36 | gibi | sean-k-mooney: thanks. I just told the story about this possibility to artom and then I wanted to see if it still works | |
| 14:53:40 | gibi | I will report back | |
| 14:54:18 | sean-k-mooney | we have the watcher-operator reporting on changes to master opendev.org prs | |
| 14:54:30 | sean-k-mooney | we had to swap it to use c10s contianers | |
| 14:54:51 | sean-k-mooney | so its defintly possibel btu this is actully going to break because we dropped 3.9 supprot | |
| 14:55:35 | sean-k-mooney | gibi: we can chat downstream about how to prot the test change form watcher-operator to nova-operator to make this work if your interested | |
| 14:55:46 | sean-k-mooney | although maybe not this evening | |
| 14:57:04 | sean-k-mooney | the short version is https://github.com/openstack-k8s-operators/watcher-operator/blob/main/.zuul.yaml#L308-L344 defiens a c10s based job that will run on all changes ot opendev master for watcher | |
| 14:57:45 | sean-k-mooney | we have both watcher and epoxy version fo those jobs so that we will be able to do some testing on stable/2025.1 to if needed | |
| 14:57:48 | sean-k-mooney | https://github.com/openstack-k8s-operators/watcher-operator/blob/main/.zuul.yaml#L153-L270 | |
| 14:58:47 | sean-k-mooney | chandan got all that working in the last month or so, if we want to improve our upstream nova testign we can definetly do that in the future | |
| 15:00:59 | gibi | thanks for the pointers | |
| 15:16:20 | gmaan | sean-k-mooney: no PTO planned until Sept. for policy, yes that is what I will be pushing now. but I will keep eyes on stephenfin series if anything new or need re-review | |
| 15:17:54 | sean-k-mooney | gmaan: just as a follow up any removal of the tempest policy tests wont happen until 2025.1 goes to unmaintaed right | |
| 15:18:40 | sean-k-mooney | so it will be removed in 2026.2 or 2027.1 ish | |
| 15:18:59 | gmaan | sean-k-mooney: there are no specific policy tests there but idea is to move the tempest tests to move to new defaults (as default) and we have one job testing tempest on old defaults also | |
| 15:19:08 | sean-k-mooney | sorry not policy | |
| 15:19:14 | sean-k-mooney | i ment the schema validation | |
| 15:19:52 | sean-k-mooney | so updating tempest to nolonger needign to validate the repsocne once stephens seriss is supported on all stable branches | |
| 15:20:51 | sean-k-mooney | assuming 2025.2 is the first release with fully coverage we need to keep the nova schema test until 2025.1 moves to unmainted right? | |
| 15:21:02 | gmaan | ok. we can discuss that if we need to remove but anyways that can only happen after 2026.1 to unmaintained (if Nova implement all in 2025.2). But we discussed in last PTG, we might keep both side | |
| 15:21:30 | sean-k-mooney | well once we have the coverage in tree | |
| 15:21:47 | sean-k-mooney | we shoudl not need to ask people to extend the tempest schdma validation for new changes | |
| 15:22:40 | sean-k-mooney | that really what im poking at. only doing that in one place, in nova going forwarwad instead o both in nova and tempest | |
| 15:23:11 | sean-k-mooney | but your right we can discuss that at the next ptg | |
| 15:23:19 | gmaan | I am ok for that but let's discuss during 2026.1 unmaintained status change time. if no objection then I am ok to remove | |
| 15:23:36 | sean-k-mooney | well its 2025.1 not 2026.1 | |
| 15:24:07 | sean-k-mooney | anyway not todays problem :) | |
| 15:24:16 | gmaan | next PTG might be too early, anyways we need to keep them until 2026.1 (after 2025.2 EOL but I am considering 2026.1 as immediate SLURP after Nova has all schema) | |
| 15:24:43 | sean-k-mooney | right but its not when the first slurp that supprot it that is imporant | |
| 15:24:54 | sean-k-mooney | it when the last slurp that does not that matters | |
| 15:25:14 | sean-k-mooney | so its when 2025.1 goies unmaintined that tempest could consider removing the vlaidation | |