| cardoe | While we’re talking about rebootless updates. How about flagging out of band inspection as not requiring the node to be serviced. | 00:00 |
|---|---|---|
| iurygregory | TheJulia, boot iso 509.41M \o/ let's see how it goes, time to grab some dinner.. | 00:48 |
| iurygregory | wtf, kernel panic ..., I think it's time to go sleep and try again tomorrow... | 02:20 |
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-ui/+/957829 | 03:07 |
| kubajj | JayF: ok, sounds well. mostepha[m], can just review it after as he had some ideas | 09:41 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/960283 | 11:19 |
| dtantsur | TheJulia: this ^^ has just successfully deployed my cirros image from quay \o/ Will test more cases later | 11:20 |
| frickler | TheJulia: JayF: iurygregory: rpittau: I don't want to hold this up too much, but do you think the two latest patches for ipa-builder should be added to the release patch? https://review.opendev.org/c/openstack/releases/+/959568 | 11:43 |
| iurygregory | checking | 11:49 |
| iurygregory | frickler, let me update to the newest hash, so we don't have to do another release later | 11:51 |
| iurygregory | frickler, done | 11:53 |
| * iurygregory brb, need to reboot the laptop | 11:54 | |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/961639 | 12:16 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/961640 | 12:16 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/961641 | 12:16 |
| tkajinam | hi I wonder if https://review.opendev.org/c/openstack/ironic-inspector/+/958068 can be merged ? | 12:53 |
| tkajinam | It already has two +2s | 12:54 |
| tkajinam | also would be nice to have https://review.opendev.org/c/openstack/ironic-inspector/+/953764 merged | 12:54 |
| dtantsur | tkajinam: I've approved the former. Given that Inspector will be removed in the 2026.1 cycle, is it worth investing more into it? | 12:55 |
| tkajinam | dtantsur, I wasn't aware of that timeline (when I asked it previously I remember it was still TBD) | 13:12 |
| tkajinam | if that's the probable plan then the others may not needed | 13:12 |
| tkajinam | that wsgi module may allow it to survive through the upcpcoming setuptools chain though it may be useless | 13:12 |
| dtantsur | yep, I've approved that one | 13:36 |
| TheJulia | good morning | 13:41 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/ironic-inspector/+/961663 | 13:54 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/ironic-inspector/+/961664 | 13:54 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/ironic-inspector/+/961665 | 13:54 |
| TheJulia | cardoe: I think the bios code shouldn't think about IPA, really. At least that is my opinion/feeling | 14:03 |
| cardoe | Well I'm talking about the flow for booting the machine into IPA | 14:04 |
| TheJulia | iurygregory: you've got to have something else getting loaded in there which is expanding the size or resulting in a larger image. Its not much, but... hmm | 14:04 |
| TheJulia | iurygregory: regarding idrac10, okay, I'll whip something up | 14:05 |
| TheJulia | ahh, yeah | 14:05 |
| cardoe | TheJulia: so if I do the "--disable-ramdisk" option, then the bits to boot into the IPA won't be setup | 14:05 |
| TheJulia | I think there is a whole other challenge which we need to also be semi-mindful | 14:05 |
| TheJulia | of | 14:05 |
| iurygregory | TheJulia, yeah, I'm thinking in re-install bifrost to see if will help... | 14:05 |
| TheJulia | Some settings *entirely* change change the firmware image loaded by some of these machines | 14:06 |
| TheJulia | which means boot settings can get reset | 14:06 |
| iurygregory | because it doesn't make any sense | 14:06 |
| TheJulia | iurygregory: ++ | 14:06 |
| iurygregory | and I'm getting a ocp cluster to test also, finally, so I will see if I can reproduce the machine stuck in inspection | 14:06 |
| TheJulia | k | 14:07 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Add WSGI module entrypoint https://review.opendev.org/c/openstack/ironic-inspector/+/958068 | 14:08 |
| dtantsur | TheJulia: the good news is: the podman image also seems to work with my patch (well, it deploys, it does not seem to be a valid image though?) | 14:09 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/networking-baremetal/+/961668 | 14:09 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/networking-baremetal/+/961669 | 14:09 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/networking-baremetal/+/961670 | 14:09 |
| dtantsur | TheJulia: do you think https://review.opendev.org/c/openstack/ironic/+/960283 is mergeable the way it is, or would you rather see my split it into some parts? | 14:11 |
| TheJulia | dtantsur: hmmm | 14:16 |
| TheJulia | looking at it for the first time now | 14:16 |
| dtantsur | okay, if I hear screaming from the western direction, I know why | 14:16 |
| TheJulia | dtantsur: quite possibly could be good, I need to take another pass when I'm not in multiple discussions at once | 14:17 |
| dtantsur | Sure, take your time, I'm not blocked by this. | 14:17 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Documentation: image_checksum is ignored for OCI URL's https://review.opendev.org/c/openstack/ironic/+/961672 | 14:26 |
| TheJulia | So I suspect that might also address the check concern around artifacts in general with tags, going back to kernel/ramdisk. Might be worthwhile to explicitly revisit | 14:29 |
| dtantsur | I want to dive into kernel/ramdisk once I get my bloody bifrost back.. | 14:29 |
| * dtantsur is wondering if he needs image_download_source customizable in metal3 | 14:35 | |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 14:37 |
| TheJulia | dtantsur: Could be useful, I guess | 14:50 |
| TheJulia | dtantsur: but, by default, I'm not sure you would really need to unless there is some base reason not to or to allow it to flow through | 14:53 |
| JayF | tkajinam: I validated that 2026.1 meets the requirements for us to retire Inspector this cycle, fwiw. I'm just waiting on announcements/paperwork until folks are done with the integrated release. Thank you for your work to help us dot the "i" and cross the "t" for some of these details across all our repos! | 15:13 |
| tkajinam | :-D | 15:15 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/961702 | 15:43 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/961703 | 15:43 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/961704 | 15:43 |
| TheJulia | adamcarthur5: hey, I'm sorry I've not revisited your spec recently. I have a semi-crazy|awesome idea you might be interested in as well when your around next | 15:54 |
| opendevreview | Merged openstack/networking-generic-switch stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/961639 | 16:22 |
| opendevreview | Merged openstack/networking-baremetal stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/networking-baremetal/+/961668 | 16:23 |
| opendevreview | Merged openstack/networking-generic-switch stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/961640 | 16:25 |
| opendevreview | Merged openstack/networking-baremetal stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/networking-baremetal/+/961669 | 16:28 |
| opendevreview | Merged openstack/ironic-inspector stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/ironic-inspector/+/961663 | 16:29 |
| opendevreview | Merged openstack/ironic-inspector stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/ironic-inspector/+/961664 | 16:31 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/961702 | 16:47 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/961703 | 16:47 |
| adamcarthur5 | TheJulia, sounds good :) Just get in touch with me about this awesome idea when you have the time :) | 17:09 |
| opendevreview | Julia Kreger proposed openstack/sushy master: Improve Dell Asynchronous task handling https://review.opendev.org/c/openstack/sushy/+/961730 | 17:10 |
| TheJulia | adamcarthur5: https://etherpad.opendev.org/p/opendev-preptg-october-2025 and likely best to look at the irc log for another channel, getting you that link | 17:10 |
| TheJulia | https://meetings.opendev.org/irclogs/%23opendev/latest.log.html | 17:11 |
| JayF | he's in there fwiw | 17:11 |
| JayF | so "scroll up" is another option lol | 17:11 |
| TheJulia | oh, okay | 17:12 |
| TheJulia | that works too | 17:12 |
| adamcarthur5 | TheJulia is that you saying you want to talk about it at the PTG? | 17:13 |
| TheJulia | Perhaps, dunno. Just sort of floating the ideas out there, seeing what sticks | 17:14 |
| adamcarthur5 | Okay sounds good :) | 17:14 |
| TheJulia | (if it wasn't obvious, this is how I roll to get the needle to move overall) | 17:14 |
| * TheJulia is evil | 17:14 | |
| TheJulia | (... for good?) | 17:15 |
| * TheJulia goes back to whisteling innocently while the oven heats up for lunch | 17:17 | |
| TheJulia | iurygregory: I added you as a reviewer on https://review.opendev.org/c/openstack/sushy/+/961730 ... It wouldn't be an awful thing to try and spin, and I would be totally okay if you want to -1 until you've had a chance to explicitly test it. | 18:48 |
| JayF | I'll note I don't have iDRAC 10 gear to test against right now but if there's anything I can do to help enable it in Ironic (including backporting if possible? 🤞) let me know. | 18:55 |
| TheJulia | Thanks, our machine availablility is also super limited. AIUI, only 4 shipping revisions have it, but obviously that will change so I'm semi-motivated to get this stuff sorted sooner rather than later | 18:59 |
| iurygregory | TheJulia, ack o/ | 19:07 |
| opendevreview | Merged openstack/ironic-inspector master: Add WSGI module entrypoint https://review.opendev.org/c/openstack/ironic-inspector/+/958068 | 19:32 |
| cardoe | TheJulia: I swear there's a spot in Ironic today that ignores the job cause it 404s | 20:07 |
| cardoe | I think I abandoned it | 20:10 |
| iurygregory | woot :eyes: | 20:13 |
| zigo | Hi there ! I noticed that for Flamingo, Ironic's API is at ironic.wsgi:application, while it's supposed to be at ironic.wsgi.api:application so it's consistent with other OpenStack projects. Would it be possible to fix this, for example by having api being an alias to __init__ ? | 20:22 |
| JayF | Can you cite the "supposed to be"? I'm game to be consistent but I am curious if we missed something. | 20:23 |
| zigo | JayF: Supposed to be <project>.wsgi.<service>:application, as per the TC resolution, and as what absolutely all other projects did, like so far: Blazar, Cyborg, Designate, Keystone and Magnum. So in the Ironic case, it should be ironic.wsgi.api:application, not just ironic.wsgi:application. | 20:32 |
| JayF | I'm pasting the last three lines of IRC into claude and seeing if it can whip up an alias :) | 20:34 |
| zigo | :) | 20:35 |
| zigo | JayF: I wrote this, this week, in an amazingly short amount of time, thanks to AI help: https://salsa.debian.org/openstack-team/services/neutron-ipv6-bgp-injector/ :) | 20:38 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add WSGI alias under ironic.wsgi.api https://review.opendev.org/c/openstack/ironic/+/961753 | 20:38 |
| zigo | Great ! :P | 20:39 |
| JayF | I'll note Ironic has a lot of wsgi services | 20:39 |
| JayF | that's just the only one we allow under an external runner | 20:39 |
| JayF | the others (e.g. IPA's API, JSON-RPC server) are started later in the process | 20:39 |
| TheJulia | cardoe: oh, my. | 20:41 |
| opendevreview | Merged openstack/networking-baremetal master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/networking-baremetal/+/961670 | 22:26 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/961704 | 22:27 |
| opendevreview | Merged openstack/ironic-inspector master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/ironic-inspector/+/961665 | 22:28 |
| iurygregory | https://github.com/openstack/ironic/blob/247ae57a22d1b48173398e940d62e531c237d66e/ironic/drivers/modules/drac/management.py#L313-L333 would it be this one ? | 22:29 |
| opendevreview | Merged openstack/networking-generic-switch master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/961641 | 22:29 |
| opendevreview | Merged openstack/sushy-tools master: Switch to pre-commit https://review.opendev.org/c/openstack/sushy-tools/+/953126 | 23:00 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!