Monday, 2026-07-20

opendevreviewSteve Baker proposed openstack/ironic master: Add supports_deploy to AnsibleDeploy for autodetect  https://review.opendev.org/c/openstack/ironic/+/99790902:35
opendevreviewSteve Baker proposed openstack/ironic master: Switch ironic-standalone-redfish to autodetect deploy  https://review.opendev.org/c/openstack/ironic/+/99791002:35
opendevreviewSteve Baker proposed openstack/ironic master: Fix ironic-standalone-redfish POST_FAILURE  https://review.opendev.org/c/openstack/ironic/+/99792102:35
stevebaker[m]Hey that POST_FAILURE fix ^^ should get some reviews02:39
rm_work[m]cardoe: ah sorry you caught me after I left, I'll link you some stuff shortly02:43
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Use cirros 0.6.3 as default cirros version  https://review.opendev.org/c/openstack/bifrost/+/93217508:26
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Use cirros 0.6.3 as default cirros version  https://review.opendev.org/c/openstack/bifrost/+/93217508:27
rpittaumoving to a more recent cirros finally  works! \o/  reviews welcome ^ thanks!08:28
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC  https://review.opendev.org/c/openstack/ironic/+/99736608:54
opendevreviewMerged openstack/virtualbmc master: Add note about lower constraints  https://review.opendev.org/c/openstack/virtualbmc/+/99756909:04
opendevreviewMerged openstack/networking-generic-switch stable/2025.1: Sanitise secret config field  https://review.opendev.org/c/openstack/networking-generic-switch/+/99698409:48
dtantsurTheJulia: I'll follow-up publicly to something we discuss in private. We need to finish rethinking our driver policy given the Redfish-dominant world10:40
dtantsurI.e. finish the cardoe's quirk work for good, drop the idrac driver, and admit the the core of Ironic is now Redfish with vendor quirks.10:41
dtantsurhttps://review.opendev.org/c/openstack/ironic/+/997366 is an example where we essentially have to cross the driver border again11:10
rpittauoh yeah, that case probably needs deeper discussion11:12
tkajinamo/ jfyi we need https://review.opendev.org/c/openstack/networking-baremetal/+/997893 to fix the broken tests there.11:50
tkajinam(actually not only tests but also actual functionality is broken due to recent change in neutron11:54
dtantsurthx, approved11:54
opendevreviewMerged openstack/ironic master: Fix ironic-standalone-redfish POST_FAILURE  https://review.opendev.org/c/openstack/ironic/+/99792112:02
tkajinamdtantsur, thanks !12:48
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC  https://review.opendev.org/c/openstack/ironic/+/99736613:07
TheJuliadtantsur: I don't really disagree, but the fine details may mismatch13:15
dtantsurWhich is why an agreement is needed :)13:16
* dtantsur senses another midcycle coming13:16
rpittaua mid-midcycle? a quartercycle?13:17
TheJuliaIt could likely just end up in our normal ptg planning13:19
* TheJulia really ponders going back to sleep13:20
dtantsurTake a good rest for sure13:21
dtantsurDo we have a PTG etherpad already?13:22
TheJulianot yet13:22
TheJuliaI have been thinking a blog post would actually be appropriate on the subject13:26
TheJuliaThat way we're setting tone/context and it is something discoverable13:26
*** iurygregory_ is now known as iurygregory13:36
iurygregorygood morning ironic13:37
*** noonedeadpunk is now known as noonedeadpunk_13:51
*** noonedeadpunk_ is now known as noonedeadpunk13:51
opendevreviewMerged openstack/networking-baremetal master: Replace BaseEvent  https://review.opendev.org/c/openstack/networking-baremetal/+/99789313:52
dtantsurI can wait for the meeting, but here's a question: if I wanted to add some generic scaling guidance to the docs, where would it go?14:04
dtantsurhttps://docs.openstack.org/ironic/latest/admin/tuning.html seems quite hardcore, not quite where you *start*14:04
fricklerdtantsur: https://docs.openstack.org/ironic/latest/install/refarch/common.html#ha-and-scalability ?14:06
dtantsurah, nice find! (please don't tell me I wrote that....:D)14:09
opendevreviewDmitry Tantsur proposed openstack/ironic master: Refresh the scaling guide in the reference architecture  https://review.opendev.org/c/openstack/ironic/+/99799814:37
dtantsurPretty opinionated, thoughts welcome ^^^14:37
iurygregory#startmeeting ironic15:00
opendevmeetMeeting started Mon Jul 20 15:00:12 2026 UTC and is due to finish in 60 minutes.  The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
dtantsuro/15:00
iurygregoryHello everyone, welcome to our weekly meeting o/15:00
TheJuliagood morning15:00
rpittauo/15:00
iurygregorylets wait to see if we have quorum =)15:01
estherd[m]o/15:01
opendevreviewMerged openstack/ironic-python-agent bugfix/11.4: Fix: Remove legacy agent_token fallback code  https://review.opendev.org/c/openstack/ironic-python-agent/+/99762515:02
iurygregoryok, we have 5 people15:03
iurygregorylet's give coffee for the others :D 15:03
TheJuliaMore coffee is always good15:03
rpittauice coffeee pretty please :D15:04
iurygregory++15:04
Mahnooro/15:04
iurygregoryour agenda can be found in our wiki page15:04
iurygregory#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_July_20.2C_202615:04
iurygregory#topic Announcements / Reminders15:04
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC  https://review.opendev.org/c/openstack/ironic/+/99736615:04
iurygregory#info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio15:04
cido/15:05
iurygregory#link https://tinyurl.com/ironic-weekly-prio-dash15:05
iurygregory#info dtantsur want us to remind that the BMO job has stabilized over the past week, please start paying attention15:05
dtantsur#link https://zuul.opendev.org/t/openstack/builds?job_name=metal3-baremetal-operator-functional&project=openstack%2Fironic&branch=master&skip=0 BMO job history15:05
iurygregory#link https://zuul.opendev.org/t/openstack/builds?job_name=metal3-baremetal-operator-functional&project=openstack%2Fironic&branch=master&skip=0 15:06
dtantsuroops, sorry :)15:06
iurygregorysorry I was a bit slow =)15:06
dtantsurjust one of us need to issue the undo command, let's not do it both :D15:06
rpittaulol15:06
* dtantsur defers to iurygregory15:06
dtantsurWhile we sort our friendly race condition, yeah, I think I finally managed to sort most or all of its issues15:07
TheJuliayeah, need to be the chair to undo15:07
kubajjo/15:07
iurygregoryjust remind me how it works lol15:07
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC  https://review.opendev.org/c/openstack/ironic/+/99736615:07
iurygregory"/ undo"?15:07
TheJuliayou should have had more coffee first ;)15:07
TheJulia#undo15:07
iurygregory#undo15:07
opendevmeetRemoving item from minutes: #link https://zuul.opendev.org/t/openstack/builds?job_name=metal3-baremetal-operator-functional&project=openstack%2Fironic&branch=master&skip=015:07
iurygregoryty15:07
dtantsurThanks!15:07
rpittau\o/15:07
dtantsurSo, if you see the BMO job red, don't assume it's its normal state15:07
dtantsurit does fail downloading images from the artifact storage, but it seems to be quite rare15:07
iurygregory#info we are at the R-10 week of hibiscus schedule, nothing to highlight 15:08
iurygregoryAnyone has something to add to Announcements / Reminders?15:08
rpittaunext week should be bugfix branch week again :)15:09
rpittauprobably doing some more releases too15:09
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC  https://review.opendev.org/c/openstack/ironic/+/99736615:09
iurygregory#info next week we should do bugfix branches15:10
iurygregorytks rpittau 15:10
iurygregory#topic Working Group Updates 15:10
iurygregory#info AsyncIO - still need reviews on the spec15:11
iurygregoryAny updates from the Security Coresec team?15:11
TheJuliaNothing to highlight this week.15:11
iurygregoryack, ty!15:12
iurygregory#topic Discussion topics 15:12
iurygregoryI see one from fungi, but there was some discussions in other meetings, not sure if is still required for this one...15:13
TheJuliaSo we basically discussed that last week but we coulnd't access the wiki that day15:13
iurygregoryack, so moving15:13
TheJuliaYeah, not sure there is anything actionable there this week unless fungi suddenly appears in a poof of magical smoke15:13
iurygregory#topic Bug Deputy Updates15:14
TheJuliaQuestion for the Bug Deputy, is that single bug only the ironic project?15:14
iurygregoryI was using the ironic bug dashboard to track ...15:14
iurygregoryit only showed 1 new bug (based on all projects)15:14
iurygregoryand 3 new  RFE..15:15
TheJuliaahh, okay15:15
TheJuliacarry on then15:15
iurygregoryI was a bit puzzled that the automation didn't change things, because the patch was merged15:15
TheJuliaI didn't see the RFEs15:15
TheJuliawhich automation?15:15
fungiiurygregory: sorry, i didn't add one for this week. someone likely copy-pasted my topic from last week into this week's agenda again15:16
iurygregorylaunchpad has an automation that knows when we pushed a change to fix a bug15:16
iurygregoryand normally it changes some fields no?15:16
TheJuliaDepends on the tag and overall release process, your going to need to highlight an specific case15:16
iurygregoryoh ok, so maybe because it was a security issue things were a bit different..15:17
iurygregorybut now the bug shows the right information, and I've assigned to you TheJulia (since you had the patch) :D 15:17
iurygregoryfungi, I just didn't removed from the agenda, I wasn't sure if we need more discussion o/15:18
TheJuliaoh, security related bugs ideally should be labeled as "Related-Bug"15:18
TheJuliabecause if an announcement goes out, the announcement is the actual closer.15:19
iurygregorythank you for the clarification TheJulia ++15:19
iurygregory#topic Who is the next bug deputy? 15:20
MahnoorI can do it15:20
TheJuliaThank you Mahnoor 15:20
iurygregory#info Mahnoor is the bug deputy this week15:20
iurygregorytyvm!15:20
Mahnoor^-^15:21
iurygregory#topic RFE Review15:21
fungiannouncements close ossa bugtasks, patches close ironic bugtasks. launchpad allows a single bug to affect multiple projects15:21
iurygregoryTheJulia, there was 3 new RFEs you created, wondering if you want to discuss them or not15:21
dtantsurIt looks like specs are in progress for them, so we can mark them "needs-spec" and move on?15:22
TheJuliaSo the first two RFE's I've largely discussed in the past. One is RDMA (remote direct memory access) as it relates to networking changes/capabilities to enable that model which is used heavily for moving data between GPUs on systems.15:22
TheJuliaSpecs have been proposed for all three15:22
TheJuliaThe second one, nvme-over-tcp support, is just filed to the backlog because I don't believe the hardware ecosystem is "ready" yet.15:23
fungithe vmt recommends patches which fix bugs related to a security advisory use the closes-bug commit message trailer, not related-bug. that will also affect release automation which leaves comments on the bug reports inticating the tags their respective backports wind up in15:23
iurygregoryok, I will update them and add a link to the spec15:23
TheJuliaThe third, is more of a longer term thought process, enable us to begin moving away from paramiko.15:23
iurygregorytyvm TheJulia 15:24
iurygregorymoving on15:25
TheJuliaI think it would be super helpful if folks can put some eyes on the RDMA and connection method specs for networking-generic-switch over the next week, thanks!15:25
iurygregory#topic Open discussion15:25
iurygregoryI've added a patch just get some feedback on the change (we finally got some information about the hardware model..)15:26
iurygregory#link https://review.opendev.org/c/openstack/ironic/+/986597/comment/db358bb9_bf3f0c37/15:26
iurygregoryThis is an attempt to fix a problem in new Lenovo hardware (Model SR645 V3)15:27
TheJulia... What is it with the SR6XXX series of gear15:28
iurygregorypreviously we had a workaround for lenovo because of the SR650 model, so I'm not sure if they use same firmware, or if new firmware fixed the issue we had on SR65015:28
TheJuliaYeah, it just seems like we end up angering that hardware in general fairly often15:30
iurygregoryI'm wondering if we should do this for vendor based or just for the model we saw the problem.. we don't have lenovo gear to test other cases =( 15:30
TheJulia... Generally I'd suggest vendor since we tend to see differences permutate across the models15:32
TheJuliabut if its in-line with past efforts cool, if not, that is worrisome. In the past it was just a subtle issue or ordering that created a situation where more than one record was added so we had to very carefully walk through the flows15:33
TheJuliai.e. clean up records before adding the singular nvram record, because adding two causes the system to blue screen on reboot and refresh to last known good state.15:34
iurygregoryyeah, this we fixed in https://github.com/openstack/ironic/blame/e839c4318873a050564e3b239a3dd84881d10e5e/ironic/drivers/modules/agent.py#L23715:34
iurygregorynow the problem seems to be on the boot flow <facepalm>15:35
TheJuliayeah, but to that point my concern is broadly did we create a situation where that is happening and then one step removed how do we verify. Honestly, I'm really wishing we had a third party lab we could watch15:35
TheJuliaAnyway, seems like heads need to get together on this15:37
iurygregorybased on what I saw in the downstream bug, I think is just because the hardware doesn't have/expose all redfish capabilities ...15:37
dtantsurIn the ideal case, we get Lenovo folks to tell us the exact thing we need to do for all their hardware..15:38
iurygregoryyeah ++15:38
iurygregorythanks for the discussion everyone!15:39
iurygregory#topic Who is chairing the next meeting? 15:39
opendevreviewDmitry Tantsur proposed openstack/ironic master: Refresh the scaling guide in the reference architecture  https://review.opendev.org/c/openstack/ironic/+/99799815:40
iurygregoryok, I will do it next week :D15:40
iurygregorytks everyone!15:41
iurygregory#endmeeting15:41
opendevmeetMeeting ended Mon Jul 20 15:41:22 2026 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:41
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-07-20-15.00.html15:41
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-07-20-15.00.txt15:41
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2026/ironic.2026-07-20-15.00.log.html15:41
fungiper earlier, if anybody has questions about the nitty-gritty details for the automation that comments on launchpad bugs when changes get pushed to gerrit, or get included in tagged releases, i'm happy to get into all of that (either here or elsewhere)15:41
fungiit's not really any different for security bugs, except that the automation can't "see" bugs that are still private and so can't update or comment on them, but that's fine since changes in public code review should never refer to bugs while they're still private15:41
fungiand once the bug is switched to public (or public security) type, gerrit integration for events from that point in work normally15:41
TheJuliadtantsur: +1M15:42
TheJuliaGoing back to some discussion last week, I wrote up a quick risc-v call to action of sorts for our blog: https://github.com/OpenStackweb/ironic-website/pull/9016:06
opendevreviewJulia Kreger proposed openstack/ironic bugfix/37.0: security: Fix poisoning of the allowed origin list  https://review.opendev.org/c/openstack/ironic/+/99801716:09
opendevreviewJulia Kreger proposed openstack/ironic stable/2026.1: security: Fix poisoning of the allowed origin list  https://review.opendev.org/c/openstack/ironic/+/99801816:09
opendevreviewJulia Kreger proposed openstack/ironic stable/2025.2: security: Fix poisoning of the allowed origin list  https://review.opendev.org/c/openstack/ironic/+/99801916:10
TheJuliaiurygregory: it was because I related-bug it all16:10
opendevreviewJulia Kreger proposed openstack/ironic bugfix/34.0: security: Fix poisoning of the allowed origin list  https://review.opendev.org/c/openstack/ironic/+/99802016:11
opendevreviewJulia Kreger proposed openstack/ironic bugfix/33.0: security: Fix poisoning of the allowed origin list  https://review.opendev.org/c/openstack/ironic/+/99802116:12
TheJuliaIs anyone else interested in a spec review jam tomorrow? I've got Dmitry, Riccardo, and Doug so far.16:14
opendevreviewMerged openstack/ironic master: Remove Python 3.10 support  https://review.opendev.org/c/openstack/ironic/+/99775616:20
opendevreviewMerged openstack/ironic master: Replace deprecated datetime.datetime.utcnow  https://review.opendev.org/c/openstack/ironic/+/99782916:20
TheJuliacardoe: sigh: https://bugs.launchpad.net/networking-baremetal/+bug/216134416:22
cardoeI think someone job made a patch for that?16:22
cardoehttps://review.opendev.org/c/openstack/networking-baremetal/+/99789316:23
TheJuliaoh yes16:24
TheJuliavery yes16:24
* TheJulia goes and does the recheck thing16:24
TheJuliatkajinam: thanks!16:27
cardoeTheJulia: there's a handful against networking-baremetal if you wanna +2/+W them... I've given my +216:31
TheJuliaCool cool, I was just going to shift gears to those16:31
fungiTheJulia: regarding your risc-v blog post, not sure if you saw but earlier today the tc approved formation of an official openstack risc-v sig now... https://governance.openstack.org/tc/reference/sigs/16:38
TheJuliaI saw16:39
TheJuliaThanks16:39
fungialso there was a thread started on the ml a few weeks ago: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/UK2YYNPXC3BGZM3IWHYM6XYNQXHPD57C/ (which got little engagement but could benefit from some follow-up)16:40
TheJuliacardoe: https://review.opendev.org/c/openstack/networking-baremetal/+/996726 if you have a minute16:44
cardoeah that was the one I wanted to discuss with hjensas 16:45
cardoeI think overall the change is good.16:45
TheJuliaYeah, I think it is likely okay16:45
cardoeBut it goes back to we should only do the teardown when it's a port we've bound16:45
TheJuliawell, the whole model is to ensure we have things begin to be pinned into place so physical ports actually work on the bridging. All that patch does is limit that scope so VM routed networks in distributed router cases (why?!) can still keep talking because I guess there is a  separate premise taking place and you could end up with a mismatch if the base substrate networking is not in place to support it16:47
TheJuliawhich is also sort of a "not our problem problem"16:48
TheJuliaunwind feeling separate, fwiw16:48
cardoeyeah it is separate. it just prompted me to want to talk about that17:10
opendevreviewMerged openstack/networking-generic-switch master: Add trunk subport for  NETCONF OpenConfig driver  https://review.opendev.org/c/openstack/networking-generic-switch/+/99145817:11
opendevreviewVerification of a change to openstack/networking-generic-switch master failed: Netconf: Document Neutron trunk port support  https://review.opendev.org/c/openstack/networking-generic-switch/+/99145917:11
hjensasI think 996726 look ok too. Regarding only teardown if we bound the port, seems reasonable as well. Can we filter on drivers listed in the binding_vif_details bound_drivers.<idx>=<driver_name> ?17:21
hjensass/<idx>/<binding level> ^^17:23
cardoeThat's what I'm gonna do with our internal driver.17:25
opendevreviewHarald Jensås proposed openstack/networking-generic-switch master: Netconf: Document Neutron trunk port support  https://review.opendev.org/c/openstack/networking-generic-switch/+/99145917:38
TheJuliacardoe: ahh, okay17:39
TheJuliaI think its fair to do on ours as well, fwiw17:39
opendevreviewMerged openstack/networking-baremetal master: Restrict range of the port option  https://review.opendev.org/c/openstack/networking-baremetal/+/99789117:43
opendevreviewMerged openstack/networking-baremetal master: Hide password option from logs  https://review.opendev.org/c/openstack/networking-baremetal/+/99789517:59
opendevreviewMerged openstack/networking-baremetal master: Drop unnecessary import/register error handling  https://review.opendev.org/c/openstack/networking-baremetal/+/99789218:18
opendevreviewMerged openstack/networking-baremetal master: router_ha_binding: Skip networks without external ports  https://review.opendev.org/c/openstack/networking-baremetal/+/99672618:33
opendevreviewMerged openstack/tenks master: Add openstack.cloud collection to requirements  https://review.opendev.org/c/openstack/tenks/+/99306020:46
*** srelf_ is now known as continuity20:50
opendevreviewHarald Jensås proposed openstack/networking-generic-switch master: Switch netconf transport from paramiko to libssh  https://review.opendev.org/c/openstack/networking-generic-switch/+/99806220:50
opendevreviewDoug Goldstein proposed openstack/ironic master: doc: provide definitions of hardware types, drivers and interfaces  https://review.opendev.org/c/openstack/ironic/+/99806320:56
cardoehjensas: something you're shooting for with libssh?20:59
cardoehjensas: Just cause libssh isn't the preferred or tested backend. paramiko still is. libssh is there for platforms that struggle with the paramiko depends. They've even said it's not fully compatible or as perfomant.21:00
hjensascardoe: there is a discussion on paramiko as a dep ref: https://review.opendev.org/c/openstack/ironic-specs/+/997653 - I initially just want to give libssh a test.21:10
cardoeah21:15
cardoewe had just looked at it as well and had concerns. I wasn't aware.21:15
opendevreviewMerged openstack/networking-baremetal master: use agent hostname for notification listener pool name  https://review.opendev.org/c/openstack/networking-baremetal/+/99700421:15
rm_work[m]so https://review.opendev.org/c/openstack/oslo.messaging/+/997783 is green (not for merging, just for demonstration) and proves its depends-on  https://review.opendev.org/c/openstack/ironic/+/997789 is working, which would be a good cheap fix IMO21:31
rm_work[m](yes the comments are AI generated but I left them because they seemed helpful but could be trimmed if we don't like the verbosity)21:32

Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!