| TheJulia | I mean, I was hoping | 00:03 |
|---|---|---|
| TheJulia | I talked to them years ago and they pushed back it was standardized and I pulled the spec and they went radio silent | 00:03 |
| shermanm | in either case, making sure my notes exist in a bug for reference: https://bugs.launchpad.net/ironic/+bug/2156955 | 01:28 |
| *** elodilles_pto is now known as elodilles | 07:50 | |
| opendevreview | Jacob Anders proposed openstack/ironic-specs master: Add spec for multi-component batched firmware updates https://review.opendev.org/c/openstack/ironic-specs/+/992208 | 10:12 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Clarify the ordering in [ipmi]cipher_suite_versions https://review.opendev.org/c/openstack/ironic/+/993731 | 11:03 |
| TheJulia | Good morning | 13:22 |
| zigo | Hi ! | 13:24 |
| TheJulia | Clearly, zigo is too awake ;) | 13:25 |
| * TheJulia sips coffee | 13:25 | |
| zigo | JayF: I did a recheck of https://review.opendev.org/c/openstack/ironic/+/993685 since the release notes job definition was broken yesterday, but the error got reverted. | 13:25 |
| zigo | Maybe other backports CI runs are broken too because of this. | 13:25 |
| cardoe | speaking of release notes. I saw that nova added minimum version or something to each of their release mappings to avoid fixes to release notes getting caught up as new. | 13:47 |
| cardoe | JayF: ^ might be something we want too? | 13:47 |
| JayF | I don't know why you point me at it :) | 14:04 |
| JayF | if you think it's a good idea JFDI or email the list | 14:04 |
| JayF | just make sure you add a release note to the change ;) | 14:04 |
| cardoe | Cause you previously had concerns about changes around those and explained to me how it all worked forever ago. So to me you're the release note SME. :D | 14:09 |
| JayF | I don't have time to be concerned anymore /s | 14:10 |
| TheJulia | VMT plate overflowith | 14:13 |
| TheJulia | cardoe: if you can link me to an example of what they did, we might have done similar at some point | 14:36 |
| TheJulia | its also just a super easy thing to forget to do and all | 14:37 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Modernize the Metal3 functional jobs https://review.opendev.org/c/openstack/ironic/+/992607 | 14:45 |
| opendevreview | Pierre Crégut proposed openstack/ironic master: Do not check pending setting for bios update https://review.opendev.org/c/openstack/ironic/+/993770 | 15:02 |
| JayF | Did we spec out what LACP support in ironic-networking will look like and not implement it? Or does it need design work still, too., | 15:34 |
| dtantsur | alegacy: ^^ | 15:34 |
| TheJulia | I feel like there was some initial work there, but not a direct focus. | 15:43 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2024.1: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/993025 | 16:02 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2024.1: ci: Pin setuptools to a range that still ships https://review.opendev.org/c/openstack/ironic-python-agent/+/993787 | 16:02 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2023.1: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/993026 | 16:05 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2023.1: ci: Pin setuptools to a range that still ships https://review.opendev.org/c/openstack/ironic-python-agent/+/993789 | 16:05 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2024.1: ci: Pin setuptools to a range that still ships https://review.opendev.org/c/openstack/ironic-python-agent/+/993787 | 16:51 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2024.1: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/993025 | 16:51 |
| alegacy | JayF: the spec touched on some aspects of it. Probably needs some refinement as we get to the implementation. | 16:53 |
| JayF | ack | 16:53 |
| JayF | I'll take a gander when I find time | 16:53 |
| * JayF pencils it in for 2034 | 16:53 | |
| * TheJulia schedules staring in 2053 | 16:58 | |
| opendevreview | Harald Jensås proposed openstack/ironic master: Fix bootfile_name_by_arch to consider boot mode https://review.opendev.org/c/openstack/ironic/+/993813 | 16:59 |
| opendevreview | Pierre Crégut proposed openstack/ironic master: Do not check pending setting for bios update https://review.opendev.org/c/openstack/ironic/+/993770 | 18:24 |
| opendevreview | Merged openstack/ironic stable/2025.2: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/993685 | 18:29 |
| JayF | That patch, 993770, looks like a regression fix | 18:46 |
| opendevreview | Merged openstack/ironic stable/2025.1: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/993684 | 19:05 |
| opendevreview | Merged openstack/ironic bugfix/34.0: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/993683 | 19:05 |
| opendevreview | Merged openstack/ironic bugfix/33.0: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/993682 | 19:05 |
| opendevreview | Merged openstack/ironic unmaintained/2024.1: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/993686 | 19:05 |
| opendevreview | Merged openstack/ironic unmaintained/2023.1: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/993687 | 19:05 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent stable/2025.1: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/993024 | 19:49 |
| cardoe | JayF: it is but the proposed fix would then break Dell's | 19:50 |
| JayF | cardoe: Yeah, which means we need to revert it outta being unconditional and turn it into a quirk | 19:51 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent unmaintained/2024.1: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/993025 | 19:51 |
| cardoe | I don't know what hardware they tested it on. | 19:52 |
| JayF | https://bugs.launchpad.net/ironic/+bug/2157002 says HPE and Lenovo | 19:52 |
| cardoe | Yeah I tested it on a HPE box and didn't have that issue. | 19:52 |
| cardoe | So it's not all HPE boxes. | 19:53 |
| JayF | can you put that info inthe bug then so we can try and narrow it down? | 19:53 |
| JayF | I just didn't want that patch to get lost in the shuffle | 19:53 |
| opendevreview | Ghanshyam Maan proposed openstack/ironic master: Remove setting of oslo_policy[enforce_scope] flag https://review.opendev.org/c/openstack/ironic/+/993827 | 19:54 |
| TheJulia | brraaaaaaains | 19:59 |
| cardoe | Reading the HPE docs on how it works, our implementation is correct. | 20:09 |
| cardoe | I mean this brings us back to how my first implementation of the patch was. | 20:23 |
| cardoe | Which set the requested BIOS settings, rebooted the box, once the box started to boot if any of the settings weren't applied we failed the job. | 20:24 |
| cardoe | Cause the ultimate version that got merged is that we keep looking to see if the value is applied and don't finish until it is. | 20:25 |
| JayF | I don't really know a lot about this interface, I only did the pointer to make sure we didn't lose hold of the fact we regressed. I assume for every person who tells us about a bug that >1 have experienced it | 20:25 |
| JayF | I don't have a strong opinion as to how it's solved | 20:25 |
| cardoe | Well the issue I've got with the current implementation is that invalid values can get you stuck like the bug report appears to imply the user got stuck | 20:26 |
| JayF | hmm | 20:26 |
| cardoe | The box stays stuck in the step and no way to abort it without a DB touch | 20:27 |
| cardoe | But it's no longer trying to apply a BIOS setting change. | 20:27 |
| cardoe | But you also cannot attempt to apply any more BIOS settings until you clear the pending changes. | 20:28 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent-builder unmaintained/2023.1: update .gitreview to point to unmaintained/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/993418 | 21:40 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent-builder unmaintained/2023.1: Pin setuptools in the requirements.txt copied from IPA https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/993420 | 21:40 |
| opendevreview | Clif Houck proposed openstack/ironic-python-agent-builder unmaintained/2023.1: Disable ipa-check-image-dib-centos9{-extra} in zuul.d https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/993850 | 21:40 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!