| blake_ | good evening :) anyone aware that tarballs.opendev.org is down? | 02:38 |
|---|---|---|
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Suppress misleading ERROR logs from OVN port lookups https://review.opendev.org/c/openstack/networking-baremetal/+/980546 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix L2VNI trunk port binding for switch config https://review.opendev.org/c/openstack/networking-baremetal/+/980547 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add OVN event-driven reconcile for L2VNI trunks https://review.opendev.org/c/openstack/networking-baremetal/+/980548 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add targeted single-VLAN reconcile for OVN events https://review.opendev.org/c/openstack/networking-baremetal/+/980549 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add VNI to L2VNI trunk subport binding profiles https://review.opendev.org/c/openstack/networking-baremetal/+/980578 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Suppress misleading ERROR logs from OVN port lookups https://review.opendev.org/c/openstack/networking-baremetal/+/980612 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix incorrect is_partial_segment() call on PortContext https://review.opendev.org/c/openstack/networking-baremetal/+/980640 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix router interface HA binding for baremetal VLAN networks https://review.opendev.org/c/openstack/networking-baremetal/+/980648 | 04:35 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: L2VNI multi-link LAG support + fix OVN LLDP filter https://review.opendev.org/c/openstack/networking-baremetal/+/980612 | 04:39 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix incorrect is_partial_segment() call on PortContext https://review.opendev.org/c/openstack/networking-baremetal/+/980640 | 04:39 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix router interface HA binding for baremetal VLAN networks https://review.opendev.org/c/openstack/networking-baremetal/+/980648 | 04:39 |
| rpittau | good morning ironic! o/ | 07:46 |
| opendevreview | Merged openstack/networking-baremetal master: Fix hash ring broken after eventlet removal https://review.opendev.org/c/openstack/networking-baremetal/+/980543 | 08:21 |
| opendevreview | Merged openstack/networking-baremetal master: fix: correct client ovs/ovn idl object creation https://review.opendev.org/c/openstack/networking-baremetal/+/980324 | 08:21 |
| opendevreview | Merged openstack/networking-baremetal master: fix neutron port reconcilator https://review.opendev.org/c/openstack/networking-baremetal/+/980437 | 08:21 |
| opendevreview | Merged openstack/networking-baremetal master: Use other_config for bridge-mapping lookup (OVN) https://review.opendev.org/c/openstack/networking-baremetal/+/980374 | 08:21 |
| *** jroll03 is now known as jroll0 | 08:38 | |
| opendevreview | Merged openstack/networking-baremetal master: Allow hostname in l2vni_network_nodes.yaml config https://review.opendev.org/c/openstack/networking-baremetal/+/980375 | 08:57 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Revert "Handle missing RelatedProperties in TransferProtocolType detection" https://review.opendev.org/c/openstack/sushy/+/980697 | 12:13 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Suppress misleading ERROR logs from OVN port lookups https://review.opendev.org/c/openstack/networking-baremetal/+/980546 | 13:10 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix L2VNI trunk port binding for switch config https://review.opendev.org/c/openstack/networking-baremetal/+/980547 | 13:13 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add OVN event-driven reconcile for L2VNI trunks https://review.opendev.org/c/openstack/networking-baremetal/+/980548 | 13:14 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add targeted single-VLAN reconcile for OVN events https://review.opendev.org/c/openstack/networking-baremetal/+/980549 | 13:23 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add VNI to L2VNI trunk subport binding profiles https://review.opendev.org/c/openstack/networking-baremetal/+/980578 | 13:24 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: L2VNI multi-link LAG support + fix OVN LLDP filter https://review.opendev.org/c/openstack/networking-baremetal/+/980612 | 13:25 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix incorrect is_partial_segment() call on PortContext https://review.opendev.org/c/openstack/networking-baremetal/+/980640 | 13:26 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add targeted single-VLAN reconcile for OVN events https://review.opendev.org/c/openstack/networking-baremetal/+/980549 | 13:56 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add VNI to L2VNI trunk subport binding profiles https://review.opendev.org/c/openstack/networking-baremetal/+/980578 | 13:56 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: L2VNI multi-link LAG support + fix OVN LLDP filter https://review.opendev.org/c/openstack/networking-baremetal/+/980612 | 13:56 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix incorrect is_partial_segment() call on PortContext https://review.opendev.org/c/openstack/networking-baremetal/+/980640 | 13:56 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix router interface HA binding for baremetal VLAN networks https://review.opendev.org/c/openstack/networking-baremetal/+/980648 | 13:56 |
| cardoe | rpittau: morning! | 13:59 |
| rpittau | hey cardoe :) | 13:59 |
| TheJulia | good morning | 13:59 |
| * TheJulia corrects her calendar due to daylight savings time | 14:00 | |
| dtantsur | o/ | 14:01 |
| dtantsur | cardoe: hi, do I recall it right that you wanted to move sushy in-tree very soon? (it's relevant for our async redfish work) | 14:01 |
| cardoe | I was thinking right after 2026.1 branching. | 14:02 |
| cardoe | We pitched it out there and discussed it last PTG. I weighed in why I was -1. But folks gave some guarantees around my concerns. So I think we should proceed since the overwhelming majority was in favor. | 14:03 |
| cardoe | I just think leaving ambiguity in the change is going to be more problematic and lead to possible duplication of work. | 14:04 |
| cardoe | So I figure giving us a full release cycle to hammer out the rough edges would be best. | 14:04 |
| cardoe | What do you think about that approach? | 14:04 |
| TheJulia | Could someone take a quick look at https://review.opendev.org/c/openstack/ironic/+/978602/7, its small | 14:05 |
| cardoe | TheJulia: the commit message is wrong if we're okay with that? | 14:12 |
| cardoe | It says it does something different than the change does | 14:12 |
| TheJulia | we can revise it directly if we want | 14:13 |
| cardoe | I gave it a +2 | 14:13 |
| cardoe | But didn't +W if ya wanna fix or whatever | 14:13 |
| TheJulia | lets get JayF to ack the issue and make the call then | 14:14 |
| TheJulia | since it might be something forgotten in the ide | 14:14 |
| cardoe | hjensas: https://review.opendev.org/c/openstack/ironic/+/633729 is that something that should be revisited or abandoned? I know we landed some events API stuff already so not sure if there's functionality you've got in your change that we should get on top of what landed or what. | 14:15 |
| cardoe | hjensas: same ask here https://review.opendev.org/c/openstack/ironic/+/637840 | 14:16 |
| cardoe | hjensas: and here https://review.opendev.org/c/openstack/ironic/+/637841 | 14:16 |
| TheJulia | Quick question: Since we're relatively AI tool friendly here, would folks be interested in joining a governing board session during the PTG to discuss impact/perceptions ? | 14:17 |
| dtantsur | cardoe: yeah, having a full cycle will definitely help | 14:17 |
| cardoe | TheJulia: yep. I'd join | 14:17 |
| dtantsur | TheJulia: i'm only relatively AI-freindly ;) | 14:17 |
| TheJulia | Well, I was more saying friendly from the embrace/recognize/possibly utilize | 14:18 |
| cardoe | TheJulia: I think JayF is right in the release note for https://review.opendev.org/c/openstack/ironic/+/959880 but are we cool with the output to users changing? Ultimately we'll be able to get people detailed messages once the API is covered. | 14:18 |
| TheJulia | standpoint, opinions either way ;) | 14:18 |
| JayF | I think we have to be cool with that output changing if we want to improve this at all, I just think we need to billboard it with a release though | 14:18 |
| JayF | ** Note | 14:19 |
| JayF | And we've also already had that effect on endpoints already migrated and just didn't realize it in review | 14:19 |
| TheJulia | cardoe: I think thats for posting *to* ironic, afaik its not covered in the client and I'd never expect a human to touch events | 14:19 |
| JayF | TheJulia: I would suggest that whoever is doing that research sit in on our internal team use of AI session as well | 14:19 |
| TheJulia | JayF: It was already noted we would have one on the board ML | 14:20 |
| cardoe | Since I tend to have work calls overlapping the ironic meeting I'm gonna toss this out here with the hopes you folks can discuss during the call and I'm okay with the answer being discuss during PTG as well... my team is using the inspection rules tester I wrote plus looking at the TBN simulator. We utilize GitHub Actions so we were going to look at putting together some reusable actions for using the two testers. Do we | 14:23 |
| cardoe | want that contributed to the Ironic repo or another OpenStack/OpenDev repo or just something my org owns that we'll make public? | 14:23 |
| TheJulia | define the scope where it needs a repo? | 14:26 |
| cardoe | It's just a script and a YAML description | 14:28 |
| cardoe | It's gotta live in a repo that gets pushed to GitHub, which the Ironic repo does. | 14:28 |
| cardoe | Not sure how familiar with GitHub Actions you are but: - uses: actions/checkout@v6 for example checks out the code in the current repo using https://github.com/actions/checkout from the "v6" tag. | 14:29 |
| cardoe | We're gonna write "rackerlabs/ironic/inspection-rules-tester" and "rackerlabs/ironic/tbn-sim" for example. | 14:30 |
| JayF | cardoe: Id just make sure it's open source in a public repo somewhere. And then you start writing the talk for the summit. | 14:32 |
| hjensas | @cardoe: oh, the neutron event handling, so I guess the main driver is https://opendev.org/openstack/ironic/src/branch/master/ironic/dhcp/neutron.py#L150-L158 | 14:32 |
| cardoe | Yeah it'd be open source for sure. | 14:33 |
| hjensas | cardoe: we get the events now, but don't actually use them. It might be worth picking up. | 14:33 |
| cardoe | hjensas: yep. Just was looking at that when reviewing your networking-baremetal changes. | 14:33 |
| cardoe | Just trying to understand where all that sits. | 14:34 |
| cardoe | Was just thinking for the upcoming PTG we could talk about picking it up | 14:35 |
| hjensas | cardoe: but the events stuff I have in proposed patches are using OVN events - the goal there is to quickly set up the trunk and ha router alignment. So that network create <- deploy node can work if there is no human that will take 10 minutes for coffebreak between net create and deploying a node. | 14:35 |
| hjensas | s/proposer patches/proposed networking-baremtal patches/ | 14:38 |
| cardoe | Yep. It's totally different event stuff but it just made me think of it while reviewing. | 14:45 |
| TheJulia | I would love to move forward on Deferred Tasks and have the event from neutron be that deferred task | 14:46 |
| cardoe | Yeah that'd be a good usage. | 14:49 |
| cardoe | Mother Nature is mad. It was 78F/25C yesterday. Today it's 38F/3C with a feels like 22F/-5C. It's also snowing hard to be nearly a whiteout. | 14:54 |
| JayF | so on friday, wake up at 5:30am, it's raining as usual | 14:54 |
| TheJulia | mother nature likely has decided we all got to go | 14:55 |
| JayF | my wife leaves for work around 6:30am, calls me 5 minutes later "it's pouring snow" and sure enough, at home it switched to just POURING snow | 14:55 |
| JayF | and we got like 4-5 inches just midday outta nowhere when they were calling for flurries at best | 14:55 |
| JayF | never even got below freezing either | 14:55 |
| TheJulia | JayF: sounds reminiscent when NC got hit when we were in high school | 14:55 |
| JayF | I didn't live in Raleigh in HS :) but I remember an incident in the late 2000s | 14:56 |
| TheJulia | I think your area got 2-3 feet, my area got 5-6 inches in like 1-2 hours | 14:56 |
| TheJulia | early '01 | 14:56 |
| JayF | where it was calling for a snowstorm all weekend, it started like 16 hours early and faster than they called for | 14:56 |
| JayF | and there was this photo which has been fully memified of i-40 looking like a zombie apocalypse | 14:56 |
| TheJulia | oh, that one | 14:56 |
| JayF | https://www.ncrabbithole.com/p/raleigh-nc-greatest-snow-picture-winter-weather this one | 14:57 |
| JayF | although that's glenwood ave, it's still hilarious | 14:57 |
| TheJulia | Yeah, the meme photo was 14. I remember it quite well. | 14:58 |
| TheJulia | Had to jerry rig my heat because the power was out for 8... days. | 14:58 |
| JayF | I had a power outage in rural NC around Spring Hope during a winter storm once that lasted like 10 days | 14:59 |
| JayF | we were one of the only houses with a fireplace so literally half the neighborhood was sleeping in our den | 14:59 |
| TheJulia | Has anyone scrubbed the release notes? | 14:59 |
| TheJulia | and updated release mappings? | 15:00 |
| cardoe | I think no to both. | 15:00 |
| TheJulia | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Mar 16 15:00:43 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 |
| TheJulia | Good morning everyone and Welcome to the weekly #ironic meeting | 15:00 |
| TheJulia | I'm TheJulia, and I'll be your host for this roast of the meeting agenda. | 15:01 |
| cardoe | JayF: what say you on https://review.opendev.org/c/openstack/ironic/+/978602 do we need to update the commit msg or no | 15:01 |
| TheJulia | Our first item on the agenda is to ensure everyone has Coffee. Everyone have coffee? | 15:01 |
| TheJulia | o/ | 15:01 |
| JayF | cardoe: meh? I can do it if folks insist, but the work is done by that commit in the one behaind it, just let it land | 15:01 |
| JayF | o/ | 15:01 |
| * JayF substitutes a can of diet pepsi | 15:02 | |
| opendevreview | Verification of a change to openstack/ironic master failed: Enable ramdisk autodetect by default https://review.opendev.org/c/openstack/ironic/+/978602 | 15:02 |
| opendevreview | Verification of a change to openstack/ironic master failed: Correct disk-format in ramdisk boot docs https://review.opendev.org/c/openstack/ironic/+/980333 | 15:02 |
| rpittau | o/ | 15:02 |
| TheJulia | Fair enough! | 15:02 |
| TheJulia | #topic Announcements / Reminders | 15:02 |
| TheJulia | Our first reminder of the week is for cores to review ironic-week-prio hashtaged items. | 15:02 |
| TheJulia | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| TheJulia | Our next item if the 2026.1 release cycle. This week is R-2 | 15:03 |
| TheJulia | #link https://releases.openstack.org/gazpacho/schedule.html | 15:03 |
| TheJulia | In other word: Time to get our end of cycle releases branched | 15:03 |
| kubajj | o/ | 15:03 |
| rpittau | we need to wrap up things, the branch creation will happen this week | 15:03 |
| TheJulia | Outside of release note cleanup and the release mappings update, what is a MUST ? | 15:04 |
| kubajj | JayF: ok, valid | 15:04 |
| rpittau | there are still some patches in ironic-week-prio | 15:05 |
| TheJulia | We can't hold forever | 15:05 |
| TheJulia | Some are bug fixes and can be backported | 15:05 |
| rpittau | yep, besides thos4e | 15:06 |
| TheJulia | I guess what I'm asking for is for us to be specific. | 15:06 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/978602 is approved and I'd /like/ it in, but it's not a must | 15:06 |
| rpittau | I will work on release notes cleanup and release mappings update to have all ready by Wednesday | 15:06 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Enable ramdisk autodetect by default https://review.opendev.org/c/openstack/ironic/+/978602 | 15:06 |
| TheJulia | I'd ask for us to try and be ready to release tomorrow instead of wednesday | 15:07 |
| JayF | (it needed a rebase to merge, I think) | 15:07 |
| TheJulia | that way we can take a bump in the road at the last minute | 15:07 |
| rpittau | TheJulia: I can probably then do one of the 2 things, not both :) | 15:07 |
| TheJulia | Then lets split the work | 15:08 |
| TheJulia | Who can take on one of the tasks? | 15:08 |
| rpittau | I've kind of started the release mappings update already, anyone up for the release notes cleanup ? | 15:08 |
| JayF | release mappings cleanup is just a ^c^v, swap in a version, update tests? | 15:08 |
| JayF | Can we be specific about what cleanup is? | 15:09 |
| JayF | for release notes? | 15:09 |
| JayF | - add a preamble (whatever it's called) | 15:09 |
| TheJulia | Scrub/cleanup any inconsistencies, and a preamble | 15:09 |
| JayF | - read the rendered version and fix text/grammar if needed | 15:09 |
| TheJulia | ++ | 15:09 |
| JayF | I didn't wanna take on something text-based today, I'm worried I don't have the juice for it | 15:10 |
| JayF | but I guess it's me | 15:10 |
| TheJulia | I can give it a shot | 15:10 |
| TheJulia | My brain is likely on the wordy side today | 15:10 |
| JayF | maybe split it? I'll write the preamble if you wanna do the edits? | 15:10 |
| TheJulia | As I have some long emails to aslo reply to as well | 15:10 |
| TheJulia | Sure | 15:10 |
| JayF | it's mainly the proofreading I'm afraid of, I've been sleeping badly and proofreading requires focus I'm not sure I have today tbh | 15:11 |
| TheJulia | fair enough | 15:11 |
| TheJulia | sounds like we have a plan! | 15:11 |
| TheJulia | Does anyone have anything else to announce or remind us of? | 15:11 |
| TheJulia | I'm taking silence as nope, and then onward! | 15:12 |
| TheJulia | #topic Working Group Updates | 15:13 |
| opendevreview | Merged openstack/python-ironicclient master: Remove dead code: define_command, define_commands_from_module, HelpFormatter https://review.opendev.org/c/openstack/python-ironicclient/+/980579 | 15:13 |
| TheJulia | Any updates regarding async io aside from the spec? | 15:13 |
| dtantsur | Not really. I'll check your comments when I have a free minute. | 15:13 |
| TheJulia | Okay, Does it need to become a PTG topic? | 15:13 |
| dtantsur | Otherwise, it slips into the next cycle for me, and I'll also keep cardoe's work to move sushy in-tree in mind | 15:13 |
| TheJulia | I guess that is a bigger question which doesn't need an immediate answer | 15:13 |
| dtantsur | Well.. maybe. At least as a status check. Maybe together with the sushy future. | 15:14 |
| TheJulia | but we should likely just sort of have a discussion around those topics | 15:14 |
| TheJulia | makes a lot of sense, could you make sure something is on the etherpad? | 15:14 |
| * cid o/ | 15:14 | |
| TheJulia | The next working group is the vxlan working group. By in large, but fixing and rough edge grinding is the order of the week. | 15:15 |
| TheJulia | Anyone have anything else to add? | 15:15 |
| JayF | I do | 15:16 |
| JayF | I would suggest, as I have, that folks communicate to stakeholders that we may need more bugfix time next cycle | 15:16 |
| JayF | just as an expectation set, given the quantity of new stuff we added last cycle and how much of that stuff touches hardware | 15:16 |
| JayF | (especially since I suspect "ironic can deliver features" expectations are at an alltime high \o/) | 15:17 |
| opendevreview | Merged openstack/ironic master: Update TBN docs for group_and_attach_ports https://review.opendev.org/c/openstack/ironic/+/980376 | 15:17 |
| TheJulia | Yeah, I think AI tooling has helped, and hurt us here. | 15:17 |
| TheJulia | Also, some of the areas we're touching really requires a dramatically deeper level of testing because its not "out of the box" in the slightest | 15:18 |
| JayF | yep | 15:18 |
| JayF | which means some of the "bugfixes" might include improving docs | 15:18 |
| JayF | or support on the list as folks implement this stuff | 15:18 |
| TheJulia | What might work just fine for someone, also may have a sharp edge for another | 15:18 |
| TheJulia | yup | 15:19 |
| JayF | I just wanna make sure we're communicating up to our management chains that we may have more work to do :D | 15:19 |
| cardoe | yeah this cycle is gonna be a lot of "digest the changes" for me. | 15:19 |
| cardoe | Like another PTG item I just thought of might be to add a "merge" mode for inspection. | 15:20 |
| TheJulia | Yeah, I need to get with hjensas and ensure we're creating items downstream for our recent bugfixes just so we're tracking the time/effort it is taking | 15:20 |
| TheJulia | cardoe: That is a good way to put it | 15:20 |
| TheJulia | "digest the changes" | 15:20 |
| hjensas | Do we want to drop the exsting periodic HA Alignment before releae? I found it is actually a no-op. | 15:20 |
| TheJulia | Anyway, Discussion Topics! | 15:20 |
| JayF | digest the changes, finish type checking on ironic repos, plan improvements for nova-compute story | 15:21 |
| JayF | nobody is allowed to talk me into more tasks at ptg ;) | 15:21 |
| TheJulia | hjensas: At this point, it can be a backported bugfix. We knew there was that risk upfront | 15:21 |
| TheJulia | #topic Discussion Topics | 15:21 |
| TheJulia | First and only item, PTG! | 15:21 |
| TheJulia | #link https://etherpad.opendev.org/p/ironic-ptg-2026.2 | 15:21 |
| TheJulia | Were building up quite a few topics, I'd encourage everyone to take a little time this week and add any items and thoughts to existing items | 15:22 |
| TheJulia | Have we reserved a "room" and times yet? | 15:23 |
| JayF | Isn't one of the DPL roles assigned to that? | 15:24 |
| TheJulia | It appears that we have not. | 15:24 |
| TheJulia | Yes, although they are not in channel at present. | 15:24 |
| JayF | Does someone want to volunteer to help schedule PTG session times and register them? | 15:26 |
| TheJulia | Well, we likely need to start to get that sorted if we want a good and consistent schedule. Some teams have already started reserving rooms. | 15:26 |
| JayF | If it's somethign you've not done before, I'm willing to show you what it takes | 15:26 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix bug in TBN NoMatch.__str__ https://review.opendev.org/c/openstack/ironic/+/980477 | 15:26 |
| JayF | it's fun to whitewash a fence | 15:26 |
| * TheJulia updates the 'twain reference counter for the day. | 15:27 | |
| cid | Sounds like something I can likely be able to do with a guide. | 15:27 |
| JayF | ++ thank you for the volunteer | 15:28 |
| JayF | we can talk it out in here after meetings | 15:28 |
| TheJulia | Next, you'll have us sitting around sipping mint juleps and negotiating while on a river boat ;) | 15:28 |
| cid | +++ | 15:28 |
| * TheJulia updates the ST:Lower Decks reference counter for the day. | 15:28 | |
| JayF | I was expecting the ST/Twain crossover to be more TNG-coded | 15:28 |
| TheJulia | Well, Lower Decks, they have to hit the entire spectrum of things. | 15:29 |
| TheJulia | Okay, onward? | 15:29 |
| TheJulia | #topic Bug Deputy Updates | 15:30 |
| TheJulia | cid, you were our deputy, what chaos do you have to report! :) | 15:30 |
| cid | Saw a lot of new bugs... :D | 15:30 |
| mumesan[m] | lol | 15:30 |
| cid | Mostly against the networking baremetal repo | 15:30 |
| TheJulia | Yeah, we knew this was going to happen on some level | 15:31 |
| cid | Yup. Out of the 10 total, 7 against NB | 15:31 |
| TheJulia | I think it would be helpful to try and have a high bandwidth call regarding that stuff today. hjensas would you have a time which could work? | 15:31 |
| TheJulia | I think we're going to just have to work through them, although I think patches are up for a number of them already | 15:32 |
| hjensas | Sure, we can do that - I am free most of the evening. | 15:32 |
| TheJulia | hjensas: ack, okay | 15:32 |
| hjensas | Most new bugs there are fixed in my proposed NB changes. | 15:32 |
| cid | I saw that. Most are just awaiting reviews | 15:33 |
| TheJulia | Cool, who would like to volunteer to be the bug deputy for this week? | 15:33 |
| Mahnoor | I could give it a go | 15:34 |
| JayF | \o/ | 15:34 |
| TheJulia | \o/ | 15:34 |
| TheJulia | #info Mahnoor has volunteered to be the bug deputy for the week! | 15:34 |
| TheJulia | Since have no RFE's to review, onward to Open Discussion | 15:34 |
| TheJulia | #topic Open Discussion | 15:34 |
| cid | Hey, Mahnoor, been a minute :-) tks | 15:34 |
| Mahnoor | It has, hello hello :) | 15:35 |
| cid | \o/ | 15:35 |
| TheJulia | So! What crazy topics of discussion do we have for today? | 15:35 |
| TheJulia | hjensas: would you be good to chat in say, 2 hours? | 15:36 |
| TheJulia | Or 2.5 hours? | 15:36 |
| JayF | We should mention the summit Call for Papers is open | 15:37 |
| JayF | if you wanna go hang out in Shanghai and talk about Ironic please submit a talk :) | 15:37 |
| hjensas | TheJulia: yes 2 hours or 2.5 hours. | 15:38 |
| TheJulia | Okay, I'll send you a meeting invite and I'll past the meet link in irc here so folks can join in | 15:39 |
| TheJulia | JayF: Great call out! | 15:39 |
| mumesan[m] | Can you send me an invite as well please 😅 | 15:40 |
| TheJulia | sure! | 15:40 |
| TheJulia | so in just under 2 hours, we'll discuss on this following gmeet link; https://meet.google.com/zqs-irew-agb?authuser=0&hs=122 | 15:43 |
| TheJulia | Invite sent, if anyone else would like an invite, please let me know | 15:43 |
| TheJulia | Thanks | 15:43 |
| TheJulia | Anything else to discuss today before we move on to figuring out who we're going to make run the next meeting? | 15:43 |
| * TheJulia takes silence as a no | 15:46 | |
| TheJulia | #topic Who will run our next meeting? | 15:46 |
| TheJulia | Volunteers? | 15:46 |
| JayF | OK, my turn again I guess :) | 15:46 |
| TheJulia | Thank you JayF for volunteering to run next week's meeting. | 15:47 |
| * JayF has a reminder set so he'll be here to run it | 15:47 | |
| TheJulia | \o/ | 15:47 |
| TheJulia | Is there anything else before we rap this week's meeting? | 15:48 |
| cid | Doesn't look like it. | 15:49 |
| TheJulia | Thank folks, have a good week! | 15:49 |
| JayF | \o | 15:50 |
| TheJulia | #endmeeting | 15:50 |
| opendevmeet | Meeting ended Mon Mar 16 15:50:42 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-03-16-15.00.html | 15:50 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-03-16-15.00.txt | 15:50 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-03-16-15.00.log.html | 15:50 |
| JayF | cid: so basically there's a few steps to doing the room bookings | 15:52 |
| JayF | cid: first of all, I always would check the times we booked last cycle and use that as a base template | 15:52 |
| JayF | cid: then you can reserve that time using the ptgbot in #openinfra-events (ptg.opendev.org has a howto for using the bot) | 15:52 |
| JayF | cid: then send an email to the list proposing those times, and asking if it works for time zones -- we can add more times that are friendly to other TZs (including yours if needed!) if folks ask | 15:53 |
| zigo | Guys, there's a missing __init__.py in sushy/oem/dell/resources/system/ | 15:53 |
| JayF | cid: other thing to consider: cross-project sessions (we appear to want one with neutron/nova) | 15:53 |
| zigo | How come it's only something that I see only in the Ironic namespace ! :) | 15:54 |
| * zigo writes a patch... | 15:54 | |
| JayF | I wasn't aware a __init__.py was a hard requirement in all cases? | 15:54 |
| TheJulia | I think for some packagers, it is :) | 15:55 |
| JayF | I'm mainly curious if this caused runtime issues or just a thing you noticed | 15:55 |
| cid | JayF, acknowledged. tks! | 15:55 |
| zigo | If there's no __init__,py, then you cannot import sushy.oem.dell.resources.system | 15:55 |
| TheJulia | oh brain, where were you | 15:55 |
| JayF | zigo: I'm mainly trying to answer the question "if this is breaky, how did it get in the codebase in the first place" | 15:55 |
| zigo | Well, failure to discover tests... | 15:55 |
| zigo | https://paste.opendev.org/show/b3QP9JYhVOJg7sC7gMU4/ | 15:55 |
| JayF | cardoe: ^ | 15:55 |
| zigo | Anyways, please give me the pleasure to make this zero char patch ! :) | 15:56 |
| zigo | Funy stuff, my last contrib to the sushy repo was: https://review.opendev.org/c/openstack/sushy/+/831515 | 15:56 |
| cardoe | Which version of Python? | 15:56 |
| zigo | Exact same thing ... | 15:56 |
| zigo | cardoe: Any version. Currently 3.14, but that's IMO, not relevant. | 15:57 |
| cardoe | hrm. I don't know how I'm not hitting it. | 15:57 |
| JayF | that's basically the question I'm trying to figure out cardoe :) | 15:58 |
| JayF | because I figured that was running downstream at your place | 15:58 |
| JayF | we should fix it anyway, but I think finding out how zigo can find these may help us in the future to detect them | 15:58 |
| cardoe | It is. | 15:58 |
| * JayF hopes the answer isn't "I read the output very carefully" | 15:59 | |
| JayF | lol | 15:59 |
| cardoe | I even looked and we don't have sushy_oem_dell installed | 15:59 |
| opendevreview | Thomas Goirand proposed openstack/sushy master: Add missing __init__.py https://review.opendev.org/c/openstack/sushy/+/980781 | 15:59 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Update release mappings for new ironic releases 34.0 and 35.0 https://review.opendev.org/c/openstack/ironic/+/980782 | 16:00 |
| cardoe | https://www.irccloud.com/pastebin/hFn5D9jK/ | 16:02 |
| cardoe | Even works locally | 16:02 |
| JayF | that imports dell | 16:02 |
| cardoe | That's what the code zigo linked is doing. | 16:03 |
| JayF | not sushy.oem.dell.resources.system | 16:03 |
| cardoe | Inside of that its importing sushy.oem.dell.resources.system | 16:03 |
| JayF | ah | 16:03 |
| zigo | cardoe: Are you saying "it works on my laptop" ? :) | 16:03 |
| * zigo runs and hides ... | 16:04 | |
| JayF | I think he's saying "it works on Rackspace Cloud control plane" | 16:04 |
| JayF | which is a bit of a stronger statement | 16:04 |
| JayF | and sounds more like "it doesn't work on /your/ laptop" ;) | 16:04 |
| cardoe | I mean I pasted it from my laptop | 16:04 |
| zigo | lol | 16:04 |
| JayF | I legitimately don't care about adding the __init__.py, I'll +2 it | 16:04 |
| cardoe | But I confirmed that the container we're using doesn't have sushy_oem_dell installed which would be the fallback import | 16:04 |
| JayF | but finding out what's different is super duper important and useful | 16:04 |
| cardoe | Yeah I'll 100% +2/+W adding the __init__.py I was just trying to understand | 16:04 |
| JayF | cardoe: lets try your test w/3.14 | 16:05 |
| cardoe | I'll have to install it. | 16:05 |
| JayF | I'm trying to get there too | 16:06 |
| JayF | we can race lol | 16:06 |
| rpittau | seems to work? | 16:06 |
| rpittau | Python 3.14.3 (main, Feb 4 2026, 00:00:00) [GCC 15.2.1 20260123 (Red Hat 15.2.1-7)] on linux | 16:06 |
| rpittau | Type "help", "copyright", "credits" or "license" for more information. | 16:06 |
| rpittau | >>> from sushy.oem.dell.resources import system | 16:06 |
| rpittau | >>> | 16:06 |
| zigo | In 2022, when I wrote https://review.opendev.org/c/openstack/sushy/+/831515 I didn't need Python 3.14 (that didn't exist) to trigger an issue... :P | 16:06 |
| JayF | can you give more detail about your testing environment? | 16:07 |
| zigo | Historically, setuptools just skiped installing .py files recursively if __init__.py is missing. Not sure these days though. | 16:07 |
| JayF | is this from an installed deb? | 16:07 |
| zigo | Yeah, sbuild. | 16:07 |
| JayF | using setuptools instead of pbr? | 16:07 |
| zigo | https://wiki.debian.org/zigo/mysbuild | 16:07 |
| zigo | No no, using pbr of course. | 16:07 |
| zigo | Gosh, hold on the +2w | 16:07 |
| JayF | the of course is not assumable :) there are downstreams that used to use !pbr for openstacky stuff | 16:07 |
| zigo | It's missing other __init__.py it seems. | 16:07 |
| JayF | just W-1 your own patch if you want to make it not-land | 16:08 |
| opendevreview | Thomas Goirand proposed openstack/sushy master: Add missing __init__.py https://review.opendev.org/c/openstack/sushy/+/980781 | 16:09 |
| JayF | I'm asking claude, seeing if maybe it finds something we missed | 16:09 |
| JayF | > if sushy uses implicit namespace packages (no __init__.py) anywhere in the chain, Debian's dh_python3 can sometimes fail to install the full tree correctly, especially when the package hierarchy is deep. | 16:10 |
| JayF | is what claude says | 16:10 |
| JayF | I'm asking it to prove it :D | 16:11 |
| TheJulia | Speaking of claude, I spotted something it couldn't see, and that makes me feellike there is hope! | 16:11 |
| cardoe | That could be it. Clearly I screwed up not including those __init__.py files. | 16:11 |
| rpittau | well it should work without those though :) | 16:12 |
| rpittau | I blame debian :D | 16:12 |
| JayF | TheJulia: have you read https://www.linkedin.com/posts/jayfaulkner_ive-thought-a-lot-about-the-last-few-days-activity-7437570841652920320-y6LN ... I sorta think if we get to the point where claude "knows more" than us, we need to stop clauding and start learning :D | 16:12 |
| JayF | I mean, it's bad of us to be inconsistent | 16:12 |
| JayF | I believe it's our bug no matter what | 16:12 |
| zigo | rpittau: Well, blame python itself, the only thing Debian is doing is "python3 -m install" ... | 16:12 |
| JayF | it just might be BOTH of us have bugs | 16:12 |
| zigo | rpittau: I don't agree it "should work". If it doesn't have an __init__.py, then it's not a module, technically speaking. | 16:13 |
| JayF | zigo: you mean python -m pip install (?) | 16:13 |
| zigo | That's by definition from Python. | 16:13 |
| zigo | JayF: No, I mean what I wrote. | 16:13 |
| rpittau | zigo: I was definitely not serious there | 16:13 |
| JayF | zigo: can you link me to upstream docs on the python install module then? I'm not familiar with it. | 16:13 |
| JayF | zigo: and can't find it, that's why I assumed type | 16:13 |
| * TheJulia very carefully reads claude's latest output and begins to wonder if someone gave sonnet 4.5 some drugs | 16:13 | |
| JayF | *type | 16:13 |
| JayF | *typo | 16:14 |
| * JayF notes that omitting __init__.py is OK for downstream modules, the problems only start coming in when they are inconsistently included | 16:14 | |
| JayF | zigo: https://github.com/pypa/setuptools/issues/1097 is suggested by claude to be a related bug, with an alternate fix. Might be good to include the nspkg.pth file as well as adding __init__.py | 16:15 |
| zigo | If you ask google "what's the difference between python package and a python module" it leads to this: | 16:15 |
| zigo | https://stackoverflow.com/questions/7948494/whats-the-difference-between-a-module-and-package-in-python | 16:15 |
| zigo | which is 100% accurate. | 16:15 |
| JayF | zigo: after reading the issue it sounds like it matches up | 16:15 |
| opendevreview | Thomas Goirand proposed openstack/sushy master: Add missing __init__.py https://review.opendev.org/c/openstack/sushy/+/980781 | 16:16 |
| dtantsur | TheJulia: why not opus? | 16:23 |
| TheJulia | Eh, I've not really found a need for opus, but it is clear that it is acting more humanlike on parsing lots of text | 16:24 |
| dtantsur | Heh, I just follow the logic "the fancier - the better" :D | 16:27 |
| JayF | opus is like asking your really smart friend who goes way in depth on everything | 16:27 |
| JayF | sometimes I just want a sentence and sonnet acts more like that | 16:27 |
| JayF | opus has ... tied itself in knots trying to do something for me before when I'm fairly sure sonnet would've said no/reprompted. It's clear claude values "uninterrupted agent time" over accuracy, and I think it's making their product worse even while it's getting smarter | 16:28 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Suppress misleading ERROR logs from OVN port lookups https://review.opendev.org/c/openstack/networking-baremetal/+/980546 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix L2VNI trunk port binding for switch config https://review.opendev.org/c/openstack/networking-baremetal/+/980547 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add OVN event-driven reconcile for L2VNI trunks https://review.opendev.org/c/openstack/networking-baremetal/+/980548 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add targeted single-VLAN reconcile for OVN events https://review.opendev.org/c/openstack/networking-baremetal/+/980549 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add VNI to L2VNI trunk subport binding profiles https://review.opendev.org/c/openstack/networking-baremetal/+/980578 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: L2VNI multi-link LAG support + fix OVN LLDP filter https://review.opendev.org/c/openstack/networking-baremetal/+/980612 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix incorrect is_partial_segment() call on PortContext https://review.opendev.org/c/openstack/networking-baremetal/+/980640 | 16:31 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Fix router interface HA binding for baremetal VLAN networks https://review.opendev.org/c/openstack/networking-baremetal/+/980648 | 16:31 |
| TheJulia | Yea, the couple times I tried opus, similar result, got in a loopy knot and I was getting frustrated. | 16:35 |
| TheJulia | tbh, even sonnet has gotten there at times, but I use a model of firm feedback when it gets like that | 16:36 |
| JayF | the newest idea I saw, which makes sense and has helped me some | 16:42 |
| JayF | is being fast to "fire" agents / sessions | 16:42 |
| JayF | if it gets too far off the path? ^c, reprompt with more explicit direction | 16:42 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: revise release notes for 2026.1 release https://review.opendev.org/c/openstack/ironic/+/980786 | 16:47 |
| TheJulia | Yeah, I have found that *generally* feedback as you go better re-enforces the session, but depending on how you access also changes the pattern entirely | 16:54 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Fix Cisco NX-OS VNI cleanup check command https://review.opendev.org/c/openstack/networking-generic-switch/+/980590 | 17:03 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Configure L2VNI mappings for trunk subports https://review.opendev.org/c/openstack/networking-generic-switch/+/980586 | 17:04 |
| JayF | TheJulia: when I found out that basically most of the chat history sessions work by just feeding the entire history of the chat back into the llm, that started to inform how I treat those kind of failure cases. Just stop it, pull out any of the good that was in there and feed it back in | 17:06 |
| JayF | I almost think of it as a way to manually edit context | 17:06 |
| TheJulia | sort of an interesting idea actually | 17:26 |
| TheJulia | in about 4 minutes, vxlan issues and fun! Lets discuss vxlan issues and solutions! meet.google.com/zqs-irew-agb | 17:27 |
| opendevreview | Merged openstack/ironic master: Enable ramdisk autodetect by default https://review.opendev.org/c/openstack/ironic/+/978602 | 17:48 |
| opendevreview | Merged openstack/ironic master: Fix bug in TBN NoMatch.__str__ https://review.opendev.org/c/openstack/ironic/+/980477 | 19:18 |
| cardoe | cid: can we make some constants of https://review.opendev.org/c/openstack/ironic/+/980153 in a follow up? Also do we need to back port? | 20:46 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/980786 <-- minor release note cleanups | 20:49 |
| cid | cardoe. Alright let me see if I can get a follow-on up in a moment. | 20:55 |
| cid | re' backportability. We can backport it just one release. | 20:55 |
| * JayF needs to get clif to point his docs-speed-magic at releasenotes target when he's back | 21:20 | |
| JayF | Next release will be my last one before I loop | 21:21 |
| JayF | when Hibiscus is released, I'll have contributed to every openstack letter | 21:22 |
| cid | yes | 21:35 |
| * cid err | 21:36 | |
| opendevreview | cid proposed openstack/ironic master: Replace boot property literals with constants https://review.opendev.org/c/openstack/ironic/+/980829 | 21:36 |
| opendevreview | Merged openstack/networking-generic-switch master: vxlan: Multicast traffic options for cisco driver https://review.opendev.org/c/openstack/networking-generic-switch/+/979332 | 21:51 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: gently increase ci callback timeouts https://review.opendev.org/c/openstack/ironic/+/980832 | 22:05 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add preamble for Gazpacho https://review.opendev.org/c/openstack/ironic/+/980833 | 22:06 |
| JayF | I went 47 rounds with restructured text to bring you that | 22:06 |
| JayF | 100% human work too, so don't ask about the AI header :D | 22:06 |
| cardoe | Look at JayF's callused fingers as a result of that effort! | 22:07 |
| TheJulia | lol | 22:08 |
| TheJulia | JayF: one minor issue on that change which is a quick correction | 22:08 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: docs: l2vni - Clean up admin documentation https://review.opendev.org/c/openstack/networking-baremetal/+/980835 | 22:08 |
| opendevreview | Verification of a change to openstack/ironic master failed: Update release mappings for new ironic releases 34.0 and 35.0 https://review.opendev.org/c/openstack/ironic/+/980782 | 22:21 |
| JayF | I also apparently spelled remote as remove so I'll fix that as well when I get back to a desk. Walking the dog right now. | 22:27 |
| JayF | I also owe you a review on your release notes work which I will get before my EOD | 22:27 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add preamble for Gazpacho https://review.opendev.org/c/openstack/ironic/+/980833 | 22:39 |
| JayF | TheJulia: quick one for you, too https://review.opendev.org/c/openstack/ironic/+/980786/1#message-a53267bc4746a6fc6092bb21425f307c9467be08 | 22:40 |
| JayF | TheJulia: if you are already away from pc lmk and I can edit | 22:41 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: revise release notes for 2026.1 release https://review.opendev.org/c/openstack/ironic/+/980786 | 22:46 |
| TheJulia | done | 22:46 |
| TheJulia | lol | 22:46 |
| TheJulia | My brain didn't catch remove, it just kind of went "thats awkward!" | 22:46 |
| TheJulia | doh | 22:46 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add preamble for Gazpacho https://review.opendev.org/c/openstack/ironic/+/980833 | 22:50 |
| JayF | found another one where I accidentally a word | 22:50 |
| TheJulia | heh | 22:52 |
| TheJulia | JayF: you have a spelling issue in that | 23:57 |
| TheJulia | tox -epep8 | 23:57 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Correct disk-format in ramdisk boot docs https://review.opendev.org/c/openstack/ironic/+/980333 | 23:59 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!