Monday, 2025-06-02

opendevreviewTakashi Kajinami proposed openstack/ironic-prometheus-exporter master: Replace pkg_resources  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/95148304:14
opendevreviewMerged openstack/sushy master: Fix type of Targets in ShareParameters for Dell  https://review.opendev.org/c/openstack/sushy/+/95069412:23
iurygregorygood morning ironic12:43
winiciusallan[m]morning o/12:44
TheJuliagood morning13:00
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job  https://review.opendev.org/c/openstack/ironic/+/95020613:37
TheJuliaJayF: Interesting little device!13:45
TheJuliadtantsur: I really like the idea you planted on https://review.opendev.org/c/openstack/ironic/+/94684314:00
FreemanBoss[m]rpittau: please what can I start working on?14:02
FreemanBoss[m]rpittau: and on my reviews that you've approved anything else I need to do to get them merged?14:02
opendevreviewMerged openstack/ironic master: Update devstack-guide to use neutron- not q-  https://review.opendev.org/c/openstack/ironic/+/95142414:05
opendevreviewJulia Kreger proposed openstack/ironic bugfix/28.0: Fix agent get_XXX_steps retries from being treated as not fresh agents  https://review.opendev.org/c/openstack/ironic/+/95152514:05
opendevreviewJulia Kreger proposed openstack/ironic bugfix/27.0: Fix agent get_XXX_steps retries from being treated as not fresh agents  https://review.opendev.org/c/openstack/ironic/+/95152614:06
opendevreviewMerged openstack/ironic master: doc: A possible improvement around multi-arch config options  https://review.opendev.org/c/openstack/ironic/+/94941614:09
TheJuliaFreemanBoss[m]: If you want some reviews, happy to take a look at some of your changes, just share a link and I'll take a look14:17
dtantsurFreemanBoss[m]: rpittau is currently away, please ask masghar for next tasks14:27
TheJuliadtantsur: I started to look at the eventlet removal etherpad on Friday in depth. I've added some comments and attempted to frame things a little more verbosely. I expect to pick it back up after the weekly meeting.14:30
dtantsurnice, thank you! I'll check soon14:31
TheJuliadtantsur: Broadly, I think it is on the right track but I can definitely see where your brain started to focus14:31
TheJulialocking/safeness... thats a stupidly good question and it is all going to depend on a few variables (and lockutils as well).14:32
* dtantsur nods14:32
TheJuliaThere *is* the broad/general idea to hand a task to the conductor as if it is new work like we do with some stuff, but I think your more so thinking error handling?!14:32
dtantsuryeah, low-level internal business, not node-level locking (which hopefully stays the same for now)14:33
dtantsurit's very well possible that nothing needs to be done14:33
dtantsurI just don't want us to assume that without checking :)14:33
TheJuliaaround taskmanager semaphors?14:34
TheJuliaI'm sort of thinking the same we might not need to, *but* if we do... yeouch14:34
dtantsurgenerally around taskmanager methods since its instances can and will be passed between threads14:34
dtantsurhere I miss the Rust's ownership model: it allows the compiler to proof that nothing is used concurrently without locking :)14:35
TheJuliaThat is what I was thinking as well around the locks, I think they are done over an abundance of caution but also its sort of like a first line guard if I'm remembering the code correctly14:35
TheJuliaSomething to slowly sip coffee and dig into :)14:36
dtantsur++14:36
JayFdtantsur: you win the prize to be the first person to allude to riir'ing ironic :D 14:43
dtantsurI'm honored :D14:44
TheJuliaSo who is driving the meeting this week?15:00
JayF#startmeeting ironic15:00
opendevmeetMeeting started Mon Jun  2 15:00:38 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
TheJuliao/15:00
JayFWelcome to the Ironic weekly meeting! 15:00
iurygregoryo/15:00
JayFAs always we're operating under the OpenInfra Code of Conduct15:01
JayF#note Standing reminder to review patches hashtagged with ironic-week-prio15:01
alegacyo/15:01
JayF#note Flamingo release cycle ongoing, currently R-1715:02
JayF#topic Working Group updates15:02
JayFAny updates on Standalone networking?15:02
cido/15:02
alegacyYa, just about done putting together a spec for that15:02
alegacysorry was distracted with some other work so took longer than expected15:02
JayF#note alegacy expecting to publish spec soon15:02
JayFThanks for the update and don't sweat it :)15:03
alegacyhave just about all the section filled out now... hoping to finish this week.15:03
TheJuliacool cool15:03
JayFAnything else for working group updates on standalone networking or anything else?15:04
JayFWe don't have Eventlet listed here; we probably should?15:04
TheJuliaWe likely should15:04
TheJuliaI was just thinking some of the cleanups also tightly relate to python versions and ultimately eventlet, but we're close on retooling the multinode jobs15:04
JayFEventlet migration is ongoing, a lot of good progress in the last couple of weeks15:04
TheJulialike... hopefully the next revision.15:04
JayF🫰15:04
JayFWe don't have anything under discussion topics category; skipping15:05
JayF#topic Bug Deputy Updates15:05
JayFTheJulia: you are listed as bug deputy this (two) weeks15:05
TheJuliacool, no new bugs for last week15:05
TheJuliaI guess it was a nice and quiet week15:06
JayFGood stuff. Does someone want to be bug deputy next week?15:06
cidI could15:06
JayFsounds great, thank you 15:06
JayF#note No new bugs this week; new deputy is cid 15:06
JayFThere are no RFEs for review. Skipping that topic.15:07
JayF#topic Open Dicsussion15:07
JayFnothing on agenda here, but I have a thing to mention15:07
JayFThis is your two week Jay warning. A week from Friday will be my last day working for six weeks. If you need anything from me, start asking now. Please help out cid, clif, adamcarthur5 with anything they might need tech help with in my absense :)15:07
iurygregoryack o/15:08
dtantsurnoted15:08
cidack15:08
JayFLast call for open discussion and meeting overall?15:09
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job  https://review.opendev.org/c/openstack/ironic/+/95020615:10
TheJuliahave we double checked our cycle priorities ?15:10
JayFthey look really good right now iirc15:10
JayFwe did a better job than usual of not throwing stuff in we had lower confidence in15:11
TheJuliaI'll try to hammer out the steps used in node history stuff at some point in the next couple of weeks. Shouldn't take long15:11
JayFeventlet migration, dynamic networking, project mercury, OEM handling in sushy, api response validation/openapi spec, redfish servicing improvements, document stpes in node history15:11
JayFI think outta that list doc steps in node history is the only one that hasn't moved15:11
JayFand yeah, == TheJulia it's not going to be a big one15:11
JayFeverything else has had meaningful progress made.15:12
JayFOn a related note; https://review.opendev.org/c/openstack/ironic-specs/+/945642 I would love to get this landed before I go15:12
TheJuliaYeah, one last item, I could use your eyes on https://review.opendev.org/c/openstack/ironic/+/950206 since I'm changing some of the logic since it re-shuffles how nodes map to hypervisor "compute" nodes.15:12
TheJuliaand the shard logic gets a little weird in the variation15:13
JayFI'll get a revise on that spec today15:13
TheJuliacool cool15:13
JayFIf the meeting is over, we can close it up and have that chat sync now()?15:13
TheJuliasure15:15
JayFThanks everyone o/15:15
JayF#endmeeting ironic15:15
opendevmeetMeeting ended Mon Jun  2 15:15:30 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:15
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-06-02-15.00.html15:15
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-06-02-15.00.txt15:15
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-06-02-15.00.log.html15:15
TheJuliahttps://meet.google.com/nch-wbdw-qwh <-- high bandwidth discussion for shards job15:16
FreemanBoss[m]TheJulia:  thank you.  These are the links. https://review.opendev.org/c/openstack/bifrost/+/946726 https://review.opendev.org/c/openstack/bifrost/+/94672415:17
queensly[m]Hi everyone, I have just started my Outreachy internship with the Ironic project. My project is about setting the bare metal host's clock remotely using Redfish.15:18
queensly[m]I will be working closely with masgar and rpittau, and I might reach out here if I run into issues.15:18
queensly[m]Looking forward to learning and contributing!15:18
queensly[m]s/masgar/masghar/15:18
FreemanBoss[m]<dtantsur> "Freeman Boss: rpittau is..." <- dtantsur: thank you. But I'm unable to tag masghar15:22
iurygregoryFreemanBoss[m], you just tagged =) 15:23
FreemanBoss[m]iurygregory: I don't understand. But I meant the @masghar isn't available again15:24
iurygregoryon irc you don't need the "@"15:24
iurygregorywhen you type masghar she will be notified (if she is online at the moment you sent the message)15:25
FreemanBoss[m]I'm just looking forward to anything to start working on to start having tangible contributions like actual coding contributions 15:25
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job  https://review.opendev.org/c/openstack/ironic/+/95020615:36
masgharHello ironic! (Lost track of time today and missed the meeting, smh) Just wanted to drop by and say Queensly will be interning with us from June - August for this Outreachy round, starting from today, like she's mentioned :) 16:00
masgharHello Freemanboss o/ I've taken a look at your patches, thank you for them! For next steps, perhaps you could take a look at the bug dashboard and see if there's anything you would like to work on? (https://bugs.launchpad.net/ironic/+bugs)16:01
masghar(case mismatch, FreemanBoss should work)16:02
JayFAh, my math is wrong from the meeting: this is the *3* week warning.16:02
JayFThat's better anyway I had too many things left to do before I go for two weeks lol16:02
TheJuliaheh16:07
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Inject network config when configdrive is empty  https://review.opendev.org/c/openstack/ironic/+/95157216:25
TheJuliaJayF: woot, shard job passed16:46
JayF\o/16:46
TheJuliamoving the existing multinode job out of normal operation and just relying upon the shard job, any preference to do that in an entirely separate change?16:47
*** awb_ is now known as awb17:02
opendevreviewMerged openstack/ironic master: Allow to unprovision instance from service wait states  https://review.opendev.org/c/openstack/ironic/+/94830117:29
TheJuliaJayF: ^ question17:55
JayFI don't care where/how, I care about end state17:55
TheJuliak17:56
TheJuliaI'll do it as a separate change then17:56
JayFJun 02 18:01:31 devstack-20250602-autoinstall setup-devstack.sh[2669]: ERROR neutron ModuleNotFoundError: No module named 'neutron.privileged.agent' <-- this is a little wacky18:03
JayFgoing to see if I can reproduce it manually18:03
TheJulia... wut?!18:30
TheJuliaJayF: actually, makes sense to really combine the two because the shard job just runs one test, the multinode runs 3 different networking tests :\18:31
JayFyep that's what I was saying18:31
JayFjust add a shard to the existing one, you cover both sets of cases18:31
JayFwell, trying to say at least :)18:32
TheJuliayeah, I think we need change the way we pass the test regex, but I'll just put a todo in for now18:38
opendevreviewJulia Kreger proposed openstack/ironic master: ci: Expand the multinode job -- remove tinyipa usage  https://review.opendev.org/c/openstack/ironic/+/95020619:24
opendevreviewJulia Kreger proposed openstack/ironic master: ci: combine networking multinode tests with shard tests  https://review.opendev.org/c/openstack/ironic/+/95159319:24
opendevreviewJulia Kreger proposed openstack/bifrost master: DNM: remove tinyipa usage  https://review.opendev.org/c/openstack/bifrost/+/95159419:28
opendevreviewJulia Kreger proposed openstack/ironic-python-agent master: Remove unused tinyipa CI job  https://review.opendev.org/c/openstack/ironic-python-agent/+/95159519:31
opendevreviewJay Faulkner proposed openstack/ironic master: PTG: Move CI job descriptions into yaml  https://review.opendev.org/c/openstack/ironic/+/94658420:21
opendevreviewJay Faulkner proposed openstack/ironic master: Automated cleaning by runbook  https://review.opendev.org/c/openstack/ironic/+/94525920:21
opendevreviewQueensly Kyerewaa Acheampongmaa proposed openstack/sushy-tools master: Add PATCH support for Redfish DateTime fields in Manager resource  https://review.opendev.org/c/openstack/sushy-tools/+/95092520:43
TheJuliadtantsur: I think we're relatively good on locks and overall process shutdown flow since the pattern has been fairly well constrainted. The key thing I'm wondering is if we can de-eventletize the condutor enough to remove the monkey patch. That being said the rpc surface access needs to really be addrssed first()20:44
TheJuliaThinking outloud for comments: If an admin were to just trigger a deploy and we wanted to be abel to inject an ssh key easily, how would we expose that capability....20:45
JayFit already exists; deploy template's inject_files20:52
JayFI remember because I kinda hate it's existence ;) 20:52
JayFor more correctly stated: we have a deploy step, inject files, that does that20:53
JayFit's more accurate to our model and our design to use configdrive to transit information to the running instance though20:53
TheJuliaYeah, unfortunately nova just silently injects based upon it's key data it has, so if we ever were to have a similar thing, then... we should be slightly intentional about it.20:58
TheJuliabut also, make it simple20:58
JayFbut we allow users to pass a configdrive in, yeah? 21:11
JayFincluding not base64'd right?21:12
JayFugh, I think the recent fixes around cleaning  failure recovery21:33
JayFcompletely broke my in-progress automated clean via runbook21:33
JayFit's running the runbook steps then restarting cleaning fresh, seemingly :( 21:33
JayFwell, it was working, it isn't now, I can't exactly explain why and logs are unclear21:38
* JayF found the issue and is confused how he ever tested this working21:44
JayFaha, it was working in disable_ramdisk=true (in runbook) case and not in a case where it's enabled22:08
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Store driver objects in a global dict  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983022:18
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Move _is_port_supported, _is_port_bound to utils  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983122:18
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Improve test coverage of NetmikoSwitch commands  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983222:18
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Clean up add_subports_on_trunk, del_subports_on_trunk  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983322:18
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Create service plugin to configure security groups  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983422:18
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Support security groups in netmiko base class  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983522:18
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Create service plugin to configure security groups  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983422:23
opendevreviewSteve Baker proposed openstack/networking-generic-switch master: Support security groups in netmiko base class  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983522:23
opendevreviewJay Faulkner proposed openstack/ironic master: Automated cleaning by runbook  https://review.opendev.org/c/openstack/ironic/+/94525922:29
opendevreviewJay Faulkner proposed openstack/ironic master: Automated cleaning by runbook  https://review.opendev.org/c/openstack/ironic/+/94525922:32
opendevreviewcid proposed openstack/python-ironicclient master: Cast string boolean from CLI  https://review.opendev.org/c/openstack/python-ironicclient/+/95160022:39
rm_workIs it possible to add custom UEFI shell scripts to be executed on boot? 🤔22:44
JayFthe ipxe config is templated22:49
JayFso you can do to it what you want, but you get the pieces if you break it :D 22:49

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!