| opendevreview | nidhi proposed openstack/ironic master: Add Redfish LLDP data collection support to the Redfish inspection interface. https://review.opendev.org/c/openstack/ironic/+/967841 | 03:54 |
|---|---|---|
| opendevreview | nidhi proposed openstack/ironic master: Add Redfish LLDP data collection support to the Redfish inspection interface. https://review.opendev.org/c/openstack/ironic/+/967841 | 04:10 |
| rpittau | good morning ironic! happy friday! o/ | 08:13 |
| opendevreview | Bartosz Bezak proposed openstack/networking-generic-switch stable/2025.2: Fix incorrect command when unplugging bond subports https://review.opendev.org/c/openstack/networking-generic-switch/+/970798 | 08:40 |
| opendevreview | Bartosz Bezak proposed openstack/networking-generic-switch stable/2025.1: Fix incorrect command when unplugging bond subports https://review.opendev.org/c/openstack/networking-generic-switch/+/970799 | 08:40 |
| opendevreview | Verification of a change to openstack/ironic master failed: Move `check_image_size` to `deploy_utils` https://review.opendev.org/c/openstack/ironic/+/970181 | 10:34 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: asynchronous Redfish client https://review.opendev.org/c/openstack/ironic/+/970450 | 11:22 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: asynchronous Redfish client https://review.opendev.org/c/openstack/ironic/+/970450 | 11:29 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: asynchronous Redfish client https://review.opendev.org/c/openstack/ironic/+/970450 | 13:59 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: demonstrate collecting sensors asynchronously https://review.opendev.org/c/openstack/ironic/+/970842 | 13:59 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Don't add a link to Storage if it's not supported https://review.opendev.org/c/openstack/sushy-tools/+/970845 | 14:17 |
| cardoe | dtantsur: so I think you can still use all the sushy objects and just pass in your own http client | 15:01 |
| dtantsur | You need to sprinkle awaits virtually everywhere, so I don't really want to go down that path | 15:05 |
| dtantsur | (I had claude creating some sort of a glue code for me, but it was ugly, and involved eagerly loading resources) | 15:05 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: asynchronous Redfish client https://review.opendev.org/c/openstack/ironic/+/970450 | 15:13 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: demonstrate collecting sensors asynchronously https://review.opendev.org/c/openstack/ironic/+/970842 | 15:13 |
| dtantsur | In the end my thinking is: if we decide to go full async for redfish, it may be easier to just use an LLM to migrate all objects from sushy on top of this new API rather than trying to get a hybrid sushy | 15:23 |
| cardoe | Yeah agreed. So you make the objects agnostic and use the awaits outside of it. | 15:24 |
| dtantsur | Yeah, possibly. I haven't thought the 2nd step completely through, we're still in a PoC phase. | 15:25 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Don't add a link to Storage if it's not supported https://review.opendev.org/c/openstack/sushy-tools/+/970845 | 15:41 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Make post-firmware-update reboot conditional on component https://review.opendev.org/c/openstack/ironic/+/966344 | 17:47 |
| *** mdfr1 is now known as mdfr | 17:59 | |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Make post-firmware-update reboot conditional on component https://review.opendev.org/c/openstack/ironic/+/966344 | 18:00 |
| opendevreview | Merged openstack/ironic master: Include all relevant error messages in exception https://review.opendev.org/c/openstack/ironic/+/942664 | 18:17 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/965877/3 what the heck is ready to submit? | 18:47 |
| dtantsur | cardoe: weird, zuul is supposed to do this automatically | 18:59 |
| dtantsur | (this is for workflows where a human merges patches) | 18:59 |
| opendevreview | Merged openstack/ironic master: docs: a little more clarity around the state of ipxe+uefi https://review.opendev.org/c/openstack/ironic/+/966724 | 19:00 |
| JayF | dtantsur: cardoe recheck can sometimes unstick those | 19:03 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily at 20:00 UTC (approximately 55 minutes from now) for a project rename maintenance: https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/message/ZPIV7CTPXZUYKUCBYMQ3OKQZDXOSOQGF/ | 19:09 | |
| opendevreview | Nahian Pathan proposed openstack/ironic master: Reduce API calls when collecting sensor data with redfish https://review.opendev.org/c/openstack/ironic/+/955484 | 19:23 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a project rename maintenance: https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/message/ZPIV7CTPXZUYKUCBYMQ3OKQZDXOSOQGF/ | 20:02 | |
| opendevreview | Merged openstack/ironic-python-agent master: enable_vlan_interfaces: support identifying interfaces by MAC https://review.opendev.org/c/openstack/ironic-python-agent/+/955629 | 20:26 |
| cardoe | cid: didn't you have a patch for this? | 20:37 |
| cardoe | https://www.irccloud.com/pastebin/3DJrAcfg/ | 20:37 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: re-enable some pycodestyle checks not covered by ruff https://review.opendev.org/c/openstack/ironic/+/970918 | 21:05 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/970922 | 21:25 |
| cardoe | dtantsur: ^^ should get that message back for you | 21:27 |
| cid | cardoe, that's exactly the sort of error the patch is supposed to fix. | 21:30 |
| cid | Was this from CI or a local log? | 21:30 |
| cardoe | My local setup. I swear I just built that container yesterday or Wednesday from stable/2025.2 | 21:31 |
| opendevreview | Verification of a change to openstack/networking-generic-switch stable/2025.2 failed: Fix incorrect command when unplugging bond subports https://review.opendev.org/c/openstack/networking-generic-switch/+/970798 | 21:31 |
| * cid let me find the patch. | 21:32 | |
| cid | It was backported: https://review.opendev.org/c/openstack/ironic/+/966567 | 21:32 |
| cid | I wonder what might be going on. | 21:32 |
| * cardoe grumbles. | 21:38 | |
| cardoe | cid: its me | 21:38 |
| cardoe | I was trying to help test OSH stuff and I used the stable/2025.2 build from OSH. Which apparently has been failing since dang near halloween | 21:40 |
| opendevreview | Merged openstack/networking-generic-switch stable/2025.1: Fix incorrect command when unplugging bond subports https://review.opendev.org/c/openstack/networking-generic-switch/+/970799 | 23:03 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!