| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Try Two: Split NGS admin docs https://review.opendev.org/c/openstack/networking-generic-switch/+/997461 | 00:18 |
|---|---|---|
| opendevreview | Julia Kreger proposed openstack/ironic master: Extend the RBAC policy matrix to steps https://review.opendev.org/c/openstack/ironic/+/992542 | 00:58 |
| opendevreview | Verification of a change to openstack/ironic master failed: Maintain driver dependencies as optional dependencies https://review.opendev.org/c/openstack/ironic/+/996882 | 01:00 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add TLS hardening for kickstart configdrive and inspection rule connections https://review.opendev.org/c/openstack/ironic/+/988804 | 01:06 |
| opendevreview | Merged openstack/networking-generic-switch stable/2026.1: Sanitise secret config field https://review.opendev.org/c/openstack/networking-generic-switch/+/996981 | 01:40 |
| opendevreview | Merged openstack/ironic master: api: Add schema for conductors API https://review.opendev.org/c/openstack/ironic/+/995763 | 01:57 |
| opendevreview | Merged openstack/ironic master: api: Add schema for volume API https://review.opendev.org/c/openstack/ironic/+/995764 | 01:57 |
| opendevreview | Rico Lin proposed openstack/sushy master: Allow explicit virtual media transfer protocols https://review.opendev.org/c/openstack/sushy/+/997463 | 02:17 |
| opendevreview | Rico Lin proposed openstack/ironic master: Pass explicit virtual media protocol to Sushy https://review.opendev.org/c/openstack/ironic/+/997464 | 02:17 |
| opendevreview | Merged openstack/ironic master: Fix autodetect deploy interface failing during node adoption https://review.opendev.org/c/openstack/ironic/+/995919 | 02:23 |
| opendevreview | Rico Lin proposed openstack/sushy master: Allow explicit virtual media transfer protocols https://review.opendev.org/c/openstack/sushy/+/997463 | 02:24 |
| opendevreview | Rico Lin proposed openstack/ironic master: Pass explicit virtual media protocol to Sushy https://review.opendev.org/c/openstack/ironic/+/997464 | 02:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Maintain driver dependencies as optional dependencies https://review.opendev.org/c/openstack/ironic/+/996882 | 02:53 |
| opendevreview | Merged openstack/ironic master: Fix Redfish firmware update handling of NEW task state https://review.opendev.org/c/openstack/ironic/+/997207 | 02:58 |
| opendevreview | Rico Lin proposed openstack/ironic master: Pass explicit virtual media protocol to Sushy https://review.opendev.org/c/openstack/ironic/+/997464 | 04:13 |
| opendevreview | Rico Lin proposed openstack/ironic master: Pass explicit virtual media protocol to Sushy https://review.opendev.org/c/openstack/ironic/+/997464 | 04:13 |
| opendevreview | Merged openstack/ironic master: Fix destroy_node race condition https://review.opendev.org/c/openstack/ironic/+/997458 | 05:42 |
| opendevreview | Merged openstack/ironic master: Add TLS hardening for kickstart configdrive and inspection rule connections https://review.opendev.org/c/openstack/ironic/+/988804 | 05:42 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Add trunk subport for NETCONF OpenConfig driver https://review.opendev.org/c/openstack/networking-generic-switch/+/991458 | 05:59 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Netconf: Document Neutron trunk port support https://review.opendev.org/c/openstack/networking-generic-switch/+/991459 | 05:59 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Refactor: extract OpenConfigModelMixin https://review.opendev.org/c/openstack/networking-generic-switch/+/992884 | 05:59 |
| opendevreview | Merged openstack/ironic master: Maintain driver dependencies as optional dependencies https://review.opendev.org/c/openstack/ironic/+/996882 | 06:28 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC https://review.opendev.org/c/openstack/ironic/+/997366 | 08:37 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2026.1: Fix Redfish firmware update handling of NEW task state https://review.opendev.org/c/openstack/ironic/+/997509 | 10:48 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/37.0: Fix Redfish firmware update handling of NEW task state https://review.opendev.org/c/openstack/ironic/+/997510 | 10:49 |
| opendevreview | Merged openstack/ironic-specs master: Add spec for multi-component batched firmware updates https://review.opendev.org/c/openstack/ironic-specs/+/992208 | 11:03 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Fix destroy_node race condition https://review.opendev.org/c/openstack/ironic/+/997524 | 11:29 |
| opendevreview | Doug Goldstein proposed openstack/ironic bugfix/37.0: Fix destroy_node race condition https://review.opendev.org/c/openstack/ironic/+/997525 | 11:30 |
| dtantsur | cardoe: hey, would you like to follow-up on your comments on https://review.opendev.org/c/openstack/ironic-specs/+/985800 or is it fine if we just approve it? | 12:05 |
| cardoe | The issue is the string values are meaningless. | 12:06 |
| cardoe | That’s coming from the PCI ids DB. Which can vary by supplier. | 12:06 |
| cardoe | Better is to use the actual code value. | 12:06 |
| dtantsur | cardoe: Fair point, could you clarify it on the spec please? I don't think the author understood your argument. | 12:07 |
| cardoe | Yep | 12:09 |
| opendevreview | Merged openstack/ironic bugfix/37.0: Fix Redfish firmware update handling of NEW task state https://review.opendev.org/c/openstack/ironic/+/997510 | 12:20 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Use cirros 0.6.3 as default cirros version https://review.opendev.org/c/openstack/bifrost/+/932175 | 13:19 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Drop explicit dependency on python-subunit https://review.opendev.org/c/openstack/virtualbmc/+/997568 | 13:37 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Add note about lower constraints https://review.opendev.org/c/openstack/virtualbmc/+/997569 | 13:38 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix Redfish firmware update handling of NEW task state https://review.opendev.org/c/openstack/ironic/+/997509 | 13:48 |
| TheJulia | good morning | 14:02 |
| TheJulia | cardoe: quick ask, I added warnings to https://review.opendev.org/c/openstack/ironic-specs/+/990532 which I'm framing as just an item to put into the backlog at this point due to everything. Maybe we chip away at some aspects (like data types and all), but since I know you've headed down that path it would be good to highlight if there are any other major issues/challenges which you've discovered in your work that I somehow haven't | 14:11 |
| TheJulia | stumbled upon | 14:11 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Add note about lower constraints https://review.opendev.org/c/openstack/virtualbmc/+/997569 | 14:12 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/virtualbmc/+/997579 | 14:12 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Update supported python versions https://review.opendev.org/c/openstack/virtualbmc/+/997580 | 14:12 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/virtualbmc/+/997579 | 14:19 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Update supported python versions https://review.opendev.org/c/openstack/virtualbmc/+/997580 | 14:19 |
| opendevreview | Takashi Kajinami proposed openstack/virtualbmc master: Add note about lower constraints https://review.opendev.org/c/openstack/virtualbmc/+/997569 | 14:19 |
| cardoe | Sure I'll give it a look. | 14:21 |
| TheJulia | Thanks | 15:05 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Devstack support for fake-hardware nodes https://review.opendev.org/c/openstack/ironic/+/983251 | 15:08 |
| TheJulia | JayF: w/r/t the ipa token stuffs, is there any reason you didn't post a backport to 2026.1? | 15:48 |
| JayF | TheJulia: lol | 15:48 |
| JayF | TheJulia: time keeps on slipping, slipping, slipping... into the future | 15:48 |
| JayF | We have a 2026.1 release? [grows a few dozen more grey hairs] | 15:49 |
| * TheJulia reads that as "time for jay to go on PTO and visit a pool or something" | 15:49 | |
| TheJulia | I'll take care of it | 15:49 |
| JayF | Soon(tm) | 15:49 |
| JayF | thanks. I'm trying to draft an mdns disable patch | 15:49 |
| TheJulia | ok | 15:49 |
| TheJulia | ... hmmmm | 15:55 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2026.1: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997615 | 15:59 |
| TheJulia | can I get some reviews on https://review.opendev.org/c/openstack/networking-generic-switch/+/997461 so we can split the documentation up in ngs to make dtantsur happy :) | 16:10 |
| * dtantsur approves of making dtantsur happy | 16:10 | |
| JayF | 👀 | 16:13 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Use cirros 0.6.3 as default cirros version https://review.opendev.org/c/openstack/bifrost/+/932175 | 16:21 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2025.1: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997624 | 17:08 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.4: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997625 | 17:09 |
| JayF | I'll note I delegated the ContainerHardwareManager fix/rework to cid, and asked him -- in addition to fixing the security issues in 2160143 -- to take a pass over the whole feature for any other missed portions of the implementation, including testing or docs. | 17:12 |
| TheJulia | k | 17:29 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix destroy_node race condition https://review.opendev.org/c/openstack/ironic/+/997524 | 17:32 |
| opendevreview | Merged openstack/ironic bugfix/37.0: Fix destroy_node race condition https://review.opendev.org/c/openstack/ironic/+/997525 | 17:32 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/11.6: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997446 | 17:44 |
| opendevreview | Merged openstack/ironic master: Add releasenote for disallow-steps feature https://review.opendev.org/c/openstack/ironic/+/993131 | 18:00 |
| opendevreview | Verification of a change to openstack/ironic-python-agent stable/2026.1 failed: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997615 | 18:38 |
| opendevreview | Verification of a change to openstack/ironic-python-agent stable/2025.2 failed: Fix: Remove legacy agent_token fallback code https://review.opendev.org/c/openstack/ironic-python-agent/+/997449 | 18:46 |
| TheJulia | https://review.opendev.org/c/openstack/networking-generic-switch/+/997461 is going to be played by a corgi who will wimper to go out and play with the rest of the docs | 19:04 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Security: Make mDNS discovery explicitly opt-in https://review.opendev.org/c/openstack/ironic-python-agent/+/997637 | 19:10 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Security: Make mDNS discovery explicitly opt-in https://review.opendev.org/c/openstack/ironic-python-agent/+/997637 | 19:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Allow limiting what config can be set in mDNS https://review.opendev.org/c/openstack/ironic-python-agent/+/997638 | 19:17 |
| JayF | dtantsur: TheJulia: ^ I don't have a good sense of mDNS use cases, so I thought a smart approach to this might be to do the work to setup the framework, but let someone familiar with the use case add a 3rd commit to set the default. | 19:18 |
| * JayF reopens Julia's NGS docs change from this morning | 19:18 | |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Security: Make mDNS discovery explicitly opt-in https://review.opendev.org/c/openstack/ironic-python-agent/+/997637 | 19:30 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Allow limiting what config can be set in mDNS https://review.opendev.org/c/openstack/ironic-python-agent/+/997638 | 19:30 |
| JayF | How "alive" is CONF.inspection_callback_url in IPA? Is that a Inspector service vestige in IPA? | 19:43 |
| * cardoe prints out Julia's NGS docs change and bangs his head against them for a while. | 20:22 | |
| cardoe | I mean the docs change... words have moved from one place to another. | 20:22 |
| cardoe | Can confirm. | 20:23 |
| JayF | yeah, reviewing beyond that would be borderline-mean lol | 20:25 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: [experiment] Remove standalone flag from IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/997647 | 20:28 |
| TheJulia | cardoe: your not supposed to print them | 21:38 |
| TheJulia | JayF: I believe it still gets used as an inspection signal | 21:38 |
| JayF | TheJulia: yeah, I think I unraveled it for the mDNS change, but that needs to be closely reviewed | 21:39 |
| cardoe | I spent a day and half on a silly neutron policy issue that was caused by a DuplicatePolicyError being swallowed | 21:39 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Libssh based switch configuration drivers? https://review.opendev.org/c/openstack/ironic-specs/+/997653 | 21:41 |
| TheJulia | ugh | 21:48 |
| TheJulia | I think I have reached ENOSPOON requiring a meeting with the cats | 21:48 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Add podman package to ironic-python-agent-ramdisk https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/970336 | 22:49 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: Add minimal element ironic-python-agent-ramdisk-base https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/997659 | 22:49 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!