| TheJulia | cardoe: ack | 00:31 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Remove integration for python-ironic-inspector-client https://review.opendev.org/c/openstack/python-ironicclient/+/972420 | 02:19 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: NetworkAdapter resource ID changes unpredictably across reboots https://review.opendev.org/c/openstack/ironic/+/972421 | 03:19 |
| rpittau | good morning ironic! o/ | 08:06 |
| abongale | good morning o/ | 09:54 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: demonstrate collecting sensors asynchronously https://review.opendev.org/c/openstack/ironic/+/970842 | 12:44 |
| alegacy | good morning folks... A few patches in my standalone networking series (starting at: https://review.opendev.org/c/openstack/ironic/+/971182) are ready to go if anyone can spare some time to set them free! | 12:53 |
| *** darmach3 is now known as darmach | 12:55 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Unedited AI dump: demonstrate collecting sensors asynchronously https://review.opendev.org/c/openstack/ironic/+/970842 | 14:00 |
| TheJulia | good morning | 14:18 |
| opendevreview | Matt Crees proposed openstack/bifrost master: Document IPA image download options https://review.opendev.org/c/openstack/bifrost/+/964145 | 14:33 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 14:43 |
| opendevreview | Clif Houck proposed openstack/ironic master: Rip out check for multiple physnets in vif_attach https://review.opendev.org/c/openstack/ironic/+/972494 | 14:43 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add ConfigLoader class to TBN and use it in TaskManager https://review.opendev.org/c/openstack/ironic/+/972495 | 14:43 |
| opendevreview | Merged openstack/ironic master: fix: report a better error in validate-interfaces https://review.opendev.org/c/openstack/ironic/+/972306 | 15:20 |
| TheJulia | clif: o/, regarding https://review.opendev.org/c/openstack/ironic/+/972494 could you update the commit message with more detail so reviewers can understand why | 15:39 |
| TheJulia | Thanks in advance | 15:39 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: fix: report a better error in validate-interfaces https://review.opendev.org/c/openstack/ironic/+/972511 | 16:15 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: fix: report a better error in validate-interfaces https://review.opendev.org/c/openstack/ironic/+/972513 | 16:16 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2024.2: fix: report a better error in validate-interfaces https://review.opendev.org/c/openstack/ironic/+/972514 | 16:17 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 16:35 |
| opendevreview | Clif Houck proposed openstack/ironic master: Rip out check for multiple physnets in vif_attach https://review.opendev.org/c/openstack/ironic/+/972494 | 16:41 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add ConfigLoader class to TBN and use it in TaskManager https://review.opendev.org/c/openstack/ironic/+/972495 | 16:41 |
| clif | TheJulia: Done, np | 16:42 |
| TheJulia | Thanks | 16:47 |
| cardoe | iurygregory: if ya got a sec... https://review.opendev.org/c/openstack/ironic/+/972413 | 17:01 |
| cardoe | or even dtantsur since you were involved in the original. | 17:01 |
| opendevreview | Merged openstack/python-ironicclient master: Remove integration for python-ironic-inspector-client https://review.opendev.org/c/openstack/python-ironicclient/+/972420 | 17:26 |
| TheJulia | cardoe: I need to double check (in a meeting at the moment, but I *suspect* your change is good there | 17:32 |
| TheJulia | and sort of what I was thinking as a possible option yesterday | 17:32 |
| dtantsur | cardoe: oh, interesting. I'm really curious why it does not affect metal3.. | 17:38 |
| TheJulia | I suspect his target version is pinned out | 17:44 |
| TheJulia | *suspect* | 17:44 |
| TheJulia | on tip of master or latest version, unless version pinned, I suspect metal3 would never hit it | 17:46 |
| cardoe | So I’m using stable/2025.2 | 17:56 |
| TheJulia | but the question I've been wondering is if the version is pinned to that specifically? | 18:01 |
| TheJulia | or older... | 18:01 |
| TheJulia | but, ultimately, running default state config for stable/2025.2 really should not hit it, which is what is bizaar | 18:02 |
| TheJulia | I guess what I'm trying to say, it doesn't make sense that target_version is (1, 0) | 18:09 |
| TheJulia | And the only way I can see influcing that is the rpc version pinning | 18:09 |
| TheJulia | Anyway, I've +2'ed the change, do agree with Dmitry's comments, not an issue really. Iury you might want to chime in but if you don't by this afternoon I'll approve it as-is. | 18:14 |
| opendevreview | Verification of a change to openstack/ironic master failed: Silence loud logging when no NetworkAdapters https://review.opendev.org/c/openstack/ironic/+/972343 | 18:31 |
| cardoe | TheJulia: I follow what you're saying. I'm happy to share any configs or whatever I can to figure out what the mis-config is. | 18:33 |
| cardoe | It seems like it started around when we upgraded to 2025.2 | 18:34 |
| TheJulia | is https://github.com/openstack/ironic/blob/master/ironic/conf/default.py#L378 set at all? | 18:34 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add retry logic for boot device changes during POST https://review.opendev.org/c/openstack/ironic/+/971150 | 18:36 |
| TheJulia | If set to 2025.1, it shouldn't create it. https://github.com/openstack/ironic/blob/master/ironic/common/release_mappings.py#L827 so there may be another issue someplace. | 18:37 |
| TheJulia | That being said, we're missing the 2025.2 release mappings... https://github.com/openstack/ironic/blob/master/ironic/common/release_mappings.py#L991 | 18:38 |
| * TheJulia cries | 18:38 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add release mapping alias for 2025.2 https://review.opendev.org/c/openstack/ironic/+/972537 | 18:43 |
| TheJulia | it looks like the mapping in cardoe's case would have to be set to 17.0 to create the case, so it feels like something else is going on | 18:44 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix heartbeat for steps with requires_ramdisk=False https://review.opendev.org/c/openstack/ironic/+/971152 | 19:42 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add retry logic for boot device changes during POST https://review.opendev.org/c/openstack/ironic/+/971150 | 19:57 |
| cardoe | No pin release is set | 20:09 |
| TheJulia | ... okay | 20:12 |
| TheJulia | hmm | 20:12 |
| opendevreview | Merged openstack/ironic master: fix: bios fields could not be fetched via the API https://review.opendev.org/c/openstack/ironic/+/972413 | 22:11 |
| cardoe | https://review.opendev.org/c/openstack/nova/+/972601 I think that'll make the error message a bit clearer to users. Probably some others that need to get included | 23:18 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: fix: bios fields could not be fetched via the API https://review.opendev.org/c/openstack/ironic/+/972602 | 23:18 |
| cardoe | That issue has to have happened with 2025.2 | 23:19 |
| cardoe | I see it immediately in logs after we went to 2025.2 | 23:20 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!