| opendevreview | cid proposed openstack/ironic master: Fix the types of the agent_containers options https://review.opendev.org/c/openstack/ironic/+/998385 | 00:08 |
|---|---|---|
| opendevreview | cid proposed openstack/ironic master: Drop the unused container_conf_file option https://review.opendev.org/c/openstack/ironic/+/998386 | 00:08 |
| opendevreview | cid proposed openstack/ironic master: doc: Fix the container-based steps guide https://review.opendev.org/c/openstack/ironic/+/998387 | 00:08 |
| opendevreview | cid proposed openstack/ironic-python-agent-builder master: Write container options as comma separated lists https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998389 | 00:12 |
| opendevreview | cid proposed openstack/ironic-python-agent-builder master: Do not require a container steps file to build https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998390 | 00:12 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: Improve autodetect test coverage https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/998398 | 01:57 |
| opendevreview | Steve Baker proposed openstack/ironic master: Switch ironic-standalone-redfish to autodetect deploy https://review.opendev.org/c/openstack/ironic/+/997910 | 01:57 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: Improve autodetect test coverage https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/998398 | 05:19 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add allow_grouping_reboots argument to firmware.update step https://review.opendev.org/c/openstack/ironic/+/998417 | 08:00 |
| *** jroll05 is now known as jroll0 | 09:09 | |
| hjensas | TheJulia: cardoe: I tested https://review.opendev.org/c/openstack/networking-generic-switch/+/998062, on Cisco it works, on Arista it only works if I use ssh-key. Tried configuring Arista's management ssh to use "password" - but it looks like it does keyboard-interactive no matter. | 12:04 |
| hjensas | wdyt, should I change the patch to make libssh/paramiko a config option? | 12:05 |
| dtantsur | I think it won't hurt to make it configurable in case of regressions like this one. | 12:07 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Switch netconf transport from paramiko to libssh https://review.opendev.org/c/openstack/networking-generic-switch/+/998062 | 12:33 |
| TheJulia | brraaains | 13:14 |
| TheJulia | Well, I don't think that is a regression necessarilly, its a defect in arista forcing the user to be keyboard-interactive | 13:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC https://review.opendev.org/c/openstack/ironic/+/997366 | 13:15 |
| dtantsur | TheJulia: well, if it worked with netmiko, it's effectively a regression if we switch to netconf | 13:15 |
| TheJulia | I really don't think we should extend the option out to the user to choose moving forward because its not like everyone will test both paths | 13:15 |
| TheJulia | everything is framed as new drivers to replace, not perfect continuiity | 13:16 |
| * TheJulia CAFFINATES | 13:17 | |
| iurygregory | good morning | 13:28 |
| TheJulia | The other issue is then we have to explicitly document "this one doesn't work with libssh" which is fine, but its still not great and then the question becomes how do we provide a paramiko-less path, if ever. Does it support the netconf over tls model? | 13:29 |
| TheJulia | I guess I'm trying to end up with an explicit matrix of "these are the drivers" as opposed to "you need to know the incantation" for specific drivers because we should document broadcast what is used and when. | 13:36 |
| TheJulia | I may also be hoping too much there | 13:37 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998479 | 14:01 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.6: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998480 | 14:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2026.1: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998481 | 14:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.4: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998482 | 14:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.3: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998483 | 14:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2025.2: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998484 | 14:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2025.1: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998485 | 14:03 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998486 | 14:03 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.6: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998487 | 14:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2026.1: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998488 | 14:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2025.2: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998489 | 14:07 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2025.1: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998490 | 14:08 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent unmaintained/2024.1: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998491 | 14:08 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent unmaintained/2023.1: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998492 | 14:08 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.4: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998493 | 14:09 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent bugfix/11.3: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998494 | 14:09 |
| TheJulia | So, folks... if it wasn't obvious by the titles, I'll need reviews ^ | 14:32 |
| iurygregory | looking now | 14:50 |
| TheJulia | Thanks | 14:58 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/37.0: bugfix/stable: lock constraints for bugfix to 2026.1 https://review.opendev.org/c/openstack/ironic/+/998511 | 15:12 |
| TheJulia | As an fyi, bugfix/37.0 is broken due to upper constraints, I've proposed a pin to https://review.opendev.org/c/openstack/ironic/+/998511 | 15:27 |
| iurygregory | +2 | 15:32 |
| TheJulia | I'll likely just workflow that one since it is a CI fix. | 15:35 |
| rpittau | we do pin UC in metal3, but not here | 15:41 |
| rpittau | maybe time to start doing that | 15:41 |
| TheJulia | reviews needed: https://review.opendev.org/c/openstack/ironic-python-agent/+/998486 | 15:43 |
| TheJulia | At least CI seems to be "mostly happy" today | 15:45 |
| TheJulia | last call on https://review.opendev.org/c/openstack/sushy/+/997463 | 15:59 |
| TheJulia | rutro, we have a conflict | 16:08 |
| TheJulia | on the constraints, 5.11.0 is requested. u-c is 5.10.1. | 16:08 |
| TheJulia | rpittau: what did you do the last time this broke one of the bugfix branches? Didn't you craft a custom constraints file for the branch? | 16:08 |
| TheJulia | I guess the inherent issue is unless we backport the change to rip the policy flag out, bugfix/37.0 is broken | 16:20 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Soft power off before virtual media ejection on inspection cleanup https://review.opendev.org/c/openstack/ironic/+/996692 | 16:29 |
| opendevreview | Merged openstack/networking-baremetal master: Clean up deprecated [ironic] options https://review.opendev.org/c/openstack/networking-baremetal/+/984216 | 16:37 |
| opendevreview | Julia Kreger proposed openstack/ironic unmaintained/2024.1: stable: limit virtualbmc so we don't get into a python conflict https://review.opendev.org/c/openstack/ironic/+/998529 | 16:47 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: doc: include info about l2vni_subport_anchor_network and cross refs https://review.opendev.org/c/openstack/networking-baremetal/+/998530 | 16:48 |
| opendevreview | Julia Kreger proposed openstack/ironic unmaintained/2024.1: stable: limit virtualbmc so we don't get into a python conflict https://review.opendev.org/c/openstack/ironic/+/998529 | 16:52 |
| TheJulia | Okay, that should fix 2024.1 | 16:55 |
| opendevreview | Merged openstack/ironic master: Add supports_deploy to AnsibleDeploy for autodetect https://review.opendev.org/c/openstack/ironic/+/997909 | 17:10 |
| opendevreview | Merged openstack/ironic-python-agent master: security: fix NTP command handling https://review.opendev.org/c/openstack/ironic-python-agent/+/998486 | 17:38 |
| TheJulia | reviews on https://review.opendev.org/q/I69e85a1c5040066674d90a6e12acc30cec96ba76 would be appreciated | 17:54 |
| opendevreview | Merged openstack/ironic-python-agent master: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998479 | 18:01 |
| opendevreview | Verification of a change to openstack/ironic-python-agent-builder stable/2025.2 failed: Fix IPA service not starting on CentOS 10 (systemd 257) https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998332 | 18:07 |
| * TheJulia wonders if all contributors have melted in the heat :) | 18:18 | |
| opendevreview | Merged openstack/networking-baremetal stable/2026.1: use agent hostname for notification listener pool name https://review.opendev.org/c/openstack/networking-baremetal/+/998079 | 18:25 |
| opendevreview | Merged openstack/networking-generic-switch stable/2025.2: Sanitise secret config field https://review.opendev.org/c/openstack/networking-generic-switch/+/996983 | 18:41 |
| opendevreview | Verification of a change to openstack/ironic-python-agent-builder stable/2025.1 failed: Fix IPA service not starting on CentOS 10 (systemd 257) https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998333 | 19:19 |
| opendevreview | Verification of a change to openstack/ironic-python-agent-builder stable/2026.1 failed: Fix IPA service not starting on CentOS 10 (systemd 257) https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998331 | 19:21 |
| * cardoe waves at Ironic | 19:34 | |
| cardoe | TheJulia: OVN issues melted me | 19:36 |
| cardoe | Somehow I ended up fighting oslo.service importing eventlet always triggering a patching. | 19:38 |
| opendevreview | Merged openstack/ironic-python-agent stable/2026.1: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998481 | 20:01 |
| opendevreview | Merged openstack/ironic-python-agent stable/2025.2: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998484 | 20:01 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/11.6: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998480 | 20:01 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/11.4: Do not expose registry pull secret to bootc container https://review.opendev.org/c/openstack/ironic-python-agent/+/998493 | 20:01 |
| cardoe | BUT... if JayF and stephenfin don't mind that I stomped on their work... https://review.opendev.org/c/openstack/oslo.service/+/992774 will remove eventlet from even being in Ironic's dep tree. | 20:17 |
| TheJulia | how was it doing that? | 21:48 |
| TheJulia | or maybe thats a bad question | 21:48 |
| TheJulia | easy patch: https://review.opendev.org/c/openstack/ironic/+/998529 | 21:50 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!