| JayF | iurygregory: can you express this coherently in a public bug someplace? | 00:56 |
|---|---|---|
| JayF | iurygregory: I'll do what I can to get it to the right folks myself too. If enough of us ask they eventually have to listen, right? RIGHT!? | 00:57 |
| iurygregory | JayF, I think I added in the bug for the change | 01:37 |
| iurygregory | JayF, https://bugs.launchpad.net/ironic/+bug/2125571 | 01:38 |
| iurygregory | While working on the different problems to provision iDRAC10, we discovered that iso images should only be used on the Virtual Media Slot 1. | 01:38 |
| iurygregory | In ironic we will pick the first slot returned that has CD/DVD in MediaTypes, the problem is that in the R7715 I have access the first slot returned is Slot2 and it has CD/DVD MediaTypes (I consider this a firmware bug on the Dell side, but we need a workaround in ironic for such case) | 01:38 |
| iurygregory | I'm wondering if we can point to https://github.com/dell/iDRAC-Redfish-Scripting/issues/324#issuecomment-2892410726 (but not 100% sure) | 01:39 |
| opendevreview | Merged openstack/ironic master: Force Virtual Media Slot 1 on iDRAC10 https://review.opendev.org/c/openstack/ironic/+/961989 | 03:35 |
| rpittau | good morning ironic! o/ | 07:06 |
| dtantsur | cardoe: I agree, it's a mess. I hope we polish all this eventually with the quirks work. | 09:00 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2025.2: Force Virtual Media Slot 1 on iDRAC10 https://review.opendev.org/c/openstack/ironic/+/962326 | 09:01 |
| dtantsur | unfortunately, iDRAC 10 support is rapidly becoming an emergency.. | 09:01 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/2025.2: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962327 | 09:14 |
| dtantsur | rpittau: don't forget bugfix branches please ^^ | 09:16 |
| rpittau | yep | 09:16 |
| rpittau | I should follow the list that I wrote :D | 09:16 |
| dtantsur | also.. since when do we have a Hitachi CI? Oo | 09:16 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/2025.1: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962328 | 09:18 |
| rpittau | I did't even notice | 09:18 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/31.0: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962329 | 09:18 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/30.0: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962330 | 09:20 |
| rpittau | mmm cherry-pick failed | 09:20 |
| rpittau | if we want 2024 stable I need to cherry pick manually it seems | 09:21 |
| dtantsur | rpittau: entirely unrelated, just occured to me: we need to EOL a bunch of bugfix branches | 09:22 |
| dtantsur | it's probably a good idea to backport to 2024.2 if you have cycles | 09:22 |
| rpittau | dtantsur: yep, I usually run a script to remove the oldest bugfix branches, but I think I skipped a couple ones, I'll do it now | 09:23 |
| rpittau | I should have time to backport that to 2024.2 as well | 09:24 |
| dtantsur | Hmm, 2025.2 is broken in an interesting way: https://tarballs.openstack.org/ironic-python-agent/dib/files/ipa-centos9-stable-2025.2.kernel is expected but does not exist | 09:26 |
| dtantsur | I wonder if the build jobs are fubar on IPA 2025.2 | 09:26 |
| rpittau | great! | 09:27 |
| dtantsur | hmm, it has never run? https://zuul.opendev.org/t/openstack/builds?job_name=ironic-python-agent-build-image-dib-centos9 | 09:28 |
| dtantsur | how is it even possible? | 09:29 |
| dtantsur | I'm asking #opendev for hints on how to track it | 09:34 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/2025.2: Handle idrac10 moving some boot items https://review.opendev.org/c/openstack/sushy/+/962331 | 09:36 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/2024.2: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962332 | 09:39 |
| dtantsur | We'll re-run the job manually while investigating | 09:59 |
| opendevreview | Merged openstack/ironic bugfix/30.0: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962330 | 10:04 |
| opendevreview | Merged openstack/ironic bugfix/31.0: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962329 | 10:06 |
| dtantsur | Okay folks, thanks to tonyb, we got 2025.2 IPA images published, so Ironic patches can be rechecked now. | 10:42 |
| dtantsur | Here is a lesson: if IPA-b is branched after IPA, we need to make sure all artifacts are built by the time of Ironic branching. | 10:42 |
| rpittau | goiogn to add that to the release how to page! | 10:43 |
| dtantsur | good point | 10:44 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add ipa-builder branching warning https://review.opendev.org/c/openstack/ironic/+/962335 | 10:48 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add ipa-builder branching warning https://review.opendev.org/c/openstack/ironic/+/962335 | 10:48 |
| rpittau | mm not sure that's the right place on the page | 10:49 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add support for multiple JSON-RPC configuration groups https://review.opendev.org/c/openstack/ironic/+/960593 | 11:56 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add configurable inspection network restoration on node deletion https://review.opendev.org/c/openstack/ironic/+/960823 | 11:56 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add force_inspection_dhcp option for managed inspection https://review.opendev.org/c/openstack/ironic/+/960824 | 11:56 |
| iurygregory | good morning ironic, happy friday \o/ | 12:04 |
| iurygregory | tooo many backports <OMG> | 12:04 |
| kubajj | iurygregory: thanks for the +2 for https://review.opendev.org/c/openstack/ironic-python-agent/+/937342, I just realised that I should probably add a release note, right? | 12:10 |
| iurygregory | kubajj, ++ to a releasenote, I totally missed! | 12:11 |
| kubajj | iurygregory: 🫡 | 12:11 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: Fix skip block devices for RAID arrays https://review.opendev.org/c/openstack/ironic-python-agent/+/937342 | 12:19 |
| kubajj | I tried to highlight both the fixes and the wee change of behaviour which could be taken as a deprecation | 12:20 |
| TheJulia | good morning | 13:00 |
| dtantsur | morning TheJulia | 13:02 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add support for multiple JSON-RPC configuration groups https://review.opendev.org/c/openstack/ironic/+/960593 | 13:45 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add configurable inspection network restoration on node deletion https://review.opendev.org/c/openstack/ironic/+/960823 | 13:45 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add force_inspection_dhcp option for managed inspection https://review.opendev.org/c/openstack/ironic/+/960824 | 13:45 |
| opendevreview | Merged openstack/ironic stable/2025.2: CI: replace the outdated Bifrost job https://review.opendev.org/c/openstack/ironic/+/961389 | 13:55 |
| opendevreview | Merged openstack/ironic stable/2025.2: Fix OCI artifacts pointing to a single manifest https://review.opendev.org/c/openstack/ironic/+/962172 | 14:21 |
| opendevreview | Verification of a change to openstack/ironic stable/2025.2 failed: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962327 | 15:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: Detail availability zone modeling in ironic https://review.opendev.org/c/openstack/ironic/+/962206 | 16:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: remove backwards looking notes https://review.opendev.org/c/openstack/ironic/+/962207 | 17:00 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: Add some meta tags to key docs https://review.opendev.org/c/openstack/ironic/+/962208 | 17:00 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: some more outdated reference fixes https://review.opendev.org/c/openstack/ironic/+/962388 | 18:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: remove severely outdated mutlitenancy docs https://review.opendev.org/c/openstack/ironic/+/962389 | 18:15 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: remove metalsmith and molteniron references https://review.opendev.org/c/openstack/ironic/+/962390 | 18:15 |
| opendevreview | Merged openstack/ironic stable/2025.2: Do not run metal3 integration job in stable branch https://review.opendev.org/c/openstack/ironic/+/962276 | 18:36 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: more legacy references https://review.opendev.org/c/openstack/ironic/+/962391 | 18:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: More legacy version references removals https://review.opendev.org/c/openstack/ironic/+/962392 | 18:39 |
| TheJulia | I swear I didn't mean to start doing this, but the AI Agent kept taunting me to make the docs better | 18:39 |
| JayF | happy to review it | 18:50 |
| JayF | if nobody else gets around to it I will single core merge soon() | 18:50 |
| JayF | because they are pretty blatantly obviously better | 18:50 |
| TheJulia | (which is permitted as almost all of it is purely docs) | 18:52 |
| TheJulia | uhh, I did go ahead and put my current AI agent results into an etherpad as well so we can discuss during the PTG, it has spotted a few other things, but its a little less of clean/direct/quick fixes. | 18:52 |
| JayF | I appreciate you taking the path of fixing the obvious stuff | 18:53 |
| JayF | every attempt I've made has been ALL OF THE THINGS which leads to more wheel spinning than useful updates | 18:53 |
| TheJulia | Yeah, I started having claude code do it | 18:57 |
| TheJulia | then shifted gears as it was much more tactical | 18:57 |
| TheJulia | FYI, I put an item on neutron's etherpad suggesting a cross-project session | 19:03 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for OVS as a virtual switch https://review.opendev.org/c/openstack/bifrost/+/962038 | 19:08 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: WIP: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 19:08 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove support for Python 3.9 https://review.opendev.org/c/openstack/ironic-python-agent/+/962156 | 19:15 |
| opendevreview | Merged openstack/ironic stable/2025.2: Force Virtual Media Slot 1 on iDRAC10 https://review.opendev.org/c/openstack/ironic/+/962326 | 19:19 |
| opendevreview | Merged openstack/python-ironicclient master: Fix parallel initial version negotiation https://review.opendev.org/c/openstack/python-ironicclient/+/956542 | 19:21 |
| opendevreview | Merged openstack/ironic master: Fix incorrect arg while getting ports by portgroup https://review.opendev.org/c/openstack/ironic/+/962043 | 19:34 |
| opendevreview | Merged openstack/ironic master: Add ipa-builder branching warning https://review.opendev.org/c/openstack/ironic/+/962335 | 19:34 |
| TheJulia | We likely need to begin closing the loops on cross project engagements in the next week. | 19:40 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Remove testing for pending-EOL items https://review.opendev.org/c/openstack/ironic-python-agent/+/962397 | 19:43 |
| opendevreview | Merged openstack/ironic master: docs: Detail availability zone modeling in ironic https://review.opendev.org/c/openstack/ironic/+/962206 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: remove backwards looking notes https://review.opendev.org/c/openstack/ironic/+/962207 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: Add some meta tags to key docs https://review.opendev.org/c/openstack/ironic/+/962208 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: some more outdated reference fixes https://review.opendev.org/c/openstack/ironic/+/962388 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: remove severely outdated mutlitenancy docs https://review.opendev.org/c/openstack/ironic/+/962389 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: remove metalsmith and molteniron references https://review.opendev.org/c/openstack/ironic/+/962390 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: more legacy references https://review.opendev.org/c/openstack/ironic/+/962391 | 20:01 |
| opendevreview | Merged openstack/ironic master: docs: More legacy version references removals https://review.opendev.org/c/openstack/ironic/+/962392 | 20:02 |
| iurygregory | TheJulia, I've sent the email for the cross-project with neutron, working on the one for nova now | 20:08 |
| JayF | we have more-than-usual patches without ironic-week-prio, I'm adding them where I can, but also if you wanna see them with me, https://tinyurl.com/all-ironic-stuff is the query I use to find lonely patches | 20:21 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix skip block devices for RAID arrays https://review.opendev.org/c/openstack/ironic-python-agent/+/937342 | 20:24 |
| opendevreview | Merged openstack/networking-generic-switch master: Add security group support to netmiko_dell_os10 https://review.opendev.org/c/openstack/networking-generic-switch/+/958499 | 20:44 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Detect device configuration errors on Dell Force10 and OS10 https://review.opendev.org/c/openstack/networking-generic-switch/+/868318 | 20:44 |
| TheJulia | JayF: makes sense, I flagged a couple earlier, myself | 20:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!