| opendevreview | Takashi Kajinami proposed openstack/ironic master: Validate [glance] allowed_direct_url_schemes https://review.opendev.org/c/openstack/ironic/+/1003010 | 01:57 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Validate [glance] allowed_direct_url_schemes https://review.opendev.org/c/openstack/ironic/+/1003010 | 03:48 |
| *** jlvillal6 is now known as jlvillal | 12:40 | |
| TheJulia | good morning | 13:07 |
| JayF | FYI, I'm on a sick day today | 13:24 |
| lroussarie[m] | Good morning,... (full message at <https://matrix.org/oftc/media/v1/media/download/AdGn1TcLYAB_FD6tvybkJNHcI--971F4xsorIyPPfouoh1HZZ76XCJk-iqO8mjqogk4wsQ_48ehsFyG6b2W8OeBCegoB_CmwAG1hdHJpeC5vcmcveGV0SXBTQ0JpWnJUZGVmSWRZWmZiUWhP>) | 13:33 |
| lroussarie[m] | I saw that TheJulia started an implementation in 2019-2020 (changes 688581, 688977, 711853)... (full message at <https://matrix.org/oftc/media/v1/media/download/AXi2E3y8EvUfDiooMqy6ymSlHY1IymfJNMbkgLUCyaj7KvhwHeB2dFSjgCyBbmu_V3E_6cLLx9Zx7Cu32dOwCuhCegoB_QAgAG1hdHJpeC5vcmcvU0FNcVBnY1V0UlNSeUVXeUFteE1QeEh6>) | 13:33 |
| TheJulia | JayF: feel better! | 13:34 |
| TheJulia | lroussarie[m]: You may want to try shorter messages, your message are getting truncated and replaced with giant links | 13:34 |
| TheJulia | Ahh, yes, kexec | 13:38 |
| lroussarie[m] | TheJulia: yes ! I have been working on kexec support in ironic for a POC with baremetal-operator / ironic / IPA | 13:40 |
| TheJulia | So, kexec often gets pushed by folks who want to save that final reboot cycle, which is not as generally applicable, but brings complexity in that happening. For example, how do you know if your image is actually bootable if you never actually boot. Its much more of an advanced user's feature | 13:40 |
| TheJulia | Generally specs are advisable, I think kexec always seemed like a "smaller" idea to implement, but if you'd like to write up a spec to explain your throught process, I would be happy to review it. | 13:41 |
| TheJulia | The other issue is the underlying security teardown, in our model we need to rip the machine off the provisioning network and place it on a tenant network. That may be okay with kexec, but we have no real way of knowing if kexec "worked" short of inventing a new "I'm still here" or "this is expected to fail" connectivity check with a super short timeout. | 13:43 |
| TheJulia | Overall, wouldn't be a bad idea to make it globally configurable as well, I'm sure some operators who are super security concious will reject it's use because they will want to ensure the machine powers off before handing it over to the user. | 13:43 |
| TheJulia | lroussarie[m]: fwiw, I think the carriage returns in your messages are what caused matrix to sending links | 13:46 |
| TheJulia | Well, now adays, its just "new lines" | 13:46 |
| opendevreview | Verification of a change to openstack/ironic master failed: json-rpc: Clarify upgrade impact for ksa 5.17.0 https://review.opendev.org/c/openstack/ironic/+/1002854 | 13:48 |
| opendevreview | Verification of a change to openstack/ironic bugfix/38.0 failed: Adopt port physical network for empty portgroups https://review.opendev.org/c/openstack/ironic/+/1002835 | 13:51 |
| lroussarie[m] | TheJulia: ok,thank you for your feedback on kexec. I saw your started some implementation in 2019-2020 but it seems like the work was stopped after that. Is there any special reason for that ? | 13:53 |
| TheJulia | Lack of interest and lack of time for the most part | 13:55 |
| TheJulia | 2019->2020 was also a weird time. | 13:55 |
| TheJulia | And I'm not sure anyone remembers 2020 after... April or May. | 13:56 |
| TheJulia | And suddenly it was 2022. | 13:56 |
| TheJulia | oh, I remember why I backed away from it! You loose secure boot when you kexec | 13:59 |
| TheJulia | and jurisdictions starting to implement manditory secure boot for some workload types meant it would be self-excluding | 13:59 |
| opendevreview | Merged openstack/ironic bugfix/37.0: Adopt port physical network for empty portgroups https://review.opendev.org/c/openstack/ironic/+/1002836 | 14:06 |
| lroussarie[m] | TheJulia: yes, 2020 was not so funny ! | 14:21 |
| lroussarie[m] | concerning secure boot, kexec now supports it | 14:21 |
| opendevreview | Merged openstack/ironic bugfix/38.0: Make AutoDetectDeploy act more consistently https://review.opendev.org/c/openstack/ironic/+/1002838 | 14:26 |
| TheJulia | lroussarie[m]: Interesting! Then seems do-able then | 14:28 |
| TheJulia | Did anyone intend to be the bug deputy for this week? | 14:28 |
| TheJulia | Well, this past week. Looks like we failed to name someone last week during the meeting | 14:28 |
| TheJulia | iurygregory: https://bugs.launchpad.net/sushy/+bug/2164964 <- why are we loading the message registries all the time?! | 14:30 |
| TheJulia | at least for getting the tasks, | 14:30 |
| TheJulia | That seems like the risk of the registry changing (afaik, not supposed to really happen, but we *have* seen it, but also seems less important overall to just getting a task state | 14:31 |
| iurygregory | TheJulia, seems like it was done when we implemented TaskMonitor | 14:35 |
| iurygregory | maybe in the standard it mentions about registries? | 14:35 |
| iurygregory | we probably forgot to move to lazy_registries when we did for other resources... | 14:35 |
| TheJulia | you can get errors/messages matching the registry, I guess using cache is fine unless we can't match it then we should retry | 14:36 |
| TheJulia | (which I would hope is the behavior already) | 14:36 |
| cardoe | I was trying to understand it myself as well and I couldn't | 14:37 |
| TheJulia | The last time I looked deep at registry stuff was... 2022 timeframe | 14:38 |
| iurygregory | I only noticed we didn't change to lazy because people were complaining about some extra time in firmware updates ... | 14:39 |
| opendevreview | Verification of a change to openstack/ironic stable/2026.1 failed: Adopt port physical network for empty portgroups https://review.opendev.org/c/openstack/ironic/+/1002837 | 14:53 |
| TheJulia | #startmeeting ironic | 15:01 |
| opendevmeet | Meeting started Mon Aug 31 15:01:07 2026 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:01 |
| TheJulia | o/ | 15:01 |
| * TheJulia distributes coffeeeeeeee | 15:01 | |
| iurygregory | o/ | 15:01 |
| estherd[m] | o/ | 15:01 |
| * TheJulia puts out some cans of diet jolt cola | 15:01 | |
| dtantsur | o/ | 15:01 |
| TheJulia | (Yes, someone makes Jolt cola again in north america) | 15:02 |
| * iurygregory has no idea what is jolt cola lol | 15:03 | |
| TheJulia | iurygregory: Go watch the movie "Hackers" from 1995 | 15:03 |
| TheJulia | This will answer your question ;) | 15:03 |
| iurygregory | ohhh I've watched, but it's been a long long time ... | 15:03 |
| TheJulia | Anyhow! Looks like we're barely at Quorum today | 15:03 |
| TheJulia | If memory serves, a couple folks are out today/this week | 15:04 |
| TheJulia | I guess we'll just run down our agenda since it seems like it was a quiet week last week! | 15:04 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_August_31_2026 | 15:04 |
| cid | o/ | 15:04 |
| TheJulia | #topic Announcements/Reminders | 15:04 |
| TheJulia | Please remember the standing reminder to review ironic-week-prio items and tag items which you feel are ready for review. | 15:05 |
| TheJulia | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:05 |
| TheJulia | We are at week R-4 of the 2026.2 Hibiscus Release Schedule. | 15:05 |
| TheJulia | #link https://releases.openstack.org/hibiscus/schedule.html | 15:05 |
| TheJulia | This means cycle highlights are due this week. | 15:05 |
| iurygregory | I will take a look at it | 15:06 |
| TheJulia | One last item to make note of, I expect I'll be unavailable on Friday, and I'll be in China next week. | 15:06 |
| iurygregory | and ask folks to review | 15:06 |
| TheJulia | Does anyone else have anything they wish to announce or remind folks of? | 15:06 |
| iurygregory | I will be out next monday Sep 7, public holiday for me | 15:07 |
| clif | same here, labor day in US | 15:07 |
| TheJulia | Shall we cancel our meeting for next week? | 15:07 |
| TheJulia | I'll be in china, so... *shrug* | 15:08 |
| iurygregory | I would say it makes sense | 15:08 |
| TheJulia | If nobody objects in the next little bit, I'll send an email out shortly. | 15:08 |
| TheJulia | #agreed Sept 7th meeting is cancelled - Julia to send the email. | 15:09 |
| TheJulia | Onward! | 15:09 |
| TheJulia | #topic Working Group Updates | 15:09 |
| TheJulia | iurygregory: anything for asyncio this week? | 15:09 |
| iurygregory | no updates, still looking at code | 15:09 |
| TheJulia | ok | 15:09 |
| TheJulia | For the security core security team, I don't believe there are any updates for the past week. | 15:10 |
| TheJulia | err | 15:10 |
| TheJulia | ironic core security team | 15:10 |
| TheJulia | Clearly I need a jolt cola! | 15:10 |
| TheJulia | Moving onward! | 15:10 |
| TheJulia | I put one discussion topic in, asking who would do the highlights. Iury your volunteering, correct? | 15:10 |
| TheJulia | #topic Discussion Topics | 15:11 |
| iurygregory | yeah | 15:11 |
| TheJulia | #agreed iurygregory to look at cycle highlights! | 15:11 |
| TheJulia | Thanks Iury! | 15:11 |
| TheJulia | #topic Bug Deputy Updates | 15:11 |
| cid | I checked this morning and just before the meeting, there were 2 new bugs this week | 15:12 |
| cid | #link https://bugs.launchpad.net/sushy/+bug/2164964 | 15:12 |
| cid | use lazy registries for get_task_monitor | 15:12 |
| cid | #link https://bugs.launchpad.net/ironic/+bug/2164916 | 15:12 |
| cid | validate_driver_interfaces can issue a synchronous HEAD request to image_source | 15:12 |
| cid | It's not clear to me if this one ^^ just become public or was filed today | 15:12 |
| cid | *not today. on the same 24th | 15:13 |
| TheJulia | Yeah, it got semi-discussed during the last meeting. I forgot about it, I think I was going to whip up a "fix" | 15:13 |
| cid | I planned to as well. But if it was private before, it may already have a patch file somewhere | 15:14 |
| cid | But that's all the Bug deputy updates and I can take it next week unless someone else wants to | 15:14 |
| opendevreview | Verification of a change to openstack/ironic master failed: json-rpc: Rename TLS options https://review.opendev.org/c/openstack/ironic/+/1002853 | 15:15 |
| TheJulia | its a pretty mild issue | 15:15 |
| TheJulia | tbh | 15:15 |
| TheJulia | Anyway! would anyone like to volunteer to be the bug deputy in advance of our next meeting, I believe that would be september 11th | 15:16 |
| iurygregory | I can | 15:17 |
| TheJulia | #agreed iurygregory volunteers to do a bug deputy sweep before the meeting on the 11th! | 15:17 |
| TheJulia | Thanks Iury! | 15:17 |
| TheJulia | Onward! | 15:18 |
| TheJulia | #topic Open Discussion | 15:18 |
| iurygregory | Sep 7 till 11 right? | 15:18 |
| TheJulia | iurygregory: yes, I'll try to do a pass before my end of week just in case | 15:18 |
| iurygregory | ok | 15:18 |
| TheJulia | So, I forgot to mention the PTG. It is coming soon! | 15:19 |
| TheJulia | An etherpad has already been started, please add items you want to discuss or which you feel is important to be discussed during the PTG. | 15:19 |
| TheJulia | #link https://etherpad.opendev.org/p/ironic-ptg-2027.1 | 15:19 |
| TheJulia | Does anyone have anything else to discuss during open discussion? | 15:21 |
| TheJulia | #topic Who shall run our next meeting (Sept 11th)? | 15:24 |
| iurygregory | Sep11 is a Friday.. | 15:24 |
| TheJulia | err | 15:25 |
| iurygregory | wouldn't be Sep 14? | 15:25 |
| TheJulia | doh | 15:25 |
| TheJulia | #undo | 15:25 |
| opendevmeet | Removing item from minutes: #topic Who shall run our next meeting (Sept 11th)? | 15:25 |
| TheJulia | #topic Who shall run our next meeting (Sept 14th)? | 15:25 |
| iurygregory | I can | 15:25 |
| TheJulia | Your right, I clearly need to wake up! | 15:25 |
| TheJulia | #agreed Iury to run our next meeting on September 14th. | 15:25 |
| TheJulia | Thanks Iury! | 15:25 |
| TheJulia | Anything else before I close the meeting for this week? | 15:25 |
| TheJulia | Thanks folks! | 15:27 |
| TheJulia | #endmeeting | 15:27 |
| opendevmeet | Meeting ended Mon Aug 31 15:27:24 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-08-31-15.01.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-08-31-15.01.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-08-31-15.01.log.html | 15:27 |
| iurygregory | tks TheJulia | 15:27 |
| opendevreview | Merged openstack/ironic bugfix/38.0: Adopt port physical network for empty portgroups https://review.opendev.org/c/openstack/ironic/+/1002835 | 15:30 |
| opendevreview | Verification of a change to openstack/ironic master failed: json-rpc: Rename TLS options https://review.opendev.org/c/openstack/ironic/+/1002853 | 15:52 |
| cardoe | Do we need to make an Ironic 35.0.2 tag? | 16:28 |
| TheJulia | cardoe: huh? | 16:31 |
| cardoe | Looks like yes. And there's a patch about to land so I'll wait. | 16:31 |
| cardoe | I'm reading the security advisories and we say versions >=35.0.2,<36 are unaffected but there's no such 35.0.2 | 16:31 |
| cardoe | As a result my usage of stable/2026.1 cannot get GitHub Security whatever to be quiet. | 16:32 |
| ildikov | iurygregory: Hi, I'm working with fungi on the improving contributor experience effort. I wasn't able to attend the meeting where he shared our analysis after Gazpacho, but I was able to caught up on the logs.I remember you mentioned that you're interested in exploring ways to make it more visible where the Ironic team tracks priorities. | 16:38 |
| TheJulia | folks, can we get a review on https://review.opendev.org/c/openstack/ironic/+/1002940 this morning, running tests on master branch screams deprecation errors this should fix | 16:38 |
| ildikov | iurygregory: Following up on that I'm pinging you to point you to a patch to add review templates to the Project Team Guide: https://review.opendev.org/c/openstack/project-team-guide/+/1002796 | 16:38 |
| iurygregory | ildikov, hey o/ | 16:39 |
| iurygregory | ack will take look | 16:39 |
| ildikov | iurygregory: The last one on the page is intended as a first response on new changes, could even be automated with a Gerrit hook. And it has a specific section to call out where priorities are tracked. | 16:40 |
| ildikov | iurygregory: awesome, thank you. I'm curious what you think about it! | 16:41 |
| iurygregory | finishing a few things, will read it later today o/ | 16:41 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add short timeout for image URL validation https://review.opendev.org/c/openstack/ironic/+/1003086 | 16:57 |
| TheJulia | cid: ^^ | 17:00 |
| cid | \o/ | 17:01 |
| TheJulia | okay, https://review.opendev.org/c/openstack/ironic/+/1002837 failing seems... odd, when some of the test jobs actually succeeded :\ | 18:08 |
| opendevreview | Merged openstack/ironic master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/ironic/+/1002940 | 18:37 |
| opendevreview | Timon Schnell proposed openstack/ironic-python-agent master: Erase frozen SATA SSDs by discarding them https://review.opendev.org/c/openstack/ironic-python-agent/+/1001323 | 18:48 |
| opendevreview | Merged openstack/ironic stable/2026.1: Adopt port physical network for empty portgroups https://review.opendev.org/c/openstack/ironic/+/1002837 | 19:27 |
| opendevreview | Merged openstack/ironic master: json-rpc: Rename TLS options https://review.opendev.org/c/openstack/ironic/+/1002853 | 21:25 |
| iurygregory | TheJulia, regarding the registries in https://review.opendev.org/c/openstack/sushy/+/1002222 if I understood what you mentioned earlier, you also want https://github.com/openstack/sushy/blob/master/sushy/main.py#L658 to have the cache decorator (utils.cache_it) ? | 22:01 |
| TheJulia | ... I think we likely should, because we shouldn't be trying to get all the registries, and AFAIK they really shouldn't be changing drastically | 22:07 |
| iurygregory | ack o/ going to update the patch | 22:13 |
| TheJulia | The only case where it could bite us, I guess... is post-upgrade of the bmc bios firmware, but even then they are not really supposed to "change" if I'm recalling correctly | 22:20 |
| iurygregory | ack | 22:40 |
| iurygregory | but post-upgrade we will request a new client session, so we should be fine for most of the cases I think | 22:41 |
| TheJulia | In theory, yeah | 22:58 |
| opendevreview | Merged openstack/ironic master: Add short timeout for image URL validation https://review.opendev.org/c/openstack/ironic/+/1003086 | 23:09 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy master: Use lazy registries for TaskMonitor https://review.opendev.org/c/openstack/sushy/+/1002222 | 23:19 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!