| *** dtantsur_ is now known as dtantsur | 00:08 | |
| opendevreview | likui proposed openstack/ironic master: CI: Remove ubuntu focal job https://review.opendev.org/c/openstack/ironic/+/894014 | 02:52 |
|---|---|---|
| dtantsur | JayF, bifrost tends to be more conservative when it comes to removing distributions. While Ironic just follows the PTI, Bifrost makes an effort to keep older versions working as long as possible. | 08:17 |
| kubajj | Good morning dtantsur, and Ironic! o/ | 08:27 |
| masghar | Good morning | 08:34 |
| dtantsur | o/ | 08:42 |
| iurygregory | good morning Ironic | 11:26 |
| JayF | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Sep 11 15:00:04 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| JayF | Good morning Ironic'ers: A reminder we operate under the OpenInfra Foundation CoC https://openinfra.dev/legal/code-of-conduct | 15:00 |
| kubajj | o/ | 15:00 |
| dtantsur | o/ | 15:00 |
| JayF | Agenda is available at https://wiki.openstack.org/wiki/Meetings/Ironic | 15:00 |
| JayF | We don't have much on the agenda so I'm going to try and get through it quickly. | 15:00 |
| JayF | #topic Announcements/Reminder | 15:00 |
| 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:00 |
| JayF | I'll additionally say: if you have anything you want to get in 2023.2, please paste it in here post-meeting | 15:01 |
| JayF | I will volunteer to review jam on them post-meeting and we can land them | 15:01 |
| JayF | that's because... | 15:01 |
| JayF | #note Bobcat release: October 4th, 2023; Ironic should release in the next week or two. | 15:01 |
| JayF | Also shortly after that is the PTG | 15:02 |
| JayF | #link https://etherpad.opendev.org/p/ironic-ptg-october-2023 | 15:02 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/890817 seems an easy win | 15:02 |
| JayF | Please begin thinking about what you want to do for 2024.1 lets get it on the PTG etherpad and/or RFEs written | 15:02 |
| JayF | dtantsur: ack, in my queueu | 15:02 |
| JayF | Moving on from announcements | 15:03 |
| JayF | No action items last meeting; skipping. | 15:03 |
| JayF | #topic Review Ironic CI Status & Update whiteboard if needed | 15:03 |
| JayF | I think we're in an OK spot? Any issues to repot? | 15:03 |
| dtantsur | We really, really hope that the most recent patches have fixed metal3 with sqlite | 15:04 |
| dtantsur | Testing results are promising so far | 15:04 |
| JayF | I kinda feel like we need to document database support? | 15:04 |
| JayF | I'm not sure anyone but metal3 uses our sqlite support | 15:04 |
| dtantsur | I've put a note about sqlite into the corresponding section | 15:05 |
| JayF | ack | 15:05 |
| JayF | moving on | 15:05 |
| JayF | #topic Branch retirement to resolve zuul-config-errors | 15:05 |
| JayF | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034854.html | 15:05 |
| JayF | I will be actioning this at some point this week, take notice. | 15:05 |
| JayF | #topic RFE Review | 15:07 |
| JayF | #link https://bugs.launchpad.net/ironic/+bug/2033288 | 15:07 |
| JayF | rpittau proposed ' [rpittau] RFE proposal "Allow non-bootable iso connection/disconnection for provisioned nodes" ' | 15:07 |
| JayF | and it was marked rfe-approved on 8/29 | 15:07 |
| JayF | this is a stale agenda entry, I believe | 15:07 |
| * JayF punts it | 15:07 | |
| dtantsur | Yeah, I believe so | 15:07 |
| JayF | #topic Open Discussion | 15:07 |
| JayF | Anything else? | 15:07 |
| dtantsur | Not from me | 15:08 |
| JayF | #endmeeting | 15:08 |
| opendevmeet | Meeting ended Mon Sep 11 15:08:40 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:08 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-09-11-15.00.html | 15:08 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-09-11-15.00.txt | 15:08 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-09-11-15.00.log.html | 15:08 |
| JayF | dtantsur: is there value in 890817 before it's all done? | 15:08 |
| JayF | dtantsur: I'm not fully up on state of inspector migration | 15:08 |
| dtantsur | JayF, it's all partly operational already, although purposely not documented | 15:09 |
| TheJulia | Not around today, still trying to get home. | 15:09 |
| dtantsur | JayF, i.e. you can use it today already, but I'd avoid hurrying it into production :) | 15:09 |
| JayF | dtantsur: looks like you have valid comments on that PR still; I'll look over it but given it's not customer-facing I feel very little benefit in hurrying it into master :D | 15:09 |
| JayF | TheJulia: have a safe trip hope | 15:13 |
| JayF | s/hope/home/ | 15:13 |
| JayF | Going to post some things here that are trivial reviews and need one more +2+A | 16:11 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/890138 | 16:11 |
| JayF | (yoga) https://review.opendev.org/c/openstack/ironic-python-agent/+/892687 | 16:14 |
| JayF | (zed) https://review.opendev.org/c/openstack/ironic-python-agent/+/892594 | 16:15 |
| JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/892686 | 16:15 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Redfish: wait for secure boot state change if it's not immediate https://review.opendev.org/c/openstack/ironic/+/863999 | 16:44 |
| dtantsur | I'm not exceptionally fond of this change ^^^ but it is what it is. | 16:44 |
| dtantsur | Redfish opinions appreciated | 16:44 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Redfish: wait for secure boot state change if it's not immediate https://review.opendev.org/c/openstack/ironic/+/863999 | 16:45 |
| JayF | There's not a case we'd ever change boot mode that wasn't a part of a provisioning action, right? | 16:49 |
| dtantsur | JayF, there is (fortunately asynchronous) API for that as well | 16:50 |
| JayF | in that case this change is a touch more problematic, right? | 16:50 |
| JayF | because across Ironic versions, something that used to leave your in-band workload alone now can cause a system reboot | 16:50 |
| dtantsur | possibly? these direct APIs are always a bit weird citizens | 16:51 |
| JayF | I think I'm going to leave a comment that we need, at a minimum, to document the change in behavior in a release note | 16:51 |
| dtantsur | Fair | 16:51 |
| JayF | maximally give a flag to restore old behavior but ... I doubt anyone cares in this case | 16:51 |
| dtantsur | same | 16:52 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [CI] Support for running with shards https://review.opendev.org/c/openstack/ironic/+/894460 | 16:55 |
| JayF | BTW; that is working | 16:56 |
| JayF | the failures were the alternate case (non-sharding) that I broke | 16:56 |
| opendevreview | Verification of a change to openstack/ironic stable/xena failed: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885242 | 17:45 |
| JayF | The devstack depends-on change for https://review.opendev.org/c/openstack/ironic/+/894460 is landing, I'd enjoy if we could land this before we branch so sharding gets hello-world tested in the gate | 17:45 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [CI] Support for running with shards https://review.opendev.org/c/openstack/ironic/+/894460 | 20:01 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [CI] Support for running with shards https://review.opendev.org/c/openstack/ironic/+/894460 | 20:01 |
| JayF | > The attribute(s) "shard" are invalid; they are not needed to create nodes. | 22:44 |
| JayF | I must have screwed up node sharding support in the client somehow? | 22:44 |
| JayF | from https://zuul.opendev.org/t/openstack/build/083d3a4460fa4ec9ae3801ca7b494838 | 22:44 |
| JayF | If anyone has any ideas please leave them, I even double checked that this should work for node create but apparently never tested it | 22:45 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [CI] Support for running with shards https://review.opendev.org/c/openstack/ironic/+/894460 | 22:50 |
| JayF | that sets it after the fact; but I do want to fix it for on-create because that seems like a super common case | 22:50 |
| TheJulia | Node create takes a weird path if memory serves | 23:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!