Friday, 2020-06-19

*** dsneddon has quit IRC00:27
*** dsneddon has joined #openstack-ironic00:28
*** diurnalist has joined #openstack-ironic00:30
*** cdearborn has quit IRC00:41
*** diurnalist has quit IRC00:49
*** diurnalist has joined #openstack-ironic00:49
*** diurnalist has quit IRC01:03
*** tzumainn has quit IRC01:06
*** Goneri has quit IRC01:07
*** Qianbiao has joined #openstack-ironic01:08
*** Qianbiao_ has joined #openstack-ironic01:11
*** Qianbiao_ has quit IRC01:11
*** Qianbiao_ has joined #openstack-ironic01:12
*** Qianbiao has quit IRC01:13
openstackgerritSteve Baker proposed openstack/ironic master: ironic-standalone, use http basic auth for json-rpc  https://review.opendev.org/73639801:41
*** diurnalist has joined #openstack-ironic01:55
*** gyee has quit IRC01:58
*** diurnalist has quit IRC02:09
*** rcernin has quit IRC02:12
*** mkrai has joined #openstack-ironic02:28
*** yaawang_ has quit IRC03:08
*** yaawang_ has joined #openstack-ironic03:09
*** rcernin has joined #openstack-ironic03:17
openstackgerritSteve Baker proposed openstack/ironic master: WIP standalone, use file:// for all images instead of glance  https://review.opendev.org/73692803:37
openstackgerritSteve Baker proposed openstack/ironic master: WIP standalone, use file:// for all images instead of glance  https://review.opendev.org/73692803:45
*** mkrai has quit IRC04:11
*** mkrai_ has joined #openstack-ironic04:11
*** rh-jelabarre has quit IRC04:15
*** bdodd has quit IRC05:50
*** yaawang_ has quit IRC05:52
openstackgerritMerged openstack/tenks master: Install python setuptools & virtualenv  https://review.opendev.org/73624905:52
*** bdodd has joined #openstack-ironic05:53
arne_wiebalckGood morning, ironic!06:08
iurygregorygood morning arne_wiebalck and Ironic! happy friday06:11
arne_wiebalckhey iurygregory o/06:11
openstackgerritRichard G. Pioso proposed openstack/ironic stable/train: Fix Redfish handle no continuous override boot src  https://review.opendev.org/73694106:28
* iurygregory wants to sleep again (it's raining)06:30
*** rpittau|afk is now known as rpittau06:44
rpittaugood morning ironic! o/06:44
* rpittau wants the logs back :(06:44
iurygregorygood morning rpittau o/06:51
rpittauhey iurygregory :)06:51
iurygregorywe wall want the logs back06:51
rpittauwhat about the ceiling ?06:52
iurygregorywith logs we  can't even find problems sometimes, imagine without them...06:52
iurygregoryceiling of what?06:52
iurygregoryO.o06:52
rpittauwall... ceiling.... nvm :P06:52
iurygregorylol06:52
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Add api-ref for indicators API  https://review.opendev.org/73214907:05
iurygregoryrpittau, good news07:08
iurygregoryhttps://review.opendev.org/#/c/736714/07:08
patchbotpatch 736714 - zuul/zuul-jobs - Fix with_* loop for ansible 2.9 (MERGED) - 1 patch set07:08
rpittau\o/07:09
iurygregorywe should probably get our logs back07:09
iurygregory\o/07:09
rpittaummm so apaprently loop helps :)07:09
rpittau:D07:09
iurygregoryrpittau, well let's hope it does for us hehehe07:09
rpittauyep, I was referring to a patch I submitted recently to bifrost with a similar change, need to do the same for the rest07:11
iurygregoryoh gotcha =)07:11
iurygregoryrpittau, if you have some time https://review.opendev.org/#/c/734592/ =)07:12
patchbotpatch 734592 - bifrost - Add openstack collection - 5 patch sets07:12
rpittauyeah, time, so underratede07:12
iurygregoryhehehe07:12
openstackgerritRiccardo Pittau proposed openstack/ironic master: Fix uefi jobs with native ubuntu ovmf package  https://review.opendev.org/71688907:12
rpittaureally close to a solution ^07:13
iurygregoryyay07:14
*** dtantsur|afk is now known as dtantsur07:17
dtantsurmorning ironic, happy Friday07:18
iurygregorymorning dtantsur o/07:18
rpittauhappy Friday! \o/07:18
rpittauiurygregory: left a comment on the collections patch07:27
rpittauif someone wants to make this Friday even better, please review https://review.opendev.org/735335 thanks!07:29
patchbotpatch 735335 - ironic-python-agent-builder - Disable automatic updates in dnf-based systems - 4 patch sets07:29
iurygregoryrpittau, tks looking now07:30
*** dougsz has joined #openstack-ironic07:33
iurygregoryrpittau, tks for the review, i think the if should stay there, the idea is that if ANSIBLE_COLLECTIONS_PATHS is not set we install without saying the directory, and you are right about using BIFROST_COLLECTIONS_PATHS in the install part07:45
*** yaawang has joined #openstack-ironic07:45
iurygregoryto use it I will just need to add the case where it was empty and a venv will be used to install07:45
rpittaummmm but if you set ANSBILE_COLLECTIONS_PATHS to be equal to BIFROST_COLLECTIONS_PATHS you can just omit it as it will be taken by default07:49
iurygregoryANSIBLE_COLLECTIONS_PATHS is empty by default07:50
iurygregoryis like is unset07:51
rpittauyeah, but if BIFROST_COLLECTIONS_PATHS is set, then you can just set ANSIBLE_COLLECTIONS_PATHS to be the same and avoid the if at L4607:51
iurygregoryso all installation will be   ${ANSIBLE_GALAXY} collection install -r ${ANSIBLE_COLLECTION_REQ} -p ${BIFROST_COLLECTIONS_PATHS}07:52
rpittaummmm no07:52
rpittauthey will be all: ${ANSIBLE_GALAXY} collection install -r ${ANSIBLE_COLLECTION_REQ}07:52
iurygregoryand that doesnt work for venv07:53
iurygregoryyou need to put -p and the dir07:53
rpittauI don't see this difference in your patch07:53
openstackgerritMerged openstack/ironic-inspector stable/ussuri: Set tempest_test_timeout for managed job  https://review.opendev.org/73051507:54
iurygregory1st case is no virtual env and the user doesnt set ANSIBLE_COLLECTIONS_PATHS - so it should install like ${ANSIBLE_GALAXY} collection install -r ${ANSIBLE_COLLECTION_REQ07:55
iurygregory2nd case no virtual env but the user have set ANSIBLE_COLLECTIONS_PATHS so it should install  ${ANSIBLE_GALAXY} collection install -r ${ANSIBLE_COLLECTION_REQ} -p ${ANSIBLE_COLLECTIONS_PATHS}07:55
rpittauiurygregory: ANSIBLE_COLLECTIONS_PATHS is always taken into considertion by the installed07:56
rpittau*installer07:56
*** rcernin has quit IRC07:59
rpittaudunno, maybe I'm wrong, just a suggestion on how to handle things in a different way07:59
iurygregorylet me re run the tests here07:59
* dtantsur rubs his eyes and tries to wake up finally08:10
rpittaudtantsur: you can do it! :)08:10
dtantsurdunno, really :) I'm in a zombie mode the whole week08:11
dtantsurgetting out of bed today was more like resurrection08:11
rpittau:D08:12
*** lucasagomes has joined #openstack-ironic08:13
*** mkrai_ has quit IRC08:23
iurygregoryrpittau, http://paste.openstack.org/show/794991/08:24
*** mkrai has joined #openstack-ironic08:24
iurygregoryonly the 1st one goes to the scenario where the -p is not used08:25
iurygregorythe only thing I notice I had to fix was BIFROST_COLLECTIONS_PATHS=$ANSIBLE_COLLECTIONS_PATHS after line 2108:29
iurygregoryand ofc I can change lines 51 and 5208:32
rpittauok08:34
openstackgerritIury Gregory Melo Ferreira proposed openstack/bifrost master: Add openstack collection  https://review.opendev.org/73459208:37
*** Lucas_Gray has joined #openstack-ironic08:42
rpittaugithub seems down, long live github08:46
rpittaummm maybe was cache, looks fine now08:47
*** mugsie has quit IRC08:48
*** mugsie has joined #openstack-ironic08:48
*** rcernin has joined #openstack-ironic08:53
*** rcernin has quit IRC09:01
iurygregoryhabemus logs \o/09:03
dtantsur\o/09:03
iurygregoryhappy dance on friday gif09:04
openstackgerritDmitry Tantsur proposed openstack/ironic master: Fix fast-track with the direct deploy interface  https://review.opendev.org/73616409:12
openstackgerritDmitry Tantsur proposed openstack/ironic master: Fix fast-track with the direct deploy interface  https://review.opendev.org/73616409:12
openstackgerritDmitry Tantsur proposed openstack/ironic master: Networking boot fallback for local boot  https://review.opendev.org/73449009:14
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Agent power interface  https://review.opendev.org/73559109:20
*** tkajinam has quit IRC09:21
*** clarkb has quit IRC09:23
*** rcernin has joined #openstack-ironic09:23
openstackgerritMerged openstack/tenks master: Trust receive filters of baremetal nodes  https://review.opendev.org/73670809:26
*** clarkb has joined #openstack-ironic09:28
*** priteau has joined #openstack-ironic09:36
*** rcernin has quit IRC09:58
*** rpittau is now known as rpittau|bbl10:22
*** dsneddon has quit IRC10:24
*** k_mouza has joined #openstack-ironic10:32
*** mkrai has quit IRC10:36
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Agent power interface  https://review.opendev.org/73559110:37
openstackgerritMark Goddard proposed openstack/tenks stable/1.0: Trust receive filters of baremetal nodes  https://review.opendev.org/73697210:40
*** rcernin has joined #openstack-ironic10:54
dtantsurcan I get another +2 on https://review.opendev.org/#/c/735955/ please?10:58
patchbotpatch 735955 - ironic - Fix agent token and URL handling during fast-track... - 1 patch set10:58
*** rcernin has quit IRC11:08
*** mkrai has joined #openstack-ironic11:29
*** dougsz has quit IRC11:33
openstackgerritMerged openstack/tenks stable/1.0: Trust receive filters of baremetal nodes  https://review.opendev.org/73697211:34
*** derekh has joined #openstack-ironic11:36
iurygregorydtantsur, not sure if it's just me but I couldn't find a place where you are using wipe_cleaning_internal_info, maybe you wanted to have it in https://review.opendev.org/#/c/735955/1/ironic/conductor/manager.py since you removed driver_internal_info.pop('clean_steps', None) but wipe_deploy_internal_info doesn't remove 'clean_steps'?11:43
patchbotpatch 735955 - ironic - Fix agent token and URL handling during fast-track... - 1 patch set11:43
dtantsuriurygregory: wiping clean steps is not needed there, but we definitely need wipe_cleaning_.. somewhere11:45
dtantsurmm, it is used in https://review.opendev.org/#/c/735955/1/ironic/conductor/cleaning.py11:45
patchbotpatch 735955 - ironic - Fix agent token and URL handling during fast-track... - 1 patch set11:45
iurygregoryI was wondering if you missed since it had the pop for clean_steps11:45
iurygregory=)11:45
dtantsurwell, I don't know why we removed clean_steps at the end of a deployment :)11:45
dtantsurprobably a copy-pasta11:45
iurygregoryyup11:45
dtantsurI'm debugging something crazy in the RDO CI11:47
dtantsuressentially, conductor stops doing anything after 'qemu-img info'11:47
dtantsurjust.. leaves the lock in place and never comes back to it again Oo11:47
iurygregoryo.O11:48
iurygregoryholy molly11:48
iurygregorynothing on qemu logs?11:49
dtantsurah, wait, the last command is 'iscsiadm -m discovery'. this is something.11:50
*** rcernin has joined #openstack-ironic11:53
dtantsurfirst action item: add a timeout to iscsi discovery11:53
iurygregoryeasy =)11:55
dtantsurand apparently subprocess on python 3 has built-in timeout support!11:56
dtantsurwe can drop ugly hacks from ipmitool.py11:57
dtantsurugh, it's not exposed in processutils.. /me complains about oslo usage again12:00
*** Lucas_Gray has quit IRC12:01
*** ociuhandu has quit IRC12:03
*** rh-jelabarre has joined #openstack-ironic12:04
*** tkajinam has joined #openstack-ironic12:04
*** mkrai has quit IRC12:06
*** rcernin has quit IRC12:06
*** Lucas_Gray has joined #openstack-ironic12:14
*** rpittau|bbl is now known as rpittau12:19
*** k_mouza has quit IRC12:24
iurygregorydamm =(12:27
iurygregorybuilt-in timeout support would be awesome12:27
dtantsuryeah12:28
dtantsuradding now, but won't be backportable12:28
iurygregorywell at least we will be able to use it12:29
dtantsuryeah12:29
iurygregoryand having the config for the command that sets the default and yay12:29
dtantsurexactly12:30
*** Qianbiao has joined #openstack-ironic12:31
*** Qianbiao_ has quit IRC12:31
*** ociuhandu has joined #openstack-ironic12:31
*** derekh has quit IRC12:33
*** k_mouza has joined #openstack-ironic12:41
*** derekh has joined #openstack-ironic12:51
TheJuliagood morning12:53
iurygregorygood morning TheJulia12:54
dtantsurTheJulia: morning12:57
rpittaugood morning TheJulia :)12:57
dtantsurTheJulia: https://review.opendev.org/#/c/736164/ should be ready for review when you have this time thing again12:58
patchbotpatch 736164 - ironic - Fix fast-track with the direct deploy interface - 3 patch sets12:58
*** rloo has joined #openstack-ironic13:07
*** dtantsur is now known as dtantsur|brb13:11
*** Goneri has joined #openstack-ironic13:36
TheJuliadtantsur|brb: okay, I should be able to look at it today as long as no fires get started13:47
openstackgerritMerged openstack/ironic master: Fix agent token and URL handling during fast-track deployment  https://review.opendev.org/73595513:50
TheJuliadtantsur|brb: btw, the requests -> urllib3 stuffs. I seem to remember funky behavior when you start mixing in some of the features in the code path they have as a result. But to kind of just settle it, I've asked our folks in the field to rip out the patch and retry since it is self contained and hopefully we'll fail or not at that point.13:51
TheJuliaoh plus select/poll :\13:52
TheJulia(except, only in terms of connection pooling)13:52
*** rpittau is now known as rpittau|brb13:54
*** rpittau|brb is now known as rpittau|afk14:00
*** ociuhandu has quit IRC14:07
*** ociuhandu has joined #openstack-ironic14:12
*** tkajinam has quit IRC14:13
iurygregoryevery request we make when using http_basic needs to have in the header "Authorization: Basic <token>" right?14:27
*** dtantsur|brb is now known as dtantsur14:28
dtantsurTheJulia: gotcha cool14:29
dtantsuriurygregory: correct14:29
iurygregorydtantsur, tks114:29
openstackgerritDmitry Tantsur proposed openstack/ironic stable/ussuri: Fix agent token and URL handling during fast-track deployment  https://review.opendev.org/73701814:29
dtantsurTheJulia: agent token is a new thing in ussuri, right? and fast-track as well?14:32
TheJuliaagent token ussuri, fast track is like a year old14:32
*** diurnalist has joined #openstack-ironic14:33
dtantsurgotcha14:36
dtantsurTheJulia: also, got a case of getting stuck in 'deploying': https://storyboard.openstack.org/#!/story/200783614:36
TheJuliafun!14:39
dtantsurindeed14:39
openstackgerritDmitry Tantsur proposed openstack/ironic master: agent: poll long-running commands till completion  https://review.opendev.org/73398514:45
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy  https://review.opendev.org/69830914:45
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps  https://review.opendev.org/73382514:45
*** Qianbiao has quit IRC14:46
dtantsursigh, did not want to rebase all these..14:47
openstackgerritDmitry Tantsur proposed openstack/ironic master: agent: poll long-running commands till completion  https://review.opendev.org/73398514:47
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy  https://review.opendev.org/69830914:47
*** Qianbiao has joined #openstack-ironic14:47
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps  https://review.opendev.org/73382514:47
*** Qianbiao has quit IRC14:53
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step on iscsi and ansible deploy  https://review.opendev.org/73118014:54
*** jtomasek has quit IRC15:08
*** ociuhandu has quit IRC15:14
*** rloo has quit IRC15:19
*** diurnalist has quit IRC15:32
*** Goneri has quit IRC15:32
*** Lucas_Gray has quit IRC15:32
*** andy_ has quit IRC15:32
*** sziviani has quit IRC15:32
*** diurnalist has joined #openstack-ironic15:32
*** Goneri has joined #openstack-ironic15:32
*** Lucas_Gray has joined #openstack-ironic15:32
*** sziviani has joined #openstack-ironic15:32
*** andy_ has joined #openstack-ironic15:32
*** Lucas_Gray has quit IRC15:33
*** mkowalski has quit IRC15:36
*** Lucas_Gray has joined #openstack-ironic15:47
*** bdodd has quit IRC15:57
*** bdodd has joined #openstack-ironic15:59
*** lucasagomes has quit IRC16:03
*** dtantsur is now known as dtantsur|afk16:13
openstackgerritMerged openstack/ironic stable/ussuri: Fix Redfish handle no continuous override boot src  https://review.opendev.org/73683916:25
*** Lucas_Gray has quit IRC16:41
*** Goneri has quit IRC16:43
*** dsneddon has joined #openstack-ironic16:47
*** derekh has quit IRC17:02
TheJuliadtantsur|afk: won't have feedback until next week on that, field team guy is off today17:03
*** k_mouza has quit IRC17:05
*** gmann is now known as gmann_afk17:13
*** Goneri has joined #openstack-ironic17:37
*** k_mouza has joined #openstack-ironic17:49
*** k_mouza has quit IRC17:54
*** dsneddon has quit IRC18:11
*** Lucas_Gray has joined #openstack-ironic18:11
*** dsneddon has joined #openstack-ironic18:11
TheJuliarpioso: thanks for backporting https://review.opendev.org/#/c/736941/1 Quick question I noticed they all bombed on dell ci18:28
patchbotpatch 736941 - ironic (stable/train) - Fix Redfish handle no continuous override boot src - 1 patch set18:28
TheJulias/they all/it completely/18:28
erbarrTheJulia: there's an olso.concurrency version problem in train, I'm trying to see if I can change it from using master upper constraints to use train18:31
TheJuliaerbarr: where exactly?19:02
*** Lucas_Gray has quit IRC19:06
*** bnemec is now known as beekneemech19:08
*** yolanda has quit IRC19:41
openstackgerritMerged openstack/ironic stable/ussuri: Fix agent token and URL handling during fast-track deployment  https://review.opendev.org/73701820:13
*** gmann_afk is now known as gmann20:15
openstackgerritMerged openstack/ironic master: agent: poll long-running commands till completion  https://review.opendev.org/73398520:15
*** Lucas_Gray has joined #openstack-ironic20:30
*** priteau has quit IRC20:36
*** gyee has joined #openstack-ironic20:46
TheJuliaerbarr: oh! saw the recheck, I get it now!21:23
TheJuliaon train in your CI?21:23
TheJuliaor is your CI trying to run master off of train branch commits?21:23
TheJulia#confused21:23
*** patchbot has quit IRC21:26
*** patchbot has joined #openstack-ironic21:26
openstackgerritJulia Kreger proposed openstack/ironic master: ISO ramdisk virtual media test enablement  https://review.opendev.org/73638321:40
NobodyCamHappy Friday Ironic'ers21:50
openstackgerritVerification of a change to openstack/ironic failed: Use unittest.mock instead of mock  https://review.opendev.org/73435021:53
TheJuliahappy Friday NobodyCam !22:37
NobodyCam\o/22:40
*** gyee has quit IRC22:42
openstackgerritMerged openstack/bifrost master: Update IPA if update_repos is true (the default)  https://review.opendev.org/73589922:58
openstackgerritMerged openstack/bifrost master: Fix fast-track deployment after discovery/inspection  https://review.opendev.org/73593322:58
openstackgerritMerged openstack/bifrost master: Only install proliantutils if ilo is enabled; support idrac  https://review.opendev.org/73555422:59
*** k_mouza has joined #openstack-ironic23:12
*** k_mouza has quit IRC23:16
*** diurnalist has quit IRC23:26
openstackgerritMerged openstack/ironic stable/ussuri: Add IPv6 ci Job  https://review.opendev.org/73561423:43
*** diurnalist has joined #openstack-ironic23:57

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!