Wednesday, 2017-08-16

Nisha_AgarwalTheJulia, re comments on SUM patch, i will let owner do it. its already 5.35 am here. she should be there in office by 8.30 am or so.00:08
openstackgerritRichard G. Pioso proposed openstack/ironic master: Add 'hardware type' for Dell EMC iDRACs  https://review.openstack.org/49126300:08
TheJuliaNisha_Agarwal: Okay, get some sleep :)00:10
TheJuliarpioso: ^^^ good?00:10
Nisha_AgarwalTheJulia, :) actually i dont have those nitty details so i cant update the patch00:11
Nisha_AgarwalTheJulia, thanks for the reviews :) now logging off. will login my day time in few hrs from now00:13
*** Nisha_Agarwal has quit IRC00:17
rpiosoTheJulia: Ack re: Redfish vendor extensions.00:27
rpiosoTheJulia: Thank you for the review.  I've left the Workflow -1 on it.  I'm waiting on a 3d party CI test result.00:28
TheJuliaUnderstood00:28
rpiosoTheJulia: As I commented, it'll be removed by midday tomorrow.  That would be US EDT.00:29
rpiosoThat's all for me.  Good day ironicers!00:30
*** rpioso has quit IRC00:35
*** stendulker has joined #openstack-ironic00:52
*** rama_y has joined #openstack-ironic00:54
*** hw_wutianwei has joined #openstack-ironic00:58
*** litao__ has joined #openstack-ironic00:58
*** phuongnh has joined #openstack-ironic01:00
*** libin_ has quit IRC01:04
*** rloo has quit IRC01:13
*** rama_y has quit IRC01:17
*** gcb has joined #openstack-ironic01:32
*** tuanluong has joined #openstack-ironic01:44
*** crushil has quit IRC01:46
*** crushil has joined #openstack-ironic01:53
*** stendulker has quit IRC02:07
*** cdearborn has quit IRC02:09
*** r-daneel has joined #openstack-ironic02:09
*** crushil has quit IRC02:12
*** crushil has joined #openstack-ironic02:12
*** rama_y has joined #openstack-ironic02:35
*** krtaylor has joined #openstack-ironic02:42
*** baoli has joined #openstack-ironic02:47
*** baoli has quit IRC02:48
*** mat128 has quit IRC02:48
*** baoli has joined #openstack-ironic02:48
*** baoli has quit IRC02:49
*** zhangfei has joined #openstack-ironic02:53
*** rama_y has quit IRC03:10
*** vishwana_ has joined #openstack-ironic03:18
*** vishwanathj has quit IRC03:21
*** stendulker has joined #openstack-ironic03:28
*** yangyapeng has quit IRC03:28
*** yangyapeng has joined #openstack-ironic03:32
openstackgerritShivanand Tendulker proposed openstack/ironic master: Documetation for 'snmp' hardware type  https://review.openstack.org/49346803:32
*** finix has joined #openstack-ironic03:34
finixHi folks. Many links in ocata installation guide is broken due to doc03:41
finix migration. This has been fixed here: https://review.openstack.org/#/c/491966/  Could somebody help to review this bug fix for making it go on? Thanks.03:41
patchbotpatch 491966 - ironic (stable/ocata) - Fix broken links to ocata installation guide03:41
*** udesale has joined #openstack-ironic03:44
*** links has joined #openstack-ironic03:47
*** makowals has quit IRC03:47
*** makowals has joined #openstack-ironic03:57
*** finix has quit IRC04:04
*** finix has joined #openstack-ironic04:04
*** Nisha_Agarwal has joined #openstack-ironic04:10
*** yangyapeng has quit IRC04:15
*** rama_y has joined #openstack-ironic04:20
*** rama_y has quit IRC04:28
*** deray has joined #openstack-ironic04:48
derayGood Morning !04:49
*** r-daneel has quit IRC04:50
*** r-daneel has joined #openstack-ironic04:54
*** aparnav has joined #openstack-ironic05:08
*** Nisha_Agarwal has quit IRC05:11
*** Nisha_Agarwal has joined #openstack-ironic05:18
*** wong has quit IRC05:19
*** wong has joined #openstack-ironic05:19
*** mvpnitesh has joined #openstack-ironic05:21
openstackgerritAparna proposed openstack/ironic master: Support SUM based firmware update as clean step for iLO drivers  https://review.openstack.org/42257205:26
*** zhangfei has quit IRC05:31
*** sasha21 has quit IRC05:32
openstackgerritAparna proposed openstack/ironic master: Support SUM based firmware update as clean step for iLO drivers  https://review.openstack.org/42257205:32
*** udesale__ has joined #openstack-ironic05:35
*** udesale has quit IRC05:35
*** mjura has joined #openstack-ironic05:38
*** rama_y has joined #openstack-ironic05:38
*** pcaruana has joined #openstack-ironic05:45
*** zhangfei has joined #openstack-ironic05:45
*** aparnav has quit IRC05:48
*** finix has quit IRC05:58
*** marios has joined #openstack-ironic06:00
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290206:03
*** namnh has joined #openstack-ironic06:05
*** hoonetorg has quit IRC06:08
*** ChubYann has quit IRC06:14
*** Nisha_Agarwal has quit IRC06:19
*** adisky__ has joined #openstack-ironic06:19
*** Nisha_Agarwal has joined #openstack-ironic06:20
*** hoonetorg has joined #openstack-ironic06:21
*** udesale has joined #openstack-ironic06:23
*** udesale__ has quit IRC06:23
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up to ``ilo`` hardware type documentation  https://review.openstack.org/49408406:29
openstackgerritHironori Shiina proposed openstack/ironic master: iRMC: Add documentation for remote volume boot  https://review.openstack.org/48962006:34
*** udesale has quit IRC06:35
*** udesale has joined #openstack-ironic06:36
*** aparnav has joined #openstack-ironic06:38
*** jaganathan has joined #openstack-ironic06:42
*** stendulker has quit IRC06:44
*** rcernin has joined #openstack-ironic06:57
*** masuberu has joined #openstack-ironic07:14
*** rcernin has quit IRC07:16
*** masber has quit IRC07:16
*** mgoddard has joined #openstack-ironic07:17
*** masuberu has quit IRC07:18
*** ccamacho has joined #openstack-ironic07:18
*** rcernin has joined #openstack-ironic07:19
*** jaypipes has joined #openstack-ironic07:25
*** masuberu has joined #openstack-ironic07:28
*** hw_wutianwei has quit IRC07:29
*** hw_wutianwei has joined #openstack-ironic07:31
*** masuberu has quit IRC07:31
*** rama_y has quit IRC07:44
*** rbartal_ has joined #openstack-ironic07:49
*** karimb has joined #openstack-ironic07:51
*** e0ne has joined #openstack-ironic08:01
*** ralonsoh has joined #openstack-ironic08:04
*** lucas-afk is now known as lucasagomes08:06
*** andymccr_ is now known as andymccr08:08
*** derekh has joined #openstack-ironic08:10
*** ralonsoh has quit IRC08:12
*** ralonsoh has joined #openstack-ironic08:12
wonghi,who knows about drac-driver?08:15
wongmy node error is like this:Timeout reached while cleaning the node. Please check if the ramdisk responsible for the cleaning is running on the node08:15
wongWhen I make the node status to provide, I can see the deploy image boot in the node,and the deploy system seems OK in the DELL server node08:16
wongthen it’s motionless.After for a long time, the deploy system shutdown and ironic show me the node clean failed. The reason is time out. I don’t know why08:17
*** mgoddard has quit IRC08:17
*** karimb has quit IRC08:19
*** kromanenko has joined #openstack-ironic08:21
vdrokgood morning ironic08:26
vsaienk0morning vdrok08:28
openstackgerritMerged openstack/bifrost master: [releasenotes] update  https://review.openstack.org/49387008:28
vdrokmorning vsaienk008:28
vsaienk0vdrok: dtantsur|afk could you please review https://review.openstack.org/#/q/I0874fe3b3628cb3e662ee01f24c4599247fdc82d,n,z increase host_subset_size for ironic case08:28
vdrokdone, this does work, can be seen at http://logs.openstack.org/12/493812/5/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/85cab0f/logs/old/screen-n-sch.txt.gz08:30
derayvdrok, g'morning08:32
derayvsaienk0, o/08:32
derayvdrok, could you approve this: https://review.openstack.org/#/c/478741/08:33
patchbotpatch 478741 - ironic - Update proliantutils version for Pike release08:33
tuanluongmorning ironic. vdrok vsaienk0 wong deray08:33
vdrokI will take a look some time today08:33
vdrokmorning tuanluong08:33
deraytuanluong, good morning o/08:34
derayvdrok, np.. tyt08:34
etingofmorning everyone ;)08:37
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove ensure_logs_exist check during upgrade  https://review.openstack.org/49411808:38
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove ensure_logs_exist check during upgrade  https://review.openstack.org/49411808:39
vdrokmorning etingof08:39
deraytuanluong, thx for looking :)08:40
derayetingof, g'morning08:40
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP: wait for resources after destroy  https://review.openstack.org/49376708:41
*** mvpnitesh has quit IRC08:47
*** mvpnitesh has joined #openstack-ironic08:48
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent stable/pike: Update .gitreview for stable/pike  https://review.openstack.org/49412808:53
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent stable/pike: Update UPPER_CONSTRAINTS_FILE for stable/pike  https://review.openstack.org/49412908:53
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent master: Update reno for stable/pike  https://review.openstack.org/49413008:53
*** sambetts|afk is now known as sambetts08:58
sambettsMorning all08:58
Nisha_Agarwalvdrok,09:00
Nisha_Agarwalmorning09:00
*** amarao has joined #openstack-ironic09:00
Nisha_Agarwalmorning ironic09:00
vdrokmorning sambetts and Nisha_Agarwal09:00
Nisha_Agarwalo/09:01
Nisha_Agarwalvdrok, please review https://review.openstack.org/422902  at your leisure. Julia gave +2 and then there was a minor comment which i updated in the morning, hence no +2 votes now on it09:02
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface09:02
deraysambetts, Nisha_Agarwal good morning09:05
Nisha_Agarwalderay, o/09:05
*** stendulker has joined #openstack-ironic09:05
*** hoonetorg has quit IRC09:07
*** lucasagomes is now known as lucas-brb09:08
*** hoonetorg has joined #openstack-ironic09:10
*** milan has joined #openstack-ironic09:13
openstackgerritM V P Nitesh proposed openstack/python-ironicclient master: Fixes multiple chassis delete  https://review.openstack.org/45566209:17
*** zhangfei has quit IRC09:22
*** yufei has joined #openstack-ironic09:27
openstackgerritMerged openstack/python-ironic-inspector-client master: Update Inspector docs links in README  https://review.openstack.org/49383509:29
*** OctopusZhang has joined #openstack-ironic09:30
*** yufei has quit IRC09:30
*** OctopusZhang has quit IRC09:30
*** zhangfei has joined #openstack-ironic09:35
*** deadnull has joined #openstack-ironic09:36
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up to ``ilo`` hardware type documentation  https://review.openstack.org/49408409:37
*** deadnull has quit IRC09:41
*** hw_wutianwei has quit IRC09:47
openstackgerritShivanand Tendulker proposed openstack/ironic master: Documetation for 'snmp' hardware type  https://review.openstack.org/49346809:50
milanmorning Ironic! :)09:51
stendulkerMorning Milan :)09:58
milanmorning stendulker! :)09:58
vdrokmorning milan stendulker09:58
milanmornign vdrok! :)09:59
stendulkerGood morning vdrok :)09:59
deraymilan, stendulker morning o/10:01
milanmorning deray! :)10:02
stendulkermorning deray :)10:03
*** namnh has quit IRC10:04
*** dtantsur|afk is now known as dtantsur10:06
dtantsurmorning ironic10:06
vdroko/ dtantsur10:06
deraydtantsur, good morning10:07
*** karimb has joined #openstack-ironic10:08
aparnavhey dtantsur, vdrok, Could you review this patch https://review.openstack.org/#/c/422572/ ?10:12
patchbotpatch 422572 - ironic - Support SUM based firmware update as clean step fo...10:12
dtantsurdepending on how things look wrt the release.. it feels like the CI is broken, so we may not be able to merge stuff today10:13
vdrokdtantsur: it's not just the feeling :) i've updated the whiteboard10:14
dtantsuraparnav: please avoid rechecking until it's fixed10:14
aparnavdtantsur, sure.10:15
*** lucas-brb is now known as lucasagomes10:18
*** Nisha_Agarwal has quit IRC10:26
*** Nisha_Agarwal has joined #openstack-ironic10:27
*** udesale has quit IRC10:29
openstackgerritIlya Etingof proposed openstack/ironic-specs master: Enumerate Inspector errors  https://review.openstack.org/49415710:37
*** mgoddard has joined #openstack-ironic10:38
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Remove ensure_logs_exist check during upgrade  https://review.openstack.org/49415910:39
dtantsurvdrok: is ^^ blocking ironic-inspector release?10:39
vdrokdtantsur: if there is no pike branch for inspector, then I believe so10:40
dtantsurvdrok: then please review :)10:41
dtantsursambetts: ^^10:42
vdrokeasy :)10:42
*** ralonsoh has quit IRC10:44
*** ralonsoh_ has joined #openstack-ironic10:44
*** priteau has joined #openstack-ironic10:44
vdrokwtf, rechecks don't work10:45
vdroki love release time :D10:46
*** stendulker has quit IRC10:49
dtantsuryep10:50
dtantsurvdrok: FYI we need to re-release networking-baremetal after adding the missing publish-to-pypi job. I'll handle it.10:50
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove ensure_logs_exist check during upgrade  https://review.openstack.org/49411810:51
vdrokdtantsur: sorry. want to retrigger the jobs faster10:51
dtantsurno problem10:52
*** zhangfei has quit IRC10:52
Nisha_Agarwaldtantsur, so ironic can not be released today?10:53
Nisha_Agarwali was just reading the scroll up10:53
dtantsurNisha_Agarwal: I'm afraid so, but we'll see10:53
Nisha_Agarwaldtantsur, :(10:53
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/49416510:54
*** jkilpatr has quit IRC10:54
*** aparnav has quit IRC10:57
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/pike: Imported Translations from Zanata  https://review.openstack.org/49383910:57
*** crushil has quit IRC10:58
*** crushil has joined #openstack-ironic10:59
*** tuanluong has quit IRC11:04
openstackgerritMerged openstack/ironic-python-agent master: Update reno for stable/pike  https://review.openstack.org/49413011:07
*** lucasagomes is now known as lucas-hungry11:10
Nisha_Agarwaldtantsur, can u do favor for https://review.openstack.org/422902  as well :)11:11
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface11:11
*** deray has quit IRC11:15
*** aparnav has joined #openstack-ironic11:16
*** phuongnh has quit IRC11:18
*** fellypefca has joined #openstack-ironic11:19
*** gcb has quit IRC11:20
TheJuliaGood morning11:22
dtantsurit's on the list, but I need to get some food and do some errands. I'll try to get to it, if nobody ends up being faster11:22
dtantsurmorning TheJulia11:22
openstackgerritM V P Nitesh proposed openstack/ironic master: Validate ipmi_address format on ipmitool power interface validation  https://review.openstack.org/43694511:22
dtantsurTheJulia: the CI went away for a walk, it does not feel like we're releasing this morning :(11:22
dtantsurTheJulia: you may have to release tomorrow instead of me (please keep in mind that both ironic and ironic-inspector require a major version bump)11:23
*** Nisha_Agarwal has quit IRC11:23
*** Nisha_Agarwal has joined #openstack-ironic11:24
* TheJulia sighs11:29
TheJuliadtantsur: so ironic to 9 instead of 8, 8.0.0 was early pike11:29
dtantsurright11:29
dtantsurIIRC we follow SemVer, not named-releases-based versioning11:30
*** jkilpatr has joined #openstack-ironic11:30
*** Nisha_Agarwal has quit IRC11:31
TheJuliaI think the resource class changes does kind of warrant 9.0.011:31
TheJuliasince "if you don't do this, your going to break your installation" things exist11:31
*** Nisha_Agarwal has joined #openstack-ironic11:32
TheJuliadtantsur: how badly did CI go for a walk overnight?11:32
*** Nisha_Agarwal has joined #openstack-ironic11:32
TheJuliadid it actually board an airplane and go on vacation?11:32
dtantsurTheJulia: rather to the nearby river. vdrok has a patch for the hard breaking grenade problem, and I think the multinode grenade is still unstable.11:33
*** aparnav has quit IRC11:34
*** aparnav has joined #openstack-ironic11:34
TheJuliaThat is not too bad.11:34
*** mat128 has joined #openstack-ironic11:44
*** rpioso has joined #openstack-ironic11:46
rpiosoGood morning!11:46
rpiosohttps://review.openstack.org/#/c/491263/ needs Workflow+1 tender loving care :)11:48
patchbotpatch 491263 - ironic - Add 'hardware type' for Dell EMC iDRACs11:48
dtantsurrpioso: morning, done11:50
dtantsurrpioso: it will not merge though, until the CI is fixed11:50
vdrokHeh11:51
dtantsursee, you've got even twice workflow+1 :D11:51
vdrokMorning rpioso11:51
* TheJulia wonders if she should make that three :)11:52
* TheJulia holds off in case we need to trick gerrit later11:53
*** litao__ has quit IRC11:54
rpiosoGood morning, vdrok :)11:55
rpiosodtantsur: Thank you.  CI :(11:55
rpioso2 Workflow+1 > CI, though11:56
rpiosoGood morning, TheJulia :)11:57
TheJuliagood morning rpioso11:57
*** lucas-hungry is now known as lucasagomes11:59
*** trown|outtypewww is now known as trown12:04
*** jcoufal has joined #openstack-ironic12:10
*** dprince has joined #openstack-ironic12:15
*** rpioso is now known as rpioso|eat12:17
*** aparnav has quit IRC12:17
*** aparnav has joined #openstack-ironic12:18
fellypefcaGood morning everyone :)12:20
milanmorning fellypefca, rpioso|eat , TheJulia and all! :)12:28
Nisha_AgarwalTheJulia, morning12:31
Nisha_AgarwalTheJulia, need the favor for https://review.openstack.org/#/c/422902 again12:32
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface12:32
*** rpioso|eat has quit IRC12:34
*** sasha21 has joined #openstack-ironic12:39
TheJuliaNisha_Agarwal: Once the gate fix merges12:39
dtantsurvdrok: https://review.openstack.org/#/c/494118/ is going to fail multinode grenade :(12:39
patchbotpatch 494118 - ironic - Remove ensure_logs_exist check during upgrade12:39
openstackgerritMerged openstack/ironic master: [reno] update  https://review.openstack.org/49394712:40
TheJuliaspeaking of whichhttp://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/console.html#_2017-08-16_12_30_37_72977612:41
TheJuliaerr, http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/console.html#_2017-08-16_12_30_37_72977612:41
dtantsurffs, what is happening? :(12:43
*** dyasny_ has quit IRC12:44
*** hw_wutianwei has joined #openstack-ironic12:46
TheJulialooks like cleaning alone took an extra 4 minutes12:47
*** jlabarre has joined #openstack-ironic12:47
*** lucasagomes is now known as lucas-brb12:48
*** jaganathan has quit IRC12:49
TheJuliaapproached 1.5 hours and didn't even get through the first half it looks like :(12:51
vdrokdtantsur: yeah, the usual resources thing12:53
*** rloo has joined #openstack-ironic12:55
sambettswhy is nova attempting to do anything to a node thats in a transition "*ing" state??12:55
*** osdev has quit IRC12:55
*** mvpnitesh has quit IRC12:57
*** osdev has joined #openstack-ironic12:57
sambettsand there are no logs of the error in the nova conductor is seeing in the n-cpu ...12:57
sambettssomething does not look happy with rabbit in the ir-api logs... http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/old/screen-ir-api.txt.gz#_2017-08-16_12_01_36_06613:01
vdroksambetts: no, it's fine. we were filtering such messages before13:02
vdrokseems like we need to add one more filter13:02
*** karimb has quit IRC13:04
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall  https://review.openstack.org/47183113:04
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Allow periodics to terminate inspector  https://review.openstack.org/48790913:04
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644813:04
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871213:04
sambettsvdrok: ok ... so the request that 409d was a DELETE vifs request... I expect that it is related to https://review.openstack.org/#/c/422678/[3~13:05
patchbotpatch 422678 - nova - Change order of _cleanup_deploy and _unprovision i... (ABANDONED)13:05
sambettsbascally if it takes too long to move from cleaning ---> clean wait then nova will try to do stuff to it but the node is locked13:05
vdrokyup, it seems so13:06
pas-havdrok: dtantsur: i've tried to improve the scheduling in nova today too - https://review.openstack.org/#/c/494136/, take a look, should kind of help with rescheduling chances, especially when using resource classes13:07
patchbotpatch 494136 - nova - Shuffle best hosts from weighed hosts13:07
sambettsvdrok: I think we need to move unplug vifs to before we issue delete on the node13:08
fellypefcaTheJulia, dtantsur Hey, just a little doubt, do you think I need to open a bug/rfe for this patch? https://review.openstack.org/#/c/493983/13:08
patchbotpatch 493983 - ironic - [WIP/DNM] Updating the minimum version of python-o...13:08
TheJuliaNot to take away from the current discussion, but did anyone look at the subside logs?13:08
sambettsvdrok: other ways I've fixed it when I've seen that error downstream is to increase the nova-->ironic timeout13:09
dtantsurfellypefca: usually we do not bump versions for bug fixes, unless everything is broken. I'd like to see a bug13:09
TheJuliafellypefca: what dtantsur said. We don't understand the issue that actually caused it.13:10
fellypefcadtantsur TheJulia Ok! thanks.13:10
openstackgerritRuby Loo proposed openstack/ironic master: [releasenotes] update irmc's boot-from-volume support  https://review.openstack.org/49400713:11
dtantsurrloo: thanks, I was thinking about ^^^ too13:11
rloodtantsur: :)13:13
d0ugalIs there something special about introspecting more than 15 nodes at the same time?13:13
rloogood morning all, dtantsur, fellypefca, TheJulia, sambetts, vdrok, pas-ha13:13
pas-hamorning rloo :)13:13
rloothis is a dumb question, but can we turn off cleaning in the gate, temporarily?13:14
vdrokmorning rloo13:14
dtantsurd0ugal: PXE, iPXE, DHCP :) all of these can be special13:14
rlooif the cleaning is the issue right now?13:14
d0ugalA strange question, I realise. However, I am testing introspection with tripleo and I have 18 nodes - every time 3 nodes time out. I have rebuilt this env a number of times.13:14
vdroksambetts: it seems like that exception happens on the server build, not tear down13:14
dtantsurrloo: this may be an unfortunate answer to our problems..13:14
dtantsura;sp ,prmomg13:14
dtantsurwow, what was that?13:15
sambettsvdrok: its on a rescheduled build, so it tries to tear down the old instance first13:15
dtantsurit was supposed to be "also, morning" :D13:15
rloodtantsur: i was waiting for you to tell us :)13:15
sambettsvdrok: at least that is what it seems like13:15
dtantsurrloo: blind typing lets me down when I'm tired :)13:15
rloodtantsur: :( Hang in there, only a few more hours for you!13:16
*** cdearborn_ has joined #openstack-ironic13:16
dtantsurright, but I won't feel safe and relaxed, knowing that our release is hanging in the air..13:16
vdroksambetts: hrm http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz#_2017-08-16_12_26_35_47813:16
vdrokthis seems to be the first try to deploy13:17
TheJuliarloo: I have one issue with that reno revision, note added13:17
TheJuliadtantsur: or perhaps cleaning for multimode ?13:17
vdrokat least I don't see anything related to server d1e316bf-5395-45de-b011-4052e9eb23d9 before that13:17
vdrokchecked the primary too13:17
fellypefcaGood morning, rloo :)13:17
vdrokmorning fellypefca13:17
rlooTheJulia: just saw it. I removed the reference cuz in the 'property' description itself, it doesn't mention subnet mask.13:17
fellypefcaMorning vdrok o/13:18
TheJuliarloo: it should :(13:18
sambettsvdrok: why would nova schedule an instance on to a transition state node?13:18
openstackgerritHironori Shiina proposed openstack/ironic master: iRMC: Add documentation for remote volume boot  https://review.openstack.org/48962013:18
rlooTheJulia: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/irmc/boot.py#L7013:18
sambettsvdrok: the call thats failing there is actually the add_instance_info_to_node13:18
rlooTheJulia: also, most of that shouldn't even be in the reno, it is way too long. we should really put a link to something.13:18
vdroksambetts: no idea. yeah, I see that13:18
TheJuliarloo: perhaps the patch just updated?13:19
sambettsvdrok: which means that the node must be in some transition state before nova even triggers on it13:19
rlooTheJulia: huh?13:19
TheJuliahttps://review.openstack.org/48962013:19
patchbotpatch 489620 - ironic - iRMC: Add documentation for remote volume boot13:19
sambettsvdrok: is there a race somewhere, and we're changing something behind nova's back??13:19
rlooTheJulia: oh, yeah, maybe that. what state is it in? I'll look. We have a bit more time now before release, right?13:20
sambettsvdrok: or nova is scheduling onto an already scehduled node :/13:20
sambettsbecause of multinode races13:20
vdroksambetts: well, this should have failed then http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz#_2017-08-16_12_26_33_59613:20
TheJuliarloo: it was just updated, I'll have to take a look.13:20
vdrok*should* :)13:20
dtantsurrloo: yeah, we cannot (IMO) release with a broken CI and a lot of patches approved with a minus from Jenkins..13:21
vdrokit says that the node is ready for use, all the resources have used=013:21
vdrokdtantsur: sambetts: oh, huh http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz#_2017-08-16_12_26_33_88613:23
vdrokresource provider does not exist13:24
vdrokthen another request grabs the node, yeah13:24
vdrokbut then this http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz#_2017-08-16_12_26_34_05413:25
TheJuliarloo: the proposed doc reads a little awkwardly, and since it is doc, it can land quite easily.13:25
TheJuliarloo: that being said, it gets the point across, at least to me.13:25
rlooTheJulia: i'm commenting/editing it now. Although after I comment, we could land and there could be a follow up.13:25
TheJuliarloo: that could work, that way the reno just becomes mostly a link13:26
rlooTheJulia: err. I am only at the first paragraph though. Which means it could take me awhile to comment. If you think it is ready to land, maybe we should land. then I'll do a followup edit?13:26
sambettsvdrok: aren't the resource provider UUID meant to match the ironic node uuids??13:26
vdroksambetts: nope. name matches the uuid of the ironic node13:27
vdrokwhich is a bit unfortunate13:27
TheJuliarloo: if your wanting to correct the awkwardness, yes, I think that is best. I have only one nit that I stumble over but that is easily followed up13:27
rlooTheJulia: we have a plan!13:27
TheJulia\o/13:27
rlooTheJulia: did you note that here too, there is no mention of subnet mask?13:28
TheJuliait says network size, which is the same thing, just different way of stating it13:28
TheJuliathe example shows a CIDR example, which works13:28
sambettsvdrok: ah... ok ... errr... I'm very confused... this is talking to the placement API but there are no placement API logs in that test output13:29
rlooTheJulia: ah yes. I thought I had that too in the reno, but i think i copied from something that wasn't that clear :-(.13:29
vdroksambetts: wut. they should be in apache13:29
sambettsah right...13:29
sambettsle sigh...13:29
* vdrok looks too13:29
*** chlong_ has joined #openstack-ironic13:31
vdroksambetts: soo, node was in clean wait at that moment http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-ir-cond.txt.gz#_2017-08-16_12_25_45_84513:31
* dtantsur wonders which release we're upgrading to which13:31
dtantsurlike, nova already has stable/pike, but we don't..13:31
openstackgerritHironori Shiina proposed openstack/ironic master: iRMC: Add documentation for remote volume boot  https://review.openstack.org/48962013:31
vdrokdtantsur: we still do ocata to master13:32
sambettsvdrok: then nova shouldn't have even attempted to schdule onto it ...13:32
vdrokwhich means that if some breaking change happens before we cut pike, we are screwed :)13:32
TheJuliarloo: looks like hshiina edited the patch13:33
rlooTheJulia: yeah, just saw that. Like a slow-motion movie :)13:33
*** Nisha_Agarwal has quit IRC13:33
vdrokdtantsur: wrt the issue we're talking about with sambetts , it seems like nova tries to schedule on a node in cleanwait13:34
dtantsuralso https://github.com/openstack-dev/devstack/commit/6b354a909535a9dfef164c3e844544989d17514a broke inspector in one more way omg...13:34
sambettscleanwait/cleaning13:34
*** Nisha_Agarwal has joined #openstack-ironic13:34
*** osdev has quit IRC13:34
TheJuliarloo: a little better, +2'ing13:35
rlooTheJulia: I +2'd again :)13:35
*** jlabarre has quit IRC13:35
sambettsvdrok: by the error it seems like cleaning rather than clean wait,  because of "transition state" part of the error13:35
TheJuliarloo: +2+A'ed13:36
rlooTheJulia: :)13:36
vdroksambetts: in conductor logs it says clean wait at that moment13:36
dtantsurTheJulia, rloo, folks, I think we'll have to release ironic-inspector :( it's CI is broken too deeply by https://github.com/openstack-dev/devstack/commit/6b354a909535a9dfef164c3e844544989d17514a13:36
sambettsvdrok: clean wait isn't a locked state13:36
dtantsurs/it13:36
vdrokI think we consider any state transition if the target_provision_state is set13:36
dtantsurs/it's/its/13:37
sambettsvdrok: oh :/13:37
vdrokhrm13:37
sambettsdtantsur: blah...13:37
TheJuliadtantsur: have to release now with broken ci?13:37
sambettsdtantsur: so frustrating...13:37
rloodtantsur: if we know why the CI is broken and it isn't due to inspector, i am fine with releasing it13:38
dtantsurTheJulia: hopefully, stable/pike will work. we have 2 breakages due to grenade already defaulting to upgrade from Pike13:38
*** jlabarre has joined #openstack-ironic13:38
*** crushil has quit IRC13:38
TheJuliaugh13:38
*** mat128 has quit IRC13:38
TheJuliadtantsur: ugh * infinity13:39
rlooif we release, we can at least see if stable/pike will work. this is a candidate, not necessarily *final*.13:39
TheJuliaYeah, I guess we could back port a couple changes to the branch from there and cut a final release after branching.13:40
*** karimb has joined #openstack-ironic13:40
dtantsurrloo: ++ and anyway the code is working, we just cannot land new changes13:41
rloodtantsur: right. So go for it!13:41
sambettsvdrok: yup... clean wait it was ... my bad...13:42
sambettsvdrok: I guess the question is why is nova attempting to schedule onto a clean wait node...13:43
vdrokyup. I'm sure this has something to do with all the hacks we do while reporting resources13:43
vdrokin the virt driver13:44
sambettsthis isn't even port-upgrade though... this is like ... stable/ocata code right?/13:44
sambettspost-upgrade*13:44
vdroksambetts: no on master13:45
*** Nisha_Agarwal has quit IRC13:45
vdrokthe ocata tests passed fine13:45
sambettsall these logs are in the /old folder though13:46
openstackgerritFellype Cavalcante proposed openstack/ironic master: Updating the minimum version of python-oneviewclient  https://review.openstack.org/49398313:46
fellypefcadtantsur, TheJulia ^ I think it's more 'understandable' now. :)13:46
*** Nisha_Agarwal has joined #openstack-ironic13:46
TheJuliadtantsur: I wonder if we could cut the release, move the branch to be rebased off a commit from master.... >.>13:47
TheJuliasince, it is really just a tag.....13:47
dtantsurTheJulia: I think the release team won't like it13:48
dtantsuranyway, chances are high that ironic-inspector stable/pike will work after branches13:48
dtantsurwe have problems now because we essentially upgrade ocata->queens13:48
milandtantsur, +1 for me13:49
TheJuliadtantsur: I believe it has been done before, either way we have options13:50
vdroksambetts: yes, because we don't upgrade nova on multinode13:51
*** crushil has joined #openstack-ironic13:51
dtantsurmeanwhile, our nova patch was approved13:52
vdrokdtantsur: ++, it seems like for inspector cutting pike should be ok13:52
*** mjturek has joined #openstack-ironic13:52
dtantsurvdrok: could you please propose a backport of https://review.openstack.org/#/c/492964/ to pike once it merges?13:53
patchbotpatch 492964 - nova - Fix reporting inventory for provisioned nodes in t...13:53
dtantsurand please recheck if it does not :)13:53
vdrokdtantsur: okie13:53
*** lucas-brb is now known as lucasagomes13:53
rlooyay for approved nova patch!13:53
dtantsuryeah.. it took Vlad and myself 4 days since last Thursday to figure out13:54
sambettsvdrok: so basically something is going weird in stable/ocata nova's resourcing when we kill and restart/upgrade ironic13:56
vdroksambetts: it's a periodic task executing each minute, and the failure happens like 12 minutes after upgrade, so it seems to be master's issue13:57
sambettsvdrok: stable/ocata nova against master ironic13:58
vdrokah, right13:58
sambettsthe nova logic shouldn't be changing... so it must be getting confused about something when we restart/upgrade ironic13:59
*** hw_wutianwei has quit IRC13:59
*** chlong_ has quit IRC13:59
sambettserrr is this expected? http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz?level=TRACE#_2017-08-16_11_51_21_20614:00
*** rama_y has joined #openstack-ironic14:00
*** udesale has joined #openstack-ironic14:01
sambettsbasically something is going wrong with the placement api14:01
TheJuliasambetts: no, but I think that is before ironic is up and running for the first time14:01
vdrokyup, this is. dtantsur 's patch works around this14:01
vdroksambetts:14:01
vdroksambetts: https://review.openstack.org/#/c/494216/1 this is the backport14:02
patchbotpatch 494216 - nova (stable/pike) - Fix reporting inventory for provisioned nodes in t...14:02
vdrokto pike though14:02
sambettsseems like it was a major issue in ocata too...14:03
sambettsand I'm wondering if its related to what we're seeing14:03
sambettsbecause if the inventory isn't zero'd out in placement then aren't some of the nodes still "avaible" to be schduled onto?14:03
*** yangyapeng has joined #openstack-ironic14:03
sambettsI'm seeing a whole ton of http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz?level=TRACE#_2017-08-16_12_15_22_79114:04
sambettsvdrok: there is a lot of http://logs.openstack.org/18/494118/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dff3d51/logs/subnode-2/old/screen-n-cpu.txt.gz?#_2017-08-16_12_15_22_74914:09
TheJuliagrenade log shows networking-generic-switch  getting upgraded at the same time14:09
* dtantsur relocates home, back in an hour14:09
vdroksambetts: yup, I see that.14:10
*** dtantsur is now known as dtantsur|bbl14:10
vdroksorry, /me has to switch to another thing for a bit14:10
*** jaypipes has quit IRC14:14
*** ralonsoh_ has quit IRC14:16
*** mat128 has joined #openstack-ironic14:20
*** radez has joined #openstack-ironic14:23
radezdtantsur: ping, hey I had talked you about uefi in the past, I've got some baremetal nodes that are provisioning now, but when they reboot the drop to the grub prompt14:25
radezif I give it a configfile (hd0,gpt3)/boot/grub2/grub.cfg it boots fine14:25
radezthere are separate partitions for efi and boot14:26
radezany thoughts that could help?14:26
*** rama_y has quit IRC14:26
*** jaypipes has joined #openstack-ironic14:27
*** rbartal_ has quit IRC14:27
*** aparnav has quit IRC14:30
openstackgerritMichael Turek proposed openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401614:31
*** links has quit IRC14:32
*** aparnav has joined #openstack-ironic14:36
openstackgerritJulia Kreger proposed openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401614:42
openstackgerritMichael Turek proposed openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401614:42
TheJuliamjturek: you have whitespace ^^^14:43
*** srobert has joined #openstack-ironic14:45
mjturekwhoops!14:46
TheJulia:)14:46
TheJuliaI tried to fix it real quick, but then you edited it14:46
mjturekyeah just saw, I'll update again14:46
mjturekweird, it passed edocs on my machine14:47
mjturekthought that'd be checked14:47
TheJuliapep8 checks it14:47
TheJuliait seems weird14:47
*** makowals has quit IRC14:48
*** makowals has joined #openstack-ironic14:49
*** fellypefca has quit IRC14:52
openstackgerritMichael Turek proposed openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401614:53
*** dtantsur|bbl is now known as dtantsur14:58
* dtantsur is back but on a meeting14:59
dtantsurradez: no instant idea, unfortunately. I don't have much experience with grub+uefi14:59
*** crushil has quit IRC15:00
*** crushil has joined #openstack-ironic15:01
*** namnh has joined #openstack-ironic15:01
*** _milan_ has joined #openstack-ironic15:01
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up to ``ilo`` hardware type documentation  https://review.openstack.org/49408415:04
*** milan has quit IRC15:04
openstackgerritAkira Yoshiyama proposed openstack/virtualbmc master: Add Serial-over-LAN (SOL) support  https://review.openstack.org/48285315:06
*** racedo has quit IRC15:07
*** rcernin has quit IRC15:07
openstackgerritStephane Robert proposed openstack/ironic master: Baytech MRP27 PDU support  https://review.openstack.org/38771815:07
*** racedo has joined #openstack-ironic15:09
*** karimb has quit IRC15:12
*** mjura has quit IRC15:12
TheJuliarloo: https://review.openstack.org/#/c/494016/ since docs can still land, in theory15:12
patchbotpatch 494016 - ironic - Add admin guide for boot from volume15:12
rlooTheJulia: yes they should, am looking at it now.15:13
TheJulia10-415:13
dtantsurmjturek, TheJulia, sorry, a few things to fix15:15
dtantsurmainly, the storage interface is missing from the hardware interfaces documentation in the install guide15:15
TheJuliadtantsur: I think that can be followed up on, the first step goal was to just get a page up so we could link to it in the prelude15:16
dtantsurdunno, it's like 5 mins to fix...15:16
* TheJulia shrugs15:16
*** rpioso has joined #openstack-ironic15:18
rloodtantsur: I *just* want a link so I can update the reno for pike :)15:20
dtantsursorry :)15:21
*** aparna has joined #openstack-ironic15:21
rloodtantsur: no worries. just that gate is so slow, even for doc-only patches :-(15:22
dtantsuroh yeah15:22
*** Sukhdev has joined #openstack-ironic15:22
anupnGood Morning Ironic'ers :)15:22
dtantsurwell, we have enough time. we still cannot merge non-docs patches..15:22
dtantsurmorning anupn15:23
dtantsurthe nova patch merged \o/15:23
*** aparnav has quit IRC15:23
anupndtantsur, morning, TG hump day ;)15:24
rloodtantsur: yay!15:24
vdrokmorning anupn15:24
*** MattMan has quit IRC15:24
*** MattMan has joined #openstack-ironic15:24
anupnvdrok, morning :)15:24
*** appu103 has joined #openstack-ironic15:25
*** udesale has quit IRC15:25
mjturekdtantsur: updating now, thanks!15:26
anupnHi sambetts: while building tinyipa, i see weird warning "warning: no previously-included files matching '*.pyc' found anywhere in distribution15:26
anupn" and the building stops on this15:26
anupnsambetts, have you seen at this anytime?15:26
*** aparna has quit IRC15:28
sambettsanupn: hmm, if you do "export PYOPTIMIZE_TINYIPA=false" and then re-run the build using "make clean && make all" does it still happen15:29
anupnsambetts, i haven't set PYOPTIMIZE_TINYIPA=false, but we don't usually set that and want .pyo instead correct?15:32
anupnsambetts, but yes, let me try that15:32
sambettspyoptimize is true by default15:32
sambettsfor tinyipa15:32
anupnsambetts, ohh yeah15:33
*** links has joined #openstack-ironic15:33
openstackgerritMerged openstack/ironic master: iRMC: Add documentation for remote volume boot  https://review.openstack.org/48962015:34
anupnso hence right now it removes all .py and .pyc15:34
sambettsexactly15:34
anupnyo, let me check that15:37
*** aparnav has joined #openstack-ironic15:38
*** appu103 has quit IRC15:40
mjturekdtantsur: other guides don't seem to further indent second lines (ie: https://docs.openstack.org/ironic/latest/admin/portgroups.html) Did you want me to align the bottom args with the top ones?15:41
dtantsurmjturek: I'd say it looks nicer, but I don't insist15:42
mjtureknp, I'll add the indent!15:42
*** yolanda has quit IRC15:44
dtantsurthanks!15:44
*** yolanda has joined #openstack-ironic15:44
*** ccamacho has quit IRC15:44
*** ccamacho has joined #openstack-ironic15:46
*** links has quit IRC15:49
openstackgerritMichael Turek proposed openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401615:50
* dtantsur waits for Jenkins on ^^^15:50
* rloo waits for dtantsur who waits for ... :)15:51
dtantsurlol15:51
TheJuliaheh15:51
mjturek:)15:51
*** marios has quit IRC15:51
*** Sukhdev_ has joined #openstack-ironic15:53
*** rama_y has joined #openstack-ironic15:55
*** pcaruana has quit IRC15:57
*** jaypipes has quit IRC15:58
*** crushil has quit IRC16:03
vdrokdtantsur: do you know someone from devstack core? I have trouble finding the person to hit approve16:05
dtantsurvdrok: sdague?16:05
dtantsuryou can get the list from gerrit16:05
vdrokdtantsur: he's on vacation I think16:05
vdrokok, then I'll just ping everyone from that list closer to the end of my day :)16:06
rama_yHi vdrok, sambetts, Trying to deploy baremetal node and I am getting this error "No VIFs found...". Cleaning works. Any suggestions?16:11
rama_yHere is the error in logs: http://paste.openstack.org/show/618440/16:11
sambettsrama_y: how are you deploying? via nova?16:11
rama_yTried via Nova; getting "no valid host found"16:12
rama_yFails immediately16:12
Nisha_Agarwalrloo, thanks for the review on https://review.openstack.org/#/c/422902/2116:12
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface16:12
rama_yTried "ironic node-set-provision-state node-0 active" and I get the above error16:12
dtantsurrama_y: do you have ports (ironic) attached to nodes?16:12
rama_yYes16:13
sambettsthats a scheduling issue, https://docs.openstack.org/ironic/latest/admin/troubleshooting.html#nova-returns-no-valid-host-was-found-error16:13
rama_yCleaning works!16:13
Nisha_Agarwalrloo, i just now replied to your queries, please have a look and let me know if the explanations look ok16:13
rama_yI am able to bring the node to available state16:13
sambettsrama_y: if you deploy using ironic directly you have to manually attach tenent network ports to the ironic node via ironic vif attach16:13
*** trown is now known as trown|lunch16:13
dtantsurFYI the pike backport of the nova fix was also approved16:13
rlooNisha_Agarwal: ok, will look in a few min16:13
Nisha_Agarwalrloo, thanks16:13
sambettsrama_y: cleaning vifs are internal to ironic so Ironic creates them, tenant vifs are either created by nova or by the user16:14
rama_ysambetts: do I need to create a neutron port?16:14
Nisha_Agarwaldtantsur, yay16:14
sambettsrama_y: if you are deploying via ironic manually... but if you are in an OpenStack environment with nova you should really deploy via nova and it'll create and manage those ports for you16:14
sambettsrama_y: if you are getting no valid host found in nova that indicates there is a bigger problem in your environment than just missing vif on the node16:15
sambettsrama_y: most likely missing scheduling information on the node, or incorrectly set up flavors16:15
rama_ysambetts: ok16:15
rama_yHere is my node show: http://paste.openstack.org/show/618555/16:16
Nisha_Agarwaldtantsur, whats the progress on gates? and whats the plan on release?16:16
*** crushil has joined #openstack-ironic16:16
rama_ysambetts: properties are set on the node and I am trying to boot flavor 'baremetal'16:17
sambettsrama_y: that looks good at first glance, does your flavor match your node's properties exactly ?16:17
dtantsurNisha_Agarwal: not much so far :( we hope that TheJulia will release it tomorrow16:17
rama_ysambetts: no it doesn't match exactly16:18
Nisha_Agarwal:)16:18
dtantsurNisha_Agarwal: vdrok is looking for someone from devstack to approve our fix16:18
dtantsurwe'll then probably need a backport to pike and ocata16:18
Nisha_Agarwalok,16:18
rama_ysambetts: i will give it a try with a new flavor to match my node; is there anything else to look for, other than the flavor, for nova to pick up the node?16:18
* TheJulia sees her name mentioned, but she is in a meeting at the moment16:19
dtantsurit's ok, nothing critical16:19
Nisha_Agarwal:)16:19
sambettsrama_y: I expect that is your issue, flavors for baremetal have to exactly match the resources your baremetal node offers for it to work, you can check if nova sees the baremetal node by looking at "nova hypervisor-stats" and seeing if the ram and cpus are as expected16:20
sambettsif they are then just fixing the flavor *should* work #16:20
rama_ysambetts: I checked that; nova doesn't see the baremetal node16:21
sambettsrama_y: is the baremetal node in the avaiable state?16:21
sambettsrama_y: it has to be in that state and not in maintenance to be useable16:21
rama_ysambetts: ok, I will bring the node to available again and check. It is deploy failed at the moment.16:22
openstackgerritStephane Robert proposed openstack/ironic master: Adds snmp driver auto discovery for APC and baytech  https://review.openstack.org/38973616:25
*** stendulker has joined #openstack-ironic16:26
rlooNisha_Agarwal: I replied. Of those three, is is possible that a node will have BootMode capability?16:26
Nisha_Agarwalno16:27
rlooNisha_Agarwal: also, wrt max_raid_level. not sure if i understand that, was it actually set in a node, or do you mean 'raid_level" was set?16:27
rlooNisha_Agarwal: I don't understand how we let capability keys into the code, that were never used.16:28
rlooNisha_Agarwal: given that they were there and your patch is removing them, there needs to be an explanation in the patch.16:29
rama_yhi sambetts: nova sees the properties now: here is the paste: http://paste.openstack.org/show/618557/16:29
Nisha_Agarwalrloo, they were added as part of spec approval in kilo release16:30
*** aparnav has quit IRC16:30
Nisha_Agarwalrloo, if you want I can retain them but actually prolaintutelis doesnt discover tehm16:31
Nisha_Agarwalthem16:31
radezdtantsur: anyone you could suggest to ask about it?16:31
*** kromanenko has quit IRC16:32
dtantsurradez: mmmm, maybe ilo folks, Nisha_Agarwal or stendulker, they may have some good experience with UEFI16:32
*** ccamacho has quit IRC16:32
radezdtantsur: I think the solution is just writing an extra grub.conf file to the EFI partition that points to the boot partition16:32
radezdoesn't ironic handle creating those partitions?16:32
dtantsurradez: it does16:33
dtantsurradez: I don't quite get why "boot partition" is different from "EFI partition" though16:33
*** e0ne has quit IRC16:33
rlooNisha_Agarwal: it doesn't make sense to retain them if they aren't used. it doesn't make sense to me as to why they were there in the first place. So they should be removed, but I want it documented as to what is going on. I commented in the patch.16:33
radezdtantsur: agreed, I'd think it would be simpler to just through the efi stuff on the boot part with grub16:33
rlooNisha_Agarwal: you can choose to remove that change since it is outside the scope of that patch. up to you.16:34
Nisha_Agarwalrloo, ok can i do follow up of this?16:34
Nisha_Agarwali will place the followup in few min16:34
rlooNisha_Agarwal: no, you can't do a followup. because commit msgs can't be edited afterwards.16:34
rlooNisha_Agarwal: that patch needs to be updated. as far as i'm concerned.16:35
Nisha_Agarwalohk16:35
Nisha_Agarwalok. i will do it. i will address other comments as well16:35
radezNisha_Agarwal: stendulker: ping, I'm installing UEFI machine with ironic, when the machine boots it falls to a grub prompt. if I execute configfile (hd0,gpt3)/boot/grub2/grub.conf the machine boots fine. Any ideas what may be amiss?16:35
sambettsvdrok: going through a second set of logs for fail multinode and for the second time nova has tried to schedule onto a node in clean wait16:35
dtantsurfolks, seems like IPA pike CI does not feel too well.. https://review.openstack.org/#/c/494128/16:36
patchbotpatch 494128 - ironic-python-agent (stable/pike) - Update .gitreview for stable/pike16:36
vdrokdtantsur: hrm, it's just the timeout16:36
vdroksambetts: ugh.16:36
dtantsurah, ok16:36
dtantsurwe have too many jobs there, we need to urgently start reducing the count16:37
* dtantsur wants a standalone job with tinyipa there16:37
sambettsvdrok: and again this happens in the nova logs just before it schdules onto a node that isn't in a avaiable state http://logs.openstack.org/88/491288/4/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/65aebf7/logs/new/screen-n-cpu.txt.gz#_2017-08-16_01_40_10_02116:37
stendulkerradez: You may have to run the steps in this method manually and see if they go through successfully. https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/image.py#L7916:38
stendulkerradez: This method takes care of installing grub2 bootloader16:39
stendulkerradez: What kind of image are you using?16:39
vdroksambetts: I think I remember a patch doing deletion of resource provider if it's not available. but, that needs investigation anyway16:39
vdroks/it's/node16:39
radezstendulker: one built with dib16:40
radezhttps://images.rdoproject.org/aarch64/ocata/delorean/current-tripleo/stable/overcloud-full.tar16:40
stendulkeris it partition image or whole disk?16:40
dtantsurwe need to remove jlvillal-bot until we have the CI fixed :)16:40
dtantsurstendulker: I think tripleo uses partition images16:40
stendulkerok16:41
dtantsurbuilt out of centos in this case16:41
radezstendulker: it's a whole disk I thought?16:41
radezdtantsur: ^16:41
stendulkerradez: Are you setting the boot_option to local?16:41
radezyup16:41
dtantsurradez: I'm pretty sure it's a partition image by default, unless images for aarch are built differently16:41
stendulkerradez: If its whole disk then Ironic does not install bootloader. It will try to use one in the image16:42
radezohoh, you're right dtantsur I thought that backwards in my mind16:42
radezstendulker: dtantsur is right it's a partition image16:42
stendulkerWhole disk image should have all necessary EFI  partitions to make it boot16:42
stendulkerFor partition image, set 'boot_option' to 'local'  in node's 'instance_info' so that Ironic installs bootloader during provisioning16:43
radezprofile:compute,boot_option:local,boot_mode:uefi16:43
radezthat's my capabilities on all my nodes16:44
*** yangyapeng has quit IRC16:44
stendulkerand same thing in node->instance_info as well?16:44
radezinstance info?   /me checks16:44
dtantsurnote that instance_info is set by nova, not by an admin16:45
radezu'capabilities': u'{"profile": "compute", "boot_option": "local"}',16:45
stendulkeryes, if its nova boot.16:45
radezso it's missing uefi there16:45
*** crushil has quit IRC16:45
radezI think the bootloader is installed16:45
stendulkerthen you need to set the boot_mode to uefi in your nova flavor16:45
radezbut efi is on a different part than boot, so I think grub.efi is loading but can't find the kernel and such16:46
stendulkerbootloader will get installed in bios boot mode as well16:46
dtantsurI think the flavor does not have to be updated for UEFI, but I may be wrong in it16:46
* dtantsur never deployed with UEFI, unfortunately16:46
radezdtantsur: not for profile matching, but maybe to set the bootloader up corectly16:46
*** _milan_ has quit IRC16:46
dtantsurif you see grub.efi booting, then you already have UEFI, so dunno. you can try though16:47
dtantsurit won't hurt trying, right?16:47
radezstendulker: what will boot_mode efi in the flavor do for me?16:47
*** yangyapeng has joined #openstack-ironic16:47
radezI expect it won't hurt... question is if it will help :)16:48
*** rcernin has joined #openstack-ironic16:49
stendulkerradez: ok. it would use one in node's capabilities. instance_info not reqd16:49
radezack16:50
stendulkerthen we would have to debug the IPA code to see if those steps are giving any warning16:50
stendulkerAlso, some of the baremetals  needs it to be in UEFI mode to install grub2 bootloader for UEFI16:50
stendulkerThese are the steps followed by IPA to install grub2 in UEFI  https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/image.py#L7916:51
radezlooking through that now16:51
*** yangyapeng has quit IRC16:52
radezstendulker: so it is correct that the efi and boot partitions are different right? so shouldnt there be a grub.conf file written to the efi part so it can find boot?16:54
*** lucasagomes is now known as lucas-afk16:54
radezI only see where it writes grub.cfg to boot16:54
stendulkerwont grub-install take care of that?16:54
radezI'm not sure16:54
radezI would guess not if you give it the boot partition to install it on16:55
radezbut I'm not sure how grub.efi makes in on the efi part16:55
stendulkergrub2-install should have taken care of that. Better to try these commands manually and see if we are missing any step here16:57
*** dprince has quit IRC16:58
radezstendulker: kk, I'll see if I can fix it manually using the grub-install commands and let you know what I dig up, thx for your help16:58
stendulkerradez: np16:59
*** leyal has quit IRC17:00
*** strigazi_OFF is now known as strigazi17:00
* TheJulia exits meeting17:00
*** leyal has joined #openstack-ironic17:00
jlvillal:q17:00
* jlvillal grumbles17:01
TheJuliaI _think_ grub install mainly just makes sure that things are in-place file wise and then does the nvram change for "hey, boot this file" when it is setting efi boot.17:01
sambettsvdrok: I think I have an idea of what might be going on... we seem to have two instances of nova-compute running on the primary node after upgrade17:01
TheJuliabut, it has been like a year and a half since I mucked with that by hand.17:02
sambettsvdrok: and they are racing17:02
*** aparnav has joined #openstack-ironic17:02
vdrokwut17:02
TheJuliasambetts: that is amazingly awesome17:02
TheJuliaand makes a lot of sense :(17:02
*** mgoddard has quit IRC17:02
sambettsvdrok: yeah ... basically I've just realised there are two sets of n-cpu logs on the primary with the same timestamps but different information17:02
sambettsTheJulia: yeah... :/17:03
sambettsvdrok: http://logs.openstack.org/88/491288/4/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/65aebf7/logs/new/screen-n-cpu.txt.gz and http://logs.openstack.org/88/491288/4/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/65aebf7/logs/old/screen-n-cpu.txt.gz17:03
sambetts*mind splode*17:03
* TheJulia thinks tequila17:04
vdroksambetts: hrm, they are different17:04
sambettslol17:04
vdrokold one log finishes at 1:2617:04
*** crushil has joined #openstack-ironic17:04
vdroknew one starts at 1:2717:04
dtantsurOo17:04
vdrokI doubt nova would even allow to register two computes with the same name17:05
sambettsah nuts... I must have been misreading the timestamps..17:05
* sambetts was excited then...#17:06
vdrok:)17:06
vdrokphew17:06
dtantsurjlvillal: could you please avoid rechecking stuff until the CI is back to us?17:06
*** derekh has quit IRC17:06
vdrokif that was true, we'd have a REALLY hard release :D17:06
*** deray has joined #openstack-ironic17:07
*** Sukhdev_ has quit IRC17:09
dtantsurif it was not hard enough...17:09
*** karthiks has quit IRC17:10
*** ElCoyote_ has joined #openstack-ironic17:11
NobodyCamGood Morning Ironic'ers17:11
anupnGood Morning NobodyCam17:12
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290217:13
*** amarao has quit IRC17:13
NobodyCamMorning anupn :)17:13
Nisha_Agarwalrloo, Addressed the comments in above patch17:13
Nisha_Agarwal^^^17:13
derayNobodyCam, anupn morning o/17:13
NobodyCamMorning deray :)17:14
dtantsurmorning NobodyCam, deray17:14
NobodyCamhey hey Morning dtantsur :)17:14
TheJuliagood morning NobodyCam17:15
jlvillaldtantsur, Sure. I just stopped my watch script17:15
dtantsurcool :)17:15
derayTheJulia, dtantsur, good morning17:15
NobodyCamGood Morning TheJulia :)17:15
* TheJulia watches the tempest tests run on the other screen17:15
deraydtantsur, https://review.openstack.org/#/c/494118/ should make the gate jobs pass, it seems..17:16
patchbotpatch 494118 - ironic - Remove ensure_logs_exist check during upgrade17:16
vdrokmorning NobodyCam jlvillal17:16
dtantsurTheJulia: I'll have to run soon, I guess I leave the release in your capable hands :)17:16
NobodyCamGood Morning vdrok :)17:16
TheJuliaderay: As I understand it, that is what we are waiting for17:16
TheJuliadtantsur: Have a wonderful evening!17:16
jlvillalHi vdrok and everyone else :)17:17
dtantsurwe could wait till Monday, but ideally we should not wait too much, or something else will break us :)17:17
derayTheJulia, okay..17:17
dtantsurthough, at this rate, we may not push everything through the gate even tomorrow....17:17
vdrokdtantsur: I've pinged all the cores that seem to be in irc and not in japan, but no luck yet17:17
derayjlvillal, o/17:17
dtantsuranyway, I'll leave it up to you folks17:17
jlvillal:)17:17
dtantsurthanks vdrok.. also please add them to the review17:17
dtantsurthey may see it later17:18
vdrokyup17:18
dtantsurhave a great weekend everyone, see you on Monday17:18
vdrokgood night dtantsur17:18
dtantsurping me on hangouts/telegram if you need anything (I can't promise a quick response)17:18
*** dtantsur is now known as dtantsur|afk17:19
Nisha_Agarwalrloo, if any other comment plz leave it on the patch. I will update it asap17:20
*** e0ne has joined #openstack-ironic17:21
rlooNisha_Agarwal: looking...17:21
Nisha_Agarwalvdrok, when u find time plz see this again https://review.openstack.org/#/c/422902/17:21
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface17:21
Nisha_Agarwalrloo, thanks17:21
vdrokNisha_Agarwal: I don't think there is a need to rush when we have a broken gate. I doubt it gets better today.17:22
vdrokand it's time to leave for /me17:22
vdrokgood night everyone!17:22
sambettssame... and my brain is fried17:22
openstackgerritStephane Robert proposed openstack/ironic master: Caching of PDU autodiscovery  https://review.openstack.org/39923417:22
sambettsnight vdrok17:23
*** sambetts is now known as sambetts|afk17:23
sambetts|afknight all o/17:23
vdroksambetts|afk: good night17:23
NobodyCamnight sambetts|afk17:23
sambetts|afkcya NobodyCam17:23
NobodyCamnight vdrok17:23
NobodyCam:017:23
NobodyCam:)17:23
anupngood night sambetts|afk17:23
rloogood night sambetts|afk and vdrok. Thx for looking into the gate!17:24
rloogood morning NobodyCam :)17:24
NobodyCamGood Morning rloo :)17:25
openstackgerritJulia Kreger proposed openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401617:25
sambetts|afkrloo: I wish I could have got further debugging it tbh ... but its a real swamp of logs17:25
Nisha_Agarwalvdrok, :) ok17:25
rloosambetts|afk: at least you were debugging!17:25
TheJuliarloo: ^^^ if you want to +2+A, i fixed the nit17:26
sambetts|afkrloo: heh yeah17:26
sambetts|afknight!17:26
*** dprince has joined #openstack-ironic17:26
rlooTheJulia: thx.17:26
anupnderay, morning :)17:26
rlooTheJulia, mjturek: who wants to address dtantsur|afk's comment? I'm giving you first dibs on it :)17:30
TheJuliaI'm working on it17:30
*** namnh has quit IRC17:30
rlooThx TheJulia!17:31
*** Sukhdev has quit IRC17:32
anupnHi rloo, TheJulia: anyone tried to clean tinyipa sometime? I am unable to clean it because it can't remove proc/ under directory tinyipafinal/17:34
*** e0ne has quit IRC17:34
rlooanupn: dunno, not me. i hate cleaning ;)17:34
anupnrloo, :)17:35
*** dsneddon is now known as dsneddon_pto17:36
mjturekd'oh. Sorry dtantsur|afk, I missed that one17:36
TheJuliayeah, not a five minute update either :(17:39
rlooTheJulia: i was wondering about that, so thx for volunteering!17:39
*** jtomasek is now known as jtomasek|afk17:41
*** mjturek has quit IRC17:43
*** mjturek has joined #openstack-ironic17:44
TheJuliabuilding now17:47
*** stendulker has quit IRC17:50
*** trown|lunch is now known as trown17:52
*** milan has joined #openstack-ironic17:53
*** strigazi is now known as strigazi_OFF17:55
openstackgerritJulia Kreger proposed openstack/ironic master: Add storage interface to enabling-drivers doc  https://review.openstack.org/49428317:56
TheJuliarloo: ^^^^17:56
TheJuliaany other cores ^^^^17:56
rlooTheJulia: thx17:57
*** e0ne has joined #openstack-ironic17:57
TheJuliaI guess in theory, redfish hardware type will work too17:58
TheJuliabut in theory only :)17:59
*** mgoddard has joined #openstack-ironic18:01
TheJuliaanupn: sorry, I've only built tiny ipa by hand once and it was with the scripts18:06
*** ChubYann has joined #openstack-ironic18:07
*** e0ne has quit IRC18:07
anupnTheJulia, np, i cloned new repo, and i figured i was having trouble because my instance was active with the tinyipa image and i was trying to delete those ramdisk/kernel18:08
*** e0ne has joined #openstack-ironic18:08
*** e0ne has quit IRC18:08
TheJuliaAnd 494118 is in the gate18:09
*** pcaruana has joined #openstack-ironic18:09
TheJuliarloo: how about s/life/lifecycle/?18:12
TheJuliaerr18:12
TheJuliano, lifetime works better18:12
TheJuliabecause it refers to the instance18:12
rlooyeah. no cycle there.18:12
rloounless you mean the big cycle of life... :)18:13
TheJuliaif we were talking about the node it's self, then it would be lifecycle18:13
rlooTheJulia: right18:13
rlooTheJulia: but we're talking about fleas on cows18:13
rlooor cattle or something like that, just not dogs.18:13
openstackgerritJulia Kreger proposed openstack/ironic master: Add storage interface to enabling-drivers doc  https://review.openstack.org/49428318:13
TheJuliawell, the fleas get on both the pets and the cattle18:14
TheJulia:)18:14
TheJuliarloo: I took the bit out of the note.  We don't want to draw too much attention but the information should be there...18:15
rlooyay, devstack patch was approved: https://review.openstack.org/#/c/493989/18:16
patchbotpatch 493989 - openstack-dev/devstack - Increase host_subset_size for ironic18:16
rlooTheJulia: thx!18:19
*** makowals_ has joined #openstack-ironic18:21
*** sasha21 has quit IRC18:23
*** yuriyz has quit IRC18:28
*** sasha21 has joined #openstack-ironic18:29
*** mgoddard has quit IRC18:41
*** ElCoyote_ has quit IRC18:42
*** Sukhdev has joined #openstack-ironic18:52
openstackgerritAparna proposed openstack/ironic master: Support SUM based firmware update as clean step for iLO drivers  https://review.openstack.org/42257218:52
*** e0ne has joined #openstack-ironic18:54
*** _milan_ has joined #openstack-ironic18:55
*** aparnav has quit IRC18:56
*** milan has quit IRC18:57
*** _milan_ has quit IRC18:57
*** ElCoyote_ has joined #openstack-ironic18:58
*** _milan_ has joined #openstack-ironic18:59
*** deray has quit IRC19:00
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/49416519:03
*** sasha21 has quit IRC19:03
TheJuliahopefully ~20-30 minutes on 49411819:06
*** lihi has quit IRC19:13
TheJuliaOne oneview folks around?19:15
TheJuliahttps://review.openstack.org/#/c/493983/19:16
patchbotpatch 493983 - ironic - Updating the minimum version of python-oneviewclient19:16
TheJuliatl;dr I feel like the release note is way too verbose.19:16
*** lihi has joined #openstack-ironic19:16
TheJuliaand actually, that should be upgrade19:17
*** mjturek has quit IRC19:22
*** mjturek has joined #openstack-ironic19:22
TheJuliaone more test....19:23
*** adisky__ has quit IRC19:23
openstackgerritMerged openstack/ironic master: Remove ensure_logs_exist check during upgrade  https://review.openstack.org/49411819:29
*** trown is now known as trown|brb19:29
openstackgerritMerged openstack/ironic master: Add admin guide for boot from volume  https://review.openstack.org/49401619:29
openstackgerritMerged openstack/ironic master: Add storage interface to enabling-drivers doc  https://review.openstack.org/49428319:29
rlooyay, gate is unblocked! https://review.openstack.org/#/c/494118/19:30
patchbotpatch 494118 - ironic - Remove ensure_logs_exist check during upgrade (MERGED)19:30
TheJuliayup, just rechecked the idrac hw type19:30
TheJulialooks like we're going to have to wait for 493421 to finish before we can recheck it19:31
TheJuliarloo: do you want to revise the prelude and we can just +A it once the idrac hw type lands?19:35
rlooTheJulia: yup, now that it looks like things are working again :)19:35
TheJulia:)19:35
TheJuliaYeah, there ism a lot churning on the gate, so it is going to be at least... six to eight hours :(\19:36
rlooTheJulia: that's fine with me :)19:36
rlooTheJulia: better late than never!19:36
TheJulia:)19:37
*** trown|brb is now known as trown19:39
openstackgerritRuby Loo proposed openstack/ironic master: [reno] Add prelude for Pike release  https://review.openstack.org/49390619:41
TheJuliaI'll check back in on the gate in ~2 hours, since it looks like there are plenty of resources in the gate at the moment19:41
rlooTheJulia: thx19:42
*** Sukhdev has quit IRC19:43
*** yangyapeng has joined #openstack-ironic19:44
*** yangyapeng has quit IRC19:49
*** rcernin has quit IRC19:55
openstackgerritRuby Loo proposed openstack/ironic master: [releasenotes] update irmc's boot-from-volume support  https://review.openstack.org/49400719:59
*** _milan_ has quit IRC20:01
*** milan has joined #openstack-ironic20:01
*** Sukhdev has joined #openstack-ironic20:13
*** srobert has quit IRC20:13
*** mat128 has quit IRC20:17
*** dprince has quit IRC20:18
*** trown is now known as trown|outtypewww20:28
*** fellypefca has joined #openstack-ironic20:33
*** makowals_ has quit IRC20:36
*** e0ne has quit IRC20:36
*** jcoufal_ has joined #openstack-ironic20:36
*** fellypefca has quit IRC20:36
*** jcoufal has quit IRC20:39
*** pcaruana has quit IRC20:41
jlvillalrloo, TheJulia: Do we think the gate is working again?20:44
rloojlvillal: yes, i hope so anyway. one patch merged and the devstack patch merged.20:44
rloojlvillal: am waiting to see what happens with the patches that were recheck'd...20:45
TheJuliaThe grenade tests passed 4x for the in tree patch to land, so I have hope :)20:45
rloowhat I don't know, is whether it is urgent for the devstack backports to land: https://review.openstack.org/#/q/I0874fe3b3628cb3e662ee01f24c4599247fdc82d,n,z20:47
jlvillalrloo, Thanks20:47
*** jlabarre has quit IRC20:47
rloojlvillal: yw. i suppose we should update the etherpad but i'm in the middle of two doc changes...20:47
*** jlabarre has joined #openstack-ironic20:47
*** rpioso has quit IRC20:48
*** rpioso has joined #openstack-ironic20:48
*** rpioso has quit IRC20:48
TheJuliaI'll take a look at the couple failures I see in a little bit and update the etherpad20:49
*** jcoufal_ has quit IRC20:52
*** ElCoyote_ has quit IRC20:54
radezslagle: ping, are you familiar with either of these messages:20:54
radezos-collect-config[2023]: No hosts in Heat, nothing written.20:54
radezcrap, I lost the other one... sec20:55
TheJuliaWow, os-collect-config, that brings back some memories20:56
radezyea sry, just realized this is more triple-o related than ironic20:57
TheJuliano worries20:57
TheJuliaone of the failures, specifically on the recheck of the idrac hw type is a job timeout. It took devstack 40 minutes to stand-up :(20:59
*** vishwana_ has quit IRC21:03
*** _milan_ has joined #openstack-ironic21:03
*** milan has quit IRC21:03
*** vishwanathj has joined #openstack-ironic21:04
*** crushil has quit IRC21:07
*** ElCoyote_ has joined #openstack-ironic21:11
*** rama_y has quit IRC21:18
TheJuliajlvillal: I'm guessing some of those early rechecks didn't actually start with the patch merged :\21:19
TheJuliasince there is the main review server, and ?9? git servers things are replicated down to... might have taken a little while :(21:20
jlvillalTheJulia, Okay. The patch merged at 12:29 for the grenade issue21:20
TheJuliaYeah, I'm saying that even minutes after that, some git servers that code was pulled from might not have had an up to date master branch21:21
jlvillalTheJulia, I think it should propogate very fast. Can look in the logfile to see what it used as the base of Ironic21:21
jlvillalTheJulia, Do you have a pointer to something specific? A failed build?21:21
TheJuliahttp://logs.openstack.org/75/490675/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/417c978/21:22
jlvillalLooking21:22
jlvillalTheJulia, That failed before it even got to the new portion of grenade I think.21:23
TheJuliaoh......21:24
TheJuliathe log fix21:24
jlvillalHmmm Failed during upgrade to ironic21:24
TheJuliait needs to be backported21:24
jlvillalI'm not sure21:24
jlvillalTheJulia, Well I doubt it, if the fix merged..21:24
TheJuliagrenade log looks like it couldn't find ir-cond.log21:25
TheJuliaone ironic rev now in the gate21:25
TheJuliagah, one of the grenade smoke tests just failed21:28
jlvillalTheJulia, You are correct. That patch did not take affect for the logs you posted.21:30
jlvillalhttp://logs.openstack.org/75/490675/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/417c978/logs/grenade.sh.txt.gz#_2017-08-16_18_31_34_28621:31
jlvillalTheJulia, The job started the recheck at 12:34 and the fix merged at 12:29. So you are correct in the lag.21:32
TheJuliawoot!21:32
jlvillalTheJulia, Things I Learned. I thought it was almost instantaneous.21:32
*** jlabarre has quit IRC21:33
*** rama_y has joined #openstack-ironic21:33
jlvillalTheJulia, Oh wait!21:34
jlvillalTheJulia, Your recheck was at 12:34, but there was already a running jenkins job21:34
jlvillalTheJulia, That job finished at 12:37...21:34
jlvillalTheJulia, It had probably started several hours before.21:34
TheJuliaugh21:34
jlvillalTheJulia, A recheck is in progress for it now21:35
TheJulialooks like we are still getting sporadic grenade failures, but at least it passes sometimes now21:35
openstackgerritRuby Loo proposed openstack/ironic master: [docs] update irmc boot-from-volume  https://review.openstack.org/49432921:41
openstackgerritMerged openstack/ironic-python-agent stable/pike: Update .gitreview for stable/pike  https://review.openstack.org/49412821:43
openstackgerritMerged openstack/ironic-python-agent stable/pike: Update UPPER_CONSTRAINTS_FILE for stable/pike  https://review.openstack.org/49412921:43
TheJuliait looks like random grenade failures and sporadic timeouts :(21:49
rloo:( random is better than 100% failure...21:50
jlvillal+0.5 ;)21:52
*** vishwana_ has joined #openstack-ironic21:54
TheJuliamy gut feeling is that 491263 will just timeout due to some reason deep inside the standalone job21:55
TheJuliaalthough I have high hopes for https://review.openstack.org/#/c/493421/ :)21:56
patchbotpatch 493421 - ironic - Fix DRAC classic driver double manage/provide21:56
TheJuliaI updated the etherpad with a new status. Time to wait some more21:57
*** vishwanathj has quit IRC21:58
*** priteau has quit IRC21:59
rloothx TheJulia21:59
TheJuliaat least... one job is in the gate queue....21:59
*** cdearborn_ has quit IRC22:07
*** _milan_ has quit IRC22:07
*** _milan_ has joined #openstack-ironic22:08
*** jkilpatr has quit IRC22:09
* TheJulia gets a cursed sign out22:19
TheJuliahttp://logs.openstack.org/17/473717/21/gate/gate-ironic-python35/2a4c759/console.html22:19
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: [TEST] [DO NOT MERGE] Testing tls-proxy enabling  https://review.openstack.org/49266422:21
openstackgerritRuby Loo proposed openstack/ironic master: [admin docs] ironic -> openstack baremetal CLI  https://review.openstack.org/49433522:38
rlooTheJulia: what's with the timeouts? :-(22:40
rlooTheJulia: we think the gate is random, so I think jlvillal should resurrect (again) his script and recheck 49342122:41
rlooTheJulia: i didn't actually look at 493421, is it really a dependency for iDRAC hw type?22:41
TheJuliaWe're basically hitting variable performance. I think we need to consider revising out job timeouts22:42
jlvillalrloo, Started up the watcher script again on 49342122:43
TheJuliarloo: not looking at review at the moment, but if it is the double provide fix, I believe so22:43
rlooTheJulia: yup, sorry should have mentioned it. ok, too bad.22:43
rloothx jlvillal!22:43
TheJuliaI'm waiting for dinner to appear with the family, will check back in a little bit22:44
rlooTheJulia: I have to take off too.22:44
rloojlvillal: am hoping this will work but maybe add this to your recheck, i think it is the only non-doc patch that is blocking the release: https://review.openstack.org/#/c/473717/22:45
patchbotpatch 473717 - ironic - Add functional API tests for volume connector and ...22:45
jlvillalrloo, done22:45
rloothx jlvillal!22:46
rloociao, might be back later...22:46
*** rloo is now known as rloo_afk22:47
*** jkilpatr has joined #openstack-ironic22:49
openstackgerritAkira Yoshiyama proposed openstack/virtualbmc master: Add Serial-over-LAN (SOL) support  https://review.openstack.org/48285322:51
*** makowals has quit IRC23:06
*** _milan_ has quit IRC23:10
*** _milan_ has joined #openstack-ironic23:11
*** makowals has joined #openstack-ironic23:11
*** Nisha_Agarwal has quit IRC23:15
*** patchbot has quit IRC23:17
jlvillalTheJulia, rloo:( Job in the gate failed on the Python 3 unit tests. Seriously???   And multi-node grenade.23:20
jlvillalhttps://review.openstack.org/#/c/473717/23:22
*** patchbot has joined #openstack-ironic23:23
* jlvillal updates his local ironic, cherry-picks the patch, and tries the python 3 tests locally.23:24
jlvillalAnd it passed23:24
TheJuliaUgh23:27
*** rama_y has quit IRC23:31
*** mjturek has quit IRC23:44
*** yangyapeng has joined #openstack-ironic23:45
*** yangyapeng has quit IRC23:54

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!