| opendevreview | cid proposed openstack/ironic master: WIP: Redfish VirtualMedia NFS and SMB/CIFS Support https://review.opendev.org/c/openstack/ironic/+/958505 | 00:44 |
|---|---|---|
| *** hroy_ is now known as hroy | 02:36 | |
| rpittau | good morning ironic! o/ | 07:04 |
| kubajj | good morning rpittau, and ironic o/ | 07:20 |
| rpittau | hey kubajj :) | 07:21 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [DNM] TEST CI https://review.opendev.org/c/openstack/bifrost/+/968791 | 07:33 |
| rpittau | new ovmf packages are out for noble and jammy, it looks like the issue with uefi has been fixed | 07:38 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/966466 | 09:05 |
| rpittau | adn we're good again to switch to DIB https://review.opendev.org/c/openstack/bifrost/+/964404 \o/ | 10:13 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/966466 | 10:26 |
| dtantsur | rpittau: job names still need fixing, everything else looks good | 10:29 |
| rpittau | /o\ | 10:32 |
| dtantsur | A new idea for today's RFE review: https://bugs.launchpad.net/ironic/+bug/2133499 | 10:47 |
| rpittau | dtantsur: just to clarify, I'm going to remove dibipa from all the jobs as we don't have other methods for ipa ramdisk now, and add "on" to indicate the node, and "src" to indicate we build the ipa image and we dont use the pre-built one | 10:52 |
| rpittau | so we'll ahve | 10:52 |
| rpittau | bifrost-integration-dibipa-debian-centos-10 -> bifrost-integration-debian-src-on-centos-10 | 10:52 |
| rpittau | bifrost-integration-dibipa-centos-10 -> bifrost-integration-on-centos-10 | 10:52 |
| rpittau | suggestions welcome for other renaming! :D | 10:52 |
| dtantsur | s/src/ipasrc/ otherwise I quite like this! | 10:54 |
| rpittau | perfect! :D | 11:01 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix slow test_find_routable_addr_no_reachable_urls https://review.opendev.org/c/openstack/ironic-python-agent/+/968488 | 12:02 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 13:13 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 13:27 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove tinyipa support and switch to DIB-based IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 14:01 |
| cid | So, I tagged Kolla on this bug, is really a Kolla bug, https://bugs.launchpad.net/ironic/+bug/2132960 | 14:01 |
| rpittau | cid: indeed | 14:02 |
| cid | Alright, thanks :D | 14:02 |
| rpittau | :) | 14:02 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use new bifrost ci job name https://review.opendev.org/c/openstack/ironic/+/969077 | 14:07 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Removed unused tinyipa CI jobs https://review.opendev.org/c/openstack/bifrost/+/969083 | 14:15 |
| TheJulia | Good morning | 14:18 |
| opendevreview | Merged openstack/ironic master: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/966466 | 14:20 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 14:26 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 14:29 |
| janders | ^ kubajj apologies I added depends-on in the wrong patch | 14:30 |
| janders | (this push removed your review vote) | 14:30 |
| * TheJulia blinks | 14:30 | |
| dtantsur | morning TheJulia | 14:31 |
| cardoe | Hello all. | 14:59 |
| TheJulia | good morning! | 14:59 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/968418 if someone will nudge that. I'll start the backports. | 14:59 |
| TheJulia | Who is going to be running today's meeting | 14:59 |
| JayF | I can but I will be eating breakfast while doing so :) | 15:00 |
| TheJulia | This is fine. | 15:00 |
| TheJulia | :) | 15:00 |
| JayF | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Dec 1 15:00:34 2025 UTC and is due to finish in 60 minutes. The chair is JayF. 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 | I've not had breakfast much less coffee yet :) | 15:00 |
| dtantsur | o/ | 15:00 |
| janders | o/ | 15:00 |
| kubajj | o/ | 15:00 |
| TheJulia | o/ | 15:00 |
| JayF | Welcome to the Ironic weekly meeting. We operate under the usual Code of Conduct. | 15:00 |
| JayF | #topic Announcements | 15:00 |
| JayF | #note Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: https://tinyurl.com/ironic-weekly-prio-dash | 15:01 |
| rpittau | o/ | 15:01 |
| JayF | It's R-17. Trailing release cycle stuff for Flamingo is due by today; that doesn't impact us. | 15:01 |
| JayF | s/today/this week/ | 15:01 |
| iurygregory | o/ | 15:01 |
| clif | o/ | 15:01 |
| JayF | #topic Working Group Updates | 15:02 |
| JayF | Do we have an update on Standalone Networking? | 15:02 |
| alegacy | o/ | 15:02 |
| alegacy | Yep | 15:02 |
| alegacy | updated my patch series last week to address everyone comments | 15:02 |
| cid | o/ | 15:02 |
| alegacy | i see that one of them has merged as of this morning. | 15:03 |
| TheJulia | \o/ | 15:03 |
| alegacy | if there are further comments on the remaining patches I'll take a look! | 15:03 |
| JayF | You wanna link the chain here for the notes? | 15:03 |
| alegacy | https://review.opendev.org/q/topic:%22feature/standalone-networking%22+status:open | 15:04 |
| JayF | #link https://review.opendev.org/q/topic:%22feature/standalone-networking%22+status:open | 15:04 |
| JayF | #note patches are landing for standalone networking; please prioritize review | 15:04 |
| JayF | Thank you for the update alegacy, I assume that's it? | 15:04 |
| alegacy | yes sir | 15:04 |
| JayF | Anything new on AsyncIO work? | 15:04 |
| dtantsur | No, and won't be for a week more | 15:06 |
| JayF | Ack. | 15:06 |
| dtantsur | 2 weeks really | 15:06 |
| JayF | #note No new info on AsyncIO, probably not for 2+ weeks. | 15:06 |
| JayF | VXLAN Networking, then? | 15:06 |
| TheJulia | I believe I revised the spec I've got posted. Discussions seem stalled at the moment. | 15:06 |
| TheJulia | dtantsur: I did reproduce the toc or lack of toc entry issue. That is weird, but I'll try to figure it out this week. | 15:07 |
| TheJulia | I won't be around next week, so likely no progress on my end until sometime until later in the month | 15:07 |
| JayF | #note VXLAN spec updated; please review. Progress note expected until later in the month. | 15:08 |
| JayF | #topic Discussion Topics | 15:08 |
| JayF | I think the first one here, based on my reading of the ML earlier, is a leftover? | 15:08 |
| JayF | We have already decided to cancel Dec. 22, Dec 29 meetings, yes? | 15:08 |
| rpittau | yep | 15:08 |
| TheJulia | Yes, I sent out that email already | 15:08 |
| TheJulia | its a left over | 15:08 |
| JayF | What about iRMC/DMTF items, new or leftover cc: janders | 15:08 |
| janders | new | 15:09 |
| JayF | you have the floor | 15:09 |
| janders | w/r/t iRMC we are very much done. Depreciation change is merged. FJ were consulted and are on board, so we're nearly done. Should I go ahead and send an email to the ML stating what we decided to close the loop? | 15:09 |
| TheJulia | Yes please | 15:10 |
| JayF | I'd say yes, plese | 15:10 |
| janders | OK, will do. Once that's done we can tick this item off. I will team up with someone more experienced since it's the first depreciation I am handling. | 15:11 |
| janders | Any closing comments on iRMC? If not I will move to the DMTF topic. | 15:11 |
| janders | OK, moving on. | 15:11 |
| janders | I approached DMTF folks about the issue of machines with key resources spread across multiple Systems. They are happy to talk about it and advised best way to get attention is to open an issue in their GH space. | 15:12 |
| JayF | I also have had an interaction with someone invested in DMTF, who pointed me at two people, one of whom is mraineri, for feedback. I suspect this will be duplicating noise but I won't turn down the opportunity to help advocate for a more Ironic-friendly Redfish spec :D | 15:13 |
| janders | Unfortunately this space is private, but I do have access to it. I think it's worth pursuing this. However I think it would be best if we can colaborate on the problem description to best explain to them what we want to know | 15:13 |
| janders | yeah I also exchanged emails with Mike | 15:13 |
| janders | JayF since you raised this issue, I thought perhaps the two of us collaborate via gDoc with whoever else may be interested? | 15:13 |
| janders | once we have a draft for the issue I can post it and raise it in upcoming DMTF meetings | 15:14 |
| janders | (upcoming/tomorrow one is heavily booked, but the following one should be better for this) | 15:14 |
| janders | would this be a good course of action? WDYT folks? | 15:14 |
| TheJulia | I think a doc makes sense to get an issue in place as a starting point | 15:14 |
| JayF | janders: I think that's a good idea, but I don't really have cycles this week. I'm flying out to Dallas on Wednesday. | 15:14 |
| janders | JayF noted. I will share the gDoc with you anyway so you can contribute when you have time. | 15:15 |
| TheJulia | Sounds like this may be a slow burn until the end of the year, tbh | 15:15 |
| janders | TheJulia would you have some time to look at the initial version of what I am putting together? | 15:16 |
| JayF | #note Those interested in collaborating on a feedback packet for DMTF should contact janders. | 15:16 |
| janders | ++ | 15:16 |
| JayF | #action janders to email the list about iRMC retirement | 15:16 |
| janders | OK, that's me, thanks folks | 15:16 |
| TheJulia | janders: I would be happy to, I'm not sure I will have anything constructive in the short term, but if you ping me I'll take a look. | 15:16 |
| janders | thank you TheJulia | 15:16 |
| JayF | #topic Bug Deputy Updates | 15:16 |
| JayF | cid: it says you were the bug deputy | 15:17 |
| cid | Yeah. I was | 15:17 |
| cid | Update, 1 RFE | 15:18 |
| JayF | #link https://bugs.launchpad.net/ironic/+bug/2133499 | 15:18 |
| TheJulia | I can see where folks might be pushing us and dmitry wants to go to solve it, I think a slightly deeper discussion may be needed because it feels like we're unlocking a closet of foot-guns. | 15:18 |
| TheJulia | Maybe that is a spec, I dunno. | 15:19 |
| JayF | At a minimum I don't fully grok the use case | 15:19 |
| TheJulia | I kind of do, but it feels like a footgun shaped giant lever | 15:19 |
| JayF | once I understand what the use case is and why a hard node delete doesn't solve it I would have a better opinion | 15:20 |
| TheJulia | That is great point | 15:20 |
| TheJulia | And it would help | 15:20 |
| janders | was thinking the same - why not force-delete? | 15:20 |
| TheJulia | we've been skittish of anything which is a forcing behavior in the past because it begins to reset expectations | 15:21 |
| TheJulia | I guess I'm worried the next is "force active with a verb", which heads us on a path to "force ipa to make me whipped cream for my pies" | 15:21 |
| JayF | We'll get more detail. | 15:21 |
| JayF | I put a comment in the bug just now. | 15:21 |
| TheJulia | I did as well earlier | 15:22 |
| JayF | I don't care if that detail is in spec or more-fleshed-RFE form | 15:22 |
| TheJulia | TBF, we're talking verbs, behavior changes, and state machine as well | 15:22 |
| TheJulia | That really puts it in spec territory | 15:22 |
| TheJulia | But, a little more detail first might help the discussion | 15:22 |
| JayF | OK, who wants to be bug deputy this week? | 15:24 |
| cid | I will be bug deputy this week. I think most folks will be out. | 15:24 |
| TheJulia | I can, but I won't be around next monday | 15:24 |
| TheJulia | cid: ack | 15:24 |
| JayF | #note CID to be bug deputy again | 15:24 |
| JayF | #topic Open Discussion | 15:24 |
| JayF | As a first note for open discussion, I'll be out the last half of this week traveling so if you need something from me speak soon :) | 15:25 |
| TheJulia | Fair, I'll be out next week | 15:26 |
| janders | Since it's quiet I will go: I have a question related to hardware health monitoring (https://review.opendev.org/c/openstack/ironic/+/966946). In one of the reviews Steve asked me if there is an issue or a spec for this. From the PTG discussion it seemed to me this is lightweight enough not to need one. Am I right or do you feel there is further | 15:27 |
| janders | discussion needed about this? Asking so I can give Steve a solid answer. | 15:27 |
| JayF | my usual personal rule is to make a rfe bug if someone asks for one | 15:28 |
| JayF | and upgrade that to a spec if demanded in the meeting | 15:28 |
| JayF | but that's not like, policy, just now I approach ti | 15:28 |
| TheJulia | I think consensus was largely "we agree to the idea", but we didn't require a spec. RFE bug does make sense to just treat as something approved to track it | 15:28 |
| * dtantsur finally back to the screen | 15:28 | |
| dtantsur | JayF: force deletion won't clean up anything, that's a big issue now | 15:28 |
| dtantsur | (cache, PXE configs, neutron configs, you name it) | 15:29 |
| dtantsur | also right now force deletion is only possible in a couple of states | 15:29 |
| JayF | That helps focus the use case for sure | 15:29 |
| dtantsur | TheJulia: retired is an entire different purpose (planned retirement vs "this thing burned in its track") | 15:30 |
| TheJulia | fair enough, why an entirely new db field though? | 15:30 |
| dtantsur | Think about getting a bricked machine out of the provision state samsara | 15:30 |
| dtantsur | TheJulia: so that various code throughout Ironic can stop doing what it's tryign to do | 15:31 |
| dtantsur | e.g. trying to power it on/off during cleaning, even though the thing will never respond to IPMI packages ever again | 15:31 |
| TheJulia | Well, if there is an open task, they aren't going see if if they currently hold another task | 15:31 |
| TheJulia | (sort of feels like something we should be doing with the state machine too, but more pondering needed) | 15:32 |
| dtantsur | That's a fair point. Another part of it is to be able to reject new tasks. | 15:32 |
| dtantsur | Also, your long-running task spec may make TaskManager re-entrant | 15:32 |
| JayF | If y'all are Ok with this discussion happening outside the meeting, I'm going to close up the official logger? | 15:33 |
| TheJulia | eh, sort of, but i have no time to revisit that at hte moment | 15:33 |
| dtantsur | sure, it's not urgent | 15:33 |
| dtantsur | in fact, I won't have time to work on it in any near future | 15:33 |
| TheJulia | Really, the deferred task bit expects the state might change on the task itself, at least, that is how I envisioned it | 15:33 |
| dtantsur | (but maybe some interns...) | 15:33 |
| TheJulia | fair far | 15:33 |
| TheJulia | dtantsur: could you put a little more clarity in to the RFE and lets try to revisit the discussion in a few days? | 15:34 |
| TheJulia | Just give brains time to do the brain thing | 15:34 |
| dtantsur | definitely. I want to write the thought down before I forget it | 15:34 |
| dtantsur | (squirell brain, you know) | 15:34 |
| TheJulia | fair fair | 15:34 |
| JayF | Any other topics for Open Discussion? | 15:35 |
| janders | I will create that RFE for monitoring. | 15:35 |
| janders | Thanks! | 15:36 |
| JayF | That does it for the Ironic meeting, thanks all, see you next week same metal-time, same metal-channel o/ | 15:36 |
| JayF | #endmeeting | 15:36 |
| opendevmeet | Meeting ended Mon Dec 1 15:36:40 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:36 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-12-01-15.00.html | 15:36 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-12-01-15.00.txt | 15:36 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-12-01-15.00.log.html | 15:36 |
| janders | dtantsur while I have you, would you have cycles to collaborate on DMTF issue gDoc for multi-System-machines? | 15:37 |
| janders | (lines timed from 16:12 in the meeting) | 15:38 |
| dtantsur | janders: not before next week, sorry | 15:38 |
| dtantsur | You don't imagine how busy I am downstream this week | 15:38 |
| janders | dtantsur totally understand, leave this with me | 15:38 |
| janders | I may share the gDoc with all interested but that's just so you have an option to comment when you have time | 15:40 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Make post-firmware-update reboot conditional on component https://review.opendev.org/c/openstack/ironic/+/966344 | 15:41 |
| opendevreview | Merged openstack/ironic master: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/968418 | 16:10 |
| dtantsur | Oh damn, should have mentioned the broken (?) SIGHUP support on the meeting | 16:59 |
| * dtantsur is curious if anyone cares in reality | 16:59 | |
| janders | dtantsur as in kill -HUP triggering low-disruption Ironic config reload? | 17:02 |
| TheJulia | so | 17:02 |
| dtantsur | Well, that's what it used to do | 17:02 |
| TheJulia | well | 17:02 |
| dtantsur | now it's reloading the entire conductor worker | 17:02 |
| TheJulia | my opinion is that oslo broke it and its actually a correct change for mutlithreaded universe | 17:03 |
| TheJulia | (likely, not the popular opinion, but its much more a side effect from my point of view) | 17:03 |
| dtantsur | From the container perspective, I don't care too much since any configuration change is a restart for us anyway | 17:03 |
| TheJulia | I think that is the other aspect, the world is shifting to that as a model | 17:03 |
| dtantsur | but I wonder if some consumers appreciated being able to e.g. change to debugging mode without draining the entire conductor | 17:04 |
| TheJulia | which gives me a bit less urgency than to fight the "mutate should mutate!" | 17:04 |
| TheJulia | Well, you can do a quick restart, you just break stuff in flight | 17:04 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/969120 | 17:04 |
| TheJulia | even then, steve has shorted a lot of that window (if I'm recalling correctly) | 17:05 |
| TheJulia | There is also proper drain and such | 17:05 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/969122 | 17:05 |
| TheJulia | Thats a bit different, and actually is broken depending on the eventlet version on older versions (*twitch* *twitch*) | 17:05 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2024.2: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/969123 | 17:06 |
| opendevreview | Doug Goldstein proposed openstack/ironic bugfix/31.0: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/969124 | 17:06 |
| janders | I retrospectively created RFE for Redfish health monitoring: https://bugs.launchpad.net/ironic/+bug/2133522 | 17:30 |
| cardoe | clif: you good with my feedback on TBN? | 18:42 |
| TheJulia | BRRAAAAINS | 18:44 |
| clif | cardoe: yep, I'm working on it | 18:45 |
| clif | didn't know about ddt | 18:45 |
| cardoe | good deal. just making sure I'm not a blocker | 18:45 |
| TheJulia | ddt is awesome, but... I think there was soemthing that makes me hesitant | 18:45 |
| clif | although I'm a little peeved that it doesn't provide an easy way to set the __name__ of the test data object | 18:46 |
| clif | even in the example it does a little dance to set it... why not just provide that as part of the library or make it easy | 18:46 |
| opendevreview | Merged openstack/python-ironicclient master: A new `instance_name` field to the node object https://review.opendev.org/c/openstack/python-ironicclient/+/955102 | 18:46 |
| clif | overall it does seem a bit simpler than doing my own subtest dance in each test so that's good | 18:47 |
| TheJulia | no declared "support" past py3.10 for ddt, looks like maybe inactivity maintenance wise :\ | 18:47 |
| clif | hrm, it does already exist in ironic's test-requirements.txt at least... | 18:48 |
| TheJulia | Yeah, we did a bunch of RBAC testing using it | 18:48 |
| clif | cardoe: the other thing about StrEnum is that py310 doesn't support it | 18:49 |
| cardoe | Well unless you specifically need to work with it as a str then a regular enum would work too. | 18:50 |
| cardoe | I just saw you using match so that helps make it exhaustive for tests and analysis. | 18:51 |
| cardoe | I believe ddt is the recommended thing by the OpenStack testing tools. | 18:52 |
| cardoe | I'm personally a pytest fan so I don't keep up with the unittest helpers that are out there. | 18:52 |
| JayF | ddt is what nova asked me to rewrite some tests in for a follow-up | 19:05 |
| JayF | so if it's unsupported it's at least everyones' problem not just ours | 19:05 |
| clif | we can support it :) | 19:06 |
| clif | pytest is good ime | 19:07 |
| JayF | there's already an ML thread about people trying to paint their test-shed that color | 19:08 |
| cardoe | That's why I suggested ddt cause I saw nova suggest that to JayF | 19:09 |
| TheJulia | ddt needs a new release, basically, I'm opening an issue on their GH regarding this | 19:14 |
| opendevreview | Merged openstack/networking-generic-switch master: tox: Enable codespell by default and limit it to relevant files https://review.opendev.org/c/openstack/networking-generic-switch/+/958154 | 21:23 |
| opendevreview | Clif Houck proposed openstack/ironic master: Generate network plan based on trait based networking config https://review.opendev.org/c/openstack/ironic/+/964895 | 21:28 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Simulator https://review.opendev.org/c/openstack/ironic/+/966202 | 21:28 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 21:28 |
| opendevreview | Clif Houck proposed openstack/ironic master: Generate network plan based on trait based networking config https://review.opendev.org/c/openstack/ironic/+/964895 | 22:24 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Simulator https://review.opendev.org/c/openstack/ironic/+/966202 | 22:24 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 22:25 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!