| opendevreview | Merged openstack/ironic master: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991717 | 03:51 |
|---|---|---|
| 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 | 04:05 |
| 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 | 04:05 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: L2VNI: Handle none bottom bound segment unbind https://review.opendev.org/c/openstack/networking-generic-switch/+/991755 | 04:05 |
| opendevreview | Merged openstack/ironic unmaintained/2023.1: security: directory transversal ISO9660 support https://review.opendev.org/c/openstack/ironic/+/991388 | 04:06 |
| opendevreview | Merged openstack/ironic unmaintained/2023.1: security: disable driver_info level pxe_template override https://review.opendev.org/c/openstack/ironic/+/991389 | 04:06 |
| opendevreview | Verification of a change to openstack/ironic unmaintained/2024.1 failed: Ensure kernel_append_params are valid kernel parameters https://review.opendev.org/c/openstack/ironic/+/991383 | 04:46 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add release mappings for ironic 36.0 and 37.0 https://review.opendev.org/c/openstack/ironic/+/991673 | 07:20 |
| opendevreview | Nuutti Hakala proposed openstack/ironic master: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/990458 | 08:45 |
| *** mdfr1 is now known as mdfr | 09:59 | |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Add SONiC multicast BUM configuration validation https://review.opendev.org/c/openstack/networking-generic-switch/+/981193 | 10:28 |
| opendevreview | Nuutti Hakala proposed openstack/ironic master: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/990458 | 11:19 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add release mappings for ironic 36.0 and 37.0 https://review.opendev.org/c/openstack/ironic/+/991673 | 12:35 |
| TheJulia | good morning | 13:16 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2026.1: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991854 | 13:22 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/34.0: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991856 | 13:29 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/33.0: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991857 | 13:30 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.2: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991858 | 13:35 |
| opendevreview | Julia Kreger proposed openstack/ironic unmaintained/2024.1: ci: stable: increase memory for ironic-base https://review.opendev.org/c/openstack/ironic/+/991859 | 13:57 |
| opendevreview | Nuutti Hakala proposed openstack/ironic master: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/990458 | 14:01 |
| TheJulia | rpittau: fyi, you should likely take a look at my comment on https://review.opendev.org/c/openstack/ironic/+/983349 | 14:07 |
| rpittau | TheJulia: oh whoops, completely overlooked that, thanks! | 14:11 |
| TheJulia | rpittau: what is it with the push behind trying to calculate a seconds provisioned on some changes recently? | 14:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: follow-up: tls getattr cleanup https://review.opendev.org/c/openstack/ironic/+/988787 | 14:21 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix send_sensor_data periodic with oslo.service spawn workers https://review.opendev.org/c/openstack/ironic/+/990251 | 14:24 |
| TheJulia | rpittau: thanks! | 14:27 |
| rpittau | TheJulia: it comes from the outreachy project we're working on with estherd[m] and hroy, we'd like to be able to tell how long a node is (was) in a specific status and work on that, for example to get the power consumption | 14:28 |
| TheJulia | So are we adding a "deployed_at_time" someplace to be able to unwind that? | 14:29 |
| TheJulia | it just feels... weird | 14:29 |
| JayF | TheJulia: rpittau: dtantsur: https://review.opendev.org/c/openstack/security-doc/+/991729 [OSSN-0099] DoS in Ironic (CVE-2026-50589) needs review. Once landed, I will announce the OSSN. Leaving for my Friday morning volunteering here shortly, but I should be back to publish around 10a/11a pacific | 14:29 |
| TheJulia | JayF: thanks | 14:30 |
| rpittau | TheJulia: I don't think we thought about a field specifically for wehn a node becomes deployed, that should be seen though from the node_history and the state changes | 14:34 |
| TheJulia | except node_history rolls off... | 14:35 |
| TheJulia | I mean, None could be a viable value I guess if you can't calculate it | 14:36 |
| opendevreview | Julia Kreger proposed 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 | 14:41 |
| TheJulia | Last call on defferred actions spec | 14:45 |
| TheJulia | Bonus points to cid and JayF who stealth reviewed it | 14:45 |
| clif | Question: in BootInterface.prepare_ramdisk is the `ramdisk_params` argument purely kernel parameters? The semantics of it seem slippery. | 14:53 |
| clif | The heredoc of `prepare_ramdisk` seems to imply it can be used for passing arbitrary parameters to the ramdisk (not necessarily just kernel parameters) | 14:55 |
| TheJulia | Well, all of the parameters can be kernel parameters or arguments to IPA I guess | 14:57 |
| TheJulia | prepare_ramdisk is generally only called for IPA, however it gets ued by the ramdisk deploy interface for user defined ramdisk interfaces which "deploy" the node (or persistently network boot the node) | 14:58 |
| clif | are they all passed as kernel parameters and IPA reads the kernel cmd line or are they split up prior somehow? | 14:58 |
| TheJulia | ipa reads the kernel command line | 14:59 |
| clif | aha | 14:59 |
| TheJulia | yeah | 14:59 |
| TheJulia | IPA *also* gets configuration via the lookup endpoint | 14:59 |
| TheJulia | and in some hyper-limited cases may also refresh some config dynamically in some cases... like root device detection if memory seves | 14:59 |
| clif | so if I'm understanding correctly, all ramdisk_params should be valid kernel parameters, even if the kernel is not necessarily using the parameter directly | 15:00 |
| TheJulia | yeah, which is why we also can't apply like a list of only known kernel parameters | 15:01 |
| clif | right | 15:01 |
| TheJulia | For example, at RH we have heavy use of a particular parameter which does ssh key injection in some cases | 15:01 |
| TheJulia | largely for troubleshooting ramdisks | 15:02 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.13 and will be offline starting at 1600 UTC. We have allocated an hour for the outage window lasting until 1700 UTC. https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/thread/UIFRQR34WYMF3KDMNQFMO222DIK3HTUD/ | 15:02 | |
| opendevreview | Merged openstack/ironic master: Add TLS hardening for HTTP image service connections https://review.opendev.org/c/openstack/ironic/+/987906 | 15:02 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Add TLS configuration to agent lookup response https://review.opendev.org/c/openstack/ironic/+/987887 | 15:12 |
| opendevreview | Merged openstack/ironic master: Add TLS hardening for OCI registry connections https://review.opendev.org/c/openstack/ironic/+/987907 | 15:14 |
| TheJulia | janders: at some point, assign yourself as the assignee of https://bugs.launchpad.net/ironic/+bug/2154614, since I can't figure out what your launchpad user name is :) | 15:19 |
| opendevreview | Merged openstack/ironic master: Add TLS hardening options for Redfish BMC connections https://review.opendev.org/c/openstack/ironic/+/987908 | 15:19 |
| opendevreview | Merged openstack/ironic master: Add TLS hardening for Ansible stream_url module https://review.opendev.org/c/openstack/ironic/+/985885 | 15:19 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Consolidate duplicated TLS code into ironic/common/tls_utils https://review.opendev.org/c/openstack/ironic/+/987909 | 15:26 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.2: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991858 | 15:32 |
| opendevreview | Merged openstack/ironic master: CI: fix discrepancy: all votings jobs must be in the gate https://review.opendev.org/c/openstack/ironic/+/988296 | 15:35 |
| Mahnoor | Hello ironic | 15:39 |
| Mahnoor | As a bug deputy, does one have to set importance as part of triaging a bug? | 15:39 |
| Mahnoor | I dont see it mentioned here https://docs.openstack.org/ironic/latest/contributor/bug-deputy.html | 15:40 |
| TheJulia | Your free to, but generally it becomes an assessment as folks look at and understand it | 15:41 |
| TheJulia | in other words, if you don't know, that is fine | 15:41 |
| TheJulia | but if you see something and think "oh, we need to fix this right now!", go ahead an set importance | 15:42 |
| dtantsur | I think it's good to at least roughly evaluate importance | 15:45 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.13 and will be offline momentarily. We have allocated an hour for the outage window lasting until 1700 UTC. https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/thread/UIFRQR34WYMF3KDMNQFMO222DIK3HTUD/ | 16:01 | |
| Mahnoor | Alrightie | 16:30 |
| Mahnoor | Thank you both! | 16:30 |
| JayF | I'm going to announce OSSN-0099 as soon as gerrit and zuul is happy again | 16:31 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Default TLS minimum version to 1.3 https://review.opendev.org/c/openstack/ironic/+/987953 | 16:42 |
| opendevreview | Merged openstack/ironic stable/2026.1: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991854 | 16:56 |
| opendevreview | Merged openstack/ironic bugfix/34.0: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991856 | 16:56 |
| opendevreview | Merged openstack/ironic bugfix/33.0: OSSN-0099: Add JSON body depth and size limiting middleware https://review.opendev.org/c/openstack/ironic/+/991857 | 16:57 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add optional ServiceRoot Vendor via SUSHY_EMULATOR_VENDOR https://review.opendev.org/c/openstack/sushy-tools/+/991904 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add empty FirmwareInventory collection stub under UpdateService https://review.opendev.org/c/openstack/sushy-tools/+/991905 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add ComputerSystem SerialNumber derived from domain UUID https://review.opendev.org/c/openstack/sushy-tools/+/991906 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add server-side $expand support with configurable depth cap https://review.opendev.org/c/openstack/sushy-tools/+/991907 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add optional domain-name Redfish identity via SUSHY_EMULATOR_IDENTITY_AS_NAME https://review.opendev.org/c/openstack/sushy-tools/+/991908 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add EthernetInterface UefiDevicePath derived from PCI topology https://review.opendev.org/c/openstack/sushy-tools/+/991909 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add empty BootOptions collection stub and Boot navigation links https://review.opendev.org/c/openstack/sushy-tools/+/991910 | 17:00 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add one-time boot via libvirt reboot listener (SUSHY_EMULATOR_BOOT_ONCE) https://review.opendev.org/c/openstack/sushy-tools/+/991912 | 17:03 |
| * TheJulia blinks | 17:50 | |
| Mahnoor | Have a good weekend everyone o/ | 17:59 |
| JayF | \o | 18:01 |
| TheJulia | you too! | 18:31 |
| opendevreview | Merged openstack/networking-generic-switch master: Add SONiC multicast BUM configuration validation https://review.opendev.org/c/openstack/networking-generic-switch/+/981193 | 18:38 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add Juniper Junos multicast BUM replication support https://review.opendev.org/c/openstack/networking-generic-switch/+/981307 | 18:52 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in MTU management for switch ports https://review.opendev.org/c/openstack/networking-generic-switch/+/984832 | 18:52 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in port carrier bounce on bind https://review.opendev.org/c/openstack/networking-generic-switch/+/984833 | 18:52 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in STP edge port and BPDU guard support https://review.opendev.org/c/openstack/networking-generic-switch/+/984847 | 18:53 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Crazy Idea Part Deux! Discussing tighter storage fabric integration https://review.opendev.org/c/openstack/ironic-specs/+/990532 | 19:03 |
| 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 | 19:22 |
| opendevreview | Merged openstack/ironic master: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/990458 | 19:56 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2026.1: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/991938 | 20:01 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/34.0: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/991939 | 20:02 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/33.0: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/991940 | 20:02 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.2: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/991941 | 20:03 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.1: Set bound for failure reason from RFB https://review.opendev.org/c/openstack/ironic/+/991943 | 20:05 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!