| cardoe | How would folks feel about backporting node history stuff like? https://review.opendev.org/c/openstack/ironic/+/981474 | 00:02 |
|---|---|---|
| cardoe | I'm gonna backport that stuff locally cause it seems good for troubleshooting but just floating it out there. | 00:02 |
| TheJulia | I would be good with treating that as a bug fix ;) | 00:05 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: WIP directly link to systemd init https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/985060 | 00:16 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Use startswith for interface validation in tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982960 | 00:27 |
| opendevreview | Merged openstack/ironic master: ci: fix guarding logic ovs qrouter logic https://review.opendev.org/c/openstack/ironic/+/984950 | 00:45 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: WIP directly link to systemd init https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/985060 | 00:54 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: WIP directly link to systemd init https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/985060 | 01:38 |
| opendevreview | Merged openstack/networking-generic-switch master: Add Dell OS10 bond trunk support https://review.opendev.org/c/openstack/networking-generic-switch/+/983131 | 02:36 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent-builder master: WIP directly link to systemd init https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/985060 | 02:58 |
| opendevreview | Victor proposed openstack/ironic master: Truncate node history event to avoid DB column overflow https://review.opendev.org/c/openstack/ironic/+/981399 | 04:10 |
| opendevreview | Kaifeng Wang proposed openstack/ironic-python-agent master: Add support to collect subsystem ids https://review.opendev.org/c/openstack/ironic-python-agent/+/968587 | 08:35 |
| opendevreview | Merged openstack/ironic master: Log node history on vif_attach and vif_detach failures https://review.opendev.org/c/openstack/ironic/+/981474 | 09:16 |
| dtantsur | JFYI: it looks like the downstream conflict has cleared up for Monday for our team. Sorry for the noise! | 12:00 |
| dtantsur | Leaving a bit earlier, have a nice weekend all | 12:00 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add missing command descriptions to docs extension https://review.opendev.org/c/openstack/networking-generic-switch/+/985101 | 13:04 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add Cumulus NVUE multicast BUM replication support https://review.opendev.org/c/openstack/networking-generic-switch/+/981018 | 13:50 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Extract inline EVPN/VXLAN commands to class constants https://review.opendev.org/c/openstack/networking-generic-switch/+/985109 | 13:50 |
| cardoe | kubajj: janders: https://bugs.launchpad.net/ironic/+bug/2148675 | 14:13 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Log node history on vif_attach and vif_detach failures https://review.opendev.org/c/openstack/ironic/+/985115 | 14:13 |
| cardoe | Unless someone has any better suggestion on how I can un-wedge this box without a database edit? | 14:15 |
| cardoe | ^ that's the backport as a bugfix like we said | 14:15 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: bios.apply_configuration needs to validate before starting https://review.opendev.org/c/openstack/ironic/+/985117 | 14:25 |
| opendevreview | Merged openstack/ironic master: Limit interfaces to fake-graphical for testing https://review.opendev.org/c/openstack/ironic/+/982270 | 14:36 |
| opendevreview | Merged openstack/networking-generic-switch master: Add missing command descriptions to docs extension https://review.opendev.org/c/openstack/networking-generic-switch/+/985101 | 14:40 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 14:42 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Fix SONiC and Arista VXLAN parameter config errors https://review.opendev.org/c/openstack/networking-generic-switch/+/985122 | 15:09 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Do not soft power off before during inspection clean up if PXE is used https://review.opendev.org/c/openstack/ironic/+/985123 | 15:15 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Do not soft power off during inspection clean up if PXE is used https://review.opendev.org/c/openstack/ironic/+/985123 | 15:46 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add SONiC multicast BUM configuration validation https://review.opendev.org/c/openstack/networking-generic-switch/+/981193 | 15:53 |
| 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 | 15:53 |
| TheJulia | cardoe: i suspect a bit more details might be helpful. I mean, its sane, but a bit more context may be helpful. | 15:59 |
| cardoe | TheJulia: so when we changed the bios.apply_configuration step to work without IPA one of the review changes that got added along the way was to check that every key=value supplied is actually set before the step completes | 16:01 |
| TheJulia | I mean in the bug :) | 16:01 |
| cardoe | Done | 16:07 |
| cardoe | Do we have a minimum version of tox defined anywhere? | 16:07 |
| TheJulia | I was under the impression that openstack was supposed to be >=3 until I saw arguing about 2 a few days ago | 16:46 |
| JayF | I thought we were >=4 lol | 16:47 |
| JayF | and that's what -infra is suggesting is their default over there | 16:47 |
| TheJulia | I think that is reasonable | 16:49 |
| cardoe | okay I'm asking in #openstack-infra as well. | 16:51 |
| cardoe | 4.28 with the new constraints syntax drops plain tox run time pretty bigly. | 17:14 |
| TheJulia | sounds like it should really be 5.0... | 17:53 |
| cardoe | There's also a random version that converted to using lock files internally that's newer as well. | 18:01 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: GenericSwitchNetmikoConfigError error details https://review.opendev.org/c/openstack/networking-generic-switch/+/985029 | 23:03 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!