| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Pre-test typing cleanup https://review.opendev.org/c/openstack/python-ironicclient/+/983360 | 01:06 |
|---|---|---|
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `tests/unit/common/` and `tests/unit/utils.py` https://review.opendev.org/c/openstack/python-ironicclient/+/984285 | 01:13 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `tests/unit` directory-level test files https://review.opendev.org/c/openstack/python-ironicclient/+/984287 | 01:41 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/tests/unit/v1` https://review.opendev.org/c/openstack/python-ironicclient/+/984290 | 03:30 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/tests/unit/osc` https://review.opendev.org/c/openstack/python-ironicclient/+/984295 | 04:17 |
| opendevreview | yatin proposed openstack/ironic master: devstack: skip qrouter namespace config for ovn https://review.opendev.org/c/openstack/ironic/+/984352 | 09:48 |
| dtantsur | Our team has a bit of a conflict with the PTG: we cannot attend the first hour. Could we move the retrospective? | 10:25 |
| dtantsur | generally, leaving some comments in the schedule (FYI TheJulia, JayF) | 10:28 |
| cardoe | stevebaker[m]: thanks for the review. | 12:40 |
| JayF | cid: ^^ fyi from dtantsur | 13:03 |
| cid | JayF, acknowledged. dtantsur, We can definitely move it to 1hr later. | 13:08 |
| cid | I will be updating the PTG kickoff by Monday to 1hr later since we usually start with a Retro | 13:09 |
| cid | ^^ that's if everyone is fine with that | 13:09 |
| cid | to: 14:00 - 17:00 UTC | 13:10 |
| cardoe | Sounds good to me. | 13:14 |
| TheJulia | Also, sounds good to me | 13:16 |
| cid | ++ | 13:37 |
| cardoe | TheJulia: can ya share what devstack local.conf you've got for running OVN? | 13:46 |
| cardoe | or maybe hjensas? | 13:47 |
| opendevreview | Merged openstack/networking-generic-switch stable/2026.1: Fix GenericSwitchNotSupported exceptions https://review.opendev.org/c/openstack/networking-generic-switch/+/983966 | 13:50 |
| TheJulia | cardoe: give me a little bit, my head is in an urgent customer case | 14:09 |
| cid | updated https://ptg.opendev.org/ptg.html | 14:12 |
| * TheJulia wonders if we'll have a meeting this week | 14:22 | |
| ykarel | Hi is the issue with ironic-tox-unit-with-driver-libs known one? Noticed while fixing https://review.opendev.org/c/openstack/ironic/+/984352 | 14:29 |
| ykarel | triggered by recent pyasn1 bump https://github.com/openstack/requirements/commit/2c6eae3260867a648c252e8f39520cef685aba65#diff-449046f51430a028478d668e3d3150c5b8f80e1a2682f1df9fd264dd6a42681aR35-R416 | 14:30 |
| JayF | that appears to be tests that weren't removed when ilo was removed | 14:30 |
| JayF | there should be no tests remaining in drivers/modules/ilo AFAICT | 14:30 |
| cardoe | ykarel: you got a devstack config for running OVN locally that you can share? | 14:30 |
| ykarel | cardoe, for just ovn it's https://github.com/openstack/neutron/blob/master/devstack/ovn-local.conf.sample | 14:31 |
| ykarel | but likely you looking along with ironic? | 14:32 |
| ykarel | for that may be you can just pull it from job and adjust/clean ci specific bits | 14:32 |
| cardoe | ykarel: I was trying to get the stock devstack config from the Ironic docs working and ran into an issue where the qrouter wasn't cleaned up. So I had 2. Everywhere else we explicitly look up the qrouter to use so I changed the code look up the qrouter in that function. But like you pointed out, that had the side effect of requiring a qrouter in that path vs skipping if non was available. | 14:32 |
| JayF | ykarel: https://review.opendev.org/c/openstack/ironic/+/982706 needs to be landed | 14:33 |
| JayF | can someone land it? | 14:33 |
| JayF | to fix CI at this point | 14:33 |
| ykarel | cardoe, ack atleast the ironic ovn job which was failing now pass with https://review.opendev.org/c/openstack/ironic/+/984352 | 14:33 |
| ykarel | thx JayF will rebase my patch over that | 14:34 |
| cardoe | JayF: +W'd it. | 14:34 |
| JayF | g'byelo | 14:34 |
| TheJulia | cardoe: https://9cd271f2961401a442f6-d0226936c9cadcaed5032c3f3c093f07.ssl.cf1.rackcdn.com/openstack/0924ec77d57d4cfda403d13c438d7716/controller/logs/local_conf.txt is along the lines of what I would use, I have a very similar config from late last year on one of my dev/test VMs | 14:36 |
| JayF | stephenfin: you or anyone you know working on python 3.14 for oslo.concurrency? | 14:37 |
| TheJulia | ykarel: that is entirely without ironic though, requires specific parameters for the baremetal side | 14:37 |
| JayF | stephenfin: I had an ask from gentoo downstream of us about it, I don't see any patches in progress, was gonna point someone at it if nobody is on it | 14:37 |
| ykarel | TheJulia, yeap that's what i meant | 14:38 |
| cardoe | stevebaker[m]: the case that I've wired up is for when you've started Ironic with the new runbook traits but not yet run the online DB migrations and a cleaning happens... it's so that the box still gets cleaned correctly. | 14:38 |
| cardoe | JayF, dtantsur: Can I get you two to agree on if the traits should be part of the runbook API? I'm leaning towards not allowing traits to be set at all via CREATE and UPDATE on the runbook itself. | 14:39 |
| JayF | I'm confused as to the sticking point? | 14:41 |
| JayF | the race I was concerned about is removed from the Ironic codebase | 14:41 |
| JayF | we can't save ourselves from client read races; only from races inside ironic itself, which at this point has been cleared up, right? | 14:41 |
| JayF | Or am I missing something | 14:41 |
| dtantsur | JayF: what I commented on was the fact that a user can set traits via the POST API on creation but cannot PATCH them later on. | 14:42 |
| dtantsur | So we pondered dropping traits from the POST API and only updating them after creation, just like it's done for nodes | 14:42 |
| JayF | I don't care enough to have a differing opinion from that | 14:43 |
| JayF | and if it's what we already do for nodes | 14:43 |
| JayF | I <3 standardization | 14:43 |
| TheJulia | regarding ilo, https://review.opendev.org/c/openstack/ironic/+/982706 hasn't merged yet | 14:43 |
| JayF | TheJulia: yep, I noticed and cardoe landed it (or said he did) | 14:43 |
| * TheJulia finally realizes she made more coffee and left the mug on the kitchen counter | 14:48 | |
| JayF | stephenfin: I JFDI https://review.opendev.org/c/openstack/oslo.concurrency/+/984403 | 14:51 |
| iurygregory | o/ | 15:01 |
| clif | o/ | 15:01 |
| TheJulia | so, we having a meeting this week? | 15:01 |
| mumesan[m] | o/ | 15:02 |
| TheJulia | whose driving? | 15:02 |
| JayF | #startmeeting ironic | 15:03 |
| opendevmeet | Meeting started Mon Apr 13 15:03:03 2026 UTC and is due to finish in 60 minutes. The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:03 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:03 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:03 |
| *** alegacy_ is now known as alegacy | 15:03 | |
| JayF | Welcome to the Ironic meeting! | 15:03 |
| abongale | 0/ | 15:03 |
| TheJulia | o/ | 15:03 |
| JayF | We are operating under the OpenInfra Code of Conduct, as usual. Behave yourselves :D | 15:03 |
| JayF | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| JayF | Please review things hashtagged ironic-week-prio, and if you are submitting changes and need reviews, just add a hashtag of "ironic-week-prio" to get it in the dashboard | 15:04 |
| JayF | The gazpacho release schedule is still in the Announcements/Remidners. | 15:04 |
| JayF | It's like R+1 now :D | 15:04 |
| JayF | #link https://releases.openstack.org/hibiscus/schedule.html | 15:04 |
| TheJulia | more like +2 ;) | 15:04 |
| JayF | it's R-24 for Hibiscus | 15:04 |
| TheJulia | \o/ | 15:04 |
| kubajj | o/ | 15:04 |
| JayF | PTG is next week -- the team meeting is traditionally cancelled for PTG week. | 15:04 |
| TheJulia | We should likely send an email just in case | 15:05 |
| JayF | #note Ironic team meeting next week is cancelled -- please participate in the PTG instead. | 15:05 |
| JayF | I'll do that | 15:05 |
| TheJulia | Thanks! | 15:05 |
| JayF | #action JayF email list about meeting cancelation | 15:05 |
| JayF | #undo | 15:05 |
| opendevmeet | Removing item from minutes: #action JayF email list about meeting cancelation | 15:05 |
| JayF | #action JayF email list about meeting cancellation | 15:05 |
| JayF | Given we're in anew release cycle, I'm going to skip working group updates | 15:05 |
| JayF | we'll hit it all at PTG next week anyway and set new WGs, I suspsect | 15:05 |
| cid | o/ | 15:06 |
| JayF | #topic PTG timing/allocations | 15:06 |
| JayF | cid has laid out a rough, high-level time allocation proposal in the etherpad | 15:06 |
| JayF | #link https://etherpad.opendev.org/p/ironic-ptg-2026.2#L16 | 15:06 |
| JayF | Please review it and provide feedback as neccessary. Thank you to cid for taking the reins on this for the H PTG. | 15:06 |
| JayF | Anything else to hit on re: PTG planning/timing/etc? | 15:06 |
| cid | \o/ | 15:07 |
| TheJulia | We likely need to ask folks to review specs in advance | 15:07 |
| cid | cross-project sessions too | 15:07 |
| TheJulia | We presently have 31 status:open ironic-week-prio items, some of which are specs | 15:07 |
| JayF | #note Please review all open, active specs before the PTG. | 15:08 |
| JayF | Moving on? | 15:09 |
| iurygregory | ++ | 15:09 |
| JayF | #topic Bug Deputy Updates | 15:09 |
| JayF | Our bug deputy was [insert name here]. [insert name here], take it away! | 15:10 |
| JayF | lol | 15:10 |
| JayF | We have 4 things listed here so someone was working; IDK who it was | 15:10 |
| TheJulia | We didn't pull off a meeting last week so *shrugs* | 15:10 |
| cid | I did for last week, didn't see bugs this week | 15:10 |
| * TheJulia thinks were in the weird "everyone ought to have taken a vacation" timeframe | 15:10 | |
| cid | Those bugs were from last week | 15:10 |
| cid | *are | 15:11 |
| JayF | #note No active bug deputy last week; info in agenda is stale. | 15:11 |
| JayF | Someone wanna dig through them this week? Or just keep it as vacant until post-ptg? | 15:11 |
| cid | I will do that | 15:11 |
| JayF | #note CID bug deputy this week | 15:12 |
| JayF | No RFEs for review; skipping that topic. | 15:12 |
| JayF | #topic Open Discussion | 15:12 |
| JayF | Any items for discussion not added to the agenda beforehand? | 15:12 |
| TheJulia | not on my end | 15:14 |
| JayF | Meeting next week is cancelled, and I'm not going to ask anyone to remember to run the meeting in 2 weeks | 15:14 |
| JayF | so we can just have whoever is around pick it up then | 15:15 |
| JayF | Thanks for coming, o/ | 15:15 |
| JayF | #endmeeting | 15:15 |
| opendevmeet | Meeting ended Mon Apr 13 15:15:10 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:15 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-04-13-15.03.html | 15:15 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-04-13-15.03.txt | 15:15 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-04-13-15.03.log.html | 15:15 |
| TheJulia | cool cool, now to create chaos! | 15:15 |
| jlvillal | I wander in here and it looks about the same as 2018. Almost all the same people :) | 15:17 |
| TheJulia | Eh, some different folk and lots of good work still being done! | 15:17 |
| JayF | we got some new folks in here too, but it's mostly been growth without too many old timers getting shed :) | 15:17 |
| TheJulia | Voids may have have been the cause ;) | 15:18 |
| jlvillal | Excellent! Good it is great to see it keeps on chugging along. | 15:18 |
| TheJulia | (as in black cats, feline gravity wells) | 15:18 |
| JayF | We got that white/tabby girl | 15:18 |
| JayF | she is the other side of the void, all the energy sucked it comes out there lol | 15:18 |
| JayF | s/it/in | 15:19 |
| jlvillal | I got motivated to setup "TheLounge" in a Docker container and now I can easily use IRC again :D | 15:19 |
| TheJulia | Our tabby and orange are the cuddly ones. The void just brings her gravity and causes the corgi to feel the need to provide emergent status updates about the void | 15:25 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: runbooks: add traits support, description field, and v1.112 API https://review.opendev.org/c/openstack/ironic/+/983215 | 15:34 |
| opendevreview | Merged openstack/ironic master: Remove iLO hardware type and interfaces https://review.opendev.org/c/openstack/ironic/+/982706 | 16:03 |
| opendevreview | Merged openstack/ironic master: Improve search experience in Ironic docs https://review.opendev.org/c/openstack/ironic/+/984213 | 16:03 |
| opendevreview | yatin proposed openstack/ironic master: devstack: skip qrouter namespace config for ovn https://review.opendev.org/c/openstack/ironic/+/984352 | 16:35 |
| TheJulia | cid: just to highlight, regarding https://review.opendev.org/c/openstack/ironic-specs/+/983183 the -1 is a super quick fix to the links in in the folder structure | 16:36 |
| cid | I will get to that now :D. I saw the notification over the weekend | 16:37 |
| cid | Thanks for the review | 16:37 |
| TheJulia | no problem, thank you for the effort! | 16:39 |
| opendevreview | cid proposed openstack/ironic-specs master: Proposal for a Bulk Node Operations API https://review.opendev.org/c/openstack/ironic-specs/+/983183 | 16:44 |
| opendevreview | Merged openstack/ironic master: devstack: skip qrouter namespace config for ovn https://review.opendev.org/c/openstack/ironic/+/984352 | 19:14 |
| cardoe | dtantsur: https://review.opendev.org/c/openstack/ironic/+/983215 can I update the flush in a follow up? I don't know how I missed your comments. | 19:40 |
| cardoe | JayF: https://review.opendev.org/c/openstack/ironic/+/983215 as well since you two had some conflicting opinions | 19:40 |
| opendevreview | cid proposed openstack/ironic-specs master: Proposal for a Bulk Node Operations API https://review.opendev.org/c/openstack/ironic-specs/+/983183 | 20:55 |
| opendevreview | Merged openstack/ironic master: runbooks: add traits support, description field, and v1.112 API https://review.opendev.org/c/openstack/ironic/+/983215 | 23:12 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!