15:00:18 <JayF> #startmeeting ironic
15:00:19 <opendevmeet> Meeting started Mon Aug 28 15:00:18 2023 UTC and is due to finish in 60 minutes.  The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:19 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:19 <opendevmeet> The meeting name has been set to 'ironic'
15:00:29 <rpittau> o/
15:00:29 <JayF> Good morning Ironic'ers: A reminder we operate under the OpenInfra Foundation CoC https://openinfra.dev/legal/code-of-conduct
15:00:37 <iurygregory> o/
15:00:45 <JayF> Agenda is available at https://wiki.openstack.org/wiki/Meetings/Ironic
15:00:56 <kubajj> o/
15:01:16 <JayF> I'll note my internet is not behaving well this morning; if I disappear in the meeting someone please take over
15:01:19 <JayF> #chair rpittau
15:01:19 <opendevmeet> Current chairs: JayF rpittau
15:01:22 <JayF> #chair iurygregory
15:01:22 <opendevmeet> Current chairs: JayF iurygregory rpittau
15:01:34 <JayF> #note  Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: https://tinyurl.com/ironic-weekly-prio-dash
15:01:58 <JayF> We are reaching the end of the release cycle. Please prioritize reviewing things that are close so we can get as much stuff into the release as possible :)
15:02:21 <JayF> #note Bobcat client library freeze is in three days; please take notice
15:02:58 <JayF> Also please update the PTG etherpad with anything you have to discuss in October during our chat: https://etherpad.opendev.org/p/ironic-ptg-october-2023
15:03:07 <JayF> I think that's all the announcements :)
15:03:41 <JayF> #topic Previous action items
15:04:41 <JayF> I had an action item to review and take action on Ironic-related zuul config errors; I have a change up here: https://review.opendev.org/c/openstack/python-ironicclient/+/892622 to fix stable/yoga ironicclient; and an email out to the mailing list announcing all other impacted branches will be retired on Sept 1 or later. Please comment there if you have an objection.
15:04:52 <JayF> #link https://review.opendev.org/c/openstack/python-ironicclient/+/892622
15:05:12 <JayF> That was the only action item. Giving a minute for comment before moving on.
15:05:57 <rpittau> I jsut approved it
15:06:16 <JayF> Awesome
15:06:19 <JayF> moving on
15:06:25 <JayF> #topic Review CI Status
15:06:28 <JayF> How is CI?
15:07:52 <JayF> Other than that metal3 CI blip on Friday was there anything else?
15:08:10 <iurygregory> there is the issue with python-ironicinspectorclient
15:08:32 <iurygregory> the functional job is broken, still trying to figure out what is happening there
15:09:11 <rpittau> we should merge https://review.opendev.org/c/openstack/ironic/+/892680 ASAP so we can merge https://review.opendev.org/c/openstack/metalsmith/+/892965 and finally release metalsmith :)
15:09:14 <JayF> there have been approximately no changes in that repo, yeah?
15:09:26 <iurygregory> JayF, correct
15:09:42 <iurygregory> https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/891823 this was the patch that mentioned our gate was broken
15:10:03 <JayF> yeah we have to make that pass then :(
15:10:08 <iurygregory> and seems like it's broken till zed =X
15:10:17 <iurygregory> https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/856419
15:10:26 <iurygregory> since we couldn't merge this one
15:10:45 <JayF> I'll note that's a repo on the way out; I'm not that opposed to us removing the func job if we're sure the client itself still works
15:11:10 <iurygregory> the weird things is the test that is failing
15:11:57 <iurygregory> according to our test we should give an exception in case we try to abort inspection if is not started
15:12:15 <iurygregory> but we return 202 :D
15:12:16 <iurygregory> testtools.matchers._impl.MismatchError: <bound method ClientV1.abort of <ironic_inspector_client.v1.ClientV1 object at 0x7f2472a1b010>> returned <Response [202]>
15:12:33 <JayF> iurygregory: I also saw complaints about function-py310 not existing, which is true: https://github.com/openstack/python-ironic-inspector-client/blob/master/tox.ini#L45
15:12:47 <iurygregory> just a heads-up, we can move forward with the meeting
15:12:52 <JayF> yeah, we should
15:12:55 <iurygregory> https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/891976
15:13:11 <JayF> good stuff
15:13:13 <JayF> ty
15:13:16 <iurygregory> yeah, but in the patch I pushed it was given the actual error =)
15:13:19 <JayF> #topic Review ongoing 2023.2 Workstreams
15:14:00 <JayF> I'm going to note I think it's unlikely sharding will land in Nova this cycle. I will do what I can but it hasn't had enough review focus since the patches were put up. I will try to get eyes on it early next cycle when there is more developer availability :(
15:15:22 <JayF> https://etherpad.opendev.org/p/IronicWorkstreams2023.2
15:15:30 <JayF> although at this point; if you have changes outstanding for this release
15:15:34 <JayF> they should be close if you want them in :(
15:17:32 <JayF> Looks like no comments on workstreams in progress. Moving on.
15:17:43 <JayF> #topic Branch retirement for Zuul config errors
15:17:56 <opendevreview> Merged openstack/python-ironicclient stable/yoga: [stable-only] Fix zuul config; comment failing CI  https://review.opendev.org/c/openstack/python-ironicclient/+/892622
15:18:10 <JayF> As mentioned above, in response to action item to fix zuul config errors in Ironic; I'm proposing mass retirement of some branches: https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034854.html
15:18:23 <JayF> please comment there if you have any concerns
15:18:31 <JayF> #topic RFE Review
15:18:34 <JayF> rpittau: you have the floor
15:18:51 <rpittau> JayF: no concerns and I think I left a comment last week about the bugfix branches
15:19:12 <JayF> ty :)
15:19:15 <JayF> you wanna talk about your RFE?
15:19:29 <rpittau> JayF: yes, thanks!
15:20:19 <rpittau> I'd like to propose a change to allow to expose ironic functionalities to be able to attach/detach non-bootable isos to provisioned nodes
15:20:43 <rpittau> the RFE should have all the info https://bugs.launchpad.net/ironic/+bug/2033288 but please let me know if there are questions/doubts/concerns
15:20:53 * iurygregory reads
15:20:59 <JayF> I'm mainly curious about the use case, it'd be nice to have one in the bug.
15:21:12 <rpittau> the changes should be minimal as the features are there already, jsut need to be exposed
15:21:25 <TheJulia> Interesting!
15:21:28 <rpittau> JayF: right! I think I have that, just forgot to add it to the RFE! :D
15:21:59 <TheJulia> It does sort of make sense to expose, sort of like setting boot mode manually
15:22:11 <TheJulia> Err, boot device
15:22:24 <JayF> Yeah, I can think of a case or two but it's nice to know the actual real world reason we're doing a thing :)
15:22:25 <rpittau> yeah
15:22:44 * TheJulia returns to the Covid infected world of trying to rest
15:22:51 <JayF> if anything my only question is if there are other low-hanging items like this we should implement
15:23:05 <iurygregory> agree, I think the missing bit is just the use case (but I think it makes sense ++)
15:23:10 <rpittau> probably :D
15:23:56 <JayF> going to mark it approved based on lazy consensus here
15:24:21 <rpittau> thanks JayF , I'll add the use case soon(TM)
15:24:50 <JayF> #topic Open Discussion
15:24:56 <JayF> Anything else for the meeting?
15:26:34 <JayF> Last call?
15:26:48 <JayF> #endmeeting