Wednesday, 2026-10-14

opendevreviewDoug Goldstein proposed openstack/ironic master: Allow retrieving and using public runbooks  https://review.opendev.org/c/openstack/ironic/+/101031502:54
iurygregory_FYI, I will probably be a few minutes late for the PTG tomorrow o/ (max 30min, I have meeting I need to attend)03:39
opendevreviewAdam Harwell proposed openstack/networking-baremetal master: Keep reporting state after a failed pass  https://review.opendev.org/c/openstack/networking-baremetal/+/101033805:34
rm_workRan into that today ^^05:38
rm_workHad a DNS blip and then suddenly no more agents05:38
opendevreviewJacob Anders proposed openstack/ironic master: Add Dell LC job status helper for multiple JIDs  https://review.opendev.org/c/openstack/ironic/+/100565613:19
opendevreviewJacob Anders proposed openstack/ironic master: Add Redfish BootProgress gate helper  https://review.opendev.org/c/openstack/ironic/+/100565713:19
opendevreviewJacob Anders proposed openstack/ironic master: Verify firmware application after reboot before resuming  https://review.opendev.org/c/openstack/ironic/+/100565813:19
opendevreviewJacob Anders proposed openstack/ironic master: Validate BMC stability only after BMC firmware updates  https://review.opendev.org/c/openstack/ironic/+/100565913:19
cardoeI'll be coming over in just a few.13:30
cardoeTheJulia: JayF: ping me if I should come over. Just doing some networking with neutron13:45
opendevreviewJacob Anders proposed openstack/ironic master: Batch non-BMC Redfish firmware updates under one reboot  https://review.opendev.org/c/openstack/ironic/+/99972014:38
opendevreviewJacob Anders proposed openstack/ironic master: Document batched Redfish firmware updates  https://review.opendev.org/c/openstack/ironic/+/99972114:38
opendevreviewJacob Anders proposed openstack/ironic master: Keep Redfish firmware update state in one record with explicit phases  https://review.opendev.org/c/openstack/ironic/+/101045914:38
opendevreviewJed Preist proposed openstack/ironic master: Retry setting boot device when told to by BMC  https://review.opendev.org/c/openstack/ironic/+/100943515:03
opendevreview葉力維 proposed openstack/networking-generic-switch master: sonic: detect "already untagged member" as a config error  https://review.opendev.org/c/openstack/networking-generic-switch/+/101047115:59
opendevreview葉力維 proposed openstack/ironic master: ironic-networking: remove old VLAN before plugging a changed switchport  https://review.opendev.org/c/openstack/ironic/+/101047215:59
cidHow not to record an architecture session in meetpad -> https://drive.google.com/file/d/1EgVnTiib3dvut-HY9Bal3DplaQhuUP_F/view?usp=sharing17:02
cidDeleted at the end of today's PTG :) 17:02
cid*deleting17:02
TheJuliaheh17:10
TheJuliac'lest la vie17:11
JayFoh no! We'll have to do the two hours of swapping context and chatting about Ironic again! 17:11
JayF(/s)17:11
TheJulia... where was my brain17:17
TheJulia.... it's only wednesday.17:25
cardoeJayF: https://review.opendev.org/c/openstack/ironic/+/1010315 that's what I was talking about17:33
cardoeI also personally disagree with the still default policy.17:34
cardoeActually nvm.17:34
opendevreview葉力維 proposed openstack/ironic master: ironic-networking: remove old VLAN before plugging a changed switchport  https://review.opendev.org/c/openstack/ironic/+/101047217:53
opendevreviewMerged openstack/sushy master: Avoid clearing an empty HttpBootUri  https://review.opendev.org/c/openstack/sushy/+/100961218:00
opendevreviewcid proposed openstack/ironic master: Consistency in handling full-boot-request vendors  https://review.opendev.org/c/openstack/ironic/+/98016318:03
opendevreviewDoug Goldstein proposed openstack/ironic stable/2026.1: Only plug ports with a tenant VIF in configure_tenant_networks  https://review.opendev.org/c/openstack/ironic/+/100978318:07
opendevreview葉力維 proposed openstack/ironic master: ironic-networking: remove old VLAN before plugging a changed switchport  https://review.opendev.org/c/openstack/ironic/+/101047218:09
opendevreviewJulia Kreger proposed openstack/ironic-specs master: Extend ironic auth models  https://review.opendev.org/c/openstack/ironic-specs/+/101029518:27
opendevreviewcid proposed openstack/ironic master: Replace boot property literals with constants  https://review.opendev.org/c/openstack/ironic/+/98082918:29
cardoeclif: can we talk TBN shortly during tomorrow's PTG?18:56
cardoeThat's what I wanted on the Network Attachment Flow topic for today18:56
clifcardoe: sure18:58
cardoeSo essentially what I wanna do is talk about the perceived gaps to deleting the old flow and going TBN only.19:00
cardoeclif: https://etherpad.opendev.org/p/ironic-ptg-2027.1 at the very end (right now that's starting at line 944)19:27
clifcardoe: is TBN on the schedule for tomorrow?19:32
clifor should we save it for friday19:32
cardoeFriday is fine.19:45
cardoeFeel free to add any extra notes20:01
clif*shrug* either is fine, just need to carve out a time and friday looks freer20:03
TheJuliacardoe: w/r/t personas, you still doing the human personas on the wiki someplace, or should I sort of mentally hold off until we land a "what maps to us" in order to help translate/connect to srbac20:06
TheJuliahas anyone seen my brain?!?20:07
opendevreviewcid proposed openstack/ironic master: Replace boot property literals with constants  https://review.opendev.org/c/openstack/ironic/+/98082921:37
cardoeTheJulia: uhh human personas wiki was OpenStack wide... I haven't done Ironic ones there.. I was doing that in secure-rbac.rst21:54
TheJuliaOh, okay then21:54
TheJuliaThat works21:54
cardoeNothing pushed yet.21:54
TheJuliaOk21:58
TheJuliaI figured, it’s PTG week21:59
TheJuliaENOBRAIN21:59
TheJuliaPasta sauce > brain21:59
TheJuliaDinner! https://usercontent.irccloud-cdn.com/file/UA1Woc1o/1792015260.JPG22:01
TheJuliaI need some holiday cooking….22:05
cardoeTheJulia: oh btw you're wrong22:30
* cardoe walks away from the keyboard.22:30
TheJuliacardoe: that is evil ;)22:31
cardoeTheJulia: https://review.opendev.org/c/openstack/ironic/+/1009783 that change which is spurred from https://launchpad.net/bugs/216976122:34
cardoea portgroup has a standalone_ports_supported which TBN actually respected22:35
TheJuliaI'm not sure I understand22:35
cardoeYou had said it was broken behavior to attach a port individually that's part of a port group22:36
TheJuliaoh yeah, we shouldn't, but yeah22:36
cardoeBut the commit message and the API comments around that flag seem to be exactly for that22:36
TheJuliagah22:36
cardoeI won't disagree with you but we have an API flag22:36
cardoeAnd we default that to True22:36
TheJuliaugh, le sigh22:36
TheJulia:(22:36
TheJulia.... if my stand mixer was onboard, I'd make cookies which is just a testimonial to the state of my brain22:37
cardoeI'll update the bug22:37
TheJuliaokay, thanks22:38
TheJuliasorry for the noise22:38
cardoeSo this is actually good.22:38
cardoeIt's giving me more test case fodder for TBN replacing the hardcoded flow.22:39
cardoeBecause guess what does the plugging wrong? (well assuming you backport my patch)22:39
cardoethe hardcoded code flow does the wrong thing with that flag.22:39
cardoeI propose we change the default state of that flag to False going forward.22:40
cardoeI'd almost argue we can change the default state of that flag on create immediately because that's the current assumption of the code.22:41
JayFTBN is young enough that I'd rather take the hard turn and fix a burr in an edge case than play 2 years of deprecation games for something that you are likely the first human to interact with (this case in particular)22:53
cardoeLet's discuss on Friday22:57
cardoeI agree with you.23:01

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