| opendevreview | Steve Baker proposed openstack/ironic master: docs: troubleshooting addition for sync period https://review.opendev.org/c/openstack/ironic/+/908342 | 01:06 |
|---|---|---|
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 01:51 |
| opendevreview | Merged openstack/ironic-inspector stable/zed: Handle LLDP parse Unicode error https://review.opendev.org/c/openstack/ironic-inspector/+/905443 | 02:25 |
| opendevreview | Merged openstack/ironic master: docs: troubleshooting addition for sync period https://review.opendev.org/c/openstack/ironic/+/908342 | 02:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Log upon completion of power sync https://review.opendev.org/c/openstack/ironic/+/891334 | 02:51 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.1: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 04:44 |
| opendevreview | Merged openstack/ironic master: Log upon completion of power sync https://review.opendev.org/c/openstack/ironic/+/891334 | 05:59 |
| rpittau | good morning ironic! o/ | 07:52 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 08:26 |
| opendevreview | Alex Welsh proposed openstack/tenks master: Fix Python dependencies https://review.opendev.org/c/openstack/tenks/+/911086 | 11:13 |
| opendevreview | Alex Welsh proposed openstack/tenks master: Fix Python dependencies https://review.opendev.org/c/openstack/tenks/+/911086 | 11:19 |
| dtantsur | Could I get some reviews on https://review.opendev.org/c/openstack/ironic/+/907991 please? It's quite important to allow people to use the new inspection, so I'd like to have it in this release. | 11:53 |
| iurygregory | good morning Ironic | 11:54 |
| iurygregory | dtantsur, I will add to my list | 11:54 |
| dtantsur | morning, thank you! | 11:54 |
| opendevreview | cid proposed openstack/ironic master: Fix data length exceeding limit error https://review.opendev.org/c/openstack/ironic/+/910973 | 12:08 |
| Sandzwerg[m] | morning ironic o/ | 12:27 |
| Sandzwerg[m] | Does someone knows what is blocking https://bugs.launchpad.net/ironic/+bug/2039458 and if only the change on sushy side needs to be merged, or if there are other changes to ironic needed? | 12:27 |
| dtantsur | iurygregory: ^^ | 12:28 |
| dtantsur | Sandzwerg[m]: I believe an Ironic change will be needed | 12:28 |
| iurygregory | Sandzwerg[m], yes we will need changes in ironic to handle it | 12:29 |
| iurygregory | Winicius is working on it, I think monday we will have a patch up for review | 12:29 |
| dtantsur | iurygregory: do you think there is a way to make this patch backportable? | 12:30 |
| dtantsur | As in, it can degrade gracefully if old sushy is present. | 12:30 |
| Sandzwerg[m] | I see. Because we ran into this with Lenovo and metal³ but then we need to use ipxie for that till it's fixed | 12:31 |
| iurygregory | dtantsur, I think so | 12:31 |
| dtantsur | yeah, this topic seems to be heating up | 12:33 |
| Sandzwerg[m] | Would that fix work in setup where hardware with the old path and the new path are both present? | 12:34 |
| iurygregory | while talking with him the idea would be to move the logic in the def _insert_vmedia/_eject_vmedia to be able to handle based on the sushy resource (System/Manager), and we would check if vmedia is avaiable trough System or not (to choose how ironic would proceed) | 12:34 |
| iurygregory | Sandzwerg[m], yup | 12:34 |
| dtantsur | Sandzwerg[m]: my assumption is that we'll try System first and fallback to Manager if it does not have VirtualMedia (or the version of sushy is old) | 12:34 |
| iurygregory | we would try to identify in ironic if the Hardware has vmedia in System or Manager and use the one available | 12:34 |
| Sandzwerg[m] | Nice :) | 12:34 |
| iurygregory | dtantsur, I was thinking a bit more conservative, we try System only if is available, if it's it wouldn't fall back to manager.. | 12:36 |
| Sandzwerg[m] | Great, that would be my idea as well but the change in sushy tools seemed the change the path completely but I'm not familiar enough with the code and how it interacts with the other parts to be sure if that matters or not | 12:36 |
| iurygregory | dtantsur, this was like I was discussing with him yesterday https://paste.opendev.org/show/bpIchqQy4xAFilc93wy6/ | 12:37 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 12:38 |
| dtantsur | iurygregory: roughly like this (modulo handling the normal attributeerror and not duplicating get_system) | 12:40 |
| opendevreview | Merged openstack/sushy-tools master: Force constraints when installing a package during tox test https://review.opendev.org/c/openstack/sushy-tools/+/908789 | 13:00 |
| iurygregory | ack | 13:04 |
| opendevreview | Merged openstack/networking-generic-switch master: Replace usage of deprecated path for test decorators https://review.opendev.org/c/openstack/networking-generic-switch/+/908550 | 13:20 |
| TheJulia | good morning | 14:16 |
| iurygregory | good morning TheJulia =) | 14:40 |
| TheJulia | \o/ the lenovo fix has been confirmed to work | 14:43 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Verify agent TLS certificate and provide diagnostics https://review.opendev.org/c/openstack/ironic/+/911116 | 14:50 |
| dtantsur | JayF: on the topic of the clock skew ^^^ | 14:50 |
| dtantsur | TheJulia: now we only need to fix virtual media for them :) | 14:51 |
| opendevreview | cid proposed openstack/ironic master: Fix multiple assignment of redfish_system_id during node creation https://review.opendev.org/c/openstack/ironic/+/909851 | 15:19 |
| TheJulia | https://twitter.com/ipxe/status/1765011419173425580 | 15:50 |
| opendevreview | Merged openstack/ironic stable/2023.1: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 15:56 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.1: ci: pin CI to dnsmasq 2.85 https://review.opendev.org/c/openstack/ironic/+/910679 | 17:03 |
| rpittau | good night! o/ | 17:27 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/zed: stable only/ci: pin CI to dnsmasq 2.85/pin proliantutils/scciclient https://review.opendev.org/c/openstack/ironic/+/911158 | 17:39 |
| dtantsur | TheJulia: \o/ re iPXE | 17:40 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/xena: stable only/ci: pin CI to dnsmasq 2.85/pin proliantutils https://review.opendev.org/c/openstack/ironic/+/911159 | 17:43 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: stable only/ci: pin CI to dnsmasq 2.85/pin proliantutils https://review.opendev.org/c/openstack/ironic/+/911160 | 17:44 |
| TheJulia | okay, I think that should at least get CI in a semi-working state back to wallaby | 17:44 |
| TheJulia | hmm, I thought I put a reno on https://review.opendev.org/c/openstack/ironic/+/896570... six months ago :( | 17:56 |
| JayF | FYI: The Ironic docs stuff we talked about GR-OSS funding a while ago is kicking off today. We're hoping to have a report with actions to improve our documentation ready by the PTG. | 18:33 |
| fungi | do the consistent ironic-tempest-bios-ipmi-direct-tinyipa failures on devstack's stable/zed branch indicate a missing backport for some known fix? e.g. https://zuul.opendev.org/t/openstack/build/75ea9330a04f4f129d30ad5bc50e775f | 18:36 |
| JayF | it's likely the dnsmasq issue | 18:38 |
| JayF | which we're backporting | 18:38 |
| JayF | ^^^ 911158 above | 18:38 |
| fungi | okay cool. so that explains why it wasn't impacting later branches | 18:38 |
| fungi | or rather isn't now | 18:38 |
| JayF | well, it's flakey :) | 18:39 |
| JayF | 911158 is landing now | 18:39 |
| fungi | aha, thanks! | 18:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Handle Power On/Off for child node cases https://review.opendev.org/c/openstack/ironic/+/896570 | 19:17 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.2: ci: stable-only: remove metal3-integration ci job https://review.opendev.org/c/openstack/ironic/+/910537 | 19:22 |
| JayF | How would contributors feel about getting some issues filed in our bugtracker that are actionable documentation improvements? ala https://github.com/etcd-io/website/issues/766 except openstack flavored cc: TheJulia | 19:28 |
| TheJulia | JayF: https://51bb6dbe041694cdacc4-bbe0f194e8a12aaea6b359f6bab3a11a.ssl.cf2.rackcdn.com/911158/1/check/ironic-tempest-functional-python3/a041285/testr_results.html | 19:34 |
| TheJulia | that is against zed | 19:34 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/zed: stable only/ci: pin CI to dnsmasq 2.85/pin proliantutils/scciclient https://review.opendev.org/c/openstack/ironic/+/911158 | 19:43 |
| TheJulia | JayF: ^^^ hopefully to pass CI in some shape, I filed a bug for the tempest issue | 19:44 |
| JayF | +2A on 911158 | 19:55 |
| JayF | TheJulia: you have time for a outta-left-field question? | 19:55 |
| TheJulia | not really | 19:55 |
| JayF | ack | 19:56 |
| JayF | I'll ask in a larger context then :) | 19:56 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: ci: pin CI to dnsmasq 2.85 https://review.opendev.org/c/openstack/ironic/+/910679 | 20:39 |
| JayF | are ya kidding me | 20:40 |
| JayF | unit test failure | 20:40 |
| JayF | lolsob > ironic.tests.unit.drivers.modules.network.test_common.TestNeutronVifPortIDMixin.test_port_changed_client_id_fail | 20:40 |
| JayF | I thought we removed that test | 20:41 |
| * JayF recheck | 20:41 | |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: ci: pin CI to dnsmasq 2.85 https://review.opendev.org/c/openstack/ironic/+/910679 | 20:48 |
| JayF | that's arm64 unit tests passing | 20:50 |
| TheJulia | oh good | 20:50 |
| * TheJulia returns to meeting | 20:51 | |
| TheJulia | so stable/zed fix, looks like standalone job failed on it due to software raid, specifically neutron port binding failed | 21:17 |
| TheJulia | which is unrelated, but the second time it has failed | 21:19 |
| JayF | I am +1 to -nv'ing jobs, generally, on stable/zed | 21:19 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/zed: stable only/ci: pin CI to dnsmasq 2.85/pin proliantutils/scciclient https://review.opendev.org/c/openstack/ironic/+/911158 | 21:23 |
| opendevreview | Merged openstack/ironic stable/2023.1: ci: pin CI to dnsmasq 2.85 https://review.opendev.org/c/openstack/ironic/+/910679 | 23:33 |
| TheJulia | woot | 23:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!