| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-04-13 | |||
| 17:35:45 | gmaan | sean-k-mooney: or the healthcheck proposal is good fit here but is the latest proposal check RPC health also? | |
| 17:35:55 | phildree[m] | it like a tcp Syn (compute send a request to any conductor), Syn-Ack (conductor replies to that tiny request), Ack (compute initiates the regular heartbeat as is) | |
| 17:35:55 | phildree[m] | This way when any part of the communction is broken the node does not appear as up | |
| 17:35:55 | phildree[m] | But in order to reduce RPCs i think also just a periodic ping without compute base initiation might be feasible but will have the issue of coordination etc.. | |
| 17:36:21 | sean-k-mooney | gmaan: it was goign to catch connection closed events and similar on the comptue and report if the rpc bus was working | |
| 17:36:26 | sean-k-mooney | but not by active probing | |
| 17:36:34 | sean-k-mooney | only by recoreding if we got an excption | |
| 17:36:34 | gmaan | k | |
| 17:36:41 | sean-k-mooney | so it woudl not really help in this case | |
| 17:36:41 | gmaan | ++ | |
| 17:36:50 | gmaan | yeah | |
| 17:38:53 | gmaan | do not have any perfect solution as of now but let's discuss it in PTG | |
| 17:39:47 | phildree[m] | Ok, thanks anyway for the engagement, I really appreciate you guys taking your time :) | |
| 19:20:17 | opendevreview | Merged openstack/nova stable/2026.1: fix: device_by_alias should respect config type https://review.opendev.org/c/openstack/nova/+/983989 | |
| 19:58:52 | opendevreview | John Garbutt proposed openstack/nova master: Make PCPUs not land on VCPUs by default https://review.opendev.org/c/openstack/nova/+/975779 | |
| 19:59:20 | opendevreview | John Garbutt proposed openstack/nova master: Fix ironic instance rebuild with API >=2.93 https://review.opendev.org/c/openstack/nova/+/963298 | |
| 20:24:21 | melwitt | dansmith: there is one more small patch that was split out from the 'user' live migration patch that is ready for review at your convenience https://review.opendev.org/c/openstack/nova/+/983505 | |
| 21:00:29 | dansmith | ah yep, the other one, sorry I guess I missed that when hitting the first one | |
| 21:00:30 | dansmith | done now | |
| 21:16:32 | melwitt | dansmith: thanks!! | |
| 22:46:32 | opendevreview | melanie witt proposed openstack/nova master: [WIP] how borken is ipv6 only ceph. https://review.opendev.org/c/openstack/nova/+/982302 | |
| 22:51:07 | opendevreview | melanie witt proposed openstack/nova master: [WIP] how borken is ipv6 only ceph. https://review.opendev.org/c/openstack/nova/+/982302 | |
| #openstack-nova - 2026-04-14 | |||
| 00:30:42 | opendevreview | melanie witt proposed openstack/nova master: CellDatabases: replace shared lock with thread-local cell routing https://review.opendev.org/c/openstack/nova/+/984480 | |
| 00:30:43 | opendevreview | melanie witt proposed openstack/nova master: Database: use file-backed SQLite for native threading https://review.opendev.org/c/openstack/nova/+/984481 | |
| 00:38:55 | melwitt | sean-k-mooney, gibi: I had read through the review comments on some database fixture patches yall have been working on and tried a couple of changes to try and help out ^ they may not be useful given my limited knowledge on the eventlet removal work | |
| 00:40:38 | melwitt | I have them rebased on Ashish's change https://review.opendev.org/c/openstack/nova/+/979850. the second patch complains of merge conflict for some reason though | |
| 01:02:48 | opendevreview | melanie witt proposed openstack/nova master: Database: use file-backed SQLite for native threading https://review.opendev.org/c/openstack/nova/+/984481 | |
| 01:15:45 | opendevreview | melanie witt proposed openstack/nova master: Database: use file-backed SQLite for native threading https://review.opendev.org/c/openstack/nova/+/984481 | |
| 06:20:42 | opendevreview | Merged openstack/nova master: mem-enc: stop using _get_mem_encryption_config() for SEV checks https://review.opendev.org/c/openstack/nova/+/967970 | |
| 07:14:59 | gibi | melwitt: thanks for that. I wil check it out | |
| 07:38:07 | opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | |
| 07:42:05 | opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | |
| 08:21:43 | gibi | bauzas: stephenfin: can one of you push this CI timeout bump through https://review.opendev.org/c/openstack/nova/+/984417 | |
| 08:45:03 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 09:25:37 | opendevreview | Kamil Sambor proposed openstack/nova master: Change nova-alt-configurations job https://review.opendev.org/c/openstack/nova/+/983179 | |
| 09:31:09 | opendevreview | ribaudr proposed openstack/nova master: Add reproducer for bug #2141722 https://review.opendev.org/c/openstack/nova/+/984521 | |
| 09:31:10 | opendevreview | ribaudr proposed openstack/nova master: Fix os-simple-tenant-usage pagination with duplicate instance_extra https://review.opendev.org/c/openstack/nova/+/984522 | |
| 10:10:06 | Uggla | sean-k-mooney, I have opened https://blueprints.launchpad.net/nova/+spec/adopt-port-trusted-vif-extension and add a comment in the bug to it. Please tell me if ok. | |
| 10:33:01 | opendevreview | Merged openstack/nova master: [openstack-tox-cover]Increase timeout further https://review.opendev.org/c/openstack/nova/+/984417 | |
| 10:44:42 | opendevreview | sean mooney proposed openstack/nova master: add agentic coding scaffold and contributor docs https://review.opendev.org/c/openstack/nova/+/984540 | |
| 12:46:22 | opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | |
| 13:08:41 | opendevreview | Merged openstack/nova master: mem-enc: refactor memory encryption trait logic for extensibility https://review.opendev.org/c/openstack/nova/+/967971 | |
| 13:58:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | |
| 13:58:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Reenable last scatter-gather unit test https://review.opendev.org/c/openstack/nova/+/984349 | |
| 13:59:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | |
| 13:59:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Reenable last scatter-gather unit test https://review.opendev.org/c/openstack/nova/+/984349 | |
| 14:21:51 | artem_vasilyev | Hi, could you please take a look at a Nova bug I’ve been investigating: https://bugs.launchpad.net/nova/+bug/2147776 (with reproducer tests https://review.opendev.org/c/openstack/nova/+/984007) and the proposed fix https://review.opendev.org/c/openstack/nova/+/984048 - it addresses a race between resize and update_available_resources affecting dedicated CPU accounting | |
| 15:05:58 | Uggla | gibi, regarding you eventlet topic. Would you prefer to have it after the "global" eventlet session (tue) ? | |
| 15:06:38 | Uggla | or it doesn't matter. | |
| 15:29:11 | Uggla | TheJulia, dansmith Thu 1500-1600UTC : Attestation integration Plugin (1h) does it work for you ? | |
| 15:31:13 | lajoskatona | Uggla: Hi, there is perhaps a beehive like topic (sorry for that) to use SDK for neutron to nova notifications (os-server-external-events API call ), see my patch in SDK: | |
| 15:32:25 | lajoskatona | Uggla: https://review.opendev.org/c/openstack/openstacksdk/+/983904 (and in Neutron to use the SDK patch: https://review.opendev.org/c/openstack/neutron/+/983905 ) we discussed today's Neutron meeting that perhaps this is a topic to chat during the PTG | |
| 15:35:17 | melwitt | bauzas: hi, there are a few small vtpm related patches that would appreciate your review if you get a chance please https://review.opendev.org/c/openstack/nova/+/978836 https://review.opendev.org/c/openstack/nova/+/983504 https://review.opendev.org/c/openstack/nova/+/983505 | |
| 15:36:24 | Uggla | lajoskatona, please add the topic at the end of the PTG doc, if not already done. | |
| 15:37:05 | bauzas | melwitt: ack | |
| 15:37:13 | Uggla | lajoskatona, then I will check if that will be an "individual" topic or something in a X session. (Note the X session with Neutron is not yet defined). | |
| 15:44:06 | Uggla | dansmith Get on the same page about unpinning availability zone --> how much time do you need ? 1h ? 30 mn ? | |
| 15:56:01 | croelandt | bauzas: hey! I just came across https://github.com/sbauza/openstack-agentic-workflows , which at the moment seems mostly used for Nova. In its current form, /code-review can only be used for Nova, if I'm not mistaken. Would you be open to having a /openstack-code-review skill that dispatches the review of a given patch to project-specific agents? | |
| 15:56:36 | TheJulia | Uggla: that window works for me | |
| 15:57:06 | bauzas | croelandt: this is more a demonstrating POC at the moment for the Nova repository indeed, but other projects could add their own knowledge files for each of them | |
| 15:59:06 | lajoskatona | Uggla: thanks, I added it | |
| 15:59:26 | croelandt | bauzas: yeah but if I define workflows/glance-review/.claude/skills/code-review/SKILL.md , then /code-review will be defined twice, right? | |
| 16:00:56 | bauzas | good call, I could change the skill name to be nova-specific | |
| 16:01:26 | croelandt | yeah so at first you'd have nova-review and we'd have glance-review | |
| 16:01:36 | croelandt | wanna do it or want me to send a PR tonight? :) | |
| 16:01:56 | bauzas | croelandt: I already have some LLM session somewhere so I can fix that quickly :) | |
| 16:02:04 | croelandt | I could get CLaude to do it and try to understand the code better while I'm at it | |
| 16:02:22 | croelandt | bauzas: you say it's more of a POC but are you open to merging stuff for other projects? | |
| 16:02:27 | bauzas | croelandt: yeah all the PRs were generated by LLMs and only curated by me | |
| 16:02:40 | croelandt | ok so I'll try and send PRs :p | |
| 16:02:53 | croelandt | well I'll try not to write a single line and let Claude do it | |
| 16:03:15 | bauzas | croelandt: I'd prefer to have the knowledge files to be on the upstream repos, but for skills, it could be on other repos | |
| 16:03:48 | croelandt | hm not sure I get it | |
| 16:03:59 | croelandt | you want to move knowledge/nova.md somewhere else? | |
| 16:07:25 | bauzas | directly in the nova repo | |
| 16:07:32 | bauzas | but we'll discuss that next week on the PTG | |
| 16:08:58 | croelandt | Is there an OpenStack-wide discussion regarding agentic AI? | |
| 16:09:55 | bauzas | hopefully on the TC meeting | |
| 16:12:13 | croelandt | ok I'll keep an eye on that | |
| 16:12:27 | croelandt | thanks for the info | |
| 16:32:41 | bauzas | croelandt: still working on the PR, hopefully be here in ~30 mins | |
| 16:43:19 | bauzas | croelandt: PR merged https://github.com/sbauza/openstack-agentic-workflows/pull/26 | |
| 16:57:34 | dansmith | Uggla: I suspect the attestation thing will need some arguing yeah, so 1h would be good | |
| 16:57:57 | dansmith | Uggla: I think 30 min for the AZ thing is probably fine, and/or tucked into a sequence of smaller things | |
| 17:26:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | |
| 17:26:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Reenable last scatter-gather unit test https://review.opendev.org/c/openstack/nova/+/984349 | |
| 17:26:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Reenable test_show_simple_tenant_usage_policy https://review.opendev.org/c/openstack/nova/+/984606 | |
| 17:26:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Delete duplicate unit test https://review.opendev.org/c/openstack/nova/+/984607 | |
| 17:26:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Skip eventlet specific tests https://review.opendev.org/c/openstack/nova/+/984608 | |
| 17:30:09 | croelandt | bauzas: thanks! Maybe I'll look into adding a bit of Glance stuff as a PoC, but ideally we sort this out next week at the PTG and decide how OpenStack wants to handle agentic AI | |
| 17:30:24 | bauzas | ++ | |
| 17:31:12 | gibi | bauzas: I pushed a bunch of unit test fixes for native threading mode above. All of them are simple | |
| 17:31:21 | gibi | ;) | |
| 17:31:49 | bauzas | gibi: i'm fried and still on a meeting, please ping me again tomorrow | |
| 17:33:35 | gibi | sure | |
| 17:48:51 | sean-k-mooney | gibi: i saw melwitt pushsed some test patches yesterday? i think as well | |
| 17:49:09 | sean-k-mooney | i didnt have a chance really to look at them bu they were related to the db setup | |
| 17:49:33 | sean-k-mooney | so hopefuly tose are all converging to makign it work? | |
| 17:50:26 | melwitt | sean-k-mooney: I did, they might not be good though. right related to databases fixtures. no not converging because what I tried is very different and might not be the way people prefer | |
| 17:51:20 | melwitt | I had read through the review comments on some of the proposed patches dealing with cell databases and database and thought to try something different | |