| janders | good morning Ironic o/ | 08:08 |
|---|---|---|
| rpittau | good morning ironic! o/ | 08:29 |
| Ayo[m] | Good morning everyone:) | 09:10 |
| Ayo[m] | Good morning rpittau o/ | 09:11 |
| opendevreview | Aykut Bulgu proposed openstack/sushy stable/2024.2: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/965896 | 09:20 |
| opendevreview | nidhi proposed openstack/ironic master: Add PCIe function fields to redfish inspection https://review.opendev.org/c/openstack/ironic/+/963179 | 11:46 |
| opendevreview | Morten Stephansen 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 | 12:49 |
| opendevreview | Merged openstack/ironic master: Fix power sync timeouts after BMC firmware update https://review.opendev.org/c/openstack/ironic/+/962560 | 13:02 |
| opendevreview | Merged openstack/ironic master: api: Prevent overlapping schema versions https://review.opendev.org/c/openstack/ironic/+/960291 | 13:11 |
| opendevreview | Merged openstack/ironic master: Remove sushy-oem-idrac from driver requirements https://review.opendev.org/c/openstack/ironic/+/965114 | 13:11 |
| TheJulia | good morning | 14:13 |
| mumesan[m] | good morning! | 14:33 |
| clif | Is the meeting 10 minutes or 1 hour and 10 minutes? | 14:50 |
| clif | good morning o/ | 14:50 |
| clif | daylight savings woes :) | 14:52 |
| cid | Should be in 2minutes (assuming my Google Calender has already accounted for daylight saving) | 14:58 |
| janders | ++ | 14:59 |
| dtantsur | This matches my expectations | 15:00 |
| alegacy | o/ | 15:00 |
| dtantsur | Who's chairing today? | 15:01 |
| TheJulia | Someone needs to chair :) | 15:01 |
| TheJulia | #startmeeting ironic | 15:02 |
| opendevmeet | Meeting started Mon Nov 3 15:02:19 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 | Someone volunteer to chair :) | 15:02 |
| dtantsur | o/ | 15:02 |
| dtantsur | thanks TheJulia! | 15:02 |
| * dtantsur not feeling all too great still | 15:02 | |
| rpittau | o/ | 15:02 |
| janders | o/ | 15:03 |
| TheJulia | #topic Announcements / Reminders | 15:03 |
| TheJulia | dtantsur: :( | 15:03 |
| TheJulia | Consider this reminder to review items tagged with ironic-week-prio, and furthermore apply the tag to any item which appears ready for reviews. | 15:03 |
| TheJulia | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| TheJulia | Also, a reminder the 2026.1 Gazpacho Release Schedule has been posted. | 15:04 |
| TheJulia | #link https://releases.openstack.org/gazpacho/schedule.html | 15:04 |
| cid | o/ | 15:04 |
| TheJulia | This week is week R-21 on the schedule. | 15:04 |
| TheJulia | As a reminder, we structurally have a little less than two months left to the end of the year. | 15:04 |
| dtantsur | Time flies! | 15:05 |
| TheJulia | *Many* contributors will be taking PTO/Vacations at various times. Please be mindful and plan/set expectations ahead. | 15:05 |
| dtantsur | I assume we'll need an intermedia release before NY? | 15:05 |
| TheJulia | Yes | 15:05 |
| TheJulia | We'll get there | 15:05 |
| rpittau | I've created this patch that should help keeping in track with releasing our projects https://review.opendev.org/c/openstack/ironic/+/965090 | 15:06 |
| TheJulia | https://review.opendev.org/c/openstack/ironic-specs/+/965865/1/priorities/2026-1-workitems.rst presents a rough schedule. | 15:06 |
| dtantsur | Huh, these don't match? | 15:07 |
| rpittau | yeah they're not in conflict, my change is much more genric and can be applied to all release cycles | 15:07 |
| dtantsur | rpittau's link says releasing at the first milestone, which is apparently R-20? | 15:07 |
| dtantsur | TheJulia's says first week of Dec | 15:07 |
| TheJulia | Lets get into it later | 15:07 |
| dtantsur | sorry, please go on :) | 15:07 |
| rpittau | we usually release between week 6 and 8 with a bugfix branch | 15:07 |
| TheJulia | bottom line is we have broad schedules and more specific suggestion, coupled with a need to be realistic | 15:08 |
| TheJulia | 6-8 is *way* too late | 15:08 |
| TheJulia | fwiw | 15:08 |
| TheJulia | around week 6, we should be prepared to cut our final release for the cycle. | 15:08 |
| TheJulia | Lets get into it when we get there | 15:08 |
| rpittau | 6 weeks after the beginning of the cycle ? I' mtalkgin about R-20 | 15:09 |
| rpittau | alright | 15:09 |
| TheJulia | We have a working group updates section, and I think the thing to do there is to skip that for now. Once we have the areas highlighted via the prioriteis consensus, we can backfeed into that. | 15:09 |
| TheJulia | In consideration of alegacy's time, do you have anything to really mention? | 15:09 |
| dtantsur | I've just added the Async IO group to the list | 15:09 |
| TheJulia | #topic Working Group Updates | 15:10 |
| alegacy | Not really. Testing is going well. Really just in need of reviews | 15:10 |
| TheJulia | cool cool, folks please review alegacy's work. | 15:10 |
| TheJulia | Is there anything else this week for working group items? | 15:10 |
| kubajj | o/ | 15:10 |
| dtantsur | Do they all have ironic-week-prio, alegacy? | 15:11 |
| dtantsur | re Async WG, I'd like to have a boostrap call this or next week, wdyt? cid and whoever else is interested? | 15:11 |
| alegacy | dtantsur: yes, I believe so... i'll double check now | 15:12 |
| dtantsur | (if you're interested in async WG, add yourself to https://etherpad.opendev.org/p/ironic-asyncio) | 15:12 |
| dtantsur | alegacy++ | 15:12 |
| cid | dtantsur, works for me either of the weeks. | 15:12 |
| dtantsur | cool | 15:12 |
| TheJulia | Since we don't have any discussion topis, bug deputy updates, or RFEs to review right now, it feels like the next stop for this meeting is Open Discussion, however we need to identify a bug deputy for the next week | 15:13 |
| cid | alegacy, will make sure to review this week. Question, adding to the review queue means in merge-able state, but you mention some of them being tested :) | 15:13 |
| cid | I can take the bug deputy task for this week, already been taking a look during the PTG | 15:14 |
| TheJulia | cool cool | 15:15 |
| cardoe | Folks can give some feedback on https://cardoe.com/neutron/evpn-vxlan-network/admin/data-center-networks.html which is where I'm trying to add a doc to neutron describing the VXLAN bits. | 15:15 |
| TheJulia | Okay then, thanks cid! | 15:15 |
| TheJulia | #topic Open Discussion | 15:15 |
| cardoe | https://review.opendev.org/c/openstack/neutron/+/965415 | 15:15 |
| TheJulia | Clearly, we have a LOT to sift through this week. | 15:16 |
| alegacy | cid: sorry, they are merge-able... by 'testing' i meant the testing I'm doing with this in a more integrated fashion with higher level software. | 15:16 |
| cid | ack'd | 15:17 |
| TheJulia | I guess at a simple CI level, maybe some additional scripting to do the needful in terms of a basic validation?! That might be feasible | 15:18 |
| TheJulia | So clearly, we need to reach consensus on cycle plans. JayF has a WIP review up already for that which I encourage people to set bounds on and keep in mind we do have a time window during the next two months where some portion of contributors will disappear for at least a couple weeks. | 15:20 |
| rpittau | #link https://review.opendev.org/c/openstack/ironic-specs/+/965865 | 15:20 |
| TheJulia | Any other items for discussion this week? | 15:21 |
| janders | From my side, the top priority follow-up from the PTG I have is iRMC support/deprecation/removal. I am intending to reach out to iRMC/FJ folks early this week. Aiming to report back what I found out in next week's meeting | 15:22 |
| TheJulia | janders: perfect, thank you! | 15:22 |
| janders | no problem. A brief follow up question on this: | 15:23 |
| TheJulia | Another item wort nothing, JayF did send an email to the mailing list regarding reviewer structure. | 15:23 |
| TheJulia | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/LUCBUPWZIRCANEHREHUPO7DZCPVJW4ML/ | 15:23 |
| TheJulia | I'm sure he would appreciate responses | 15:23 |
| TheJulia | Also, does anyone feel interested in sending a PTG summary to the mailing list? | 15:23 |
| kubajj | This is a wee bugfix of mostepha[m] , but it is blocking the progress on is_root_volume implementation: https://review.opendev.org/c/openstack/ironic-python-agent/+/965797 | 15:23 |
| janders | in my past comms with iRMC/FJ folks they indicated they need it in 2025.1, they seemed less concerned about what happens later | 15:24 |
| janders | if we fasttrack deprecation/removal in master how does it affect still maintained branches? | 15:24 |
| janders | I understand feature/bugfix workflow way better than deprecation/removal hence the question | 15:25 |
| TheJulia | kubajj: eek, thats kind of a bad bug | 15:25 |
| dtantsur | kubajj, mostepha[m], a release note won't hurt | 15:26 |
| kubajj | TheJulia: it was also weird to debug, because we only hit it on one of our qa nodes | 15:26 |
| TheJulia | Yeah, I think a release note is kind of key since we'll want to backport that | 15:26 |
| TheJulia | janders: That may always change, but yeah, worth syncing up with them and letting us know | 15:27 |
| kubajj | TheJulia, dtantsur: ok, will quickly add | 15:27 |
| TheJulia | kubajj: thanks! | 15:27 |
| TheJulia | janders: doesn't impact released branches. | 15:28 |
| TheJulia | janders: but the reality is the snmp portions of the driver, which we've pushed back against in the past, need to minimally go away entirely. | 15:28 |
| janders | TheJulia ack and thank you | 15:29 |
| janders | I totally understand, I will start the discussion and report back | 15:29 |
| TheJulia | janders: Concievably, if the snmp bits and dependency can be excised, that could be a bugfix | 15:29 |
| TheJulia | thanks! | 15:30 |
| TheJulia | but, because we also can't test or assure it, we're sort of forced to keep the dividing line at the whole of the driver | 15:30 |
| janders | yeah | 15:30 |
| TheJulia | Okay, anything else this morning? | 15:31 |
| TheJulia | Do we have a PTG summary volunteer? | 15:31 |
| JayF | TheJulia: I'd prefer we get work items landed and just send that as the summary | 15:32 |
| JayF | Once we get that patch to a reasonable spot, I also plan on doing a video for gross YouTube | 15:33 |
| TheJulia | PTGs are always 1-2 cycles worth of details and context, I'd prefer we draw a line or awareness since there is more there than we can execute upon as well. | 15:33 |
| TheJulia | I guess ^^^ is where my concern with the current doc is, but it can be revised, I think the key aspect is for ironic contributors to review/comment *today* | 15:34 |
| JayF | Yeah the doc is 100% in need of editing. I'm not by any means saying it's there. | 15:35 |
| TheJulia | cool cool | 15:35 |
| JayF | I may or may not have leaned on fancy new fake brain tooling to help build it 😂 | 15:35 |
| TheJulia | okay, anything else before we wrap this week's meeting then? | 15:35 |
| TheJulia | lol | 15:35 |
| TheJulia | I suspected that based upon one of the noted reasons which didn't add up | 15:36 |
| TheJulia | (which is sort of weight there anyhow!) | 15:36 |
| TheJulia | I'm taking silence as reason to call it a meeting. | 15:38 |
| TheJulia | Thanks Folks! | 15:38 |
| dtantsur | \o/ | 15:38 |
| janders | thank you all o/ | 15:38 |
| * cardoe silences. | 15:38 | |
| TheJulia | Go review all the things! | 15:38 |
| TheJulia | #endmeeting | 15:38 |
| opendevmeet | Meeting ended Mon Nov 3 15:38:57 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-11-03-15.02.html | 15:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-11-03-15.02.txt | 15:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-11-03-15.02.log.html | 15:38 |
| TheJulia | cardoe: no silence, only zuuuuuuul :) | 15:39 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: Fix for matching hints with lists of strings https://review.opendev.org/c/openstack/ironic-python-agent/+/965797 | 15:39 |
| kubajj | TheJulia, dtantsur: added the reno ^ | 15:51 |
| kubajj | hope it is ok | 15:51 |
| dtantsur | thx, I'll check soon | 15:51 |
| kubajj | dtantsur: thanks | 15:51 |
| dtantsur | kubajj: sorry for not realizing it earlier, but would it be possible to add unit tests for the fixed behavior? | 16:02 |
| dtantsur | the code itself lgtm | 16:02 |
| kubajj | dtantsur: ah, yeah, of course | 16:02 |
| opendevreview | Morten Stephansen proposed openstack/ironic-python-agent master: Fix for matching hints with lists of strings https://review.opendev.org/c/openstack/ironic-python-agent/+/965797 | 16:38 |
| cardoe | Sooo one of my many yaks (not happening today) is to give some better examples of building a container of some openstack services to loci / kolla. Would there be an interest in me submitting the Containerfile for Ironic to the tree? | 16:39 |
| opendevreview | Doug Goldstein proposed openstack/networking-generic-switch master: fix: segment binding level for VLAN support https://review.opendev.org/c/openstack/networking-generic-switch/+/965948 | 16:46 |
| frickler | cardoe: you triggered my attention by mentioning kolla, but I'm not sure what you are proposing. kolla is building ironic containers today, if you think that there is something wrong about how this is done, a bug report or patch submission would sound more useful. or maybe (likely?) I am lacking some context | 16:52 |
| cardoe | I would like to try and combine some of the efforts of kolla and loci. | 16:52 |
| cardoe | I've also thought about the fact that maybe each project would want to have some integration | 16:53 |
| JayF | cardoe: do you plan on contributing upstream resources to achieve the port behavior toggling work discussed at PTG? | 18:47 |
| JayF | cardoe: trying to evaluate if it stays on our work items | 18:47 |
| cardoe | What was the port toggling? | 18:47 |
| JayF | turning a port from "normal"/storage | 18:48 |
| JayF | iirc | 18:48 |
| cardoe | ah okay looking at the existing list, I would probably drop that from 2026.1 | 18:48 |
| cardoe | We'll be more focused on the VXLAN and the inspection side of things. | 18:49 |
| cardoe | For 2026.1 | 18:49 |
| JayF | okie doke | 18:51 |
| TheJulia | alegacy: looks like your big ironic change needs a rebase. Are you already aware of that? | 19:00 |
| JayF | TheJulia: fwiw that one thing you ID'd as an AI line, I don't recall if I edited it to say that, but it did match what my understanding was so... not just AI hallucination, it's Jay-hallucination (or more likely: the original reason is lost to time and/or not the one we wanna document) | 19:01 |
| JayF | alegacy: lmk if you start rebasing that, reviewing that is on my list and if it's landable I can put +2s on if ready :D | 19:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: ci: dial back check intervals https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/942220 | 19:02 |
| JayF | https://review.opendev.org/c/openstack/ironic-specs/+/965249 someone wanna CR this? It's in the workitems chain. | 19:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: 2026.1 Ironic work items https://review.opendev.org/c/openstack/ironic-specs/+/965865 | 19:03 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: 2026.1 Ironic work items https://review.opendev.org/c/openstack/ironic-specs/+/965865 | 19:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: cover what happens if someone changes iscsi passwords https://review.opendev.org/c/openstack/ironic/+/963442 | 19:26 |
| TheJulia | JayF: more so, folks interpret selenium more for verification, and because of that RH didn't want to ship it | 19:31 |
| JayF | I just dropped the "why" for that line item :) | 19:32 |
| TheJulia | k | 19:33 |
| JayF | avoiding selenium is a goal in it's own way :P | 19:36 |
| opendevreview | Merged openstack/ironic master: docs: provide a little more clarity/guidance w/r/t redfish https://review.opendev.org/c/openstack/ironic/+/965030 | 19:39 |
| opendevreview | Merged openstack/ironic master: docs: add a "i want a pony" bit of text https://review.opendev.org/c/openstack/ironic/+/965035 | 19:49 |
| alegacy | TheJulia: JayF: Yep, I noticed that a rebase is required. I'll try to do that in the morning. | 20:00 |
| opendevreview | Doug Goldstein proposed openstack/networking-generic-switch master: fix: segment binding level for VLAN support https://review.opendev.org/c/openstack/networking-generic-switch/+/965948 | 20:06 |
| JayF | alegacy: I have a -1 with some pretty significant feedback you may want to consider first | 20:11 |
| JayF | at least on the one at the top of the chain | 20:11 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/962493/6#message-125163124b9d3c9628a1d3c04ebdc248fd324a59 | 20:11 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: pass along physical_network to neutron from the baremetal port https://review.opendev.org/c/openstack/ironic/+/964570 | 20:14 |
| opendevreview | Verification of a change to openstack/ironic master failed: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 20:26 |
| cardoe | So does anyone look at our arm64 tests? | 20:48 |
| opendevreview | Verification of a change to openstack/ironic master failed: api: Add schema for inspection rules API (requests) https://review.opendev.org/c/openstack/ironic/+/954039 | 20:48 |
| alegacy | JayF: ack | 20:53 |
| TheJulia | I haven't recently, but are they failing now? | 21:03 |
| cardoe | Yeah. They are. | 21:14 |
| cardoe | So AT&T fiber fell over earlier. Then their cell network followed. Now Verizon’s cell network fell over. Starlink saving the day until it falls over. | 21:15 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix heredoc for SystemdConsoleContainer.start_container https://review.opendev.org/c/openstack/ironic/+/965980 | 21:16 |
| TheJulia | oh joy | 21:38 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: cover what happens if someone changes iscsi passwords https://review.opendev.org/c/openstack/ironic/+/963442 | 21:40 |
| opendevreview | Verification of a change to openstack/ironic master failed: Enable fake-graphical console for all hardware types https://review.opendev.org/c/openstack/ironic/+/963151 | 21:42 |
| TheJulia | cardoe: well, passing but did find a failure which feels like a race condition | 22:00 |
| TheJulia | maybe not, looks like one failure was the code itself | 22:02 |
| cardoe | pre-commit run --all-files is triggering codespell changes so something in there is not catching the config. | 22:27 |
| cardoe | hrm so in the tests when we're creating port and portgroup objects... if we create a portgroup and then create and add ports to the port group. If those ports have a physical_network should the portgroup be updated? | 22:40 |
| cardoe | e.g. should we have a DB trigger ensuring this? | 22:40 |
| cardoe | If not that's fine but I'll try to update the test helpers. | 22:44 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: cs10 - remove excess intel firmware https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/965984 | 22:47 |
| TheJulia | rpittau: ^ | 22:47 |
| TheJulia | uhhhh dunno. | 22:48 |
| TheJulia | cardoe: dunno I think I just reached outofbrain for the day | 22:48 |
| cardoe | I was trying to test my physical_network change and switching the behavior to using portgroup.physical_network instead of portgroup.ports[0].physical_network causes it to all fail. | 22:49 |
| TheJulia | .. I think I commented on that change expecting something to break soon | 22:52 |
| cardoe | Which change? my physical_network one? There's no comments on it. | 22:56 |
| cardoe | But the physical_network portgroup thing passed tests but its a total fuke. | 22:58 |
| cardoe | https://opendev.org/openstack/ironic/src/commit/6b84d88a5d1c8952560e6218ac73e39888e600c0/ironic/tests/unit/drivers/modules/network/test_common.py#L43 the def _objects_setup() doesn't really work. | 22:59 |
| cardoe | All the ports have a physical_network but the portgroup itself does not have a physical_network. So the tests validate that the dict'd object has the same fields as the DB objects. And yes physical_network=None == physical_network=null in the DB. | 23:00 |
| TheJulia | Looks like the code has changed in that change since I last looked at it | 23:06 |
| TheJulia | its likely fair to just fix test_common.py | 23:07 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: ensure that portgroup physical_network is updated for tests https://review.opendev.org/c/openstack/ironic/+/965985 | 23:10 |
| cardoe | ^ I expect that to fail | 23:10 |
| cardoe | and then I'll fix up test_common.py after the fact. | 23:11 |
| TheJulia | k | 23:12 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!