| *** irclogbot_3 has quit IRC | 00:16 | |
| *** irclogbot_0 has joined #openstack-ironic | 00:19 | |
| *** ociuhandu has joined #openstack-ironic | 00:25 | |
| *** ociuhandu has quit IRC | 00:30 | |
| *** diurnalist has quit IRC | 01:14 | |
| *** ricolin has joined #openstack-ironic | 01:43 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root path / to use plain JSON https://review.opendev.org/740142 | 02:15 |
|---|---|---|
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root controller to plain controller https://review.opendev.org/741064 | 02:15 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert v1 controller to plain, return JSON https://review.opendev.org/741065 | 02:15 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Remove File type https://review.opendev.org/741862 | 02:15 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Remove Link type https://review.opendev.org/741863 | 02:15 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Rename Response to PassthruResponse https://review.opendev.org/741864 | 02:15 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: WIP Remove Driver, DriverList types https://review.opendev.org/741865 | 02:15 |
| *** uzumaki has joined #openstack-ironic | 02:19 | |
| *** rcernin has quit IRC | 03:02 | |
| *** yaawang has quit IRC | 03:08 | |
| *** yaawang has joined #openstack-ironic | 03:08 | |
| *** rcernin has joined #openstack-ironic | 03:21 | |
| *** rcernin has quit IRC | 03:25 | |
| *** rcernin has joined #openstack-ironic | 03:26 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 03:36 |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 03:38 |
| *** diurnalist has joined #openstack-ironic | 03:46 | |
| *** diurnalist has quit IRC | 03:51 | |
| *** yaawang has quit IRC | 03:54 | |
| *** yaawang has joined #openstack-ironic | 03:55 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 04:23 |
| *** uzumaki has quit IRC | 04:27 | |
| *** ociuhandu has joined #openstack-ironic | 04:28 | |
| *** ociuhandu has quit IRC | 04:33 | |
| *** Lucas_Gray has quit IRC | 04:33 | |
| *** abdysn has joined #openstack-ironic | 04:52 | |
| *** vmud213 has joined #openstack-ironic | 05:56 | |
| *** vmud213 has quit IRC | 06:21 | |
| *** yaawang has quit IRC | 06:41 | |
| *** yaawang has joined #openstack-ironic | 06:42 | |
| iurygregory | good morning Ironic! | 07:02 |
| *** ociuhandu has joined #openstack-ironic | 07:12 | |
| *** rcernin has quit IRC | 07:24 | |
| *** yaawang has quit IRC | 07:27 | |
| *** dtantsur|afk is now known as dtantsur | 07:27 | |
| dtantsur | morning ironic | 07:27 |
| *** yaawang has joined #openstack-ironic | 07:28 | |
| arne_wiebalck | Good morning iurygregory, dtantsur and ironic+ | 07:29 |
| dtantsur | iurygregory: if you have a few minutes this morning: https://review.opendev.org/#/c/740304/ https://review.opendev.org/#/c/740217/ and https://review.opendev.org/#/c/740224/ please | 07:31 |
| patchbot | patch 740304 - ironic-inspector - Do not try to set local_gb to -1 when the matched ... - 2 patch sets | 07:31 |
| patchbot | patch 740217 - bifrost - Get rid of ci_testing variable in roles - 1 patch set | 07:31 |
| patchbot | patch 740224 - bifrost - Add git_url_root to override root for all reposito... - 1 patch set | 07:31 |
| *** yolanda has joined #openstack-ironic | 07:31 | |
| dtantsur | also, it's not yet officially announced, but: https://ironicbaremetal.org/ :) | 07:31 |
| arne_wiebalck | boom! | 07:33 |
| arne_wiebalck | very nice! | 07:33 |
| iurygregory | morning arne_wiebalck and dtantsur =) | 07:34 |
| iurygregory | sure! | 07:34 |
| arne_wiebalck | will this be announce this afternoon? | 07:34 |
| dtantsur | arne_wiebalck: yep, on a keynote | 07:34 |
| iurygregory | we won't have upstream meeting today | 07:34 |
| dtantsur | we probably won't indeed, but let's wait for TheJulia to confirm | 07:35 |
| iurygregory | "It occurs to me that we should likely cancel the meeting in the morning since it is during opendev" | 07:35 |
| iurygregory | she said yesterday on the channel | 07:35 |
| dtantsur | ah, I see it | 07:38 |
| *** rcernin has joined #openstack-ironic | 07:38 | |
| *** derekh has joined #openstack-ironic | 07:39 | |
| *** rcernin has quit IRC | 07:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Account for power interfaces that cannot power on https://review.opendev.org/741650 | 07:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add agent power interface https://review.opendev.org/735591 | 07:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Document fast-track and the agent power interface https://review.opendev.org/741668 | 07:45 |
| *** rcernin has joined #openstack-ironic | 07:48 | |
| *** yaawang has quit IRC | 07:52 | |
| *** rcernin has quit IRC | 07:52 | |
| *** yaawang has joined #openstack-ironic | 07:52 | |
| iurygregory | dtantsur, fyi seems like now it's working https://zuul.opendev.org/t/openstack/build/a82ea77d72ab4ef5ab89ff9f55dd6424/log/controller/logs/tempest_log.txt both ids are in the tempest.log \o/ | 07:53 |
| dtantsur | nice, lemme check | 07:55 |
| dtantsur | iurygregory: I'm worried about the multinode job not passing.. | 07:56 |
| *** uzumaki has joined #openstack-ironic | 07:58 | |
| *** lucasagomes has joined #openstack-ironic | 07:58 | |
| dtantsur | can it be that we need more VMs there? | 08:01 |
| dtantsur | hmm, we already have 6 | 08:01 |
| *** janders has joined #openstack-ironic | 08:02 | |
| openstackgerrit | Merged openstack/bifrost master: Get rid of ci_testing variable in roles https://review.opendev.org/740217 | 08:03 |
| * iurygregory checks | 08:04 | |
| uzumaki | good afternoon ironic! o/ | 08:06 |
| janders | g'day o/ | 08:07 |
| iurygregory | hey uzumaki | 08:07 |
| iurygregory | hey janders =) | 08:07 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop running test_schedule_to_all_nodes in the multinode job https://review.opendev.org/741884 | 08:07 |
| dtantsur | iurygregory, TheJulia, maybe ^^^ | 08:07 |
| iurygregory | dtantsur, worth a try =) | 08:08 |
| dtantsur | iurygregory: but it's nonetheless suspicious that BaremetalBasicOpsPartitionImage failed on both rechecks | 08:08 |
| iurygregory | yeah | 08:08 |
| iurygregory | I will look a bit more on the tests to see | 08:09 |
| uzumaki | how's it going people? janders iurygregory \o/ | 08:10 |
| janders | pretty good, had a relaxing weekend, didn't do much except for a couple drone flights | 08:10 |
| janders | I fixed up the bifrost install doco page which I butchered on Friday (don't know what was I thinking :) | 08:11 |
| janders | how are you guys? | 08:11 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option https://review.opendev.org/741886 | 08:12 |
| iurygregory | uzumaki, doing good and you? | 08:12 |
| uzumaki | I'm well, weekend was good actually, I feel quite relaxed, started picking up on my reading janders iurygregory | 08:12 |
| iurygregory | nice =) | 08:13 |
| uzumaki | \o/ | 08:14 |
| iurygregory | dtantsur, I only used default_boot_option so we would get the value from IRONIC_DEFAULT_BOOT_OPTION hehehe | 08:14 |
| dtantsur | iurygregory: I'm fine with the conf name, but I think on the class level it should be just "boot_option" | 08:14 |
| iurygregory | gotcha | 08:15 |
| janders | thank you for the doco review dtantsur! :) | 08:18 |
| dtantsur | np :) | 08:18 |
| janders | sorry about the noise regarding paths in the previous iterations | 08:18 |
| janders | I either made a mistake somewhere or the libselinux bug was breaking venv setup in my install back when testing that | 08:19 |
| janders | all good now so removed all that unnecessary stuff and only left relevant bits | 08:19 |
| dtantsur | nice | 08:21 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 08:21 |
| dtantsur | yeah, we need the install docs in a good shape now that we link to it from https://ironicbaremetal.org/ | 08:21 |
| openstackgerrit | Merged openstack/bifrost master: Add git_url_root to override root for all repositories at once https://review.opendev.org/740224 | 08:23 |
| dtantsur | iurygregory: oh, may I also bother you with https://review.opendev.org/#/c/741627/ and https://review.opendev.org/#/c/741638/ please? | 08:26 |
| patchbot | patch 741627 - bifrost - Make skip_install a normal boolean variable - 1 patch set | 08:26 |
| patchbot | patch 741638 - bifrost - Enable metadata cleaning by default - 2 patch sets | 08:26 |
| *** yaawang has quit IRC | 08:29 | |
| *** yaawang has joined #openstack-ironic | 08:30 | |
| *** derekh has quit IRC | 08:34 | |
| dtantsur | also at some point we should work on enabling redfish virtual media in bifrost - will simply our testing a lot | 08:36 |
| arne_wiebalck | did anyone receive the zoom link/password for the event this afternoon? Last time, the details were on the eventbrite ticket, does not seem to be the case this time ... | 08:45 |
| dtantsur | arne_wiebalck: look for an email with subject "24 Hours Left! OpenDev Participation Information" | 08:49 |
| *** Lucas_Gray has joined #openstack-ironic | 08:51 | |
| iurygregory | dtantsur, np will look in a few | 08:51 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Do not try to set local_gb to -1 when the matched root device size is 0 https://review.opendev.org/740304 | 08:54 |
| arne_wiebalck | dtantsur: thanks, will check! | 08:55 |
| *** k_mouza has joined #openstack-ironic | 09:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support configuring credential-less deploy https://review.opendev.org/741894 | 09:02 |
| *** rcernin has joined #openstack-ironic | 09:10 | |
| *** rcernin has quit IRC | 09:15 | |
| *** ricolin has quit IRC | 09:17 | |
| janders | I am having some difficulties figuring out where bifrost keeps ironic's codebase. For example I see two copies of steps.py: | 09:22 |
| janders | [and512@xps stack]$ find . -name steps.py | 09:22 |
| janders | ./bifrost/lib/python3.8/site-packages/ironic/conductor/steps.py | 09:22 |
| janders | ./ironic/ironic/conductor/steps.py | 09:22 |
| janders | [and512@xps stack]$ | 09:22 |
| janders | sorry pastebin | 09:23 |
| janders | http://paste.openstack.org/show/796106/ | 09:23 |
| janders | better :) | 09:23 |
| janders | which one is "live"? | 09:23 |
| dtantsur | janders: ./bifrost is the actual venv | 09:24 |
| dtantsur | ./ironic is a source checkout from which it was installed | 09:25 |
| dtantsur | a good TODO item would be to support develop installation (pip -e) | 09:25 |
| *** sshnaidm|off is now known as sshnaidm | 09:27 | |
| *** diurnalist has joined #openstack-ironic | 09:31 | |
| *** yaawang has quit IRC | 09:33 | |
| *** yaawang has joined #openstack-ironic | 09:34 | |
| *** diurnalist has quit IRC | 09:35 | |
| janders | when installing bifrost with the intention of doing some testing in KVM instances running on the node, what do I need to specify as --network-interface? virbr0? | 09:42 |
| janders | or does --testenv take care of that? | 09:45 |
| dtantsur | janders: only --testenv | 09:49 |
| janders | thank you dtantsur | 09:50 |
| openstackgerrit | Merged openstack/bifrost master: Make skip_install a normal boolean variable https://review.opendev.org/741627 | 09:53 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support extra-vars in bifrost-cli https://review.opendev.org/740246 | 09:55 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support configuring credential-less deploy https://review.opendev.org/741894 | 09:55 |
| *** ociuhandu has quit IRC | 10:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support extra-vars in bifrost-cli https://review.opendev.org/740246 | 10:07 |
| openstackgerrit | Merged openstack/bifrost master: Enable metadata cleaning by default https://review.opendev.org/741638 | 10:08 |
| *** ociuhandu has joined #openstack-ironic | 10:08 | |
| *** yaawang has quit IRC | 10:08 | |
| *** k_mouza has quit IRC | 10:08 | |
| *** Lucas_Gray has quit IRC | 10:08 | |
| *** janders has quit IRC | 10:08 | |
| *** irclogbot_0 has quit IRC | 10:08 | |
| *** bdodd_ has quit IRC | 10:08 | |
| *** vesper11 has quit IRC | 10:08 | |
| *** fdegir has quit IRC | 10:09 | |
| *** stevebaker has quit IRC | 10:09 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support extra-vars in bifrost-cli https://review.opendev.org/740246 | 10:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support configuring credential-less deploy https://review.opendev.org/741894 | 10:10 |
| *** yaawang has joined #openstack-ironic | 10:10 | |
| *** k_mouza has joined #openstack-ironic | 10:10 | |
| *** Lucas_Gray has joined #openstack-ironic | 10:10 | |
| *** janders has joined #openstack-ironic | 10:10 | |
| *** irclogbot_0 has joined #openstack-ironic | 10:10 | |
| *** bdodd_ has joined #openstack-ironic | 10:10 | |
| *** vesper11 has joined #openstack-ironic | 10:10 | |
| *** fdegir has joined #openstack-ironic | 10:10 | |
| *** stevebaker has joined #openstack-ironic | 10:10 | |
| *** Lucas_Gray has quit IRC | 10:10 | |
| *** Lucas_Gray has joined #openstack-ironic | 10:11 | |
| *** ociuhandu has quit IRC | 10:13 | |
| *** derekh has joined #openstack-ironic | 10:20 | |
| *** tkajinam has quit IRC | 10:29 | |
| *** ociuhandu has joined #openstack-ironic | 10:32 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop running test_schedule_to_all_nodes in the multinode job https://review.opendev.org/741884 | 10:33 |
| openstackgerrit | Merged openstack/tenks master: Set min version of tox to 3.2.1 https://review.opendev.org/741194 | 10:35 |
| *** trident has quit IRC | 10:35 | |
| *** trident has joined #openstack-ironic | 10:36 | |
| janders | I'm having some hassles trying to enroll my VMs into ironic | 10:39 |
| janders | http://paste.openstack.org/show/796110/ | 10:39 |
| janders | am I doing something dumb, or is my install missing ansible modules somehow? | 10:39 |
| janders | inventory output seems cool | 10:42 |
| janders | http://paste.openstack.org/show/796112/ | 10:42 |
| openstackgerrit | Sandeep Yadav proposed openstack/ironic-python-agent-builder master: [dnm] test tripleo-buildimage-ipa c8 https://review.opendev.org/741917 | 10:45 |
| *** janders has quit IRC | 11:17 | |
| *** janders has joined #openstack-ironic | 11:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop running test_schedule_to_all_nodes in the multinode job https://review.opendev.org/741884 | 11:39 |
| janders | wrapping up for the day, see you tomorrow o/ | 11:47 |
| janders | when you have time, I'd appreciate a pointer on http://paste.openstack.org/show/796110/ | 11:47 |
| janders | kinda stuck there | 11:47 |
| janders | doesn't look like my ansible install has openstack.cloud.baremetal_node module | 11:48 |
| janders | (if I read correctly it is a module) | 11:49 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support extra-vars in bifrost-cli https://review.opendev.org/740246 | 11:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support configuring credential-less deploy https://review.opendev.org/741894 | 11:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Update to cirros 0.5.1 https://review.opendev.org/741923 | 12:00 |
| dtantsur | janders: I'll direct your question to iurygregory, he has experience with the collections | 12:01 |
| *** rh-jelabarre has joined #openstack-ironic | 12:03 | |
| iurygregory | janders, can you check if you have the collections in the env? | 12:04 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Add wsme core types, remove WSME https://review.opendev.org/704490 | 12:08 |
| *** yolanda has quit IRC | 12:14 | |
| *** yolanda has joined #openstack-ironic | 12:15 | |
| *** Goneri has joined #openstack-ironic | 12:17 | |
| *** rh-jelabarre has quit IRC | 12:23 | |
| rpioso | Good morning, ironic | 12:27 |
| *** rh-jelabarre has joined #openstack-ironic | 12:28 | |
| iurygregory | morning rpioso | 12:29 |
| rpioso | iurygregory: :-) | 12:30 |
| * rpioso didn't receive the "24 Hours Left! OpenDev Participation Information" email | 12:31 | |
| arne_wiebalck | rpioso: neither did I | 12:32 |
| arne_wiebalck | rpioso: and on the eventbrite ticket I do not see the zoom link (like last time) | 12:33 |
| TheJulia | So we have consensus on cancelling the meeting this morning? | 12:34 |
| arne_wiebalck | +1 | 12:36 |
| * iurygregory gave +1 yesterday | 12:36 | |
| *** ricolin has joined #openstack-ironic | 12:37 | |
| TheJulia | email sent | 12:41 |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent-builder failed: [tinyipa] Fix getting MTU via dhcp https://review.opendev.org/731095 | 12:50 |
| dtantsur | morning TheJulia | 12:51 |
| *** tzumainn has joined #openstack-ironic | 12:57 | |
| TheJulia | Time for OpenDev! | 13:00 |
| TheJulia | did you guys register for the second event? | 13:01 |
| TheJulia | you had to register for each one separately | 13:01 |
| *** zbitter is now known as zaneb | 13:07 | |
| *** rcernin has joined #openstack-ironic | 13:07 | |
| *** ricolin_ has joined #openstack-ironic | 13:08 | |
| *** k_mouza has quit IRC | 13:09 | |
| *** rloo has joined #openstack-ironic | 13:10 | |
| *** ricolin has quit IRC | 13:10 | |
| clarkb | https://www.eventbrite.com/e/opendev-hardware-automation-registration-104569991660 is the link to register for this event | 13:11 |
| *** rcernin has quit IRC | 13:12 | |
| *** ricolin has joined #openstack-ironic | 13:16 | |
| *** ricolin_ has quit IRC | 13:17 | |
| dtantsur | to pile on an old discussion: I've just screwed up myself by force-deleting a node | 13:18 |
| dtantsur | making me wonder if we should proactively cleanup local PXE environment of items that do not belong to any nodes | 13:19 |
| TheJulia | so we should disucss next step for the baremetal sig soon | 13:19 |
| TheJulia | we kind of also hit tuning bu tmaybe perofrmance requirements gathering? | 13:19 |
| TheJulia | my lighting is awful | 13:21 |
| TheJulia | light kit willbe broken out for tomorrow | 13:21 |
| openstackgerrit | Merged 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 | 13:22 |
| *** uzumaki has quit IRC | 13:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ussuri: Do not try to set local_gb to -1 when the matched root device size is 0 https://review.opendev.org/741936 | 13:28 |
| *** ociuhandu_ has joined #openstack-ironic | 13:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/train: Do not try to set local_gb to -1 when the matched root device size is 0 https://review.opendev.org/741937 | 13:29 |
| dtantsur | fun https://storyboard.openstack.org/#!/story/2007933 | 13:29 |
| *** k_mouza has joined #openstack-ironic | 13:29 | |
| *** ociuhandu has quit IRC | 13:31 | |
| *** bnemec has joined #openstack-ironic | 13:32 | |
| *** k_mouza has quit IRC | 13:34 | |
| *** derekh has quit IRC | 13:34 | |
| dtantsur | also yay, tooz is broken with the new pymysql | 13:36 |
| *** k_mouza has joined #openstack-ironic | 13:37 | |
| *** k_mouza has quit IRC | 13:41 | |
| TheJulia | woot | 13:45 |
| dtantsur | as you can guess, I learned it when my FIPS-related patch got approved :( | 13:46 |
| TheJulia | oh jeeze | 13:46 |
| TheJulia | *sigh* | 13:46 |
| dtantsur | and isn't this lovely? https://github.com/PyMySQL/PyMySQL/commit/e06bbac052bd8cb978bbf1083989cc05c6c19b27 | 13:47 |
| *** k_mouza has joined #openstack-ironic | 13:47 | |
| TheJulia | "no automated way of resetting it" <-- not in our current feature set but itcould be | 13:47 |
| *** k_mouza has quit IRC | 13:51 | |
| TheJulia | I feel like a "multi-os" deployment to hardware tempest test could be useful | 13:52 |
| TheJulia | a "hardware qualification" helper? | 13:55 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option https://review.opendev.org/741886 | 13:58 |
| *** k_mouza has joined #openstack-ironic | 14:03 | |
| *** ociuhandu_ has quit IRC | 14:05 | |
| *** ociuhandu has joined #openstack-ironic | 14:06 | |
| *** derekh has joined #openstack-ironic | 14:06 | |
| *** k_mouza has quit IRC | 14:06 | |
| dtantsur | iurygregory: do you have anything ready or half-ready for basic auth in bifrost? if not, I can take a look (want to have it reasonable soon). | 14:07 |
| iurygregory | dtantsur, half ready, haven't tested to see the state (will probably need to solve some merge conflicts before pushing) | 14:08 |
| dtantsur | iurygregory: do you want to push it or should I start over? or do you want to finish it as well? | 14:08 |
| iurygregory | you can start if you want, I didn't pushed at the time because we were waitting for the keystoneauth release (and now we merged a lot of things hehe) | 14:09 |
| dtantsur | heh, okay | 14:09 |
| *** aedc has joined #openstack-ironic | 14:10 | |
| andrein | hi guys, I'm getting this error when provisioning a server with bifrost https://seashells.io/p/EUgTWDM. I'm using tinyipa-stable-train from https://tarballs.opendev.org/openstack/ironic-python-agent/tinyipa/files/tinyipa-stable-train.gz | 14:19 |
| *** abdysn has quit IRC | 14:23 | |
| *** k_mouza has joined #openstack-ironic | 14:27 | |
| TheJulia | adrianc: 404 on your link | 14:28 |
| andrein | TheJulia: https://seashells.io/p/EUgTWDMM sorry, my mouse ate an M at the end | 14:30 |
| andrein | I'm using the direct deploy method, I suspect https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/extensions/image.py#L736 should be guarded by a check for the deploy interface | 14:31 |
| andrein | but I'm not sure how to get that | 14:32 |
| TheJulia | andrein: anything in the conductor log? | 14:33 |
| andrein | checking | 14:33 |
| TheJulia | that shouldn't fail the deploy | 14:33 |
| *** diurnalist has joined #openstack-ironic | 14:34 | |
| TheJulia | wow that is bizzar | 14:36 |
| TheJulia | I guess the issue is we're getting an unexpected exception and things are going *boom* as a result | 14:37 |
| andrein | TheJulia: http://paste.openstack.org/show/796127/ | 14:40 |
| TheJulia | Interesting | 14:40 |
| andrein | I suspect tgtadm is not installed on the TinyIPA image | 14:40 |
| TheJulia | yeah, we need to catch ProcessExecutionError | 14:40 |
| TheJulia | yeah, most likely | 14:40 |
| openstackgerrit | Sandeep Yadav proposed openstack/ironic-python-agent-builder master: Add Tripleo IPA build image job in zuul layout https://review.opendev.org/741917 | 14:43 |
| andrein | TheJulia: looks like ProcessExecutionError is caught in _execute, it's actually a ironic_python_agent.errors.ISCSICommandError that's uncaught | 14:44 |
| *** cdearborn has joined #openstack-ironic | 14:45 | |
| TheJulia | andrein: sorry, slightly distracted at the moment trying to eat a snack during a break, but yeah... it just seems super weird with the errors. Seems like easy enough fix and I think it would likely be okay to catch it in the caller | 14:45 |
| andrein | TheJulia: I'm not sure blindly catching ISCSICommandError is the way to go (I'm not familiar with the ISCSI deploy interface, but swallowing errors sounds bad) That's why I was suggesting adding a check for the deploy interface used in https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/extensions/image.py#L736 | 14:48 |
| andrein | basically, skip that call if the interface is not ISCSI | 14:49 |
| andrein | however, I'm not sure how to get that information. if someone can point me in the right direction I'd appreciate it | 14:50 |
| TheJulia | The agent doesn't really know what the deploy interface is. I think we cache a node object internally in ipa, but using it would be the question and I'm not sure that would be great. I think we explicitly tear down iscsi and install_bootloader does it for good measure, but I'd need to check the conductor side code path | 14:51 |
| *** rcernin has joined #openstack-ironic | 14:55 | |
| *** stendulker has joined #openstack-ironic | 14:57 | |
| andrein | TheJulia: shall I open a storyboard issue? | 14:57 |
| *** rcernin has quit IRC | 15:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 15:09 |
| dtantsur | iurygregory: ^^ | 15:11 |
| iurygregory | dtantsur, ack | 15:11 |
| dtantsur | TheJulia: opinions welcome: https://review.opendev.org/#/c/741884/ | 15:12 |
| patchbot | patch 741884 - ironic - Stop running test_schedule_to_all_nodes in the mul... - 3 patch sets | 15:12 |
| TheJulia | can I +2 the spirit without looking? | 15:13 |
| dtantsur | :D | 15:13 |
| *** jgwentworth is now known as melwitt | 15:19 | |
| gudrutis0 | There is no Monday meeting today? | 15:31 |
| ajya | gudrutis0: no, http://lists.openstack.org/pipermail/openstack-discuss/2020-July/016007.html | 15:32 |
| *** stendulker has quit IRC | 15:42 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 15:50 |
| *** gyee has joined #openstack-ironic | 15:51 | |
| *** sshnaidm is now known as sshnaidm|afk | 15:57 | |
| TheJulia | sorry for the late notice gudrutis0 | 15:58 |
| *** lucasagomes has quit IRC | 16:00 | |
| *** JamesBenson has joined #openstack-ironic | 16:09 | |
| *** diurnalist has quit IRC | 16:12 | |
| *** ricolin has quit IRC | 16:14 | |
| *** k_mouza has quit IRC | 16:15 | |
| *** diurnalist has joined #openstack-ironic | 16:16 | |
| dtantsur | ugh.. the HTTP basic middleware does not account for public routes | 16:21 |
| dtantsur | stevebaker: this ^^ at least breaks inspector (/ and /v1 must be without authentication) | 16:23 |
| dtantsur | iurygregory: FYI ^^ we cannot use the basic auth until it's fixed | 16:26 |
| dtantsur | oh, and /v1/continue will probably be broken as well, so inspector completely broken | 16:27 |
| dtantsur | the same likely applies for ironic. unless I'm missing something? | 16:28 |
| *** dtantsur is now known as dtantsur|afk | 16:29 | |
| dtantsur|afk | will look tomorrow if nobody beats me to it (please) | 16:29 |
| *** ociuhandu_ has joined #openstack-ironic | 16:30 | |
| TheJulia | it should... but... I thought steve turned one of the jobs on to use auth | 16:31 |
| *** ociuhandu has quit IRC | 16:33 | |
| *** ociuhandu_ has quit IRC | 16:34 | |
| iurygregory | seems we only enabled auth_strategy http_basic for json_rpc | 16:40 |
| TheJulia | *sigh* yeah.... | 16:41 |
| TheJulia | ugh | 16:41 |
| iurygregory | I've tested some api calls for ironic with gophercloud using http_basic and it worked fine | 16:42 |
| iurygregory | (haven't done deployment...) | 16:43 |
| TheJulia | yeah, if the middleware is quashing the unauthenticated endpoint, that is a hard break | 16:43 |
| TheJulia | I need to go lay down, spoons low at the moment after opendev | 16:44 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Add wsme core types, remove WSME https://review.opendev.org/704490 | 16:49 |
| *** rcernin has joined #openstack-ironic | 16:56 | |
| *** derekh has quit IRC | 17:00 | |
| *** rcernin has quit IRC | 17:01 | |
| TheJulia | looks like we can have the hash ring rebuild collide with an action and somehow we end up failing the deploy step :( https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_868/704490/11/gate/ironic-tempest-ipa-partition-uefi-pxe-grub2/868ca82/controller/logs/screen-ir-cond.txt | 17:18 |
| *** ociuhandu has joined #openstack-ironic | 17:34 | |
| *** ociuhandu has quit IRC | 17:38 | |
| *** Lucas_Gray has quit IRC | 17:45 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Minor exception fix related to read timeout patch https://review.opendev.org/742006 | 18:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: set EVENTLET_NO_GREENDNS to 'yes' https://review.opendev.org/742011 | 18:56 |
| TheJulia | of all the silly things in the world ^ | 18:57 |
| *** rcernin has joined #openstack-ironic | 18:57 | |
| *** rcernin has quit IRC | 19:01 | |
| *** dsneddon has joined #openstack-ironic | 19:35 | |
| *** ociuhandu has joined #openstack-ironic | 19:36 | |
| andrein | TheJulia: https://storyboard.openstack.org/#!/story/2007937 i've added a story about the failure i wrote about a while ago. I tried to just swallow the ISCSICommandError but I'm having trouble building an IPA image to test. | 19:44 |
| janders | good evening / early morning ironic o/ | 19:59 |
| janders | iurygregory regarding collections - how do I check for/install collections? | 20:00 |
| janders | iurygregory just saw the PM checking | 20:01 |
| janders | how was the opendev baremetal event? | 20:05 |
| *** ociuhandu has quit IRC | 20:06 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Strip some excess from the centos dib image https://review.opendev.org/741819 | 20:09 |
| TheJulia | andrein: awesome | 20:10 |
| TheJulia | good morning janders | 20:10 |
| TheJulia | janders: it went well, I think | 20:10 |
| *** hjensas has joined #openstack-ironic | 20:22 | |
| TheJulia | andrein: got an idea, poking at the code to see if it works | 20:25 |
| andrein | TheJulia: should tgtd be running in tinyIPA? because I've been poking around and it looks like it isn't started | 20:26 |
| TheJulia | no, but I do think you've got fun valid bug. I'm _REALLY_ surprised we've not hit this before | 20:27 |
| stevebaker | morning | 20:34 |
| stevebaker | dtantsur|afk: I'll take a look | 20:34 |
| TheJulia | Guten Morgan stevebaker | 20:34 |
| andrein | TheJulia: it's really odd, I've been reviewing the code and nothing seems to have changed around there for months. I've reverted to a previous IPA ramdisk which I'm reasonably sure worked about 3 months ago when I provisioned some servers, and that one also fails. | 20:36 |
| * TheJulia blinks | 20:37 | |
| TheJulia | These are the times I think "Maybe I should take up brewing" | 20:37 |
| stevebaker | TheJulia: been there, done that. Surprisingly hard work ;) | 20:38 |
| TheJulia | stevebaker: drat! | 20:38 |
| TheJulia | What if I open a brewery/pizzaria after the pandemic is over? | 20:38 |
| stevebaker | I would endorse this plan | 20:39 |
| andrein | TheJulia: just gave the iSCSI deploy driver a whirl, it was surprisingly fast and it says it successfully installed grub on the server :) waiting for it to boot now | 20:41 |
| TheJulia | andrein: awesome | 20:41 |
| TheJulia | Yeah, each driver depending on how has its benefits | 20:41 |
| TheJulia | So I think I know what happened | 20:42 |
| TheJulia | Still super-surprised things went kaboom | 20:42 |
| andrein | TheJulia: do tell, I'm really curious what I stumbled across | 20:43 |
| TheJulia | so I think we backported changes to force bootloader check/application for raid/uefi and I guess the train agent kind of broke in some cases and we just never spotted it... but maybe the behavior of the underlying libraries also changed | 20:44 |
| TheJulia | because it shouldn't make it CI if it broke in the gate | 20:44 |
| *** janders has quit IRC | 20:44 | |
| stevebaker | TheJulia, dtantsur|afk: OK, my local ironic running master and latest ironic-image is returning 200 responses for / and /v1 with http_basic as expected. I'll take a look at that bifrost change | 20:51 |
| *** bnemec has quit IRC | 20:55 | |
| *** rcernin has joined #openstack-ironic | 20:58 | |
| TheJulia | stevebaker: does /v1/lookup work? | 21:00 |
| *** rcernin has quit IRC | 21:02 | |
| stevebaker | TheJulia: let me add a node and find out | 21:03 |
| TheJulia | ++ | 21:03 |
| *** bnemec has joined #openstack-ironic | 21:10 | |
| *** cdearborn has quit IRC | 21:14 | |
| *** aedc has quit IRC | 21:14 | |
| *** aedc has joined #openstack-ironic | 21:15 | |
| TheJulia | okay, i figured it out, I was trying to fix it the wrong way | 21:18 |
| *** aedc has quit IRC | 21:20 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Prevent un-needed iscsi cleanup https://review.opendev.org/742029 | 21:28 |
| TheJulia | vi andrein ^^^ | 21:29 |
| TheJulia | hot off the code press | 21:29 |
| andrein | TheJulia: wohoo! can't wait for the backport! | 21:34 |
| andrein | thanks for looking into this, I'll give it a go as soon as the images are published | 21:35 |
| andrein | (or, if you have an image handy *wink*wink*) | 21:35 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Extend base build timeouts https://review.opendev.org/741818 | 21:42 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Extend base build timeouts https://review.opendev.org/741818 | 21:43 |
| *** rloo has quit IRC | 22:22 | |
| TheJulia | andrein: Sadly not right now, but it seems fairly problematic to get moving forward so a couple days most likely | 22:34 |
| TheJulia | dtantsur|afk: https://review.opendev.org/#/c/740344 has not failed since the last change to include content on the 202s. But it looks like the lower constraint will need to be rasied to Flask 1.1.0. Any thoughts? | 22:47 |
| patchbot | patch 740344 - ironic-inspector - DNM: Respond so a ContentType is included - 8 patch sets | 22:47 |
| *** tkajinam has joined #openstack-ironic | 22:53 | |
| *** Lucas_Gray has joined #openstack-ironic | 22:55 | |
| *** rcernin has joined #openstack-ironic | 22:58 | |
| *** rcernin has quit IRC | 22:59 | |
| *** rcernin has joined #openstack-ironic | 22:59 | |
| openstackgerrit | Merged openstack/ironic master: Set min version of tox to 3.2.1 https://review.opendev.org/741185 | 23:03 |
| *** bnemec has quit IRC | 23:20 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add ``ipxe_template`` override option https://review.opendev.org/739841 | 23:38 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!