| *** rh-jelabarre has quit IRC | 00:43 | |
| *** rh-jelabarre has joined #openstack-ironic | 00:44 | |
| *** rh-jelabarre has quit IRC | 00:48 | |
| *** vkmc_ is now known as vkmc | 00:54 | |
| *** prometheanfire has left #openstack-ironic | 01:04 | |
| *** Goneri has quit IRC | 01:18 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add json and param parsing to args https://review.opendev.org/704489 | 01:26 |
|---|---|---|
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add wsme core types, remove WSME https://review.opendev.org/704490 | 01:26 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root path / to use plain JSON https://review.opendev.org/740142 | 01:26 |
| *** gyee has quit IRC | 01:38 | |
| *** ricolin has joined #openstack-ironic | 02:02 | |
| *** mkrai has joined #openstack-ironic | 02:18 | |
| openstackgerrit | liuyuanfeng proposed openstack/ironic stable/ussuri: error word confiuration to configuration https://review.opendev.org/740148 | 02:26 |
| *** mkrai has quit IRC | 02:49 | |
| *** tzumainn has quit IRC | 02:50 | |
| *** mkrai has joined #openstack-ironic | 02:56 | |
| *** Lucas_Gray has quit IRC | 02:56 | |
| *** rcernin has quit IRC | 03:00 | |
| *** diurnalist has quit IRC | 03:01 | |
| *** rcernin has joined #openstack-ironic | 03:03 | |
| *** yaawang has quit IRC | 03:10 | |
| *** yaawang has joined #openstack-ironic | 03:10 | |
| *** Lucas_Gray has joined #openstack-ironic | 03:20 | |
| *** Lucas_Gray has quit IRC | 03:32 | |
| *** diurnalist has joined #openstack-ironic | 03:33 | |
| *** diurnalist has quit IRC | 03:38 | |
| *** diurnalist has joined #openstack-ironic | 03:49 | |
| *** diurnalist has quit IRC | 03:54 | |
| *** ociuhandu has joined #openstack-ironic | 03:56 | |
| *** ociuhandu has quit IRC | 04:01 | |
| *** diurnalist has joined #openstack-ironic | 04:04 | |
| *** diurnalist has quit IRC | 04:10 | |
| *** diurnalist has joined #openstack-ironic | 04:29 | |
| *** abdysn has joined #openstack-ironic | 05:02 | |
| *** zzzeek has quit IRC | 05:19 | |
| *** zzzeek has joined #openstack-ironic | 05:23 | |
| *** diurnalist has quit IRC | 05:32 | |
| *** mkrai has quit IRC | 05:50 | |
| arne_wiebalck | Good morning, ironic! | 06:04 |
| *** yaawang has quit IRC | 06:32 | |
| *** yaawang has joined #openstack-ironic | 06:33 | |
| iurygregory | good morning arne_wiebalck and Ironic | 06:47 |
| *** ildikov has quit IRC | 06:55 | |
| arne_wiebalck | hey iurygregory o/ | 06:56 |
| *** ildikov has joined #openstack-ironic | 06:58 | |
| openstackgerrit | uchenily proposed openstack/networking-generic-switch master: change segment id type to int https://review.opendev.org/740171 | 07:03 |
| rpittau | good morning ironic! o/ | 07:03 |
| iurygregory | morning rpittau o/ | 07:03 |
| rpittau | hey iurygregory :) | 07:04 |
| iurygregory | just FYI don't try recheck yet on the patches =) | 07:08 |
| rpittau | what's broken? | 07:08 |
| iurygregory | we need https://review.opendev.org/#/c/714453/ fixing it | 07:08 |
| patchbot | patch 714453 - ironic - add tempest boot_mode config - 21 patch sets | 07:08 |
| rpittau | fun fact: I used to start a lot of my past working days with that question | 07:08 |
| iurygregory | ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa | 07:09 |
| iurygregory | and ironic-tempest-ipa-partition-pxe_ipmitool | 07:09 |
| iurygregory | the fix is on my patch since it was introduced by my change on the tempest plugin that got merged | 07:09 |
| rpittau | who breaks it, fixes it :) | 07:10 |
| rpittau | another common sentence used in the past | 07:10 |
| iurygregory | hehehe | 07:10 |
| rpittau | just kidding of course, some memories of stuff... broken... badly :) | 07:11 |
| iurygregory | well I knew that was going to break, but probably people didn't see my comment =X | 07:11 |
| rpittau | reading is overrated | 07:11 |
| iurygregory | yeah | 07:12 |
| * iurygregory grabs more coffee | 07:12 | |
| *** dougsz has joined #openstack-ironic | 07:20 | |
| arne_wiebalck | good morning, rpittau o/ | 07:34 |
| rpittau | hey arne_wiebalck :) | 07:34 |
| * arne_wiebalck missed irc most of yesterday | 07:34 | |
| iurygregory | this was a funny error from our multinode job https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_90b/714453/21/check/ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode/90bed6c/testr_results.html | 07:35 |
| rpittau | heh missed a lot of logs too :/ | 07:35 |
| iurygregory | never saw before =D | 07:35 |
| rpittau | well I just got 5300 tests failed on 6383, something's not right | 07:37 |
| iurygregory | wut?! | 07:38 |
| rpittau | lower-constraints, Python 3.8, nightmares ahead | 07:40 |
| openstackgerrit | liuyuanfeng proposed openstack/ironic master: error wold confiuration to configuration https://review.opendev.org/740174 | 07:40 |
| iurygregory | rpittau, ouch =( if you need help let me know | 07:46 |
| iurygregory | only me or gerrit is a bit slow today? | 07:49 |
| openstackgerrit | liuyuanfeng proposed openstack/ironic master: error word confiuration to configuration https://review.opendev.org/740175 | 07:53 |
| openstackgerrit | liuyuanfeng proposed openstack/ironic master: error word confiuration to configuration https://review.opendev.org/740175 | 07:54 |
| *** sshnaidm has joined #openstack-ironic | 07:55 | |
| *** Lucas_Gray has joined #openstack-ironic | 08:01 | |
| rpittau | oh thanks iurygregory, I'm back to Failed: 0 :) | 08:02 |
| *** gmann has quit IRC | 08:03 | |
| *** erbarr has quit IRC | 08:03 | |
| iurygregory | rpittau, yay! | 08:04 |
| *** vkmc has quit IRC | 08:04 | |
| *** erbarr has joined #openstack-ironic | 08:04 | |
| * iurygregory resets the router brb | 08:04 | |
| *** v1k0d3n has quit IRC | 08:05 | |
| *** gmann has joined #openstack-ironic | 08:06 | |
| openstackgerrit | liuyuanfeng proposed openstack/ironic stable/ussuri: fix error word confiuration to configuration in docs https://review.opendev.org/740148 | 08:06 |
| *** v1k0d3n has joined #openstack-ironic | 08:07 | |
| *** vkmc has joined #openstack-ironic | 08:07 | |
| maelk | Hi! dtantsur|afk I was trying to test your patch https://review.opendev.org/#/c/739964. However, the fixes you added in Metal3 make it so that IPA is behaving properly now, so there is no lookup happening anymore. So we do not need urgently this fix anymore. anyways it would be good to fix it still so I'll try again without the metal3 fix and keep | 08:08 |
| maelk | you updated | 08:08 |
| patchbot | patch 739964 - ironic - [WIP] Wipe agent token during reboot or power off - 1 patch set | 08:08 |
| openstackgerrit | liuyuanfeng proposed openstack/ironic master: fix error word confiuration to configuration in docs https://review.opendev.org/740175 | 08:09 |
| *** iurygregory has quit IRC | 08:10 | |
| *** lucasagomes has joined #openstack-ironic | 08:15 | |
| *** iurygregory has joined #openstack-ironic | 08:24 | |
| *** uzumaki has joined #openstack-ironic | 08:34 | |
| maelk | also, dtantsur|afk if we are using the fast track, and we have a powerloss or anything that causes the server to reboot independently of Ironic's control, IPA will need a new token, but ironic won't give it a new one because it did not wipe the old token at any point. is it right ? | 08:34 |
| *** derekh has joined #openstack-ironic | 08:36 | |
| iurygregory | yay CI gave +1 | 08:40 |
| iurygregory | tks zuul, I will give you a cookie XD | 08:40 |
| *** benj_ has quit IRC | 08:41 | |
| *** k_mouza has joined #openstack-ironic | 08:44 | |
| uzumaki | iurygregory, I'd like to present myself as an ambassador to mr zuul to recive the property, thusly described as 'the cookie' | 08:48 |
| iurygregory | uzumaki, hehehehehe | 08:48 |
| uzumaki | sup iurygregory o/ xD how's it going? | 08:49 |
| iurygregory | uzumaki, doing good, and you? | 08:49 |
| iurygregory | here is the cookie https://media.giphy.com/media/7GYHmjk6vlqY8/giphy.gif | 08:49 |
| uzumaki | I'm well! | 08:49 |
| iurygregory | please give to zuul | 08:49 |
| uzumaki | iurygregory, already on it! love the cookie, looks delicious xD | 08:50 |
| *** benj_ has joined #openstack-ironic | 08:58 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Remove non-inclusive language in pxe filter https://review.opendev.org/740194 | 09:01 |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Remove non-inclusive language in pxe filter https://review.opendev.org/740194 | 09:03 |
| iurygregory | hjensas, ++ | 09:10 |
| *** uzumaki has quit IRC | 09:20 | |
| *** uzumaki has joined #openstack-ironic | 09:20 | |
| *** rcernin has quit IRC | 09:23 | |
| *** dtantsur|afk is now known as dtantsur | 09:29 | |
| dtantsur | morning ironic | 09:34 |
| dtantsur | mgoddard: hey, https://review.opendev.org/#/c/737695/ got 2x +2 and is close to getting approve, PTAL | 09:34 |
| patchbot | patch 737695 - ironic - agent_base: support inserting in-band deploy steps - 5 patch sets | 09:34 |
| mgoddard | dtantsur: looking | 09:39 |
| iurygregory | morning dtantsur | 09:40 |
| dtantsur | iurygregory: I'm updating the incorrect upper-constraints patches | 09:41 |
| iurygregory | dtantsur, ack | 09:42 |
| iurygregory | do they have victoria available already? | 09:43 |
| dtantsur | iurygregory: yep, it's a redirect to master | 09:43 |
| iurygregory | dtantsur, oh awesome! otherwise we would need to update everything in the end of the cycle =X | 09:44 |
| *** k_mouza has quit IRC | 09:47 | |
| dtantsur | maelk: re fast-track: yes, that's why we need my patch still | 09:52 |
| ftarasenko | Hi team! | 09:56 |
| *** k_mouza has joined #openstack-ironic | 09:56 | |
| maelk | @dtantsur, would ironic find out on its own that it needs to wipe the token ? there is nothing telling it to power off the server | 09:57 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Do not clone repos with source_install==false https://review.opendev.org/739930 | 09:57 |
| dtantsur | hi ftarasenko | 09:57 |
| dtantsur | maelk: if you tell ironic to power off, it will wipe the token | 09:57 |
| maelk | yes, but what if the user powers it off without ironic | 09:58 |
| ftarasenko | arne_wiebalck: iurygregory: dtantsur: have question regarding EFI bootloader and software RAID. Why do we put EFI bootloader only on one partition outside of mdraid? | 09:58 |
| dtantsur | in case of power loss.. I'm not sure | 09:58 |
| dtantsur | ftarasenko: I thought we put it on a partition on each holder disk.. arne_wiebalck remembers it better though | 09:58 |
| maelk | maybe the token would need to be wiped whenever ironic powers on a node also | 09:58 |
| maelk | or when ironic polls the power status and the node is off | 09:59 |
| dtantsur | mmm, probably the former. I need to think a bit. | 09:59 |
| maelk | of course that does not cover an unexpected reboot undetected by ironic | 09:59 |
| ftarasenko | dtantsur: I might be wrong. Checked again and see both partitions with blkid. | 10:00 |
| ftarasenko | Do I correctly remember that UEFI search for EFI partition on each mounted drive? | 10:00 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Do not clone repos with source_install==false https://review.opendev.org/739930 | 10:03 |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Remove non-inclusive language in pxe filter https://review.opendev.org/740194 | 10:03 |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Remove non-inclusive language in pxe filter https://review.opendev.org/740194 | 10:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Do not clone repos with source_install==false https://review.opendev.org/739930 | 10:06 |
| openstackgerrit | Merged openstack/ironic master: add tempest boot_mode config https://review.opendev.org/714453 | 10:07 |
| iurygregory | yay CI is back | 10:09 |
| iurygregory | recheck all the patches! | 10:09 |
| iurygregory | ftarasenko, it will look for the efi partition on the device https://github.com/openstack/ironic-python-agent/blob/9d9a6bce5c246d18d50dc405265e865fb77142eb/ironic_python_agent/utils.py#L613 | 10:11 |
| *** hjensas is now known as hjensas|afk | 10:12 | |
| ftarasenko | iurygregory: Thank you. Still think that we need to set boot order with efibootmgr, but haven't tested yet | 10:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Get rid of ci_testing variable in roles https://review.opendev.org/740217 | 10:15 |
| *** ricolin has quit IRC | 10:15 | |
| *** ociuhandu has joined #openstack-ironic | 10:16 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/ussuri: add tempest boot_mode config https://review.opendev.org/740221 | 10:18 |
| arne_wiebalck | ftarasenko: I think we put it on partitions on all drives. | 10:21 |
| maelk | dtantsur I verified your patch https://review.opendev.org/#/c/739964 . It fixes the issue! | 10:23 |
| patchbot | patch 739964 - ironic - [WIP] Wipe agent token during reboot or power off - 1 patch set | 10:23 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: migrate testing to ubuntu focal https://review.opendev.org/739448 | 10:23 |
| dtantsur | maelk: great! | 10:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add git_url_root to override root for all repositories at once https://review.opendev.org/740224 | 10:24 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: migrate testing to ubuntu focal https://review.opendev.org/739448 | 10:24 |
| *** janders has quit IRC | 10:25 | |
| *** uzumaki has quit IRC | 10:26 | |
| *** uzumaki has joined #openstack-ironic | 10:26 | |
| openstackgerrit | Merged openstack/ironic master: fix error word confiuration to configuration in docs https://review.opendev.org/740175 | 10:28 |
| *** uzumaki has quit IRC | 10:33 | |
| *** uzumaki has joined #openstack-ironic | 10:33 | |
| *** uzumaki has quit IRC | 10:38 | |
| *** tkajinam has quit IRC | 11:02 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Account for bugfix branches in bifrost-cli https://review.opendev.org/740244 | 11:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support extra-vars in bifrost-cli https://review.opendev.org/740246 | 11:22 |
| *** jungleboyj_ is now known as jungleboyj | 11:46 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Remove non-inclusive language https://review.opendev.org/740251 | 11:49 |
| *** rh-jelabarre has joined #openstack-ironic | 11:57 | |
| dtantsur | hmm, are we broken again? https://zuul.openstack.org/builds?job_name=ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa | 12:20 |
| dtantsur | yeah, same problem, I guess | 12:22 |
| iurygregory | dtantsur, it's fixed | 12:24 |
| iurygregory | =) | 12:24 |
| dtantsur | iurygregory: well, it still fails | 12:24 |
| iurygregory | stable/ussuri is probably affected | 12:24 |
| iurygregory | dtantsur, I didn't get your comment in https://review.opendev.org/#/c/740221/ | 12:26 |
| patchbot | patch 740221 - ironic (stable/ussuri) - add tempest boot_mode config - 1 patch set | 12:26 |
| iurygregory | O.o | 12:26 |
| dtantsur | iurygregory: that job also needs to be updated (although it's not running, so let's update it separately) | 12:26 |
| dtantsur | okay, I'll do another round of rechecks to be sure | 12:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support extra-vars in bifrost-cli https://review.opendev.org/740246 | 12:28 |
| iurygregory | oh we have ironic-tempest-ipa-wholedisk-bios-agent_ipmitool but we don't run that job | 12:29 |
| iurygregory | we run ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa | 12:29 |
| rpittau | until ussuri it was still tinyipa | 12:29 |
| iurygregory | # NOTE(rpittau): converted job but not running for now as there | 12:31 |
| iurygregory | # could be an issue with the lookup in ironic-python-agent | 12:31 |
| iurygregory | it's the dib version of it | 12:31 |
| iurygregory | but we don't run because you are still testing right rpittau ? | 12:32 |
| rpittau | yeah | 12:34 |
| rpittau | there's a little issue with networkmanager https://review.opendev.org/734759 | 12:34 |
| patchbot | patch 734759 - ironic - Convert ironic-tempest-ipa-wholedisk-bios-agent_ip... - 10 patch sets | 12:34 |
| rpittau | I confused jobs :D | 12:35 |
| rpittau | that's the only one still with tinyipa | 12:35 |
| iurygregory | yeah! | 12:35 |
| iurygregory | XD | 12:36 |
| iurygregory | just be aware now that you may need to increase the amount of VMS | 12:36 |
| iurygregory | =) | 12:36 |
| openstackgerrit | Merged openstack/ironic master: agent_base: support inserting in-band deploy steps https://review.opendev.org/737695 | 12:42 |
| dtantsur | \o/ | 12:42 |
| dtantsur | now, could I lure someone into reviewing the remaining two patches? https://review.opendev.org/#/q/topic:story/2006963+status:open | 12:42 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Convert ironic-tempest-ipa-wholedisk-bios-agent_ipmitool to dib https://review.opendev.org/734759 | 12:47 |
| openstackgerrit | Merged openstack/ironic stable/ussuri: fix error word confiuration to configuration in docs https://review.opendev.org/740148 | 12:54 |
| *** hjensas|afk is now known as hjensas | 13:09 | |
| *** ricolin has joined #openstack-ironic | 13:10 | |
| TheJulia | good morning | 13:10 |
| rpittau | good morning TheJulia :) | 13:17 |
| TheJulia | hopefully everyone had a wonderful evening | 13:18 |
| TheJulia | dtantsur: I'll happilly review th elast... after I've been coffeed | 13:18 |
| iurygregory | good morning TheJulia =) | 13:19 |
| *** Goneri has joined #openstack-ironic | 13:20 | |
| dtantsur | morning TheJulia | 13:22 |
| iurygregory | should we only run test_baremetal_multitenancy+test_schedule_to_all_nodes on ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode ? | 13:24 |
| TheJulia | does anyone remember what the ansible behavior change with using | bool was that was coming in the next major release? | 13:27 |
| TheJulia | iurygregory: I guess likely just that job because we're pushing nova scheduling to the maxiumum during a release, across an upgrade, I'm not sure there is value there | 13:28 |
| TheJulia | dtantsur: you've been around for a long time as well, any thoughts on ^^^ | 13:28 |
| dtantsur | on which of the two questions? | 13:29 |
| iurygregory | upgrade? o.o | 13:29 |
| dtantsur | re ansible: they'll stop treating strings as booleans eventually | 13:29 |
| dtantsur | so {{ variable | bool }} is safer | 13:29 |
| dtantsur | iurygregory: re tests, we probably should (although the ironic-tempest-plugin CI uses this job to verify the regular scenarios) | 13:30 |
| iurygregory | dtantsur, aha | 13:31 |
| iurygregory | ok now I see the reason =) | 13:31 |
| dtantsur | TheJulia: the good news are: we've done our intermediate releases and the world did not blow up :) | 13:33 |
| dtantsur | we need branch mapping for devstack CI jobs on the new branches though (I'll look into it after some tea) | 13:34 |
| TheJulia | dtantsur: I remember some awesome errors recently that basically said they were going to remove a form of use, so maybe it was just uncast strings. I really don't remember | 13:37 |
| TheJulia | dtantsur: I was wondering about the schedule all nodes | 13:38 |
| TheJulia | dtantsur: \o/ | 13:38 |
| TheJulia | ++ | 13:38 |
| TheJulia | I have a meeting starting in ~20 minutes | 13:38 |
| * TheJulia sips coffee and still tries to wake up | 13:38 | |
| *** tzumainn has joined #openstack-ironic | 13:48 | |
| dtantsur | schedule all nodes... dunno. there is value in it, not sure how much. | 13:50 |
| TheJulia | It is a test that seems... easy to fail | 13:50 |
| TheJulia | and by that I mean we've got a lot of moving parts going on there | 13:51 |
| TheJulia | and we're trying to artificially compress everything into the CI window. | 13:51 |
| TheJulia | I'd prefer minimalistic and only on the larger multinode job because that will also be less of a grenade headache in my mind | 13:51 |
| iurygregory | since we started moving to dib (we only have 1 VM) | 13:53 |
| dtantsur | I *think* multinode jobs still use tinyIPA, no? | 13:53 |
| iurygregory | dtantsur, correct | 13:54 |
| openstackgerrit | Merged openstack/ironic-inspector master: Remove non-inclusive language in pxe filter https://review.opendev.org/740194 | 13:54 |
| openstackgerrit | Merged openstack/bifrost master: Account for bugfix branches in bifrost-cli https://review.opendev.org/740244 | 14:02 |
| *** diurnalist has joined #openstack-ironic | 14:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Do not try to set local_gb to -1 when the matched root device size is 0 https://review.opendev.org/740304 | 14:16 |
| dtantsur | TheJulia: 1st part of what we've just discussed downstream ^^ | 14:16 |
| *** abdysn has quit IRC | 14:24 | |
| openstackgerrit | Merged openstack/ironic master: Decompose the core deploy step of the direct deploy https://review.opendev.org/698309 | 14:30 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Ignore devices with size 0 when collecting inventory https://review.opendev.org/740309 | 14:32 |
| dtantsur | TheJulia: 2nd part ^^ | 14:32 |
| dtantsur | I should file a story for that, shouldn't I? | 14:36 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Do not try to set local_gb to -1 when the matched root device size is 0 https://review.opendev.org/740304 | 14:40 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Ignore devices with size 0 when collecting inventory https://review.opendev.org/740309 | 14:41 |
| *** Lucas_Gray has quit IRC | 14:55 | |
| *** mnasiadka has joined #openstack-ironic | 14:58 | |
| *** Lucas_Gray has joined #openstack-ironic | 15:11 | |
| dtantsur | arne_wiebalck: so, if RAID is broken, what exactly shows with size zero? and what are we trying to do about it? | 15:22 |
| arne_wiebalck | dtantsur: from what I remember, if a RAID is broken (not degraded), e.g. a RAID-0 with one disk missing, and the instance is being cleaned, the corresponding RAID device is reported as 0 size and was not cleaned properly (as it was not reported); this then broke RAID creation afterwards | 15:27 |
| dtantsur | arne_wiebalck: does cleaning still work for such devices? | 15:28 |
| arne_wiebalck | dtantsur: with cleaning I am referring to delete_configuration (our h/w manager does this on auto cleaning, hence the not very precise wording) | 15:30 |
| arne_wiebalck | dtantsur: the configuration on the other leg of the RAID needs to be removed | 15:30 |
| arne_wiebalck | dtantsur: but the other leg cannot be found since the md device is not reported | 15:30 |
| dtantsur | arne_wiebalck: mmm, so if I add a flag that only includes empty devices for delete_configuration, it will be fine? | 15:30 |
| * dtantsur checks the code | 15:31 | |
| *** k_mouza has quit IRC | 15:31 | |
| arne_wiebalck | dtantsur: I would think so, yes | 15:31 |
| arne_wiebalck | dtantsur: I just remember this one as it took me a while to figure out :) | 15:32 |
| dtantsur | arne_wiebalck: it doesn't seem like delete_configuration uses list_all_block_devices.. | 15:32 |
| arne_wiebalck | dtantsur: that broken md devices have size 0 | 15:32 |
| dtantsur | oh, we can allow empty md devices but skip all other? | 15:33 |
| arne_wiebalck | dtantsur: this one? https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L1782 | 15:36 |
| arne_wiebalck | dtantsur: yep, that should also work | 15:36 |
| *** k_mouza has joined #openstack-ironic | 15:37 | |
| dtantsur | oh, how did I manage to miss it? | 15:43 |
| arne_wiebalck | I missed it on the first pass, too :-D | 15:46 |
| *** gyee has joined #openstack-ironic | 15:50 | |
| openstackgerrit | Merged openstack/ironic master: Stop using md5 for __repr__ of objects https://review.opendev.org/739735 | 15:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Stop using md5 for __repr__ of objects https://review.opendev.org/740329 | 16:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/train: Stop using md5 for __repr__ of objects https://review.opendev.org/740330 | 16:04 |
| *** k_mouza has quit IRC | 16:19 | |
| TheJulia | gah corgis | 16:21 |
| TheJulia | dtantsur: thanks on the ironic-inspector patch | 16:21 |
| dtantsur | np.. now wrapping my head around the IPA part | 16:21 |
| *** ociuhandu_ has joined #openstack-ironic | 16:23 | |
| openstackgerrit | Doug Szumski proposed openstack/ironic-python-agent master: WIP: Fix bootloader install issue with MDRAID https://review.opendev.org/740331 | 16:24 |
| dtantsur | ugh | 16:24 |
| TheJulia | looks like ironic-inspector-tempest-managed-non-standalone has some intermittent failures every so often | 16:24 |
| dtantsur | I think it's the same problem with Bad Gateway | 16:25 |
| TheJulia | ahh, looks like the latest... it unpacked the initramfs and nothing else was logged | 16:26 |
| *** ociuhandu has quit IRC | 16:27 | |
| TheJulia | 4 cores, 3 gb of ram... | 16:27 |
| *** ociuhandu_ has quit IRC | 16:28 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Ignore devices with size 0 when collecting inventory https://review.opendev.org/740309 | 16:28 |
| dtantsur | arne_wiebalck: something like ^^? | 16:29 |
| *** dougsz has quit IRC | 16:31 | |
| openstackgerrit | Merged openstack/ironic master: Minor agent token log handling https://review.opendev.org/736214 | 16:38 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Do not validate driver on changing non-driver fields https://review.opendev.org/739699 | 16:38 |
| *** lucasagomes has quit IRC | 16:39 | |
| TheJulia | dtantsur: yup, bad gateway... | 16:46 |
| *** ricolin has quit IRC | 16:47 | |
| *** phrobb_ has joined #openstack-ironic | 16:48 | |
| *** andreaf has quit IRC | 16:48 | |
| *** mnasiadka_ has joined #openstack-ironic | 16:48 | |
| *** m1p_ has joined #openstack-ironic | 16:48 | |
| TheJulia | dtantsur: I wonder if the proxy server needs to be primed or the config is unhappy? | 16:50 |
| dtantsur | I tried to figure out, but couldn't :( | 16:52 |
| dtantsur | I think it's coming from oslo.messaging | 16:54 |
| *** phrobb has quit IRC | 16:55 | |
| *** m1p has quit IRC | 16:55 | |
| *** mnasiadka has quit IRC | 16:55 | |
| *** m1p_ is now known as m1p | 16:55 | |
| *** mnasiadka_ is now known as mnasiadka | 16:55 | |
| *** phrobb_ is now known as phrobb | 16:55 | |
| *** mkowalski has quit IRC | 16:56 | |
| *** derekh has quit IRC | 16:59 | |
| *** mkowalski has joined #openstack-ironic | 17:03 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:12 | |
| TheJulia | hmm | 17:14 |
| * TheJulia digs | 17:14 | |
| arne_wiebalck | dtantsur: yes, that looks good to me | 17:15 |
| TheJulia | dtantsur: interesting, I do see some weirdness on the message bus | 17:16 |
| TheJulia | wow, talk about errors not adding up | 17:18 |
| *** Lucas_Gray has quit IRC | 17:19 | |
| TheJulia | so iptables and ipmitool commands are failing all over the place | 17:23 |
| TheJulia | I see what is going on | 17:28 |
| TheJulia | The test aborted it | 17:30 |
| arne_wiebalck | bye everyone o/ | 17:31 |
| TheJulia | goodnight! | 17:31 |
| TheJulia | heh, yeah, it is cascading out from the original failure | 17:32 |
| dtantsur | wow | 17:35 |
| TheJulia | it makes no sense | 17:38 |
| TheJulia | like the "please go inspect now, kthxbai" command is actually processed but the API is returning an error | 17:39 |
| dtantsur | yup :( | 17:42 |
| dtantsur | and oslo.messaging seems configured the same way in inspector as in ironic | 17:43 |
| *** dtantsur is now known as dtantsur|afk | 17:45 | |
| dtantsur|afk | o/ | 17:45 |
| TheJulia | i have a theory, it may be bunnies! | 17:48 |
| TheJulia | I think the executor submit is blowing up | 17:57 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Additional debugging around inspection start https://review.opendev.org/740344 | 18:10 |
| TheJulia | 85% CPU wait... | 18:13 |
| TheJulia | yeeeahhhhhh | 18:13 |
| TheJulia | Not really sure, 5 GB of ram in full use, moderate IO usage but also high CPU wait times, high new thread/process counts | 18:23 |
| TheJulia | okay, not that high on wait, still odd | 18:34 |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/ussuri: Correct Redfish boot once fallback conditional https://review.opendev.org/740356 | 20:11 |
| *** ociuhandu has joined #openstack-ironic | 20:12 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove locks before RPC bus is started https://review.opendev.org/740360 | 20:14 |
| *** diurnalist has quit IRC | 20:14 | |
| *** diurnalist has joined #openstack-ironic | 20:16 | |
| *** diurnalist has quit IRC | 20:22 | |
| *** diurnalist has joined #openstack-ironic | 20:22 | |
| *** ociuhandu has quit IRC | 20:31 | |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/train: Correct Redfish boot once fallback conditional https://review.opendev.org/740368 | 20:37 |
| *** ociuhandu has joined #openstack-ironic | 20:47 | |
| TheJulia | this inspector issue is just bizzar | 20:55 |
| TheJulia | its not the message bus, api does as expected returns a 202 | 20:56 |
| TheJulia | apache blows up | 20:56 |
| *** ociuhandu has quit IRC | 20:59 | |
| *** yolanda has quit IRC | 21:16 | |
| *** yolanda has joined #openstack-ironic | 21:17 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: WIP: Additional debugging around inspection start https://review.opendev.org/740344 | 21:23 |
| TheJulia | so it seems if you return an empty body, there is no character encoding... | 21:25 |
| TheJulia | and no character encoding can cause apache to get bent out of shape | 21:25 |
| TheJulia | and try to return an error document, which may result in Bad Gateway errors when it tries to get the error document | 21:25 |
| * TheJulia calls it a day | 22:04 | |
| eandersson | Interesting. Is that a 100% failure or just sometimes? | 22:23 |
| eandersson | I have a Bad Gateway error for Designate I have been trying to figure out for ages, but related to many consecutive Unauthorized calls. | 22:24 |
| *** rh-jelabarre has quit IRC | 22:30 | |
| *** jamesdenton has quit IRC | 22:40 | |
| *** tkajinam has joined #openstack-ironic | 23:00 | |
| *** diurnalist has quit IRC | 23:30 | |
| *** dmellado has quit IRC | 23:39 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add json and param parsing to args https://review.opendev.org/704489 | 23:39 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add wsme core types, remove WSME https://review.opendev.org/704490 | 23:39 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root path / to use plain JSON https://review.opendev.org/740142 | 23:39 |
| *** gyee has quit IRC | 23:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
