| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [PoC] Unedited AI dump: asynchronous Redfish client https://review.opendev.org/c/openstack/ironic/+/970450 | 02:46 |
|---|---|---|
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add RestconfSwitch base class for HTTP transport https://review.opendev.org/c/openstack/networking-generic-switch/+/992887 | 09:25 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add RestconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/992888 | 09:26 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add RESTCONF documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/992889 | 09:27 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Rename netconf_models to yang_models https://review.opendev.org/c/openstack/networking-generic-switch/+/993090 | 09:28 |
| dtantsur | cardoe: oh, interesting! I have some use cases too, let's cooperate. | 10:33 |
| dtantsur | JayF, iurygregory_, anything blocking W+1 on https://review.opendev.org/c/openstack/ironic/+/1004653 and https://review.opendev.org/c/openstack/sushy/+/1004664? | 11:55 |
| *** iurygregory_ is now known as iurygregory | 12:17 | |
| iurygregory | good morning ironic | 12:18 |
| iurygregory | dtantsur, looking | 12:18 |
| iurygregory | changed my votes to include +W | 12:18 |
| dtantsur | cardoe: I got samples for Dell R6525 and HPE DL380. The latter has no SimpleStorage and no members in Storage, which is pretty unfortunate. | 12:22 |
| dtantsur | iurygregory: thx! | 12:22 |
| iurygregory | yw | 12:25 |
| dtantsur | If you have more time today, https://review.opendev.org/c/openstack/ironic/+/1005010 and https://review.opendev.org/c/openstack/ironic/+/1005083 are the next fixes | 12:29 |
| iurygregory | I do, will look at it in a bit o/ | 12:29 |
| iurygregory | checking the bugs since I'm the bug deputy :D | 12:30 |
| kubajj | iurygregory: if https://bugs.launchpad.net/ironic/+bug/2166702 does not make sense at all or there's a reason for it not being checked, lmk | 12:31 |
| iurygregory | WOW =O | 12:32 |
| iurygregory | interesting scenario! | 12:32 |
| iurygregory | gotta love BMCs sending 202 and rejecting things and what you request doesn't work lol | 12:33 |
| kubajj | Indeed, was also surprised. I tried to draft https://review.opendev.org/c/openstack/ironic/+/1004570 but wanted to sanity check before implementing tests | 12:34 |
| iurygregory | I saw that, I've skipped because it was marked with WIP | 12:35 |
| iurygregory | This case sounds a bit familiar with something a saw, I will do some double check also | 12:36 |
| kubajj | iurygregory: silly me, should have not marked it WIP to collect feedback | 12:36 |
| iurygregory | yeah! | 12:38 |
| *** mdfr1 is now known as mdfr | 12:39 | |
| iurygregory | will also add to my list to review later today | 12:39 |
| kubajj | thanks | 12:39 |
| opendevreview | Merged openstack/sushy master: Fix accessing PCIe functions when Redfish does not support them https://review.opendev.org/c/openstack/sushy/+/1004664 | 13:08 |
| opendevreview | Merged openstack/ironic master: Guard inspection PCIe devices against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/1004653 | 13:48 |
| cardoe | dtantsur: ^ needs to depend on a newer sushy with the other change | 14:23 |
| cardoe | No? | 14:24 |
| cardoe | oh wait no. | 14:24 |
| dtantsur | cardoe: not necessarily. What the sushy change does is to turn HTTP 404 into MissingAttributeError. | 14:24 |
| cardoe | yes let's work together on the BMC settings for sure | 14:28 |
| cardoe | hjensas: https://bugs.launchpad.net/neutron/+bug/2167105 wanted your thoughts... my fix would likely mess up NGS / networking-baremetal which assumes something does that for it | 14:33 |
| JayF | dtantsur: Unless it's something that needs to urgently land, I usually leave workflowing on sushi to folks who have worked more in that codebase directly | 14:44 |
| JayF | *sushy | 14:44 |
| dtantsur | okie, noted | 14:45 |
| opendevreview | Merged openstack/ironic master: Improve compatibility between IPA and Redfish inspection https://review.opendev.org/c/openstack/ironic/+/1005010 | 14:45 |
| opendevreview | Doug Goldstein proposed openstack/sushy master: Read the ReadOnly attribute property in any case https://review.opendev.org/c/openstack/sushy/+/1005271 | 14:58 |
| opendevreview | Doug Goldstein proposed openstack/sushy master: Let callers ask a Dell manager for a named attribute resource https://review.opendev.org/c/openstack/sushy/+/1005272 | 14:58 |
| cardoe | dtantsur: ^ just some things that got noticed while looking into the BMC stuff | 14:59 |
| dtantsur | Readonly, sigh. | 14:59 |
| cardoe | vendors... the gift that keeps on giving | 15:00 |
| dtantsur | Indeed. Both lgtm. | 15:01 |
| cardoe | Those two changes now let me replace my requests scripts with sushy | 15:03 |
| dtantsur | \o/ | 15:03 |
| opendevreview | fivetime proposed openstack/networking-generic-switch master: Configure link aggregations declaratively, never their member ports https://review.opendev.org/c/openstack/networking-generic-switch/+/1005277 | 15:04 |
| opendevreview | fivetime proposed openstack/networking-generic-switch master: Route trunk subport changes to the aggregate as well https://review.opendev.org/c/openstack/networking-generic-switch/+/1005278 | 15:04 |
| cardoe | fivetime are you on IRC? | 15:06 |
| cardoe | I think this will rub weird with JayF's LAG stuff. | 15:06 |
| cardoe | It's also a different approach than we use in our mechanism. | 15:06 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: Add BMC settings management https://review.opendev.org/c/openstack/ironic/+/1005281 | 15:08 |
| dtantsur | wow, you're not losing any time :) | 15:09 |
| cardoe | very rough WIP but I figured I'd post it. | 15:10 |
| cardoe | I didn't take into account your feedback yet. | 15:10 |
| dtantsur | Sure, no worries. Nothing there is super blocking. As long as the discussion happens, it's all good. | 15:10 |
| cardoe | I fleshed out some rough parts of the interface and committed it. And then I told Claude to use some of my prior changes as a template for how to do the API. And to write a bunch of tests and then to walk through using it in a mock environment to set a bunch of settings. | 15:12 |
| opendevreview | Merged openstack/ironic master: Fix device class detection for PCI devices during Redfish inspection https://review.opendev.org/c/openstack/ironic/+/1005083 | 15:49 |
| dtantsur | cardoe: do you have any HPEs there? I'm looking at this DL380, and it does not have anything in its Storage collection even on the latest firmware version. And it uses non-standard PCIDevices (vs PCIeDevices) | 16:01 |
| dtantsur | (this is iLO 5, maybe iLO 6 is less of a mess) | 16:01 |
| cardoe | yeah that's an iLO5 thing. They've not included it until iLO6 | 16:10 |
| dtantsur | Okay, it's good to know that at least iLO 6 has it. | 16:10 |
| cardoe | You have to use the proliantutils and the ilo5 driver in Ironic to get that data but we ripped it out. | 16:10 |
| dtantsur | I don't care all that much about iLO 5, I just don't have iLO 6 handy for testing. | 16:10 |
| cardoe | That was how we walked the Oem entries in Redfish for HPE | 16:11 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2026.1: Guard inspection PCIe devices against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/1005291 | 16:19 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/38.0: Guard inspection PCIe devices against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/1005292 | 16:19 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/37.0: Guard inspection PCIe devices against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/1005293 | 16:20 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/2026.2: Fix accessing PCIe functions when Redfish does not support them https://review.opendev.org/c/openstack/sushy/+/1005294 | 16:20 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/2026.1: Fix accessing PCIe functions when Redfish does not support them https://review.opendev.org/c/openstack/sushy/+/1005295 | 16:20 |
| TheJulia | I see I’m going to be very busy next week | 18:20 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: Fail loudly when the console browser does not start https://review.opendev.org/c/openstack/ironic/+/1005360 | 22:01 |
| cardoe | stevebaker[m]: ^ | 22:02 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!