| rpittau | good morning ironic! o/ | 07:58 |
|---|---|---|
| rpittau | happy friday :) | 08:04 |
| opendevreview | Merged openstack/ironic-specs master: CI: Make old spec compliant with new lint https://review.opendev.org/c/openstack/ironic-specs/+/914703 | 08:06 |
| opendevreview | Merged openstack/python-ironicclient master: Fix on-creation ability https://review.opendev.org/c/openstack/python-ironicclient/+/908534 | 09:12 |
| opendevreview | Piotr Czarnik proposed openstack/sushy-tools master: Normalize relative path for emulator config file https://review.opendev.org/c/openstack/sushy-tools/+/914756 | 11:58 |
| rpittau | bye everyone have a great weekend! o/ | 15:59 |
| TheJulia | cid: mind if I rebase your ironic-tempest-plugin patch on another patch to see if that silences the microversion errors? | 16:07 |
| TheJulia | cid: Ahh, I see you +1'ed my change, I'm going to update it and then I'll put your patch on top. That should cause it to actually pass if I'm understanding the entire condundrum | 16:22 |
| cid | https://www.irccloud.com/pastebin/f2YEiarE | 16:25 |
| TheJulia | I'm thinking I should just go put the same skip check into the scenario jobs as well | 16:26 |
| cid | :D I definitely did not want to reply within a pastebin or whatever that was | 16:27 |
| TheJulia | irccloud and multiline text ends up doing that :) | 16:27 |
| cid | TheJulia: I don't mind. And yea, looking at your change, it seems it might cause the i-t-p to pass . | 16:27 |
| TheJulia | https://www.irccloud.com/pastebin/GHMkkEgc/ | 16:28 |
| TheJulia | see, easy to do! | 16:28 |
| cid | Discovery of the day | 16:28 |
| TheJulia | lol | 16:28 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: automatically set the branch for IPA to match TARGET_BRANCH https://review.opendev.org/c/openstack/ironic/+/914250 | 16:30 |
| opendevreview | Merged openstack/ironic master: Do not rely on predefined UUIDs in tests https://review.opendev.org/c/openstack/ironic/+/881485 | 16:34 |
| opendevreview | cid proposed openstack/ironic master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic/+/913793 | 16:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Leverage the remote microversion without requiring human config https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914270 | 16:56 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Add max microversion for ipmi classic drivers https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914760 | 16:56 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 16:56 |
| TheJulia | That should sort it | 16:56 |
| TheJulia | I guess lets wait and see | 16:56 |
| cid | let's wait... | 16:58 |
| TheJulia | pretty much | 17:02 |
| TheJulia | I think I'm going to go run into town for an errand while waiting | 17:03 |
| TheJulia | since I've failed to do it all week until now | 17:03 |
| cid | ack | 17:04 |
| JayF | https://www.openwall.com/lists/oss-security/2024/03/29/4 this is not really specifically ironic-relevant | 17:14 |
| JayF | but it's bad enough I've been linking it anywhere I haven't seen it discussed (yet) | 17:14 |
| TheJulia | oh jesus | 17:17 |
| TheJulia | that is very bad | 17:19 |
| TheJulia | inside the tarball distributions at that | 17:20 |
| JayF | yep | 17:23 |
| JayF | think about this in context of openstack | 17:23 |
| JayF | and our pypi maintainer project | 17:23 |
| JayF | and just let that feeling sit in the pit of your stomach for a few minutes :) | 17:23 |
| TheJulia | yeah, not liking that feeling | 17:29 |
| * JayF links that with a comment under the pypi-maintainer-cleanup ptg item | 17:31 | |
| opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Replace appdirs by platformdirs https://review.opendev.org/c/openstack/python-ironicclient/+/914762 | 17:37 |
| TheJulia | cid: doesn't look like it :( | 17:44 |
| TheJulia | I'll take a look a little later | 17:44 |
| cid | hmm. I see, the code is not working yet, but I believe the idea should work. | 17:51 |
| cid | happy weekend! o/ | 18:52 |
| JayF | o/ see you when I'm back week after next cid | 18:54 |
| cid | JayF: enjoy your holiday | 18:55 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: automatically set the branch for IPA to match TARGET_BRANCH https://review.opendev.org/c/openstack/ironic/+/914250 | 19:38 |
| TheJulia | oh, hmmm weird | 19:39 |
| TheJulia | cid: so the base challenge is because of the depends on, the test runs locally with 1.91 available on the remote, so it won't auto exclude | 19:54 |
| TheJulia | that is fine, really, but you have 4 other issues in the methods you need to fix | 19:54 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Get extra debug info when the partition image deploy fails https://review.opendev.org/c/openstack/ironic-python-agent/+/914771 | 21:15 |
| opendevreview | Merged openstack/python-ironicclient master: Replace appdirs by platformdirs https://review.opendev.org/c/openstack/python-ironicclient/+/914762 | 21:18 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Try to figure out missing /bin/sh failures with partition tests https://review.opendev.org/c/openstack/ironic/+/914772 | 21:23 |
| cid | TheJulia: I will take a look by Monday. | 22:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!