| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 01:11 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 02:39 |
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 03:04 |
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 03:59 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 04:40 |
| *** dtantsur_ is now known as dtantsur | 05:54 | |
| rpittau | good morning ironic! o/ | 06:20 |
| queensly[m] | Good morning! | 07:01 |
| rpittau | alegacy_: hey was this discussed with the community https://review.opendev.org/c/openstack/requirements/+/962691 ? | 08:30 |
| rpittau | asking because I'm not sure is a good idea putting ironic in the upper-constraints | 08:30 |
| dtantsur | my first reaction was "WAIT WUT", but we also have Neutron there, so I guess there is precedent.. | 08:53 |
| opendevreview | nidhi proposed openstack/sushy master: Add comprehensive PCIe function support to Sushy https://review.opendev.org/c/openstack/sushy/+/962680 | 08:58 |
| opendevreview | Merged openstack/networking-baremetal stable/2025.2: Fix min version of Neutron https://review.opendev.org/c/openstack/networking-baremetal/+/961858 | 10:35 |
| alegacy_ | rpittau: I raised this a while back when I introduced this (https://review.opendev.org/c/openstack/networking-generic-switch/+/962037). It needs to import Ironic to access a new base class for the new driver interface. | 11:29 |
| alegacy_ | would have been an ideal use case for ironic-lib, but... | 11:29 |
| dtantsur | Maybe we need to revisit ironic-lib.. | 11:40 |
| dtantsur | I have one idea though, I'll try to formulate it after the current meeting | 11:41 |
| jrosser | in openstack-ansible we have to edit upper-constraints on-the-fly in order to remove services that have been added, horizon is another example | 11:41 |
| jrosser | pip blows up with conflicting installation of horizon and horizon==some-specific-version in that case | 11:42 |
| dtantsur | yeah, bifrost will be affected too most likely | 11:42 |
| rpittau | jrosser: we do the same in metal3, I don't like it :) | 11:42 |
| jrosser | yeah, agreed i don't like it also, and the error from pip is pretty confusing if you don't know whats going on | 11:43 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add support for multiple JSON-RPC configuration groups https://review.opendev.org/c/openstack/ironic/+/960593 | 12:14 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add force_dhcp option for managed inspection https://review.opendev.org/c/openstack/ironic/+/960824 | 12:14 |
| opendevreview | Allain Legacy proposed openstack/ironic master: WIP: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/962493 | 12:14 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for OVS as a virtual switch https://review.opendev.org/c/openstack/bifrost/+/962038 | 12:15 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 12:16 |
| TheJulia | good morning | 13:04 |
| dtantsur | alegacy_: I haven't finalized what I was thinking about, but how terrible would be to keep a copy of SwitchDriverBase in n-g-s? | 13:31 |
| dtantsur | initially, I thought we would have two separate services talking via JSON RPC.. but I guess we're moving away from this model? | 13:33 |
| dtantsur | Or should n-g-s (instead of Ironic) become the core of this service (with a potential to split it away entirely)? | 13:33 |
| TheJulia | central driver code and exceptions are sort of the issue | 13:35 |
| dtantsur | I guess I had a different mental model of all this | 13:35 |
| TheJulia | I sort of did as well | 13:35 |
| dtantsur | I thought that everything Ironic-related stays in Ironic (its network interface) | 13:35 |
| dtantsur | and n-g-s or $something turns into a JSON RPC service that knows nothing about nodes and ports but operates on a lower level | 13:36 |
| TheJulia | $something ended up in ironic, and n-g-s part is pluggable | 13:36 |
| TheJulia | but then another part is in ngs | 13:36 |
| TheJulia | because it exposes it as a driver | 13:36 |
| dtantsur | so, not JSON RPC, just ironic+ngs? | 13:36 |
| TheJulia | no, with json rpc | 13:37 |
| dtantsur | wut | 13:37 |
| dtantsur | the main point of JSON RPC was to avoid problem we're discussing now: coupling n-g-s and ironic in code | 13:37 |
| dtantsur | if we don't do that, why have an RPC? | 13:37 |
| TheJulia | ironic network interface -> jsonrpc -> ironic-networking service -> ironic-networking-service drivers -> ngs-driver -> switch driver -> switch | 13:37 |
| dtantsur | why not: ironic network interface -> ironic-networking-service drivers -> ngs-driver -> switch driver -> switch | 13:38 |
| TheJulia | because there seems to be a desire to make it pluggable to use a different plugin for the switch config | 13:38 |
| TheJulia | (if anyone says networking-ansible, I'm going to just go in to the corner and cry) | 13:38 |
| dtantsur | we can do it with entrypoints as it's already done in alegacy_'s patches | 13:38 |
| TheJulia | dtantsur: parent driver class is in ironic | 13:39 |
| TheJulia | along with the exceptions he wants to raise | 13:39 |
| dtantsur | I guess I had the same picture as you in mind, but I assumed that ironic-networking service is disjointed from ironic and its concepts | 13:39 |
| TheJulia | it is, minus the desire to have type and exact same class matching | 13:39 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 13:40 |
| dtantsur | oh, I understand what is odd to me. I thought that JSON RPC will take the role of the plug-in point. | 13:40 |
| dtantsur | essentially, ironic network interface -> jsonrpc -> ngs-service -> switch driver -> switch | 13:40 |
| dtantsur | or, ironic network interface -> jsonrpc -> something else -> switch driver -> switch | 13:41 |
| TheJulia | Yeah, I was always thinking, no matter how we end up doing it, we didn't try for eact matching and just figured it what was going on via jsonrpc, not explicit exceptions. Granted, there are huge downsides to that crazy idea. | 13:45 |
| dtantsur | Maybe, for the sake of keeping alegacy_'s sanity, we table the whole JSON RPC idea for now and just make it all direct Python calls... | 13:46 |
| dtantsur | I'd still probably just copy over the base class (and avoid using isinstance in Ironic) | 13:46 |
| dtantsur | maintaing Ironic version compatibility may be fun | 13:46 |
| opendevreview | nidhi proposed openstack/ironic master: Add PCIe function fields to redfish inspection https://review.opendev.org/c/openstack/ironic/+/963179 | 13:47 |
| TheJulia | dtantsur: that doesn't make any of it any better though | 13:49 |
| TheJulia | the issue is the explicit pluggable driver interface being added below | 13:49 |
| dtantsur | it reduces the operational complexity quite a bit | 13:49 |
| dtantsur | I mean, we're not getting ironic/n-g-s separation, so why complicate everything with an RPC bus between ironic and ironic? | 13:49 |
| TheJulia | It eliminates the ability to build trust, but that is not his product concern | 13:50 |
| alegacy_ | i thought the point of the new service was to gain the ability to give that to another department and let them run and manage it. | 13:50 |
| TheJulia | I feel like, at this point, that is change for change's sake and not addressing the root challenge | 13:50 |
| TheJulia | alegacy_: yeah, exactly, allow for a barrier to exist | 13:51 |
| dtantsur | that's a pure theory | 13:51 |
| dtantsur | and if we want the barrier, we cannot be coupling the code so tightly | 13:51 |
| alegacy_ | IMO, that new service does need to have the ability to pull in drivers as a pluggable interface and that's where that base class came from. i.e., not just tightly coupled to ngs. | 13:52 |
| TheJulia | Its operational reality based upon numerous operators providing feedback | 13:52 |
| TheJulia | That is where the extra complexity seems rooted, fwiw | 13:53 |
| TheJulia | at least, the complexity which we're finding challenging and going back and forth on | 13:54 |
| dtantsur | Well, I guess Metal3 is a bit specific case in this sense | 13:54 |
| dtantsur | Anyway | 13:54 |
| dtantsur | Just let's really not import Ironic :) | 13:54 |
| * TheJulia sighs | 13:55 | |
| dtantsur | Similarly, importing Ironic is not a problem for Metal3 (we have one image anyway, and everything is run together), but it seems to be a problem for other folks | 13:57 |
| alegacy_ | ironic-lib would have been perfect. alternatively, i could look into moving the new entry point from ngs back into ironic and have it import ngs. | 13:57 |
| alegacy_ | It would still be a pluggable driver, but unfortunately it would never be possible to have a driver live purely outside of Ironic. | 13:57 |
| alegacy_ | IMO, the current way all of the drivers inside of Ironic works is a bit limiting in that way... not possible to externalize them without Ironic. | 13:58 |
| TheJulia | who is saying its a problem? | 13:58 |
| dtantsur | yeah, it's also not a clean solution: then we need to add n-g-s to requirements | 13:58 |
| alegacy_ | exactly | 13:58 |
| alegacy_ | not saying its a problem perse, but it is definitely a limitation... anyone wanting to implement their own driver needs to get it into the Ironic repo. | 13:59 |
| TheJulia | well, in the code I looked at last week, the base is in the ironic repo | 14:00 |
| dtantsur | TheJulia: jrosser for openstack-ansible | 14:00 |
| dtantsur | everyone will have to hack upper-constraints to be able to install ironic | 14:00 |
| TheJulia | oh, ugh | 14:00 |
| TheJulia | yeah, if they are trying to enforce it, yeah | 14:00 |
| TheJulia | that is a good point | 14:01 |
| TheJulia | Anyway, lets stop trying to push the problem around the plate | 14:01 |
| TheJulia | what is the cleanest path? | 14:01 |
| TheJulia | a small library? | 14:01 |
| dtantsur | again, why cannot we copy the base class? | 14:02 |
| TheJulia | ... I'm fine with that | 14:02 |
| * dtantsur is in a meeting and might have missed some answers | 14:02 | |
| opendevreview | nidhi proposed openstack/sushy master: Add comprehensive PCIe function support to Sushy https://review.opendev.org/c/openstack/sushy/+/962680 | 14:02 |
| TheJulia | I floated that as well, but... yeah | 14:02 |
| jrosser | example for horizon mess is here https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/770283 | 14:02 |
| JayF | JFYI I'm on a sick day today | 14:10 |
| dtantsur | JayF: get better :( | 14:11 |
| kubajj | dtantsur: or any IPA pro, is there any simple way of passing information between cleaning steps? (other than global variables - we are trying to avoid this) | 14:20 |
| TheJulia | kubajj: trying to save what the root device should be according to a prior step for a later step to pickup? | 14:21 |
| kubajj | TheJulia: yes, exactly | 14:22 |
| TheJulia | uhhhh nothing in the steps themselves, but you can reset/save it if memory serves | 14:23 |
| * TheJulia looks | 14:23 | |
| TheJulia | https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L1840-L1902 | 14:25 |
| TheJulia | That is the block of code you want to influence, you, could.. in theory modify the cached node, but that is overwritable. I think the right thing would be to create a new flag of "this is it" which could just return quickly if set. | 14:25 |
| TheJulia | and a prior step could save that value on the hardware manager. | 14:26 |
| TheJulia | Seems like that ?might? work | 14:26 |
| kubajj | TheJulia: yes, so we will just move the global variable to the HW manager, sounds like a plan, thanks | 14:27 |
| TheJulia | kubajj: the cached node is a similar concept, fwiw, but it can be refreshed in some steps | 14:30 |
| dtantsur | kubajj: I don't think so.. | 14:32 |
| * dtantsur -> short walk, may miss the meeting | 14:32 | |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 14:46 |
| clif | o/ | 15:00 |
| alegacy_ | o/ | 15:00 |
| TheJulia | I guess it is time | 15:02 |
| TheJulia | #startmeeting ironic | 15:02 |
| opendevmeet | Meeting started Mon Oct 6 15:02:50 2025 UTC and is due to finish in 60 minutes. The chair is TheJulia. 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 | So, this will be our last meeting until after the PTG! | 15:03 |
| kubajj | o/ | 15:03 |
| TheJulia | The PTG is the week of the 27th. Our next meeting will be November 3rd. | 15:03 |
| cid | o/ | 15:04 |
| TheJulia | #topic Announcements / Reminders | 15:04 |
| TheJulia | A general reminder to review items tagged with the hashtag "ironic-week-prio" | 15:04 |
| TheJulia | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:04 |
| rpittau | o/ | 15:05 |
| TheJulia | For this next release, gazpacho, the schedule has been posted. | 15:05 |
| TheJulia | #link https://releases.openstack.org/gazpacho/schedule.html | 15:05 |
| TheJulia | Generally, in terms of the release schedule, the PTG is a little later into the cycle than we're typically used to based upon the past history. Also, Ironic will likely need to cut it's final release in R-2 which is the third week of March. | 15:06 |
| TheJulia | Any questions or concerns regarding the schedule? | 15:07 |
| TheJulia | #topic Working Group Updates | 15:08 |
| TheJulia | Any working group updates for us this week? | 15:08 |
| alegacy_ | networking... | 15:08 |
| alegacy_ | First, thanks to those that have provided review comments so far. I've done my best at addressing most of those. Still a couple to go...but making progress | 15:08 |
| TheJulia | Thanks alegacy_! | 15:09 |
| alegacy_ | I seem to have hit a snag with the whole Ironic import business though. | 15:09 |
| TheJulia | Yeah, some discussion earlier regarding that | 15:09 |
| alegacy_ | Should I maybe organize a separate call to kick that around a bit? I feel like there are some questions about the direction... | 15:09 |
| TheJulia | I think that will likely be a good idea | 15:09 |
| TheJulia | I also suspect we're at the low end of critical mass for that topic here at the moment. | 15:10 |
| TheJulia | Onward? | 15:10 |
| alegacy_ | ya... i'll try to find a suitable timeslot and post it after the meeting. | 15:11 |
| TheJulia | Okay | 15:11 |
| TheJulia | #topic Discussion Topics | 15:11 |
| TheJulia | rpittau has a single topic this week. | 15:12 |
| TheJulia | rpittau: would you like to bring it up or do you want me to? | 15:12 |
| rpittau | yep, just a reminder that I'm going to count the number for the ironic dinner at OIS tomorrow at the latest | 15:12 |
| TheJulia | #link https://etherpad.opendev.org/p/Ironic_OIS_community_dinner_in_Paris | 15:12 |
| rpittau | and then put a reservation :) | 15:12 |
| rpittau | we're 12 at the moment | 15:13 |
| TheJulia | 13 by my count | 15:13 |
| rpittau | I can't count :D | 15:13 |
| rpittau | that's all | 15:14 |
| TheJulia | It happens :) | 15:14 |
| TheJulia | #topic Bug Deputy Updates | 15:14 |
| TheJulia | cid: I think you've been quietly updating the agenda (thanks!) | 15:14 |
| cid | Yep | 15:15 |
| cid | I was bug deputy for the week. | 15:15 |
| cid | Already updated https://wiki.openstack.org/wiki/Meetings/Ironic. | 15:15 |
| cid | There were 4 newly filed bugs, and 1 RFE | 15:15 |
| cid | The RFE is about allowing setting BOOTIF whether or not there is a VIF | 15:15 |
| cid | https://bugs.launchpad.net/ironic/+bug/2126644 | 15:15 |
| cid | I think TheJulia already commented on it on LaunchPad. | 15:15 |
| TheJulia | Yeah, I think it makes sense to do | 15:16 |
| TheJulia | I think we likely *ought* to rename some of the internals | 15:16 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: ci: Remove ironic-inspector related ci job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962997 | 15:16 |
| TheJulia | i.e. "pxe_interface" | 15:16 |
| TheJulia | Since PXE, it is not. | 15:16 |
| TheJulia | Any questions or dicsussion regarding the RFE? | 15:18 |
| * TheJulia suspects the coffee maker needs to be turned on and coffee distributed | 15:18 | |
| TheJulia | Since we're on hold the next few weeks, the expectation is that contributors do take time to sporadically look at bugs and triage as time permits. | 15:19 |
| TheJulia | As such, no need to name a bug deputy for the next few weeks. | 15:19 |
| TheJulia | #topic Open Discussion | 15:19 |
| cid | ++ | 15:20 |
| TheJulia | Do we have anything else to discuss today? Aside from making more coffee! | 15:21 |
| esimone | Hi, first time here... there is a backport in stand by due to a CI issue (https://review.opendev.org/c/openstack/ironic/+/962993) | 15:21 |
| esimone | I tried to investigate the issue (web server restart fault on devstack setup) and commented | 15:21 |
| rpittau | esimone: looks ok to me | 15:22 |
| TheJulia | yeah, I'll approve it once my browser is signed in | 15:23 |
| TheJulia | Done! | 15:23 |
| TheJulia | Anything else? | 15:23 |
| * TheJulia takes silence as time for everyone to have more coffee.... ;) | 15:25 | |
| TheJulia | Thanks everyone! | 15:26 |
| TheJulia | #endmeeting | 15:26 |
| opendevmeet | Meeting ended Mon Oct 6 15:26:59 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:26 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-10-06-15.02.html | 15:26 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-10-06-15.02.txt | 15:26 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-10-06-15.02.log.html | 15:26 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: remove tinyipa job alias usage https://review.opendev.org/c/openstack/ironic/+/963001 | 15:28 |
| alegacy_ | how about 1500UTC tomorrow for an opportunity to discuss the Ironic--NGS import issues and related topics? | 15:31 |
| TheJulia | That time is clear for me | 15:32 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: WIP: Implement functionality for the is_root_volume RAID config https://review.opendev.org/c/openstack/ironic-python-agent/+/963200 | 15:33 |
| kubajj | TheJulia: does ^ this look any sensible? (wanted to push a draft before we prepare any tests) | 15:34 |
| kubajj | (we did not test it yet, but will do tomorrow) | 15:35 |
| alegacy_ | Ok, tentatively let's go with 1500UTC then... This is the link: https://meet.google.com/mzm-xtky-jbr ... i'll re-post tomorrow an hour or so beforehand as a reminder. | 15:36 |
| TheJulia | kubajj: yeah, along those lines is what I was thinking | 15:37 |
| kubajj | TheJulia: nice, will tests and get it ready in the coming days | 15:40 |
| opendevreview | Merged openstack/ironic stable/2025.1: Allow multiple HTTP redirects for image source https://review.opendev.org/c/openstack/ironic/+/962993 | 16:37 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: disable multi-n-cpu on ironic multinode job https://review.opendev.org/c/openstack/ironic/+/962980 | 17:19 |
| TheJulia | I hope that sorts out the intermittent failure issues for that job | 17:31 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: remove snmp ci job https://review.opendev.org/c/openstack/ironic/+/963215 | 17:44 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: remove metalsmith ci job invocation https://review.opendev.org/c/openstack/ironic/+/963216 | 17:50 |
| TheJulia | stevebaker[m]: doesn't https://review.opendev.org/c/openstack/ironic/+/963151/3/ironic/drivers/fake_hardware.py seem like the opposite of what is wanted? | 18:07 |
| stevebaker[m] | TheJulia: It has moved to a parent class. Uh which isn't in that change :P I'll refresh it in a bit | 18:12 |
| TheJulia | okay! :) | 18:12 |
| TheJulia | I was like 'wait, what?!" | 18:13 |
| TheJulia | Thanks! | 18:13 |
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 19:03 |
| TheJulia | So, an observation, it looks like the ovn jobs seem to be fail happy, like dhcp is not working. I'll check some more logs tomorrow, it is certinly weird | 20:43 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 21:01 |
| opendevreview | Steve Baker proposed openstack/ironic master: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 22:55 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!