| opendevreview | Verification of a change to openstack/ironic master failed: Avoid import-time AttributeError with PySNMP 7.x https://review.opendev.org/c/openstack/ironic/+/976275 | 07:20 |
|---|---|---|
| frickler | looks like the ironic tox jobs are broken the same way, fun https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py313&project=openstack/ironic | 07:31 |
| iurygregory | good morning ironic | 10:36 |
| iurygregory | metal3 issue still not fixed, our CI is currently busted .-. | 10:38 |
| opendevreview | nidhi proposed openstack/ironic master: Add storage controller discovery to Redfish inspection https://review.opendev.org/c/openstack/ironic/+/976626 | 11:00 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: be specific on exception checks https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 14:24 |
| iurygregory | meeting time? <eyes> | 15:02 |
| cid | I doubt we have quorum. Let's see though | 15:04 |
| TheJulia | good morning | 15:05 |
| TheJulia | Yeah, I was just thinking about 10 minutes ago, we definitely don't have quorum | 15:05 |
| TheJulia | That being said, we can still do some stuff surely like fix CI | 15:06 |
| iurygregory | yeahh | 15:06 |
| iurygregory | metal3 broke us .-. | 15:06 |
| iurygregory | and their fix doesn't seem to be passing their CI https://github.com/metal3-io/metal3-dev-env/pull/1637 | 15:06 |
| TheJulia | So, step 0, likely disable the metal3 job for now | 15:08 |
| iurygregory | yeah, I was thinking the same | 15:08 |
| TheJulia | That being said, it sounds like we have a compound breakage right now | 15:08 |
| iurygregory | ohhh | 15:09 |
| TheJulia | Does anyone know what else has changed which has resulted in mass unhappiness with ironic.tests.unit.drivers.modules.redfish.test_firmware.RedfishFirmwareTestCase.test_update_bmc_with_explicit_wait | 15:09 |
| TheJulia | as frickler asked earlier, looks like its brand new and I *can* reproduce it locally | 15:09 |
| iurygregory | let me see | 15:09 |
| iurygregory | last change on the test was 3 months ago | 15:14 |
| opendevreview | Takashi Kajinami proposed openstack/networking-generic-switch master: Remove unused sphinxcontrib-seqdiag https://review.opendev.org/c/openstack/networking-generic-switch/+/976988 | 15:15 |
| TheJulia | its not the test, its dependencies it seems | 15:15 |
| iurygregory | yeah | 15:15 |
| iurygregory | The following tests exited without returning a status and likely segfaulted or crashed Python: | 15:15 |
| iurygregory | maybe something bumped and is causing trouble on us .-. | 15:15 |
| TheJulia | My pip freeze... https://www.irccloud.com/pastebin/OKtpKctU/ | 15:18 |
| iurygregory | https://paste.opendev.org/show/bynpzBwbhVch9vA7KP9b/ | 15:23 |
| TheJulia | possibiliites: https://paste.opendev.org/show/bP01pEgUYgRHZqRIPXaE/ | 15:36 |
| Sandzwerg[m] | [setuptools==82.0.0 breaks pkg_](https://setuptools.pypa.io/en/stable/history.html#v82-0-0) | 15:38 |
| TheJulia | yup | 15:38 |
| Sandzwerg[m] | s/[setuptools==82.0.0 breaks pkg_](// | 15:38 |
| TheJulia | its also the literally only thing which is recent | 15:38 |
| TheJulia | But that doesn't explain why the runtime crashes | 15:41 |
| TheJulia | and forcing it downward has no effect | 15:42 |
| TheJulia | I'm starting to wonder if its the test itself | 15:58 |
| TheJulia | so my gut feeling is the test doesn't mock enough of the lower levels | 16:14 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Fix the broken test?!? https://review.opendev.org/c/openstack/ironic/+/976999 | 16:25 |
| TheJulia | so I think it was getting hung up internally in sushy due to lack of mocking, that doesn't explain why now.... | 16:26 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix CI - One unit/One CI job https://review.opendev.org/c/openstack/ironic/+/976999 | 16:55 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Re-enable metal3-integration https://review.opendev.org/c/openstack/ironic/+/977006 | 16:55 |
| TheJulia | Iury +2+A'ed the unit test fix, so we should be good to go once that merges | 17:06 |
| TheJulia | Granted, we need to be super careful until the metal3 job is back up and running | 17:06 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [WIP] PLDM channel not initialized after BMC update https://review.opendev.org/c/openstack/ironic/+/976706 | 17:10 |
| opendevreview | Merged openstack/ironic master: Fix CI - One unit/One CI job https://review.opendev.org/c/openstack/ironic/+/976999 | 20:14 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: CI: Temporary setuptools pin https://review.opendev.org/c/openstack/networking-generic-switch/+/976728 | 20:31 |
| TheJulia | if I can get a review on https://review.opendev.org/c/openstack/networking-generic-switch/+/976728?usp=search to get n-g-s into a passing shape so we can hopefully land some code there, it would be much appreciated | 20:33 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: Add a reconciler to fix bug 1995078 https://review.opendev.org/c/openstack/networking-baremetal/+/975333 | 21:25 |
| TheJulia | I have rechecked a few already approved fixes | 21:37 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add an autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 22:02 |
| opendevreview | Merged openstack/networking-generic-switch master: CI: Temporary setuptools pin https://review.opendev.org/c/openstack/networking-generic-switch/+/976728 | 23:01 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!