| opendevreview | Clif Houck proposed openstack/ironic master: Add a new 'class' field to the Port object https://review.opendev.org/c/openstack/ironic/+/955447 | 01:16 |
|---|---|---|
| opendevreview | Clif Houck proposed openstack/ironic master: Add a new 'class' field to the Port object https://review.opendev.org/c/openstack/ironic/+/955447 | 01:49 |
| rpittau | good morning ironic! o/ | 06:13 |
| queensly[m] | Good morning. | 06:33 |
| FreemanBoss[m] | Good morning o/ | 08:06 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Build ironic-python-agent DIB image with CentOS Stream 10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/950965 | 08:08 |
| iurygregory | good morning ironic | 10:58 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Build ironic-python-agent DIB image with CentOS Stream 10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/950965 | 11:27 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Skip initial reboot to IPA when updating firmware out-of-band https://review.opendev.org/c/openstack/ironic/+/954311 | 11:28 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix wrong result of filter_device in the base interface https://review.opendev.org/c/openstack/ironic-python-agent/+/955465 | 11:51 |
| dtantsur | my bad folks, this needs urgent reviews ^^^ | 11:52 |
| dtantsur | frickler: this is your problem too ^^^ | 11:52 |
| opendevreview | cid proposed openstack/ironic-python-agent master: doc: How hardware managers ignore certain devices https://review.opendev.org/c/openstack/ironic-python-agent/+/955469 | 12:20 |
| cid | oh, dtantsur, :facepalm: ^^ | 12:21 |
| cid | So, I missed that subtle gap, but I think the doc update is still valid, kindly take a look when you some free cycle. | 12:28 |
| guilhermesp | hey team! mornings! Im trying to use https://docs.openstack.org/ironic/latest/admin/cleaning.html#configuring-automated-cleaning-with-runbooks but as it is stating in the docs, we are just going to be able to use them with automated clean in flamingo? I can use it for manual cleaning in 2024.2 cc cid :P | 12:32 |
| cid | guilhermesp. \o, good morning. | 12:34 |
| cid | Indeed, you should be able to use it for manual cleaning in the Flamingo release | 12:36 |
| guilhermesp | morning cid ! Automated you mean? coz i did test it with manual cleaning in 2024.2 these last days | 12:38 |
| guilhermesp | and if runbooks + automated clean is tested in the current main ( since i can see options in the docs to use it ) -- im willing to go for main now lol | 12:41 |
| guilhermesp | becasue that fits my use case here perfectly | 12:41 |
| guilhermesp | which is running all of these clean steps on every commission/decommission of a node: | 12:42 |
| guilhermesp | https://www.irccloud.com/pastebin/seKNblrP/ | 12:42 |
| guilhermesp | i could not figure how to add apply_configuration on the clean steps priority | 12:42 |
| cid | So, automated cleaning works for both automated and manual cleaning. Basically, it's runbook in an automated fashion. | 12:43 |
| cid | And runbook works for both use cases already. | 12:44 |
| guilhermesp | huuuum i see... i tried to do this: | 12:45 |
| guilhermesp | https://www.irccloud.com/pastebin/NDfuMM6k/ | 12:46 |
| guilhermesp | and just added the trait to the node but its just bypassing it | 12:46 |
| guilhermesp | thats on 2024.2 yeah | 12:46 |
| cid | Ok, let's try to walk through it. First, this is a good place to start, https://docs.openstack.org/ironic/2025.1/admin/runbooks.html on the creation of a runbook. Any gaps you identify will make a good LP. | 12:47 |
| cid | By 'bypassing' do you mean cleaning doesn't start? | 12:48 |
| guilhermesp | yup, runbook its there in my env and works perfectly when I do: openstack baremetal node clean --runbook CUSTOM_BASELINE_BIOS 57139cfd-d347-4c71-b9d1-4ad64a5adc4f | 12:49 |
| guilhermesp | but | 12:49 |
| guilhermesp | if i move a node from manage to provide... i just see erease_express taking place which kinda of tell me that its just doing the default steps for automated clean I guess | 12:49 |
| cid | Yup. That looks like one of the runbook's steps(?) | 12:51 |
| guilhermesp | yeah, but why does it run all when I do it manually and just that one when doing automated? that's what was kinda of confused thinking that somehow it was like...just "bypassing it" | 12:51 |
| cid | Oh, so only one step (that one step, or just any one step) get's executed with automated cleaning with runbook | 12:54 |
| guilhermesp | just erase_devices_express | 12:54 |
| guilhermesp | at least on the 4 tests i did | 12:55 |
| cid | Ok, got it. | 12:55 |
| guilhermesp | moving the node from manage -> provide | 12:55 |
| guilhermesp | and then when i read this https://docs.openstack.org/ironic/latest/admin/cleaning.html#configuring-automated-cleaning-with-runbooks | 12:55 |
| guilhermesp | i was like "well, i think im gonna get this just on flamingo " lol | 12:55 |
| cid | Well, you're technically using flamingo only in dev mode :P. | 12:58 |
| cid | I will take a look at the code and see if anything is out of the ordinary. If anything also jumps out to you based on teh logs, that could help, maybe logs. | 12:59 |
| cid | Testing is the best way to confirm what's going going, maybe some filtering. | 12:59 |
| cid | ... is happneing somewhere | 13:00 |
| guilhermesp | heheheh yeah i can enable debug and get outputs when doing automated | 13:00 |
| guilhermesp | i was just wondering if that was tested in the current main so I could bump images here on my end and see if with the current main things might work | 13:01 |
| cid | I was under the impression you were already using the main branch. If that's not the case, then use the feature will only be available in the main branch and/or flamingo release. | 13:08 |
| cid | So there's automated cleaning which already works, and there's manual cleaning. As of 2024.2 you can do both of those with a runbook. And in Flamingo you can automate both under the new 'automated cleaning with runbook' feature. | 13:08 |
| guilhermesp | yeah sorry for the confusion. Im actually using 2024.2 but open to use main just to test | 13:10 |
| guilhermesp | what im doing here to contribute is enabling debug and pasting my configs + node definition. I can get those shortly | 13:11 |
| cid | Ok great. | 13:12 |
| cid | Yeah. So, only runbook is available in your current 2024.2. | 13:14 |
| cid | Automated cleaning with runbook is not until Flamingo. | 13:14 |
| cid | And yes, so far so good, these have been tested. | 13:14 |
| guilhermesp | ah ok then, so my tests here with 2024.2 wont probably do good. but yeah if those has been tested on the current main, I guess I would try bumping my ironic to main and see how it goes | 13:21 |
| frickler | dtantsur: thx, I had just gotten to the point that I could confirm that reverting https://review.opendev.org/c/openstack/ironic-python-agent/+/954024 does resolve my issue. I can test with your fix tomorrow | 13:26 |
| cid | guilhermesp, ++ | 13:30 |
| opendevreview | cid proposed openstack/ironic-python-agent master: doc: How hardware managers ignore certain devices https://review.opendev.org/c/openstack/ironic-python-agent/+/955469 | 13:44 |
| opendevreview | cid proposed openstack/ironic-python-agent master: doc: How hardware managers ignore certain devices https://review.opendev.org/c/openstack/ironic-python-agent/+/955469 | 13:47 |
| TheJulia | good morning | 13:49 |
| dtantsur | morning TheJulia | 13:58 |
| cardoe | morning Ironic | 14:25 |
| cardoe | give someone a high five if they review and +W https://review.opendev.org/c/openstack/ironic/+/953414 cause I'm trying to rebase a patch on top and Depends-On isn't working | 14:26 |
| dtantsur | done | 14:34 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove rootwrap https://review.opendev.org/c/openstack/ironic/+/953404 | 14:34 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Use specific util methods in vfat image creation https://review.opendev.org/c/openstack/ironic/+/953417 | 14:34 |
| opendevreview | Nahian Pathan proposed openstack/ironic master: Reduce API calls when collecting sensor data with redfish https://review.opendev.org/c/openstack/ironic/+/955484 | 14:35 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove rootwrap https://review.opendev.org/c/openstack/ironic/+/953404 | 14:38 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Use specific util methods in vfat image creation https://review.opendev.org/c/openstack/ironic/+/953417 | 14:38 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix wrong result of filter_device in the base interface https://review.opendev.org/c/openstack/ironic-python-agent/+/955465 | 14:39 |
| opendevreview | Nahian Pathan proposed openstack/ironic master: Reduce API calls when collecting sensor data with redfish https://review.opendev.org/c/openstack/ironic/+/955484 | 14:42 |
| opendevreview | cid proposed openstack/ironic master: Clear `last_error` on power match/sync https://review.opendev.org/c/openstack/ironic/+/955432 | 14:43 |
| opendevreview | Nahian Pathan proposed openstack/ironic master: Reduce API calls when collecting sensor data with redfish https://review.opendev.org/c/openstack/ironic/+/955484 | 14:44 |
| cardoe | Thank you :) | 14:49 |
| TheJulia | cid: a unit test or three on that last_error change please :) | 14:55 |
| opendevreview | Merged openstack/sushy master: Optimize unit testing https://review.opendev.org/c/openstack/sushy/+/954691 | 14:57 |
| cid | TheJulia. Alright, let me try add some :). tks | 14:58 |
| TheJulia | thanks! | 14:58 |
| TheJulia | #startmeeting ironic | 15:01 |
| opendevmeet | Meeting started Mon Jul 21 15:01:47 2025 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:01 |
| TheJulia | o/ | 15:01 |
| rpittau | #startmeeting ironic gaaaaa | 15:01 |
| opendevmeet | rpittau: Error: Can't start another meeting, one is in progress. Use #endmeeting first. | 15:01 |
| alegacy | o/ | 15:01 |
| rpittau | o/ | 15:01 |
| dtantsur | o/ | 15:02 |
| masghar | o/ | 15:02 |
| TheJulia | #chair rpittau | 15:02 |
| opendevmeet | Current chairs: TheJulia rpittau | 15:02 |
| rpittau | thanks :D | 15:02 |
| * TheJulia lets rpittau chair ;) | 15:02 | |
| iurygregory | o/ | 15:02 |
| rpittau | Hello everyone! | 15:02 |
| rpittau | Welcome to our weekly meeting! | 15:02 |
| cid | o/ | 15:02 |
| clif | o/ | 15:02 |
| rpittau | The meeting agenda can be found here: | 15:02 |
| rpittau | https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_July_21.2C_2025 | 15:02 |
| queensly[m] | o/ | 15:02 |
| rpittau | lots of people today, good good :) | 15:03 |
| rpittau | #topic Announcements/Reminders | 15:03 |
| rpittau | Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: | 15:03 |
| rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| rpittau | quite some patches with +2 missing +W | 15:03 |
| rpittau | there's also an intruder patch from neutron :D | 15:04 |
| TheJulia | I commented on the intruder ;) | 15:04 |
| rpittau | thanks! | 15:04 |
| TheJulia | specifically to encourage it | 15:04 |
| rpittau | I guess we'll most of the patches there merging soon | 15:05 |
| rpittau | forgot a "see" there | 15:05 |
| rpittau | anyhow! | 15:06 |
| rpittau | 2025.2 Flamingo Release Schedule | 15:06 |
| rpittau | #link https://releases.openstack.org/flamingo/schedule.html | 15:06 |
| rpittau | we're in the second half of the dev cycle | 15:06 |
| rpittau | at R-10 | 15:06 |
| rpittau | just 10 weeks to go | 15:06 |
| rpittau | I'm going to propose bugfix branches this week | 15:07 |
| rpittau | anything else to announce/remind ? | 15:07 |
| TheJulia | Okay, that will likely be the last bugfix/branch with eventlet if we're able to move futurist changes along | 15:07 |
| rpittau | oh yeah | 15:07 |
| rpittau | right | 15:07 |
| rpittau | great! | 15:07 |
| rpittau | (had so many issues downstream with eventlet) | 15:08 |
| TheJulia | we can talk about eventlet when we get there | 15:08 |
| rpittau | yeah | 15:08 |
| rpittau | moving on! | 15:08 |
| rpittau | #topic Working Group Updates | 15:08 |
| rpittau | any update from standalone networking? | 15:08 |
| alegacy | yep, making good progress here. a few things I want to raise here today. | 15:09 |
| alegacy | First, I've got things roughed in from the API down to the switch. Wondering if there's any interest in a short demo to validate the approach before I start cleaning things up and submitting patches | 15:09 |
| TheJulia | a quick demo would be awesome | 15:10 |
| TheJulia | but at your convenience obviously | 15:10 |
| cardoe | I had a question about some of the specs that seem to be tied in. | 15:10 |
| alegacy | k, i'll follow-up on this outside of meeting to find a suitable time. | 15:10 |
| cardoe | We've got specs for port binding that reference the standalone work. | 15:10 |
| cardoe | But there's almost two competing ones. | 15:11 |
| cardoe | There's adding of a "vendor" field but no clarity on how that will be used or defined. | 15:11 |
| TheJulia | I wouldn't describe them as competing | 15:11 |
| TheJulia | Its entirely differnet problem spaces and use patterns | 15:11 |
| opendevreview | Merged openstack/ironic-python-agent master: doc: How hardware managers ignore certain devices https://review.opendev.org/c/openstack/ironic-python-agent/+/955469 | 15:11 |
| alegacy | ya, i didn't think there would be any overlap there | 15:11 |
| TheJulia | For example, one pattern is nova integrated, another is "operator knows what they want" | 15:11 |
| TheJulia | another is "operator is asking for baremetal and asking nova to bind ports and is explicitly using tags because they know all the secrets in between because they the operator is the user, not an end user" | 15:12 |
| TheJulia | And yes, that is all very confusing and not obvious | 15:12 |
| * TheJulia would be happy to go rev specs with clarifying text, fwiw | 15:13 | |
| cardoe | So along the lines of the demo. Just wanted to get more definition on use case and also consideration of failure case. | 15:13 |
| alegacy | use case for the demo specifically... or use cases for this feature in general? | 15:13 |
| cardoe | But maybe I'm speaking to the wrong section. | 15:13 |
| cardoe | Not for the demo specifically. | 15:15 |
| cardoe | I derailed the convo. Not my intention back to being quiet. | 15:15 |
| rpittau | no worries :) | 15:15 |
| alegacy | :) | 15:15 |
| rpittau | should we move on? | 15:15 |
| alegacy | a couple more things from me... | 15:15 |
| rpittau | alegacy: please :) | 15:15 |
| TheJulia | heh, cardoe your pointing out an easy to confuse area, I'll sync up with you after the meeting | 15:16 |
| alegacy | I've been using bifrost to do some testing. | 15:16 |
| alegacy | but it doesn't have any support for virtual switches that can be controlled by ngs. | 15:16 |
| alegacy | has there been any discussions in the past to add OVS into the bifrost testenv mix? | 15:16 |
| alegacy | is that a good/bad idea to propose? | 15:17 |
| TheJulia | It seems possible, we have some of the base structure in the ironic devstack plugin. Honestly there is no real reason not to aside from just its a bit more complexity. I think when I last approached adding neutron support in bifrost I ended up with a bunch of playbooks some of which did similar things but I also ended up building a lab locally with hardware | 15:18 |
| alegacy | ok, maybe we can discuss this a bit more outside of this meeting. likely need a plan to CI this feature... | 15:19 |
| alegacy | I'll take a look at what it would take. | 15:19 |
| TheJulia | it would make the most sense to do such for CI, specifically | 15:19 |
| alegacy | ok | 15:20 |
| alegacy | 1 more topic from me. | 15:20 |
| alegacy | Looking at the node transitions and the calls to add_*_network... it looks like there are states that either don't expect any network access or assume that the switch is already setup. | 15:20 |
| alegacy | For instance, when the node is available it doesn't look like there are any calls to setup any network stuff. | 15:21 |
| TheJulia | Correct | 15:21 |
| TheJulia | it is considered detached | 15:21 |
| alegacy | am I reading that right? Is network access not required/expected in that (and other) state(s). | 15:21 |
| alegacy | ah | 15:21 |
| alegacy | also, when it is first enrolled... same? no net access required/expected? | 15:22 |
| TheJulia | correct, we don't want to leave the node attached needlessly in case it gets powered on or someone also sets up a DHCP+PXE server in a specific network (outside of our control) and tries or manages to make use of the machines | 15:22 |
| TheJulia | yeah, no networking generally expected. | 15:22 |
| alegacy | ok, that explains what I have been seeing. Thought maybe there were some missing hooks somewhere. | 15:23 |
| alegacy | thanks | 15:23 |
| TheJulia | dtantsur: you might have thoughts here given.... I'm drawing a blank on what the protocol is in the bmc layer to share an ethernet port with the bmc | 15:23 |
| TheJulia | dtantsur: I blanked it from my mind in horror.... | 15:23 |
| * dtantsur reads scrollback | 15:23 | |
| alegacy | that's it from me for standalone networking | 15:24 |
| rpittau | ack, thanks alegacy | 15:24 |
| dtantsur | attachment in available depends on fast track | 15:24 |
| dtantsur | but this should be handled by Ironic already | 15:24 |
| alegacy | k, i'll double check and follow-up with you offline | 15:24 |
| cardoe | You need some kind of "safe" state. | 15:25 |
| rpittau | anythign else for the standalone networking ? | 15:25 |
| jrosser | TheJulia: nc-si ? | 15:25 |
| TheJulia | jrosser: that! | 15:25 |
| * TheJulia hides in terror | 15:25 | |
| rpittau | lol | 15:25 |
| cardoe | But the question for "safe" state is do you want it configured so that LLDP works so that inspection works to find the linkage? | 15:25 |
| alegacy | inspecting doesn't apply a network config | 15:26 |
| dtantsur | jrosser: oh noes | 15:26 |
| alegacy | sorry, does | 15:26 |
| TheJulia | cardoe: inspection is an entirely separate state in the state machine, and also doesn't really apply networking config | 15:26 |
| alegacy | it does apply network config | 15:26 |
| dtantsur | yeah, there is inspection_network and all the business | 15:26 |
| TheJulia | Yeah, exactly | 15:27 |
| TheJulia | So onward to eventlet removal? | 15:27 |
| rpittau | yep | 15:27 |
| TheJulia | So, regarding eventlet removal, we're super close, but also have a little bit of a ways. We're in the phase of finding the things out we didn't know or expect, and overall we seem to be heading in a good direction. dtantsur has been awesome and driving forward a change in futurist which we'll end up using. The profile will be a little different but I don't think we're going to have to do much more aside put some warnings | 15:28 |
| TheJulia | about increased memory usage | 15:28 |
| * dtantsur nods | 15:28 | |
| TheJulia | and by usage, I mean more around allocations. I fully expect we might find some things we never expected either, but one step at a time. | 15:28 |
| TheJulia | (not the allocations feature, but thread launch reserved stack size, which I'm thinking of setting to the smallest available value) | 15:29 |
| TheJulia | Any questions regarding eventlet removal? | 15:30 |
| rpittau | looks like we're good | 15:31 |
| rpittau | #topic Discussions topics | 15:32 |
| rpittau | we have 2 topics today | 15:32 |
| rpittau | first one from queensly[m] :) | 15:32 |
| queensly[m] | Hi everyone, I’m 8 weeks into the Outreachy internship and would like to give an update on my project. I’m working on setting a bare metal host's clock remotely using Redfish.... (full message at <https://matrix.org/oftc/media/v1/media/download/Adi0_wK6ZUl3pIczSde2O0zEYgSL0rwg6DoU5WekkLulsOjibVB4GC349Js5zzCHpLBD9jl8oFfTS2fQQOWqHIZCeYdbcJbgAG1hdHJpeC5vcmcvcUhjSlpJekpDaVFIbVpIUGxKd05qY1Jy>) | 15:33 |
| TheJulia | queensly[m]: cool, how can we be of assistance? | 15:34 |
| queensly[m] | So far, I have done the following:... (full message at <https://matrix.org/oftc/media/v1/media/download/AWJV0NBsoIoK-tlCz01-HwjVZCVDAfWfgBH2GdpoKl0eOj5AalTRt1-SmWwwyhEvOVXYvfCkI12L6XC1DkmzZ9BCeYdbhtrQAG1hdHJpeC5vcmcvTFloUk5MT1VpSUxkV2xqU01sbmxxWlNM>) | 15:34 |
| * cid I just blinked and we moved on already D. I was just about to ask, the checkpointed task approach, that's till on the table(?). I'm getting around a PoC this week. TheJulia, dtantsur. I'm effectively playing catching currently with all things eventlet as a result of doing non-eventlet removal things the past week. | 15:35 | |
| queensly[m] | TheJulia thanks. I need reviews on my current patches. I think that would help me know what to add or remove. | 15:36 |
| dtantsur | cid: I'm not sure what you're referring to (re checkpointed tasks) | 15:37 |
| cid | You're doing great, queensly[m], o/ | 15:37 |
| TheJulia | cid: I think it is something we might want to look at and consider, but I don't think that is something we need to do right now(). It does solve a lot of things later on and with some more complex flows thought | 15:37 |
| dtantsur | ahh, remembered | 15:37 |
| queensly[m] | cid: Thank you | 15:38 |
| dtantsur | I agree with Julia, we need this, but it's not a blocker priority | 15:38 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add Tempest API tests for inspection rules https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951761 | 15:38 |
| TheJulia | queensly[m]: Yes, your doing great. I'll try to look at your ironic change later today | 15:38 |
| cid | Oh, good thing I asked. | 15:38 |
| queensly[m] | Alright thank you. | 15:39 |
| TheJulia | cid: great you asked, but yeah, ironic works surprisingly well at the core, and oddly even better in some ways, but the big issue we've spotted is the power state sync | 15:39 |
| TheJulia | And... it is distinctly possible we'll find *more* issues as time moves on | 15:39 |
| rpittau | thanks queensly[m] for your contributions :) | 15:39 |
| TheJulia | but they might not be "in the next 2 months" sort of efforts | 15:39 |
| TheJulia | but they might be "oh wow, this is bad, we need to backport this thing which feels like a feature but it fixes a bug" | 15:40 |
| queensly[m] | rpittau: thanks :) | 15:40 |
| cid | TheJulia, that 'll already be a heavy lift, :) | 15:40 |
| TheJulia | Indeed, but we're definitely heading into a better place overall by removing eventlet. | 15:41 |
| rpittau | I agree! :D | 15:41 |
| queensly[m] | I have the following ongoing:... (full message at <https://matrix.org/oftc/media/v1/media/download/Adh3F3hIIZ2UUE3bxIOlu2cpI4-PMPyZsf6BaqsG_dcJKYZ4D515tjJxM-aCUwE-LC44ND3epGxyAfymzIXxhaxCeYdb7TcAAG1hdHJpeC5vcmcvQ3lBVXZuQ21QSFFNcVNYWnpPcWFFekt2>) | 15:41 |
| dtantsur | meanwhile, I've added https://review.opendev.org/c/openstack/futurist/+/955217 to our list, and more eyes are welcome | 15:41 |
| cid | Kudos, TheJulia, dtantsur, just ping me if you think there's anywhere I could help out with. | 15:42 |
| cardoe | speeaking of bifrost... what do we want to do with https://review.opendev.org/c/openstack/bifrost/+/946724 ? it's sitting in the weekly queue for a bit. | 15:42 |
| TheJulia | dtantsur: I'll spin it up after the meeting | 15:42 |
| rpittau | shoud lwe move on? | 15:43 |
| queensly[m] | I would like to thank my mentors rpittau and masghar for their support so far and also to the community. I'm happy to answer any questions or suggestions you may have. | 15:43 |
| rpittau | :) | 15:43 |
| queensly[m] | rpittau: Yeah, I'm done with my update. | 15:43 |
| rpittau | thanks! | 15:43 |
| cid | tks | 15:43 |
| masghar | thanks for your contributions queensly :) | 15:44 |
| queensly[m] | You're welcome :) | 15:44 |
| rpittau | next topic is from me, it's just yet another headsup that we're going to remove support for python 3.9 soon | 15:45 |
| rpittau | ideally after the next bugfix branches are cut | 15:45 |
| rpittau | what's missing now is just moving cs9 jobs in bifrost to non-voting | 15:45 |
| TheJulia | Are we pinning the CS10 usage to specific providers? | 15:45 |
| rpittau | related to that, the ipa-builder patch to build ipa ramdisk with CS10 is now ok with CI, the jobs are pinned to el10 hardware compatible nodes https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/950965 | 15:45 |
| TheJulia | cool cool | 15:46 |
| rpittau | a reminder that we're using python 3.12 in the current CS9 ipa ramdisks | 15:47 |
| rpittau | so migrating to CS10 is not super urgent | 15:48 |
| rpittau | I think that's all | 15:48 |
| rpittau | any questions/doubts/concerns? | 15:48 |
| TheJulia | none on my end | 15:48 |
| rpittau | ok, moving on! | 15:49 |
| rpittau | anything for BUG updates or RFEs ? | 15:49 |
| rpittau | alright! | 15:51 |
| rpittau | any Open Discussion topics? | 15:51 |
| TheJulia | I don't have anything right now | 15:52 |
| rpittau | then I'm declaring the meeting over! | 15:52 |
| rpittau | Thanks everyone! | 15:52 |
| rpittau | #endmeeting | 15:52 |
| opendevmeet | Meeting ended Mon Jul 21 15:52:47 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-07-21-15.01.html | 15:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-07-21-15.01.txt | 15:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-07-21-15.01.log.html | 15:52 |
| cid | Nothing else, zoomed out a little before bug deputy updates. There were just two bugs. 0 RFEs. | 15:53 |
| cid | \o/ | 15:53 |
| TheJulia | cardoe: back to your concern from what you viewed as competing specs. Did resetting context help or is there a concern beyond base confusion? | 15:53 |
| cardoe | nah I need to go back and look at it. | 15:54 |
| cardoe | I was more just thinking about the operations for standalone when I was reading the stuff on adding the vendor field. | 15:55 |
| cardoe | And thinking of the standalone case and just thinking about bad ports or ports under maintenance. | 15:55 |
| cardoe | and just all the operations or state checks | 15:55 |
| TheJulia | you did raise a super good point, we need an "enabled" field | 15:56 |
| cardoe | I think dtantsur was trying to skip adding a port because he doesn't want it to ever be used. But I was thinking that an inspection rule might be easier to always disable those ports. | 15:58 |
| dtantsur | btw I ended up with a more nuclear option of ignoring such ports on IPA level entirely | 15:59 |
| cardoe | And I started thinking about writing down some cases on an etherpad but that didn't seem right. | 15:59 |
| TheJulia | I think it makes sense to denote the case where you know of a port, but you must administratively disable it | 16:03 |
| opendevreview | Merged openstack/ironic master: Add interface name to inventory for inspection hooks https://review.opendev.org/c/openstack/ironic/+/953414 | 16:03 |
| TheJulia | and by administratively disable it, I mean tell ironic to ginroe it | 16:03 |
| TheJulia | to ignore it | 16:03 |
| iurygregory | tks for the +2 dtantsur o/ https://review.opendev.org/c/openstack/ironic/+/953394 | 16:10 |
| TheJulia | cardoe: I guess, it does make sense to write down some cases, but at the same time I wouldn't want to discount other cases because everyone is looking issues or challenges before them through their own lens | 16:14 |
| TheJulia | cardoe: sort of why I was like, "okay, there is a use case disconnect" early on | 16:14 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Switch from local RPC to automated JSON RPC on localhost https://review.opendev.org/c/openstack/ironic/+/954755 | 16:16 |
| clif | TheJulia: "class" is a keyword in Python, which causes a bit of a headache when trying to add it as an attribute to Port objects. Any objection to changing the attribute name from 'class' to 'category' or 'classification'? | 16:16 |
| clif | This is in regard to trait based port scheduling | 16:16 |
| TheJulia | clif: no objection to any suggested workaround to that :) | 16:16 |
| clif | ty | 16:16 |
| dtantsur | TheJulia: I think https://review.opendev.org/c/openstack/ironic/+/954755 is ready | 16:18 |
| opendevreview | Merged openstack/sushy-tools master: Validate JSON content type before parsing manager PATCH requests https://review.opendev.org/c/openstack/sushy-tools/+/954945 | 16:20 |
| frickler | soo ... more fun with cisco it seems ... looks like they consider it an error to try to forceoff a server that is already off. this means after a successful deploy the ironic-triggered forceoff races with the ipa-triggered shutdown and sometimes fails | 16:49 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a new 'category' field to the Port object https://review.opendev.org/c/openstack/ironic/+/955447 | 16:59 |
| iurygregory | cardoe, tks for the review, I have a doc appointment, when I'm back I`ll push a separate patch with the docs update | 17:13 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Updated pinned upper-constraints for Python 3.9 https://review.opendev.org/c/openstack/bifrost/+/955181 | 17:16 |
| TheJulia | frickler: ugh, ugh | 17:19 |
| TheJulia | frickler: #patcheswelcome ? :) | 17:19 |
| opendevreview | Merged openstack/ironic master: Update firmware schema to require 'created_at' and 'updated_at' fields https://review.opendev.org/c/openstack/ironic/+/953352 | 17:21 |
| frickler | TheJulia: yeah, I guess so ;) working on it | 17:25 |
| cardoe | iurygregory: sounds good | 17:33 |
| TheJulia | queensly[m]: I noticed one minor issue | 17:37 |
| TheJulia | queensly[m]: just you'll need to update requirements.txt | 17:38 |
| queensly[m] | TheJulia: Thanks for the heads up. I will update the requirements.txt file to set the minimum sushy version. | 17:53 |
| TheJulia | frickler: if task.node.properties['vendor'] is getting set to cisco (we would hope, but we've seen some OEMs put garbage responses in those values like some vendors just put numbers... then you can base logic on it | 17:55 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 18:20 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 20:52 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix up redfish inspection mock data https://review.opendev.org/c/openstack/ironic/+/955536 | 20:52 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish processor inspection https://review.opendev.org/c/openstack/ironic/+/955537 | 20:52 |
| opendevreview | cid proposed openstack/ironic master: Clear `last_error` on power match/sync https://review.opendev.org/c/openstack/ironic/+/955432 | 21:02 |
| clif | Anyone know if portgroup should have a `hide_fields_in_newer_versions` function? It's present for allocation, derive, node, and port, but not portgroup. | 21:09 |
| clif | driver* | 21:09 |
| clif | and that function only appears to be tested directly for ports in unit tests | 21:09 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish processor inspection https://review.opendev.org/c/openstack/ironic/+/955537 | 21:14 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 21:14 |
| cardoe | dtantsur: I dunno how to call / use default_hooks like you want in that change so I need some feedback/help. | 21:18 |
| cardoe | clif: likely never been needed before? | 21:18 |
| clif | maybe, but portgroup is on version 1.5 so it's undergone some revision in the past I assume? | 21:18 |
| clif | if I'm adding a new field/attribute to Portgroup should I go ahead and also implement a new `hide_fields_in_newer_versions` for Portgroup? | 21:20 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish processor inspection https://review.opendev.org/c/openstack/ironic/+/955537 | 21:52 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 21:52 |
| cardoe | Sure | 21:52 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Hard stop on image download duration threshold https://review.opendev.org/c/openstack/ironic-python-agent/+/954245 | 21:56 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish processor inspection https://review.opendev.org/c/openstack/ironic/+/955537 | 22:43 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 22:43 |
| TheJulia | clif: likely should be there so it is hidden as appropriate for older versions, in reality most operators don't seem to do pinned rpc upgrades | 23:26 |
| cardoe | I hate mock so much right now. | 23:40 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!