opendevreview | Takashi Kajinami proposed openstack/ironic-prometheus-exporter master: Replace pkg_resources https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/951483 | 04:14 |
---|---|---|
opendevreview | Merged openstack/sushy master: Fix type of Targets in ShareParameters for Dell https://review.opendev.org/c/openstack/sushy/+/950694 | 12:23 |
iurygregory | good morning ironic | 12:43 |
winiciusallan[m] | morning o/ | 12:44 |
TheJulia | good morning | 13:00 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 13:37 |
TheJulia | JayF: Interesting little device! | 13:45 |
TheJulia | dtantsur: I really like the idea you planted on https://review.opendev.org/c/openstack/ironic/+/946843 | 14: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 |
opendevreview | Merged openstack/ironic master: Update devstack-guide to use neutron- not q- https://review.opendev.org/c/openstack/ironic/+/951424 | 14:05 |
opendevreview | Julia 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/+/951525 | 14:05 |
opendevreview | Julia 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/+/951526 | 14:06 |
opendevreview | Merged openstack/ironic master: doc: A possible improvement around multi-arch config options https://review.opendev.org/c/openstack/ironic/+/949416 | 14:09 |
TheJulia | FreemanBoss[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 look | 14:17 |
dtantsur | FreemanBoss[m]: rpittau is currently away, please ask masghar for next tasks | 14:27 |
TheJulia | dtantsur: 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 |
dtantsur | nice, thank you! I'll check soon | 14:31 |
TheJulia | dtantsur: Broadly, I think it is on the right track but I can definitely see where your brain started to focus | 14:31 |
TheJulia | locking/safeness... thats a stupidly good question and it is all going to depend on a few variables (and lockutils as well). | 14:32 |
* dtantsur nods | 14:32 | |
TheJulia | There *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 |
dtantsur | yeah, low-level internal business, not node-level locking (which hopefully stays the same for now) | 14:33 |
dtantsur | it's very well possible that nothing needs to be done | 14:33 |
dtantsur | I just don't want us to assume that without checking :) | 14:33 |
TheJulia | around taskmanager semaphors? | 14:34 |
TheJulia | I'm sort of thinking the same we might not need to, *but* if we do... yeouch | 14:34 |
dtantsur | generally around taskmanager methods since its instances can and will be passed between threads | 14:34 |
dtantsur | here I miss the Rust's ownership model: it allows the compiler to proof that nothing is used concurrently without locking :) | 14:35 |
TheJulia | That 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 correctly | 14:35 |
TheJulia | Something to slowly sip coffee and dig into :) | 14:36 |
dtantsur | ++ | 14:36 |
JayF | dtantsur: you win the prize to be the first person to allude to riir'ing ironic :D | 14:43 |
dtantsur | I'm honored :D | 14:44 |
TheJulia | So who is driving the meeting this week? | 15:00 |
JayF | #startmeeting ironic | 15:00 |
opendevmeet | Meeting 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 |
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 | o/ | 15:00 |
JayF | Welcome to the Ironic weekly meeting! | 15:00 |
iurygregory | o/ | 15:00 |
JayF | As always we're operating under the OpenInfra Code of Conduct | 15:01 |
JayF | #note Standing reminder to review patches hashtagged with ironic-week-prio | 15:01 |
alegacy | o/ | 15:01 |
JayF | #note Flamingo release cycle ongoing, currently R-17 | 15:02 |
JayF | #topic Working Group updates | 15:02 |
JayF | Any updates on Standalone networking? | 15:02 |
cid | o/ | 15:02 |
alegacy | Ya, just about done putting together a spec for that | 15:02 |
alegacy | sorry was distracted with some other work so took longer than expected | 15:02 |
JayF | #note alegacy expecting to publish spec soon | 15:02 |
JayF | Thanks for the update and don't sweat it :) | 15:03 |
alegacy | have just about all the section filled out now... hoping to finish this week. | 15:03 |
TheJulia | cool cool | 15:03 |
JayF | Anything else for working group updates on standalone networking or anything else? | 15:04 |
JayF | We don't have Eventlet listed here; we probably should? | 15:04 |
TheJulia | We likely should | 15:04 |
TheJulia | I was just thinking some of the cleanups also tightly relate to python versions and ultimately eventlet, but we're close on retooling the multinode jobs | 15:04 |
JayF | Eventlet migration is ongoing, a lot of good progress in the last couple of weeks | 15:04 |
TheJulia | like... hopefully the next revision. | 15:04 |
JayF | 🫰 | 15:04 |
JayF | We don't have anything under discussion topics category; skipping | 15:05 |
JayF | #topic Bug Deputy Updates | 15:05 |
JayF | TheJulia: you are listed as bug deputy this (two) weeks | 15:05 |
TheJulia | cool, no new bugs for last week | 15:05 |
TheJulia | I guess it was a nice and quiet week | 15:06 |
JayF | Good stuff. Does someone want to be bug deputy next week? | 15:06 |
cid | I could | 15:06 |
JayF | sounds great, thank you | 15:06 |
JayF | #note No new bugs this week; new deputy is cid | 15:06 |
JayF | There are no RFEs for review. Skipping that topic. | 15:07 |
JayF | #topic Open Dicsussion | 15:07 |
JayF | nothing on agenda here, but I have a thing to mention | 15:07 |
JayF | This 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 |
iurygregory | ack o/ | 15:08 |
dtantsur | noted | 15:08 |
cid | ack | 15:08 |
JayF | Last call for open discussion and meeting overall? | 15:09 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 15:10 |
TheJulia | have we double checked our cycle priorities ? | 15:10 |
JayF | they look really good right now iirc | 15:10 |
JayF | we did a better job than usual of not throwing stuff in we had lower confidence in | 15:11 |
TheJulia | I'll try to hammer out the steps used in node history stuff at some point in the next couple of weeks. Shouldn't take long | 15:11 |
JayF | eventlet migration, dynamic networking, project mercury, OEM handling in sushy, api response validation/openapi spec, redfish servicing improvements, document stpes in node history | 15:11 |
JayF | I think outta that list doc steps in node history is the only one that hasn't moved | 15:11 |
JayF | and yeah, == TheJulia it's not going to be a big one | 15:11 |
JayF | everything else has had meaningful progress made. | 15:12 |
JayF | On a related note; https://review.opendev.org/c/openstack/ironic-specs/+/945642 I would love to get this landed before I go | 15:12 |
TheJulia | Yeah, 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 |
TheJulia | and the shard logic gets a little weird in the variation | 15:13 |
JayF | I'll get a revise on that spec today | 15:13 |
TheJulia | cool cool | 15:13 |
JayF | If the meeting is over, we can close it up and have that chat sync now()? | 15:13 |
TheJulia | sure | 15:15 |
JayF | Thanks everyone o/ | 15:15 |
JayF | #endmeeting ironic | 15:15 |
opendevmeet | Meeting ended Mon Jun 2 15:15:30 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:15 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-06-02-15.00.html | 15:15 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-06-02-15.00.txt | 15:15 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-06-02-15.00.log.html | 15:15 |
TheJulia | https://meet.google.com/nch-wbdw-qwh <-- high bandwidth discussion for shards job | 15: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/+/946724 | 15: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 masghar | 15:22 |
iurygregory | FreemanBoss[m], you just tagged =) | 15:23 |
FreemanBoss[m] | iurygregory: I don't understand. But I meant the @masghar isn't available again | 15:24 |
iurygregory | on irc you don't need the "@" | 15:24 |
iurygregory | when 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 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: CI Science - Expand the multinode job https://review.opendev.org/c/openstack/ironic/+/950206 | 15:36 |
masghar | Hello 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 |
masghar | Hello 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 |
JayF | Ah, my math is wrong from the meeting: this is the *3* week warning. | 16:02 |
JayF | That's better anyway I had too many things left to do before I go for two weeks lol | 16:02 |
TheJulia | heh | 16:07 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Inject network config when configdrive is empty https://review.opendev.org/c/openstack/ironic/+/951572 | 16:25 |
TheJulia | JayF: woot, shard job passed | 16:46 |
JayF | \o/ | 16:46 |
TheJulia | moving 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 awb | 17:02 | |
opendevreview | Merged openstack/ironic master: Allow to unprovision instance from service wait states https://review.opendev.org/c/openstack/ironic/+/948301 | 17:29 |
TheJulia | JayF: ^ question | 17:55 |
JayF | I don't care where/how, I care about end state | 17:55 |
TheJulia | k | 17:56 |
TheJulia | I'll do it as a separate change then | 17:56 |
JayF | Jun 02 18:01:31 devstack-20250602-autoinstall setup-devstack.sh[2669]: ERROR neutron ModuleNotFoundError: No module named 'neutron.privileged.agent' <-- this is a little wacky | 18:03 |
JayF | going to see if I can reproduce it manually | 18:03 |
TheJulia | ... wut?! | 18:30 |
TheJulia | JayF: 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 |
JayF | yep that's what I was saying | 18:31 |
JayF | just add a shard to the existing one, you cover both sets of cases | 18:31 |
JayF | well, trying to say at least :) | 18:32 |
TheJulia | yeah, I think we need change the way we pass the test regex, but I'll just put a todo in for now | 18:38 |
opendevreview | Julia Kreger proposed openstack/ironic master: ci: Expand the multinode job -- remove tinyipa usage https://review.opendev.org/c/openstack/ironic/+/950206 | 19:24 |
opendevreview | Julia Kreger proposed openstack/ironic master: ci: combine networking multinode tests with shard tests https://review.opendev.org/c/openstack/ironic/+/951593 | 19:24 |
opendevreview | Julia Kreger proposed openstack/bifrost master: DNM: remove tinyipa usage https://review.opendev.org/c/openstack/bifrost/+/951594 | 19:28 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Remove unused tinyipa CI job https://review.opendev.org/c/openstack/ironic-python-agent/+/951595 | 19:31 |
opendevreview | Jay Faulkner proposed openstack/ironic master: PTG: Move CI job descriptions into yaml https://review.opendev.org/c/openstack/ironic/+/946584 | 20:21 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Automated cleaning by runbook https://review.opendev.org/c/openstack/ironic/+/945259 | 20:21 |
opendevreview | Queensly 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/+/950925 | 20:43 |
TheJulia | dtantsur: 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 |
TheJulia | Thinking 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 |
JayF | it already exists; deploy template's inject_files | 20:52 |
JayF | I remember because I kinda hate it's existence ;) | 20:52 |
JayF | or more correctly stated: we have a deploy step, inject files, that does that | 20:53 |
JayF | it's more accurate to our model and our design to use configdrive to transit information to the running instance though | 20:53 |
TheJulia | Yeah, 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 |
TheJulia | but also, make it simple | 20:58 |
JayF | but we allow users to pass a configdrive in, yeah? | 21:11 |
JayF | including not base64'd right? | 21:12 |
JayF | ugh, I think the recent fixes around cleaning failure recovery | 21:33 |
JayF | completely broke my in-progress automated clean via runbook | 21:33 |
JayF | it's running the runbook steps then restarting cleaning fresh, seemingly :( | 21:33 |
JayF | well, it was working, it isn't now, I can't exactly explain why and logs are unclear | 21:38 |
* JayF found the issue and is confused how he ever tested this working | 21:44 | |
JayF | aha, it was working in disable_ramdisk=true (in runbook) case and not in a case where it's enabled | 22:08 |
opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Store driver objects in a global dict https://review.opendev.org/c/openstack/networking-generic-switch/+/949830 | 22:18 |
opendevreview | Steve 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/+/949831 | 22:18 |
opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Improve test coverage of NetmikoSwitch commands https://review.opendev.org/c/openstack/networking-generic-switch/+/949832 | 22:18 |
opendevreview | Steve 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/+/949833 | 22:18 |
opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Create service plugin to configure security groups https://review.opendev.org/c/openstack/networking-generic-switch/+/949834 | 22:18 |
opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Support security groups in netmiko base class https://review.opendev.org/c/openstack/networking-generic-switch/+/949835 | 22:18 |
opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Create service plugin to configure security groups https://review.opendev.org/c/openstack/networking-generic-switch/+/949834 | 22:23 |
opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Support security groups in netmiko base class https://review.opendev.org/c/openstack/networking-generic-switch/+/949835 | 22:23 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Automated cleaning by runbook https://review.opendev.org/c/openstack/ironic/+/945259 | 22:29 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Automated cleaning by runbook https://review.opendev.org/c/openstack/ironic/+/945259 | 22:32 |
opendevreview | cid proposed openstack/python-ironicclient master: Cast string boolean from CLI https://review.opendev.org/c/openstack/python-ironicclient/+/951600 | 22:39 |
rm_work | Is it possible to add custom UEFI shell scripts to be executed on boot? 🤔 | 22:44 |
JayF | the ipxe config is templated | 22:49 |
JayF | so 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/!