| *** bodgix2 is now known as bodgix | 07:20 | |
| *** mmalchuk_ is now known as mmalchuk | 07:20 | |
| *** edebeste1 is now known as edebeste | 07:25 | |
| iurygregory | good morning Ironic | 11:16 |
|---|---|---|
| TheJulia | good morning | 13:08 |
| iurygregory | good morning TheJulia =) | 13:08 |
| TheJulia | brraaaains | 13:24 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 13:48 |
| JayF | https://github.com/seekbytes/IPA up to like, 14 different things acronym'd IPA :D | 15:37 |
| JayF | an IPA (our IPA :D) job just failed with > 2024-08-28 15:28:49.747168 | controller | /bin/sh: 1: osv-vsctl: not found | 15:43 |
| JayF | this is a new one to me; has anyone else seen it before? | 15:43 |
| JayF | eh, they all didn't seem to fail the same way | 15:45 |
| TheJulia | osv-vsctl seems like a typo | 15:45 |
| TheJulia | ovs-vsctl | 15:45 |
| TheJulia | bad devstack patch maybe? | 15:46 |
| JayF | oooh | 15:47 |
| TheJulia | did I create a lightbulb ? | 15:47 |
| JayF | no, this is not a patch editing anything ipa | 15:47 |
| JayF | but I didn't notice it was osv not ovs | 15:47 |
| JayF | I rechecked it b/c I saw the earlier change didn't fail that same way | 15:47 |
| JayF | so WTF was many | 15:47 |
| TheJulia | parallel WTFs | 15:47 |
| iurygregory | https://opendev.org/openstack/ironic/src/branch/master/playbooks/ci-workarounds/get_extra_logging.yaml#L29 | 15:48 |
| JayF | yeah, I found it too | 15:48 |
| TheJulia | doh! | 15:48 |
| JayF | that's not the failure cause, but an easy fix | 15:48 |
| JayF | I will fix it | 15:48 |
| iurygregory | happy to +2 | 15:48 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Use the correct command to fetch ovs state https://review.opendev.org/c/openstack/ironic/+/927408 | 15:50 |
| iurygregory | if you want I can +W, since is a simple change | 16:04 |
| TheJulia | workflowed | 16:05 |
| cardoe | dtantsur: hopefully that's what you're looking for. I did want to talk to ask you about the ironic operator you're working on and see if you published something around what its scope will be | 16:09 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Check for the existence of an IPMI device https://review.opendev.org/c/openstack/ironic-python-agent/+/926973 | 18:26 |
| opendevreview | Merged openstack/ironic master: Use the correct command to fetch ovs state https://review.opendev.org/c/openstack/ironic/+/927408 | 19:09 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Add inspection rules https://review.opendev.org/c/openstack/ironic/+/918303 | 19:44 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Add inspection rules https://review.opendev.org/c/openstack/ironic/+/918303 | 19:47 |
| iurygregory | I would say is worth backporting https://review.opendev.org/c/openstack/ironic/+/927408 | 19:55 |
| TheJulia | eh, we don't have to troubleshoot those issues often, and when we do it is typically master branch | 20:03 |
| iurygregory | oh, I thought we would also see the issue on stable branches | 20:08 |
| TheJulia | those commands failing should never be fatal | 20:08 |
| iurygregory | ohhh ok! | 20:13 |
| JayF | yeah, that was me seeing an error and stopping | 20:26 |
| JayF | the original CI failure I saw that on, we tracked to an actual-valid-ci-failure | 20:26 |
| JayF | cid: https://review.opendev.org/c/openstack/ironic-python-agent/+/926973 if you can delete the two lines I indicated outta the docstring, I can +2 this | 20:29 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Check for the existence of an IPMI device https://review.opendev.org/c/openstack/ironic-python-agent/+/926973 | 20:46 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Check for the existence of an IPMI device https://review.opendev.org/c/openstack/ironic-python-agent/+/926973 | 20:47 |
| cid | JayF: patch updated ^^ | 20:48 |
| JayF | awesome, ty | 20:48 |
| cid | Good night ironic o/ | 20:48 |
| cid | ++ | 20:48 |
| JayF | have a good one o/ | 20:48 |
| cid | TY | 20:48 |
| JayF | that's going to be a nice QOL fix to get in \o/ | 20:49 |
| JayF | This cycle we've had a couple of things to really speed up IPA; this and the change I put in earlier that caches the result of evaluate_hardware_support | 20:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!