| opendevreview | Merged openstack/networking-generic-switch master: Add opt-in MTU management for switch ports https://review.opendev.org/c/openstack/networking-generic-switch/+/984832 | 03:15 |
|---|---|---|
| opendevreview | Merged openstack/networking-generic-switch master: Add opt-in port carrier bounce on bind https://review.opendev.org/c/openstack/networking-generic-switch/+/984833 | 03:35 |
| opendevreview | Merged openstack/ironic-python-agent master: Erase frozen SATA SSDs by discarding them https://review.opendev.org/c/openstack/ironic-python-agent/+/1001323 | 05:44 |
| dtantsur | cardoe: the more I look at inventory, the more desperate I get. We somehow managed to merge the completely non-standard storage_controllers collection, while mostly neglecting standard disks... | 12:42 |
| cardoe | The disk info from IPA was useless though. | 12:42 |
| cardoe | The storage controller stuff includes enough details to drive RAID configs. | 12:43 |
| dtantsur | cardoe: I can get it, but we made it even more useless | 12:43 |
| cardoe | Works for me for RAID | 12:43 |
| dtantsur | while virtually all current consumers rely on IPA format (e.g. gophercloud and thus metal3) | 12:43 |
| dtantsur | "disks": [ | 12:43 |
| dtantsur | { | 12:44 |
| dtantsur | "name": "Solid State Disk 0:0", | 12:44 |
| dtantsur | "size": 400088457216 | 12:44 |
| dtantsur | }, | 12:44 |
| dtantsur | { | 12:44 |
| dtantsur | "name": "Solid State Disk 1:1", | 12:44 |
| dtantsur | "size": 200049647616 | 12:44 |
| dtantsur | } | 12:44 |
| dtantsur | ], | 12:44 |
| dtantsur | This doesn't even have fields available in storage_controller.. | 12:44 |
| dtantsur | I also think we managed to boot pci_devices into inventory instead of plugin_data | 12:47 |
| dtantsur | which means the accelerators hook won't work on it. Sigh. SIGH. | 12:48 |
| cardoe | PCI-devices should be in inventory | 13:01 |
| cardoe | It’s lspci data | 13:01 |
| dtantsur | I don't disagree with that, but it hasn't been there | 13:01 |
| dtantsur | Even our own code assumes it to be in plugin_data | 13:02 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Improve compatibility between IPA and Redfish inspection https://review.opendev.org/c/openstack/ironic/+/1005010 | 13:09 |
| dtantsur | cardoe: this should be a reasonable improvement ^^ | 13:09 |
| * dtantsur tests | 13:09 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Improve compatibility between IPA and Redfish inspection https://review.opendev.org/c/openstack/ironic/+/1005010 | 13:40 |
| cardoe | dtantsur: how did IPA upload to plugin_data in the first place/ | 15:07 |
| cardoe | I guess that's allowed via the API. I thought plugin_data was private to the hooks so it wouldn't be uploaded by IPA at all. | 15:07 |
| dtantsur | cardoe: everything that is not under the "inventory" root key ends up in "plugin_data" | 15:07 |
| cardoe | oh | 15:07 |
| cardoe | So I'm guessing a whoops on the IPA side for pci_devices? | 15:07 |
| cardoe | Cause I would think before hooks run that plugin_data would be entirely empty. | 15:08 |
| dtantsur | cardoe: less a whooops, more of an evolution of how we see this entire topic | 15:08 |
| dtantsur | but yeah. we tried to make it less of a mess, and we kind of did, but it's still a mess | 15:08 |
| dtantsur | btw, I got a sample for my ancient PowerEdge FC430, if that's of interest | 15:09 |
| cardoe | yeah. I still need to pull from mine | 15:09 |
| * dtantsur is pondering constructing /dev/disk/by-path/* from Redfish | 15:23 | |
| JayF | https://bugs.launchpad.net/ironic/+bug/2166877 we need to address this ASAP | 16:18 |
| dtantsur | ugh | 16:18 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix device class detection for PCI devices during Redfish inspection https://review.opendev.org/c/openstack/ironic/+/1005083 | 16:25 |
| dtantsur | cardoe: and another fun one ^^ (I'm soooo annoyed by this enum behavior) | 16:25 |
| cardoe | dtantsur: str(enum_thing) only sucks on Python 3.10 and older. Now that we're Python 3.11+ we can fix that. | 16:38 |
| dtantsur | cardoe: I'm quite sure I'm on 3.12. Or do you mean we need to do something else? | 16:39 |
| * dtantsur will check back tomorrow | 16:44 | |
| cardoe | dtantsur: we need to update sushy. we're using the old enum behavior | 16:47 |
| cardoe | We need to tell it to use new enum behavior | 16:48 |
| cardoe | dtantsur: btw... https://review.opendev.org/c/openstack/ironic-specs/+/1004909 if ya get time | 20:29 |
| cardoe | cid: I think that might be related to something you looked at too? | 20:29 |
| cid | ++, will look at tomorrow | 20:30 |
| JayF | https://bugs.launchpad.net/ironic/+bug/2166877 was on 29.0.5 which implies to me the bug was already fixed | 20:37 |
| JayF | I'm digging but I think we're OK | 20:37 |
| JayF | marking 2166877 as invalid; 29.0.6 has teh OSSA-2026-026 fix in it which removes that issue | 20:42 |
| JayF | zigo: ^ JFYI; that was a debian user, if you wanna reach out | 20:42 |
| zigo | Who? | 20:43 |
| JayF | the linked ironic bug | 20:45 |
| JayF | the reporter is a debian user, in case you wanted to reach out | 20:46 |
| JayF | I closed as invalid as they have 29.0.5 and the bug they reported is fixed in 29.0.6 | 20:46 |
| opendevreview | Doug Goldstein proposed openstack/ironic-specs master: Add spec for BMC settings management https://review.opendev.org/c/openstack/ironic-specs/+/1004909 | 20:55 |
| zigo | JayF: I do have 1:29.1.0-1~bpo13+1 packaged, but in osbpo.debian.net, not in Debian proper, I'll let him know in the bug. | 21:24 |
| zigo | JayF: For Debian proper, maybe I can backport the patch if you can tell me where to find it? | 21:25 |
| zigo | I replied in the launchpad bug. | 21:28 |
| JayF | the advisory is linked there in my comment, which links to the patches | 21:28 |
| zigo | BTW, is 38.0.0 the final version for Hibiscus, or should I hold my breath for a new one ? | 21:29 |
| JayF | new one will be incoming | 21:29 |
| JayF | deadline is week of 9/20ish | 21:29 |
| zigo | Ok. | 21:29 |
| zigo | The version he's talking about should have the fix already, normally. | 21:30 |
| zigo | It's green here: https://wiki.debian.org/OpenStack/CVE-2026?pow_referer= | 21:30 |
| JayF | I know it's happy in the branch; IDK what that user's got going on :) | 21:30 |
| * zigo goes to bed, it's late here | 21:31 | |
| zigo | Night night ! | 21:31 |
| TheJulia | Good morning | 21:55 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!