| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Lenovo SR645 V3 boot loop after RHCOS install https://review.opendev.org/c/openstack/ironic/+/986597 | 04:12 |
|---|---|---|
| *** ykarel_ is now known as ykarel | 04:52 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: parsers: use safe dict access in redfish _build_labels https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/988216 | 08:55 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: parsers: validate health field in redfish power/fan/drive metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/988217 | 08:59 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Fix broken LOG.debug format strings in is_allowed() https://review.opendev.org/c/openstack/networking-generic-switch/+/988219 | 09:13 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Modernize GenericSwitchDevice to use abc.ABC https://review.opendev.org/c/openstack/networking-generic-switch/+/988220 | 09:15 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Fix typos in comment and copyright https://review.opendev.org/c/openstack/networking-generic-switch/+/988221 | 09:18 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Align cover tox env with stestr-based test runner https://review.opendev.org/c/openstack/networking-generic-switch/+/988222 | 09:23 |
| opendevreview | Merged openstack/python-ironicclient master: simplify is not None check https://review.opendev.org/c/openstack/python-ironicclient/+/988179 | 09:44 |
| opendevreview | Merged openstack/ironic master: Fix anaconda deploy on multitenant networks https://review.opendev.org/c/openstack/ironic/+/988125 | 11:00 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: GenericSwitchNetmikoConfigError error details https://review.opendev.org/c/openstack/networking-generic-switch/+/985029 | 12:09 |
| opendevreview | Merged openstack/python-ironicclient master: fix codespell config https://review.opendev.org/c/openstack/python-ironicclient/+/988180 | 13:11 |
| opendevreview | Merged openstack/python-ironicclient master: remove unused script which does not pep8 https://review.opendev.org/c/openstack/python-ironicclient/+/988181 | 13:11 |
| opendevreview | Merged openstack/python-ironicclient master: switch to using pre-commit for lint checks https://review.opendev.org/c/openstack/python-ironicclient/+/988182 | 13:13 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Add TLS configuration to agent lookup response https://review.opendev.org/c/openstack/ironic/+/987887 | 13:39 |
| cardoe | last part of the pre-commit migration of python-ironicclient https://review.opendev.org/c/openstack/python-ironicclient/+/988183 | 13:53 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Add TLS configuration to agent lookup response https://review.opendev.org/c/openstack/ironic/+/987887 | 13:59 |
| dtantsur | Our RBAC is driving me nuts.. shouldn't a project reader be allowed to read nodes? | 14:44 |
| * dtantsur thinks more | 14:47 | |
| TheJulia | I think I need more data to answer that comment much less assist | 14:47 |
| dtantsur | I think I keep getting confused by the fact that you need to switch projects explicitly | 14:48 |
| TheJulia | yeah, that doesn't help | 14:50 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Lenovo SR645 V3 boot loop with shim bootloader https://review.opendev.org/c/openstack/ironic/+/986597 | 15:14 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix Redfish BIOS settings on systems without BootProgress https://review.opendev.org/c/openstack/ironic/+/988275 | 15:41 |
| dtantsur | cardoe, janders FYI ^^ | 15:41 |
| dtantsur | We really need to CI this (I'm working on a Metal3 test, this is why how I caught this) | 15:42 |
| * dtantsur adds a release note | 15:42 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix Redfish BIOS settings on systems without BootProgress https://review.opendev.org/c/openstack/ironic/+/988275 | 15:43 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/34.0: Configure CI for bugfix/34.0 https://review.opendev.org/c/openstack/ironic/+/988293 | 16:44 |
| dtantsur | Let's see if I still remember how to do it ^^ | 16:44 |
| dtantsur | In fact, will anyone cry if I remove all non-voting jobs from bugfix branches? We rarely look after them on master, I have little hope about bugfix | 16:47 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/34.0: Configure CI for bugfix/34.0 https://review.opendev.org/c/openstack/ironic/+/988293 | 16:48 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/33.0: Configure CI for bugfix/33.0 https://review.opendev.org/c/openstack/ironic/+/988295 | 16:49 |
| TheJulia | dtantsur: no, go right ahead | 16:51 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: fix discrepancy: all votings jobs must be in the gate https://review.opendev.org/c/openstack/ironic/+/988296 | 16:52 |
| dtantsur | also, not sure how ^^ happened but I've just noticed it, so fixing | 16:52 |
| dtantsur | and finally, could one more person spare a +2 for the nasty bug? https://review.opendev.org/c/openstack/ironic/+/988275 | 16:54 |
| dtantsur | thx! | 16:57 |
| cardoe | dtantsur: The RBAC stuff is why I'm trying to write up that persona doc. So we can agree with how it should model. | 17:01 |
| dtantsur | ++ | 17:03 |
| cardoe | Jay gave me some good feedback so I need to get back to it | 17:04 |
| dtantsur | TheJulia: recovering bugfix jobs will be a proper nightmare this time around, I'll have to continue tomorrow.. | 17:09 |
| dtantsur | (they're hit by the aki-vs-raw issue, and by pyasn-pysnmp incompatibility, and potentially by who knows what else) | 17:09 |
| TheJulia | ugggh | 17:10 |
| TheJulia | joy | 17:10 |
| JayF | just backport snmp driver removal? It wasn't working anyway... | 17:10 |
| TheJulia | well, they are pointing requirements at master, they need to point to back in time | 17:10 |
| dtantsur | JayF: it's also dragged into by irmc, and I'm not comfortable removing that | 17:10 |
| dtantsur | TheJulia: yeah, and now I need to figure out how to override-checkout for jobs that we don't define. and maybe override glance too | 17:11 |
| JayF | I'm fairly sure none of the stuff that needs pysnmp/pyasn has worked for months, to be clear | 17:11 |
| dtantsur | it's optional in irmc, so there is a path where it works (and you'll break anyone who has enabled_hardware_types=irmc even if they don't use the driver in reality) | 17:12 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/34.0: Configure CI for bugfix/34.0 https://review.opendev.org/c/openstack/ironic/+/988293 | 17:12 |
| dtantsur | anyway, will continue tomorrow | 17:12 |
| dtantsur | unless anyone takes over while I'm asleep | 17:12 |
| JayF | fair | 17:13 |
| JayF | cardoe gets a gold star for using the IPv4 documentation prefix for IPs in 986233 | 17:15 |
| cardoe | uh oh did I do a bad? | 17:16 |
| cardoe | I learned about 192.0.2.0/24 recently and have been trying to use it. | 17:18 |
| JayF | a gold star is good | 17:18 |
| JayF | I rarely see people use it | 17:18 |
| JayF | and it makes me feel warm and fuzzy | 17:18 |
| JayF | only thing better would've been real, identifyable rackspace IPs :D | 17:18 |
| cardoe | JayF: did ya use a Rivian charger or other chargers? | 17:25 |
| JayF | https://www.plugshare.com/location/481217 (the most recent picture is my truck) | 17:28 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch stable/2026.1: Fix typos in comment and copyright https://review.opendev.org/c/openstack/networking-generic-switch/+/988302 | 17:29 |
| cardoe | ah nice. I never pay attention to plugshare. I should. | 17:30 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch stable/2026.1: Fix broken LOG.debug format strings in is_allowed() https://review.opendev.org/c/openstack/networking-generic-switch/+/988303 | 17:30 |
| JayF | plugshare is great | 17:30 |
| TheJulia | I do love plugshare | 17:34 |
| cardoe | https://review.opendev.org/c/openstack/releases/+/988323 would that be okay? | 18:20 |
| cardoe | Or do I need that against gazpacho? | 18:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988325 | 18:24 |
| TheJulia | cardoe: is the commit hash against master branch | 18:29 |
| cardoe | yes | 18:29 |
| TheJulia | then hibiscus | 18:29 |
| opendevreview | Julia Kreger proposed openstack/ironic master: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988325 | 18:45 |
| opendevreview | Merged openstack/networking-generic-switch master: Fix typos in comment and copyright https://review.opendev.org/c/openstack/networking-generic-switch/+/988221 | 18:57 |
| opendevreview | Merged openstack/networking-generic-switch master: Fix broken LOG.debug format strings in is_allowed() https://review.opendev.org/c/openstack/networking-generic-switch/+/988219 | 18:57 |
| opendevreview | Merged openstack/networking-generic-switch master: Modernize GenericSwitchDevice to use abc.ABC https://review.opendev.org/c/openstack/networking-generic-switch/+/988220 | 18:57 |
| opendevreview | Merged openstack/networking-generic-switch stable/2026.1: Fix broken LOG.debug format strings in is_allowed() https://review.opendev.org/c/openstack/networking-generic-switch/+/988303 | 19:00 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix Redfish BIOS settings on systems without BootProgress https://review.opendev.org/c/openstack/ironic/+/988275 | 19:26 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Deferred Actions https://review.opendev.org/c/openstack/ironic-specs/+/954612 | 19:47 |
| TheJulia | cardoe, if you have a minute, https://review.opendev.org/c/openstack/ironic/+/988325 would be appreciated. | 19:58 |
| cardoe | I was actually looking at it and squirreled on your 3 slashes comment | 20:00 |
| JayF | I remember all that | 20:02 |
| JayF | it's just a cheap way to dedupe '/' | 20:02 |
| JayF | similar to how https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/types.py#L43 does, it's just a stdlib implementation of some string manipulation | 20:03 |
| JayF | (I think, at least?) | 20:03 |
| TheJulia | eh, it was more urlparse doesn't properly split when you only have two slashes | 20:17 |
| TheJulia | because then it assumes the first folder name is a path | 20:17 |
| TheJulia | so if you do file://folder/foo, it becomes /foo | 20:17 |
| JayF | oooh | 20:18 |
| JayF | you mean if there's bad input | 20:18 |
| JayF | file://blah v file:///blah | 20:18 |
| JayF | I'm surprised that doesn't just kaboom | 20:18 |
| TheJulia | it doesn't really know to, I figured I'd address that outside of the issue itself | 20:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix Redfish BIOS settings on systems without BootProgress https://review.opendev.org/c/openstack/ironic/+/988275 | 20:36 |
| cardoe | Can we drop Python 3.10 and raise our requires to 3.11? | 20:39 |
| cardoe | We no longer test with 3.10. We only test with 3.11 and 2026.2 will have that as the minimum. | 20:41 |
| * TheJulia double checks her local python | 20:50 | |
| TheJulia | 3.11 is not happy with hacking right now | 20:50 |
| TheJulia | so.... | 20:51 |
| * TheJulia glares at her recently dist-updated debian machine and wonders "why you want py3.11!?!" | 20:53 | |
| TheJulia | Jay wins the well written thread response of the day award. | 20:57 |
| opendevreview | Merged openstack/python-ironicclient master: remove separate codespell job https://review.opendev.org/c/openstack/python-ironicclient/+/988183 | 21:15 |
| opendevreview | Merged openstack/networking-generic-switch stable/2026.1: Fix typos in comment and copyright https://review.opendev.org/c/openstack/networking-generic-switch/+/988302 | 21:19 |
| TheJulia | JayF: cid: a review tomorrow would be greatly appreciated on https://review.opendev.org/c/openstack/ironic-specs/+/954612... so I can start my ai prompt on it during some spare time | 21:57 |
| opendevreview | Merged openstack/ironic master: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988325 | 22:02 |
| cid | TheJulia, sure. It's on my list of first todo items for tomorrow alongside sending virtualpdu retirement announcement to the ML. | 22:03 |
| TheJulia | cool cool | 22:06 |
| TheJulia | go to sleeeep :) | 22:06 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2026.1: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988355 | 22:11 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.2: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988356 | 22:12 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.1: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988357 | 22:12 |
| TheJulia | NOOOO merge conflicts | 22:13 |
| TheJulia | le sigh | 22:13 |
| TheJulia | cardoe, fyi, your baremetal networking email went to my spam folder, you might want to send a truncated reply dropping the links/attachments just to see if there is visibility in a few days | 22:44 |
| opendevreview | Julia Kreger proposed openstack/ironic unmaintained/2024.1: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988359 | 22:48 |
| opendevreview | Merged openstack/ironic master: Fix Redfish BIOS settings on systems without BootProgress https://review.opendev.org/c/openstack/ironic/+/988275 | 22:52 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Fix Redfish BIOS settings on systems without BootProgress https://review.opendev.org/c/openstack/ironic/+/988363 | 23:04 |
| cardoe | So before I disappear to cook dinner as requested... | 23:06 |
| cardoe | https://opendev.org/openstack/openstack-helm-images/src/commit/977951e9969442e2945d0589beb6fd0a22d077d1/ironic/Dockerfile working on iterating that down to a small generic container build | 23:07 |
| cardoe | Not sure if testing via containers like metal3 would be something we'd entertain. | 23:07 |
| TheJulia | it likely needs to be a high bandwidth discussion, fwiw | 23:25 |
| opendevreview | Julia Kreger proposed openstack/python-ironicclient master: trivial: fix author in latex doc output https://review.opendev.org/c/openstack/python-ironicclient/+/988367 | 23:56 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!