| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Redfish boot mode handling for read-only BMCs https://review.opendev.org/c/openstack/ironic/+/975521 | 06:33 |
|---|---|---|
| TheJulia | cardoe: oh, agree, but more so on my end trying to extract base requirements and if we can or understand a need which is different, then that is also a data point… and as with all things if enough data points, then why not at some point to make it more generally applicable | 06:37 |
| TheJulia | cardoe:I forgot about about the nested socket, doh. | 06:37 |
| TheJulia | I’ll try and sort that today | 06:38 |
| TheJulia | cardoe: 767-400 was the evil aircraft :) | 06:46 |
| TheJulia | Newer-ish long haul info-tainment systems with older style USB A ports :\ | 06:47 |
| TheJulia | Granted, 1A ports at like… 140 passengers means the odds of infotainment loads really spiking unexpectedly is sort of low. | 06:49 |
| TheJulia | iurygregory: janders: I don’t know if either of you know Jad, but a release note would be good :) | 07:30 |
| rpittau | good morning ironic! o/ | 07:43 |
| opendevreview | Merged openstack/ironic master: Fix retrieval of inventory data from Swift https://review.opendev.org/c/openstack/ironic/+/975824 | 08:08 |
| opendevreview | David Nwosu proposed openstack/ironic master: Fix firmware update error handling and test empty MessageId https://review.opendev.org/c/openstack/ironic/+/974307 | 08:38 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: l2vni baremetal mech driver https://review.opendev.org/c/openstack/networking-baremetal/+/973889 | 09:25 |
| TheJulia | good morning rpittau | 09:29 |
| rpittau | hey TheJulia :) | 09:31 |
| rpittau | oh wait, odd hours for you :D | 09:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Refactor redfish parser to eliminate code duplication https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975891 | 09:39 |
| TheJulia | rpittau: I'm in Waterford, IE this week | 09:51 |
| rpittau | oh nice! enjoy :) | 09:56 |
| opendevreview | Merged openstack/ironic master: Add standalone networking service installation guide https://review.opendev.org/c/openstack/ironic/+/966471 | 10:12 |
| opendevreview | Merged openstack/bifrost stable/2024.2: [stable only][2024.2] Ensure firewalld is unmasked before enabling it https://review.opendev.org/c/openstack/bifrost/+/975169 | 10:31 |
| opendevreview | Jad Haj Yahya proposed openstack/ironic master: Fix Redfish boot mode handling for read-only BMCs https://review.opendev.org/c/openstack/ironic/+/975521 | 10:40 |
| pas-ha[m] | hi team, have a question about redfish-https - is ironic expected to eject boot media when using 'direct' deploy when booting into provisioned instance? reason I am asking is that we have one specific configuration failing in our downstream tests because the non-ejected boot iso that is still seen by provisioned instance, and the config drive supplied by nova - both have 'config-2' label, which confuses cloud-init into not finding | 11:26 |
| pas-ha[m] | the nova one. | 11:26 |
| pas-ha[m] | besides, AFAIU boot iso might have some secrets inside that the provisioned instance not exactly should be able to access... 😕 | 11:26 |
| iurygregory | good morning ironic | 11:37 |
| iurygregory | TheJulia, https://review.opendev.org/c/openstack/ironic/+/975521 has a relasenote , or there is another patch ? | 11:41 |
| TheJulia | pas-ha[m]: so ejecting the vmedia, I honestly don't remember, if your referring to the agent token I don't think we would be worried about that. | 13:52 |
| rpittau | pas-ha[m]: in general, yes, Ironic is expected to eject the virtual media, what you're seeing could be a race when transitioning the node into active state, or even a bug, but you need to check the logs to see exactly what's happening, timeline is important here | 13:52 |
| rpittau | also yes, you're right in the boot iso there are information that the provisioned node should not access, so if the eject is not happening when transitioning to provisioned, we need to understand why | 13:52 |
| TheJulia | we likely should be ejecting | 13:52 |
| pas-ha[m] | rpittau: what this boils down | 13:53 |
| pas-ha[m] | to is the following | 13:53 |
| pas-ha[m] | in certain scenarios - at least with redfish-https + ironic node using portgroups - ironic generates boot iso with 'openstack/*' files for networking config, and then the _label function decides that this ISO should have label config-2, and I am really not sure this is right. | 13:55 |
| pas-ha[m] | I'll file a bug + patch and we can discuss there. | 13:56 |
| TheJulia | That is right for forcing the device to self configure from vmedia | 13:56 |
| TheJulia | The issue, it sounds like is it somehow remaining attached | 13:56 |
| rpittau | config-2 is the standard label for the config drive specification | 13:56 |
| rpittau | TheJulia: yeah for some reason the eject is not happening, might be a problem with redfish-https ? | 13:57 |
| pas-ha[m] | so, you mean config-2 on the boot iso is so that cloud-init in it can find "itself"/the same root disk it is booted from, and configure itself.. ok | 13:57 |
| TheJulia | well | 13:57 |
| TheJulia | redfish-https should be boot from the iso via the url, nothing else | 13:57 |
| TheJulia | so there shouldn't be actual attachment | 13:57 |
| TheJulia | pas-ha[m]: yeah | 13:57 |
| rpittau | I mean, if we don't clear the http boot url once the node moves to active state, the node will find that attached afterwards | 14:01 |
| pas-ha[m] | so does redhish-https boot iface + direct deploy iface has any meaning/is supported? | 14:01 |
| pas-ha[m] | if not, might this be the reason why redfish-https is listed as 'not supporting config drive'? | 14:01 |
| rpittau | pas-ha[m]: mmm well redfish-https not supporting config drive because of the limitation of the redfish http boot, yeah | 14:07 |
| pas-ha[m] | so what happens with redhish-https + direct, is the final instance boots of boot iso or from kernel/whatever written to disk? | 14:09 |
| rpittau | pas-ha[m]: it hsould boot from local disk | 14:13 |
| rpittau | pas-ha[m]: I suggest you open a bug, I think we may need to just vberify that the url is actually removed before rebooting, but we will need to check logs and code before giving a final answer | 14:14 |
| rpittau | also please check that there is nothing wrong with the BMC behavior like something blocking the cleanup of the http boot url | 14:14 |
| pas-ha[m] | BMC is sushy simulator in this particular case | 14:16 |
| pas-ha[m] | Ok, will file a bug | 14:16 |
| TheJulia | I wonder if the issue here, and this is likely an issue with sushy-tools is the detachment is likley not happening there | 14:26 |
| TheJulia | yeah, so that is exactly it | 14:26 |
| rpittau | wondering the same :/ | 14:26 |
| TheJulia | I thought sushy-tools detaches it, but yeah, its a sushy-tools bug | 14:28 |
| TheJulia | the base issue and root of the problem with testing http boot funcitonality with vmedia is the e2dk firmware has no concept of forcing it directly so your seeing our single run ci "hack" expecting similarity with hardware | 14:29 |
| rpittau | TheJulia: it's also true that we don't verify the ejection in https://opendev.org/openstack/ironic/src/commit/44184af8135e938b2d44ca83686186373b97c8a1/ironic/drivers/modules/redfish/boot.py#L449 | 14:39 |
| TheJulia | there is no ejection with booting from http | 14:40 |
| TheJulia | it is the reboot | 14:40 |
| rpittau | mmm right | 14:41 |
| TheJulia | https://opendev.org/openstack/sushy-tools/commit/feb88a5c461955e077151e6c33f1c1327077b138 | 14:41 |
| TheJulia | Basically, sushy tools needs to rip it out upon the next reboot | 14:44 |
| rpittau | yeah | 14:44 |
| rpittau | I can write a patch for that if you're not on it already | 14:46 |
| Sandzwerg[m] | ohai ironic, I have a question. Is anyone aware of ipxe built into the uefi shell and is there a way to use it with ironic? | 14:46 |
| TheJulia | rpittau: that would be much appreciated | 14:46 |
| TheJulia | in team building meetings so can't really work on code stuffs this week | 14:47 |
| rpittau | ok :) | 14:48 |
| rpittau | pas-ha[m]: do you have the bug link maybe ? | 14:53 |
| pas-ha[m] | not yet | 14:54 |
| JayF | I can start the meeting in 5 minutes if no one else gets to it before then | 15:01 |
| JayF | #startmeeting Ironic | 15:05 |
| opendevmeet | Meeting started Mon Feb 9 15:05:07 2026 UTC and is due to finish in 60 minutes. The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:05 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:05 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:05 |
| iurygregory | o/ | 15:05 |
| JayF | Hello and welcome to the Ironic meeting. As usual we are operating under the OpenInfra Code of Conduct | 15:05 |
| janders | o/ | 15:05 |
| JayF | #note A reminder to all: Please prioritize reviews on our priority dashboard, and feel free to add hashtag "ironic-week-prio" to patches that are ready for review. | 15:05 |
| cid | o/ | 15:06 |
| JayF | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:06 |
| JayF | We are in week R-7 of Gazpacho cycle. Only meaningful deadline this week is Extra-ACs deadline | 15:06 |
| JayF | If you know of any contributors to Ironic who may not have had code commits please contact a DPL asap in order to get them added to the list. | 15:06 |
| JayF | #link https://releases.openstack.org/gazpacho/schedule.html | 15:06 |
| JayF | #topic Working Group Updates | 15:07 |
| JayF | First up is standalone networking; alegacy do you have an update? | 15:07 |
| clif | o/ | 15:07 |
| iurygregory | I don't think alegacy is around .-. | 15:08 |
| JayF | Well that's OK, since I think that feature is mostly merged. | 15:08 |
| JayF | What about updates for either Async IO or VXLAN Networking. | 15:08 |
| JayF | I believe Julia is out; so I'll plug https://review.opendev.org/q/topic:ironic-vxlan+status:open on her behalf -- code is complete and needs reviews | 15:09 |
| iurygregory | Async IO we have the spec under review | 15:09 |
| rpittau | o/ | 15:09 |
| JayF | Care to link it Iury? | 15:10 |
| iurygregory | #link https://review.opendev.org/c/openstack/ironic-specs/+/972754/ | 15:10 |
| JayF | Good stuff. Thanks for that. We're reaching the end of gazpacho so it's a good time to be reviewing things close to the finish line or items for future releases. | 15:10 |
| iurygregory | ++ | 15:10 |
| JayF | Next up; discussion topics | 15:11 |
| JayF | #topic Discussion Topics | 15:11 |
| JayF | iurygregory: had the only agenda item here | 15:11 |
| iurygregory | yup | 15:11 |
| iurygregory | so, I'm working on https://bugs.launchpad.net/ironic/+bug/2137364 | 15:12 |
| iurygregory | tl;dr HPE machine changes the NetworkAdapter ID after a we do a NIC firmware update :D | 15:12 |
| iurygregory | this is a bit complicated, because currently we use the ID to know which NIC we are updating prefix `nic:` + NetworkAdapterID | 15:13 |
| iurygregory | there is a patch up for review | 15:13 |
| iurygregory | #link https://review.opendev.org/c/openstack/ironic/+/972421 | 15:13 |
| JayF | I put a review on that yesterday, the approach looks good but I like Dmitry's ideas around namespacing the id | 15:13 |
| iurygregory | the only problem I see is if we store that as the `component` on the DB | 15:14 |
| iurygregory | we will hit the same problem we have in case we update the nic | 15:14 |
| JayF | so you've gotta account for it in parsing code | 15:15 |
| JayF | and it just makes it a thing | 15:15 |
| iurygregory | oh ok | 15:15 |
| JayF | OK, seems sensible to not wanna go down that path. Can you make sure that's in gerrit | 15:15 |
| JayF | I'm agreeing with you :) wait another 20 seconds before conceding ;) | 15:15 |
| iurygregory | we just care about the `nic:<SN>` to identify the right component? | 15:15 |
| iurygregory | we can go, but it would be a bit hack and ugly I think | 15:15 |
| iurygregory | unless we have some field that will contain the ID information | 15:16 |
| JayF | that's a good reason to leave it be | 15:16 |
| JayF | which means the namespace isn't a quick or easy win | 15:16 |
| iurygregory | yeah, at least I have this feeling, not sure what others think | 15:16 |
| JayF | I'm sold and so far the meeting is me, you, and a few o/'s :D | 15:17 |
| iurygregory | rpittau, do you have thoughts on this? | 15:17 |
| JayF | lol | 15:17 |
| iurygregory | yup! | 15:17 |
| rpittau | iurygregory: I haven't checked the patch, sorry, I'll have a look at it | 15:18 |
| iurygregory | rpittau, ack | 15:18 |
| JayF | Yeah sounds like that can be worked out in review, let's move on | 15:18 |
| iurygregory | I will add info in the patch to explain what we discussed here on irc | 15:18 |
| iurygregory | yup, lets move on | 15:18 |
| JayF | #topic Bug Deputy updates | 15:18 |
| kubajj | o/ | 15:18 |
| JayF | I think this is saying iurygregory is the bug deputy? or maybe iurygregory is the next one? | 15:18 |
| JayF | unsure who current one is, but the only reported bug already has a fix merged so | 15:19 |
| JayF | uh, go us? | 15:19 |
| iurygregory | yeah, I'm the bug deputy for this week | 15:19 |
| iurygregory | cid was last week | 15:19 |
| cid | ++ | 15:19 |
| cid | That's right, saw only one new bug for the week | 15:19 |
| JayF | good work, thanks for being bug deputy next week Iury | 15:21 |
| JayF | There are no RFEs for review; skipping that section. | 15:21 |
| JayF | #topic Open Discussion | 15:21 |
| JayF | fungi: I assume the Bridging the Gap retrospective is an old item that didn't get cleaned up? | 15:21 |
| JayF | I have an item for Open Discussion: Thursday will be my last day online working on OpenStack through (up to) the end of the month. I'll be out for a minor surgery which will lead to restricted use of my mousin' arm for a couple of weeks. | 15:22 |
| cid | re'Bridging...'. I think that's a left over | 15:23 |
| clif | I'd like ask about this patch I put up to remove shellinabox from Ironic since it's considered abandoned at this point | 15:23 |
| clif | #link https://review.opendev.org/c/openstack/ironic/+/975864/comment/6d06fd5c_0c090d2f/ | 15:23 |
| janders | JayF take care, hope the surgery goes well! | 15:23 |
| janders | best wishes | 15:23 |
| iurygregory | take care JayF o/ | 15:24 |
| JayF | clif: I would vote to also remove the iLO console support which depends on shellinabox; the supporting software is old/unsupported enough we should be drastic in our removal. | 15:24 |
| clif | Essentially removing IPMIShellinaboConsole means that the class that IloConsoleInterface inherits from goes away, leaving me wondering what it should derive from now. JayF pointed out that iLO is also going away | 15:24 |
| fungi | JayF: yes, sorry, i should have pulled that off the agenda, was only planning to discuss in-meeting for two weeks, which we did. thanks! | 15:24 |
| cid | Jay, best wishes. | 15:25 |
| cid | er, JayF. | 15:25 |
| clif | so maybe I should rip out the iLO driver as well in this patch or a direct follow-up patch | 15:25 |
| JayF | the ilo driver itself can't go until next cycle, with the snmp driver, per deprecation policy | 15:25 |
| clif | JayF: I too hope your surgery goes well and wish you a speedy recovery | 15:25 |
| JayF | however I would remove the console support from the ilo driver to enable shellinabox to be fully removed | 15:25 |
| clif | I see, that makes sense | 15:26 |
| JayF | I think about it this way: ideally we would've billboarded loss of the iLO console, too | 15:26 |
| iurygregory | ++ to remove the console support like JayF mentioned | 15:26 |
| JayF | but realistically, nobody should be running this crap in 2026 anyway | 15:26 |
| JayF | and I'd rather rip the bandaid off | 15:26 |
| cid | I had a patch to remove ILo, but I think the review was that there needs to be a deprecation first. https://review.opendev.org/c/openstack/ironic/+/965009 | 15:27 |
| clif | well then I'll leave the driver removal to cid when the time comes | 15:28 |
| clif | but just remove the ilo console support for now unless someone objects | 15:28 |
| JayF | ++ Thank you | 15:28 |
| cid | ++, no objections from me. clif, maybe you start the ilo removal with the deprecation patch? | 15:29 |
| JayF | My belief is we did deprecate it | 15:29 |
| JayF | and if we didn't... whoa boy we're in one | 15:29 |
| JayF | becaues it's gotta go away next cycle; it hard-deps on the SNMP driver | 15:29 |
| JayF | (proliantutils does) | 15:29 |
| JayF | and proliantutils is near-death | 15:29 |
| janders | yeah since we're removing iRMC I think it makes sense to try clean up old drivers | 15:30 |
| janders | looks like the future is converging on Redfish | 15:30 |
| JayF | I guess I'll take an action to see if we formally deprecated iLO, if we didn't I'll do a "formal" patch | 15:30 |
| JayF | but there's no reality where the SNMP bits of the iLO driver are in the repo after 2026.2 release | 15:31 |
| JayF | and I don't see a line of volunteers to modernize it | 15:31 |
| JayF | Last call for discussion on this item, or new items for Open Discussion? | 15:32 |
| janders | just a quick mention from me | 15:33 |
| janders | after a couple months on and off in Europe I am heading back to AUS next week | 15:33 |
| janders | so unfortunately won't be attending this meeting as much as I have recently | 15:33 |
| JayF | Australia? | 15:33 |
| janders | will do my best to keep up being active in reviewing patches though! | 15:33 |
| janders | yes | 15:33 |
| janders | it's been great collaborating with you folks in real time for some time! | 15:34 |
| JayF | ˙ƃuᴉʇǝǝɯ ǝɥʇ ʇɐ noʎ ssᴉɯ ll,ǝM ¡ɟlǝsɹnoʎ ʎoɾuƎ | 15:34 |
| janders | JayF LOL! thanks! :) | 15:34 |
| clif | hold on, gotta turn my monitor upside down | 15:34 |
| JayF | Anything else for the meeting? | 15:34 |
| JayF | Thanks folks! Have a good week. | 15:35 |
| JayF | #endmeeting | 15:35 |
| opendevmeet | Meeting ended Mon Feb 9 15:35:46 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-02-09-15.05.html | 15:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-02-09-15.05.txt | 15:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-02-09-15.05.log.html | 15:35 |
| kubajj | janders: I was wondering how come you've been around in reasonable times recently (reasonable for me), didn't know you were in Europe | 15:39 |
| kubajj | hope everything goes well, JayF! | 15:40 |
| kubajj | How bad of an idea is it to change the inspect_interface directly in the DB (question about https://docs.openstack.org/ironic/2025.1/admin/inspection/migration.html#migration-process step 10). The problem is that I can't change it for active nodes, at least I think | 15:41 |
| clif | JayF: I suspect removing the ilo console stuff is going to break a lot of the ilo driver, just sayin | 15:53 |
| JayF | I am ... extremely skeptical of that statement | 15:53 |
| JayF | especially given shellinabox is just a frontend to the console | 15:54 |
| JayF | I have time to look at it together if you want? | 15:54 |
| clif | well I would trust your judgement on the matter, but if I remove IloConsoleInterface then the IloHardware has no support console interfaces left | 15:54 |
| clif | supported* | 15:55 |
| JayF | Yeah, that's 100% OK | 15:55 |
| JayF | console is a bonus feature :) | 15:55 |
| clif | let me poke a little more | 15:56 |
| clif | I just needed to persevere, a lot of tests were breaking but I just needed to remove ilo as a valid console interface | 16:05 |
| JayF | makes sense | 16:05 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-tools master: Eject HttpBootUri media on boot target change https://review.opendev.org/c/openstack/sushy-tools/+/976138 | 16:12 |
| rpittau | pas-ha[m]: this should fix the issue ^ | 16:12 |
| rpittau | TheJulia: FYI ^ | 16:13 |
| opendevreview | Clif Houck proposed openstack/ironic master: Remove IloConsoleInterface since it depends on shellinabox https://review.opendev.org/c/openstack/ironic/+/976139 | 16:17 |
| iurygregory | JayF, question: do you think we should only use `nic:<SN>` to HPE change it by default and fallback to `nic:<ID>` in case <SN> is not present? | 16:23 |
| iurygregory | currently the patch only takes in consideration `nic:<SN>` to HPE, since its where we observed the problem | 16:24 |
| JayF | I think changing things we don't have to change just exposes us to the opposite of this case (Where SN is not valid and ID is) | 16:24 |
| rpittau | iurygregory, JayF, I left a comment on the patch | 16:54 |
| opendevreview | Merged openstack/networking-baremetal master: Remove MANIFEST.in https://review.opendev.org/c/openstack/networking-baremetal/+/975983 | 16:56 |
| JayF | rpittau: I had already responded to your comment :D | 16:58 |
| opendevreview | Luca Del Monte proposed openstack/ironic-prometheus-exporter master: Fix KeyError when timestamp is missing from notification payload https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/976153 | 17:15 |
| opendevreview | Luca Del Monte proposed openstack/ironic-prometheus-exporter master: Fix KeyError when timestamp is missing from notification payload https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/976153 | 18:56 |
| opendevreview | Luca Del Monte proposed openstack/ironic-prometheus-exporter master: Fix KeyError when timestamp is missing from notification payload https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/976153 | 19:01 |
| opendevreview | Merged openstack/bifrost stable/2024.2: Add the ability to override 'disable_boot_timeout' https://review.opendev.org/c/openstack/bifrost/+/974588 | 19:10 |
| opendevreview | Luca Del Monte proposed openstack/ironic-prometheus-exporter master: Fix KeyError when timestamp is missing from notification payload https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/976153 | 19:53 |
| opendevreview | Verification of a change to openstack/ironic-ui master failed: Remove MANIFEST.in https://review.opendev.org/c/openstack/ironic-ui/+/975982 | 20:59 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Remove MANIFEST.in https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/975990 | 21:39 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!