| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add Cisco NDFC REST API driver for VXLAN L2VNI https://review.opendev.org/c/openstack/networking-generic-switch/+/968484 | 02:30 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add multicast BUM replication support to NDFC driver https://review.opendev.org/c/openstack/networking-generic-switch/+/984815 | 02:30 |
| janders | dtantsur I realised I never responded to your question about https://review.opendev.org/c/openstack/ironic/+/991300 last week - yes, it does look like a genuine schema issue. I vetted Claude's fixes looking at the actual schema and the fix seems solid and grounded. There was some confusion around drives in the current version but also DMTF changed | 04:55 |
| janders | terminology around Controllers/StorageController and at what nesting level each live. PTAL if/when you have time, would be great to have your input since I think you created the previous schema. | 04:55 |
| opendevreview | Konstantinos Karampogias proposed openstack/ironic master: Add vendor/model/serial_number to firmware component endpoint https://review.opendev.org/c/openstack/ironic/+/992034 | 06:26 |
| rpittau | when any core has a moment I think we should merge https://review.opendev.org/c/openstack/ironic/+/991673, we're already a bit late for the bugfix branch cut and would be great to not have further API changes, thanks! :) | 07:28 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add netconf_models package with OpenConfig lib https://review.opendev.org/c/openstack/networking-generic-switch/+/990059 | 08:59 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NetconfSwitch base class for NETCONF transport https://review.opendev.org/c/openstack/networking-generic-switch/+/990060 | 08:59 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NetconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/990061 | 08:59 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NETCONF OpenConfig driver documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/990062 | 08:59 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add trunk subport for NETCONF OpenConfig driver https://review.opendev.org/c/openstack/networking-generic-switch/+/991458 | 08:59 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Netconf: Document Neutron trunk port support https://review.opendev.org/c/openstack/networking-generic-switch/+/991459 | 08:59 |
| hroy | JayF, cardoe can you take a look at the disallow steps patch https://review.opendev.org/c/openstack/ironic/+/989017, it already has +2 and CI is green, thanks! | 11:44 |
| opendevreview | Merged openstack/ironic stable/2026.1: Fix kernel parameter parsing for quoted values and whitespace https://review.opendev.org/c/openstack/ironic/+/992340 | 12:31 |
| opendevreview | Merged openstack/ironic bugfix/34.0: Fix kernel parameter parsing for quoted values and whitespace https://review.opendev.org/c/openstack/ironic/+/992367 | 12:36 |
| opendevreview | Merged openstack/ironic bugfix/33.0: Fix kernel parameter parsing for quoted values and whitespace https://review.opendev.org/c/openstack/ironic/+/992368 | 12:47 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Pass physnet VLAN segments to device drivers https://review.opendev.org/c/openstack/networking-generic-switch/+/990510 | 12:48 |
| opendevreview | Verification of a change to openstack/ironic stable/2025.2 failed: Fix kernel parameter parsing for quoted values and whitespace https://review.opendev.org/c/openstack/ironic/+/992341 | 12:58 |
| iurygregory | good morning ironic, I'm back after the long weekend | 12:59 |
| dtantsur | wb iurygregory, I hope you enjoyed it :) | 13:01 |
| iurygregory | yeah, it was pretty good =D | 13:03 |
| TheJulia | brraaains | 13:20 |
| TheJulia | Could I get a review on https://review.opendev.org/c/openstack/ironic/+/990430 as well as hroy's https://review.opendev.org/c/openstack/ironic/+/989017 change this morning? | 13:23 |
| rpittau | I thought I approved hroy change already, did I dream about it ? :D | 13:24 |
| TheJulia | maybe? | 13:25 |
| TheJulia | Was everything in technicolor? | 13:25 |
| rpittau | :D | 13:27 |
| opendevreview | Konstantinos Karampogias proposed openstack/ironic master: Add vendor/model/serial_number to firmware component endpoint https://review.opendev.org/c/openstack/ironic/+/992034 | 13:41 |
| opendevreview | Merged openstack/ironic stable/2025.1: Fix kernel parameter parsing for quoted values and whitespace https://review.opendev.org/c/openstack/ironic/+/992345 | 14:09 |
| opendevreview | Merged openstack/ironic master: Add release mappings for ironic 36.0 and 37.0 https://review.opendev.org/c/openstack/ironic/+/991673 | 14:09 |
| opendevreview | Merged openstack/ironic master: Add operator-configurable step disallow lists https://review.opendev.org/c/openstack/ironic/+/989017 | 14:11 |
| TheJulia | https://doodle.com/group-poll/participate/dyALj4nd <-- doodle for a midcycle | 14:24 |
| TheJulia | rpittau: once the current change merges: https://review.opendev.org/q/I8366088731aa34da0e99941924c222ffed455c8b is next ;) | 14:48 |
| rpittau | \o/ | 14:48 |
| TheJulia | Wow, you guys responded and are like... "actually live!" ;) | 15:55 |
| dtantsur | JFYI: the infra folks are looking into why gerrit is barely responding | 15:55 |
| TheJulia | Didn't notice, board meeting this morning | 15:56 |
| opendevreview | Merged openstack/networking-generic-switch master: Pass physnet VLAN segments to device drivers https://review.opendev.org/c/openstack/networking-generic-switch/+/990510 | 16:07 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/networking-baremetal master: Upgrade OVN SSL connections to TLS 1.3 with hybrid PQC key exchange https://review.opendev.org/c/openstack/networking-baremetal/+/992495 | 17:21 |
| opendevreview | cid proposed openstack/ironic master: Allow disabling port pre-creation during inspection https://review.opendev.org/c/openstack/ironic/+/990340 | 17:46 |
| opendevreview | Merged openstack/ironic master: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/990430 | 18:04 |
| opendevreview | Merged openstack/ironic unmaintained/2024.1: Ensure kernel_append_params are valid kernel parameters https://review.opendev.org/c/openstack/ironic/+/991383 | 18:04 |
| TheJulia | If I could get reviews on https://review.opendev.org/#/q/I8366088731aa34da0e99941924c222ffed455c8b to just to ahead and get the stable backports landed. Thanks! | 18:05 |
| JayF | +2A on bugfix branches + 2026.1 | 18:12 |
| TheJulia | thanks! | 18:13 |
| TheJulia | any issue with 2025.1 ? | 18:14 |
| TheJulia | Thanks | 18:29 |
| opendevreview | Merged openstack/ironic stable/2026.1: Convert string values in BIOS settings before sending them to the BMC https://review.opendev.org/c/openstack/ironic/+/988999 | 18:32 |
| opendevreview | Merged openstack/ironic master: Fix Redfish interop profile Storage requirements https://review.opendev.org/c/openstack/ironic/+/991300 | 18:32 |
| JayF | Just trying to ensure things land in somewhat of the correct order :) | 18:33 |
| JayF | technically you can't approve (N+1) until (N) lands | 18:33 |
| TheJulia | Much appreciated | 18:33 |
| TheJulia | yup | 18:34 |
| opendevreview | Merged openstack/ironic stable/2026.1: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992321 | 19:12 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2026.1: Add operator-configurable step disallow lists https://review.opendev.org/c/openstack/ironic/+/992516 | 19:40 |
| clif | eventlet is entirely gone from ironic right? | 19:41 |
| clif | or do we still depend on some things that use eventlet? | 19:42 |
| TheJulia | We've excised it although the test runner can still load it for $reasons outside of our control, why | 19:43 |
| JayF | there's a lot of code shaped like (if eventlet is importable: do eventlet things) in oslo | 19:43 |
| TheJulia | yeah | 19:43 |
| JayF | so if it's in a venv, you can end up with it running and patched in even if you don't wanna | 19:43 |
| clif | eugh | 19:44 |
| TheJulia | but also, when you declare it upfront you can only load threading and not eventlet | 19:44 |
| TheJulia | anyway, why do you ask? | 19:44 |
| * TheJulia listens to happy music hoping there is not a big gnarly issue | 19:44 | |
| clif | whenever I run a local python script that imports anything from ironic I get a long-ish screed printed out that eventlet is deprecated | 19:45 |
| JayF | does it have eventlet in the venv it's loaded up to? | 19:45 |
| clif | this is from inside an ironic tox created venv | 19:45 |
| JayF | I assume this is a given: but from master with no tox jobs run against something 2025.2 or older? | 19:45 |
| clif | I created this venv via `tox -re venv` from ironic master | 19:46 |
| clif | so something is pulling it in: eventlet==0.41.0 | 19:46 |
| * JayF flips a table | 19:47 | |
| clif | just did it again to confirm, yes eventlet is in the venv | 19:48 |
| JayF | can you maybe do it with a -v, see what is pulling it in? | 19:48 |
| clif | I'm guessing its this: https://opendev.org/openstack/requirements/raw/branch/master/upper-constraints.txt | 19:50 |
| clif | which has eventlet==0.41.0 in it | 19:50 |
| JayF | that constrains the version installed *if* a package requirement wants it | 19:50 |
| clif | ah | 19:50 |
| clif | so there's something wanting it | 19:50 |
| clif | well eventlet is not mentioned when I add -v to the tox command line | 19:51 |
| JayF | -v is at least 20% more useless than it used to be | 19:51 |
| JayF | I'm doing this stuff locally now too fighing for it | 19:51 |
| JayF | **fishing for it | 19:52 |
| JayF | oslo.service has an unconditional dep on it? no way | 19:52 |
| JayF | https://opendev.org/openstack/oslo.service/src/branch/master/requirements.txt#L7 | 19:52 |
| JayF | stephenfin: ^ How is any OpenStack project supposed to keep eventlet away :( | 19:53 |
| opendevreview | Merged openstack/ironic bugfix/34.0: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992322 | 19:58 |
| opendevreview | Merged openstack/ironic bugfix/33.0: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992323 | 19:58 |
| TheJulia | I think the plan is to excise that once every project has the unconditional invocation setting upfront so for service ops they can't launch in eventlet mode | 19:58 |
| opendevreview | Merged openstack/ironic stable/2025.2: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992325 | 19:58 |
| opendevreview | Merged openstack/ironic stable/2025.1: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992326 | 19:58 |
| opendevreview | Merged openstack/ironic unmaintained/2023.1: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992335 | 19:58 |
| JayF | I just wish eventlet wasn't in the environment at all | 20:00 |
| TheJulia | Yeah, fair | 20:00 |
| JayF | but i guess in situations like debian packaging, we don't have that choice | 20:01 |
| JayF | I've seen enough "if eventlet: do x" code that it makes me nervous | 20:01 |
| TheJulia | totally fair | 20:04 |
| clif | yea if I manually remove it things seem fine, just annoying to have that screed pop up everywhere for seemingly no good reason | 20:08 |
| TheJulia | yeah, when we ripped eventlet originally, we were manually removing eventlet and just making sure everything was good | 20:39 |
| opendevreview | Merged openstack/ironic stable/2025.2: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991858 | 20:59 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Extend the RBAC policy matrix to steps https://review.opendev.org/c/openstack/ironic/+/992542 | 21:06 |
| JayF | I'm backporting https://review.opendev.org/c/openstack/ironic/+/992516 as the ability to disallow steps is a security hardening feature which can be trivially backported. Sorta an obvious thing I'm surprised it took us this long to do! If you have any concerns about this backport, feel free to DM me. | 22:36 |
| TheJulia | +2 | 22:56 |
| opendevreview | Merged openstack/networking-baremetal master: docs: fix docs build https://review.opendev.org/c/openstack/networking-baremetal/+/990304 | 23:03 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Switch centos9 jobs to centos10 https://review.opendev.org/c/openstack/ironic-python-agent/+/992571 | 23:16 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix retrieval of volume name if the Name does not include hostname https://review.opendev.org/c/openstack/ironic-python-agent/+/990110 | 23:40 |
| opendevreview | Merged openstack/ironic master: Retry on Redfish vmedia insert server errors https://review.opendev.org/c/openstack/ironic/+/990924 | 23:43 |
| opendevreview | Merged openstack/ironic unmaintained/2024.1: Security: Fix sensitive properties returned on volume targets https://review.opendev.org/c/openstack/ironic/+/992333 | 23:43 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!