| *** hroy_ is now known as hroy | 02:37 | |
| rpittau | good morning ironic! o/ | 07:25 |
|---|---|---|
| kubajj | good morning, ironic! o/ | 07:41 |
| *** mdfr1 is now known as mdfr | 12:55 | |
| *** dking is now known as Guest33140 | 13:56 | |
| *** Guest33140 is now known as dking | 13:57 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Update supported Python versions https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/969473 | 14:13 |
| janders | good afternoon Ironic o/ | 15:02 |
| kubajj | o/ | 15:04 |
| janders | is it weekly meeting time, or am I TZ confused? | 15:04 |
| kubajj | janders: I also though it was now | 15:05 |
| clif | o/ I believe it is time | 15:05 |
| janders | let's ping the chairs | 15:05 |
| kubajj | it's not one of those weeks when it's canceled, right? that's 22 and 29th, I think | 15:05 |
| janders | JayF TheJulia dtantsur rpittau are you around? | 15:06 |
| janders | kubajj yeah that's my understanding as well | 15:06 |
| rpittau | janders: I kind of am, very distracted | 15:06 |
| cid | I think dtantsur is out this week, so is TheJulia. Maybe no weekly meeting this week. | 15:07 |
| janders | thanks rpittau. Let's see who else is here, from scrollback it seems most folks are out | 15:07 |
| JayF | Yeah, sorry I'm not in a position this morning to chair the meeting either. | 15:08 |
| dtantsur | I'm in, just very tired and pretty busy :) | 15:08 |
| dtantsur | does not look like we have a quorum? | 15:09 |
| kubajj | how many people do we need for quorum? | 15:09 |
| dtantsur | let's start with anyone who can chair the meeting :D | 15:10 |
| dtantsur | #startmeeting ironic | 15:10 |
| opendevmeet | Meeting started Mon Dec 8 15:10:14 2025 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:10 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:10 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:10 |
| dtantsur | Hey folks, do we have a quorum? | 15:10 |
| alegacy | o/ | 15:10 |
| dtantsur | hands-up! | 15:10 |
| rpittau | o/ | 15:10 |
| janders | o/ | 15:10 |
| mostepha[m] | o/ | 15:10 |
| dtantsur | This may be a very short meeting since everyone is obviously busy wrapping up the business before winter holidays | 15:11 |
| dtantsur | #topic Announcements / Reminders | 15:11 |
| dtantsur | Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: https://tinyurl.com/ironic-weekly-prio-dash | 15:11 |
| dtantsur | 2026.1 Gazpacho Release Schedule https://releases.openstack.org/gazpacho/schedule.html | 15:11 |
| dtantsur | I believe we got our first intermediate releases out, right rpittau? | 15:11 |
| dtantsur | Welcome Ironic 33.0 and its friends | 15:12 |
| rpittau | yep :0 | 15:12 |
| dtantsur | Another reminder: the meetings on Dec 22nd and 29th are canceled | 15:12 |
| dtantsur | anything else? | 15:12 |
| kubajj | o/ | 15:13 |
| janders | couple updates from my side: | 15:13 |
| janders | 1) iRMC deprecation email sent to ML, so AFAIK we can tick this item off | 15:13 |
| dtantsur | yeah, the agenda has not been updated | 15:13 |
| janders | should I note this somewhere? | 15:14 |
| * dtantsur not sure | 15:14 | |
| janders | (wasn't thinking agenda, just general work-plan tracking) | 15:14 |
| dtantsur | If we got the deprecation patch in, it's enough | 15:14 |
| janders | agreed, sounds like we have iRMC topic sorted till next PTG | 15:15 |
| janders | 2) I raised multi-system-systems topic with DMTF folks, it's in the queue. Hasn't been discussed yet, I will update you folks once that happens | 15:16 |
| janders | that's me w/r/t follow ups from the last meeting, I have one more RFE topic once the time is right | 15:16 |
| dtantsur | Thanks! | 15:16 |
| dtantsur | Anything else from anyone? | 15:16 |
| dtantsur | okay, moving on | 15:17 |
| janders | sounds like there's nothing else so here is the said RFE: https://bugs.launchpad.net/ironic/+bug/2133522 | 15:17 |
| dtantsur | let's wait until the RFE review | 15:17 |
| janders | OK! | 15:17 |
| dtantsur | #topic Working Group Updates | 15:17 |
| dtantsur | first, anything new in the networking world, alegacy? | 15:17 |
| dtantsur | I know we've ticked off at least two patches | 15:17 |
| alegacy | Continuing to make progress with some patches being merged. | 15:17 |
| alegacy | Made some changes on the weekend based on discussions on Friday about inspections being optional | 15:18 |
| alegacy | will wait for reviews on that | 15:18 |
| alegacy | There is one patch though that has a +2 that is ready to go. | 15:18 |
| dtantsur | nice | 15:19 |
| alegacy | that's it from me on networking | 15:19 |
| dtantsur | Great, thank you! | 15:19 |
| dtantsur | I don't have anything for the Async WG. But I do plan on trying some sort of a proof of concept (maybe with (ab)using AI) to start moving here. | 15:20 |
| dtantsur | Maybe literally "create me a small async client from Redfish based on this <sushy> code and reusing its data structures" to claude, we'll see | 15:20 |
| dtantsur | I don't believe we have anyone from the VXLAN group on? | 15:21 |
| dtantsur | Any updates on it? Or any comments or questions for any WG? | 15:21 |
| janders | one small one for me for the async one | 15:21 |
| janders | I wonder if this work could really help with various patches handling BMC unresponsiveness | 15:21 |
| janders | may be worth a chat once I'm back from PTO | 15:22 |
| dtantsur | Keep in mind that the initial scope is only metrics | 15:22 |
| dtantsur | so yeah, maybe in the future, but not in the near future | 15:22 |
| janders | fair point | 15:22 |
| janders | thanks for thoughts/pointers here | 15:22 |
| JayF | I still do think it could help when we get there. We just got to walk before we can run. | 15:23 |
| dtantsur | indeed | 15:23 |
| dtantsur | We don't have any new discussion topics, so I guess that brings us to... | 15:23 |
| dtantsur | #topic Bug Deputy Updates | 15:23 |
| dtantsur | Who was the bug deputy this time? The agenda has an old update from cid, I think | 15:24 |
| cid | No updates. 2 new bugs | 15:24 |
| dtantsur | anything requiring our discussion? | 15:24 |
| cid | Not really. | 15:25 |
| dtantsur | Okay. Before we discuss the Jacob's RFE, any volunteers from the next bug deputy? | 15:25 |
| dtantsur | s/from/for/ of course | 15:25 |
| dtantsur | only crickets :) | 15:26 |
| cid | I don't mind. | 15:26 |
| cid | I will always be bug deputy :D | 15:26 |
| dtantsur | You're doing such a great job, nobody risks competing ;) | 15:27 |
| cid | :D | 15:27 |
| dtantsur | #topic RFE Review | 15:27 |
| dtantsur | janders: your turn | 15:27 |
| janders | thank you dtantsur | 15:27 |
| janders | as discussed last week I created the RFE for Redfish health monitoring patches: https://bugs.launchpad.net/ironic/+bug/2133522 | 15:28 |
| dtantsur | #link https://bugs.launchpad.net/ironic/+bug/2133522 Enable basic hardware health monitoring for Redfish driver | 15:28 |
| janders | the code itself is nearly ready, request for the RFE came up in reviews hence creating it | 15:28 |
| janders | I'd appreciate your feedback, if you feel we can approve it I can work towards finalising the patch | 15:29 |
| dtantsur | From purely the RFE perspective, it lacks information on what the values of "Health" can be | 15:29 |
| dtantsur | (where "whatever BMC reports" is not the correct answer, to be clear) | 15:29 |
| janders | is "expected values" based on collection of various BMC dumps a valid option? | 15:29 |
| dtantsur | So maybe agree on an enum: OK, Warning, Error (or Failure?), null (= unknown)? | 15:29 |
| dtantsur | what I'm saying is: I'd much rather have some fixed values for all drivers/implementations than desiging the API around vendor-provided values | 15:30 |
| dtantsur | imagine one day we find hardware that says "Warn" and one that says "Warning", something like that | 15:30 |
| janders | totally fair | 15:31 |
| dtantsur | or upper vs lower case | 15:31 |
| janders | I can have a look at Redfish spec to see how prescriptive they are for extra context | 15:31 |
| dtantsur | in fact, since everything in Ironic is lower case (provision and power states, for example), maybe it should be OK/warning/error | 15:31 |
| dtantsur | yes please. and keep in mind the API design, not just the backend's behavior | 15:31 |
| janders | will do, thank you for your inputs | 15:32 |
| dtantsur | any other comments for Jacob? | 15:32 |
| dtantsur | From my side, I'll be fine with approving the RFE directly once we sort out the valid values issue | 15:33 |
| dtantsur | #topic Open discussion | 15:34 |
| janders | thank you dtantsur | 15:34 |
| dtantsur | Sure thing | 15:34 |
| dtantsur | Anyone has anything? | 15:34 |
| JayF | Thanks for running the meeting! | 15:34 |
| janders | ++ | 15:35 |
| janders | nothing more from me | 15:35 |
| cid | janders, a little nit is to tag the RFE on your change so that it's automatically tracked and closed. | 15:36 |
| janders | thanks cid, will do | 15:36 |
| janders | I will add the reference while re-spinning the patch with the enum approach | 15:37 |
| cid | ++ | 15:37 |
| dtantsur | great, irccloud bouncer died, using the web UI now | 15:38 |
| dtantsur | Thank you all, let me try to close the meeting for the 3rd time already: | 15:38 |
| dtantsur | #endmeeting | 15:38 |
| opendevmeet | Meeting ended Mon Dec 8 15:38:33 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-12-08-15.10.html | 15:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-12-08-15.10.txt | 15:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-12-08-15.10.log.html | 15:38 |
| janders | thank you folks | 15:38 |
| dtantsur | Remind me, why I'm I paying for something that works worse than just ZNC on a random server? Sigh | 15:38 |
| janders | :( | 15:39 |
| dking | dtantsur: Would you happen to be available for some IrSO questions? | 16:21 |
| dtantsur | dking: you caught me today :) | 16:21 |
| dking | I thought so. :) | 16:22 |
| dking | dtantsur: So, I'm looking at switching over to IrSO, but one thing we're doing now with BMO is that we ignore the standard ramdisk download and instead we have a container that checks regularly for ramdisk images at a URL (in JFrog) and pulls down new images when a new one is available. Unfortunately, I don't know of a way to do that with IrSO, as the ramdisk downloader is an initcontainer, and and the ironic pod would be cre | 16:25 |
| dking | ed dynamically by the operator, using an emptydir mount. | 16:25 |
| dtantsur | dking: you can combine two parts: disableDownloader https://github.com/metal3-io/ironic-standalone-operator/blob/main/docs/api.md#ironicspecdeployramdisk to turn on the default init container, then add your initContainer to https://github.com/metal3-io/ironic-standalone-operator/blob/main/docs/api.md#ironicspecoverrides | 16:27 |
| dtantsur | we'll be in a similar boat in OpenShift: we also use a different approach to getting IPA | 16:27 |
| dtantsur | so if it does not work out for you, let me know, I'll have to fix it :) | 16:27 |
| dking | dtantsur: Great! The first oen I'm already doing, and the second is what I was wanting. I was going to even consider writing code to make that happen. | 16:29 |
| dtantsur | yeah, not just you. we'll have to use overrides quite actively too. | 16:29 |
| dking | I didn't see that in the code, but I'm sure I was looking in the wrong place. | 16:29 |
| dking | dtantsur: Forgive my ignorance, but I see that this requires the Overrides feature gate, but I'm having trouble finding where to set that from the documentation. | 16:50 |
| dtantsur | dking: the gate is on by default btw. Otherwise.. yeah, not documented yet, but it's a CLI option/env variable for the operator, e.g. https://github.com/metal3-io/ironic-standalone-operator/blob/main/config/testing/manager_irso_config_patch.yaml#L12-L13 | 16:51 |
| dking | Thank you. I'll give it a try. | 16:53 |
| dking | dtantsur: Welp. I must be doing something wrong. I get 'Ironic in version "v1alpha1" cannot be | 17:15 |
| dking | handled as a Ironic: strict decoding error: unknown field "spec.overrides.containers"' Does that mean that I'm still not getting the feature gate correct? I added 'FEATURE_GATES: Overrides=true' to the irso operator and even redeployed. | 17:15 |
| dtantsur | dking: are you sure you have a recent version of IrSO *and* its CRDs? | 17:17 |
| dtantsur | CRDs themselves do not depend on feature gates | 17:18 |
| dking | I just downloaded it last week, but I can check. it accepts overrides, but just not overrides.containers, so I suppose it wouldn't be the feature gate anyway. I'll look at the CRDs. | 17:23 |
| dtantsur | it does seem to be present in the upstream definitions: https://github.com/metal3-io/ironic-standalone-operator/blob/main/config/crd/bases/ironic.metal3.io_ironics.yaml#L311-L325 | 17:23 |
| dking | That would do it. I don't know where I got my copy, but hose lines aren't in it. Thanks. | 17:26 |
| dking | dtantsur: Okay. I got it from https://github.com/metal3-io/ironic-standalone-operator/releases/download/v0.6.1/install.yaml. This option was added into the main branch, but there's no release since v0.6.1 10 days ago, and that doesn't include that code. | 17:41 |
| dtantsur | ahhhhh, sure, I see | 17:41 |
| dtantsur | we're about to make another release, you can wait for a couple of days, or you can pick up the main branch for now | 17:42 |
| * dtantsur has lost track of what was added when | 17:43 | |
| *** jgwentworth is now known as melwitt | 19:01 | |
| JayF | training a new ironic reviewer https://usercontent.irccloud-cdn.com/file/op3DTSxR/juno.jpg | 19:42 |
| opendevreview | Steve Baker proposed openstack/ironic master: Swap to bootc deploy interface for bootc images https://review.opendev.org/c/openstack/ironic/+/966761 | 20:49 |
| opendevreview | cid proposed openstack/ironic master: Move `check_image_size` to `deploy_utils` https://review.opendev.org/c/openstack/ironic/+/970181 | 21:09 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove ovn vtep support https://review.opendev.org/c/openstack/ironic/+/966140 | 21:28 |
| JayF | unit tests++ ^^ caught an error in my rebase \o/ | 21:44 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Separate states code from states constants https://review.opendev.org/c/openstack/ironic/+/969318 | 22:17 |
| *** melwitt is now known as jgwentworth | 22:31 | |
| *** jgwentworth is now known as melwitt | 22:32 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
