| opendevreview | Jacob Anders proposed openstack/ironic master: Update documentation to include servicing abort. https://review.opendev.org/c/openstack/ironic/+/957825 | 00:35 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/ironic master: Update documentation to include servicing abort. https://review.opendev.org/c/openstack/ironic/+/957825 | 00:36 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Update documentation to include servicing abort. https://review.opendev.org/c/openstack/ironic/+/957825 | 00:46 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Update documentation to include servicing abort. https://review.opendev.org/c/openstack/ironic/+/957825 | 00:47 |
| opendevreview | Merged openstack/ironic master: Update documentation to include servicing abort. https://review.opendev.org/c/openstack/ironic/+/957825 | 03:17 |
| opendevreview | Merged openstack/ironic master: Fix servicing abort to respect abortable flag https://review.opendev.org/c/openstack/ironic/+/957189 | 03:52 |
| rpittau | good morning ironic! o/ | 06:37 |
| queensly[m] | Good morning! | 07:05 |
| opendevreview | Merged openstack/ironic master: Update the state machine diagram https://review.opendev.org/c/openstack/ironic/+/957990 | 09:58 |
| rpittau | bye everyone, see you in 3 weeks o/ | 13:40 |
| opendevreview | Elod Illes proposed openstack/metalsmith unmaintained/2023.1: [CI] Remove centos8 based jobs https://review.opendev.org/c/openstack/metalsmith/+/958279 | 14:03 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: WIP: Fix skip block devices for RAID arrays https://review.opendev.org/c/openstack/ironic-python-agent/+/937342 | 15:05 |
| *** dking is now known as Guest24776 | 16:16 | |
| opendevreview | Merged openstack/metalsmith unmaintained/2023.1: [CI] Remove centos8 based jobs https://review.opendev.org/c/openstack/metalsmith/+/958279 | 16:25 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add request logging middleware for API requests https://review.opendev.org/c/openstack/ironic/+/958103 | 16:38 |
| opendevreview | Merged openstack/networking-generic-switch master: Update author information https://review.opendev.org/c/openstack/networking-generic-switch/+/957208 | 16:55 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add node.instance_name https://review.opendev.org/c/openstack/ironic/+/952790 | 17:12 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add node.instance_name https://review.opendev.org/c/openstack/ironic/+/952790 | 17:14 |
| opendevreview | Merged openstack/ironic master: Handle race conditions in power state changes https://review.opendev.org/c/openstack/ironic/+/956674 | 17:45 |
| opendevreview | Merged openstack/ironic master: api: Allow more types for updates https://review.opendev.org/c/openstack/ironic/+/957960 | 17:59 |
| opendevreview | Merged openstack/bifrost master: Clarify testenv and install usage order in testenv.rst https://review.opendev.org/c/openstack/bifrost/+/946116 | 18:03 |
| JayF | I've given a review pass on almost all of ironic-week-prio; we have some things which are mergable if we wanna give the gate an exercise over the weekend | 18:04 |
| opendevreview | Merged openstack/metalsmith unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/936685 | 18:32 |
| JayF | FYI Jay Faulkner proposed openstack/governance master: Retire metalsmith project from bare metal program https://review.opendev.org/c/openstack/governance/+/958297 | 18:33 |
| JayF | Who is our release liason? We had a release cut for metalsmith this cycle. I would've hoped that would've been -1'd. | 18:42 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: etcd: Discover api version automatically https://review.opendev.org/c/openstack/networking-generic-switch/+/954331 | 18:44 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: devstack: Drop explicit etcd api version https://review.opendev.org/c/openstack/networking-generic-switch/+/957207 | 18:44 |
| opendevreview | Verification of a change to openstack/ironic master failed: No admin-set maintenance override on power sync https://review.opendev.org/c/openstack/ironic/+/956993 | 18:46 |
| opendevreview | Verification of a change to openstack/ironic master failed: api: Add schema for inspection rules API (versioning) https://review.opendev.org/c/openstack/ironic/+/954038 | 18:51 |
| frickler | JayF: https://opendev.org/openstack/governance/src/branch/master/reference/projects.yaml#L559-L562 note that you (plural, as a team) could have multiple liaisons for any of these positions if you want | 19:33 |
| JayF | ack | 19:34 |
| JayF | there's a nonzero part of this that is "we don't care about it to the point where nobody thought to retire it | 19:35 |
| JayF | " | 19:35 |
| opendevreview | Merged openstack/ironic master: Add periodic cleanup of stale conductors https://review.opendev.org/c/openstack/ironic/+/956500 | 19:58 |
| opendevreview | Merged openstack/ironic master: Add a new 'category' field to the Port object https://review.opendev.org/c/openstack/ironic/+/955447 | 19:58 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Add request logging for API requests https://review.opendev.org/c/openstack/ironic-python-agent/+/958307 | 20:06 |
| opendevreview | Merged openstack/ironic master: Memoize calls to bcrypt.checkpw https://review.opendev.org/c/openstack/ironic/+/958130 | 20:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic bugfix/31.0: Memoize calls to bcrypt.checkpw https://review.opendev.org/c/openstack/ironic/+/958308 | 20:18 |
| JayF | I think that's the right branch for it to go to first? I assume bugfix consumers are highly impacted by this bug | 20:24 |
| opendevreview | Merged openstack/ironic master: api: Add schema for bios API (responses) https://review.opendev.org/c/openstack/ironic/+/952149 | 20:42 |
| opendevreview | Merged openstack/ironic master: api: Add schema for inspection rules API (versioning) https://review.opendev.org/c/openstack/ironic/+/954038 | 21:35 |
| TheJulia | JayF: yeah, it is | 22:14 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/30.0: Memoize calls to bcrypt.checkpw https://review.opendev.org/c/openstack/ironic/+/958315 | 23:13 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.1: Memoize calls to bcrypt.checkpw https://review.opendev.org/c/openstack/ironic/+/958316 | 23:14 |
| TheJulia | I guess https://review.opendev.org/c/openstack/ironic/+/955625/8/releasenotes/notes/portgroup-physical_network-ea6df85c3e7a18ea.yaml#7 worries me, in that what if someone sets the physical_network separately from the ports. Could it be intentional, could it be unintentional. On ports it has a specific use to match and map up for neutron side scheduling. | 23:19 |
| JayF | do you mean: exposing it in the api before it's hooked up or using the parallel naming in general? | 23:31 |
| JayF | https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/trait-based-port-scheduling.html#data-model-impact fwiw is what this is for, although it's not going to be 100% this cycle, so I understand if we wanna feature flag it or just hold the API side of the change | 23:33 |
| JayF | clif: ^ | 23:33 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!