| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-25 | |||
| 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 | |
| 16:44:53 | gmaan | stephenfin: one question about doc generation, if we allow rdp in schema then how we reflect that it is gone in latest version https://review.opendev.org/c/openstack/nova/+/952213/comment/e644f810_7e7b50c2/ | |
| 17:24:53 | stephenfin | gmaan: that depends on whether you're talking about the API perspective (i.e. clients) or the docs (i.e. humans/AI scrapers) | |
| 17:25:47 | stephenfin | from the API/client perspective, we're already handling that by returning HTTP 400 (Bad Request) if you attempt to use it on a newer version of Nova (or an earlier version not using HyperV) so nothing needs to change there, IMO | |
| 17:27:03 | stephenfin | gmaan: From the latter perspective, we'd likely handle that by either (a) indicating it in the 'description' text or (b) using a custom property that os-api-ref or its replacement can pick up | |
| 17:29:29 | gmaan | stephenfin: yeah, we should update the description saying it is gone but my main concern in this change is how to indicate that now via response schema | |
| 17:30:42 | stephenfin | gmaan: beyond what we're doing here? https://review.opendev.org/c/openstack/nova/+/952213/4/nova/api/openstack/compute/remote_consoles.py#161 | |
| 17:31:38 | stephenfin | With this patch, we're moving validation (and rejection) from the schema into the method itself. The end result is the same though. | |
| 17:31:52 | gmaan | I am thinking either we need to version (add different schema for each releases if changed) schema based on releases like we do for microversions by adding the info in current api-ref description | |
| 17:32:35 | gmaan | stephenfin: code behavior wise it is all good that is why i am +1. I am thinking what is usage of showing it in schema and how to handle the case where we can say it is gone after caracal | |
| 17:33:06 | stephenfin | So to be clear, you're worried about what the OpenAPI-derived replacement for api-ref will look like? | |
| 17:33:13 | gmaan | exactly | |
| 17:33:44 | gmaan | I do not want new OpenAPI-derived replacement show rdp as valid value | |
| 17:33:52 | gmaan | if it consume response schema as it is | |
| 17:34:19 | stephenfin | Well it is valid *if* you're using pre-Caracal and Hyper-V | |