Monday, 2026-01-12

opendevreviewcid proposed openstack/ironic master: Fix order of `disable_ramdisk` validation  https://review.opendev.org/c/openstack/ironic/+/97301500:04
opendevreviewcid proposed openstack/ironic master: Support `disable_ramdisk` during servicing  https://review.opendev.org/c/openstack/ironic/+/97301600:04
opendevreviewcid proposed openstack/ironic master: Fix order of `disable_ramdisk` validation  https://review.opendev.org/c/openstack/ironic/+/97301501:52
opendevreviewcid proposed openstack/ironic master: Support `disable_ramdisk` during servicing  https://review.opendev.org/c/openstack/ironic/+/97301601:52
rpittaugood morning ironic! o/07:58
*** srelf_ is now known as ContinuitySR09:25
*** ContinuitySR is now known as Continuity09:25
abongalegood morning o/09:40
ContinuityMorning Ironic10:48
*** dmellado4 is now known as dmellado11:41
opendevreviewMerged openstack/ironic master: Add two phase driver factor initialization  https://review.opendev.org/c/openstack/ironic/+/97118213:49
opendevreviewMerged openstack/ironic master: Introduce switch driver base class  https://review.opendev.org/c/openstack/ironic/+/97118313:50
opendevreviewMerged openstack/ironic master: Add generic switch driver support  https://review.opendev.org/c/openstack/ironic/+/96646913:50
opendevreviewMerged openstack/ironic master: Fix async periodic on redfish for servicewait  https://review.opendev.org/c/openstack/ironic/+/97158114:11
TheJuliagood morning14:33
cardoeshould we alias direct deploy interface to agent?14:45
TheJuliaIt wouldn't be a bad idea at this point14:45
cardoeMaybe we make a bug about all the things that are inconsistently named so we can track it for a big update and WAAAAAAAY future deprecation.14:55
dtantsurI don't mind it either. The distinction is irrelevant nowadays.14:55
TheJulia3 step: add an alias, migrate test initalizaitons and add a migration, likely next cycle. And then finally remove the old alias at some point down the road after it has been communicated14:56
TheJuliaits a light lift, but we do need to let it roll for a while14:56
opendevreviewJulia Kreger proposed openstack/ironic-specs master: VXLAN networking  https://review.opendev.org/c/openstack/ironic-specs/+/95940114:59
opendevreviewJacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface  https://review.opendev.org/c/openstack/ironic/+/96694614:59
JayFI can chair the meeting in like 5 minutes if no one else wants to15:01
TheJulia#startmeeting ironic15:01
opendevmeetMeeting started Mon Jan 12 15:01:26 2026 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
opendevmeetThe meeting name has been set to 'ironic'15:01
alegacyo/15:01
TheJuliaGood morning Ironic!15:01
dtantsuro/15:01
clifo/15:01
JayFo/15:01
janderso/15:01
TheJuliaThis will be our first meeting of 2026!15:02
clif\o/15:02
TheJuliaConsider it a slightly slow meeting, because it looks like we need to update the agenda15:02
TheJuliaOur agenda, as always can be found on the wiki.15:03
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic15:03
TheJuliaEveryone have coffee?15:04
TheJulia#topic Announcements / Reminders15:04
JayFThat's what the 5 minutes was for 😂15:04
TheJulialol, true!15:04
kubajjo/15:04
TheJuliaOur first item is a general reminder for folks to review the patches tagged ironic-week-prio.15:04
TheJulia#link https://tinyurl.com/ironic-weekly-prio-dash15:04
TheJuliaThere are 31 patches, lets try to get that below 25 again15:04
TheJuliaFurthermore, in terms of general reminder of where we are in the release cycle for 2026.115:05
TheJulia#link https://releases.openstack.org/gazpacho/schedule.html15:05
dtantsurThere is no such thing as enough coffee this January15:05
TheJuliaThis week is week R-11. A few reminders will follow on.15:05
janderscoffee++15:05
TheJuliaR-6 is final release for non-client libraries. R-5 is final release for client libraries. If your working on API surface changes, or have one which will impact the API surface, please highlight your work and remember to use the hashtag ironic-week-prio.15:06
TheJuliaMoving on!15:06
TheJulia#topic Working Group Updates15:07
TheJuliaFirst update, alegacy are you around regarding standalone networking?15:07
alegacyyep, i'm here15:07
alegacyLooks like a few patches merged this morning.  thank you!15:07
alegacyhoping to be able to continue that momentum in the coming weeks and wrap this up.15:07
alegacyonly 1 big patch left in the ironic repo... the other 3 small ones are cleanup/doc.15:08
TheJuliacool cool15:08
alegacyand then the 1 other patch over in bifrost.15:08
cido/15:08
TheJuliaVery cool, anything else?15:08
alegacynope, i'll keep an eye on incoming review comments.15:09
TheJuliaAwesome!15:09
TheJuliaOnward! Async IO. cid/dtantsur, any update? I'll apologize now I didn't look at the outstanding WIP patches yet.15:09
dtantsurI have some, yes15:09
dtantsurFirst, we have a proof-of-concept for asynchronous sensor collection: https://review.opendev.org/c/openstack/ironic/+/970450 and https://review.opendev.org/c/openstack/ironic/+/97084215:10
dtantsurThe latter is just for testing without refactoring the conductor15:10
dtantsurThe former can be treated as a preview of how the asynchronous API may look like (don't worry too much about implementation details so far).15:10
TheJuliaGood details, thanks!15:10
dtantsurI made two very opinionated decisions there15:10
dtantsur1) The new client is inside Ironic not Sushy. Given the discussion around moving Sushy in, I think it's a valid thing to do.15:11
dtantsur2) The new client is also low-level, it does not come with ready-to-use structures.15:11
dtantsurHere is where we may end up arguing quite a bit. I'm honestly quite tired of keeping sushy's structures up-to-date, especially since we use a tiny share of them.15:11
rpittauo/15:12
dtantsurCheck the script in the 2nd patch for how the API is *used* in practice. It's not as crazy as it may initially sound.15:12
dtantsurFinally, I'm working on the spec for all this:15:12
TheJuliaI think #2 makes a ton of sense given some of the object instantiation overhead which exists, but it really just depends more so on the end result of "what"15:12
dtantsur#link https://review.opendev.org/c/openstack/ironic-specs/+/972754 WIP spec for async sensor collection15:12
dtantsurIt has the motivational section and the high-level implementation proposal, so ready for early feedback.15:12
TheJuliacool cool15:12
TheJuliaAnything else?15:12
opendevreviewJacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface  https://review.opendev.org/c/openstack/ironic/+/96694615:13
dtantsurI've also tried a purely threaded approach, and our futurist pool is not suitable for 3500 threads15:13
cidtks, dtantsur. I took a quick look at what you have going on in those patches. No updates on my side regarding Async15:13
dtantsurWhile the async script finishes in 30 seconds on 3500 fake nodes from sushy-tools (not VM based).15:13
dtantsurThat's all from me. A bit verbose, but it summarizes a month worth of work :)15:13
cid++15:13
TheJuliaCool cool15:14
TheJuliaOkay, onward!15:14
TheJuliaVXLAN Networking. That is my area mostly, although cardoe feel free to chime in.15:14
cardoebleep blorp15:15
TheJuliaThe spec is in a ready to review state over at https://review.opendev.org/c/openstack/ironic-specs/+/959401 and provides the overview of attachments and required steps.15:15
TheJuliaThe implementation is basically 3 parts, two of which are needed as well for Neutron EVPN to be used later on if an operator chooses to do so, but the model is more a physcial bridge/connection model to the fabric allowing for operators to make decisions based on their traffic *and* loads, i.e. leaning more towards ASICs doing things than CPUs and network card acceleration features.15:16
TheJuliaWe're starting to see some initial implementation patches at this point in time.15:17
TheJuliacardoe: do you have anything else to add?15:17
cardoeNo I think that's spot on.15:17
* TheJulia reloads the agenda to see if anyone snuck a discussion topic in15:17
cardoeIf I could get the little time trinket from Happy Potter that Dumbledore gives Hermoine, I'd make a lot more progress.15:18
TheJuliaOkay, Given we have no discussion topics, we can proceed to Bug Deputy updates!15:18
TheJulia#topic Bug Deputy Updates15:18
cidThat's me15:19
cidI triaged two bugs and a new RFE enabling direct deploy interface to switch to ramdisk https://bugs.launchpad.net/ironic/+bug/2137729.15:19
JayFthat's my RFE if anyone has questions when we get there15:19
cidAlso, I would love to get feedback on this change: https://review.opendev.org/c/openstack/ironic/+/97301615:20
TheJuliaOkay, so who would like to serve as the deputy for next week?15:20
TheJulia(for the next week)15:21
cardoecid: yeah I'm a +1 on that change. I've got it opened to fully review it so that I can give it a +2.15:21
dtantsurhuh, yeah, disable_ramdisk makes sense for servicing15:21
cidAlright cool. Because it felt like something that might or not need a fix, I just was not sure15:22
cidTheJulia, sign me up for bug deputy for next week.15:22
TheJuliacid: okay!15:22
TheJuliaOnward!15:22
cardoeJayF: I think switching the deploy interface dynamically based on the image overall makes sense.15:22
TheJulia#topic RFE Review15:22
TheJuliaso, JayF, you have the floor15:22
JayFI mean, I'm following the pattern laid out by stevebaker[m] for the direct->bootc swap15:23
JayFI was worried it'd become an "X to Y" deploy interface change problem, but realistically bootc/direct/ramdisk/anaconda is probably the entire list of ones reasonable for this feature15:24
JayFso I don't think there's a need to genericize it too much, just follow Steve's pattern15:24
JayFI already validated w/Dan that Nova+Glance shouldn't need code changes (only docs)15:24
JayFso I15:24
dtantsurMy only request would be on the code level: please make a reasonable abstraction for it instead of just sticking more untested logic into deploy_utils15:24
TheJuliaYeah, the image itself hints/supplies almost every ounce of missing detail15:24
JayF**so I'm hoping to get this in for gazpacho15:24
dtantsur(maybe teh ship has sailed already)15:24
JayFdtantsur: review stevebaker[m]'s change15:25
JayFdtantsur: I will follow that pattern :)15:25
dtantsurhas it merged already?15:25
JayFno15:25
JayFhe owes me a guardrail15:25
JayF(so we don't try to enable bootc on conductors without bootc)15:25
dtantsurWell, at least it's not deploy_utils :D15:25
TheJuliaDo we have anything else to discuss regarding this RFE?15:26
dtantsur(oh I don't like this code oh oh)15:26
JayFdtantsur: yeah I figured you might have that reaction.15:27
TheJuliaThe bottom line is it needs to be driven by the data on hand, not a user requesting it in large part because all of the data and information is present based upon the image source/service, so somehow that needs to be reconcilable.15:27
dtantsurI don't hate the feature, just using validate() for that15:28
TheJuliaerr, I'm not a fan on validate()15:28
TheJuliaI'd much rather see it based upon the flow deeper in, but that leans towards deploy_utils as well15:28
TheJuliaI've not looked at it in a while though15:28
JayFI'm happy my RFE pointed more eyes at this :)15:29
TheJuliaThere are only so many ways to solve the problem though short of making changes invasive.15:29
* TheJulia suspects brains need to ponder15:30
TheJuliaDo we want to discuss the idea cardoe proposed just before the meeting?15:30
JayFIf you care, please prioritize the review as I really, really want to get it in for G15:30
dtantsurWill you hate me for suggesting a new "dynamic" deploy interface?15:30
JayFno, I actually think that's probably a solid idea15:30
cardoeWill you hate me if I say yes? :D15:30
JayFOnly real downside is my use case is "try ramdisk driver with less risk"15:31
cardoeI actually think dynamic makes sense and it can change and the others won't change.15:31
JayFand pushing down a "change all your interfaces" is not going to make people happy15:31
JayFbut that's an operational problem not an architectural one15:31
TheJulia... There is a point where that is a reasonable thing to do15:31
cardoeWe can make "direct" be an alias for "dynamic".15:31
cardoeAnd the existing direct becomes "agent"15:31
JayFI dislike that15:32
cardoeAnd then no change required of folks to get the functionality enabled.15:32
TheJuliaI *suspect*, if we act quickly, it won't be a big deal15:32
TheJuliaAnd by that meaning, the interfacing and patterning... as long as it lands around the same time, its not a big deal and folks can begin to adopt it15:33
* dtantsur https://review.opendev.org/c/openstack/ironic/+/966761/comment/1cbc9c11_d65ed2bf/15:33
TheJuliasgtm15:36
TheJuliaAre we good with RFE review?15:36
* TheJulia hears the crickets15:36
TheJulia#topic Open Discussion15:37
* dtantsur enjoys the cricket sound15:37
dtantsurJust as a heads-up: I'll have a bit of an odd availability in January because of a move15:38
dtantsurDon't get surprised if I take unusually long to respond or review something. Please bear with me for now.15:38
jandersdtantsur good luck, hope it all goes smooth!15:38
dtantsurThx! It's, as usual, quite a show :D15:38
JayFI will be out (up to) the last two weeks in Feburary for a medical thing, similarly will be unavailable.15:38
TheJuliadtantsur: I hope it all goes well!15:39
jandersI have a couple topics. First - I had a brief chat to DMTF folks about multi-system-systems.15:39
TheJuliaI have nothing really coming up so I should be around albeit likely very heads down on vxlan for the next month or two15:39
TheJuliajanders: seems like the floor is yours15:39
jandersThank you TheJulia15:40
jandersregarding DMTF discussion: short version: "Anything that isn't bootable isn't a system. A chassis can be used to group together components."15:40
jandersso it would seem to me that as far as they are concerned, if a System doesn't have enough components to boot up a workable OS, that's not right15:40
jandersso it seems we're not the only ones of an impression that the hardware that JayF mentioned is a bit weird - it's not just us Ironic'ers15:41
dtantsurWe cannot do much about the already existing hardware but I wonder if they can spread this message somehow15:41
dtantsurto prevent more instances of this15:41
JayFjanders: is that published somewhere?15:41
JayFjanders: or hallway conversations?15:42
TheJuliaHmm, Interesting15:42
JayFI have to have something in writing I can toss at the vendor architects/presales15:42
jandersJayF inconveniently, in DMTF-member-only repo15:42
jandersbut we can see what we can do (I can ask folks)15:42
jandersbut there is more15:42
JayFif you can email me details about it even if I don't have access, it may help15:42
janderslong version: they shared some docs to look at composability, etc which we can look at together if you're interested JayF15:42
JayFI'm academically interested; I15:43
TheJuliaI'd ask them for some way to close the loop since JayF can't access that repo without membership in the DMTF15:43
jandersthey further reinforce the "it needs to have enough kit to boot OS" line15:43
JayF**I'm not convinced it will wrap into value for my downstream15:43
jandersI can certainly help gathering documentation to potentially share with the $vendor15:43
dtantsurIf we can at least prevent this from becoming a new norm...15:43
TheJuliadtantsur: yeah15:43
jandersagreed15:43
jandersI can follow up in the thread15:44
janderseveryone is welcome to participate in the discussion but unfortunately DMTF membership is required to see the issue I raised15:44
TheJuliaSo, one thing I'm wondering, as a devil's advocate... what if the device *is* booting it's own OS but its not user managable? I realize that is a bit blurring of the exact case JayF and some others have observed with some of the hardware out there15:44
JayFYep. 15:45
TheJuliaAnd then... what if vendors view it that way and believe they are in compliance.15:45
JayFI think that's likely true for the hardware I'm talking about here15:45
JayFwow way to read between the lines TheJulia 15:45
JayFI hadn't even put that all together15:45
TheJuliaJayF: this is what working for lawyers for like... a third of my career has done to me.15:45
dtantsur"Everything with firmware is booting an OS" is quite a take..15:45
dtantsurbut yeah, I can see it being appealing15:45
JayFdtantsur: in some of this hardware, it's a real OS15:46
JayFdtantsur: think a DPU or a BMC which has an optionally operator-accessible OS15:46
JayFdtantsur: I know on this gear you can put the DPU into a operator-accessible OS mode15:46
JayFit's computers all the way down /o\15:46
JayFand... shit15:46
jandersLOL15:46
dtantsurI guess where the line could be drown is the question of self-sufficiency as a system15:46
JayFputting the computer in a mode where the DPU is NIC only15:46
JayFreduces the number of systems to 215:47
JayFthis is EXACTLY what is happenin15:47
TheJuliayup, now without I2C for inter-device communications!15:47
* TheJulia hides15:47
* dtantsur sighs15:47
* JayF nominates TheJulia to chair the DMTF, too15:47
jandersI think there are two action items here: 1) I will share the info I have with JayF and 2) update the thread with our concerns/questions15:47
TheJuliadtantsur: I think that is the most "ironic" response ever :)15:47
dtantsurlol15:47
TheJuliaJayF: ouch! But if that is what the members want, who am I to stand in the way ?!? ;)15:48
JayFyou can not only read lawyerspeak, you can use it!?!?!15:48
JayFAGREED this is dangerous ;) 15:48
jandersI see some opportunities here15:49
dtantsurIf DMTF was chaired by people like TheJulia rather than just vendors.. okay, I shut up here15:49
jandersdtantsur++15:49
TheJulialol15:50
TheJuliaY'all have made my day, thank you!15:50
TheJuliado we have anything else to discuss? Pending chaos, or crazy ideas?15:50
jandersTheJulia in seriousness are you a DMTF member?15:50
jandersif not, would you consider joining?15:50
TheJuliaI am not, I guess I could, send me the link or page to the process15:51
janderswill do15:51
TheJuliaThanks15:51
JayFThese organizations need a hook-point for stakeholders like Ironic15:51
* TheJulia might have looked at it ages ago but I really don't remember at this point15:51
JayFone that doesn't require handing over of money or becoming members or whatnot15:51
jandersOK: if nothing more on multi-system I would like to jump on the monitoring patch15:51
JayFIt's unfortunate it's structured this way.15:51
TheJuliaJayF: We pointed that out to them in the past and the answer was bascially "join the forum"15:51
JayFMost collaborations locked behind a user login is hostile to OSS15:52
TheJuliajanders: have you made a parallel patch for the client and yet another by sdk ?15:52
jandersI have a pair - API side and client side15:52
TheJuliaJayF: I agree, and even getting issues really resolved, ala this exact issue15:52
jandershttps://review.opendev.org/c/openstack/ironic/+/96694615:52
jandershttps://review.opendev.org/c/openstack/python-ironicclient/+/96705515:52
TheJuliajanders: openstacksdk as well15:53
jandersTheJulia noted. What do I need to break out to or add to SDK?15:53
janders(my first change requiring client side work, hence asking)15:53
TheJuliajanders: the api version and the field itself, AIUI15:55
jandersTheJulia ACK15:55
janderswill do15:55
TheJuliaAnything else today?15:55
janderswill I need to update the API/client patches again once that's done, or is SDK chenge self contained?15:55
TheJuliaSomehow we managed to use virtually the whole hour. I'm sure people need to refill their coffee or switch to tea... or something stronger.15:56
TheJuliajanders: it is elf contained15:56
TheJuliaself15:56
TheJulianot, elf15:56
jandersnoted, thank you15:56
TheJuliano elfs are located in the sdk, as far as I'm aware.15:56
jandersif cardoe and JayF have time I would appreciate re-reviews15:56
jandersall comments should be addressed15:56
janderslocal tests pass, just waiting for CI15:56
cardoewill do15:57
jandersclient change has one +2 15:57
jandersthank you!15:57
jandersI appreciate your time folks15:57
jandersthat's all from me15:57
dtantsurMeanwhile, I think I like the idea of a "dynamic" RAID interface as the next step after the deploy one. Both of these will make Metal3's code somewhat less annoying.15:57
TheJuliaCould be super useful16:00
TheJuliaand I think this is generally the next/logical progression for easing the user experience16:00
TheJuliaby removing, as JayF puts it, part of a decoder ring being necessary16:00
JayFwe've been big on being explicit16:01
JayFbut need to go back and add hooks -- like here -- where we can use metadata to store explicit decisions16:01
JayFlike "use this image with bootc/ramdisk/anaconda" or "pick the fibrechanel interface if the network is turquoise" 16:01
TheJuliaor make the decisions to ease the use case/patterning16:02
TheJuliaheh16:02
TheJuliaOkay folks, we're moving to colors16:02
TheJuliaThere will be no green and purple bins in this channel ;)16:02
TheJuliaThanks everyone! Have a great week!16:02
dtantsurMy favourite colors :(16:02
TheJulia#endmeeting16:02
opendevmeetMeeting ended Mon Jan 12 16:02:42 2026 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:02
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-01-12-15.01.html16:02
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-01-12-15.01.txt16:02
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-01-12-15.01.log.html16:02
janderso/16:02
jandersthanks folks, great discussion, feels like we well made up for the skipped weeks16:02
TheJuliadtantsur: but, we can't emulate the Drazi from Babylon 516:03
JayFpurple image, get purple deploy16:03
JayFgreen image, get green deploy16:03
TheJuliaSee: https://babylon5.fandom.com/wiki/The_Geometry_of_Shadows16:03
JayFnow there are two deploy images16:03
TheJuliaI guess that is fair...16:05
*** darmach2 is now known as darmach17:09
JayF\18:04
TheJulia????18:08
JayFmistake18:09
TheJuliabeep boop... whirrrrrl.18:25
opendevreviewcid proposed openstack/ironic master: Support `disable_ramdisk` during servicing  https://review.opendev.org/c/openstack/ironic/+/97301621:33
opendevreviewJay Faulkner proposed openstack/ironic master: DNM: Autodetect deploy interface (unedited from claude, seriously dont review)  https://review.opendev.org/c/openstack/ironic/+/97318721:48
opendevreviewJay Faulkner proposed openstack/ironic master: DNM: Autodetect deploy interface (unedited from claude, seriously dont review)  https://review.opendev.org/c/openstack/ironic/+/97318723:22
JayFdtantsur: stevebaker[m]: TheJulia: There are a couple things I don't likke baout how https://review.opendev.org/c/openstack/ironic/+/973187 is shaped, but that's a rough PoC for a autodetect deploy_interface  23:25
JayFI need to test it and to see if I can find a less-common path for the try;catch code23:25
stevebaker[m]JayF: I'm good with a dedicated autodetect deploy_interface if that is the consensus. Do we know enough now to move forward on the first review in that series? https://review.opendev.org/c/openstack/ironic/+/966760. I'd also be fine if JayF wants to do the dev work for autodetect. Likewise I can adapt my change in that direction, as long as we're not both working on it23:48
JayFstevebaker[m]: I just have been kiting along a claude session all day to help surface ideas; this isn't a strong investment from me other than just trying to present something to ensure the conversation continues. I'd really like us to get this behavior landed in G :)23:49
JayFI wouldn't have done this without getting a mutex with you if it wasn't a trivial time investment :D 23:49
JayFstevebaker[m]: also if you haven't; you may want to read the associated discussion from the meeting this (pst)morning23:50
stevebaker[m]JayF: OK, I can spend some time on this from now and evolve my change towards some kind of autodetect.23:56
JayFfeel free to use any or none of what I tossed together there23:57
JayFsometimes Claude thinks of things I wouldn't have (although the "just pass the instance_info modification method as a sidecar to the exceptoin" is all me)23:57
JayFthat's one thing my case has that's a bit more pathological than yours: I'm going to have to modify instance_info to fit what ramdisk driver expects23:57

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!