| opendevreview | Merged openstack/ironic master: Fix Lenovo UEFI boot loop in prepare_instance flow https://review.opendev.org/c/openstack/ironic/+/986597 | 02:13 |
|---|---|---|
| opendevreview | Merged openstack/networking-baremetal stable/2026.2: Don't let stale agent cleanup abort agent startup https://review.opendev.org/c/openstack/networking-baremetal/+/1007546 | 05:43 |
| opendevreview | Merged openstack/networking-baremetal stable/2026.1: Don't let stale agent cleanup abort agent startup https://review.opendev.org/c/openstack/networking-baremetal/+/1007547 | 06:21 |
| mnasiadka | Hmm, is that normal that bifrost upgrade jobs are failing? https://zuul.opendev.org/t/openstack/build/bb99021b72df4aa18586e10dc14de28b | 08:21 |
| rpittau | mnasiadka: yes... well, no! :D we're working to fix that | 08:23 |
| mnasiadka | Thanks, so I’ll wait ;) | 08:23 |
| rpittau | couple of patches still missing, should be quick | 08:24 |
| *** jroll09 is now known as jroll0 | 08:56 | |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:08 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:41 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:43 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:43 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:43 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:44 |
| *** Unknown123 is now known as Mike-- | 09:45 | |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:46 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 09:48 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 10:22 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost master: Experimental support for ARM64 hosts on Debian https://review.opendev.org/c/openstack/bifrost/+/920057 | 11:38 |
| dtantsur | cardoe, TheJulia, hey, I'm not sure which comments are calls for actions in https://review.opendev.org/c/openstack/ironic-specs/+/1006205 please clarify | 12:06 |
| opendevreview | Merged openstack/bifrost stable/2026.2: Update .gitreview for stable/2026.2 https://review.opendev.org/c/openstack/bifrost/+/1006453 | 12:15 |
| opendevreview | Merged openstack/networking-baremetal master: docs: Clean up the diagram to be more explicit https://review.opendev.org/c/openstack/networking-baremetal/+/990294 | 12:22 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2026.2: [CI] Upgrade non-SLURP 2026.2 from 2026.1 https://review.opendev.org/c/openstack/bifrost/+/1007855 | 12:37 |
| opendevreview | Jed Preist proposed openstack/ironic master: Enable redfish-virtual-media on iDRAC10 https://review.opendev.org/c/openstack/ironic/+/1007856 | 13:06 |
| TheJulia | dtantsur: I think it is more discussion required and figure out along what lines to consider more "crud style" interactions than higher level workflows of specific desired patterns. | 13:09 |
| TheJulia | Overall, both cardoe's and your specs are fairly agreeable, the key is more so make sure we don't shoot each other in the foot and ensure we're complimenting | 13:10 |
| dtantsur | TheJulia: unless we want to open pass-thru API to Redfish, we cannot avoid having specific workflows | 13:10 |
| dtantsur | our specs are completely orthogonal to the best of my knowledge | 13:10 |
| opendevreview | Jed Preist proposed openstack/ironic master: Enable redfish-virtual-media on iDRAC10 https://review.opendev.org/c/openstack/ironic/+/1007856 | 13:11 |
| TheJulia | dtantsur: well, I'm more thinking the right way to handle some aspects is to have lower level items such as "configure_bmc_setting_x" and a higher level step which is "configure_ldap" or "configure_oidc" and it expands out to the lower level steps with vendor context | 13:11 |
| TheJulia | dtantsur: they are overlapping quite a bit from my point of view | 13:12 |
| dtantsur | TheJulia: the confusion you may be operating it is that these things are configurable via some generic "BMC settings" API, like the one cardoe is proposing. | 13:12 |
| dtantsur | Which is why they're *NOT* overlapping :) | 13:12 |
| TheJulia | that doesn't mean either are wrong | 13:12 |
| TheJulia | So your saying that there is entirely separate specific endpoints for each and every action you've proposed? | 13:12 |
| dtantsur | TheJulia: correct. I think I listed them in the spec but maybe I forgot. | 13:13 |
| TheJulia | I didn't see them offhand, but that doesn't mean they are not there | 13:13 |
| dtantsur | Password -> ManagerAccount, LDAP -> AccountService, networking -> EthernetInterface, what cardoe proposes -> @Settings (IIRC) | 13:13 |
| dtantsur | It's possible that some of them are also exposed via Settings. This is something that is not 100% clear to me so far (I lost access to hardware because of internal lab stuff) | 13:14 |
| TheJulia | So, I don't think every bmc modeles remote auth via just account service, I seem to remember dell and hp having a bunch of oem settings | 13:14 |
| TheJulia | but I'd have to pull them up again and look | 13:14 |
| dtantsur | This stuff changes a lot between generations | 13:14 |
| TheJulia | indeed | 13:15 |
| TheJulia | and breaks on how willing they are to maintain/keep/make usable the generic endpoints | 13:15 |
| dtantsur | TheJulia: example: https://github.com/dell/iDRAC-Redfish-Scripting/blob/master/Redfish%20Python/CreateIdracUserCsvFileREDFISH.py#L76-L80 | 13:16 |
| TheJulia | I guess what I'm saying is I'd slightly smaller building blocks which can be interated upon, but I'd like to see both of y'all talk before I toss +2's on each item. | 13:17 |
| dtantsur | Sure. As long as it's not in limbo for weeks.. | 13:18 |
| TheJulia | This is where we replace cardoe's coffee with decaf and see what happens ;) | 13:18 |
| cardoe | So I was seeing it as all generic settings and then Dmitri's on top | 13:18 |
| TheJulia | cardoe: yeah, that is sort of how I'm thinking. doing user actions is distinctly different | 13:19 |
| dtantsur | Authoritative answer about LDAP: https://redfishforum.com/thread/913/configure-ldap-redfish-api | 13:19 |
| TheJulia | I guess the thing is "if the abstracted endpoint is not available", the default first step will always be lower level settings | 13:19 |
| TheJulia | so, fwiw, you'll likely need to also support different models of ldap config | 13:20 |
| dtantsur | I'd like to start with the official one and work from there. I don't have cycles to boil the ocean, unfortunately. | 13:21 |
| TheJulia | mraineri's comment has a server password, not every server will require that. Folks who permit anonymous searching authenticate as the user requesting upon finding the entry | 13:21 |
| * TheJulia ran ldap ages ago | 13:21 | |
| TheJulia | oh yeah, not saying to boil the ocean. That is for cardoe's AI prompt to go crazy on for us. ;) | 13:21 |
| TheJulia | The whole purpose of a spec is to spread the context around and enable discusison, I think that is what is happening | 13:22 |
| * TheJulia wonders if we should have instead replaced cardoe's coffee with something stronger | 13:23 | |
| TheJulia | Speaking of stalled review items, I'd really love some reviews on the outstanding networking specs since I have folks demanding more and more advanced networking all the time. | 13:28 |
| TheJulia | cardoe: you owe JayF a reply on https://review.opendev.org/c/openstack/ironic-specs/+/999448 | 13:29 |
| opendevreview | Merged openstack/networking-baremetal stable/2026.1: Remove duplicate OVN LocalnetPortEvent registration https://review.opendev.org/c/openstack/networking-baremetal/+/1006940 | 13:30 |
| opendevreview | Merged openstack/networking-baremetal stable/2026.2: Remove duplicate OVN LocalnetPortEvent registration https://review.opendev.org/c/openstack/networking-baremetal/+/1006938 | 13:47 |
| cardoe | TheJulia: I'll get on it | 13:54 |
| cardoe | dtantsur: so for example I won't find the LDAP part useful since I don't use that. But generically the account service is useful. | 13:55 |
| dtantsur | cardoe: so you're thinking more of a generic update_account_service? | 13:55 |
| cardoe | probably? | 13:56 |
| dtantsur | As in, have a curated list fields that can be PATCHed in? | 13:58 |
| dtantsur | TheJulia, any thoughts? | 13:58 |
| TheJulia | I think that would be pretty solid | 13:58 |
| TheJulia | and if we want to put a wrapper step around that, I'm all for treating our steps that way ;) | 13:59 |
| dtantsur | And the same instead of just updating Password? Or is specific to AccountService configuration? | 13:59 |
| dtantsur | I'm trying to strike a balance between enabling features and this turning into passthru PATCH API | 14:00 |
| dtantsur | I.e. to which extent do we want it fully specified (vs accepting a JSON blob and PATCHing it in)? | 14:01 |
| TheJulia | for accounts themselves, I think we're going to need a little more variety of options on specific actions. Honestly. | 14:02 |
| TheJulia | everyone's policy/model is viable to vary | 14:03 |
| TheJulia | and thats okay, just need to find what is reasonable for us to do and what we should avoid | 14:03 |
| dtantsur | I can model both the way I model LDAP data right now: we receive a DMTF JSON that we validate with a (stripped down) version of their schema | 14:03 |
| dtantsur | and then PATCH. | 14:04 |
| dtantsur | I can even roll all of this into one step if that's preferred. I'm not sure what to do with the networking part though. | 14:06 |
| TheJulia | The networking one is a bit harder because its such an opinionated space and even at simplest config there are... ?4? dramatically different configuraitons | 14:08 |
| dtantsur | And btw, cardoe, if we go down that ^^^ path, you'll need to update your approach too. Right now, you have several steps for what is essentially Dell attributes. | 14:11 |
| TheJulia | Dell and HPE hide their oem things in entirely separate areas | 14:12 |
| TheJulia | fwiw | 14:12 |
| dtantsur | Yeah, but the patch is about Dell specifically | 14:13 |
| TheJulia | Doug's though, is targetted at those lowest level settings. "EnsureSpecialPowerForPCISlot5" and being able to toggle it is the mental space I think he is at | 14:13 |
| TheJulia | yeah | 14:13 |
| dtantsur | I mean the other one, https://review.opendev.org/c/openstack/ironic/+/1007083 | 14:13 |
| TheJulia | oh! | 14:14 |
| TheJulia | I haven't looked at his actual change yet | 14:14 |
| dtantsur | All 3 are set_dell_attributes in a trench coat | 14:14 |
| TheJulia | heh | 14:14 |
| TheJulia | So how many layers of trench coats is that?! | 14:14 |
| dtantsur | Yes! | 14:14 |
| TheJulia | 'Hey server admin! I've got some stuff to make your severs go brrrrrrrrr!" | 14:15 |
| JayF | dtantsur: I don't know if my downstream woulduse Ironic to configure it, but we use the LDAP support in BMCs. | 14:15 |
| JayF | dtantsur: (including on a dell so you can increment the !root/calvin counter to 1) | 14:16 |
| dtantsur | \o/ | 14:16 |
| TheJulia | .... Hopefully the administration in the US never tries with data center ops folks like what they did to all pilots/flight attendants last week. | 14:16 |
| dtantsur | Do I want to know? | 14:17 |
| TheJulia | I mean, if you follow anything flying related on reddit, you would have seen the flight crew responses by now | 14:17 |
| * dtantsur does not | 14:17 | |
| JayF | cardoe: I'll be doing my VMT slides for the OIF presentation on Thursday today; if you wanna work on Ironic ones at well I'll be engaged for cross-review | 14:18 |
| TheJulia | Thinking outloud before I walk the corgi, does anyone have any decent tools to fake message chats like via text messages? I've got a talk coming up in november and it might be a fun way to convey across some of the key context as a dialog | 14:29 |
| cardoe | yeah later this afternoon | 14:35 |
| dtantsur | TheJulia: by the way, was there a reason why you didn't add reattach_networking to ironic-networking? Or did it simply not exist back then? | 14:52 |
| TheJulia | Timeline wise, I don't think it existed back then | 14:53 |
| TheJulia | I... honestly... don't remember even doing that work and I pulled it up and was like "oh!" | 14:53 |
| dtantsur | heh | 14:53 |
| TheJulia | well, I remember I did it, but didn't remember when | 14:53 |
| dtantsur | it may come handy for one of our obscure cases | 14:53 |
| TheJulia | good to hear | 14:54 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Also support reattach_network step in ironic-networking https://review.opendev.org/c/openstack/ironic/+/1007887 | 14:59 |
| opendevreview | Merged openstack/networking-baremetal master: Validate device options on startup https://review.opendev.org/c/openstack/networking-baremetal/+/1002522 | 15:26 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Declare Python 3.14 support https://review.opendev.org/c/openstack/ironic/+/1007894 | 15:38 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Declare Python 3.14 support https://review.opendev.org/c/openstack/ironic/+/1007894 | 15:39 |
| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Declare Python 3.15 support https://review.opendev.org/c/openstack/python-ironicclient/+/1007895 | 15:40 |
| opendevreview | Merged openstack/ironic master: Fix source version in SLURP job https://review.opendev.org/c/openstack/ironic/+/1007686 | 15:50 |
| TheJulia | JayF: When you get a chance to glance, the vmt side task on https://bugs.launchpad.net/ironic/+bug/2160548 is marked incomplete. I don't really think it makes sense to issue anything, but just wanted to higlight it for housekeeping purposes | 15:54 |
| JayF | commented | 15:55 |
| JayF | marked invalid | 15:55 |
| JayF | er, wontfix | 15:55 |
| TheJulia | k, thx | 15:57 |
| opendevreview | Merged openstack/networking-generic-switch master: docs: revise the docs depth for improved browsing clarity https://review.opendev.org/c/openstack/networking-generic-switch/+/1006823 | 16:35 |
| opendevreview | Merged openstack/ironic stable/2026.1: Don't raise if an object's attribute is None in _retrieve_attribute https://review.opendev.org/c/openstack/ironic/+/1007758 | 18:06 |
| opendevreview | Verification of a change to openstack/ironic stable/2026.2 failed: Don't raise if an object's attribute is None in _retrieve_attribute https://review.opendev.org/c/openstack/ironic/+/1007757 | 18:09 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: netconf: Add Juniper NETCONF switch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/1007523 | 18:48 |
| opendevreview | Verification of a change to openstack/ironic master failed: Declare Python 3.14 support https://review.opendev.org/c/openstack/ironic/+/1007894 | 21:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Declare Python 3.14 support https://review.opendev.org/c/openstack/ironic/+/1007894 | 21:46 |
| TheJulia | looks like ironic-python-agent-builder is unhappy: https://paste.opendev.org/show/bGbfVr9dsMhJxBcJ2kxh/ anyone aware of this? | 22:02 |
| opendevreview | Verification of a change to openstack/ironic master failed: Use official service type name for cinder https://review.opendev.org/c/openstack/ironic/+/1007590 | 22:13 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Don't remove polkit-libs on el10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007981 | 22:43 |
| TheJulia | That should fix dib up | 22:43 |
| TheJulia | well, ironic-python-agent-builder | 22:43 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Don't remove polkit-libs on el10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007981 | 22:55 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Don't remove polkit-libs on el10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007981 | 22:59 |
| TheJulia | okay, fun mere conflict that git review should have highlighted :( | 23:02 |
| JayF | pre-applied a +2 if CI likes it | 23:17 |
| JayF | thank you :) | 23:17 |
| TheJulia | Anyway, it looks like 9-stream building is no longer viable... the checksum files are missing for everything but azure and it does a "this is old popup" | 23:35 |
| TheJulia | so.... yeeah. | 23:35 |
| TheJulia | Any opinions on disabling the 9-stream building and only running 10? | 23:35 |
| TheJulia | https://paste.opendev.org/show/bKo0MHcO7Sfl9LmHvwef/ | 23:38 |
| TheJulia | I've pinged spotz specifically | 23:41 |
| 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 | 23:50 |
| 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 | 23:50 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Do all package removals with install-packages -e https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1007323 | 23:50 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!