| opendevreview | Doug Goldstein proposed openstack/ironic master: Add iDRAC BMC config service steps (NTP, DNS, OIDC) https://review.opendev.org/c/openstack/ironic/+/1007083 | 03:16 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/networking-generic-switch master: Replace reference to [ml2] tenant_network_types https://review.opendev.org/c/openstack/networking-generic-switch/+/1006980 | 03:58 |
| rpittau | please any core when got a minute approve https://review.opendev.org/c/openstack/bifrost/+/1006697 to start chain fixing bifrost CI, thanks! :) | 07:32 |
| iurygregory | good morning ironic | 11:38 |
| opendevreview | Merged openstack/bifrost stable/2025.1: [stable only] CI: Fall back to HTTPS when cirros 0.5.3 is not cached https://review.opendev.org/c/openstack/bifrost/+/1006697 | 12:40 |
| rpittau | \o/ | 12:49 |
| iurygregory | <happy dance gif> | 12:50 |
| rpittau | iurygregory: if you have a moment, can you also please check https://review.opendev.org/c/openstack/ironic/+/1006947 ? thanks! | 12:51 |
| iurygregory | sure! | 12:54 |
| opendevreview | Merged openstack/ironic master: Skip package install in pep8 tox env https://review.opendev.org/c/openstack/ironic/+/1006947 | 13:20 |
| cardoe | So some sketchy requests of me for anyone with some gear handy to run some commands... don't worry it's totally fine... run this strangers commands... | 13:30 |
| cardoe | curl -k -s -u "${BMC_USER}:${BMC_PASS}" "https://${BMC_HOST}/redfish/v1/UpdateService/FirmwareInventory" | jq -r '.Members[]."@odata.id"' | 13:30 |
| cardoe | I make it sound scary as a joke. | 13:30 |
| cardoe | But really what I'm interested in is seeing what kind of firmware folks see out there | 13:31 |
| cardoe | I'm wanting to use this data for our spec review and PTG sessions | 13:31 |
| iurygregory | cardoe, let me see what I can find for you =) | 13:39 |
| TheJulia | brraaains | 14:27 |
| TheJulia | so, w/r/t bifrost, I'll just point out its presently trying to jump 2025.2 to 2026.2 | 14:35 |
| rpittau | TheJulia: yep, I am chain fixing the ci | 14:35 |
| rpittau | this is the second patch https://review.opendev.org/c/openstack/bifrost/+/1006542 | 14:36 |
| rpittau | then we will need at least another one | 14:36 |
| TheJulia | JayF: your -1 is still present on that one from before the other fix landed, jfyi | 14:37 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [CI] Upgrade non-SLURP 2026.2 from 2026.1 https://review.opendev.org/c/openstack/bifrost/+/1007252 | 14:47 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [CI] Upgrade master from 2026.2 and add slurp job https://review.opendev.org/c/openstack/bifrost/+/1007253 | 14:50 |
| JayF | in those cases I'll never be annoyed to have it landed over the objection | 15:02 |
| JayF | -1 for CI failing is basically me just trying to point "I looked at this and it's not a random failure" :D | 15:03 |
| JayF | (I already alnded it fwiw) | 15:03 |
| TheJulia | k, thx | 15:20 |
| cardoe | TheJulia: https://review.opendev.org/c/openstack/networking-baremetal/+/1002522 what do we wanna do with that? | 15:32 |
| TheJulia | I *think* a rebase and land, tbh | 15:33 |
| TheJulia | because the list definition stuff I did doesn't really change that, but sort of does as it shifts hydration | 15:33 |
| JayF | I think in general my team has been bitten with the "evaluate config early" bug, based on that patch and stuff clif has been working on. Quite nice enhancement, really | 15:38 |
| JayF | sorta small UX things that make operations a bunch easier | 15:38 |
| TheJulia | yeah, the listing is separated from the loading | 15:41 |
| JayF | I've been thinking about how to voice this, like as a goal | 15:41 |
| JayF | but stuff like that where ... maybe we don't remove a decoder ring, but improve the iteration cycle | 15:42 |
| TheJulia | oh, wait, that was NGS I'm thinking of | 15:42 |
| JayF | e.g. making it fail immediately on boot, have stronger types on configs so "wrong" is identified earlier | 15:42 |
| JayF | a bad setting in ironic.conf should be known as bad, if possible, on startup | 15:42 |
| TheJulia | tests running on rebase | 15:43 |
| TheJulia | yeah, the issue depending on invocation, that validation doesn't actualy occur on service start | 15:43 |
| JayF | that behavior was improved in ironic last cycle aiui | 15:44 |
| JayF | when clif ran across that behavior while working on the kernel parameter stuff | 15:45 |
| JayF | I think he has a patch into oslo config | 15:45 |
| JayF | basically values weren't being coerced into their declared type until usage time | 15:45 |
| TheJulia | even a patch to oslo.config, late plugin loading still means its later on | 15:45 |
| JayF | not at loading time | 15:45 |
| TheJulia | That makes sense | 15:45 |
| JayF | yeah, you need to call oslo-config-validator first or have code that specifically insantiates them | 15:45 |
| JayF | for Ironic I think wejust have a method we call when loading opts that just specifically loads in the ones we know have a type validator | 15:46 |
| TheJulia | I hear you, but its not that simple when your a neutron plugin | 15:46 |
| JayF | ah yeah | 15:47 |
| JayF | we can't make them load us early | 15:47 |
| * TheJulia wonders if networking-baremetal ci even works right now | 15:48 | |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: Validate device options on startup https://review.opendev.org/c/openstack/networking-baremetal/+/1002522 | 15:50 |
| JayF | cardoe: can I draft you into doing an Ironic update for OpenInfra Live on Thursday, Oct 1 | 15:51 |
| cardoe | yeah what time are they? | 15:52 |
| JayF | my calendar invite is @ 630am PT | 15:54 |
| JayF | I'll let Allison know you're in, and we'll have to get (a) slide(s) ready for Ironic. | 15:55 |
| JayF | nothing too detailed or tough, just hit the high notes | 15:55 |
| JayF | and I'll be presenting on VMT items as well, so you can assume I'll cover security with broad strokes | 15:55 |
| cardoe | okay. you wanna start a slide deck and share to me? | 16:04 |
| JayF | I just replied on the mail thread | 16:05 |
| JayF | it's a single shared deck w/all presenting projects | 16:06 |
| cardoe | okay great | 16:14 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Replace own utility to wrap IPv6 address https://review.opendev.org/c/openstack/ironic-python-agent/+/1007273 | 16:28 |
| TheJulia | we're down to 240 bugs \o/ | 16:37 |
| opendevreview | Merged openstack/bifrost stable/2025.2: [stable only] CI: Fall back to HTTPS when cirros 0.5.3 is not cached https://review.opendev.org/c/openstack/bifrost/+/1006542 | 16:47 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Map iptables to iptables-nft on redhat family https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007320 | 20:52 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Handle passwd coming from shadow-utils on rhel-10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007321 | 20:52 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: ironic-python-agent-ramdisk-base don't depend on epel https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007322 | 20:52 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Do all package removals in package-installs.yaml https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007323 | 20:52 |
| JayF | oooh, TIL about phase: in package-installs.yaml | 20:54 |
| cardoe | PC LOAD LETTER? | 21:50 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: Add iDRAC BMC config service steps (NTP, DNS, OIDC) https://review.opendev.org/c/openstack/ironic/+/1007083 | 21:56 |
| TheJulia | JFYI, we should keep an eye on "[cinder][ubuntu] dd from Rust coreutils breaks Cinder host copies on Ubuntu 26.04", since IPA uses odirect with dd as well | 22:47 |
| TheJulia | cardoe: I once had a VP whose nick name was "PC".... you can imagine how many requests he receieved. | 22:47 |
| JayF | I'll tell any users broken to contact their nearest linux distribution smart enough to not replace decades-old softwares :| | 22:47 |
| JayF | that's a case of them choosing to break users so they can see how much of the API they can get away with not respecting in those tools, and that's kinda crappy | 22:48 |
| TheJulia | espescialy o_direct... | 22:48 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!