| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-25 | |||
| 08:31:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix neutron client dict grabbing https://review.opendev.org/c/openstack/nova/+/952966 | |
| 08:35:54 | gibi | dansmith sean-k-mooney thanks for the review on https://review.opendev.org/c/openstack/nova/+/952966 I did a bit of simplification and added a comment. Please check it when you are around | |
| 08:37:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix neutron client dict grabbing https://review.opendev.org/c/openstack/nova/+/952966 | |
| 08:54:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213 | |
| 11:29:12 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing py3.13 eventlet bug workaround https://review.opendev.org/c/openstack/nova/+/951749 | |
| 12:15:39 | sean-k-mooney | gibi: by the way ^ some how seams to be triggering issues with mysql, rather then using pyenv i think i need to enabel the deadsnakes ppa or try a diffent approch to get 3.13 working | |
| 12:16:05 | sean-k-mooney | gibi: how did you test with 3.13 locally | |
| 12:20:33 | gibi | I tested it with py312, jkulik tested it with 3.13. I don't have a 3.13 devstack env. | |
| 12:21:16 | gibi | I only tested the small change in isoltion with py313 about the dict grabbing | |
| 12:21:33 | gibi | we can also ask zigo to test it :) | |
| 12:22:09 | sean-k-mooney | im ok with merging it without full 3.13 testing provided it works on older verion as it clean up the code in general | |
| 12:22:24 | sean-k-mooney | i was just wondering how to pivot my WIP patch | |
| 12:22:47 | zigo | You'd need the latest version of eventlet, plus maybe a fix for https://github.com/python/cpython/issues/135552 then hopefully, we can spawn VMs ... :P | |
| 12:22:52 | sean-k-mooney | i was condierign using either UV to try and provide a python env in the hope that avoid teh mysql issue ro deadsnake ppa | |
| 12:23:09 | sean-k-mooney | althernitivly i coudl try using a diffent distor for my 3.13 poc job | |
| 12:24:31 | gibi | zigo: I guess you are grabbing this fix from nova until the py3.13 GC bug is fixed upstream in cpython. So I guess if it works for you then that is good enough for us. | |
| 12:25:15 | zigo | I didn't have time to work on OpenStack on Trixie (ie: Debian 13 and Python 1.13) for a week or so. | |
| 12:25:20 | zigo | I intend to resume that work again. | |
| 12:25:29 | zigo | I currently still can't tell what works or not. | |
| 12:25:41 | zigo | At least I was able to spawn *one* VM, but that's all I can say. | |
| 12:25:48 | zigo | Maybe other stuff are broken too. | |
| 12:26:03 | zigo | I wont be able to tell until I run the full tempest set of tests. | |
| 12:26:15 | gibi | zigo: thanks. I appreciate all the effor you do and the feeback you provide to us | |
| 12:28:58 | zigo | Whenever I can, I'll push my CI up to end and report in the list my findings. | |
| 12:33:49 | gibi | thanks | |
| 12:48:03 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 14:27:29 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 15:00:17 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 15:08:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 15:08:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 15:08:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 15:08:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 15:08:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 15:08:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 15:08:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 15:08:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 15:08:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 15:08:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 15:08:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 15:08:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 15:08:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 15:36:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 15:36:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 15:36:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 15:36:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 15:36:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 15:36:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 15:36:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 15:36:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 15:36:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 15:36:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 15:36:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 15:36:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 15:36:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 16:01:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Note on RPC error decorators around build_and_run_instance https://review.opendev.org/c/openstack/nova/+/953121 | |
| 16:02:41 | gibi | dansmith: ^^ I put back the error handling decorators as we discussed | |
| 16:49:49 | dansmith | gdi | |
| 16:54:45 | gibi | hm | |
| 17:10:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/953338 | |
| 17:23:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/953339 | |
| #openstack-nova - 2025-06-26 | |||
| 01:17:06 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove Python 3.9 support https://review.opendev.org/c/openstack/placement/+/953367 | |
| 01:17:24 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove Python 3.9 support https://review.opendev.org/c/openstack/placement/+/953367 | |
| 02:23:23 | opendevreview | Merged openstack/nova master: api: Add response body schemas for remote consoles https://review.opendev.org/c/openstack/nova/+/940114 | |
| 02:24:53 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove Python 3.9 support https://review.opendev.org/c/openstack/placement/+/953367 | |
| 07:19:58 | opendevreview | Merged openstack/nova master: api: Add response body schemas for keypairs APIs https://review.opendev.org/c/openstack/nova/+/940115 | |
| 07:21:07 | opendevreview | Rodolfo Alonso proposed openstack/nova master: [DNM] Test "grenade-skip-level-always" job https://review.opendev.org/c/openstack/nova/+/953402 | |
| 08:15:26 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 08:45:52 | stephenfin | sean-k-mooney: Could you grab this openapi follow-up? https://review.opendev.org/c/openstack/nova/+/952214/2 There's another one here but it's failing functional tests currently (I'm working on it) https://review.opendev.org/c/openstack/nova/+/952213 | |
| 09:03:23 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213 | |
| 10:36:02 | sean-k-mooney | stephenfin: looks fine to me | |
| 11:34:22 | opendevreview | Merged openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 11:35:07 | opendevreview | Merged openstack/nova master: api: Add response body schemas for image metadata APIs https://review.opendev.org/c/openstack/nova/+/940299 | |
| 12:11:53 | sahid | o/ | |
| 12:12:46 | sahid | have you already suffered issue like Live Migration failure: <mac address> | |
| 12:13:15 | sahid | I'm not able to find what could be the issue and unfortunatly the exception is logged like that | |
| 12:13:48 | sahid | LOG.error("Live Migration failure: %s", e, instance=instance) | |
| 12:13:48 | sahid | with excutils.save_and_reraise_exception(): | |
| 12:13:48 | sahid | except Exception as e: | |
| 12:14:05 | sahid | it s in live migration operation | |
| 12:28:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove unnecessary test module level code https://review.opendev.org/c/openstack/nova/+/953436 | |
| 12:47:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 12:56:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 12:58:58 | stephenfin | gmaan: Got a follow-up for you to re-approve here, when you've time https://review.opendev.org/c/openstack/nova/+/952213/ | |
| 13:10:14 | stephenfin | sean-k-mooney: Oh, I also addressed your comment on https://review.opendev.org/c/openstack/nova/+/952281/ here https://review.opendev.org/c/openstack/nova/+/953281/ | |
| 13:10:28 | stephenfin | and thanks for the review 🙏 | |
| 13:26:18 | sean-k-mooney | stephenfin: cool +2w on teh organl patch and +2 on the fixup patch | |
| 13:28:12 | sean-k-mooney | sahid: was it for an sriov port? i know that there have been issues with sriov ports adn attach/detach if there are more then one port with the same mac. havign a vm with multipel prot with the same mac is sort of unsupproted. | |
| 13:28:15 | sean-k-mooney | its not | |
| 13:28:37 | sean-k-mooney | but it was never expiclty added as a feature and kind fo bolted on as bugs were reported | |
| 13:28:56 | sean-k-mooney | it woudl never happen with nova networks and it cant happen in a signel neutron network | |
| 13:29:43 | sean-k-mooney | so nova never really blocked it. over time as neturon becausem the default we got bug reports that usign the mac to attach/dtach the prots failed fi you had a vm with 2 prots htat had the same mac | |
| 13:30:19 | sean-k-mooney | sahid: we coudl also hith that internally when live migrating vms with sriov vnic-type=direct ports | |
| 13:30:47 | sean-k-mooney | sahid: that is the only mac related live migration failure that i recall | |
| 13:31:52 | opendevreview | Merged openstack/nova master: api: Add response body schemas for server topology API https://review.opendev.org/c/openstack/nova/+/940300 | |
| 13:37:23 | sahid | sean-k-mooney: thanks for this, we don't use srio port, but it's more or less realted to vms AVI loadbalancer | |
| 13:53:26 | sahid | it's strange, when i'm doing a port list for given vm that is buggy, there is 2 ports attached, and when looking at the xml, 5 interfaces are indicated | |
| 13:53:59 | sahid | sean-k-mooney: have you already noticed something like that, where xml is not well cleaned? | |
| 15:03:35 | opendevreview | Pierre Riteau proposed openstack/nova stable/2024.2: Add repoducer test for bug 2074219 https://review.opendev.org/c/openstack/nova/+/953468 | |
| 15:35:44 | opendevreview | Pierre Riteau proposed openstack/nova stable/2024.2: Fix detaching devices by alias with mdevs https://review.opendev.org/c/openstack/nova/+/953474 | |
| 15:40:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |