opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP Add HTTPBoot support https://review.opendev.org/c/openstack/ironic-specs/+/896474 | 00:39 |
---|---|---|
TheJulia | muahahahahahaha | 00:39 |
gmann | JayF: heads up for tempest release things, this change will fail ironic jobs https://review.opendev.org/c/openstack/tempest/+/895985/3 and this will pass them again https://review.opendev.org/c/openstack/tempest/+/895986/3 | 01:18 |
gmann | JayF: we need these steps for tempest release, we try to approve/merge them together but in case there is window in between of these two change merging, you can see ironic job failing and just wait for 895986 to merge | 01:19 |
JayF | thank you for the heads up | 02:33 |
iurygregory | bye everyone, see you on Wednesday since tomorrow I'm traveling to give a talk =) | 02:37 |
rpittau | good morning ironic! o/ | 07:27 |
TheJulia | good morning! | 13:48 |
opendevreview | Julia Kreger proposed openstack/ironic master: Kickstart: Don't error unit tests ksvalidate is present https://review.opendev.org/c/openstack/ironic/+/896469 | 14:11 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Assume the parent node needs to be powered on https://review.opendev.org/c/openstack/ironic/+/896570 | 16:34 |
TheJulia | dtantsur: w/r/t power on of DPU topics, ^ | 16:35 |
TheJulia | seems... simple... enough. #FamousLastWords | 16:35 |
JayF | this seems like it might be a nephew to the "devices you can't power off" | 16:35 |
JayF | (the topic for the ptg) | 16:36 |
TheJulia | Eh, that is a good point, power off can cascade | 16:36 |
JayF | Yeah; I'm all for simple but for power it seems like we need some guardrails, too | 16:37 |
JayF | I guess locking protects most of those cases? | 16:37 |
JayF | Hmmm. Child node is updating firmware. I call parent node power off API explicitly. What happens? | 16:38 |
TheJulia | Yeah, it should, we likely need to have at least a stance on power off. If someone bypasses, it could become “interesting” regardless | 16:38 |
TheJulia | Well, firmware should be out of band, but yeah, you can brick one of the firmware sets easily in that case, it should go to secondary, in theory | 16:39 |
JayF | well your comment around this | 16:39 |
JayF | is that power off parent node has a side effect of power off child node | 16:39 |
TheJulia | But yeah, what a fun topic! | 16:39 |
TheJulia | Yup | 16:39 |
JayF | so "firmware should be oob" may not matter in that case? | 16:39 |
JayF | if we just yank the plug from the device, the band stops playing entirely :P | 16:40 |
TheJulia | Yeah, but definitely a thread to pull on | 16:40 |
TheJulia | Yeah, yay for new tech | 16:40 |
JayF | especially since our driver model allows a power action in that case to be (unlikely but possible) SNMP to a PDU | 16:40 |
* JayF gets the dog on a walk before rain + before TC meeting; bbs | 16:41 | |
TheJulia | ++++++ | 16:41 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: WIP add disk aliases to the inventory https://review.opendev.org/c/openstack/ironic-python-agent/+/896577 | 17:01 |
JayF | btw; had a nice chat with someone over zoom yesterday about Ironic | 17:01 |
JayF | I was going to say, if you see Kangie be nice to him; but they already joined so the gig is up! | 17:02 |
TheJulia | Woot | 17:02 |
opendevreview | Merged openstack/metalsmith stable/xena: stable-only: Constrain the upper Ansible version https://review.opendev.org/c/openstack/metalsmith/+/895672 | 19:42 |
opendevreview | Merged openstack/metalsmith stable/wallaby: stable-only: Constrain the upper Ansible version https://review.opendev.org/c/openstack/metalsmith/+/895673 | 19:42 |
Kangie | :) | 20:44 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Migrate completed specs to release version https://review.opendev.org/c/openstack/ironic-specs/+/896597 | 22:10 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Migrate completed specs to release version https://review.opendev.org/c/openstack/ironic-specs/+/896597 | 22:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!