Wednesday, 2019-07-24

*** ianychoi has quit IRC00:18
*** ianychoi has joined #openstack-ironic00:20
*** gyee has quit IRC00:31
*** ricolin has joined #openstack-ironic00:59
*** igordc has quit IRC01:04
openstackgerritDigambar proposed openstack/ironic stable/rocky: DRAC: Fix OOB introspection to use pxe_enabled flag in idrac driver  https://review.opendev.org/64836001:12
openstackgerritVanou Ishii proposed openstack/ironic master: Add logic to determine Ironic node is HW or not into configure_ironic_dirs  https://review.opendev.org/67195701:18
*** mkrai_ has joined #openstack-ironic01:19
*** rloo has quit IRC01:49
*** zzzeek has quit IRC02:16
*** rh-jelabarre has quit IRC02:18
*** zzzeek has joined #openstack-ironic02:19
openstackgerritMerged openstack/ironic master: Clean lower-constraints.txt  https://review.opendev.org/67174003:20
*** dsneddon has quit IRC03:40
*** dsneddon has joined #openstack-ironic03:44
*** gkadam has joined #openstack-ironic03:50
*** gkadam has quit IRC03:50
*** whoami-rajat has joined #openstack-ironic03:52
*** igordc has joined #openstack-ironic03:58
*** gregoryo has joined #openstack-ironic03:59
*** igordc has quit IRC04:01
*** pcaruana has joined #openstack-ironic04:27
*** kbaegis has joined #openstack-ironic04:32
*** kbaegis has quit IRC04:40
*** jaypipes has joined #openstack-ironic04:49
*** jmccrory has quit IRC04:52
*** jmccrory has joined #openstack-ironic05:04
*** pcaruana has quit IRC05:12
*** jmccrory has quit IRC05:14
*** jmccrory has joined #openstack-ironic05:14
*** pcaruana has joined #openstack-ironic05:25
*** jmccrory has quit IRC05:41
*** rachit7 has joined #openstack-ironic05:46
*** jmccrory has joined #openstack-ironic06:11
*** hwoarang has quit IRC06:21
*** hwoarang has joined #openstack-ironic06:25
arne_wiebalckgood morning, ironic!06:30
*** dtantsur|afk is now known as dtantsur06:35
*** jawad_axd has joined #openstack-ironic06:35
dtantsurgood hot July morning, ironic :)06:35
*** iurygregory has joined #openstack-ironic06:40
iurygregorymorning Ironic o/06:42
dtantsurrpittau|afk: sorry for so many iterations on your SDK patches, I cannot properly think in this weather06:42
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Support for testing adoption in the standalone job  https://review.opendev.org/67228606:43
*** dsneddon has quit IRC06:46
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228606:52
openstackgerritMerged openstack/ironic master: Fix serial/wwn gathering for ansible+python3  https://review.opendev.org/67231007:01
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: Fix serial/wwn gathering for ansible+python3  https://review.opendev.org/67240707:04
*** rcernin has quit IRC07:04
*** tesseract has joined #openstack-ironic07:05
*** dsneddon has joined #openstack-ironic07:12
*** rpittau|afk is now known as rpittau07:27
rpittaugood morning ironic! o/07:27
iurygregorymorning rpittau07:28
*** dsneddon has quit IRC07:29
rpittaudtantsur: are you kidding? No worries at all! Thanks for taking the time to review and keeping an eye on that! :)07:31
rpittauhey iurygregory :)07:31
*** hjensas has joined #openstack-ironic07:35
*** tssurya has joined #openstack-ironic07:39
dtantsuryou're welcome :)07:40
rpittaummm something weird is happening with uppper-constraints07:42
rpittauoh I see they're discussing already on infra07:42
rpittauyay the gate is broken07:44
dtantsuryep, even running tests locally is broken :( I had to prepare a local upper-constraints file07:45
rpittaudtantsur: doing the same :/07:45
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Release Notes support  https://review.opendev.org/67228007:51
*** hjensas has quit IRC07:51
*** lucasagomes has joined #openstack-ironic07:51
*** mkowalski has joined #openstack-ironic07:58
*** e0ne has joined #openstack-ironic07:58
*** gregoryo has quit IRC08:10
openstackgerritLi Yingjun proposed openstack/ironic-python-agent master: Fixes physical memory fetching for aarch64  https://review.opendev.org/67241708:11
*** Goneri has joined #openstack-ironic08:11
*** dougsz has joined #openstack-ironic08:13
iurygregoryrpittau, yup 2.4.1 pyparsing08:16
iurygregoryXD08:16
iurygregorythey have a tag in the repository but its not on pypi last time I checked08:17
rpittauiurygregory: the apocalypse is happening, that issue breaks pip-compile08:17
iurygregoryrpittau, woa08:18
* iurygregory wonders if it's zombie apocalypse 08:19
*** priteau has joined #openstack-ironic08:19
*** jtomasek has joined #openstack-ironic08:25
*** dsneddon has joined #openstack-ironic08:25
openstackgerritMerged openstack/metalsmith master: Bring formatting in line with other OpenStack CLI  https://review.opendev.org/67199508:37
*** dsneddon has quit IRC08:42
*** priteau has quit IRC08:43
*** priteau has joined #openstack-ironic08:44
*** priteau has quit IRC08:44
*** iurygregory has quit IRC08:46
*** S4ren has joined #openstack-ironic08:47
*** altlogbot_0 has quit IRC08:51
*** irclogbot_3 has quit IRC08:51
*** altlogbot_3 has joined #openstack-ironic08:52
*** irclogbot_0 has joined #openstack-ironic08:52
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228609:03
*** derekh has joined #openstack-ironic09:13
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from raid to jbod  https://review.opendev.org/67103809:21
*** priteau has joined #openstack-ironic09:31
*** dsneddon has joined #openstack-ironic09:38
*** rachit7 has quit IRC09:41
*** eandersson has quit IRC09:42
*** iurygregory has joined #openstack-ironic09:48
*** dsneddon has quit IRC09:52
*** rachit7 has joined #openstack-ironic10:01
*** ajya[m] has quit IRC10:04
*** trident has quit IRC10:06
*** trident has joined #openstack-ironic10:08
*** csmart has quit IRC10:08
rpittaudtantsur: perfect timing :D10:38
dtantsurheh10:39
iurygregorythanks for the reviews rpittau dtantsur o/10:39
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228610:43
dtantsurnp10:45
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228610:45
openstackgerritMerged openstack/ironic-prometheus-exporter master: Release Notes support  https://review.opendev.org/67228010:45
rpittauiurygregory: wait before thanking, now you'll have to add relese notes all the time! :D10:46
iurygregoryrpittau, hehehehe =P10:47
iurygregorynp I would say, except if need to push for the current patches that I have open  =X10:48
openstackgerritIlya Etingof proposed openstack/ironic master: [WIP] Add iDRAC boot interface  https://review.opendev.org/67249810:51
*** csmart has joined #openstack-ironic10:51
*** Lucas_Gray has joined #openstack-ironic11:15
TheJuliagood morning everyone11:18
*** iurygregory has quit IRC11:19
*** Lucas_Gray has quit IRC11:21
*** Lucas_Gray has joined #openstack-ironic11:24
* TheJulia tries to wake up11:25
*** iurygregory has joined #openstack-ironic11:26
dtantsurmorning TheJulia11:27
*** iurygregory has quit IRC11:27
*** irclogbot_0 has quit IRC11:33
*** irclogbot_3 has joined #openstack-ironic11:34
TheJuliagood morning dtantsur11:37
TheJuliadtantsur: thank you for picking up adoption in standalone testing. I feel bad I've never gotten back to it11:37
dtantsurwe're all too busy11:37
dtantsurnow I'm really curious how it's going to work out with VIF attachment :)11:38
*** jcoufal has joined #openstack-ironic11:42
TheJuliaI guess you saw my note :)11:45
TheJuliaI'm kind of curious too actually11:45
*** ajya[m] has joined #openstack-ironic11:46
*** dsneddon has joined #openstack-ironic11:49
dtantsuryep :)11:53
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228611:54
* TheJulia changes locations to the mobile command center11:56
xXraphXxhello :). I've got a question regarding root device hints, more precicely here is the test:12:00
xXraphXxhttps://paste.ubuntu.com/p/Cf4FDzFdqS/12:00
xXraphXxI was wondering why the output had to be in lower case ?12:01
xXraphXxActually asking this because it triggers an  issue in the ansible module when one tries to provide the serial as device hint12:02
*** rh-jelabarre has joined #openstack-ironic12:03
xXraphXxDuring deployment the serial is digested (and set to lower case in deploy.py:_parse_root_device_hint) before being confronted in root_hint.py to the real serial (and thus not matching).12:03
*** dsneddon has quit IRC12:06
TheJuliaI... forgot... how slow... logs.o.o can be from my mother and law's house12:06
*** dsneddon has joined #openstack-ironic12:08
*** dsneddon has quit IRC12:15
TheJuliaHm... my upgrade of tinycore from 8 to 10 failed because of c code unhappiness :(12:20
dtantsurxXraphXx: I think we're trying to do it in a case-insensitive way to simplify life for operators12:30
dtantsursince different hardware may report things differently12:30
dtantsurwhat you describe may be (yet another, doh!) bug in the ansible deploy12:31
xXraphXxok I see how it can easily be fixed, incoming12:31
TheJulia\o/12:31
dtantsurxXraphXx: thanks!! I wonder if we should improve the CI coverage for the ansible module somehow..12:32
*** dsneddon has joined #openstack-ironic12:37
*** mkowalski has quit IRC12:39
*** mkrai_ has quit IRC12:43
*** mkrai has joined #openstack-ironic12:43
*** mkrai has quit IRC12:48
*** dsneddon has quit IRC12:50
TheJuliaIt would be good, but the bugs largely seem to be either py3 related or related to cases slightly off the main path, like root device hints support in the module with case12:51
*** e0ne has quit IRC12:57
openstackgerritJulia Kreger proposed openstack/ironic-python-agent-builder master: Update tinycore from 8.x to 10.x  https://review.opendev.org/66659112:57
*** e0ne has joined #openstack-ironic12:57
*** jawad_axd has quit IRC13:01
*** dsneddon has joined #openstack-ironic13:04
TheJuliaI hope ^^^ works13:16
*** dsneddon has quit IRC13:16
TheJuliabecause if it does, then maybe we will be able to test with it and v613:17
*** bfournie has joined #openstack-ironic13:18
* dtantsur fingers crossed13:22
*** dsneddon has joined #openstack-ironic13:43
*** dsneddon has quit IRC13:50
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228613:53
dtantsurthe last run was past adoption, but I made a silly mistake ^^13:54
*** dsneddon has joined #openstack-ironic13:57
openstackgerritraphael.glon proposed openstack/ironic master: Ansible driver: fix deployment with serial specified as root device hint  https://review.opendev.org/67253013:59
openstackgerritJulia Kreger proposed openstack/ironic-python-agent-builder master: Update tinycore from 8.x to 10.x  https://review.opendev.org/66659114:00
*** dsneddon has quit IRC14:04
*** cdearborn has joined #openstack-ironic14:11
cdearborngood morning ironic!14:11
cdearbornwanted to seek advice on implementing a new cleaning step for iDRAC14:12
cdearbornthe OOB cleaning step is to take the iDRAC out of recovery mode, but when the iDRAC is in recovery mode, it rejects set_boot_device call and power on call.  these calls happen during cleaning as a result of ironic trying to PXE boot the ramdisk to run IB cleaning steps of which there are none14:13
cdearbornso to take the iDRAC out of recovery mode in a cleaning step, we would have to first take the iDRAC out of recovery mode - chicken and egg14:14
cdearbornis there any way to make ironic skip PXE booting the ramdisk for IB cleaning steps?14:15
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316914:16
*** lennyb has quit IRC14:18
*** dsneddon has joined #openstack-ironic14:21
TheJuliacdearborn: hmmmmmm14:22
TheJuliacdearborn: and I take it priority order doesn't work because we set it up first?14:23
cdearbornTheJulia, well, it's a manual cleaning step, so priority is 014:23
TheJuliahmmm14:24
TheJuliathat is a chicken and egg14:24
*** iurygregory has joined #openstack-ironic14:25
cdearbornTheJulia, ideally, it would be great if ironic knew there were no IB cleaning steps and skip the PXE boot14:26
cdearbornTheJulia, or thoughts on running OOB cleaning steps first?14:26
TheJuliaThat would be the only real way to hide it...14:26
TheJuliamaybe a new decorator option14:26
dtantsurwe could assume that if we have an OOB step, it won't be overridden by an IB step14:27
dtantsurthen we can figure out if the steps or the initial part of them are all OOB14:28
TheJuliayeah, or if we only have inband, don't try to do inband setup14:28
TheJuliaerr, only have out of band14:28
cdearbornTheJulia, dtantsur, possibly optionally tag the cleaning step command sent to ironic with OOB or IB so it knows what to do?14:28
TheJuliaThat may be another option, but that requires operator knowledge of the steps14:30
TheJuliaAnd would any one the possible override....14:30
*** dsneddon has quit IRC14:32
*** zaneb has joined #openstack-ironic14:35
*** ricolin_ has joined #openstack-ironic14:42
*** ricolin has quit IRC14:45
*** igordc has joined #openstack-ironic14:49
rpiosoGood morning, ironic.14:51
*** mgoddard has joined #openstack-ironic14:51
* rpioso was aluminum tubing14:53
iurygregoryrpioso, morning14:57
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Support for testing adoption in the standalone job  https://review.opendev.org/67228614:58
*** dsneddon has joined #openstack-ironic15:01
*** ricolin_ is now known as ricolin15:02
rpiosoiurygregory: :-)15:04
*** Goneri has quit IRC15:05
*** verma-varsha has joined #openstack-ironic15:11
*** rachit7 has quit IRC15:12
*** gyee has joined #openstack-ironic15:13
verma-varshaetingof rpittau Could you please find some time to review https://review.opendev.org/667328/?15:13
patchbotpatch 667328 - sushy-tools - Add Simple Storage resource support - 6 patch sets15:13
rpittauverma-varsha: did you check the issue with py27 job ?15:18
*** dsneddon has quit IRC15:19
verma-varsharpittau: The tests are passing locally on my machine :/15:19
*** iurygregory has quit IRC15:20
*** rachit7 has joined #openstack-ironic15:24
rpittauverma-varsha: I was able to reproduce the issue locally, it happens one time out of two, so it seems there's something wrong with that15:29
*** tssurya has quit IRC15:32
* TheJulia sighs15:38
etingofverma-varsha, I hope to do it tonight15:40
openstackgerritJulia Kreger proposed openstack/ironic-python-agent-builder master: Update tinycore from 8.x to 10.x  https://review.opendev.org/66659115:43
*** rachit7 has quit IRC15:47
*** donnyd has joined #openstack-ironic15:54
arne_wiebalckverma-varsha: I had a quick look at your test case16:05
arne_wiebalckverma-varsha: I think the test fails sometimes as the order changes sometimes16:06
*** lucasagomes has quit IRC16:06
arne_wiebalckverma-varsha: How about rather than assertEqual you use assertItemsEqual to make sure all elements are there?16:07
arne_wiebalckverma-varsha: Or is the order relevant?16:07
*** michchap has joined #openstack-ironic16:09
rpittauarne_wiebalck: that method is for python 2 only though16:10
arne_wiebalckrpittau: ok, but the issue remains the same, no?16:11
rpittauI think adding sorted might help keeping the same method16:11
rpittauif the order is not relevant ofc16:12
arne_wiebalckrpittau: my point is that the comparison as it is now requires correct order (I think) which may vary16:12
*** verma-varsha has quit IRC16:12
arne_wiebalckrpittau: yes, sorting would be another option16:12
*** pcaruana has quit IRC16:13
arne_wiebalckrpittau: or check for each element if it is present16:14
rpittauassertItemsEqual count just the number of items, in this case seems the actual content of the list is important16:14
rpittauarne_wiebalck: yep, that also works16:15
arne_wiebalckrpittau: oh, rly?16:15
arne_wiebalckrpittau: misleading name16:15
arne_wiebalckrpittau: anyway, the test fails since the order in the response changes, while the tests insists on a certain order :)16:16
*** dsneddon has joined #openstack-ironic16:16
*** verma-varsha has joined #openstack-ironic16:20
rpittauarne_wiebalck: yeah, it really depends if the order is important in the end, otherwise also using set could help16:21
*** dsneddon has quit IRC16:22
verma-varshaarne_wiebalck, rpittau: The order is not important but the content is16:22
rpittauverma-varsha: maybe using set instead of list, or just compare using sorted16:23
*** ricolin has quit IRC16:23
dtantsuradoption test passed \o/ http://logs.openstack.org/86/672286/10/check/ironic-standalone/f2e9294/testr_results.html.gz16:23
rpittaudtantsur: nice!16:23
dtantsurcomments are welcome before I un-WIP https://review.opendev.org/#/c/672286/16:24
patchbotpatch 672286 - ironic-tempest-plugin - [WIP] Support for testing adoption in the standalo... - 10 patch sets16:24
arne_wiebalckbye, everyone o/16:26
*** rachit7 has joined #openstack-ironic16:26
rpittaubye all! o/16:26
*** rpittau is now known as rpittau|afk16:26
openstackgerritDmitry Tantsur proposed openstack/ironic master: Use openstacksdk for accessing ironic-inspector  https://review.opendev.org/66985516:28
*** rachit7 has quit IRC16:32
*** S4ren has quit IRC16:35
*** marta_lais has joined #openstack-ironic16:35
*** verma-varsha has quit IRC16:35
*** verma-varsha has joined #openstack-ironic16:36
openstackgerritVarsha Verma proposed openstack/sushy-tools master: Add Simple Storage resource support  https://review.opendev.org/66732816:36
*** dsneddon has joined #openstack-ironic16:40
openstackgerritVarsha Verma proposed openstack/sushy master: Add MappedListField  https://review.opendev.org/67122816:50
*** dtantsur is now known as dtantsur|afk16:52
*** Lucas_Gray has quit IRC16:55
*** rachit7 has joined #openstack-ironic16:55
*** derekh has quit IRC16:56
*** dougsz has quit IRC16:58
*** mmethot_ is now known as mmethot17:02
*** verma-varsha has quit IRC17:05
*** igordc has quit IRC17:16
*** igordc has joined #openstack-ironic17:17
*** igordc has quit IRC17:27
*** priteau has quit IRC17:30
*** e0ne has quit IRC17:37
*** pcaruana has joined #openstack-ironic17:41
*** henriqueof has joined #openstack-ironic17:47
*** verma-varsha has joined #openstack-ironic17:54
*** igordc has joined #openstack-ironic18:01
openstackgerritJulia Kreger proposed openstack/ironic-python-agent-builder master: Update tinycore from 8.x to 10.x  https://review.opendev.org/66659118:07
*** rachit7 has quit IRC18:08
openstackgerritIlya Etingof proposed openstack/ironic master: [WIP] Add iDRAC boot interface  https://review.opendev.org/67249818:14
openstackgerritIlya Etingof proposed openstack/ironic master: Allow Legacy boot over virtual media  https://review.opendev.org/67225318:19
openstackgerritIlya Etingof proposed openstack/ironic master: [WIP] Add iDRAC boot interface  https://review.opendev.org/67249818:19
*** rachit7 has joined #openstack-ironic18:21
*** henriqueof has quit IRC18:40
*** rachit7 has quit IRC18:46
*** tesseract has quit IRC18:49
*** hwoarang has quit IRC18:53
*** hwoarang has joined #openstack-ironic18:54
*** marta_lais has quit IRC19:03
*** marta_lais has joined #openstack-ironic19:04
*** verma-varsha has quit IRC19:13
*** henriqueof has joined #openstack-ironic19:13
*** verma-varsha has joined #openstack-ironic19:17
*** e0ne has joined #openstack-ironic19:25
*** hwoarang has quit IRC19:30
*** hwoarang has joined #openstack-ironic19:31
*** jtomasek has quit IRC19:39
*** priteau has joined #openstack-ironic19:40
*** e0ne has quit IRC19:46
*** verma-varsha has quit IRC19:50
*** e0ne has joined #openstack-ironic19:57
*** henriqueof has quit IRC20:01
*** priteau has quit IRC20:02
*** e0ne has quit IRC20:19
*** hwoarang has quit IRC20:25
*** hwoarang has joined #openstack-ironic20:26
*** jcoufal has quit IRC20:39
*** e0ne has joined #openstack-ironic21:04
*** pcaruana has quit IRC21:05
*** henriqueof has joined #openstack-ironic21:07
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316921:13
*** andy_ has quit IRC21:22
*** zaneb has quit IRC21:25
*** henriqueof has quit IRC21:27
*** e0ne has quit IRC21:30
*** e0ne has joined #openstack-ironic21:39
*** e0ne has quit IRC21:46
openstackgerritJulia Kreger proposed openstack/ironic-tempest-plugin master: Make single tenant ip version configurable  https://review.opendev.org/67260122:04
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316922:06
*** rcernin has joined #openstack-ironic23:30
*** cdearborn has quit IRC23:30
*** rh-jelabarre has quit IRC23:43

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