| opendevreview | Verification of a change to openstack/ironic master failed: Add PCIe function fields to redfish inspection https://review.opendev.org/c/openstack/ironic/+/963179 | 02:04 |
|---|---|---|
| *** hroy_ is now known as hroy | 02:37 | |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Add podman package to ironic-python-agent https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/970336 | 03:35 |
| opendevreview | Merged openstack/ironic master: Add PCIe function fields to redfish inspection https://review.opendev.org/c/openstack/ironic/+/963179 | 04:11 |
| rpittau | good morning ironic! o/ | 09:15 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Soft power off before cleaning up inspection ramdisk https://review.opendev.org/c/openstack/ironic/+/970388 | 13:52 |
| dtantsur | TheJulia: hey, have you seen this? https://bugs.launchpad.net/ironic-python-agent-builder/+bug/2133509 | 13:55 |
| dtantsur | stevebaker[m]: if you're familiar with that part ^^ | 14:01 |
| dtantsur | rpittau: could we release IPA-buidler to get https://opendev.org/openstack/ironic-python-agent-builder/commit/9aca8fd38fbef23719675bb71fd12d133f228460 out? It seems that the folks may be hitting issues without it. | 14:08 |
| rpittau | dtantsur: sure | 14:08 |
| opendevreview | cid proposed openstack/ironic master: `is-empty` inspection rule to handle missing field https://review.opendev.org/c/openstack/ironic/+/968999 | 14:32 |
| opendevreview | cid proposed openstack/ironic master: Move `check_image_size` to `deploy_utils` https://review.opendev.org/c/openstack/ironic/+/970181 | 14:33 |
| rpittau | dtantsur: https://review.opendev.org/c/openstack/releases/+/970400 | 15:16 |
| dtantsur | rpittau: thanks! | 16:16 |
| TheJulia | dtantsur: it is on my list to look at when I'm back next week | 16:40 |
| dtantsur | TheJulia: probably don't worry about it, it could be outdated IPA-builder on their side | 16:44 |
| TheJulia | it should be a quiet week :)_ | 16:45 |
| JayF | don't put that curse on us | 16:50 |
| dtantsur | :D | 16:51 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/969318 this has 2x+2, if someone wants to +a it I would be able to update the nova follow-up patch | 17:13 |
| cardoe | done. I had meant to but I squirreled | 17:15 |
| JayF | danke | 17:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: asynchronous Redfish client https://review.opendev.org/c/openstack/ironic/+/970450 | 17:29 |
| dtantsur | the outcome of claude hallucinating for 2 days ^^ | 17:29 |
| dtantsur | I haven't read it myself, and I don't have a good feeling about it given that it did not manage to fix the tests it wrote | 17:30 |
| dtantsur | but maybe there is some value, I'll check later this week | 17:30 |
| JayF | FYI, the nova-ironic driver fix around servicing/hold states is landing in 2024.2, once merged it'll be in all supported branches. I'm sending it thru UM as well | 17:34 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Redfish: guard retrieve_nic_components against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/969788 | 17:59 |
| opendevreview | Verification of a change to openstack/ironic master failed: `is-empty` inspection rule to handle missing field https://review.opendev.org/c/openstack/ironic/+/968999 | 18:11 |
| opendevreview | Merged openstack/ironic stable/2024.2: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/969123 | 18:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: Separate states code from states constants https://review.opendev.org/c/openstack/ironic/+/969318 | 18:29 |
| opendevreview | Verification of a change to openstack/bifrost master failed: ILO driver retirement https://review.opendev.org/c/openstack/bifrost/+/965177 | 18:29 |
| opendevreview | cid proposed openstack/ironic master: Include all relevant error messages in exception https://review.opendev.org/c/openstack/ironic/+/942664 | 18:31 |
| JayF | https://bugs.launchpad.net/ironic/+bug/2134566 is a conspicuous miss | 19:16 |
| JayF | makes me wonder if there's something hard under the hood | 19:16 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [api-ref] Fix api-ref to mention list port by cg https://review.opendev.org/c/openstack/ironic/+/970482 | 19:17 |
| opendevreview | Verification of a change to openstack/ironic master failed: Separate states code from states constants https://review.opendev.org/c/openstack/ironic/+/969318 | 20:13 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Support filtering portgroups by shard https://review.opendev.org/c/openstack/ironic/+/970560 | 22:01 |
| JayF | cid: ^ as discussed earlier fyi | 22:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Support filtering portgroups by shard https://review.opendev.org/c/openstack/ironic/+/970560 | 22:10 |
| cid | ++++ | 22:19 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient master: Filter portgroups by shard https://review.opendev.org/c/openstack/python-ironicclient/+/970562 | 22:23 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient master: Filter portgroups by shard https://review.opendev.org/c/openstack/python-ironicclient/+/970562 | 22:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Separate states code from states constants https://review.opendev.org/c/openstack/ironic/+/969318 | 23:19 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!