Tuesday, 2025-04-15

*** gmann is now known as gmann_pto01:16
AmarachiOrdor[m]Good Morning Ironic!06:37
*** jroll05 is now known as jroll007:09
rpittaugood morning ironic! o/07:14
abongaleGood morning ironic!07:30
arne_wiebalckGood morning, rpittau abongale and Ironic!07:37
rpittauhey arne_wiebalck :)07:37
queensly[m]Good morning08:19
Ayo[m]Good morning ironic09:22
freemanboss[m]Good morning everyone 09:31
freemanboss[m]I up we are already rounding up our final application?09:31
freemanboss[m]Best of luck to each of us09:31
opendevreviewMerged openstack/networking-baremetal master: fix awkward logic that the linter didn't like  https://review.opendev.org/c/openstack/networking-baremetal/+/94581909:46
opendevreviewMerged openstack/networking-baremetal master: fix sphinx-lint issues  https://review.opendev.org/c/openstack/networking-baremetal/+/94582009:46
queensly[m]Freeman Boss: Yes.09:52
opendevreviewMerged openstack/networking-baremetal master: add pre-commit and adjust tox to utilize it  https://review.opendev.org/c/openstack/networking-baremetal/+/94582110:54
opendevreviewAdeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation  https://review.opendev.org/c/openstack/bifrost/+/94660310:56
opendevreviewKaifeng Wang proposed openstack/ironic master: Add several NV GPU types  https://review.opendev.org/c/openstack/ironic/+/94721311:47
TheJuliagood morning13:15
TheJuliacardoe: any chance we can get a few words on the summary in the etherpad w/r/t notificaitons/metrics discussion from last Tuesday sometime in the next couple of hours ?13:19
rpittauTheJulia: thanks for the summary, I'm finishing reading it now :)13:23
TheJuliastill needs a few more words in one spot.13:24
rpittauyeah but besides that I really like it :)13:25
TheJuliacool cool13:25
cardoeYes14:01
cardoeTheJulia: any specific spot?14:02
cardoeOh I see it now. Line 21814:02
cardoelet me know if that's good14:16
TheJuliacardoe: https://etherpad.opendev.org/p/ironic-ptg-april-2025#L77214:32
cardoeah okay I'll write up there.14:34
TheJuliaThanks!14:38
TheJuliarpittau: minimal links, do you feel anything else needs a reference or pointer?14:59
JayFTheJulia: suggestion: lets revise and land the work items so you can link form the email15:03
JayFrpittau: ^15:03
TheJuliaI'd like to get the summary email out today if possible15:03
JayFI was suggesting a timeline of now()ish15:04
JayFliterally was opening it now to fix the one objection (tinyipa replacement is unassigned and should be removed)15:04
TheJuliaoh, okay15:04
JayFwe can always revise it15:04
JayFbut having the URL live is useful15:04
opendevreviewJay Faulkner proposed openstack/ironic-specs master: 2025.2 Work Items from PTG  https://review.opendev.org/c/openstack/ironic-specs/+/94692115:05
opendevreviewJay Faulkner proposed openstack/ironic-specs master: 2025.2 Work Items from PTG  https://review.opendev.org/c/openstack/ironic-specs/+/94692115:09
JayF^^ should be mergeable, if there's feedback ping me and I can fix asap15:09
TheJuliaJayF: two items of feedback, lmk15:26
TheJuliato stress, it seems like without firm commitments, they may continue to linger which sends the opposite of the intended message15:27
JayFI almost wanna add a "incomplete and dropped" section15:28
JayFbut the marketing half of my brain says don't give it the space15:29
TheJuliamaybe a "notable, but still determining priority/consensus"15:29
TheJuliain some shape/form15:29
TheJuliayeah15:29
TheJuliaI can see that as well15:29
JayFin the intro, I'll mention some items dropped in priority15:29
TheJuliasounds good to me15:29
JayFwdyt https://usercontent.irccloud-cdn.com/file/Ujq8gVcm/image.png15:31
TheJuliaperfect!15:31
JayFI'm going to reword the last sentence15:31
JayFto say committed tim15:31
TheJuliak15:31
JayFor something like that15:31
TheJuliaregarding in-band disk crypto15:31
TheJuliaI think I exposed the knob on bootc to pass through the "do it" flag15:32
TheJuliaso it would really be "agent deploy"15:32
TheJuliabut, overall, its still accurate in the average path15:32
JayFI'm (well, if travel gets approved) thinking of things for the summit15:33
JayF"Silly Ironic Tricks" is an idea I've been tossing around15:33
TheJuliacool cool15:33
JayF"hah, in-band FDE outta nowhere"15:33
JayFis absolutely one of those things :D even if via bootc15:33
opendevreviewJay Faulkner proposed openstack/ironic-specs master: 2025.2 Work Items from PTG  https://review.opendev.org/c/openstack/ironic-specs/+/94692115:35
JayFfeedback reflected and note about removals added ^15:35
TheJuliaAdded a note to the review for now, if that somehow merges in the next hour or two then I'll swap it out15:39
JayFI'm honestly tempted to suggest a quick land as a single core as it's kinda docs15:40
JayFbut it's also kinda not docs15:40
JayFbut I have a strong preference for "something OK merged is better than nothhing"15:40
TheJuliaI'm good with another review can swing past real quick, but I wonder if we're a bit short handed this week on reviewers because people may have smartly taken pto15:41
JayFcid: ^ if you're around can you look at the work items change?15:41
JayFcardoe: ^ similarly15:41
JayFI've seen cardo-e this morning and I think c-d is around :D 15:41
TheJuliayeah15:42
* TheJulia context switches into code review for a little while15:43
rpittauTheJulia: had another look at the summary email and it looks good15:46
rpittauJayF: even if the work items patch does not merge, we can still link it to the mail, I believe we should give more time to people to review it15:46
JayFI gotta be honest, I hate the pattern of linking to reviews for something that's gonna be a permanent artifact15:47
JayFIt's just another thing for people to figure out, because it's not like it's an easy link from the review to the rendered version15:47
JayFbut if we wanna give people more time, we can... I have little confidence it will get significant additional review attention, based on the past cycles I've authored them15:48
cidJayF, I will ensure to do that  15:49
* cid Right about now15:50
TheJuliaIt is also kind of awful to wait for weeks to reach that front end consensus15:51
TheJuliaI'd rather just merge and send the email than wait for more reviews. folks can always propose changes, but if they are really engaged the would be in here and raise it before we send the email15:52
JayFyou said it better than I did :)15:52
TheJuliaThanks, I guess ;)16:14
opendevreviewSatoshi Shirosaka proposed openstack/ironic master: Add shared image support  https://review.opendev.org/c/openstack/ironic/+/94711516:40
cardoeJayF: which work items?16:59
JayFhttps://review.opendev.org/c/openstack/ironic-specs/+/94692116:59
cardoeoh the spec16:59
TheJuliaAI cat with 6 legs wanting networking gear for the humans https://usercontent.irccloud-cdn.com/file/imSBKoHx/halpnetworkingkitten.png17:07
opendevreviewMerged openstack/ironic master: Improve is_image_available  https://review.opendev.org/c/openstack/ironic/+/94657517:22
opendevreviewJay Faulkner proposed openstack/ironic master: Release notes title to "unreleased" for in-progress  https://review.opendev.org/c/openstack/ironic/+/94693317:36
TheJuliarebased to re-root since no tests ran17:37
cardoeWhat's our gerrit syntax for partial bug fix?18:09
cardoejust Partial-Bug: #XXX ?18:09
JayFaye18:09
opendevreviewSatoshi Shirosaka proposed openstack/ironic master: Add shared image support  https://review.opendev.org/c/openstack/ironic/+/94711518:21
opendevreviewJulia Kreger proposed openstack/ironic master: WIP provide host_id to neutron early on  https://review.opendev.org/c/openstack/ironic/+/94637818:30
opendevreviewSatoshi Shirosaka proposed openstack/ironic stable/2025.1: Improve is_image_available  https://review.opendev.org/c/openstack/ironic/+/94727618:46
opendevreviewJulia Kreger proposed openstack/ironic master: WIP provide host_id to neutron early on  https://review.opendev.org/c/openstack/ironic/+/94637818:58
TheJulialets see if we can make metalsmith happy :)18:58
opendevreviewMerged openstack/ironic-tempest-plugin master: CI: Increment jobs for 2025.1 release  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/94659319:19
opendevreviewJulia Kreger proposed openstack/ironic master: WIP provide host_id to neutron early on  https://review.opendev.org/c/openstack/ironic/+/94637819:49
opendevreviewJulia Kreger proposed openstack/ironic master: provide host_id to neutron early on  https://review.opendev.org/c/openstack/ironic/+/94637821:02
TheJuliaOkay, I think that sort of addresses the issues there and hopefully will now pass CI in general21:11
TheJulia(along with metalsmith)21:11
opendevreviewMerged openstack/ironic-specs master: 2025.2 Work Items from PTG  https://review.opendev.org/c/openstack/ironic-specs/+/94692121:26

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