| 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 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: Database: use file-backed SQLite for native threading https://review.opendev.org/c/openstack/nova/+/984481 | 00:30 |
| 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: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 | 00:40 |
| opendevreview | melanie witt proposed openstack/nova master: Database: use file-backed SQLite for native threading https://review.opendev.org/c/openstack/nova/+/984481 | 01:02 |
| 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 |
| opendevreview | Merged openstack/nova master: mem-enc: stop using _get_mem_encryption_config() for SEV checks https://review.opendev.org/c/openstack/nova/+/967970 | 06:20 |
| *** sambork_ is now known as sambork | 06:44 | |
| gibi | melwitt: thanks for that. I wil check it out | 07:14 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | 07:38 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | 07:42 |
| gibi | bauzas: stephenfin: can one of you push this CI timeout bump through https://review.opendev.org/c/openstack/nova/+/984417 | 08:21 |
| * bauzas looks | 08:22 | |
| * gibi thanks | 08:26 | |
| opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | 08:45 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Change nova-alt-configurations job https://review.opendev.org/c/openstack/nova/+/983179 | 09:25 |
| opendevreview | ribaudr proposed openstack/nova master: Add reproducer for bug #2141722 https://review.opendev.org/c/openstack/nova/+/984521 | 09:31 |
| opendevreview | ribaudr proposed openstack/nova master: Fix os-simple-tenant-usage pagination with duplicate instance_extra https://review.opendev.org/c/openstack/nova/+/984522 | 09:31 |
| 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:10 |
| opendevreview | Merged openstack/nova master: [openstack-tox-cover]Increase timeout further https://review.opendev.org/c/openstack/nova/+/984417 | 10:33 |
| opendevreview | sean mooney proposed openstack/nova master: add agentic coding scaffold and contributor docs https://review.opendev.org/c/openstack/nova/+/984540 | 10:44 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | 12:46 |
| opendevreview | Merged openstack/nova master: mem-enc: refactor memory encryption trait logic for extensibility https://review.opendev.org/c/openstack/nova/+/967971 | 13:08 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | 13:58 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Reenable last scatter-gather unit test https://review.opendev.org/c/openstack/nova/+/984349 | 13:58 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | 13:59 |
| 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 |
| 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 | 14:21 |
| Uggla | gibi, regarding you eventlet topic. Would you prefer to have it after the "global" eventlet session (tue) ? | 15:05 |
| Uggla | or it doesn't matter. | 15:06 |
| Uggla | TheJulia, dansmith Thu 1500-1600UTC : Attestation integration Plugin (1h) does it work for you ? | 15:29 |
| 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:31 |
| 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:32 |
| 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:35 |
| Uggla | lajoskatona, please add the topic at the end of the PTG doc, if not already done. | 15:36 |
| bauzas | melwitt: ack | 15:37 |
| 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:37 |
| Uggla | dansmith Get on the same page about unpinning availability zone --> how much time do you need ? 1h ? 30 mn ? | 15:44 |
| 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 |
| TheJulia | Uggla: that window works for me | 15:56 |
| 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:57 |
| lajoskatona | Uggla: thanks, I added it | 15:59 |
| croelandt | bauzas: yeah but if I define workflows/glance-review/.claude/skills/code-review/SKILL.md , then /code-review will be defined twice, right? | 15:59 |
| bauzas | good call, I could change the skill name to be nova-specific | 16:00 |
| croelandt | yeah so at first you'd have nova-review and we'd have glance-review | 16:01 |
| croelandt | wanna do it or want me to send a PR tonight? :) | 16:01 |
| bauzas | croelandt: I already have some LLM session somewhere so I can fix that quickly :) | 16:01 |
| croelandt | I could get CLaude to do it and try to understand the code better while I'm at it | 16:02 |
| croelandt | bauzas: you say it's more of a POC but are you open to merging stuff for other projects? | 16:02 |
| bauzas | croelandt: yeah all the PRs were generated by LLMs and only curated by me | 16:02 |
| croelandt | ok so I'll try and send PRs :p | 16:02 |
| croelandt | well I'll try not to write a single line and let Claude do it | 16:02 |
| 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 |
| croelandt | hm not sure I get it | 16:03 |
| croelandt | you want to move knowledge/nova.md somewhere else? | 16:03 |
| bauzas | directly in the nova repo | 16:07 |
| bauzas | but we'll discuss that next week on the PTG | 16:07 |
| croelandt | Is there an OpenStack-wide discussion regarding agentic AI? | 16:08 |
| bauzas | hopefully on the TC meeting | 16:09 |
| croelandt | ok I'll keep an eye on that | 16:12 |
| croelandt | thanks for the info | 16:12 |
| bauzas | croelandt: still working on the PR, hopefully be here in ~30 mins | 16:32 |
| bauzas | croelandt: PR merged https://github.com/sbauza/openstack-agentic-workflows/pull/26 | 16:43 |
| dansmith | Uggla: I suspect the attestation thing will need some arguing yeah, so 1h would be good | 16:57 |
| dansmith | Uggla: I think 30 min for the AZ thing is probably fine, and/or tucked into a sequence of smaller things | 16:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | 17:26 |
| 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 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Delete duplicate unit test https://review.opendev.org/c/openstack/nova/+/984607 | 17:26 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [py313-threading]Skip eventlet specific tests https://review.opendev.org/c/openstack/nova/+/984608 | 17:26 |
| 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 |
| bauzas | ++ | 17:30 |
| gibi | bauzas: I pushed a bunch of unit test fixes for native threading mode above. All of them are simple | 17:31 |
| gibi | ;) | 17:31 |
| bauzas | gibi: i'm fried and still on a meeting, please ping me again tomorrow | 17:31 |
| gibi | sure | 17:33 |
| sean-k-mooney | gibi: i saw melwitt pushsed some test patches yesterday? i think as well | 17:48 |
| sean-k-mooney | i didnt have a chance really to look at them bu they were related to the db setup | 17:49 |
| sean-k-mooney | so hopefuly tose are all converging to makign it work? | 17:49 |
| 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:50 |
| 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 | 17:51 |
| sean-k-mooney | ok i wasm more just makign sure you and gibi were both aware of yoru paralle efforts | 17:51 |
| sean-k-mooney | ack | 17:51 |
| melwitt | yes I told gibi about them and did not want to disrupt what he's doing because it might not matter | 17:52 |
| sean-k-mooney | next week is obvioulsy hte ptg so we will all be busy. not sure we want to pend time talkign about unit testign there in any great detail but it might make sense to use the next eventlet call after the ptg to sync on the related efforts | 17:52 |
| sean-k-mooney | melwitt: although iam not sure if that is too early for you to be around or not | 17:53 |
| gibi | sean-k-mooney: melwitt: I'm aware and thankful, I haven't had the time to read it. Maybe tomorrow | 17:53 |
| melwitt | sean-k-mooney: is the time and gmeet documented anywhere? I went to look for it a few times but didn't find it | 17:53 |
| sean-k-mooney | i dont know if its on the openstakc meeting patch i tought it might be? | 17:54 |
| gibi | melwitt: I guess it is in the IRC logs :D but it is not on the meeting page, sorry | 17:54 |
| gibi | https://meet.google.com/bcy-uqoz-hje | 17:54 |
| melwitt | I had special interest in the cell databases and database fixtures | 17:54 |
| melwitt | thanks! | 17:54 |
| gibi | https://docs.google.com/document/d/1vCOmD7kxmRFk9IG8WbVhyqKlCpOPDZFJgkUSwYdQHbM/edit?usp=sharing | 17:55 |
| * melwitt bookmarks | 17:55 | |
| sean-k-mooney | it happens at 14:30 UTC on wednesdays | 17:56 |
| sean-k-mooney | right? i hate timezones | 17:56 |
| sean-k-mooney | my cal invite say 22/04/2026, 15:30 (22/04/2026, 16:30, Europe/Budapest) | 17:57 |
| melwitt | looks like yes | 17:57 |
| melwitt | (according to what it says on the gmeet) | 17:57 |
| sean-k-mooney | ack Europe/Budapest is currentl utc+2 i belive | 17:58 |
| sean-k-mooney | we can also just chat asyn i just have not had time to look at the diffent patchs that are in flight and i likely wont next week | 17:59 |
| melwitt | sure np. I will look through some more things too, since my WIP ones were hastily done | 18:02 |
| sean-k-mooney | Uggla: https://blueprints.launchpad.net/nova/+spec/adopt-port-trusted-vif-extension looks fien to me we can dicuss it in a future irc meeting if/whenw e go do it | 18:16 |
| sean-k-mooney | its honestly a pretty small chagne to make | 18:16 |
| opendevreview | melanie witt proposed openstack/nova master: Fix flaky monitor metric tests due to timestamp format mismatch https://review.opendev.org/c/openstack/nova/+/984617 | 18:41 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test ceph ipv6 https://review.opendev.org/c/openstack/nova/+/984623 | 18:56 |
| aravindh | Hey I have a query with respect to multipath and iscsid, I have 2 interfaces that connects to storage, I have setup 2 paths in iscsiadm, and iscsid is set to start automatically with discovery on hypervisor boot. When it boots, every multipath list shows me 4 paths. But when I start a VM from nova that creates only 2 paths rather than 4. If i rescan the iscsi bus and restart multipathd, that 2 paths move to 4 paths. Is t | 19:01 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test ceph ipv6 https://review.opendev.org/c/openstack/nova/+/984623 | 19:05 |
| melwitt | gmaan: easy flaky unit test fix if you might be interested https://review.opendev.org/c/openstack/nova/+/984617 | 21:45 |
| gmaan | checking | 21:47 |
| gmaan | melwitt: thanks for fixing, lgtm | 21:51 |
| melwitt | thanks gmaan | 21:51 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!