| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Drop unnecessary allowlist_externals https://review.opendev.org/c/openstack/python-ironicclient/+/999360 | 05:05 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: ruff: Enable common checks https://review.opendev.org/c/openstack/python-ironicclient/+/999361 | 05:09 |
| opendevreview | Bartosz Bezak proposed openstack/networking-generic-switch stable/2026.1: batching: Fix missed batch result watches https://review.opendev.org/c/openstack/networking-generic-switch/+/999365 | 07:01 |
| rpittau | TheJulia: about the comment on https://review.opendev.org/c/openstack/ironic/+/989994, I don't think we needed to bump api version for nodehistory until the release? | 07:13 |
| opendevreview | Merged openstack/ironic-python-agent master: Set a minimal default mdns config key list for IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/999337 | 08:41 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: Add HPE iLO-style firmware update simulation for BMC, BIOS, NIC and SSD https://review.opendev.org/c/openstack/sushy-tools/+/999404 | 11:23 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: [WIP] Add HPE iLO-style firmware update simulation for BMC, BIOS, NIC and SSD https://review.opendev.org/c/openstack/sushy-tools/+/999404 | 11:24 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: [WIP] Add HPE iLO-style firmware update simulation for BMC, BIOS, NIC and SSD https://review.opendev.org/c/openstack/sushy-tools/+/999414 | 13:08 |
| TheJulia | rpittau: your bumping the object itself | 13:23 |
| TheJulia | so the version needs to increment when you do that. The object version is disjointed from the api version | 13:23 |
| rpittau | oh... right :/ | 13:24 |
| TheJulia | Sorry :( | 13:24 |
| * TheJulia feels bad | 13:24 | |
| rpittau | no, no, my bad! :D | 13:24 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: [WIP] Add HPE iLO-style firmware update simulation for BMC, BIOS, NIC and SSD https://review.opendev.org/c/openstack/sushy-tools/+/999414 | 13:35 |
| TheJulia | rpittau: a sushy release soon? I know there is one huge outstanding patch in review right now... unfortunately I think that will require sitting down with the schema to give a solid review for | 13:56 |
| rpittau | TheJulia: I was actually looking at that, I've hold it because of that specific patch, but if we're good I'm going to propose the release now | 14:01 |
| TheJulia | I went to review it yesterday and was like "I don't have the spoons for 1400 lines at the moment" | 14:01 |
| TheJulia | My plan is to review it today, but I suspect I'm no tthe only on ewith the limited spoon issue | 14:01 |
| TheJulia | The author really should have broken it up... and I really suspect it is AI given how insanely verbose the commit message is without actually telling us "why" | 14:02 |
| TheJulia | (I mean, aside from the schema) | 14:02 |
| rpittau | I opened it up myself multiple times but had to leave it there :/ | 14:04 |
| TheJulia | Honestly, I think that is feedback I'm going to leave on it. | 14:18 |
| TheJulia | And stress that if they wanted it sooner, they *really* ought to be advocating during our weekly meetings and whatnot. | 14:18 |
| rpittau | I will back that up :) | 14:20 |
| TheJulia | I'm currently on cid's IPA patches, once done with those and after a dog walking, I'm going to start on the sushy patch | 14:20 |
| dtantsur | TheJulia: hey, for the attestation work, have you by chance considered getting TPM information from Redfish instead of in-band? (I might be talking nonsense, I've just realized that Redfish exposes something in this area) | 14:21 |
| TheJulia | I loosely have | 14:21 |
| TheJulia | but honestly haven't floated it in here beyond a high level idea in some high bandwidth dsicussions | 14:21 |
| TheJulia | The *downside* is if we do that we're sort of more becoming a read only state observer, but that is still powerful in terms of information we can provide to operators. | 14:22 |
| TheJulia | you don't get *ALL* of the details, but we would need to almost become the "attestation tracker" as well which means firmware cleaning would also likely need to set a flag saying "hey, some of these masurements (0-7 might change, don't freak out TOO much" | 14:23 |
| cid | \o/ | 14:26 |
| TheJulia | cid: I'm a little concerned around just explicitly supporting a variety of transport options in https://review.opendev.org/c/openstack/ironic-python-agent/+/998381. I guess the big question is why? | 15:39 |
| TheJulia | did someone forget to make gerrit coffee this morning?!? | 15:42 |
| TheJulia | ugh, it was my vpn's dns died | 15:43 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add state, target_provision_state, and duration_seconds to node history https://review.opendev.org/c/openstack/ironic/+/989994 | 15:53 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Populate node_history state fields https://review.opendev.org/c/openstack/ironic/+/993208 | 15:53 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add request context user tracking to node history records https://review.opendev.org/c/openstack/ironic/+/993901 | 15:53 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add power tracking telemetry in node history https://review.opendev.org/c/openstack/ironic/+/996347 | 15:53 |
| JayF | TheJulia: dtantsur: rpittau: For mDNS, do we want to advisory that with an OSSN? The patches aren't really safe to backport imo | 17:19 |
| TheJulia | I'd just say OSSN, I'm not really sure the implementation was really functional, and definitely not in out of the box images. | 17:30 |
| TheJulia | rpittau: I semi-successfully made it through the sushy change | 17:31 |
| JayF | TheJulia: I said mDNS, you answered as if I said Container HWM | 17:34 |
| JayF | TheJulia: can you re-confirm that you responded with the right bug in mind? | 17:34 |
| TheJulia | oh oh oh | 17:34 |
| TheJulia | yeah | 17:34 |
| TheJulia | no, soryr | 17:35 |
| TheJulia | I guess I'm split on an advisory for mdns because 1) you basically had to force it into that state to begin with which is way outside of default cases, and 2) it would be semi-breaking to backport without doing additional actions | 17:36 |
| JayF | I'm thinking mainly booting blind agents | 17:40 |
| JayF | that's the irl likely scary scenario | 17:40 |
| TheJulia | you have to explicitly walk into that | 17:40 |
| JayF | yeah | 17:40 |
| JayF | but the OSSN would at least warn them they are gonna walk into it? | 17:41 |
| JayF | IDK, no advisory is fine too | 17:41 |
| TheJulia | so, an OSSN saying "don't do that unless you *REALLY* know what your doing | 17:41 |
| JayF | yeah | 17:41 |
| TheJulia | and if you do, here is a risk, and you may want some patches, but.... yeah. | 17:41 |
| TheJulia | Sorry for the confusion, my brain is sort of on a fast track to "weekend" | 17:47 |
| JayF | I'll draft an OSSN for mDNS | 17:56 |
| TheJulia | ok | 17:59 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: WIP: Add Portgroup/LAG support to ironic-networking https://review.opendev.org/c/openstack/ironic-specs/+/999448 | 18:48 |
| JayF | TheJulia: dtantsur: rpittau: RFR https://review.opendev.org/c/openstack/security-doc/+/999451 | 19:11 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!