| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Fix codespell job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/994831 | 01:19 |
|---|---|---|
| opendevreview | Merged openstack/ironic-python-agent-builder master: Fix codespell job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/994831 | 03:48 |
| *** hroy_ is now known as hroy | 03:59 | |
| rpittau | cardoe: would really love an answer for https://review.opendev.org/c/openstack/networking-generic-switch/+/994433 when you got a moment, as I wrote I'm ok removing just neutron (and it probably makes more sense), thanks! :) | 07:57 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter stable/2025.1: [stable] only] Replace pkg_resources with pathlib for resource file loading https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/994768 | 08:34 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Modernize the Metal3 functional jobs https://review.opendev.org/c/openstack/ironic/+/992607 | 11:38 |
| dtantsur | JayF: hey, I recall you didn't feel well about a service step to attach an arbitrary ISO. Do you have the same feelings about a clean step? | 12:53 |
| cardoe | rpittau: oh. I didn’t -1 so it’s good merge it. | 12:58 |
| cardoe | In the future I wanna nudge them to have smaller deps. | 12:58 |
| dtantsur | cardoe: oh hi, while we have you here: you mentioned you had some thoughts on hardware discovery, didn't you? | 13:00 |
| dtantsur | do you have similar requests or something brewing already? | 13:01 |
| rpittau | cardoe: ok, thanks, but thinking about what you wrote it's probably better to remove just neutron for the time being :) | 13:07 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Make neutron an optional dependency https://review.opendev.org/c/openstack/networking-generic-switch/+/994433 | 13:13 |
| cardoe | rpittau: agreed | 13:28 |
| cardoe | dtantsur: got something for me to give feedback on? | 13:28 |
| cardoe | I’m in the woods currently so not caught up on context. | 13:30 |
| dtantsur | No, not yet, but may in a couple of hours | 13:30 |
| dtantsur | it's re your comment on the midcycle etherpad | 13:30 |
| rpittau | CI is finally happy here \o/ https://review.opendev.org/c/openstack/bifrost/+/991128 | 13:46 |
| JayF | dtantsur: My objection was primarily related to booting a completely unmanaged by ironic workload on an active machine. So I think it should be okay as a clean step. | 13:57 |
| dtantsur | Got it. I'll see what these people actually want. | 13:58 |
| cardoe | dtantsur: ah yes. can we chat next week? | 14:22 |
| dtantsur | cardoe: sure! More reasonable weather can help me think :) | 14:23 |
| dtantsur | JayF: so, apparently some vendors refuse to implement firmware updates via Redfish, providing an ISO instead :( | 14:32 |
| JayF | I mean, sounds like a vendor who shouldn't have their hardware purchased. | 14:39 |
| dtantsur | Oh yeah, it's on my personal naughty list already. But I'm not the one making purchases.. | 14:40 |
| JayF | Or that the cloud users need to do some engineering work to extract the actual firmware writes from the ISO and put it into IPA in-band | 14:40 |
| JayF | I don't think we should compromise our security stance because a vendor is being obtuse. | 14:40 |
| JayF | I am getting more and more convinced that being willing to compromise for vendors contributes to the problem | 14:40 |
| dtantsur | We're leaning towards the same downstream, but it's never an easy conversation | 14:41 |
| JayF | dtantsur: There is some in-band standardization for firmware updating in linux -- mainly fwupd | 14:56 |
| JayF | dtantsur: the big downside there is that many vendors' firmware bundles used for fwupd (e.g. almost all the framework ones) are EFI-based | 14:56 |
| JayF | so it's just doing efi boot stuff to make the firmware update happen | 14:56 |
| TheJulia | its actually the best case in-band model since there are guard rails built in | 14:59 |
| TheJulia | FWIW | 14:59 |
| JayF | you mean fwupd or EFI based or both? | 14:59 |
| TheJulia | EFI based model | 15:03 |
| TheJulia | that is functionally what the BMC model is doing in some cases | 15:03 |
| TheJulia | presenting a virtual thumb drive with the file | 15:03 |
| TheJulia | At least for parts and pieces which can't be bit-banged or directly accessed | 15:03 |
| TheJulia | bit-banged is being avoided in favor of proper A/B or slotting models | 15:04 |
| TheJulia | like... one vendor wants to hold up to 8 firmware images for some devices | 15:04 |
| TheJulia | Active, Secondary, Tertiary 1-6 | 15:04 |
| TheJulia | JayF: That is a most excellent email. | 15:05 |
| opendevreview | Merged openstack/ironic master: docs: Augment bootc docs https://review.opendev.org/c/openstack/ironic/+/993421 | 15:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Modernize the Metal3 functional jobs https://review.opendev.org/c/openstack/ironic/+/992607 | 16:18 |
| JayF | https://review.opendev.org/c/openstack/ironic-specs/+/992741/2#message-3def7fa9fcbaa5d33b59c26ebe2a87e62f0ab4a2 please take a look at the container interface proposed here | 17:03 |
| JayF | it's pretty simple spec and almost got a +2 from me on first pass | 17:03 |
| opendevreview | Merged openstack/ironic master: docs: Add a Getting Started map to the landing page https://review.opendev.org/c/openstack/ironic/+/994770 | 17:07 |
| TheJulia | JayF: you -1'ed it... unless its been revised somewhere. | 17:35 |
| TheJulia | and my browser is still showing old records | 17:35 |
| JayF | almost :) | 17:36 |
| JayF | the -1 is for naming | 17:37 |
| JayF | but the implementation itself is solid | 17:37 |
| JayF | The names of the interfaces there describe the method for spinning up the container, not the container engine | 17:37 |
| TheJulia | ... true | 17:41 |
| * TheJulia glares at claude for saying "I ran pep8" when it failed doing so | 17:44 | |
| JayF | my favorite is when it goes to make a venv to run pep8 | 17:48 |
| JayF | instead of using tox like it says to do in the claude.md, personal instructions, and examples earlier in the chat | 17:48 |
| JayF | claude models get dumber over time, I'm completely convinced of it | 17:48 |
| JayF | and the version numbers are fantasies | 17:49 |
| TheJulia | Yeah, and also the "oh, this looks like super cheap model can answer it" logic overriding user selections | 17:52 |
| JayF | I worked this weekend on improving my local email (I now sync to a local IMAP proxy + use imapfilter on office365 for filtering, it's great) | 17:53 |
| JayF | and mid-project, claude compacted the giant conversation we had about it | 17:53 |
| JayF | and it was like self-lobotomization. Suggesting things we knew wouldn't work. Making tradeoffs on my requirements without asking. | 17:53 |
| JayF | I was more pissed off than I've been at an AI in a while, just because it was so useful and became worse-than-useless in the blink of an eye, completely outside of my control | 17:53 |
| TheJulia | Yeah, I've started to keep an eye on the remaining context at times and explicitly clear it. | 17:54 |
| JayF | I almost wish I could "sticky" context? | 17:54 |
| JayF | like in a spreadsheet, you save the top 10 rows or something so you only scroll the rest | 17:55 |
| TheJulia | https://github.com/anthropics/claude-code/issues/49861 :( | 17:55 |
| JayF | When I think about it, I tend to have it "self-compact" at that point by telling it to write me a prompt to start on (implementation, step 2, etc) so I can edit the prompt and make sure it didn't drop naything useful | 17:55 |
| TheJulia | Yeah, I've tried to drive it twoards very stepwise ordering for my sanity | 17:56 |
| JayF | Writing an issue in the claude-code github is about as useful as writing your issue on a post-it note, giving it a little kiss, shredding it, and spreading the shreds on the wind | 17:56 |
| TheJulia | I did that with deferred task stuff pretty heavily and its not awful now thatI'm going through and reviewing the wip changes in fine detail and cleaning them up. | 17:56 |
| * TheJulia gets out the giant post-it notes | 17:57 | |
| * TheJulia also gets out the torch | 17:57 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add deferred actions data model, DB API, and versioned object https://review.opendev.org/c/openstack/ironic/+/989224 | 18:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add config options, reschedule exception, and action creation helper https://review.opendev.org/c/openstack/ironic/+/989225 | 18:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add two-step deferred actions cleanup DB methods https://review.opendev.org/c/openstack/ironic/+/994948 | 18:21 |
| opendevreview | Merged openstack/ironic stable/2025.1: Add operator-configurable step disallow lists https://review.opendev.org/c/openstack/ironic/+/992732 | 18:38 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add deferred action launcher periodic and executor https://review.opendev.org/c/openstack/ironic/+/989226 | 19:13 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add deferred action cleanup periodic and stuck recovery https://review.opendev.org/c/openstack/ironic/+/989227 | 20:13 |
| TheJulia | cid: if you want to carry on rebasing the changes on that series while I'm on vacation, your welcome to. | 20:13 |
| cid | I will poke at it if I get a window. | 20:21 |
| JayF | at this rate he might be writing enhanced maintenance | 20:21 |
| cid | Enjoy the break! | 20:21 |
| JayF | that scheduled maintenance window thing | 20:21 |
| JayF | so obvious but how did I never think of it | 20:21 |
| TheJulia | oh yeah, totally makes so much sesne | 20:39 |
| TheJulia | because I've literally had to shutdown transformers | 20:39 |
| TheJulia | "oh noes! that takes row 3/4 out" | 20:39 |
| TheJulia | I'm basically engaging the cloaking device and heading towards vacation | 20:44 |
| JayF | I will see you (literally!) Tuesday. | 20:47 |
| TheJulia | Woot! | 21:20 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Enable centos10 publish job https://review.opendev.org/c/openstack/ironic-python-agent/+/992571 | 21:26 |
| opendevreview | Steve Baker proposed openstack/ironic master: Switch jobs to build centos10 IPA images https://review.opendev.org/c/openstack/ironic/+/992573 | 21:49 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Switch ironic-standalone-ipa-src to centos10 ipa https://review.opendev.org/c/openstack/ironic-python-agent/+/994982 | 21:52 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!