| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Lenovo UEFI boot loop in prepare_instance flow https://review.opendev.org/c/openstack/ironic/+/986597 | 03:02 |
|---|---|---|
| kubajj | Good morning Ironic! o/ | 06:57 |
| estherd[m] | hroy: rpittau Happy first day! Following up on Riccardo's email from last week. I'm available for our first weekly meeting. Let me know when works for you. Looking forward to working with you both :) | 09:43 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Lenovo UEFI boot loop in prepare_instance flow https://review.opendev.org/c/openstack/ironic/+/986597 | 10:00 |
| hroy | estherd[m], hey o/, looking forward to working with you too :) Riccardo is not available today. I would recommend going through the latest official ironic documentation and then look into the existing node history api. We will send out an invite for our sync soon. | 12:34 |
| hroy | estherd[m], let us know if you hit any issues or don't understand something | 12:34 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Lenovo UEFI boot loop in prepare_instance flow https://review.opendev.org/c/openstack/ironic/+/986597 | 12:51 |
| iurygregory | good morning Ironic o/ | 12:52 |
| opendevreview | Bartosz Bezak proposed openstack/tenks master: CI: move do debian trixie https://review.opendev.org/c/openstack/tenks/+/988970 | 13:15 |
| opendevreview | Bartosz Bezak proposed openstack/tenks master: CI: move do debian trixie https://review.opendev.org/c/openstack/tenks/+/988970 | 13:21 |
| TheJulia | good morning | 13:31 |
| mumesan[m] | o/ | 13:36 |
| JayF | FYI I'm taking a personal day today. If you need something feel free to pay me but it might be high latency on the response. | 13:55 |
| JayF | **ping me 😂 | 13:55 |
| TheJulia | heh, okay | 13:57 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Consolidate duplicated TLS code into ironic/common/tls_utils https://review.opendev.org/c/openstack/ironic/+/987909 | 14:12 |
| mumesan[m] | Hey there, can I get some reviews on these -... (full message at <https://matrix.org/oftc/media/v1/media/download/AZRotYm92RIzWJHzugNgiOh7X1Q3UVuJ387QrpsRDnn4KVLDh0T_pffeYC_OoSOkj23cMmZ2hGYztJQ0DrF__n5Ceeg4rNNgAG1hdHJpeC5vcmcvY2hoZUNsZmlFbFJrSFJMc0J4RkxLUmlD>) | 14:16 |
| TheJulia | mumesan[m]: smaller messages are likely better ;) | 14:19 |
| mumesan[m] | Ah right sorry, I'm using matrix so it comes up as a single message, did it get parsed into single lines on an irc client TheJulia | 14:19 |
| TheJulia | no worries :) | 14:20 |
| iurygregory | JayF, payment == beer? XD | 14:20 |
| mumesan[m] | Sorry again 😅 i forgot it did that | 14:20 |
| opendevreview | Merged openstack/ironic master: doc: Ignore ironic.wsgi module https://review.opendev.org/c/openstack/ironic/+/988604 | 14:25 |
| opendevreview | Julia Kreger proposed openstack/ironic unmaintained/2024.1: stable only: cleanup 2024.1 ci jobs https://review.opendev.org/c/openstack/ironic/+/988767 | 14:28 |
| cardoe | TheJulia: https://review.opendev.org/c/openstack/ironic/+/987916 abandon that one then? | 14:30 |
| TheJulia | Yeah, doing so. | 14:30 |
| TheJulia | last call on https://review.opendev.org/c/openstack/ironic/+/987441 unless we're all for lazy consensus ;) | 14:34 |
| iurygregory | I think we are good to go with it | 14:36 |
| TheJulia | we will likely want to backport, it walks a bit of a line, but as a hidden detail really | 14:38 |
| TheJulia | fixing a bug | 14:38 |
| iurygregory | ++ | 14:38 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic-python-agent master: Apply TLS configuration from Ironic lookup response https://review.opendev.org/c/openstack/ironic-python-agent/+/988150 | 14:40 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic-python-agent master: Apply TLS configuration from Ironic lookup response https://review.opendev.org/c/openstack/ironic-python-agent/+/988150 | 14:40 |
| TheJulia | If folks could also review https://review.opendev.org/c/openstack/ironic-specs/+/954612 it would be helpful to begin moving that item forward | 14:41 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add SONiC multicast BUM configuration validation https://review.opendev.org/c/openstack/networking-generic-switch/+/981193 | 14:46 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add Juniper Junos multicast BUM replication support https://review.opendev.org/c/openstack/networking-generic-switch/+/981307 | 14:46 |
| Sandzwerg[m] | Ohai wonderful people of ironic. I have a question regarding: https://bugs.launchpad.net/ironic/+bug/2121702 a colleague now had the idea to just add a auth_from_config=True here: https://opendev.org/openstack/ironic/src/branch/master/ironic/common/neutron.py#L305 Would that work? Do you have thoughts on this? (He also had another idea but one after the other) | 14:57 |
| TheJulia | Sandzwerg[m]: I likely don't have the time to dig/answer that before our weekly meeting | 14:59 |
| TheJulia | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon May 18 15:00:20 2026 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| TheJulia | o/ | 15:00 |
| iurygregory | o/ | 15:00 |
| dtantsur | o/ | 15:00 |
| Mahnoor | O/ | 15:00 |
| clif | o/ | 15:00 |
| TheJulia | Good morning and welcome to this week's Ironic meeting, where we discuss the irony... I mean, Bare metal things. | 15:01 |
| TheJulia | As always, our meeting occurs under the governance of the OpenStack project and the code of conduct set forth there. | 15:01 |
| cid | o/ | 15:01 |
| TheJulia | #topic Announcements / Reminders | 15:01 |
| TheJulia | As a standing reminder, please review items with the ironic-weke-prio hashtag. | 15:02 |
| TheJulia | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| TheJulia | This week is week R-19, which means already past the Hibiscus-1 milestone in the community. | 15:02 |
| TheJulia | #link https://releases.openstack.org/hibiscus/schedule.html | 15:03 |
| TheJulia | Does anyone have anything to announce or remind us of? | 15:03 |
| kubajj | o/ | 15:03 |
| mumesan[m] | o/ | 15:04 |
| TheJulia | As a note, I'll be on vacation the week of June 29th. | 15:04 |
| iurygregory | enjoy the time off =) | 15:04 |
| TheJulia | Thanks! | 15:04 |
| TheJulia | Moving on if nobody else has anything to announce or remind us of | 15:05 |
| TheJulia | #topic Working Group Updates | 15:05 |
| TheJulia | The first and only group is Async IO. Is there any update this week? | 15:05 |
| dtantsur | nope | 15:05 |
| TheJulia | Okay! | 15:05 |
| TheJulia | Onward | 15:05 |
| TheJulia | #topic Discussion topics | 15:05 |
| * TheJulia hands the podium to cid | 15:06 | |
| TheJulia | Virutal PDU Retirement :) | 15:06 |
| iurygregory | :D | 15:06 |
| iurygregory | wow | 15:06 |
| cid | Yup, some lots of merges | 15:06 |
| cid | Almost there | 15:06 |
| cid | I sent out the mail to the ML | 15:07 |
| TheJulia | Anything you need from ironic-cores or liaisons to wrap that up? | 15:07 |
| cid | I don't think so, | 15:07 |
| TheJulia | Excellent! | 15:07 |
| iurygregory | ack | 15:07 |
| cid | The government repo change has gotten all the sign-off | 15:08 |
| TheJulia | awesome | 15:08 |
| TheJulia | Onward then! | 15:08 |
| TheJulia | #topic Bug Deputy Updates | 15:08 |
| TheJulia | clif: this looks like it was you, here is a shiny podium to speak from! :) | 15:08 |
| clif | Hello, from what I can tell it's been a quiet week, with only one new bug reported | 15:09 |
| clif | To ironic, but is more of an RFE | 15:09 |
| clif | so not much to report | 15:09 |
| TheJulia | excellent! Does that include sushy, ironic-python-agent, etc? | 15:10 |
| clif | Please correct me if I missed something, but all other projects don't have a new/open open within the last week afaict | 15:10 |
| clif | yes, all other projects accounted for | 15:10 |
| TheJulia | cool | 15:10 |
| clif | so I'll hand it off again | 15:10 |
| TheJulia | Okay, quiet weeks are always nice. Does anyone want to be the bug deputy for this next week? | 15:10 |
| TheJulia | Also, clif, the one you noted was more of an RFE, was that Dmitry's item added on Friday? | 15:11 |
| iurygregory | next week or this week? | 15:11 |
| clif | TheJulia: Yes, this one: https://bugs.launchpad.net/ironic/+bug/2152718 | 15:11 |
| cid | iurygregory. I think this week. | 15:12 |
| cid | I can deputy this week. | 15:12 |
| cid | re' VirtualPDU... this is all the patches I created so far. Maybe I missed something, https://review.opendev.org/q/topic:%22retiring-virtualpdu%22. | 15:12 |
| cid | I created this etherpad to lay out the steps I followed. https://etherpad.opendev.org/p/retiring-virtualpdu | 15:12 |
| cid | Reviews alway welcome | 15:12 |
| TheJulia | iurygregory: this next week, 7 days starting now() ish :) | 15:13 |
| TheJulia | cid, okay, thanks! | 15:13 |
| TheJulia | dtantsur: would you like to discuss that item in RFE review, or was it you just getting a high level idea down into the bug tracker? | 15:14 |
| * TheJulia takes silence as a no | 15:16 | |
| TheJulia | Skipping directly to Open Discussion | 15:17 |
| TheJulia | #topic Open Discussion | 15:17 |
| TheJulia | Does anyone have anything to discuss? | 15:17 |
| dtantsur | yeah, it was just a quick brain dump | 15:18 |
| TheJulia | I guess everything is good and we have nothing else to discuss today | 15:19 |
| cid | ++ | 15:19 |
| TheJulia | This leaves the most critical question, who shall run our meeting next week ? | 15:20 |
| iurygregory | I can | 15:20 |
| TheJulia | Okay, thanks Iury, I'll update the agenda! | 15:21 |
| TheJulia | And with that, have a great week everyone! | 15:21 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add a BMO job https://review.opendev.org/c/openstack/ironic/+/988766 | 15:21 |
| cardoe | stephenfin: https://opendev.org/openstack/python-ironicclient/src/commit/e5f25fd2a50bfc4c22507478fdb31698f1db6934/ironicclient/v1/runbook.py#L43 should that be Runbook instead of Resource? Looking at line 30 above it has Runbook as a type. Trying to figure out what the right thing is here. | 15:22 |
| TheJulia | #endmeeting | 15:23 |
| opendevmeet | Meeting ended Mon May 18 15:23:41 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:23 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-05-18-15.00.html | 15:23 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-05-18-15.00.txt | 15:23 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-05-18-15.00.log.html | 15:23 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM Science: try --ha in the IrSO job https://review.opendev.org/c/openstack/ironic/+/988760 | 15:24 |
| TheJulia | Sandzwerg[m]: I guess that might work as proposed, as long as we know for sure its only called for ramdisk booting | 15:26 |
| opendevreview | Merged openstack/sushy-tools master: Drop pbr from requirements https://review.opendev.org/c/openstack/sushy-tools/+/988752 | 15:27 |
| Sandzwerg[m] | The comment speaks about ramdisk so I'd assume that but I'm not sure myself. But the issue seems to be that when a human admin makes a state change from "clean failed" -> managed -> cleaning -> available the project context is used instead of system context. Or could that be related to our still custom & not RBAC ready policy? | 15:30 |
| TheJulia | Yeah, thinking about it I suspect it should be system context because we're proxing through authority, the issue which comes to mind is flat networking | 15:31 |
| TheJulia | since in flat, we try to re-use vifs | 15:31 |
| TheJulia | but... well | 15:31 |
| TheJulia | yeah | 15:31 |
| * TheJulia ponders | 15:31 | |
| opendevreview | Ivan Anfimov proposed openstack/ironic-ui master: Restore explicit package list for module detection https://review.opendev.org/c/openstack/ironic-ui/+/988997 | 15:36 |
| opendevreview | Ivan Anfimov proposed openstack/ironic-ui master: Restore explicit package list for module detection https://review.opendev.org/c/openstack/ironic-ui/+/988997 | 15:36 |
| opendevreview | Ivan Anfimov proposed openstack/ironic-ui master: Restore explicit package list for module detection https://review.opendev.org/c/openstack/ironic-ui/+/988997 | 15:37 |
| TheJulia | Sandzwerg[m]: I just gave my concerns/thoughts to claud and I'm letting it chew on them for a little bit. my concern over flat is likely misplaced, but its doing the code walking as well | 15:38 |
| Sandzwerg[m] | Alright | 15:39 |
| opendevreview | Merged openstack/ironic master: Convert string values in BIOS settings before sending them to the BMC https://review.opendev.org/c/openstack/ironic/+/988467 | 15:42 |
| TheJulia | so, generally speaking claude agrees, but highlights we also need to update remove_neutron_ports. That being said, I'm having super strong deja vu and having it continue to dig | 15:43 |
| stephenfin | cardoe: It should, yes. Though the base class doesn't expose a `get` method for some reason | 15:46 |
| TheJulia | Sandzwerg[m]: claude has convinced me, if you want to propose a patch for both methods, I'd +2 it | 15:48 |
| opendevreview | Merged openstack/networking-generic-switch master: ci: add "vxlan" ci job https://review.opendev.org/c/openstack/networking-generic-switch/+/981024 | 15:54 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Restructure admin documentation by topic and vendor https://review.opendev.org/c/openstack/networking-generic-switch/+/981418 | 15:57 |
| cardoe | stephenfin: yeah I'm gonna clean that up | 16:04 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Convert string values in BIOS settings before sending them to the BMC https://review.opendev.org/c/openstack/ironic/+/988999 | 16:04 |
| Sandzwerg[m] | TheJulia: yeah, the colleague proposed that as well but I forgot to copy it here 😅 | 16:06 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: fix incorrect annotation for list_ports method https://review.opendev.org/c/openstack/python-ironicclient/+/989000 | 16:07 |
| Sandzwerg[m] | TheJulia: We just tried it in our QA and it does seem to solve the issue. I'll test some more tomorrow and ask the colleague if he can open the PR (as it was his idea) but so far it looks good. Gosh I whish I've spent that time already a year ago instead of suffering with the consequences for so long 🫠| 16:08 |
| TheJulia | heh, okay :) | 16:08 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: remove pbr as runtime dependency https://review.opendev.org/c/openstack/python-ironicclient/+/989001 | 16:10 |
| estherd[m] | hroy: Absolutely, thank you. | 16:19 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Lenovo UEFI boot loop in prepare_instance flow https://review.opendev.org/c/openstack/ironic/+/986597 | 16:19 |
| TheJulia | Jad_Haj: out of curiosity, have you taken that for a test yet on the hardware you have access to? | 16:24 |
| cardoe | hot dang we've got a lot of code in python-ironicclient | 16:28 |
| cardoe | Like we've got a whole CrudManager class that seems unused. I dunno if that's something for openstacksdk or what. | 16:30 |
| dtantsur | cardoe: one of the things we always wanted and never did was to reap it all out and replace with openstacksdk for real.. | 16:31 |
| cardoe | Like a lot of the apiclient directory seems unused and duplicated. | 16:31 |
| dtantsur | IIRC apiclient used to be cargo-culted around back in earlier OpenStack days | 16:31 |
| dtantsur | (few people here remember how before Oslo we just copy-pasted common code....) | 16:32 |
| TheJulia | A fun exercise may be to ask claude to clean it up but not break the user interface | 16:32 |
| dtantsur | ....wwwwow, my new BMO job just passed? .... | 16:33 |
| * dtantsur happy dance | 16:33 | |
| TheJulia | congrats! | 16:35 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Add TLS configuration to agent lookup response https://review.opendev.org/c/openstack/ironic/+/987887 | 16:41 |
| stephenfin | cardoe: if you're cleaning stuff up, I suspect the entire ironic.common.apiclient package could go. Looks like it was an import from oslo a long time ago and is basically unused. jfyi | 16:49 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: A software raid guard rail? https://review.opendev.org/c/openstack/ironic-python-agent/+/989009 | 16:56 |
| Jad_Haj | TheJulia: Actually i do not have access to Lenovo but i can ask the reporter of the bug to so. | 16:56 |
| TheJulia | stephenfin: you caused my brain to go sideways for a moment until I realized it was ironicclient.common.apiclient you were referring to | 16:57 |
| TheJulia | Jad_Haj: okay, thanks! | 16:57 |
| stephenfin | TheJulia: oops, yeah 😅 | 16:58 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add a BMO job https://review.opendev.org/c/openstack/ironic/+/988766 | 17:00 |
| cardoe | TheJulia: dtantsur: if you're cool with what stephenfin is suggesting, I would likely make that change. | 17:39 |
| cardoe | It looks like that code is an import like stephenfin said and has some old relationship to what openstacksdk has become. | 17:39 |
| dtantsur | if it works, I'm cool with it | 17:40 |
| TheJulia | I could be good with that, my only real concern in all of that is if version negotiation is handled properly. I think we did get the baseline logic cleaned up and matching appropriately but... its been such a very very very long time I just don't remember | 17:40 |
| cardoe | I'm just looking to get the type stuff better... like ran into https://review.opendev.org/c/openstack/python-ironicclient/+/989000 | 17:44 |
| cardoe | Ultimately we can get rid of a ton of duplicated code if we used CrudManager from apiclient but it's signature and process is all wrong. | 17:45 |
| cardoe | We've got these "resource_class" and "_resource_name" and the helper functions use them if you don't specify it. | 17:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Document the Metal3 ironic-standalone-operator job https://review.opendev.org/c/openstack/ironic/+/989014 | 17:46 |
| TheJulia | cardoe: likely why I would give it to an agent with a bunch of specific guardrails | 17:51 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: Remove dead code in apiclient module https://review.opendev.org/c/openstack/python-ironicclient/+/989016 | 18:22 |
| opendevreview | Himanshu Roy proposed openstack/ironic master: Disallow certain steps for users in different deployments https://review.opendev.org/c/openstack/ironic/+/989017 | 18:25 |
| opendevreview | Himanshu Roy proposed openstack/ironic master: Disallow certain steps for users in different deployments https://review.opendev.org/c/openstack/ironic/+/989017 | 18:27 |
| opendevreview | Himanshu Roy proposed openstack/ironic master: WIP: Disallow certain steps for users in different deployments https://review.opendev.org/c/openstack/ironic/+/989017 | 18:28 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: remove pbr as runtime dependency https://review.opendev.org/c/openstack/python-ironicclient/+/989001 | 18:57 |
| cardoe | TheJulia: yeahhhh agent no happy | 19:40 |
| TheJulia | did you give it digital ativan? | 19:40 |
| cardoe | During the add typing work we changed the common "Resource" class to inherit from apiclient instead of from ironicclient/v1/resource_fields.py | 19:41 |
| TheJulia | is the command to try and type it or re-shape it? | 19:45 |
| cardoe | I'm trying to type it | 19:46 |
| cardoe | We've got a Resource class that's suppose to be a generic dict like thing that then there's specific versions of it to limit the parameters but we stopped inheriting from that so we stopped enforcing the checks. | 19:48 |
| TheJulia | so, your trying to retype it to keep python types in place, maybe if we keep it for that, we just move straight to a major version bump | 20:50 |
| TheJulia | alternatively, really everyone doing python clienty things should be using the sdk, but the library holds the underlying interaction logic for the cli and the original python client sdk | 20:50 |
| TheJulia | for ironic itself | 20:50 |
| cardoe | I'm gonna put stuff in the openstacksdk | 20:52 |
| cardoe | You wanna depend on openstacksdk? | 20:52 |
| TheJulia | we're already dependent upon it | 20:54 |
| cardoe | alright | 21:13 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!