| opendevreview | Jacob Anders proposed openstack/sushy master: settings: GET Settings endpoint ETag when parent omits @Redfish.Settings.ETag https://review.opendev.org/c/openstack/sushy/+/1006671 | 01:49 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/1006680 | 04:22 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add Redfish BootProgress gate helper https://review.opendev.org/c/openstack/ironic/+/1005657 | 04:44 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Verify firmware application after reboot before resuming https://review.opendev.org/c/openstack/ironic/+/1005658 | 04:44 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Validate BMC stability only after BMC firmware updates https://review.opendev.org/c/openstack/ironic/+/1005659 | 04:44 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2025.2: [stable only] CI: Fall back to HTTPS when cirros 0.5.3 is not cached https://review.opendev.org/c/openstack/bifrost/+/1006542 | 07:27 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Switch netconf transport from paramiko to libssh https://review.opendev.org/c/openstack/networking-generic-switch/+/998062 | 07:50 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add Dell LC job status helper for multiple JIDs https://review.opendev.org/c/openstack/ironic/+/1005656 | 08:33 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add Redfish BootProgress gate helper https://review.opendev.org/c/openstack/ironic/+/1005657 | 08:33 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Verify firmware application after reboot before resuming https://review.opendev.org/c/openstack/ironic/+/1005658 | 08:33 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Validate BMC stability only after BMC firmware updates https://review.opendev.org/c/openstack/ironic/+/1005659 | 08:33 |
| opendevreview | Merged openstack/sushy master: Remove note about old pip's behavior https://review.opendev.org/c/openstack/sushy/+/1006413 | 09:21 |
| opendevreview | Merged openstack/sushy master: Skip tempest jobs for more local files https://review.opendev.org/c/openstack/sushy/+/1006405 | 09:21 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2025.1: [stable only] CI: Fall back to HTTPS when cirros 0.5.3 is not cached https://review.opendev.org/c/openstack/bifrost/+/1006697 | 09:39 |
| opendevreview | Merged openstack/ironic master: Cast `cpu.frequency` to a string https://review.opendev.org/c/openstack/ironic/+/1005718 | 09:55 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Switch netconf transport from paramiko to libssh https://review.opendev.org/c/openstack/networking-generic-switch/+/998062 | 10:33 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add MTU support to ironic-networking https://review.opendev.org/c/openstack/ironic/+/1006662 | 11:28 |
| opendevreview | Roussarie proposed openstack/ironic-specs master: Add spec for rebooting into the user image via kexec https://review.opendev.org/c/openstack/ironic-specs/+/1005717 | 11:51 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2025.1: [stable only] CI: Fall back to HTTPS when cirros 0.5.3 is not cached https://review.opendev.org/c/openstack/bifrost/+/1006697 | 12:11 |
| iurygregory | good morning ironic | 12:29 |
| TheJulia | good morning | 13:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Clarify VXLAN networking documentation https://review.opendev.org/c/openstack/ironic/+/1006731 | 14:01 |
| JayF | TheJulia: stevebaker[m]: I assume you all tried something like https://bugs.launchpad.net/ironic-python-agent/+bug/2167949 with the 4k image stuff? | 14:50 |
| TheJulia | we considered it, but that it would at best be an approximation and complete re-map | 14:51 |
| TheJulia | which gets into super weird territory | 14:51 |
| TheJulia | because your basically going to have to take a look at the partition format, fit it into your desired shape on the new target, and then mount/stream | 14:51 |
| TheJulia | that itself is asking for issues | 14:51 |
| JayF | a concern around fidelity of the converted image? | 14:52 |
| TheJulia | Well, there is no post-checksum matching | 14:53 |
| TheJulia | the actual structural layouts and sizes change | 14:53 |
| TheJulia | the padding all changes | 14:53 |
| TheJulia | so, is it *really* the same image at that point | 14:54 |
| JayF | that's more or less the feeling I had | 14:54 |
| JayF | but I'm not sure it'll be compelling to operators unless we can point to a specific issue | 14:54 |
| TheJulia | It starts to get to a point of just because we can, doesn't really mean we should. | 14:54 |
| TheJulia | which is a separate question/discussion | 14:54 |
| JayF | I mean, having to manage two images based solely on generation of hardware *sucks a lot* and breaks a lot of our abstractions | 14:54 |
| TheJulia | I guess, I didn't push it because I didn't feel like having a "my partitions are different" discussion with someone who got a silent conversion | 14:55 |
| JayF | well that's where I was going | 14:55 |
| JayF | if we did it, given we can't assure fidelity | 14:55 |
| JayF | we could do it opt-in | 14:55 |
| TheJulia | when really, it should be documented starting point for the hardware to know you need 4k blocks | 14:55 |
| TheJulia | maybe, that is a trait we should have | 14:55 |
| JayF | yeah, but some of the value we provide is covering up some of the sharp hardware edges | 14:56 |
| JayF | this one is extra sharp | 14:56 |
| TheJulia | oh yeah | 14:56 |
| TheJulia | super sharp edge | 14:56 |
| JayF | I'll put it on the PTG, with the suggestion being to have a conversion as an opt-in option | 14:56 |
| JayF | either in IPA config, Ironic config, or per-node | 14:56 |
| TheJulia | sounds good | 14:56 |
| TheJulia | I went ahead and replied with "why I didn't do this" in the past, basically captured some of what we just hit upon | 15:06 |
| TheJulia | Definitely a good ptg topic | 15:06 |
| TheJulia | Could I get some quick reviews on https://review.opendev.org/q/project:openstack/networking-generic-switch+topic:ngs-configdoc which are cleanups of the configuraiton documentation. It would make things more clear for someone I have reading the docs to do downstream docs. | 15:14 |
| JayF | oh that one looks real good | 15:24 |
| JayF | and good timing on when to land it too | 15:25 |
| JayF | GR-OSS team group review is happening in ~10 minutes, if anyone wants to join just ask and I'll drop the zoom link here when it starts | 16:50 |
| TheJulia | I'd be happy to, saves me from Jira | 16:56 |
| JayF | https://us06web.zoom.us/j/81185922171?pwd=2cMJ6jDDJoXbKylXGCQiUmTJrObH2o.1 | 17:01 |
| JayF | I'm grabbing a soda then jumping in | 17:01 |
| TheJulia | k | 17:01 |
| * TheJulia grabs a jolt cola | 17:01 | |
| TheJulia | is that the right link? | 17:02 |
| TheJulia | JayF: ^^ | 17:03 |
| JayF | I gotta sign in to kick some other meeting out, apparently lol | 17:03 |
| TheJulia | doh! | 17:03 |
| JayF | https://us06web.zoom.us/j/84752772101?pwd=cWGuPg94g4fh4IiH7jXSUNqOh8ebHX.1&jst=2 | 17:04 |
| JayF | totally the wrong link | 17:04 |
| JayF | sorry clif and cid who I probably just kicked out | 17:04 |
| JayF | TheJulia: ^ | 17:04 |
| TheJulia | Another quickie docs update: https://review.opendev.org/c/openstack/networking-baremetal/+/1006635 | 17:16 |
| cardoe | TheJulia: so OpenStack Helm seems to have changed rabbit_quorum_queue and rabbit_transient_quorum_queue to true | 17:17 |
| cardoe | Should transient not be quorum? | 17:17 |
| TheJulia | Well... its complicated | 17:17 |
| TheJulia | Realistically you use quorum queues to make the contents durable, but then they never delete for messages and such. Most of the services and patterns were not written with that neverending message in the queue model, because the result expects write once with multiple durable readers needing to be able to access the prior data, where as our model is "hey, I just put the thing in, move on!" | 17:19 |
| TheJulia | so... its a mismatch but partially rooted because you reduce message bus performance | 17:19 |
| TheJulia | because also, with quorum queues, your message is not submitted/accepted until its on multiple nodes | 17:20 |
| cardoe | so it seems like transient shouldn't be quorum then | 17:36 |
| cardoe | so neutron uses some exclusive feature for fanout which is why those need to be quorum now too? | 17:42 |
| * cardoe shrugs. | 17:42 | |
| cardoe | I am not a rabbit expert. | 17:43 |
| frickler | can we expect 2026.2 ipa images soon or should we disable the job in devstack for now? https://review.opendev.org/c/openstack/devstack/+/1006721/1 | 18:02 |
| opendevreview | Merged openstack/ironic master: Clarify VXLAN networking documentation https://review.opendev.org/c/openstack/ironic/+/1006731 | 18:06 |
| JayF | I would've expected some to have been posted | 18:06 |
| JayF | https://review.opendev.org/q/repo:openstack/ironic-python-agent+and+branch:stable/2026.2 | 18:07 |
| JayF | we have branched and landed the rote packages | 18:07 |
| JayF | s/packages/patches/ | 18:07 |
| JayF | unsure what else we might need to do? | 18:07 |
| opendevreview | Merged openstack/networking-generic-switch master: Describe internal per-switch options as cfg.Opt instances https://review.opendev.org/c/openstack/networking-generic-switch/+/1006314 | 18:13 |
| opendevreview | Merged openstack/networking-generic-switch master: Generate per-switch config option reference from cfg.Opt https://review.opendev.org/c/openstack/networking-generic-switch/+/1006315 | 18:13 |
| opendevreview | Merged openstack/networking-generic-switch master: Deduplicate switch option docs against generated reference https://review.opendev.org/c/openstack/networking-generic-switch/+/1006316 | 18:13 |
| TheJulia | ... that should have created artifacts | 18:15 |
| TheJulia | https://tarballs.opendev.org/openstack/ironic-python-agent/dib/ipa-centos9-stable-2026.2.tar.gz | 18:16 |
| TheJulia | so... done? | 18:16 |
| JayF | https://tarballs.opendev.org/openstack/ironic-python-agent/dib/files/ the file layout here is different than devstack change wants | 18:17 |
| JayF | ++ /opt/stack/ironic/devstack/lib/ironic:download_with_retry:830 : wget --timeout=30 --tries=3 -q https://tarballs.openstack.org/ironic-python-agent/dib/files/ipa-centos9-2026.2.kernel -O /opt/stack/devstack/files/ir-deploy-ipmi.kernel | 18:18 |
| JayF | I need to finish talking w/Cardoe and then get lunch before I could dig into this. If someone else has time that'd be ideal but if not I'll look into it | 18:20 |
| TheJulia | ahh, yeah, because the switch to centos10 was done right around release so it wouldn't surprise me that something is missing | 18:28 |
| TheJulia | rpittau: ^^^ | 18:28 |
| opendevreview | Merged openstack/sushy-tools master: Skip tempest jobs for more local files https://review.opendev.org/c/openstack/sushy-tools/+/1006406 | 18:52 |
| opendevreview | Merged openstack/sushy-tools master: Add ComputerSystem SerialNumber derived from domain UUID https://review.opendev.org/c/openstack/sushy-tools/+/991906 | 18:52 |
| opendevreview | Verification of a change to openstack/networking-baremetal stable/2026.1 failed: Guard rabbit config access in _get_notification_transport_url https://review.opendev.org/c/openstack/networking-baremetal/+/1005794 | 18:54 |
| opendevreview | Merged openstack/sushy-tools master: Add empty FirmwareInventory collection stub under UpdateService https://review.opendev.org/c/openstack/sushy-tools/+/991905 | 19:00 |
| TheJulia | cardoe: were you going to follow-up on those doc items or did you want me to? | 19:09 |
| opendevreview | Merged openstack/networking-baremetal master: Docs: Fix L2VNI driver name and reconcile driver ordering https://review.opendev.org/c/openstack/networking-baremetal/+/1006635 | 19:18 |
| cardoe | TheJulia: either way | 19:31 |
| TheJulia | cardoe: We're seeking best clarity, why don't you post quick updates for what you think and I can reply | 19:31 |
| TheJulia | (or just workflow) | 19:31 |
| TheJulia | (that keeps the context switch cost a little loswer | 19:31 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal stable/2026.2: Docs: Fix L2VNI driver name and reconcile driver ordering https://review.opendev.org/c/openstack/networking-baremetal/+/1006807 | 19:32 |
| * TheJulia has a wash crew washing the rig now and the corgi is filing all possible complaints | 19:32 | |
| cardoe | that works | 19:32 |
| cardoe | We can nudge those in for now | 19:32 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal stable/2026.1: Docs: Fix L2VNI driver name and reconcile driver ordering https://review.opendev.org/c/openstack/networking-baremetal/+/1006808 | 19:32 |
| rm_work | You have a corg? I feel like mentioning that makes corg tax due | 19:32 |
| TheJulia | Corgi tax https://usercontent.irccloud-cdn.com/file/jY80lOzQ/1790105663.JPG | 19:34 |
| rm_work | cardoe: Become an un-rabbit expert like me 👍 | 19:34 |
| rm_work | TheJulia: Perfect, noted and filed | 19:35 |
| TheJulia | he just got a sedative because he also somehow hurt himself yesterday and he needs to rest as well. | 19:37 |
| JayF | my doggo has been sick and is on bland food; which for him is rice+ground beef+pumpkin | 19:39 |
| JayF | only time I have ever made dog food that I was like "that smells GREAT" lol | 19:39 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: Name the listener pool queue after [DEFAULT] host https://review.opendev.org/c/openstack/networking-baremetal/+/1006663 | 20:07 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: Make enable_l2vni_trunk_reconciliation the master switch https://review.opendev.org/c/openstack/networking-baremetal/+/1006660 | 20:07 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: Remove duplicate OVN LocalnetPortEvent registration https://review.opendev.org/c/openstack/networking-baremetal/+/1006816 | 20:07 |
| cardoe | JayF: ^ should address your concerns | 20:07 |
| cardoe | I'm also not sold on making things the master switch | 20:07 |
| cardoe | But that's how we've documented things. | 20:07 |
| TheJulia | Any ops questions to surface up: https://etherpad.opendev.org/p/ops-devs-qanda-2026 | 20:08 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix file descriptor leak in Swift client connections https://review.opendev.org/c/openstack/ironic/+/1005971 | 20:30 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix file descriptor leak in Cinder client connections https://review.opendev.org/c/openstack/ironic/+/1005972 | 20:30 |
| TheJulia | rebases | 20:30 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: docs: revise the docs depth for improved browsing clarity https://review.opendev.org/c/openstack/networking-generic-switch/+/1006823 | 20:48 |
| JayF | frickler: https://review.opendev.org/c/openstack/devstack/+/1006721/ it's a bug in the change; but since hopefully you're asleep I'll push a patchset | 21:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Validate ansible_playbooks_path against permitted directories https://review.opendev.org/c/openstack/ironic/+/1006848 | 22:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Enforce ansible_playbooks_path by default https://review.opendev.org/c/openstack/ironic/+/1006849 | 22:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Write ansible configdrive to a private temporary directory https://review.opendev.org/c/openstack/ironic/+/1006850 | 22:39 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily while we upgrade to a new patch release, but should return within a few minutes | 23:01 | |
| opendevreview | Merged openstack/networking-baremetal stable/2026.1: Guard rabbit config access in _get_notification_transport_url https://review.opendev.org/c/openstack/networking-baremetal/+/1005794 | 23:09 |
| opendevreview | Merged openstack/networking-baremetal stable/2026.1: Docs: Fix L2VNI driver name and reconcile driver ordering https://review.opendev.org/c/openstack/networking-baremetal/+/1006808 | 23:09 |
| opendevreview | Merged openstack/networking-baremetal stable/2026.2: Docs: Fix L2VNI driver name and reconcile driver ordering https://review.opendev.org/c/openstack/networking-baremetal/+/1006807 | 23:09 |
| opendevreview | Merged openstack/networking-generic-switch master: Switch netconf transport from paramiko to libssh https://review.opendev.org/c/openstack/networking-generic-switch/+/998062 | 23:46 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
