| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-specs master: Asynchronous sensor data collection https://review.opendev.org/c/openstack/ironic-specs/+/972754 | 00:03 |
|---|---|---|
| *** darmach10026 is now known as darmach1002 | 00:25 | |
| opendevreview | Adam McArthur proposed openstack/ironic master: api: Add schema for events API https://review.opendev.org/c/openstack/ironic/+/959880 | 05:04 |
| zigo | Hi there! I saw new activity on https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 does it mean it looks an ok fix for CVE-2026-43003 ? | 10:21 |
| opendevreview | mabulgu proposed openstack/ironic master: Improve vmedia insertion error messages https://review.opendev.org/c/openstack/ironic/+/982626 | 11:06 |
| iurygregory | good morning ironic | 11:07 |
| dtantsur | zigo: it seems close enough if you're in a hurry | 11:13 |
| zigo | dtantsur: Patch header says "Add Ironic conductor side change to add a config + plumb it through" so it's probably useless at this point to use the patch. | 11:24 |
| zigo | (it says that's "TODO") | 11:24 |
| dtantsur | zigo: the patch does change the default to False, which mitigates the CVE at the expense of being potentially breaking | 11:26 |
| zigo | Oh ok, thanks. | 11:26 |
| TheJulia | good morning | 13:10 |
| opendevreview | cid proposed openstack/ironic master: Retry on Redfish vmedia insert server errors https://review.opendev.org/c/openstack/ironic/+/990924 | 13:51 |
| opendevreview | cid proposed openstack/ironic master: Stop skipping Redfish vmedia insert on stale BMC state https://review.opendev.org/c/openstack/ironic/+/990925 | 13:51 |
| opendevreview | cid proposed openstack/ironic master: Retry on Redfish vmedia insert server errors https://review.opendev.org/c/openstack/ironic/+/990924 | 14:01 |
| opendevreview | cid proposed openstack/ironic master: Stop skipping Redfish vmedia insert on stale BMC state https://review.opendev.org/c/openstack/ironic/+/990925 | 14:01 |
| TheJulia | o/ | 15:00 |
| kubajj | o/ | 15:00 |
| iurygregory | o/ | 15:00 |
| iurygregory | wondering if we have quorum to start the meeting :D | 15:00 |
| TheJulia | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Jun 1 15:00:54 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 |
| JayF | o/ | 15:01 |
| TheJulia | We can always start the meeting, do we feel we have sufficient attendance to have quorum to make decisions is the question | 15:01 |
| iurygregory | o/ | 15:02 |
| kubajj | o/ | 15:02 |
| TheJulia | Do we feel we have enough folks to consider quorum? | 15:02 |
| TheJulia | I count 4 hopefully caffinated individuals | 15:02 |
| JayF | full disclosure; I'm only a half-cup in | 15:03 |
| iurygregory | 3.5 in this case lol | 15:03 |
| opendevreview | Merged openstack/ironic master: Improve vmedia insertion error messages https://review.opendev.org/c/openstack/ironic/+/982626 | 15:03 |
| clif | o/ | 15:03 |
| TheJulia | I'm only 1 cup in | 15:03 |
| TheJulia | okay, another victi^Wattendee ;) | 15:03 |
| TheJulia | I guess we have enough quorum to go through the motions | 15:04 |
| TheJulia | Welcome to this week's Ironic meeting of Irony where we shall discuss all things Ironic! | 15:04 |
| TheJulia | Our agenda can be found on the wiki! | 15:04 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:04 |
| TheJulia | #topic Announcements / Reminders | 15:04 |
| rpittau | o/ | 15:05 |
| TheJulia | Our standing reminder to review outstanding items tagged with ironic-week-prio. I realize folks are also starting to take summer time vacations as well, but lets just try to keep momentum up! | 15:05 |
| cid | o/ | 15:05 |
| TheJulia | In terms of the release schedule, this is week is R-17. | 15:06 |
| TheJulia | #link https://releases.openstack.org/hibiscus/schedule.html | 15:06 |
| dtantsur | o/ | 15:06 |
| rpittau | related to that, I'm planning to cut bugfix branches for ironic and IPA tomorrow | 15:06 |
| rpittau | I think we're in good shape, but just in case please review the outstanding ironic patches | 15:06 |
| TheJulia | rpittau: can I ask you to do that wednesday? | 15:06 |
| rpittau | sure! :) | 15:06 |
| TheJulia | Thanks! | 15:06 |
| TheJulia | Moving on! | 15:07 |
| TheJulia | #topic Working Group Updates | 15:07 |
| TheJulia | dtantsur: any async io updates? | 15:07 |
| iurygregory | I have o/ | 15:07 |
| TheJulia | go ahead! | 15:07 |
| dtantsur | TheJulia: I've passed the lead on this to iurygregory :) | 15:08 |
| iurygregory | I took over the spec, and updated to address most of the comments https://review.opendev.org/c/openstack/ironic-specs/+/972754 | 15:08 |
| iurygregory | so please review and let me know if there is something I should address on it | 15:08 |
| TheJulia | dtantsur: ack ack | 15:08 |
| TheJulia | iurygregory: Cool, thanks! | 15:08 |
| TheJulia | Folks, you know what to do! | 15:08 |
| TheJulia | #topic Discussion topics! | 15:08 |
| TheJulia | We have one discussion topic today! | 15:08 |
| * TheJulia places the podium and a microphone in front of the janders stand-in | 15:09 | |
| iurygregory | ok, I think it`s on me | 15:09 |
| TheJulia | Yes! | 15:10 |
| iurygregory | so, after the PTG we did some investigation about Firmware Upgrades | 15:10 |
| iurygregory | we came up with and etherpad with details | 15:10 |
| iurygregory | #link https://etherpad.opendev.org/p/ironic-firmware-updates-v2 | 15:10 |
| iurygregory | and a bug tracker was created also | 15:11 |
| iurygregory | #link https://bugs.launchpad.net/ironic/+bug/2153965 | 15:11 |
| iurygregory | not sure if folks had a chance to read it and have questions about it | 15:11 |
| dtantsur | I wonder if a spec would be a more familiar format.. | 15:12 |
| iurygregory | janders shared with some questions to help drive the discussion | 15:13 |
| TheJulia | Overall, I think I agree with everything I've read on this topic so far, but it feels a bit too verbose and I'd prefer simplification. I also would really wonder if we could get some data points to to help answer the lingering why question in a more concrete form. It almost feels like a "we're heading down this path because we must interate it. | 15:13 |
| TheJulia | and I guess without a solid reason or answer to the question of "why?", I'm just a little skeptical | 15:13 |
| iurygregory | TheJulia, got it! | 15:14 |
| JayF | My only question/request is that we have a good swath of redfish hardware being looked at for this feature | 15:14 |
| JayF | as long as we aren't modeling it against just one specific model it all seems OK to me | 15:14 |
| TheJulia | That is my other concern, because there is such a variety of behavior, this effort needs to be super careful as a result. | 15:14 |
| iurygregory | yeah, we have tested with two Dells and one HPE if I recall | 15:14 |
| TheJulia | Any other thoughts/discussions/questions/concerns/cups of coffee to sip, or suddenly appearing wet cat. | 15:15 |
| TheJulia | ? | 15:15 |
| dtantsur | I think the gist of it is something along the lines "if you submit several components as part of a single step, we're free to merge or rearrange them" | 15:15 |
| JayF | Given we already know that lenovos act weird w/r/t booting, that might be a good one to look at as well if we have one availble | 15:15 |
| iurygregory | R640, XR8620t, DL380 (I think) | 15:15 |
| dtantsur | which does leave the escape hatch of submitting updates separately | 15:15 |
| iurygregory | JayF, I wish we had some lenovo to test this =( | 15:15 |
| TheJulia | dtantsur: that is kind of my take as well, fwiw. | 15:16 |
| TheJulia | I guess the original thought was in operator supplied order | 15:16 |
| TheJulia | but, maybe thats not great, dunno | 15:16 |
| dtantsur | yeah, the current implementation treats the updates as separate steps | 15:17 |
| dtantsur | which is not great if you want to update 3-4 components, and each does a reboot just because | 15:17 |
| TheJulia | Yeah | 15:17 |
| iurygregory | if we are ok with rearranging the order of the updates it would help a lot based on our investigation, so we can avoid unnecessary reboots (because we could do a single reboot for non-bmc updates) | 15:17 |
| TheJulia | But some of them may or might and only the BMC may know for sure | 15:17 |
| TheJulia | Which will also govern responses | 15:17 |
| TheJulia | "like, configuration job pending reboot" | 15:17 |
| dtantsur | It's again the "vendors will only agree to disagree" situation, and we want to give a humam more choice :) | 15:18 |
| TheJulia | yeah, agree | 15:18 |
| dtantsur | the only potentially issue is that we're changing the behavior of an already working scenario | 15:18 |
| TheJulia | the thing which worries me is mostly just a bmc saying "I've got this, I can't do anything else until I reboot and do this with a configuration job because of how I do this firmware update as compared to my own update" | 15:18 |
| dtantsur | if that's a concern, just submit several steps, which will work the same way as now | 15:20 |
| TheJulia | Well, the operator has no way to know that *until* the are in the middle of it | 15:20 |
| TheJulia | so... *shrug* | 15:20 |
| TheJulia | Anyway, Are we good to proceed onward? | 15:20 |
| dtantsur | Many operators that I deal with have a very-very carefully picked hardware | 15:20 |
| dtantsur | yeah, we're digging too deeply into it :) | 15:21 |
| TheJulia | yup | 15:21 |
| dtantsur | I guess the question is what janders is supposed to do next | 15:21 |
| iurygregory | yeah | 15:21 |
| TheJulia | dtantsur: keep on keeping on? | 15:21 |
| dtantsur | Like, assume the proposal is accepted and work on the code? | 15:21 |
| iurygregory | does it needs a spec? | 15:21 |
| dtantsur | Or do more to convince us? | 15:21 |
| dtantsur | Or address certain specific concerns? | 15:21 |
| TheJulia | I'm semi-convinced... however... we need to give a clear answer to the why question | 15:21 |
| TheJulia | and if that means a spec, so be it | 15:21 |
| dtantsur | Why is essentially a downtime window | 15:22 |
| dtantsur | 5 components times 10 minutes reboot is nearly an hour of downtime | 15:22 |
| TheJulia | yup, we shouldn't get into that in irc, that needs to be in the recorded artifacts | 15:22 |
| TheJulia | i.e. RFE or even spec | 15:22 |
| dtantsur | Agreed. I assume janders will read this discussion tomorrow | 15:22 |
| TheJulia | because operators are going to have different wants/demands/opinions | 15:22 |
| TheJulia | ++ | 15:23 |
| TheJulia | Onward? | 15:23 |
| dtantsur | yep | 15:23 |
| TheJulia | #topic Bug Deputy Updates | 15:23 |
| * TheJulia pushes the podium over to cid | 15:23 | |
| cid | \o/ | 15:23 |
| cid | Moderately busy week. | 15:23 |
| cid | I entered 5 newly filled bugs to the triaged state. | 15:23 |
| cid | One around DMTF Redfish schema is something I left open as needing help to triage | 15:23 |
| cid | https://bugs.launchpad.net/ironic/+bug/2154614 | 15:23 |
| cid | 5 new RFEs but there's really only 3 valid RFEs on the list because of the 5, one is a duplicate of discussion topic and the other I filled as an RFE and is ending up as just a fix. | 15:24 |
| cid | That leaves only: | 15:24 |
| cid | https://bugs.launchpad.net/networking-generic-switch/+bug/2154349: Refresh ngs_trunk_ports with full VLAN segment list on each network operation | 15:24 |
| cid | https://bugs.launchpad.net/ironic/+bug/2154494: Please support Docker for console container provide | 15:24 |
| cid | https://bugs.launchpad.net/ironic/+bug/2154610: Permit RBAC decisions based on JSON key update | 15:24 |
| JayF | I'll note that docker console container RFE is from our downstream, via StackHPC, for integration into kolla-ansible | 15:24 |
| * cid list of all newly triaged bugs are visible on the wiki page | 15:25 | |
| dtantsur | The schema one is interesting. Someone needs to double-check both the DMTF documents and what Ironic actually uses. May end up being a simple typo. | 15:25 |
| * cid https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_June_1.2C_2026 | 15:25 | |
| JayF | So I know that was a loose end on containers, GR-OSS is getting it tied up | 15:25 |
| TheJulia | dtantsur: I think its more dell went in one direction early on and the profile was contributed with that and not updated as things shifted | 15:25 |
| TheJulia | dtantsur: but yeah, someone just needs to tak ethat one and roll with it | 15:25 |
| TheJulia | JayF: cool | 15:25 |
| dtantsur | any volunteers for the schema one? I don't want it to be me, but in the worst case I can occupy claude with it.. | 15:26 |
| TheJulia | I was about ask something similar to ^ | 15:26 |
| iurygregory | I can try to take a look in my free time | 15:26 |
| TheJulia | .... Could Jacob be a good resource for this? | 15:26 |
| TheJulia | It should be relatively straight forward, he has engaged with the DMTF so he is also aware of the profile importance | 15:27 |
| iurygregory | I can check with him today | 15:27 |
| dtantsur | iurygregory: free.. WHAT?? Oo | 15:27 |
| TheJulia | iurygregory to take the action item to sync up with others. cool! | 15:27 |
| TheJulia | ... wait, free time?! | 15:27 |
| dtantsur | right? | 15:27 |
| TheJulia | WUT!? | 15:27 |
| iurygregory | hahaha | 15:27 |
| dtantsur | iurygregory: thanks! just please make sure we don't drop it to the floor, it's quite bad if we declare all modern hardware as incompatible.. | 15:28 |
| iurygregory | when I have my priorities under review/or ci is running lol | 15:28 |
| TheJulia | Thanks! | 15:28 |
| iurygregory | its free time and I need to find something to work on lol | 15:28 |
| TheJulia | So who shall be the next bug deputy? | 15:28 |
| Mahnoor | I can do it | 15:28 |
| TheJulia | Mahnoor, thanks! | 15:29 |
| cid | tks | 15:29 |
| TheJulia | Onward if there is nothing else for the Bug Deputy Updates? | 15:29 |
| cid | Nothing else at this time :-) | 15:29 |
| TheJulia | #topic RFE Review | 15:29 |
| TheJulia | We have *FIVE* RFEs... eek | 15:30 |
| cid | Yup | 15:30 |
| TheJulia | I think the NGS one can be approved, it just makes sense and we have some switch vendors where it is basically required to do things properly. | 15:30 |
| TheJulia | I don't knwo why there is another firmware RFE ixed in?! | 15:32 |
| cid | Duplicate... | 15:32 |
| TheJulia | i guess | 15:32 |
| TheJulia | To me the rest of the RFE's make sense to me and seem logical | 15:32 |
| cid | So we should add the rfe-approved tags. | 15:33 |
| TheJulia | If there are no objections, yes | 15:33 |
| dtantsur | It pains me that we're still dealing with JSON free-form fields.. but we definitely are, so it makes sense to protect them better | 15:34 |
| dtantsur | Re https://bugs.launchpad.net/ironic/+bug/2154382, I'm not sure like relying on add_ports there, although I see the logic there.. | 15:35 |
| dtantsur | also i see that cid's patch does add a new option, so it's probably fine | 15:36 |
| JayF | CID's patch is a upstream-port of a downstream fix GR team is running | 15:36 |
| TheJulia | That was sort of what I was thinking as well | 15:36 |
| cid | Yeah, we found out the add_ports approach was not necessarily the right one. | 15:37 |
| dtantsur | I think I'm good with all of them | 15:37 |
| cid | New gating flag, then remove the additonal pre port creation in the redfish interface completely. | 15:37 |
| TheJulia | So onward? | 15:38 |
| cid | ++ | 15:38 |
| TheJulia | #topic Open Discussion | 15:38 |
| TheJulia | What items do we have to discuss? | 15:39 |
| TheJulia | Bueller? | 15:40 |
| TheJulia | Clearly no items today! | 15:41 |
| TheJulia | #topic Who shall run the next meeting? | 15:42 |
| JayF | put me in coach | 15:42 |
| TheJulia | The agenda suggested iurygregory, should we let you two debate it out? | 15:42 |
| JayF | I think that usually is saying who volunteered *last time* | 15:43 |
| JayF | but imbw | 15:43 |
| JayF | believe it or not, IDC who runs the meeting as long as it is run ;) | 15:43 |
| TheJulia | same! | 15:43 |
| * TheJulia puts JayF in | 15:44 | |
| TheJulia | Anything else for this weekly meeting?! | 15:44 |
| TheJulia | In that case, thanks everyone for attending! | 15:45 |
| TheJulia | #endmeeting | 15:46 |
| opendevmeet | Meeting ended Mon Jun 1 15:46:14 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-06-01-15.00.html | 15:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-06-01-15.00.txt | 15:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-06-01-15.00.log.html | 15:46 |
| TheJulia | https://www.youtube.com/watch?v=KS6f1MKpLGM | 15:46 |
| dtantsur | cid: JFYI I'll add a test on pre-creating MAC addresses to the BMO suite https://github.com/metal3-io/baremetal-operator/pull/3299 but we cannot yet use Depends-On with metal3 changes.. | 16:02 |
| * dtantsur needs to fix that as well | 16:02 | |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent master: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 16:32 |
| clif | what's the canonical way to create a new release note for IPA? it doesn't seem to follow the same step(s) as ironic | 16:40 |
| clif | at least reno is not available in the venv | 16:41 |
| clif | I guess I just install it then continue on, but maybe we should add it by default? and document the process? | 16:42 |
| TheJulia | I just have it installed | 16:42 |
| TheJulia | locally that is :) | 16:42 |
| clif | fair | 16:50 |
| clif | I tend to install as little as possible in my os-wide pip | 16:50 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent master: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 16:58 |
| TheJulia | clif: it is basically the only tool I do that with | 17:03 |
| JayF | gentoo packages reno | 17:06 |
| JayF | I don't think we have a good reason to, and I hope nobody ever notices and tries to last-rite it | 17:06 |
| JayF | because it makes it super easy for doing release notes lol | 17:06 |
| JayF | clif: https://review.opendev.org/c/openstack/ironic-python-agent/+/987391/5#message-abca80b16fdcfa8bbf1cf1066ba8d6c384c5f1df on the IPA change, minor feedback depending on which choice you make to resolve it :) | 17:18 |
| cid | dtantsur, ++ re' https://github.com/metal3-io/baremetal-operator/pull/3299. | 17:27 |
| cid | /me assumes context is related to the just landed docs/test suite changes on the Ironic repo. | 17:27 |
| cardoe | clif: I use uvx | 17:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add TLS hardening for HTTP image service connections https://review.opendev.org/c/openstack/ironic/+/987906 | 17:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add TLS hardening for OCI registry connections https://review.opendev.org/c/openstack/ironic/+/987907 | 17:54 |
| clif | JayF: landing the conductor side first seems easier to me, wdyt? | 18:02 |
| JayF | I have zero preference, up to you | 18:02 |
| JayF | add the depends-on if we wanna do that though please so we can indicate the logical dep | 18:03 |
| clif | https://review.opendev.org/c/openstack/ironic/+/990724 <- Ironic side | 18:03 |
| clif | needs a release note at a minimum but please review | 18:03 |
| clif | ok | 18:03 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent master: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 18:03 |
| JayF | clif: dtantsur: TheJulia: https://review.opendev.org/c/openstack/ironic/+/990724/1#message-c4eb93279ad6600dd28d8fa506634e4228f05f3a are we disabling-by-default bootloader installs, or disabling-for-deprecation? | 18:04 |
| TheJulia | disabling by default I think | 18:05 |
| TheJulia | meaning, not depreating afaik | 18:07 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add TLS hardening options for Redfish BMC connections https://review.opendev.org/c/openstack/ironic/+/987908 | 18:08 |
| JayF | TheJulia: Can you put that in gerrit then, please? This is basically walling off that corner of the code to rot for ~forever if we don't deprecate it :| | 18:09 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NetconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/990061 | 18:14 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NETCONF OpenConfig driver documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/990062 | 18:14 |
| JayF | cardoe: wouldn't mind review feedback on my review feedback :D https://review.opendev.org/c/openstack/ironic/+/990340/4#message-0b4a628d13ab1ea29200f9b6ca0b5814b3236b1c | 20:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add TLS hardening for Ansible stream_url module https://review.opendev.org/c/openstack/ironic/+/985885 | 21:11 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add TLS hardening for kickstart configdrive and inspection rule connections https://review.opendev.org/c/openstack/ironic/+/988804 | 21:11 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Add TLS configuration to agent lookup response https://review.opendev.org/c/openstack/ironic/+/987887 | 21:12 |
| opendevreview | Merged openstack/ironic master: Fix accumulation of any_left flag across storage controllers https://review.opendev.org/c/openstack/ironic/+/990471 | 21:17 |
| cardoe | JayF: I don’t disagree. The names of the options all suck. I think we agreed before but didn’t have a better name. | 21:57 |
| cardoe | Alas this is my formal review comment. https://usercontent.irccloud-cdn.com/file/pjahb4DN/IMG_0425.JPG | 21:58 |
| TheJulia | I approve of that comment | 22:09 |
| JayF | if this counts as code review, I'm going to have to bill for this trip to Scenic Beach State Park on Sunday. https://usercontent.irccloud-cdn.com/file/wNOOp14N/16805.jpg | 22:41 |
| JayF | (I have photos of unknown, but hopefully better quality on my camera SD card, unprocessed) | 22:42 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add validation for servicing https://review.opendev.org/c/openstack/ironic/+/990996 | 23:10 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [WIP] Require custom networks for custom ramdisks https://review.opendev.org/c/openstack/ironic/+/990997 | 23:10 |
| janders | iurygregory TheJulia dtantsur JayF thank you for your pointers w/r/t firmware updates. I will update the RFE to address some of the points you made. Will ponder the idea of a spec for this. | 23:18 |
| janders | I'm also happy to have a look at https://bugs.launchpad.net/ironic/+bug/2154614 | 23:21 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
