*** gmann is now known as gmann_pto | 01:16 | |
AmarachiOrdor[m] | Good Morning Ironic! | 06:37 |
---|---|---|
*** jroll05 is now known as jroll0 | 07:09 | |
rpittau | good morning ironic! o/ | 07:14 |
abongale | Good morning ironic! | 07:30 |
arne_wiebalck | Good morning, rpittau abongale and Ironic! | 07:37 |
rpittau | hey arne_wiebalck :) | 07:37 |
queensly[m] | Good morning | 08:19 |
Ayo[m] | Good morning ironic | 09: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 us | 09:31 |
opendevreview | Merged openstack/networking-baremetal master: fix awkward logic that the linter didn't like https://review.opendev.org/c/openstack/networking-baremetal/+/945819 | 09:46 |
opendevreview | Merged openstack/networking-baremetal master: fix sphinx-lint issues https://review.opendev.org/c/openstack/networking-baremetal/+/945820 | 09:46 |
queensly[m] | Freeman Boss: Yes. | 09:52 |
opendevreview | Merged openstack/networking-baremetal master: add pre-commit and adjust tox to utilize it https://review.opendev.org/c/openstack/networking-baremetal/+/945821 | 10:54 |
opendevreview | Adeola Adesina proposed openstack/bifrost master: Add notes to provide more clarity for bifrost installation https://review.opendev.org/c/openstack/bifrost/+/946603 | 10:56 |
opendevreview | Kaifeng Wang proposed openstack/ironic master: Add several NV GPU types https://review.opendev.org/c/openstack/ironic/+/947213 | 11:47 |
TheJulia | good morning | 13:15 |
TheJulia | cardoe: 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 |
rpittau | TheJulia: thanks for the summary, I'm finishing reading it now :) | 13:23 |
TheJulia | still needs a few more words in one spot. | 13:24 |
rpittau | yeah but besides that I really like it :) | 13:25 |
TheJulia | cool cool | 13:25 |
cardoe | Yes | 14:01 |
cardoe | TheJulia: any specific spot? | 14:02 |
cardoe | Oh I see it now. Line 218 | 14:02 |
cardoe | let me know if that's good | 14:16 |
TheJulia | cardoe: https://etherpad.opendev.org/p/ironic-ptg-april-2025#L772 | 14:32 |
cardoe | ah okay I'll write up there. | 14:34 |
TheJulia | Thanks! | 14:38 |
TheJulia | rpittau: minimal links, do you feel anything else needs a reference or pointer? | 14:59 |
JayF | TheJulia: suggestion: lets revise and land the work items so you can link form the email | 15:03 |
JayF | rpittau: ^ | 15:03 |
TheJulia | I'd like to get the summary email out today if possible | 15:03 |
JayF | I was suggesting a timeline of now()ish | 15:04 |
JayF | literally was opening it now to fix the one objection (tinyipa replacement is unassigned and should be removed) | 15:04 |
TheJulia | oh, okay | 15:04 |
JayF | we can always revise it | 15:04 |
JayF | but having the URL live is useful | 15:04 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: 2025.2 Work Items from PTG https://review.opendev.org/c/openstack/ironic-specs/+/946921 | 15:05 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: 2025.2 Work Items from PTG https://review.opendev.org/c/openstack/ironic-specs/+/946921 | 15:09 |
JayF | ^^ should be mergeable, if there's feedback ping me and I can fix asap | 15:09 |
TheJulia | JayF: two items of feedback, lmk | 15:26 |
TheJulia | to stress, it seems like without firm commitments, they may continue to linger which sends the opposite of the intended message | 15:27 |
JayF | I almost wanna add a "incomplete and dropped" section | 15:28 |
JayF | but the marketing half of my brain says don't give it the space | 15:29 |
TheJulia | maybe a "notable, but still determining priority/consensus" | 15:29 |
TheJulia | in some shape/form | 15:29 |
TheJulia | yeah | 15:29 |
TheJulia | I can see that as well | 15:29 |
JayF | in the intro, I'll mention some items dropped in priority | 15:29 |
TheJulia | sounds good to me | 15:29 |
JayF | wdyt https://usercontent.irccloud-cdn.com/file/Ujq8gVcm/image.png | 15:31 |
TheJulia | perfect! | 15:31 |
JayF | I'm going to reword the last sentence | 15:31 |
JayF | to say committed tim | 15:31 |
TheJulia | k | 15:31 |
JayF | or something like that | 15:31 |
TheJulia | regarding in-band disk crypto | 15:31 |
TheJulia | I think I exposed the knob on bootc to pass through the "do it" flag | 15:32 |
TheJulia | so it would really be "agent deploy" | 15:32 |
TheJulia | but, overall, its still accurate in the average path | 15:32 |
JayF | I'm (well, if travel gets approved) thinking of things for the summit | 15:33 |
JayF | "Silly Ironic Tricks" is an idea I've been tossing around | 15:33 |
TheJulia | cool cool | 15:33 |
JayF | "hah, in-band FDE outta nowhere" | 15:33 |
JayF | is absolutely one of those things :D even if via bootc | 15:33 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: 2025.2 Work Items from PTG https://review.opendev.org/c/openstack/ironic-specs/+/946921 | 15:35 |
JayF | feedback reflected and note about removals added ^ | 15:35 |
TheJulia | Added a note to the review for now, if that somehow merges in the next hour or two then I'll swap it out | 15:39 |
JayF | I'm honestly tempted to suggest a quick land as a single core as it's kinda docs | 15:40 |
JayF | but it's also kinda not docs | 15:40 |
JayF | but I have a strong preference for "something OK merged is better than nothhing" | 15:40 |
TheJulia | I'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 pto | 15:41 |
JayF | cid: ^ if you're around can you look at the work items change? | 15:41 |
JayF | cardoe: ^ similarly | 15:41 |
JayF | I've seen cardo-e this morning and I think c-d is around :D | 15:41 |
TheJulia | yeah | 15:42 |
* TheJulia context switches into code review for a little while | 15:43 | |
rpittau | TheJulia: had another look at the summary email and it looks good | 15:46 |
rpittau | JayF: 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 it | 15:46 |
JayF | I gotta be honest, I hate the pattern of linking to reviews for something that's gonna be a permanent artifact | 15:47 |
JayF | It'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 version | 15:47 |
JayF | but 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 them | 15:48 |
cid | JayF, I will ensure to do that | 15:49 |
* cid Right about now | 15:50 | |
TheJulia | It is also kind of awful to wait for weeks to reach that front end consensus | 15:51 |
TheJulia | I'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 email | 15:52 |
JayF | you said it better than I did :) | 15:52 |
TheJulia | Thanks, I guess ;) | 16:14 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic master: Add shared image support https://review.opendev.org/c/openstack/ironic/+/947115 | 16:40 |
cardoe | JayF: which work items? | 16:59 |
JayF | https://review.opendev.org/c/openstack/ironic-specs/+/946921 | 16:59 |
cardoe | oh the spec | 16:59 |
TheJulia | AI cat with 6 legs wanting networking gear for the humans https://usercontent.irccloud-cdn.com/file/imSBKoHx/halpnetworkingkitten.png | 17:07 |
opendevreview | Merged openstack/ironic master: Improve is_image_available https://review.opendev.org/c/openstack/ironic/+/946575 | 17:22 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Release notes title to "unreleased" for in-progress https://review.opendev.org/c/openstack/ironic/+/946933 | 17:36 |
TheJulia | rebased to re-root since no tests ran | 17:37 |
cardoe | What's our gerrit syntax for partial bug fix? | 18:09 |
cardoe | just Partial-Bug: #XXX ? | 18:09 |
JayF | aye | 18:09 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic master: Add shared image support https://review.opendev.org/c/openstack/ironic/+/947115 | 18:21 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP provide host_id to neutron early on https://review.opendev.org/c/openstack/ironic/+/946378 | 18:30 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic stable/2025.1: Improve is_image_available https://review.opendev.org/c/openstack/ironic/+/947276 | 18:46 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP provide host_id to neutron early on https://review.opendev.org/c/openstack/ironic/+/946378 | 18:58 |
TheJulia | lets see if we can make metalsmith happy :) | 18:58 |
opendevreview | Merged openstack/ironic-tempest-plugin master: CI: Increment jobs for 2025.1 release https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/946593 | 19:19 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP provide host_id to neutron early on https://review.opendev.org/c/openstack/ironic/+/946378 | 19:49 |
opendevreview | Julia Kreger proposed openstack/ironic master: provide host_id to neutron early on https://review.opendev.org/c/openstack/ironic/+/946378 | 21:02 |
TheJulia | Okay, I think that sort of addresses the issues there and hopefully will now pass CI in general | 21:11 |
TheJulia | (along with metalsmith) | 21:11 |
opendevreview | Merged openstack/ironic-specs master: 2025.2 Work Items from PTG https://review.opendev.org/c/openstack/ironic-specs/+/946921 | 21:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!