| *** pmannidi|AFK is now known as pmannidi | 00:14 | |
| rpittau | good morning ironic! o/ | 08:38 |
|---|---|---|
| dtantsur | good morning | 08:39 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/xena: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829660 | 08:53 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/19.0: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829637 | 08:54 |
| dtantsur | I'll backport a couple of branches at a time to catch errors | 08:54 |
| rpittau | sounds good | 08:54 |
| iurygregory | good morning Ironic o/ | 09:34 |
| opendevreview | Ameya Raut proposed openstack/ironic-tempest-plugin master: Add iDRAC BIOS cleaning steps tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/829665 | 09:49 |
| opendevreview | Ameya Raut proposed openstack/ironic-tempest-plugin master: Add iDRAC BIOS cleaning steps tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/829665 | 09:57 |
| janders | hey rpittau dtantsur iurygregory and Ironic o/ | 10:43 |
| rpittau | hey janders :) | 10:44 |
| MahnoorAsghar | o/ | 11:05 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/18.1: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829678 | 11:52 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/wallaby: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829679 | 11:52 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/victoria: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829680 | 11:52 |
| iurygregory | dtantsur, hey if you have some minutes can you review https://review.opendev.org/c/openstack/metalsmith/+/827219 so we can include in the release for Yoga? | 12:25 |
| iurygregory | or we can just move on and later we can backport things etc ? | 12:25 |
| dtantsur | looking | 12:26 |
| dtantsur | iurygregory: +A | 12:28 |
| iurygregory | dtantsur, tks! | 12:28 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-lib master: Follow-up Allow customizing roles to use JSON RPC https://review.opendev.org/c/openstack/ironic-lib/+/829688 | 12:56 |
| opendevreview | Merged openstack/sushy master: Stop logging MessageRegistry objects https://review.opendev.org/c/openstack/sushy/+/825071 | 13:15 |
| opendevreview | Merged openstack/metalsmith master: Allow both 'network' and 'subnet' in NIC https://review.opendev.org/c/openstack/metalsmith/+/827219 | 13:53 |
| dtantsur | iurygregory, rpittau, https://review.opendev.org/q/I15189e7f5928126c6b336b1416ce6408a4950062 seems to behave well and is ready for reviews | 13:54 |
| iurygregory | dtantsur, ack looking now | 13:55 |
| iurygregory | 4/5 green WOW | 13:55 |
| * iurygregory gives cookies to zuul | 13:55 | |
| dtantsur | I'm impressed myself :D zuul++ | 13:55 |
| iurygregory | new t-shirt cookies for zuul <3 | 13:56 |
| TheJulia | good morning | 13:58 |
| iurygregory | I've spent 1min trying to find why the commit message wasn't showing cherry-pick =X | 13:58 |
| iurygregory | good morning TheJulia =) | 13:58 |
| TheJulia | Wonderful to see nova CI broken :\ | 13:58 |
| dtantsur | yay | 13:58 |
| dtantsur | good morning | 13:58 |
| TheJulia | I realized fixing the hash ring/cache stuff in nova allowed me to fix another outstanding bug last night | 13:59 |
| iurygregory | that's a good thing :D | 14:01 |
| TheJulia | yeah | 14:01 |
| iurygregory | about fixing other bug, not the fact that their CI is broken... | 14:01 |
| iurygregory | XD | 14:01 |
| iurygregory | who is looking forward for the long weekend? o/ | 14:02 |
| dtantsur | people who have the long weekend, I assume? :D | 14:05 |
| TheJulia | I definitely am | 14:06 |
| ajya | Hi dtantsur is [1] missing in this comment https://review.opendev.org/c/openstack/ironic/+/823312/4/ironic/drivers/modules/redfish/raid.py#1041 ? I also couldn't find an example I can follow. | 14:06 |
| dtantsur | ajya: I mean, self._submit_create_configuration(....)[1] | 14:07 |
| TheJulia | oh, well that is depressing | 14:07 |
| dtantsur | to fetch the 2nd component of the resulting tuple | 14:07 |
| ajya | ok, got it | 14:07 |
| TheJulia | Hmm, for some reason I thought it was a four day weekend not a 3 day weekend, but not according to calendars | 14:08 |
| TheJulia | :( | 14:08 |
| TheJulia | Had to work on monday regardless | 14:08 |
| dtantsur | you have a holiday there? | 14:09 |
| TheJulia | Federal holiday on the 21st, but looks like they replaced the holiday with the recharge day | 14:10 |
| iurygregory | huh?! | 14:10 |
| iurygregory | O.o | 14:10 |
| MahnoorAsghar | why work on a recharge day | 14:10 |
| TheJulia | MahnoorAsghar: the recharge day we have is tomorrow | 14:11 |
| MahnoorAsghar | ah | 14:11 |
| TheJulia | ya know what | 14:11 |
| TheJulia | I'm looking at the wrong year's schedule | 14:11 |
| MahnoorAsghar | xDDD | 14:11 |
| TheJulia | this is what happens when you don't maintain wikis | 14:12 |
| iurygregory | it happens | 14:13 |
| iurygregory | and you are still trying to wake up =) | 14:13 |
| TheJulia | indeed | 14:15 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix Redfish RAID for non-immediate controllers https://review.opendev.org/c/openstack/ironic/+/823312 | 14:16 |
| TheJulia | hmm | 14:18 |
| TheJulia | I think we broke ipa builder jobs somewhere along the way | 14:18 |
| TheJulia | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b52/829391/4/check/ironic-standalone-ipa-src/b52f686/controller/logs/ironic-bm-logs/node-1_no_ansi_2022-02-17-09%3A45%3A22_log.txt | 14:18 |
| iurygregory | I'm getting Not Found | 14:19 |
| dtantsur | I can open, probably your IRC client | 14:19 |
| dtantsur | yeah, this looks like an attempt to deploy a Cirros image that was not built by my script | 14:20 |
| dtantsur | TheJulia: have a link to all zuul results? | 14:21 |
| TheJulia | https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/829391 <-- this is the change | 14:22 |
| dtantsur | iniset /opt/stack/tempest/etc/tempest.conf baremetal default_boot_option netboot | 14:23 |
| opendevreview | Verification of a change to openstack/ironic-lib master failed: Add platform:base-py[2|3] to bindep https://review.opendev.org/c/openstack/ironic-lib/+/829272 | 14:23 |
| dtantsur | ehmmm, why is it using grub? Oo | 14:23 |
| TheJulia | did we change tempest? | 14:24 |
| TheJulia | nope | 14:24 |
| dtantsur | the tempest plugin handles boot_option for standalone, I wonder if it does it somehow wrong.. | 14:27 |
| dtantsur | {"path": "/instance_info/capabilities", "op": "add", "value": {"boot_option": "local"}} WHY | 14:28 |
| dtantsur | no, wait, it probably IS correct: I have an image that works with local boot. The question is why this image is not used. | 14:28 |
| dtantsur | (and why the standalone job passes on ironic btw) | 14:28 |
| dtantsur | okay, I see why. Thinking how to fix it best. | 14:30 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Stop defaulting the standalone job to netboot https://review.opendev.org/c/openstack/ironic-python-agent/+/829719 | 14:32 |
| dtantsur | TheJulia: this is the most straightforward fix ^^ | 14:32 |
| TheJulia | dtantsur: works for me, hopefully :) | 14:33 |
| dtantsur | I think the logic in the devstack plugin can use some hardening as well | 14:33 |
| dtantsur | but this should be enough | 14:33 |
| TheJulia | If it passes, I'm likely to just single core approve it. I need to get the ipa-b fix merged for fips images to be happy | 14:36 |
| dtantsur | yep | 14:37 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Build the new cirros image even when netboot is the default https://review.opendev.org/c/openstack/ironic/+/829720 | 14:37 |
| dtantsur | and this is the hardening ^^^ | 14:37 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent stable/xena: Stop defaulting the standalone job to netboot https://review.opendev.org/c/openstack/ironic-python-agent/+/829721 | 14:41 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.3: Stop defaulting the standalone job to netboot https://review.opendev.org/c/openstack/ironic-python-agent/+/829722 | 14:42 |
| TheJulia | dtantsur: if wholedisk image does not equal true? | 14:47 |
| dtantsur | is wholedisk image is true, we shouldn't replace the default image | 14:47 |
| TheJulia | well, I guess that is right, since its being run as a partition image | 14:47 |
| TheJulia | ahh, okay | 14:47 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:15 |
| dtantsur | I think this ^^ is getting really close, and I'm quite excited about it | 15:15 |
| TheJulia | \o/ | 15:19 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:26 |
| dtantsur | okay, now undeploy | 15:33 |
| opendevreview | Merged openstack/ironic-lib master: Allow customizing roles to use JSON RPC https://review.opendev.org/c/openstack/ironic-lib/+/826456 | 15:51 |
| opendevreview | Merged openstack/ironic stable/xena: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829660 | 15:51 |
| opendevreview | Verification of a change to openstack/ironic bugfix/19.0 failed: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829637 | 16:19 |
| * dtantsur wants DIB to produce an EFI partition by default... | 16:21 | |
| TheJulia | ditto, I'm a little frustrated so many just focus only on bios booting *still* | 16:24 |
| TheJulia | but their focus use case is often VMs | 16:24 |
| iurygregory | zuul is unhappy https://review.opendev.org/c/openstack/ironic-python-agent/+/829719 | 16:57 |
| dtantsur | can it be something transient? we have a high rate of PXE failures. | 16:58 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/598b1d8efc1043559cade13b6408a4aa/log/controller/logs/ironic-bm-logs/node-1_no_ansi_2022-02-17-15:41:47_log.txt#2049-2054 | 16:59 |
| iurygregory | humm this doesn't look good .-. | 16:59 |
| dtantsur | red herring | 17:00 |
| dtantsur | basically, we wipe the token before we shut down IPA | 17:00 |
| iurygregory | ohhhh | 17:00 |
| dtantsur | collect_system_logs is executed right before IPA is powered down | 17:01 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/598b1d8efc1043559cade13b6408a4aa/log/controller/logs/ironic-bm-logs/node-0_no_ansi_2022-02-17-16:09:41_log.txt how about the ones here? | 17:02 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Update inventory format documentation with more details https://review.opendev.org/c/openstack/bifrost/+/829748 | 17:04 |
| iurygregory | Looking up node with addresses '52:54:00:c0:25:d5,52:54:00:c3:9a:9f' and UUID None at https://173.231.255.251/baremetal _do_lookup /opt/ironic-python-agent/lib64/python3.6/site-packages/ironic_python_agent/ironic_api_client.py:203 O.o | 17:05 |
| dtantsur | iurygregory: yep, these are bad. I *think* I've seen something like that, and I *think* they're transient. | 17:05 |
| iurygregory | =( | 17:05 |
| dtantsur | dhcp-interface@ens1.service: Failed at step EXEC spawning /sbin/ifup: No such file or directory | 17:06 |
| dtantsur | oooouch | 17:06 |
| iurygregory | nice... | 17:06 |
| dtantsur | dhcp-all-interfaces went for a walk? | 17:06 |
| iurygregory | a very long walk | 17:07 |
| *** eandersson8 is now known as eandersson | 17:19 | |
| opendevreview | Merged openstack/ironic stable/wallaby: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829679 | 17:19 |
| opendevreview | Verification of a change to openstack/ironic stable/victoria failed: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/829680 | 17:19 |
| rpittau | bye evryone, see you on monday! o/ | 17:24 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Update inventory format documentation with more details https://review.opendev.org/c/openstack/bifrost/+/829748 | 17:24 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Update inventory format documentation with more details https://review.opendev.org/c/openstack/bifrost/+/829748 | 17:28 |
| dtantsur | I don't quite like the defaults here ^^^ but at least they're now documented | 17:28 |
| iurygregory | https://review.opendev.org/c/openstack/releases/+/829075 metalsmith release | 18:05 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Remove questionable defaults from the network configuration https://review.opendev.org/c/openstack/bifrost/+/829754 | 18:06 |
| iurygregory | Before the person pushed 1.7.0 (but we didn't have any new features... so I pushed 1.6.2) | 18:06 |
| dtantsur | need to go, see you on Monday folks | 18:11 |
| dtantsur | have a good weekend o/ | 18:11 |
| iurygregory | bye dtantsur enjoy the weekend! | 18:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Shorten error messages in commonly used modules https://review.opendev.org/c/openstack/ironic/+/829755 | 18:17 |
| *** sshnaidm is now known as sshnaidm|afk | 18:17 | |
| iurygregory | quick review if any core is around https://review.opendev.org/c/openstack/ironic-lib/+/829688 =) | 18:22 |
| iurygregory | tks rloo =) | 18:25 |
| rloo | yw :) | 18:25 |
| opendevreview | Merged openstack/ironic master: Build the new cirros image even when netboot is the default https://review.opendev.org/c/openstack/ironic/+/829720 | 18:49 |
| iurygregory | Nice \o/ | 18:52 |
| TheJulia | rechecked builder patch, so hopefully that will now pass | 18:54 |
| iurygregory | fingers crossed | 18:56 |
| opendevreview | Verification of a change to openstack/ironic-lib master failed: Follow-up Allow customizing roles to use JSON RPC https://review.opendev.org/c/openstack/ironic-lib/+/829688 | 20:04 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Create random character devices so systemd does not crash https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/829391 | 23:09 |
| opendevreview | Steve Baker proposed openstack/python-ironic-inspector-client master: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/829830 | 23:49 |
| opendevreview | Steve Baker proposed openstack/ironic master: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/ironic/+/829831 | 23:50 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/metalsmith/+/829832 | 23:51 |
| opendevreview | Steve Baker proposed openstack/metalsmith stable/xena: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/metalsmith/+/829833 | 23:58 |
| opendevreview | Steve Baker proposed openstack/metalsmith stable/wallaby: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/metalsmith/+/829834 | 23:59 |
| opendevreview | Steve Baker proposed openstack/metalsmith stable/victoria: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/metalsmith/+/829835 | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!