| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Schema for maintenance types https://review.opendev.org/c/openstack/ironic/+/989305 | 01:19 |
|---|---|---|
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Introduce maintenance types https://review.opendev.org/c/openstack/ironic/+/989244 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: DB-API methods for maintenance entries https://review.opendev.org/c/openstack/ironic/+/989242 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Introduce a NodeMaintenance object https://review.opendev.org/c/openstack/ironic/+/989310 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Add maintenance helpers https://review.opendev.org/c/openstack/ironic/+/989311 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Record and enforce maintenance states https://review.opendev.org/c/openstack/ironic/+/989312 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Add policy rules and microversion bump https://review.opendev.org/c/openstack/ironic/+/989313 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: API endpoints https://review.opendev.org/c/openstack/ironic/+/989314 | 01:19 |
| opendevreview | cid proposed openstack/ironic master: WIP [LOTO]: Document node maintenance/types https://review.opendev.org/c/openstack/ironic/+/989315 | 01:19 |
| opendevreview | Verification of a change to openstack/ironic unmaintained/2024.1 failed: security: Use sandbox rendering for jinja2 https://review.opendev.org/c/openstack/ironic/+/987777 | 02:15 |
| opendevreview | Bertrand Lanson proposed openstack/ironic master: fix: skip storage controllers without Volumes link https://review.opendev.org/c/openstack/ironic/+/987853 | 06:52 |
| opendevreview | Bertrand Lanson proposed openstack/ironic master: fix: skip storage controllers without Volumes link https://review.opendev.org/c/openstack/ironic/+/987853 | 06:58 |
| blanson[m] | Hello cid ! regarding https://review.opendev.org/c/openstack/ironic/+/987853/comment/13e1852b_8fcf21ec/ , do you think we should log here (even without the early return) ? | 09:02 |
| blanson[m] | just so I can make everything nice and clean | 09:03 |
| cid | Actually, yeah, we can just retain the log there | 09:05 |
| cid | Not sure what level the log will be so that it's not too noisy. | 09:05 |
| blanson[m] | I'd say warning but idk how noisy that would be considered ? | 09:05 |
| blanson[m] | it's genuinely a concerning outcome imo | 09:06 |
| cid | warning is a good start. Another core will also get to look at it, so, it's perfectly okay | 09:06 |
| opendevreview | Bertrand Lanson proposed openstack/ironic master: fix: skip storage controllers without Volumes link https://review.opendev.org/c/openstack/ironic/+/987853 | 09:29 |
| opendevreview | Ivan Anfimov proposed openstack/ironic master: wip https://review.opendev.org/c/openstack/ironic/+/989356 | 10:40 |
| *** darmach1002 is now known as darmach100 | 11:09 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add a BMO job https://review.opendev.org/c/openstack/ironic/+/988766 | 12:01 |
| opendevreview | Merged openstack/sushy-tools master: Preserve types for BIOS settings https://review.opendev.org/c/openstack/sushy-tools/+/989124 | 12:29 |
| Sandzwerg[m] | Morning ironic. I have a bug regarding the release notes. I know I saw the switch to the new RBAC model in the 2024.1 release notes and I still find the note itself https://opendev.org/openstack/ironic/src/branch/master/releasenotes/notes/change-default-rbac-policy-f2f154043910f26a.yaml but I don't find it in the release notes anymore: https://docs.openstack.org/releasenotes/ironic/2024.1.html I also checked other release notes | 12:30 |
| Sandzwerg[m] | from 2023.2 till now without seeing this. | 12:30 |
| TheJulia | good morning | 13:11 |
| TheJulia | I Interesting, those release notes seem truncated too | 13:12 |
| dtantsur | Hmm, I see that we can timeout doing soft power off, and I wonder if it's a libvirt problem or cirros or something else | 13:17 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add a BMO job https://review.opendev.org/c/openstack/ironic/+/988766 | 13:20 |
| TheJulia | soft power-off is just a signal | 13:22 |
| TheJulia | like an interrupt to the VM, the VM has to choose to respond | 13:23 |
| TheJulia | it may also be that it starts, but then doesn't make it there in time. | 13:23 |
| Sandzwerg[m] | TheJulia: Should I open a bug regarding this? | 13:35 |
| TheJulia | yes please | 13:35 |
| dtantsur | it is somewhat random though. it's the first time the servicing test failed this way for me. | 13:39 |
| TheJulia | dtantsur: I guess the question to me is did the host even register the signal and even start shutting down int he VM ? | 13:51 |
| dtantsur | TheJulia: I don't think so. I'm also quite sure that cirros wouldn't have taken long to shutdown | 13:57 |
| TheJulia | ... I wonder if there is an acpi bug in the kernel that can cause messages not to be picked up | 13:58 |
| TheJulia | I guess the thing to also do is check for anything in the bmc log | 13:58 |
| TheJulia | Folks, last week we briefly discussed software raid. https://review.opendev.org/c/openstack/ironic-python-agent/+/989009 is a WIP | 14:32 |
| dtantsur | TheJulia: do we really need it on the IPA side? I think we expose the step on the ironic side | 14:36 |
| TheJulia | great question, dunno. I was thinking IPA is as close to it as it gets, we do have the AgentRaid class if I'm recalling correctly, so I guess we could just guardrail there | 14:36 |
| TheJulia | likely better, tbh | 14:36 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Add deferred actions data model, DB API, and versioned object https://review.opendev.org/c/openstack/ironic/+/989224 | 14:45 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Deferred Actions https://review.opendev.org/c/openstack/ironic-specs/+/954612 | 14:50 |
| 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:17 |
| 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:17 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Restructure admin documentation by topic and vendor https://review.opendev.org/c/openstack/networking-generic-switch/+/981418 | 15:17 |
| TheJulia | If I could get some folks to review ^, it would be greatly appreciated. hjensas you may be of interest ^ | 15:19 |
| TheJulia | dtantsur: we could always drop "agent" from the "enabled_raid_interfaces" list by default and put some doc updates in ironic | 15:31 |
| TheJulia | Sort of more breaking on an upfront state, but also simplified escape hatch | 15:31 |
| TheJulia | Sandzwerg[m]: w/r/t the issue from the other day, did you post a patch yet? | 15:38 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Add config options, reschedule exception, and action creation helper https://review.opendev.org/c/openstack/ironic/+/989225 | 15:39 |
| JayF | TheJulia: I like that idea; but are any of the raid-based steps exposed generally for cleaning/etc? | 15:47 |
| TheJulia | it all gets filtered through the attachment of the interface to the driver | 15:47 |
| JayF | I guess we don't have an active security issue here where we'd need to restrict it, but if the idea is to reduce attack surface, is disabling the official interface enough? | 15:47 |
| JayF | oh, good | 15:47 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a Metal3 baremetal-operator functional job https://review.opendev.org/c/openstack/ironic/+/988766 | 15:50 |
| TheJulia | yeah, begin to message "you may want this, but the barriers are getting bigger and its sort of outside of our control" | 15:51 |
| JayF | "This requires significant manual operator configuration and testing, changing the default is reflective of that." | 15:52 |
| TheJulia | ++ | 15:52 |
| TheJulia | I'd update the docs to be a bit more verbose | 15:52 |
| TheJulia | but yeah | 15:52 |
| opendevreview | Merged openstack/ironic-ui master: Restore explicit package list for module detection https://review.opendev.org/c/openstack/ironic-ui/+/988997 | 15:57 |
| opendevreview | Merged openstack/ironic stable/2025.1: [stable-only][ci] Remove non-voting, obsolete jobs https://review.opendev.org/c/openstack/ironic/+/989215 | 16:04 |
| opendevreview | Verification of a change to openstack/ironic unmaintained/2024.1 failed: security: move file url validation up into deploy_utils main path https://review.opendev.org/c/openstack/ironic/+/988359 | 16:04 |
| opendevreview | Verification of a change to openstack/ironic unmaintained/2024.1 failed: Shell-quote console command passed to `socat` https://review.opendev.org/c/openstack/ironic/+/986417 | 16:57 |
| opendevreview | Merged openstack/networking-generic-switch master: Align cover tox env with stestr-based test runner https://review.opendev.org/c/openstack/networking-generic-switch/+/988222 | 17:09 |
| opendevreview | Sebastian Lohff proposed openstack/ironic master: Use service auth for Neutron port operations https://review.opendev.org/c/openstack/ironic/+/989431 | 17:16 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: WIP: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 17:31 |
| JayF | TheJulia: ^ early feedback before we finish it? | 17:35 |
| TheJulia | overall, looks on the right track. I'd just make sure there is not any place else where _install_grub2 gets called | 17:37 |
| JayF | ++ | 17:38 |
| JayF | you agree with wiring conf thru to conductor? | 17:38 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: DNM: testing hierarchical port binding https://review.opendev.org/c/openstack/networking-baremetal/+/989433 | 17:39 |
| TheJulia | Yeah, it really means backporting a config option on the ironic side as well, but it is what it is | 17:49 |
| JayF | it's really the only way to do this change as an OSSA | 17:49 |
| TheJulia | yeah | 17:52 |
| TheJulia | it is what it is | 17:52 |
| opendevreview | Merged openstack/tenks master: Removes VirtualPDU from supported emulators list https://review.opendev.org/c/openstack/tenks/+/988681 | 17:58 |
| Sandzwerg[m] | TheJulia: Done. https://bugs.launchpad.net/ironic/+bug/2153436 | 21:27 |
| TheJulia | awesome | 21:27 |
| Sandzwerg[m] | TheJulia: That's this one: https://review.opendev.org/c/openstack/ironic/+/989431 | 21:29 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!