| opendevreview | Takashi Kajinami proposed openstack/sushy-tools master: Drop redundant variable definition https://review.opendev.org/c/openstack/sushy-tools/+/953162 | 01:24 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:39 |
| opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 08:15 |
| queensly[m] | Good morning | 09:04 |
| rpittau | I would have voted Godzilla for G release :) | 12:45 |
| TheJulia | ++ | 13:34 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Add Mellanox Spectrum to firmware to remove list https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/952756 | 13:41 |
| dking | Is anybody here from the Metal3 team? | 13:54 |
| masghar | dking: present, among others :) | 13:57 |
| dking | masghar: Thanks! I have a couple of questions that I've been meaning to ask. So, we're using CAPI/BMO/Ironic to deploy our servers. We have a task that we would like to trigger (simple GET HTTP request, so a curl would be fine) upon deprovision before the server goes into cleaning with the IPA image. Is there a good hook somewhere for that sort of thing? | 14:00 |
| TheJulia | I guess it could be a automated cleaning step in a hardware manager... | 14:11 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: WIP Add Tempest tests for inspection rules in Ironic https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951761 | 14:26 |
| dtantsur | dking: it's either what TheJulia suggested or your own controller that listens for BareMetalHost object changes | 14:29 |
| dking | TheJulia: I'm already doing that, but I want it to happen before it reaches IPA as it affects the PXE boot. | 14:30 |
| dking | dtantsur: Okay. I thought I would ask about any pre-existing hooks before I tried to make my own controller or sidecar. | 14:31 |
| dtantsur | dking: the canonical way would be a new deploy interface with a new deploy step that runs before deploy.deploy. But BMO does not allow changing deploy interfaces for hosts. | 14:33 |
| dking | dtantsur: I'm not sure. I'm attempting a step at deprovision, rather than at deploy. | 14:42 |
| 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 | 15:02 |
| dtantsur | dking: right, so I guess I meant a clean step.. but we currently boot IPA unconditionally first | 15:02 |
| 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 | 15:04 |
| dking | dtantsur: Yeah, it would have to be before IPA. I'm attempting to reconfigure switch ports which would impact PXE booting, so it would be best if it could be done prior to the IPA image, or even the same time as (because a lot of servers take a little time to get around to actually PXE). | 15:05 |
| dtantsur | Ah, so our standalone networking effort may help you one day | 15:26 |
| dtantsur | until then, you may as well just hack something together | 15:26 |
| opendevreview | Takashi Kajinami proposed openstack/sushy-tools master: Switch to pre-commit https://review.opendev.org/c/openstack/sushy-tools/+/953126 | 15:37 |
| dking | dtantsur: I forgot about my other question. When we cycle through machines, something, and I think it's Baremetal Operator, not Ironic, makes them reboot between cleaning and provisioning. Is there a way to disable that so that machines can be deprovisioned and go directly into provisioning without needing a power down? | 16:47 |
| dtantsur | dking: I'm not sure I've heard about this behavior. Worth an upstream bug with logs. | 16:59 |
| dking | dtantsur: Okay, so it's not intended? I'll have to look at it again when I re-provision some servers, but I think what happens is that somehow after a deprovision, a server is marked offline somewhere before it gets a provision state. This sounds like an honest assumption somebody could have been made about what was expected, but as we're not wanting to actually remain deprovisioned (available?), we don't actually want the st | 17:03 |
| dking | e to go to offline. | 17:03 |
| dtantsur | dking: you have online=true on the BMH, right? | 17:03 |
| dking | dtantsur: I have to check again, but I think that's what's getting set to false. | 17:05 |
| dtantsur | hmm, I don't think we ever change that | 17:06 |
| TheJulia | final call on https://review.opendev.org/c/openstack/ironic-specs/+/945642 | 17:49 |
| TheJulia | of note, the following specs have one +2 and need another core reviewer: https://review.opendev.org/c/openstack/ironic-specs/+/940861 https://review.opendev.org/c/openstack/ironic-specs/+/952126 | 17:49 |
| TheJulia | adamcarthur5: Are you hoping to revise https://review.opendev.org/c/openstack/ironic-specs/+/952533 this week? | 17:50 |
| adamcarthur5 | TheJulia Yes, apologies, it's been full-on with my PhD this week. I'll shoot for some revisions tomorrow | 17:52 |
| TheJulia | adamcarthur5: no worries! I figured but just wanted to follow-up :) | 17:55 |
| adamcarthur5 | Appreciate it, thank you :)) | 17:56 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/11.0: Update .gitreview for bugfix/11.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/952568 | 19:29 |
| guilhermesp | dtantsur: hey! got some time to test your pr here | 20:26 |
| guilhermesp | i ended up forgetting to add ipa-debug to the kernel parameters here my bad -- but to let you know that fails the same way. Will get you a full traceback with debug | 20:27 |
| guilhermesp | dtantsur: here you go https://paste.openstack.org/raw/bYN654bisQDkqhjnRyRg/ -- let me know if there`s anything missing and thank you! | 20:49 |
| opendevreview | cid proposed openstack/sushy master: Add retry logic to connection failures https://review.opendev.org/c/openstack/sushy/+/953248 | 21:07 |
| cardoe | Are we good with Python 3.9 and up syntax for the Ironic code base? | 22:41 |
| cardoe | I don't intend on backporting any of this. | 22:41 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix port vif attach with networks with dynamic segments https://review.opendev.org/c/openstack/ironic/+/952168 | 23:33 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!