| opendevreview | Adam McArthur proposed openstack/ironic master: api: Add schema for conductors API https://review.opendev.org/c/openstack/ironic/+/995763 | 00:55 |
|---|---|---|
| opendevreview | Adam McArthur proposed openstack/ironic master: api: Add schema for volume API https://review.opendev.org/c/openstack/ironic/+/995764 | 00:58 |
| opendevreview | Adam McArthur proposed openstack/ironic-specs master: Add node workflows https://review.opendev.org/c/openstack/ironic-specs/+/952533 | 02:39 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Add pyright type checking https://review.opendev.org/c/openstack/ironic-python-agent/+/958333 | 03:25 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Add pyright type checking https://review.opendev.org/c/openstack/ironic-python-agent/+/958333 | 03:52 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter stable/2025.1: [stable only] Replace pkg_resources with importlib.resources https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/994768 | 06:56 |
| opendevreview | Merged openstack/ironic master: Fix release codename in webapi version history https://review.opendev.org/c/openstack/ironic/+/995419 | 07:32 |
| elodilles | JayF: in general i'm helping out with CI maintenance on all open unmaintained branches. if the gates are not unrepairably broken, then i'll fix them, if it is reasonable to fix (with relevant CI coverage). so in short, i say let's try to leave it open, and if we reach the point when it is not feasible to keep them, then of course those branches needs to be EOL'd. | 08:02 |
| opendevreview | Merged openstack/bifrost master: Drop support for ironic-staging-drivers https://review.opendev.org/c/openstack/bifrost/+/986131 | 09:52 |
| opendevreview | Jed Preist proposed openstack/tenks master: Add Redfish support https://review.opendev.org/c/openstack/tenks/+/995697 | 10:04 |
| skrobul | cid / JayF :hi there! Quick question about https://specs.openstack.org/openstack/ironic-specs/specs/approved/kea-dhcp-backend.html - did you manage to start any implementation on the Ironic's DHCP provider? If so, is the code available anywhere (even if it's not ready)? | 11:00 |
| cid | skrobul. We did. It's mostly unfinished. | 11:29 |
| cid | https://review.opendev.org/c/openstack/ironic/+/935645 | 11:29 |
| skrobul | cid: thank you! that's still very helpful | 11:35 |
| cid | ++ | 11:39 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: feat: add state, target_provision_state, and duration_seconds to node history https://review.opendev.org/c/openstack/ironic/+/989994 | 11:42 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: feat: populate node_history state fields https://review.opendev.org/c/openstack/ironic/+/993208 | 11:42 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add request context user tracking to node history records https://review.opendev.org/c/openstack/ironic/+/993901 | 11:42 |
| opendevreview | Elod Illes proposed openstack/ironic-python-agent unmaintained/zed: Fix CI issues https://review.opendev.org/c/openstack/ironic-python-agent/+/995201 | 13:00 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: feat: add state, target_provision_state, and duration_seconds to node history https://review.opendev.org/c/openstack/ironic/+/989994 | 14:40 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: feat: populate node_history state fields https://review.opendev.org/c/openstack/ironic/+/993208 | 14:40 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add request context user tracking to node history records https://review.opendev.org/c/openstack/ironic/+/993901 | 14:40 |
| clif | JayF: I am loathe to admit it but I think claude was actually right, while *some* options are validated at config loading time (namely cli opts), the rest are not type validated until access time (this is a HUGE wtf to me) | 15:31 |
| clif | I have been nerd sniped into caring about oslo.config on my day off | 15:31 |
| clif | I cannot fathom *why* you would want to wait until you try to use or access a configuration option to validate it | 15:32 |
| clif | I want my program to blow up before it starts if the configuration is wrong at such a basic level that a configuration option fails type checking | 15:33 |
| clif | so claude was subtly wrong while being mostly right | 15:34 |
| clif | the fact that type coercion/validation doesn't happen until access time is making me irrationally angry | 15:37 |
| clif | oslo-config-validator exists, but why wouldn't you want something like that to run when your program ingests the config at startup? | 15:48 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: Add ironic-manage command for driver discoverability https://review.opendev.org/c/openstack/ironic/+/995847 | 16:15 |
| JayF | clif: always remember the answer can be "someone wrote it this way in a hurry 15 years ago and changing it across dozens of applications may not be worth it" | 16:16 |
| JayF | clif: sounds like we have a path forward though: explicitly access any of our config items that require validation after loading of configs | 16:17 |
| cardoe | clif: speaking of config options... ^ that's for discoverability... and I'm looking to add your TBN evaluator into that command if folks are good with it | 16:21 |
| JayF | sgtm | 16:24 |
| JayF | clif: I thought you were off today? I didn't know you were susceptible to this level of nerd sniping :D | 16:24 |
| opendevreview | cid proposed openstack/ironic master: WIP: Console container provider configuration https://review.opendev.org/c/openstack/ironic/+/995852 | 16:30 |
| opendevreview | cid proposed openstack/ironic master: WIP: Console container provider https://review.opendev.org/c/openstack/ironic/+/995853 | 16:30 |
| opendevreview | cid proposed openstack/ironic master: WIP: Console container provider plugin support https://review.opendev.org/c/openstack/ironic/+/995854 | 16:30 |
| opendevreview | cid proposed openstack/ironic master: WIP: Console container provider documentation https://review.opendev.org/c/openstack/ironic/+/995855 | 16:30 |
| opendevreview | cid proposed openstack/ironic master: WIP: Console container provider tests https://review.opendev.org/c/openstack/ironic/+/995856 | 16:30 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix manual-management hardware type to use noop per the docs https://review.opendev.org/c/openstack/ironic/+/995857 | 16:51 |
| cardoe | cid: ^ I believe that's what you had intended with your original change so I've marked that as backport | 17:05 |
| cid | Indeed. Thanks! | 17:53 |
| opendevreview | Merged openstack/ironic master: Default TLS minimum version to 1.3 https://review.opendev.org/c/openstack/ironic/+/987953 | 18:19 |
| clif | I'm going to hurl a chair through a window, does oslo.config even validate configuration type coercion when using `oslo-config-generator`? because so far in my limited testing it does not | 19:20 |
| clif | I have a string specified for an option that is explicitly an IntOpt and the validator doesn't bat an eye about it | 19:20 |
| JayF | The folks who work on that are in #openstack-oslo, but it's like, 10% time of two people. | 19:21 |
| JayF | If you can make it better, please do. | 19:21 |
| clif | it looks like validate makes sure the config parses and the options in the config file map to options defined in the config namespace specified | 19:23 |
| clif | and that's about it | 19:24 |
| clif | that's a problem for future me | 19:24 |
| JayF | Sounds good :) there's a lot of stuff like this hidden in the depths; a long time for tech debt to accumulate and sometimes you don't see the dusty corner until you go looking | 19:33 |
| opendevreview | Merged openstack/ironic master: fix manual-management hardware type to use noop per the docs https://review.opendev.org/c/openstack/ironic/+/995857 | 19:41 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: fix manual-management hardware type to use noop per the docs https://review.opendev.org/c/openstack/ironic/+/995900 | 20:56 |
| cardoe | clif: I believe it does not validate type coercion. It was a limitation of argparse and I had a mental note to revisit it. | 20:58 |
| opendevreview | Doug Goldstein proposed openstack/ironic bugfix/37.0: fix manual-management hardware type to use noop per the docs https://review.opendev.org/c/openstack/ironic/+/995902 | 20:59 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: add types to hardware types for supported interfaces https://review.opendev.org/c/openstack/ironic/+/995904 | 21:06 |
| cardoe | If I was a hardware type that was effectively two paired devices with two different BMCs would you make one a parent? No relationship at all? Or make some fake parent device and both real things as children? | 22:20 |
| opendevreview | Merged openstack/ironic-python-agent unmaintained/zed: Fix CI issues https://review.opendev.org/c/openstack/ironic-python-agent/+/995201 | 22:21 |
| JayF | cardoe: depending on how my workflows worked, I might make that *one* custom hardware type with an expectation that the less-common bmc get a --management-interface= when required | 23:47 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!