| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: Fix support for Aruba AOS CX devices https://review.opendev.org/c/openstack/networking-generic-switch/+/958234 | 07:38 |
|---|---|---|
| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: Aruba: Add error message patterns to detect errors https://review.opendev.org/c/openstack/networking-generic-switch/+/958231 | 07:39 |
| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: Detect device configuration errors on Dell Force10 and OS10 https://review.opendev.org/c/openstack/networking-generic-switch/+/868318 | 07:40 |
| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: Cast boolean Netmiko kwargs to native types https://review.opendev.org/c/openstack/networking-generic-switch/+/958079 | 07:40 |
| abongale | good morning ironic o/ | 08:07 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Add type annotations and checking https://review.opendev.org/c/openstack/ironic-python-agent/+/958333 | 09:32 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job https://review.opendev.org/c/openstack/ironic/+/959690 | 14:04 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP run the ironic-standalone-operator tests as non-voting job https://review.opendev.org/c/openstack/ironic/+/959690 | 14:18 |
| * dtantsur is curious if we even have quorum for the meeting | 14:51 | |
| JayF | \o | 14:52 |
| dtantsur | maybe people are just silent :D (so was I) | 14:54 |
| kubajj | o/ | 14:59 |
| clif | o/ | 15:00 |
| masghar | o/ | 15:00 |
| dtantsur | Who is chairing today btw? I'm waiting for a delivery within 15 minutes | 15:00 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Run the ironic-standalone-operator tests as non-voting job https://review.opendev.org/c/openstack/ironic/+/959690 | 15:01 |
| TheJulia | I'm taking today off, so not me :) | 15:01 |
| JayF | #startmeeting ironic | 15:02 |
| opendevmeet | Meeting started Mon Sep 8 15:02:39 2025 UTC and is due to finish in 60 minutes. The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:02 |
| TheJulia | OTOH, I saw an awesome post from someone on LinkedIn this past weekend which spoke very highly of the Ironic project. The comments were fairly positive, one that redfish is basically awful and doesn't work which I encouraged them to file bugs. The other comments were mostly around our docs. | 15:02 |
| dtantsur | Thanks Jay! | 15:02 |
| JayF | Hello everyone, welcome to the ironic meeting! Going to wait a few minutes for folks to get ready to meet; totally not using it to open the agenda :D | 15:03 |
| dtantsur | TheJulia: nice, thanks for sharing! Docs are .. docs. | 15:03 |
| JayF | dtantsur: link me that please | 15:03 |
| dtantsur | JayF: you meant to ask TheJulia, right? | 15:03 |
| TheJulia | dtantsur: yeah, I think the perception is largely rooted in the layout overall and the fact we call back to prior behavior a lot | 15:03 |
| JayF | So current agenda in the wiki is 8/25 | 15:03 |
| JayF | yes, I did | 15:03 |
| JayF | I'm going to wing it | 15:03 |
| JayF | #notes Reminder to review things in our ironic-week-prio dashboard: https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| JayF | All flamingo releases have been cut now, right? | 15:04 |
| JayF | I didn't check to see if the patches merged, but I reviewed them | 15:04 |
| dtantsur | I feel like Ironic is pending still | 15:04 |
| dtantsur | at least I don't see the branch | 15:04 |
| JayF | #note Ironic releases are pending review from release team: https://review.opendev.org/q/topic:%22ironic-releases-2025.2%22 | 15:04 |
| TheJulia | JayF: https://www.linkedin.com/posts/nilesh-chandekar-304b4b57_opensource-osa-openstack-activity-7365744037502345218-Nx3A should work | 15:05 |
| TheJulia | Yeah, I am not a release manager so there is no "trust" of me putting the patch in to release | 15:05 |
| JayF | We generally ask members of the community, core reviewers, contributors, operators, to take some time before the official release late this month/early October to help test. | 15:05 |
| JayF | #topic Working Group Updates | 15:05 |
| JayF | Do we have any updates for Standalone networking? | 15:06 |
| JayF | I assume it's probably been a little quiet with release activities? | 15:06 |
| TheJulia | Alan is out this week as well, he should be back next week | 15:06 |
| TheJulia | or, maybe he is back this week but just getting back today | 15:06 |
| TheJulia | so I wouldn't expect anything until sometime next week. | 15:06 |
| JayF | For eventlet removal team, https://review.opendev.org/c/openstack/ironic/+/959870 was posted last week and needs to land, both in master and be backported to the (eventual) 2025.2 | 15:06 |
| JayF | We thought we were done but surprise! | 15:06 |
| JayF | I suspect we'll find a few more surprises as time continues. | 15:07 |
| JayF | There are no discussion topics on the agenda. Skipping that section. New business can go in Open Discussion. | 15:07 |
| JayF | #topic Bug Deputy Updates | 15:08 |
| JayF | clif was bug deputy for the last 2 weeks, since the meeting was cancelled | 15:08 |
| JayF | clif: do you have anything of note for the meeting? | 15:08 |
| JayF | Also; who wants to be the next deputy? | 15:08 |
| clif | Pretty quiet two weeks. I think there was just one new ironic bug filed in that time. | 15:08 |
| cid | o/ | 15:08 |
| JayF | cid: is that a volunteer to be deputy next week? | 15:09 |
| clif | Perhaps two? I self-assigned this https://bugs.launchpad.net/ironic/+bug/2121702 | 15:09 |
| clif | Will take a look at it soon | 15:09 |
| JayF | #link https://bugs.launchpad.net/ironic/+bug/2121702 | 15:09 |
| JayF | > When triggering a automated cleaning the ports are not created in the cleaning network Edit | 15:09 |
| cid | JayF, I guess so. It also doubles as attendance :D | 15:09 |
| JayF | haha be careful when you raise your hand :D thank you for your volunteering :D | 15:10 |
| cid | :) | 15:10 |
| JayF | #note Next bug deputy: cid | 15:10 |
| JayF | No RFEs for review. | 15:10 |
| JayF | #topic Open Discussion | 15:10 |
| JayF | I have one thing for this | 15:10 |
| JayF | We have multiple repos in Ironic/Bare Metal program which are coming up on EOL. | 15:11 |
| JayF | Metalsmith. *inspector* | 15:11 |
| JayF | I will likely be taking action (starting with a formal ml discussion) around retiring these in October. | 15:11 |
| JayF | Please speak now if you have interest in keeping these alive | 15:11 |
| JayF | or if you have other repos under our governance which are hiding under a rock that need to be removed from our governance | 15:12 |
| TheJulia | ++ | 15:12 |
| kubajj | I am just excited to try using Ironic without inspector as a separate repo ;P | 15:12 |
| dtantsur | ++ | 15:13 |
| JayF | #agreed Metalsmith and Inspector-related repos being considered for retirement early in the *G* cycle. Please speak quickly if you have good reason for them to stay alive, and/or want to volunteer to maintain them. Note that inspection will still exist and has been migrated to primary Ironic repos. | 15:13 |
| JayF | Anything else for open discussion | 15:13 |
| JayF | One other interesting question: Anyone else going to be at Summit in Paris? | 15:14 |
| dtantsur | Yep | 15:14 |
| JayF | Enough for an Ironic dinner, perhaps? | 15:14 |
| TheJulia | I might be, awaiting approval. | 15:14 |
| JayF | I'll be there +2 folks from the downstream GR team | 15:14 |
| dtantsur | From us, at least rpittau, masghar and myself, I think, and I don't remember about janders and iurygregory | 15:15 |
| JayF | oh, nice! So yeah, we can pencil something in I hope. | 15:15 |
| dtantsur | Let's do it rather early than late if possible please since we're also trying to have a team dinner :) | 15:15 |
| JayF | I know little about the area and don't speak French so that may be a limitation, but I doubt we'll have trouble finding a nice place for a few folks to eat in *Paris* | 15:16 |
| dtantsur | it's not quite *in* Paris | 15:16 |
| JayF | Here is where someone volunteers to organize it who isn't an ocean away and only speaking english | 15:16 |
| dtantsur | we can wait for rpittauto return next week and make puppy eyes | 15:16 |
| TheJulia | dtantsur: that sounds like an excellent plan | 15:16 |
| masghar | ++ | 15:16 |
| JayF | #voluntold rpittau to help arrange team dinner (please?) | 15:17 |
| kubajj | JayF: I will be there | 15:17 |
| TheJulia | #prettypleaseultiplier | 15:17 |
| JayF | #agreed 10+ Ironic contributors/operators will be at France summit. We're going to try to do a team meal. | 15:17 |
| JayF | Anything else before I close the meeting? | 15:18 |
| dtantsur | yep, 1 sec | 15:18 |
| dtantsur | I want to highlight https://review.opendev.org/c/openstack/ironic/+/959690 | 15:18 |
| dtantsur | (I hope the delivery does not interrupt me right now lol) | 15:18 |
| dtantsur | This is an attempt to bring a more granular Metal3 job to Ironic | 15:18 |
| dtantsur | Together with a potential addition of a baremetal-operator functional tests job, this should eventually replace the cryptic metal3-integration job which nobody really knows how to debug :) | 15:19 |
| kubajj | I would also want to highlight https://review.opendev.org/c/openstack/ironic-python-agent/+/937342 as it might spark some discussion | 15:19 |
| JayF | Does this have external dependencies which will break at release barriers? | 15:19 |
| dtantsur | JayF: fewer. Neither depend on CAPI, which is a big source of these breakages. | 15:20 |
| dtantsur | Of course, we need to observe how the newly added jobs behave on OpenInfra nodes | 15:20 |
| JayF | ack; sounds good. My only real question is if experimental makes more sense; but it's a weak question as experimental jobs are easily forgotten :D | 15:20 |
| dtantsur | I want some statistics | 15:20 |
| * dtantsur picks up delivery, sorry | 15:20 | |
| JayF | We can pickup the discussion in gerrit | 15:21 |
| JayF | similarly kubajj I've added your patch to my review queue | 15:21 |
| kubajj | JayF: thanks, it is the one where you said you will not like to review 🫠| 15:22 |
| JayF | #note please review 959690 (ironic standalone operator testing) and 937342 (IPA raid-device-cleaning-skip fixes) | 15:22 |
| JayF | kubajj: I like patches sitting with no feedback for weeks much, much less :( | 15:22 |
| JayF | Last call (chapter 2) for anything new for open discussion | 15:24 |
| TheJulia | ptg? | 15:24 |
| JayF | Has anyone done serious planning for the PTG other than creating the etherpad? | 15:25 |
| dtantsur | We already have an etherpad? :D | 15:25 |
| JayF | #note Ironic already has a PTG etherpad: https://etherpad.opendev.org/p/ironic-ptg-2026.1 | 15:25 |
| JayF | it's been in the whiteboard for a couple of weeks | 15:25 |
| kubajj | (I had to look up when it is - October 27 to 31) | 15:27 |
| TheJulia | I've already added a few topics to the etherpad | 15:28 |
| TheJulia | Largely I'm getting tons of network/storage asks these days | 15:29 |
| TheJulia | Strong enough I'm tempted to just publish a prioritized list | 15:29 |
| JayF | that sounds really useful | 15:29 |
| JayF | "prioritized list" = operator menu, and I can see if my downstream wants a #3 (proverbially) | 15:30 |
| TheJulia | Well, where folks can collaborate around most value | 15:30 |
| JayF | Absolutely. Or maybe find a common thread that can unite some of the asks | 15:31 |
| TheJulia | like, my networking stuff is drastically different than yours, but... there is overlap and opportunity collaborate regardless | 15:31 |
| TheJulia | exactly | 15:31 |
| JayF | Sounds good. | 15:32 |
| JayF | A couple reminders along these lines: | 15:32 |
| JayF | we need to get on scheduling the ptg in the next week or two | 15:32 |
| JayF | we need to ensure release liasons get +1s on the pending release PRs that were linked at the top of the meeting | 15:32 |
| JayF | Most of the rest is stuff that can be coordinated in etherpad | 15:32 |
| TheJulia | ++ | 15:34 |
| JayF | I think that's it for the meeting? | 15:34 |
| JayF | Last call v3 :) | 15:34 |
| JayF | #endmeeting | 15:37 |
| opendevmeet | Meeting ended Mon Sep 8 15:37:17 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-09-08-15.02.html | 15:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-09-08-15.02.txt | 15:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-09-08-15.02.log.html | 15:37 |
| dtantsur | An example IrSO job run: https://zuul.opendev.org/t/openstack/build/b641d9b173f740cfb58a7bd0f2a8acc1/artifacts. And its report https://e0d088a46d3a62d38611-0fe503db5fc22ef43aac4f060a09645b.ssl.cf2.rackcdn.com/openstack/b641d9b173f740cfb58a7bd0f2a8acc1/controller/report.html | 15:47 |
| JayF | Does that operator do any network fun yet? | 16:04 |
| JayF | and/or keystone? | 16:05 |
| JayF | mainly wondering if it's 'standalone' (basic auth, no network) or 'standalone' (no nova) | 16:05 |
| kubajj | JayF: sadly not yet | 16:12 |
| JayF | kubajj: so from my reading of your change | 16:15 |
| JayF | kubajj: a sufficiently motivated root user on a machine going to cleaning can kill the logical name of a raid device and prevent cleaning | 16:15 |
| JayF | kubajj: but I can't quite wrap my mind around if this is a new security risk, or an existing one | 16:15 |
| * JayF puts it on gerrit patch with another piece of feedback | 16:16 | |
| kubajj | JayF: can they remove the logical name? | 16:16 |
| JayF | logical name is a piece of metadata in the software raid | 16:17 |
| JayF | right? | 16:17 |
| kubajj | yes | 16:17 |
| JayF | so a root user could remove that and cause a cleanfail | 16:17 |
| JayF | maybe that's OK, but that's what my brain is thinking | 16:17 |
| kubajj | I mean, technically they could, but I would advise against it :D | 16:18 |
| kubajj | We aim to deploy this downstream for our hypervisors | 16:19 |
| kubajj | JayF: maybe I am wrong, but I think it would not cause a clean fail, but rather just cleaning of the array | 16:21 |
| JayF | kubajj: https://review.opendev.org/c/openstack/ironic-python-agent/+/937342/6/ironic_python_agent/hardware.py#3563 ? | 17:14 |
| kubajj | JayF: yes, but we only raise that error if the skip list contains a volume name - i.e. the user is using RAID safeguards - which do not work now, without the change | 17:16 |
| JayF | so what I'm saying is | 17:16 |
| JayF | you have a node with skip list configured | 17:16 |
| JayF | I am a nefarious evil, evil admin who wants to use this as step 1 in a chain of vulns to access cleaning network | 17:16 |
| JayF | so I remove the configured logical name on my raid array and circumvent cleaning, so my payload keeps running for longer | 17:17 |
| JayF | I suspect a sufficiently determined root user can likely already cause cleanfails, but it's the sorta thing I look out for | 17:17 |
| kubajj | JayF: sorry, I was thinking about the other comment you left on the change | 17:17 |
| kubajj | my bad | 17:17 |
| JayF | oh, backwards compat, got it | 17:17 |
| JayF | that makes sense then | 17:17 |
| JayF | can you make sure to reflect that in a release note? | 17:17 |
| kubajj | will do | 17:18 |
| kubajj | for the first comment, as I said, I think it would not fail, because we check in cleaning the existing raids and try to match them with the property, since the name will not match, it will just clean the disks - which is the same as if the root user removed the property from Ironic | 17:19 |
| JayF | I haven't forgotten you, am looking still | 17:26 |
| * JayF is trying to get claude to help | 17:27 | |
| * JayF is going to make an IRC bot called "claude" which just responds occasionally with "That's right!" | 17:34 | |
| JayF | kubajj: I found a concrete issue; I'll put it on the gerrit change | 17:34 |
| JayF | kubajj: more or less verbatim quoted the claude stuff in the review after filtering out some invalid stuff, I think we found a couple of minor things we can cleanup | 17:38 |
| JayF | kubajj: if you push an update to this and I don't look organically in a couple of business days, feel free to ping me in here | 17:39 |
| TheJulia | So, just putting it out there, I forgot to update the release mappings. worst comes to worst I can do it and backport it tomorrow, but I have stuff I need to do today which is still a day off for me and tomorrow I have time during a meeting where I can do it. | 17:43 |
| JayF | So update ironic release mappings; update the releases PR for ironic to HEAD() once it's landed? | 17:45 |
| JayF | If that's what's needed I can do it today | 17:45 |
| TheJulia | or just let it land and backport the change | 17:47 |
| TheJulia | either, its all good | 17:47 |
| TheJulia | it just has to bed done early on post-release | 17:47 |
| TheJulia | (since your not going to pin to the release itself, you pin to latest or unset it. | 17:48 |
| TheJulia | ) | 17:48 |
| TheJulia | (or, your going to pin to an even older version) | 17:48 |
| JayF | ack I'm on it | 17:51 |
| TheJulia | much appreciated! | 17:52 |
| TheJulia | Okay, I'm going to go annoy some neighbors before we disappear for a few weeks | 17:52 |
| frickler | is anyone aware of ironic sdk jobs failing for almost a month? I'm assuming based on what lutimura wrote in the sdk channel that the issue is caused by changes in the sdk, but maybe some still might want to help fixing? :) https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack-ironic&project=openstack%2Fopenstacksdk&skip=0 | 18:09 |
| JayF | looks like allocations API schema issues | 18:10 |
| JayF | which was changed this cycle on our end | 18:10 |
| JayF | stephenfin and adamcarthur5 have been working on those changes | 18:10 |
| JayF | hopefully one of them can take a look if not I'll try to get around to it | 18:10 |
| lutimura | i think it's related to this one: https://review.opendev.org/c/openstack/ironic/+/945218 | 18:10 |
| JayF | agreed | 18:11 |
| frickler | yes, I was just wondering how the regression could come from the sdk itself when that ironic job is voting | 18:11 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/945218/3/ironic/api/schemas/v1/allocation.py#78 it should work if on the correct microversion | 18:11 |
| JayF | "should" being the most evil word in software | 18:12 |
| JayF | lol | 18:12 |
| lutimura | frickler: i'm pretty new to openstacksdk/ironic, so i wasn't really sure where this should be fixed (my bad!) | 18:12 |
| JayF | we're all figuring it out together lutimura :) | 18:12 |
| JayF | at least you have the excuse of being new :D | 18:12 |
| frickler | lutimura: no problem, it was a good step to find the issue and mention it already | 18:13 |
| lutimura | JayF: i'm not sure if that's what you suggested, but i tried to bump the microversion of these failing tests (specifically): https://review.opendev.org/c/openstack/openstacksdk/+/959332 -- unfortunately, it didn't work :( | 18:17 |
| JayF | Honestly this makes me wanna cross gate with these jobs | 18:19 |
| kubajj | JayF: thanks, will have a look tomorrow 😉 | 18:19 |
| JayF | because it seems like it's giving us bonus api coverage | 18:19 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Update release mappings for 32.0.x https://review.opendev.org/c/openstack/ironic/+/960173 | 18:21 |
| JayF | cid: cardoe: dtantsur: other cores ^ can I get a quick set of reviews on this? | 18:21 |
| cardoe | JayF: cid and I +2’d | 19:18 |
| JayF | I'll wait for V+1 before I self-approve, I don't love self-approving but for something like this it seems correct | 19:19 |
| JayF | Adam just closed the loop with me, he's going to be looking at the SDK CI failure tomorrow. | 19:53 |
| opendevreview | Merged openstack/ironic master: Update release mappings for 32.0.x https://review.opendev.org/c/openstack/ironic/+/960173 | 21:33 |
| opendevreview | Arnaud Morin proposed openstack/ironic master: Avoid the wsgi application to start with eventlet https://review.opendev.org/c/openstack/ironic/+/959870 | 22:31 |
| opendevreview | Arnaud Morin proposed openstack/ironic master: Deprecate usage of ironic.api.wsgi:initialize_wsgi_app https://review.opendev.org/c/openstack/ironic/+/960209 | 22:31 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!