| opendevreview | Merged openstack/ironic master: api: Add schema for portgroups API https://review.opendev.org/c/openstack/ironic/+/959881 | 09:02 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: api: Add schema for volume_target/_connector API https://review.opendev.org/c/openstack/ironic/+/993284 | 09:09 |
| sean-k-mooney | clif: https://review.opendev.org/q/topic:%22startup-time%22 i rebased/updated the startup time patches into one seriese if you want to check them out again | 09:45 |
| sean-k-mooney | that seriese also neeed https://github.com/openstack/nova/commit/2714c71d05e8501ddb3a8fc0c36f3406ef974072 which is already merged | 09:48 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 09:51 |
| opendevreview | Merged openstack/python-ironicclient master: Add portgroup uuid to vif attach OSC client https://review.opendev.org/c/openstack/python-ironicclient/+/990409 | 10:34 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 12:32 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 12:59 |
| TheJulia | good morning | 13:03 |
| opendevreview | Konstantinos Karampogias proposed openstack/ironic master: Add vendor/model/serial_number to firmware component endpoint https://review.opendev.org/c/openstack/ironic/+/992034 | 13:06 |
| opendevreview | Nuutti Hakala proposed openstack/ironic master: Limit LLDP TLV value length and interfaces https://review.opendev.org/c/openstack/ironic/+/990452 | 13:30 |
| dtantsur | morning TheJulia | 13:36 |
| iurygregory | good morning ironic | 13:39 |
| TheJulia | https://github.com/openstack/ironic/pull/14 <-- I thought there was a bot which handled people opening PRs? | 13:43 |
| iurygregory_ | I also thought there was a bot that handle that, I do remember seeing something like this... | 13:56 |
| cardoe | There was and I think someone mentioned it was busted. | 14:01 |
| JayF | Fyi, I will not be at the meeting today. | 14:01 |
| TheJulia | k | 14:02 |
| TheJulia | cardoe: figures | 14:03 |
| cardoe | A Racker opened one recently and it didn’t work right. | 14:06 |
| cardoe | So I kinda let fable go crazy over the weekend and eat all my usage on ironic. | 14:06 |
| TheJulia | oh. my. | 14:07 |
| cardoe | So most of it wasn't code. It was analysis which I believe is what Fable is suppose to be better at. | 14:09 |
| cardoe | Well not all my Fable usage. My son has been wanting to learn programming. And we had come up with an idea of writing an iOS game that he could put in the app store for free and send his family a link to it to show off what he did. Given how the world is changing with the LLMs I decided to focus more on the programming concepts and behaviors and let him use the LLM for the actual code but told it to write lots of comments. | 14:11 |
| cardoe | He saw a pop up that Fable is the best model so lemme use that... | 14:11 |
| cardoe | Wish I had a CLI to launchpad cause I'd create a bunch of issues as reminders for the stuff. | 14:20 |
| JayF | the API is trivial, but I've only used it unauthenticated/ro | 14:23 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: add ironic-manage to CLI references doc section https://review.opendev.org/c/openstack/ironic/+/996104 | 14:26 |
| cardoe | JayF: if I'd like https://review.opendev.org/c/openstack/ironic/+/989431 backported, do I need to tell the creator to make a release note? Or should I just push one for them? | 14:27 |
| TheJulia | Its not that complicated to auth against based upon some of the examples I've seen over the years. | 14:28 |
| JayF | cardoe: yes | 14:28 |
| JayF | cardoe: that change needs a release note anyway | 14:29 |
| * TheJulia is all for doing release notes for folks | 14:29 | |
| JayF | cardoe: and it's questionable if that behavior change can be backported | 14:29 |
| TheJulia | I think its a bugfix, fwiw | 14:29 |
| TheJulia | Just, one which is semi-rare | 14:30 |
| JayF | yeah, I'm not saying it can't be, I'm saying it's a clear behavioral change | 14:30 |
| TheJulia | more so because most ironic operators are using elevated credentials | 14:30 |
| TheJulia | ++ | 14:30 |
| JayF | so it's a matter of how it's framed/what the failure/success cases are | 14:30 |
| cardoe | So been playing with having some roles with less access or targeted access | 14:30 |
| JayF | if this is relevant context for 989431 make sure it also gets into the commit/release note/bug/etc :D | 14:31 |
| cardoe | Well I'm more than happy to rewrite the commit message and the release note. | 14:31 |
| cardoe | Cause the issue is a bit gross | 14:31 |
| TheJulia | cardoe: I'd say go right ahead | 14:32 |
| cardoe | If someone does a manual service or cleaning with steps, the network the device gets put on has ports that are created in the user's context which fails if they don't have access to the service/cleaning networks. | 14:32 |
| cardoe | and it kinda throws some lego pieces all around | 14:32 |
| TheJulia | Its admin set networks, so.. yeah | 14:33 |
| TheJulia | or at least, *should* be admin-set networks | 14:33 |
| JayF | yeah that should 100% be detailed in the release note then backported | 14:33 |
| frickler | github bot is broken, yes. it leaks credentials, thus it is effectively not running currently | 14:33 |
| cardoe | yeah that's why I'm asking if I can write one on behalf | 14:33 |
| TheJulia | frickler:fun! | 14:33 |
| TheJulia | cardoe: yes, we view that as acceptable behavior | 14:34 |
| TheJulia | cardoe: ideally, you sign-off on the commit as well | 14:34 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 14:42 |
| TheJulia | so who is running this week's meeting ? | 14:50 |
| clif | sean-k-mooney: ty, I'm going to try to get them whipped into shape very soon, at least the ones we've confirmed downstream | 14:50 |
| cardoe | So for once I don't have a work meeting over the top. | 14:51 |
| clif | cardoe: why is it a limitation of argparse and not oslo.config itself? | 14:51 |
| cardoe | So I could do it but I honestly don't know how. | 14:51 |
| cardoe | clif: I think it was to do with old Python limitation and just hasn't been updated for newer Python | 14:52 |
| clif | aha, well good thing we live in the future | 14:52 |
| clif | I'm going to spend some time in-between finishing the kernel params patches and nova startup perf to add type coercion checking to oslo.config | 14:53 |
| cardoe | yeah like I wanted to toss a ton of code in pbr since the behavior was in stdlib from Python 3.8+ and pbr's behavior was slightly different. And I was told they tried to maintain compat with Python 2.6 | 14:53 |
| cardoe | That should not be the case with oslo.config | 14:53 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 14:59 |
| * TheJulia blinks | 15:00 | |
| cardoe | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Jul 6 15:00:53 2026 UTC and is due to finish in 60 minutes. The chair is cardoe. 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 | cardoe: you beat me to it! | 15:01 |
| clif | o/ | 15:01 |
| TheJulia | o/ | 15:01 |
| rpittau | o/ | 15:01 |
| dtantsur | o/ | 15:01 |
| cardoe | TheJulia: consider it my gift to you since you've been on vacation. And I've needed to do it. | 15:01 |
| Mahnoor | o/ | 15:01 |
| * TheJulia hands cardoe a fearless leader shirt | 15:01 | |
| cardoe | I believe that's enough for a quorum. | 15:01 |
| cardoe | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:02 |
| cardoe | #topic Announcements / Reminders | 15:02 |
| cid | o/ | 15:02 |
| cardoe | Please review outstanding changes in the ironic-week-prio topic. | 15:03 |
| cardoe | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| cardoe | Does anyone have any other announcements? | 15:03 |
| cardoe | Alright moving on. | 15:03 |
| cardoe | #topic Working Group Updates (Async IO) | 15:04 |
| cardoe | #link https://etherpad.opendev.org/p/ironic-asyncio | 15:04 |
| dtantsur | I think iurygregory_ is just back from PTO, so likely no updates | 15:04 |
| cardoe | Fair. I saw some patches sitting in the queue so I think he'd appreciate some reviews. | 15:05 |
| cardoe | #topic Working Group Updates (Security Coresec team) | 15:05 |
| cardoe | Any security team updates? | 15:05 |
| dtantsur | Imagine a very load scream :) | 15:05 |
| dtantsur | loud | 15:05 |
| TheJulia | Endless, right ? | 15:05 |
| * dtantsur slowly nods | 15:06 | |
| cardoe | So you want me to bury my Fable analysis? | 15:06 |
| TheJulia | So, I don't think there is much of an update this week, but please keep an eye out this week. | 15:06 |
| dtantsur | Queue up, we already have a backlog to work with :) | 15:06 |
| TheJulia | heh | 15:06 |
| cardoe | Alright. Then we'll move on. | 15:07 |
| cardoe | #topic Bridging the Gap Gazpacho Cycle Survey and Metrics Analysis (fungi) | 15:07 |
| TheJulia | Was that the week before last? | 15:07 |
| cardoe | Possibly, I was out then and I thought I saw it when I reviewed. | 15:08 |
| cardoe | The meeting logs | 15:08 |
| TheJulia | or weeks before that? I just don't remember and it feels like a repeat since I've already had someone express displeasure over how metrics are being modeled/handled/framed | 15:08 |
| cardoe | okay. We can move on. | 15:08 |
| cardoe | #topic Bug Deputy Updates | 15:08 |
| cardoe | cid: what ya got for us? | 15:08 |
| cid | Just two bugs this week, 3 the last | 15:09 |
| cid | I triaged one as invalid | 15:09 |
| cardoe | Our software is 33% better in just 1 week! | 15:09 |
| TheJulia | lol | 15:10 |
| cid | :D | 15:10 |
| cid | https://bugs.launchpad.net/ironic/+bug/2158578 : Graphical console periodic task should detect stopped containers | 15:10 |
| cid | https://bugs.launchpad.net/ironic/+bug/2157002 : BIOS setting update never completes with some vendors | 15:10 |
| cid | https://bugs.launchpad.net/ironic/+bug/2157027 : ipxe bootfile_name_by_arch causes boot_mode check to be skipped | 15:10 |
| cid | https://bugs.launchpad.net/ironic/+bug/2158986 : | 15:10 |
| cid | https://bugs.launchpad.net/ironic/+bug/2158976 : | 15:10 |
| cardoe | So 2158986 is mine. | 15:10 |
| cardoe | I cannot decide if its a documentation issue or something we should change. | 15:11 |
| cardoe | The bug was looking for opinions. | 15:11 |
| cardoe | Essentially if you set something like "default_boot_interface = ipxe" then EVERY hardware type you put in your config MUST report that it supports "ipxe" otherwise the conductor will not start | 15:11 |
| TheJulia | That feels like a lot of issues | 15:12 |
| cardoe | The documentation states that every hardware type must support at least ONE enabled_*_interface to load | 15:12 |
| TheJulia | I think 578 makes sense to just approve as an rfe enhancement | 15:12 |
| cardoe | But default_*_interface says when there's multiple interfaces for the hardware type, this is the one that will be used. | 15:12 |
| cardoe | But if my hardware type only supports 1 interface, default shouldn't matter from the reading of the docs. | 15:13 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 15:13 |
| TheJulia | 002 seems like a legit bug | 15:13 |
| cardoe | Agreed on 578 needing to be done as an enhancement | 15:14 |
| cardoe | 002 is a legit bug. There's a patch in ironic-week-prio waiting for review. | 15:14 |
| TheJulia | looks like 027 is already fixed in master branch | 15:14 |
| TheJulia | cardoe: I didn't see that one linked | 15:15 |
| TheJulia | if you could make sure its linked, that would be awesome | 15:15 |
| cardoe | Yeah the author didn't link it. | 15:15 |
| TheJulia | for 986... you've got a solid point | 15:16 |
| TheJulia | I'm not sure the right path to "fix" it is more configuration | 15:16 |
| TheJulia | since its largely an operational disconnect due to docs | 15:16 |
| TheJulia | but we've seen folks lean hard towards the default values in the past and get into similar siutations | 15:16 |
| TheJulia | situations | 15:16 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996110 | 15:16 |
| TheJulia | I guess I'm not 100% sure docs would fix that, but maybe we should also be actively discouraging the use of driver defaults?! | 15:17 |
| opendevreview | Doug Goldstein proposed openstack/ironic bugfix/37.0: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996111 | 15:17 |
| cardoe | Well that was my other thought... we deprecate the idea of defaults. | 15:17 |
| cardoe | And if ya want something else, you supply it when you create the node or set it after the fact. | 15:18 |
| TheJulia | Yeah, I'm on the fence on that idea | 15:18 |
| TheJulia | but I think maybe I might need more or less coffee to reasonably think about that | 15:18 |
| TheJulia | defaults make ironic easier too, but they lead to pitfalls exactly like this | 15:18 |
| TheJulia | I feel like "more" with "less work" is the documentation/help text revision | 15:19 |
| TheJulia | I feel like new options are just a lot of more risk because then people totally need to have the mental model in their heads | 15:19 |
| TheJulia | (and we do, but users, not so much) | 15:19 |
| cardoe | It came up because a number of deployment docs or tooling use "default_deploy_interface = direct" | 15:20 |
| cardoe | There was confusion about the priority that an interface is selected when I looked at kolla docs for example. | 15:21 |
| cardoe | if you have enable_deploy_interface = ramdisk,direct which one will be used? | 15:21 |
| TheJulia | yeah, its always priority order of availability from first to last | 15:22 |
| cardoe | The has supported_deploy_interfaces = [direct, ramdisk] | 15:22 |
| TheJulia | so, if they remove the default, they will end up with ramdisk | 15:22 |
| TheJulia | This is the challenge, it was all well documented in the design docs, but that doesn't necessarilly translate to operational how | 15:22 |
| TheJulia | and the convience of settings take over in that transition | 15:22 |
| cardoe | Yep. | 15:23 |
| cardoe | Well I'll take this one on with some documentation improvements. | 15:23 |
| cardoe | Because ultimately what folks can achieve with this is enable_deploy_interfaces = direct,ramdisk | 15:23 |
| TheJulia | Yeah, I think that is likely the cleanest path, and most likely for folks to notice if we also issue some sort of reminder in release notes | 15:23 |
| cardoe | and they don't need default_deploy_interface | 15:23 |
| TheJulia | yup | 15:23 |
| TheJulia | but they need to understand, in as few words as possible (i.e. not llm output), how that configuration translates | 15:24 |
| cardoe | Yeah. I'll human write it not LLM write it. | 15:24 |
| TheJulia | :) | 15:24 |
| cid | #link https://bugs.launchpad.net/ironic/+bug/2158976 <- any feedback on this one | 15:24 |
| TheJulia | yeah, I'm pondering that one | 15:25 |
| TheJulia | We're in the union of security hardening and side effects, combined with they want private images | 15:25 |
| TheJulia | when Ironic was largely geared around public cloud/operator approved images | 15:25 |
| TheJulia | All *do-able* but yeah.... | 15:25 |
| TheJulia | I feel like they make a valid point about access but that would need to be a whole new feature to dig into the access and translate that across | 15:26 |
| TheJulia | I think their use case is a little off from the intent which is why they are having the experience they are as well | 15:26 |
| TheJulia | and yeah | 15:26 |
| opendevreview | Merged openstack/ironic-specs master: Move standalone-networking to done in 35.0 https://review.opendev.org/c/openstack/ironic-specs/+/995944 | 15:26 |
| opendevreview | Merged openstack/ironic-specs master: Fix the horribly outdated release index https://review.opendev.org/c/openstack/ironic-specs/+/995947 | 15:26 |
| cid | Right! The reporter has a point even though I don't think the improvement on that code path was literally to share all images with the conductor | 15:26 |
| cid | But that all depends on what their workflow is | 15:27 |
| cardoe | Yeah the reporter has a valid point | 15:27 |
| TheJulia | absolutely | 15:27 |
| TheJulia | and yeah, the only other option then is "hey, operator, what is ironic's project ID" | 15:27 |
| TheJulia | and depending on how they ran their configuration/setup, that could be problematic | 15:27 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996113 | 15:27 |
| TheJulia | because it is all rooted in the credentials and the project for the account used to configure it | 15:27 |
| cid | ++ | 15:28 |
| cardoe | We almost need an image model option | 15:28 |
| cardoe | "public" model and "private" model... Bad names | 15:28 |
| TheJulia | yeah | 15:28 |
| TheJulia | I think we're going to need to ponder this one a little bit, in all honesty | 15:29 |
| cardoe | Yeah I'll make a point to read it and the code and come back to it. | 15:29 |
| cid | \o | 15:29 |
| cid | allow_image_access_via_auth_token is still there fwiw. For operators that will make that explicit config, Maybe we keep it beyond the H cycle. | 15:29 |
| TheJulia | Yeah, I'm also the last person to dig into that area AFAIK | 15:29 |
| TheJulia | yeah | 15:30 |
| TheJulia | that might be the path, in all actuality | 15:30 |
| TheJulia | It doesn't "feel right" | 15:30 |
| TheJulia | but lets think/ponder, and circle back | 15:30 |
| TheJulia | who wants to take ownership of comms on this one? | 15:30 |
| cardoe | Sounds good. | 15:30 |
| cid | +++ Noted. | 15:30 |
| cid | For the time being, I will just lift the invalid tag | 15:31 |
| cid | on the bug for now | 15:31 |
| cardoe | I'll set a reminder to ping the 3 of us and we can decide from there. | 15:31 |
| cardoe | Who will do comms and such. | 15:31 |
| cardoe | Who wants to be our next bug deputy? | 15:31 |
| Mahnoor | I can do it this week | 15:32 |
| cardoe | sounds good. | 15:32 |
| cardoe | Moving on.. No RFEs in the list. | 15:32 |
| cardoe | There are some in the ironic-week-prio | 15:32 |
| cardoe | #topic SRBAC enforce_scope removal: always enforce scope in RBAC | 15:33 |
| cardoe | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/M72AY5ABQFXQ7XHLVEGHLBBK4XFQGVFK/ | 15:33 |
| cardoe | #link https://review.opendev.org/c/openstack/oslo.policy/+/986475 | 15:33 |
| cardoe | It's gmaan's topic but he said he probably wouldn't be able to attend | 15:34 |
| cardoe | Here are his notes | 15:34 |
| cardoe | As ironic support system scope policy, I would like the team to check the impact of enforce_scope removal, and if it is ok to make scope enforcement always enabled and not configurable. | 15:34 |
| cid | Some of those were carried over from the previous week's agenda. | 15:34 |
| cardoe | If the ironic team thinks it should remain configurable, we can add a flag (something like enforce_ironic_scope, if it does not exist) on the ironic side to control it instead of oslo.policy? | 15:34 |
| cardoe | ah well did we have resolution? | 15:34 |
| cid | Nope. The meeting was deferred to today | 15:35 |
| TheJulia | so, I think the right thing to do is just to remove the option and not worry about configurability, but I don't have operator insight if they have been disabling the option | 15:36 |
| cardoe | Yeah | 15:36 |
| cardoe | I unfortunately think what Ironic does is correct and the other projects are wrong. | 15:36 |
| TheJulia | downside, that is going to get interpretted as breaking for folks who *have* been disabling it, but they were warned a very very long time ago | 15:36 |
| TheJulia | yeah | 15:36 |
| cardoe | e.g. global scoped neutron resources should be controlled by system-scope access | 15:37 |
| TheJulia | Yeah, we rolled forward on original scoped design | 15:37 |
| cardoe | Not by any project scoped admin | 15:37 |
| cardoe | I also think it's just removed and don't worry about configurability. | 15:38 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996115 | 15:39 |
| cardoe | Silence means move on? | 15:39 |
| cardoe | #topic Open Discussion | 15:39 |
| TheJulia | Then I think that is the right path to proceed along unless someones screams soon | 15:39 |
| cardoe | What do folks have for open discussion? | 15:39 |
| TheJulia | vacation was sort of relaxing | 15:39 |
| TheJulia | Although, I chose a hotel not far from the public viewing screens for the world cup | 15:39 |
| TheJulia | translate that to: didn't sleep the best. :( | 15:40 |
| cardoe | Well that's a bummer. Hopefully back to your own bed is a plus. | 15:40 |
| cid | :D. Welcome back | 15:40 |
| TheJulia | Downside, lots of fireworks due to a particular holiday | 15:41 |
| TheJulia | :) | 15:41 |
| cardoe | I've written a basic "ironic-manage" CLI which takes subcommands | 15:42 |
| cardoe | I was hoping for folks to express how they felt. | 15:42 |
| cardoe | We've been gathering some checkers like TBN and inspection_rules and I figured they could be as subcommands of that. | 15:43 |
| TheJulia | I think that is a great idea | 15:43 |
| cardoe | The first subcommand is around hardware types (drivers) and their interfaces... showing what's available | 15:43 |
| cardoe | I've been writing a blog post for the ironic site on the side for adding your own hardware type | 15:44 |
| TheJulia | cool cool | 15:44 |
| TheJulia | so going back to that image access issue reported, I'm thinking it might be feasible to allow access to be granted if the image is *owned* by the requesting project | 15:45 |
| cardoe | The mental model for me / us has been every piece of hardware we're managing in one of our racks for provisioning that's hooked to the switch will also be a node in Ironic | 15:45 |
| cardoe | So we've been extending the usage of "fake-hardware" with some other hardware types | 15:45 |
| TheJulia | The downside is we're going to have to pass the requestor's project_id *WAY* down into show and download | 15:45 |
| TheJulia | and do that consistently, unless it is on *context* somehow, but I don't think it gets preserved there | 15:46 |
| cardoe | This has led to me writing a bunch of typing on a number of our classes and interfaces. | 15:46 |
| TheJulia | since that is conductor context most likely | 15:46 |
| cardoe | I've put up the low hanging typing into ironic-week-prio | 15:46 |
| TheJulia | Interesting, would you be up for writing a blog post? | 15:46 |
| cardoe | There's a couple of places where things disagree and that'll come as later changes. | 15:46 |
| TheJulia | on that specifically "creative uses of ironic" ? | 15:46 |
| cardoe | Yeah that's what my thought was. | 15:47 |
| cardoe | We can pass project_id way down. We just need to validate all that. | 15:47 |
| cardoe | Anyway, that's the end of my open topic. | 15:48 |
| cardoe | Anyone have anything more? | 15:48 |
| cardoe | Going once? | 15:48 |
| cardoe | Twice? Three times? | 15:49 |
| cardoe | #endmeeting | 15:50 |
| opendevmeet | Meeting ended Mon Jul 6 15:50:01 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:50 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-07-06-15.00.html | 15:50 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-07-06-15.00.txt | 15:50 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-07-06-15.00.log.html | 15:50 |
| JayF | cardoe: TheJulia: (reading backlog): I wonder if default_*_interface needs to be a mapping with keys of "hardware type" and values of "interface" | 15:51 |
| JayF | the issue is not that we have a default; it's that we express a single default when in reality it's a "per driver" thing in most cases | 15:51 |
| cardoe | Yeah that was one of my suggested approaches in the bug as well. | 15:53 |
| TheJulia | Sort of similar to cardoe's "other" idea :) | 15:53 |
| TheJulia | I guess I'm gunshy of extending the configuration forward because I've seen people stub their toes on default interfaces before | 15:53 |
| TheJulia | because there is a wall mounted footgun trigger right there | 15:53 |
| TheJulia | At least, that is how it *feels* for me | 15:54 |
| TheJulia | cardoe: fyi, I asked some clarifying questions on https://bugs.launchpad.net/ironic/+bug/2158976 | 15:54 |
| iurygregory_ | sorry, missed the meeting because I went for lunch with folks from the Office, but yeah no updates, just waiting for reviews in the spec | 16:08 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2025.1: [stable only] Replace pkg_resources with importlib.resources https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/994768 | 18:32 |
| opendevreview | Merged openstack/ironic stable/2025.1: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996115 | 18:35 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996110 | 18:44 |
| cardoe | last one in the ^ https://review.opendev.org/c/openstack/ironic/+/996111 if someone wants to give it a nudge | 18:49 |
| TheJulia | k | 18:50 |
| opendevreview | Merged openstack/ironic master: add types to driver interfaces base class for steps and task https://review.opendev.org/c/openstack/ironic/+/995571 | 18:53 |
| opendevreview | Merged openstack/ironic stable/2025.2: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/996113 | 18:54 |
| opendevreview | Merged openstack/ironic master: Add ironic-manage command for driver discoverability https://review.opendev.org/c/openstack/ironic/+/995847 | 20:40 |
| JayF | Just giving y'all a heads up: in two weeks, the week from July 20-28 (Monday through Monday) I'll be offline and not working on OpenStack. If you need Ironic things, just say something loudly in here and I'm sure someone will help :D | 20:49 |
| TheJulia | sounds like a plan | 20:51 |
| opendevreview | Merged openstack/ironic master: annotate the base driver/interface methods as stubs https://review.opendev.org/c/openstack/ironic/+/995572 | 20:52 |
| opendevreview | Merged openstack/networking-generic-switch master: Add NetconfSwitch base class for NETCONF transport https://review.opendev.org/c/openstack/networking-generic-switch/+/990060 | 22:23 |
| opendevreview | Merged openstack/ironic master: Truncate node history event to avoid DB column overflow https://review.opendev.org/c/openstack/ironic/+/981399 | 22:43 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!