| opendevreview | Victor proposed openstack/ironic master: Truncate node history event to avoid DB column overflow https://review.opendev.org/c/openstack/ironic/+/981399 | 13:16 |
|---|---|---|
| TheJulia | Good morning | 13:20 |
| cardoe | morning | 13:20 |
| cardoe | TheJulia: if ya get a few cycles today https://review.opendev.org/c/openstack/ironic/+/983215 | 13:21 |
| cardoe | It's a selfish ask cause like I said to deal with the trait performance explosion I plan on stuffing that into 2026.1 for me and if its slotted as the first DB change then it'll make my upgrades easier in the future. | 13:21 |
| cardoe | There's a few DB changes on deck in the review pipeline. | 13:22 |
| TheJulia | Sure | 13:22 |
| cardoe | So selfishly trying to say "ooo me first me first!" | 13:22 |
| TheJulia | Yeah, version tagging and all ;) | 13:22 |
| cardoe | I am fleshing out some of our upcoming development sprints and have got 2026.1 + TBN + L2VNI from networking-baremetal all on deck. | 13:23 |
| cardoe | So hopefully my team will provide some decent tire kicking. | 13:23 |
| TheJulia | I wonder if we will have critical mass for a meeting today | 13:23 |
| cardoe | oh and firmware wrangling. | 13:23 |
| TheJulia | Cool cool | 13:23 |
| cardoe | We don't at the office. It's UK bank holiday season (as I call it) so others have taken a longer holiday. | 13:24 |
| cardoe | I feel like Pablo from Narcos. /me looks despondent at my desk. /me looks despondent on a swing. | 13:24 |
| TheJulia | Oh, yeah, even my mailbox is minimalist and only not messages on slack ;) | 13:25 |
| TheJulia | Wheeeeeee swing?! | 13:25 |
| * TheJulia never claimed to be normal | 13:25 | |
| TheJulia | Sadly we only have golf, pickleball, tennis sorts of things nearby | 13:26 |
| TheJulia | And pools…. | 13:26 |
| TheJulia | No swing sets :( | 13:26 |
| cardoe | I work with enough non-US folks that I regularly call it "holiday" instead of "vacation". Or I'll quote temps in Celsius. | 13:26 |
| cardoe | It's about 10 out... no pool. | 13:27 |
| cardoe | My children tell me to be normal. I just laugh. | 13:28 |
| TheJulia | lol | 13:28 |
| TheJulia | yeah, quoting temps in celsius is normal | 13:28 |
| * clif laughs in hard to reason about units | 13:29 | |
| TheJulia | :) | 13:31 |
| * TheJulia sips coffee and tries to wake up | 13:31 | |
| cardoe | I went off the deep end Friday afternoon and Saturday with uWSGI... https://review.opendev.org/c/openstack/devstack/+/983361 | 13:57 |
| cardoe | And clearly my git commit message configuration is bad cause I keep wrapping messages awkwardly. | 13:57 |
| TheJulia | heh | 14:00 |
| TheJulia | cardoe: did you verify that your migration actually migrates data? | 14:03 |
| * cardoe thinks. | 14:03 | |
| cardoe | I don't know if I started with a fresh DB. | 14:03 |
| cardoe | Lemme do it proper like and confirm. | 14:03 |
| TheJulia | The only thing I wonder is if someone creates a record after schema upgrade but before the api gets upgraded | 14:05 |
| TheJulia | and maybe that is immaterial, typically what you did in the structural migration, we also want to make sure is done in in online data migrations, but that also being said... the actual change is a bit hard to do there because its more a one-shot, I'm not sure you can cleanly revisit | 14:06 |
| TheJulia | ponder me that batman! | 14:06 |
| * TheJulia suspects we're skipping the meeting today | 14:46 | |
| TheJulia | cardoe: so thinking about it, a migration that checks, looks for there just being no more records to duplicate over | 14:46 |
| * TheJulia sees no meeting today :) | 15:06 | |
| opendevreview | Victor proposed openstack/ironic master: Fix API pagination limit for conductor filtering https://review.opendev.org/c/openstack/ironic/+/983449 | 15:09 |
| * TheJulia gives claude all of the attestation work and sees what ferments | 15:21 | |
| JayF | I'm out today btw | 15:23 |
| TheJulia | cool cool, I'm being evil with claude | 15:23 |
| opendevreview | Victor proposed openstack/ironic master: Fix API pagination limit for conductor filtering https://review.opendev.org/c/openstack/ironic/+/983449 | 15:30 |
| opendevreview | Victor proposed openstack/ironic master: Truncate node history event to avoid DB column overflow https://review.opendev.org/c/openstack/ironic/+/981399 | 15:37 |
| victor_obi_ | Hello everyone. I have two patches and would greatly appreciate a review when anyone has time: | 15:46 |
| victor_obi_ | Bug #2097102 (Conductor pagination limit): Pushes the conductor_affinity filter down to the DB API. https://review.opendev.org/c/openstack/ironic/+/983449 | 15:46 |
| victor_obi_ | Bug #2097659 (Node history truncation): Updated to fix the PEP8 formatting failure. https://review.opendev.org/c/openstack/ironic/+/981399 | 15:46 |
| TheJulia | victor_obi_: okay, I'll try to take a little later today | 15:51 |
| victor_obi_ | TheJulia: Thank you | 15:56 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline briefly to upgrade to a newer 3.11 patch release and apply some configuration updates in preparation for an upgrade to 3.12 in the near future | 16:39 | |
| TheJulia | cardoe: you around for a little bit for a quick but distracting question? | 16:46 |
| cardoe | yeah. about to do some testing on that trait stuff | 16:46 |
| TheJulia | I swear it should should be a quick and short distraction | 16:46 |
| TheJulia | for your users, if you were doing attestation with the current flow, what do you think would feel "better" to "acknowledge" a change to permissively allow the machine to move forward as-is on a pcr change. getting link real quick | 16:47 |
| TheJulia | starting state, the node is in "clean fail", measurement has failed, customer did something: https://paste.opendev.org/show/bRATuU418rDTjHE1TWg3/ | 16:48 |
| TheJulia | so the ideal would be to basically acknowledge and move onward. | 16:50 |
| cardoe | Yes I agree with that. | 16:55 |
| cardoe | I had to walk through it and in the middle someone walked into my office to demand petting. | 16:55 |
| TheJulia | any preference seen around specific options to flow | 16:56 |
| TheJulia | oh, I had the same thing happen, the void demanded cuddles | 16:56 |
| TheJulia | I guess 1,3 as claude notes is kind of reasonable | 16:57 |
| TheJulia | as long as the endpoint is just a shortcut to the clean step invocation | 16:57 |
| TheJulia | or being state aware, but really this should only happen in cleaning | 16:58 |
| cardoe | So I think Path A can fit within Path B. | 17:02 |
| cardoe | But that explicit approval feels more correct. | 17:02 |
| cardoe | We can also do firmware upgrades with attestation updates. | 17:02 |
| TheJulia | yup | 17:02 |
| cardoe | Requires that you trust your firmware not to be a baddie. | 17:02 |
| TheJulia | likely, need to explicitly expect to do so | 17:03 |
| TheJulia | yup | 17:03 |
| * TheJulia goes "om nom nom nom tokens" | 17:03 | |
| opendevreview | Anishere Mariam proposed openstack/ironic master: Add node history state timestamps https://review.opendev.org/c/openstack/ironic/+/982555 | 17:27 |
| cardoe | janders, iurygregory: we did get around to kicking the tires on the IPA-less BIOS update on the Dells as merged upstream. It works. It has to attempt to boot something successfully. So failing to boot off the wiped disk isn't good enough for the OS transition check. But attempting to PXE boot is good enough. So I'm gonna call this a win. | 17:31 |
| cardoe | They were able to recover a box that was BIOS reset and set to UEFI boot. | 17:31 |
| cardoe | https://docs.openstack.org/ironic/2024.2/admin/drivers/idrac.html#pxe-reset-with-factory-reset-bios-clean-step is now a case that's recoverable. | 17:32 |
| cardoe | TheJulia: while ya nom tokens... nom https://review.opendev.org/c/openstack/python-ironicclient/+/981200 | 17:35 |
| TheJulia | \o/ | 17:39 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: make target_datetime optional for set_bmc_clock step https://review.opendev.org/c/openstack/ironic/+/983466 | 17:47 |
| cardoe | speaking of attestation... just saw https://review.opendev.org/c/openstack/python-ironicclient/+/755837 | 17:54 |
| TheJulia | oh, missed that one | 17:54 |
| TheJulia | Yeah, I'm working on revising the attestation design overall | 17:54 |
| TheJulia | I think i'm about 10 minutes from uploading a new spec | 17:55 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Firmware caching before OS boot complete https://review.opendev.org/c/openstack/ironic/+/983467 | 17:56 |
| opendevreview | Merged openstack/python-ironicclient master: remove dangling functional test https://review.opendev.org/c/openstack/python-ironicclient/+/981200 | 17:57 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Fix Premature reboot interrupting NIC update https://review.opendev.org/c/openstack/ironic/+/983468 | 17:57 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: add node trait add/remove commands for consistent UX https://review.opendev.org/c/openstack/python-ironicclient/+/981805 | 18:05 |
| clif | TheJulia: JayF I expanded the TBN sections to four slides for the OpenInfra Live presentation. Please take a look when you get a chance and lmk if you have any feedback for me. ty! | 18:06 |
| TheJulia | Thanks! | 18:06 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Attestation Interface - Revised Design (v2) https://review.opendev.org/c/openstack/ironic-specs/+/983469 | 18:13 |
| TheJulia | cardoe: ^ | 18:13 |
| cardoe | clif: send me the presentation? I'm not sure if I need to add slides. | 18:14 |
| cardoe | TheJulia: I'll read it over lunch | 18:14 |
| TheJulia | k | 18:14 |
| TheJulia | cardoe: https://docs.google.com/presentation/d/1SDNGtEYIP45AkjMxAiyBfTjJDO4yQQcjaDMjosm17aM/edit?slide=id.gd58c7df0e6_1_13#slide=id.gd58c7df0e6_1_13 | 18:15 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_conductor.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981688 | 18:16 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_driver.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981689 | 18:17 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_chassis.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981690 | 18:22 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_deploy_template.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981691 | 18:22 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_volume_connector.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981697 | 18:28 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_allocation.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981698 | 18:28 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/osc/v1/baremetal_inspection_rule.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981699 | 18:35 |
| opendevreview | Merged openstack/ironic stable/2025.2: make target_datetime optional for set_bmc_clock step https://review.opendev.org/c/openstack/ironic/+/982870 | 19:06 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Attestation Interface - Revised Design (v2) https://review.opendev.org/c/openstack/ironic-specs/+/983469 | 20:06 |
| cardoe | Guess who didn't backup his devstack VM and accidentally nuked it earlier? | 20:10 |
| * cardoe goes and sits in the shame corner. | 20:10 | |
| cardoe | I am testing from scratch following the docs... except I switch ipmi -> redfish. I might submit a patch to make that the default. | 20:10 |
| TheJulia | cardoe: your not allowed to sit in the shame corner | 20:11 |
| TheJulia | we put claude there with the continiously regenerating dunce cap | 20:11 |
| TheJulia | which is next to the barista setup.... which I don't know its not staffed ;) | 20:12 |
| cardoe | heh | 20:13 |
| * TheJulia suddenly craves a soy latte | 20:14 | |
| TheJulia | cardoe: okay, just let me know, because I think we should likely just move the db migration all into an online migration, and I'll make that comment as such | 20:14 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix idrac-redfish SKU to serial_number override to run before rule evaluation https://review.opendev.org/c/openstack/ironic/+/982990 | 20:16 |
| TheJulia | cardoe: fwiw, I put that on https://review.opendev.org/c/openstack/ironic/+/983215 | 20:33 |
| cardoe | Yeah I think that's totally legit. Getting my devstack back up. Then gonna make the changes so that I can confirm online migration. | 20:35 |
| TheJulia | k, yeah, its a fairly straight forward pattern, but yeah, its also one of the harder things to explicitly test | 20:37 |
| cardoe | So our devstack setup still has us using tinyipa | 20:54 |
| TheJulia | what?! | 21:24 |
| TheJulia | ugh.... | 21:24 |
| TheJulia | the default value is dib, but if you define a run with tinyipa that is sitll supported, I guess your noting we likely need to rip the rest out | 21:30 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix nova rebuilds w/ramdisk driver https://review.opendev.org/c/openstack/ironic/+/982877 | 21:36 |
| TheJulia | yeah, our examples | 21:45 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Remove IRONIC_RAMDISK_TYPE devstack setting https://review.opendev.org/c/openstack/ironic/+/983490 | 21:52 |
| TheJulia | cardoe: ^ | 21:52 |
| TheJulia | Looks like we still need slides for Redfish Enhancements and other Enhancements | 21:56 |
| iurygregory | cardoe, we tested a few scenarios and they seem to work well downstream, people only said "yeah it avoid the unnecessary reboot" XD | 22:01 |
| opendevreview | Merged openstack/networking-generic-switch master: CI: Remove setuptools pin https://review.opendev.org/c/openstack/networking-generic-switch/+/983283 | 22:37 |
| opendevreview | Merged openstack/ironic stable/2026.1: make target_datetime optional for set_bmc_clock step https://review.opendev.org/c/openstack/ironic/+/983466 | 22:46 |
| opendevreview | Merged openstack/ironic stable/2026.1: Firmware caching before OS boot complete https://review.opendev.org/c/openstack/ironic/+/983467 | 22:46 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix Premature reboot interrupting NIC update https://review.opendev.org/c/openstack/ironic/+/983468 | 22:47 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: DNM: Add debug logging for console_interface investigation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/983129 | 22:52 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: DNM: Add debug logging for console_interface investigation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/983129 | 22:58 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: DNM: Add debug logging for console_interface investigation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/983129 | 23:10 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: DNM: Add debug logging for console_interface investigation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/983129 | 23:10 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!