| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Deferred Tasks https://review.opendev.org/c/openstack/ironic-specs/+/954612 | 01:49 |
|---|---|---|
| opendevreview | Pierre Riteau proposed openstack/networking-generic-switch stable/2026.1: Add Dell OS10 bond trunk support https://review.opendev.org/c/openstack/networking-generic-switch/+/986063 | 09:57 |
| opendevreview | Pierre Riteau proposed openstack/networking-generic-switch stable/2025.2: Add Dell OS10 bond trunk support https://review.opendev.org/c/openstack/networking-generic-switch/+/986064 | 09:58 |
| opendevreview | Pierre Riteau proposed openstack/networking-generic-switch stable/2025.1: Add Dell OS10 bond trunk support https://review.opendev.org/c/openstack/networking-generic-switch/+/986065 | 09:58 |
| opendevreview | IKEH CHUKWUKA FAVOUR proposed openstack/ironic-python-agent master: Fix 401 auth failures not aborting image download https://review.opendev.org/c/openstack/ironic-python-agent/+/986069 | 10:36 |
| M0xull[m] | hello, submitted a fix for bug #2098146 in IPA. IPA was retrying downloads on 401 auth failures instead of aborting. would appreciate a review when you get a chance (https://review.opendev.org/c/openstack/ironic-python-agent/+/986069) | 10:47 |
| opendevreview | Verification of a change to openstack/ironic master failed: Shell-quote console command passed to `socat` https://review.opendev.org/c/openstack/ironic/+/985899 | 11:02 |
| iurygregory | good morning ironic | 11:49 |
| iurygregory | cardoe, hey I'm wondering if 5pm your time on Wed would work for the meeting, it would be 8am (Thursday) for janders and 7pm (Wed) for me - I'm totally fine with it, since my working hours aren't the normal ones =) | 11:50 |
| cardoe | Sure. You’re -0300? | 11:51 |
| iurygregory | yeah =) Brazil | 11:51 |
| dtantsur | JayF: hey, let me know if you have a few minutes today to talk about GPU in the inventory (cc janders since you were interested) | 12:10 |
| JayF | dtantsur: it remains PTG for those of us who attend leadership sessions | 12:12 |
| JayF | So that'll be a next week thing most likely because I don't think all the TC sessions are over until 10:00 or 11:00 my time, which is usually after you're gone. | 12:13 |
| dtantsur | right, I see. next week then? | 12:13 |
| dtantsur | okie | 12:13 |
| JayF | Yes. Although I'll be traveling so it'll have to be Monday, Tuesday or early Wednesday | 12:13 |
| TheJulia | good morning | 12:22 |
| * TheJulia tries to wake up | 12:25 | |
| cardoe | iurygregory: ah I was wondering where there was land at -0300. My northern hemisphere brain was thinking you lived in Greenland. | 13:13 |
| opendevreview | Merged openstack/ironic master: CI: Set multinode jobs to fail if binding fails https://review.opendev.org/c/openstack/ironic/+/941712 | 13:48 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix IPA boot during out-of-band service steps https://review.opendev.org/c/openstack/ironic/+/984894 | 15:14 |
| opendevreview | Doug Szumski proposed openstack/tenks master: Support setting Ironic shard key and conductor group https://review.opendev.org/c/openstack/tenks/+/983901 | 15:18 |
| opendevreview | Will Szumski proposed openstack/tenks master: Switch to Rocky 10 for kayobe testing https://review.opendev.org/c/openstack/tenks/+/986117 | 15:31 |
| opendevreview | Doug Szumski proposed openstack/tenks master: Support setting Ironic shard key and conductor group https://review.opendev.org/c/openstack/tenks/+/983901 | 15:33 |
| JayF | we've broken the record for most patches proposed to tenks in a day /s | 15:43 |
| JayF | hehehe | 15:43 |
| TheJulia | lol, nah | 15:43 |
| TheJulia | The record is at least 6 in a single day ;) | 15:43 |
| cardoe | https://review.opendev.org/dashboard/?title=Open+ironic-week-prio&foreach=is:open+-is:wip+hashtag:ironic-week-prio&To+Review=(-reviewedby:self+and+-owner:self)&Reviewed+By+Me=(reviewedby:self)&My+Changes=(owner:self) | 15:58 |
| cardoe | That's my new ironic-week-prio view. | 15:58 |
| cardoe | It's splitting things up to "Things I Need to Review", "Things I Have Reviewed" and "Changes Owned By Me" | 15:59 |
| TheJulia | nice | 16:05 |
| opendevreview | Merged openstack/ironic stable/2026.1: fix: bios.apply_configuration needs to validate before starting https://review.opendev.org/c/openstack/ironic/+/985744 | 16:06 |
| cardoe | Looking to make some sharable views with the rest of the ironic community to help out. | 16:09 |
| opendevreview | Verification of a change to openstack/ironic master failed: Shell-quote console command passed to `socat` https://review.opendev.org/c/openstack/ironic/+/985899 | 16:10 |
| dtantsur | cardoe: Neat! But you need to rename "Reviewed By Me" because it includes patches where you're merely cc'ed, even if you haven't seen them in reality | 16:10 |
| cardoe | Does it? The query is "reviewedby:self" | 16:11 |
| JayF | wait, really? | 16:11 |
| dtantsur | hmmm, maybe I'm wrong, hold on | 16:11 |
| JayF | I've been using that in my filters a LOT | 16:11 |
| JayF | it'd be hilarious if I'm specifically filtering out things people wanted me to look at :| | 16:11 |
| dtantsur | I thought I haven't reviewed one of the patches, but I have | 16:11 |
| dtantsur | Okay, ignore me, I guess works as expected | 16:13 |
| * dtantsur gets back to shaving a pretty chunky yak | 16:15 | |
| TheJulia | Why is everyone shaving yak's this week?! | 16:16 |
| TheJulia | oh right, PTG! | 16:16 |
| dtantsur | it's not even PTG-related for me | 16:16 |
| TheJulia | doh :( | 16:17 |
| dtantsur | I wanted to file a bug in a certain feature. So I tried to test it and found it completely broken. So I wanted to file that bug. But when debugging, I realized my bifrost VM ran out of disk space. So I realized that my VM-building script does not expand the disk. So I went on to fix the script. Then rebuild the VM. Then rebuild bifrost. So that I can file a bug in the feature. And then finally the initial bug related to that | 16:17 |
| dtantsur | feature. | 16:17 |
| TheJulia | speaking of things and stuff, cid, if you could take a look at the deferred tasks spec, since I did a huge revision of it yesterday and tried to lay some of the higher level foundation for building bulk actions upon | 16:18 |
| dtantsur | I hope I don't forget the initial bug by the time I get to it..... | 16:18 |
| cid | TheJulia, tks. I will start the absorption process later, likely tomorrow. | 16:20 |
| TheJulia | I *think* I got the needful for basically all the flows we've discussed at a super high level to also get to Jay's point on 409s and all | 16:21 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Drop support for ironic-staging-drivers https://review.opendev.org/c/openstack/bifrost/+/986131 | 16:25 |
| dtantsur | I cannot believe ^^ is still a thing | 16:25 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Drop support for ironic-staging-drivers https://review.opendev.org/c/openstack/bifrost/+/986131 | 16:25 |
| TheJulia | whaaat?!? | 16:33 |
| dtantsur | Aaand, why does ironicclient import eventlet, I wonder? | 16:34 |
| dtantsur | ah, because https://opendev.org/openstack/oslo.concurrency/src/branch/master/oslo_concurrency/lockutils.py#L44-L47, and the released version does not silence the warning yet | 16:35 |
| dtantsur | Here is another contribution to Facepalm Friday: https://bugs.launchpad.net/ironic/+bug/2150252 | 16:41 |
| TheJulia | I didn't realize we permitted that... | 16:42 |
| dtantsur | Me neither | 16:42 |
| dtantsur | Honestly, I don't know if we should fix it or silently remove this ability since nobody has ever complained that it does not work | 16:45 |
| TheJulia | put a check in to explicitlly block it since your not moving ports, imho | 16:45 |
| opendevreview | Merged openstack/tenks master: Switch to Rocky 10 for kayobe testing https://review.opendev.org/c/openstack/tenks/+/986117 | 16:50 |
| dtantsur | yeah. I'm trying to figure out if it ever worked (where ever == 2024.2). If not, I assume I can safely disable it instead of trying to understand how and where to apply the fix. | 16:55 |
| JayF | I think we disable it anyway, tbh | 16:56 |
| JayF | there's not really a case where that is a logically sane thing to do | 16:56 |
| JayF | the only place it'd logically make sense is when you switch a NIC *while leaving the NIC plugged into the same physical ports* | 16:56 |
| dtantsur | it's broken the same way in 2024.2, so yay | 17:00 |
| cardoe | dtantsur: so I tested the query... my query is right.. there's "reviewer:self" and "reviewedby:self"... I'm using the later and the later means I've got a non-0 vote. | 17:05 |
| TheJulia | ship the block! | 17:06 |
| * TheJulia needs an evil hat | 17:06 | |
| cardoe | https://review.opendev.org/dashboard/?title=Open+backports&foreach=is:open+(projects:openstack/ironic+OR+projects:openstack/python-ironic+OR+project:openstack/networking-generic-switch+OR+project:openstack/networking-baremetal+OR+project:openstack/bifrost+OR+projects:/openstack/sushy)&stable=(branch:%5Estable/.*)&bugfix=(branch:%5Ebugfix/.*)&unmaintained=(branch:%5Eunmaintained/.*) | 17:11 |
| cardoe | There's another query... it still needs more projects added. I'm wondering if we can somehow parent a bunch of projects to a meta ironic project in gerrit and then we can just query on the parent=ironic | 17:12 |
| cardoe | I'd like to abandon some of those old bugfix ones | 17:13 |
| opendevreview | Merged openstack/tenks master: Support setting Ironic shard key and conductor group https://review.opendev.org/c/openstack/tenks/+/983901 | 17:14 |
| opendevreview | Merged openstack/ironic master: Shell-quote console command passed to `socat` https://review.opendev.org/c/openstack/ironic/+/985899 | 18:36 |
| opendevreview | IKEH CHUKWUKA FAVOUR proposed openstack/ironic-python-agent master: Fix 401 auth failures not aborting image download https://review.opendev.org/c/openstack/ironic-python-agent/+/986069 | 20:14 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: doc: highlight firmware interface and deprecate management for updates https://review.opendev.org/c/openstack/ironic/+/986233 | 21:34 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2026.1: Shell-quote console command passed to `socat` https://review.opendev.org/c/openstack/ironic/+/986235 | 21:50 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix 401 auth failures not aborting image download https://review.opendev.org/c/openstack/ironic-python-agent/+/986069 | 22:32 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!