Wednesday, 2019-06-05

*** rh-jelabarre has quit IRC00:02
*** hamzy has joined #openstack-ironic00:02
*** rh-jelabarre has joined #openstack-ironic00:03
*** itlinux_ has quit IRC00:04
*** rh-jelabarre has quit IRC00:05
*** goldyfruit has quit IRC00:29
*** goldyfruit has joined #openstack-ironic00:29
*** rloo has quit IRC00:45
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Incorporate bandit support in CI  https://review.opendev.org/66268600:50
*** hwoarang has quit IRC00:51
openstackgerritKaifeng Wang proposed openstack/ironic master: Incorporate bandit support in CI  https://review.opendev.org/66268900:54
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent master: Incorporate bandit support in CI  https://review.opendev.org/66269900:55
*** hwoarang has joined #openstack-ironic00:57
*** goldyfruit has quit IRC01:15
*** goldyfruit has joined #openstack-ironic01:16
*** yedongcan has joined #openstack-ironic01:48
*** hwoarang has quit IRC01:58
*** hwoarang has joined #openstack-ironic02:00
*** itlinux has joined #openstack-ironic02:27
*** goldyfruit has quit IRC03:11
*** hwoarang has quit IRC03:34
*** hwoarang has joined #openstack-ironic03:37
openstackgerritDongcan Ye proposed openstack/ironic-inspector master: Tests: Using bmc_addresss for node_driver_info  https://review.opendev.org/66306203:48
*** ricolin has joined #openstack-ironic03:58
*** gyee has quit IRC04:02
*** threestrands has joined #openstack-ironic04:15
*** hwoarang has quit IRC04:21
*** bnemec has quit IRC04:23
*** gmann has quit IRC04:23
*** hwoarang has joined #openstack-ironic04:24
*** bnemec has joined #openstack-ironic04:25
*** gmann has joined #openstack-ironic04:27
*** pcaruana has joined #openstack-ironic04:30
*** itlinux has quit IRC05:22
*** bdodd has quit IRC05:29
*** bdodd has joined #openstack-ironic05:40
*** jtomasek has joined #openstack-ironic05:49
*** bdodd has quit IRC05:51
*** hwoarang has quit IRC05:54
*** hwoarang has joined #openstack-ironic05:58
openstackgerritMerged openstack/ironic master: Remove XML support in parsable_error middleware  https://review.opendev.org/66278006:02
*** bdodd has joined #openstack-ironic06:24
*** dtantsur|afk is now known as dtantsur\06:29
*** dtantsur\ is now known as dtantsur06:29
dtantsurmorning ironic06:29
*** e0ne has joined #openstack-ironic06:31
arne_wiebalckgood morning, ironic!06:35
openstackgerritDongcan Ye proposed openstack/ironic-inspector master: Tests: Using bmc_addresss for node_driver_info  https://review.opendev.org/66306206:41
arne_wiebalckdtantsur: Thanks for your patient reviews on the RAID patches :)06:44
dtantsurthank you for working on it :) it's a very highly demanded feature.06:44
dtantsurarne_wiebalck: I'd really appreciate if you could add some docs once the ironic side merges06:44
arne_wiebalckdtantsur: For the create/delete one: you're ok with a follow-up patch (as the patch has a +W)?06:44
arne_wiebalckdtantsur: re docs: I will06:45
dtantsurarne_wiebalck: yep (that's implied by the approval :)06:45
arne_wiebalckdtantsur: ok, just wanted to be sure06:45
arne_wiebalckdtantsur: I'll update the deploy support once more, though?06:46
arne_wiebalckdtantsur: 63939006:46
dtantsursure06:47
arne_wiebalckI think it'd be good to check if the partition exists as you suggest.06:47
arne_wiebalckIt's one of the short cuts I took.06:47
arne_wiebalckThis area needs to be revisited, though.06:47
arne_wiebalckOk, thanks.06:47
*** rcernin has quit IRC06:59
*** hwoarang has quit IRC07:06
*** hwoarang has joined #openstack-ironic07:07
*** tssurya has joined #openstack-ironic07:13
*** rpittau|afk is now known as rpittau07:14
rpittaugood morning ironic! o/07:14
*** e0ne has quit IRC07:15
*** yolanda has joined #openstack-ironic07:15
openstackgerritPradip Kadam proposed openstack/ironic master: Adding reset_idrac and known_good_state cleaning steps  https://review.opendev.org/65827107:20
*** tesseract has joined #openstack-ironic07:25
openstackgerritMerged openstack/ironic-python-agent master: Software RAID: Create/delete configurations  https://review.opendev.org/63468107:34
dtantsur\o/07:42
*** whoami-rajat has joined #openstack-ironic07:45
*** iurygregory has joined #openstack-ironic08:00
arne_wiebalck\o/08:04
iurygregorymorning all08:06
rpittauhi iurygregory :)08:06
arne_wiebalckrpittau: iurygregory: o/08:07
rpittauhey arne_wiebalck :)08:07
iurygregory\o08:07
*** dougsz has joined #openstack-ironic08:09
*** Lucas_Gray has joined #openstack-ironic08:13
*** lucasagomes has joined #openstack-ironic08:15
*** derekh has joined #openstack-ironic08:18
*** threestrands has quit IRC08:39
*** mkowalski has joined #openstack-ironic08:44
*** openstackgerrit has quit IRC08:47
*** openstackgerrit has joined #openstack-ironic08:55
openstackgerritVarsha Verma proposed openstack/sushy-tools master: Add contributor docs on running sushy-tools locally  https://review.opendev.org/66297908:55
*** rcernin has joined #openstack-ironic08:57
*** mkowalski has quit IRC09:16
*** mkowalski has joined #openstack-ironic09:16
*** mkowalski has quit IRC09:18
*** mkowalski has joined #openstack-ironic09:18
*** Emine has joined #openstack-ironic09:21
*** priteau has joined #openstack-ironic09:22
*** e0ne has joined #openstack-ironic09:30
*** mkowalski has quit IRC09:35
*** mkowalski has joined #openstack-ironic09:36
*** dsneddon has quit IRC10:04
openstackgerritPradip Kadam proposed openstack/ironic master: Adding reset_idrac and known_good_state cleaning steps  https://review.opendev.org/65827110:04
openstackgerritPradip Kadam proposed openstack/ironic master: Adding reset_idrac and known_good_state cleaning steps  https://review.opendev.org/65827110:19
*** dsneddon has joined #openstack-ironic10:29
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management harness to ManagementInterface  https://review.opendev.org/64967510:30
jrollmorning ironic :)10:44
*** ricolin has quit IRC10:47
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: [WIP] Use allocation name for hostname instead of a custom field  https://review.opendev.org/66269110:56
dtantsurmorning jroll10:57
jroll\o10:57
*** dsneddon has quit IRC11:00
*** goldyfruit has joined #openstack-ironic11:11
*** kplant has joined #openstack-ironic11:16
*** Lucas_Gray has quit IRC11:24
*** dsneddon has joined #openstack-ironic11:34
*** dsneddon has quit IRC11:39
*** yolanda has quit IRC11:48
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type  https://review.opendev.org/65274011:48
*** gmann has quit IRC12:03
*** ricolin has joined #openstack-ironic12:03
*** hwoarang has quit IRC12:09
*** hwoarang has joined #openstack-ironic12:11
*** dsneddon has joined #openstack-ironic12:14
*** dsneddon has quit IRC12:19
*** hjensas has quit IRC12:20
*** tzumainn has joined #openstack-ironic12:24
*** goldyfruit has quit IRC12:24
*** jaypipes has joined #openstack-ironic12:28
*** rcernin has quit IRC12:37
*** priteau has quit IRC12:41
*** dsneddon has joined #openstack-ironic12:49
*** rloo has joined #openstack-ironic12:50
*** hjensas has joined #openstack-ironic12:52
*** priteau has joined #openstack-ironic12:54
*** dsneddon has quit IRC12:54
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: [WIP] Use allocation name for hostname instead of a custom field  https://review.opendev.org/66269112:55
TheJuliaoh wow, my grub boot loader job worked12:55
*** rh-jelabarre has joined #openstack-ironic12:55
dtantsurit did :)12:56
dtantsurmorning TheJulia12:56
TheJuliaamazing how broken that code has been in our devstack lib for all this time....12:56
dtantsuroh12:56
TheJuliaand now we'll be able to clean up the elilo code13:00
dtantsur\o/13:01
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type  https://review.opendev.org/65274013:01
etingofscary things peek at us from the depths of devstack13:01
*** Goneri has joined #openstack-ironic13:14
*** rloo has quit IRC13:19
*** rloo has joined #openstack-ironic13:19
*** goldyfruit has joined #openstack-ironic13:23
openstackgerritJulia Kreger proposed openstack/ironic master: Add a pxe/uefi/grub2 CI job  https://review.opendev.org/61399813:25
openstackgerritJulia Kreger proposed openstack/ironic master: Add a pxe/uefi/grub2 CI job  https://review.opendev.org/61399813:26
*** dsneddon has joined #openstack-ironic13:28
openstackgerritRiccardo Pittau proposed openstack/sushy master: Use collections.abc instead of collections when available  https://review.opendev.org/66332913:28
*** jcoufal has joined #openstack-ironic13:31
*** dsneddon has quit IRC13:32
*** iurygregory has quit IRC13:32
TheJuliaohhh and reviews on my proposed devstack fixes that should allow us to drop neutron-legacy13:33
TheJuliatoday feels like a good day13:33
*** yedongcan has quit IRC13:35
*** iurygregory has joined #openstack-ironic13:37
*** rcernin has joined #openstack-ironic13:41
arne_wiebalckdtantsur: seems like removing 'shell=True' from https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L1454 broke the RAID creation13:47
dtantsurarne_wiebalck: maybe because it needs an array not a string?13:47
TheJuliaarne_wiebalck: because your building a formatted string?13:47
dtantsurI did not remember when reviewing13:47
TheJuliadtantsur: jinx13:47
* arne_wiebalck can't follow13:49
TheJuliaarne_wiebalck: with shell, it will take a string and pass it to a shell which will then break the command into an array13:50
TheJuliathat array will be [command_name, arg1, arg2, arg3, etc]13:50
TheJuliaremoving shell, means the exec is literally trying to execute the command_name portion as the entire thing, which doesn't work13:50
*** yedongcan has joined #openstack-ironic13:50
arne_wiebalckai!13:51
dtantsurarne_wiebalck: yeah, you need something like https://github.com/openstack/ironic-python-agent/blob/73208b48b73f3f4b8f0b77a65699a37cd21bff0e/ironic_python_agent/extensions/image.py#L10913:51
dtantsurshould be an easy follow-up13:51
arne_wiebalckTheJulia: dtantsur: thx13:52
arne_wiebalckI guess this applies to more than just this one ... I remember having issues with this at some point ...13:53
dtantsuryeah. without shell=True it's just native exec, with shell it's doing `bash -c "<your string>"`13:54
arne_wiebalck... and probably found it is working with shell ... which I then used everywhere13:54
arne_wiebalckyou prefer the array, or shell=True, or no matter which?13:54
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316913:56
TheJulia^^^ moved the tcpdump because I'm going to loose my mind to make this work13:56
hjensasTheJulia: I've been trying to reproduce using the conf from that job locally in devstack today. So far it looks like you are getting further in the CI job, locally I'm not getting IP's at all in iPXE.13:59
TheJulia\o/13:59
hjensasTheJulia: not much help, but I'll keep poking at it and shall shout if I find something.14:00
TheJuliahjensas: did you get the devstack patch?14:00
hjensasTheJulia: not the new one you just pushed, but the I used the revision from last night.14:00
TheJuliashoot, I need to fix that14:00
* TheJulia pulls down the updated v6 job to fix var names14:01
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316914:04
*** dsneddon has joined #openstack-ironic14:07
*** dsneddon has quit IRC14:12
*** Lucas_Gray has joined #openstack-ironic14:18
*** bnemec has quit IRC14:23
*** bnemec has joined #openstack-ironic14:25
*** iurygregory_ has joined #openstack-ironic14:37
*** iurygregory has quit IRC14:38
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.opendev.org/64550714:40
*** dsneddon has joined #openstack-ironic14:40
*** dsneddon has quit IRC14:44
*** goldyfruit has quit IRC14:47
*** iurygregory has joined #openstack-ironic14:49
*** tzumainn has quit IRC14:49
*** iurygregory_ has quit IRC14:50
*** goldyfruit has joined #openstack-ironic14:53
*** itlinux has joined #openstack-ironic14:56
*** itlinux has quit IRC14:59
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type  https://review.opendev.org/65274015:03
*** lennyb has quit IRC15:07
*** lennyb has joined #openstack-ironic15:07
*** goldyfruit has quit IRC15:18
*** dsneddon has joined #openstack-ironic15:20
*** derekh has quit IRC15:23
*** derekh has joined #openstack-ironic15:23
*** Goneri has quit IRC15:24
*** dsneddon has quit IRC15:25
*** derekh has quit IRC15:25
*** rcernin has quit IRC15:26
*** yolanda has joined #openstack-ironic15:26
*** derekh has joined #openstack-ironic15:26
*** gyee has joined #openstack-ironic15:27
*** derekh has quit IRC15:29
*** derekh has joined #openstack-ironic15:29
*** derekh has quit IRC15:31
*** derekh has joined #openstack-ironic15:31
*** derekh has quit IRC15:34
*** derekh has joined #openstack-ironic15:34
*** tssurya has quit IRC15:36
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Fix shell commands  https://review.opendev.org/66336115:39
arne_wiebalckTheJulia: dtantsur : This ^^ should fix the shell issue that was merged earlier today.15:41
TheJuliathanks15:41
dtantsurthx!15:41
*** goldyfruit has joined #openstack-ironic15:49
*** Lucas_Gray has quit IRC15:55
*** itlinux has joined #openstack-ironic15:56
*** dsneddon has joined #openstack-ironic15:59
*** iurygregory_ has joined #openstack-ironic16:00
*** dtantsur is now known as dtantsur|afk16:01
dtantsur|afko/16:01
*** iurygregory has quit IRC16:02
*** dsneddon has quit IRC16:04
*** goldyfruit has quit IRC16:04
*** iurygregory_ is now known as iurygregory16:06
*** goldyfruit has joined #openstack-ironic16:07
*** lucasagomes has quit IRC16:07
rpittaugood night! o/16:08
*** rpittau is now known as rpittau|afk16:08
*** yedongcan has left #openstack-ironic16:10
*** Goneri has joined #openstack-ironic16:10
*** ociuhandu_ has joined #openstack-ironic16:19
*** Emine has quit IRC16:30
arne_wiebalckbye, everyone o/16:32
*** dsneddon has joined #openstack-ironic16:33
*** bnemec has quit IRC16:44
*** e0ne has quit IRC16:53
*** itlinux has quit IRC16:56
*** itlinux has joined #openstack-ironic16:58
*** dougsz has quit IRC16:59
*** derekh has quit IRC17:00
*** itlinux has quit IRC17:04
*** itlinux has joined #openstack-ironic17:09
*** ociuhandu_ has quit IRC17:36
*** goldyfruit has quit IRC17:44
*** goldyfruit has joined #openstack-ironic17:45
*** jcoufal has quit IRC17:55
*** tesseract has quit IRC17:57
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316918:02
*** goldyfruit has quit IRC18:08
*** goldyfruit has joined #openstack-ironic18:08
*** e0ne has joined #openstack-ironic18:20
*** itlinux has quit IRC18:32
*** ricolin has quit IRC18:41
*** jcoufal has joined #openstack-ironic18:42
*** goldyfruit has quit IRC18:57
*** jcoufal has quit IRC18:57
openstackgerritVlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support  https://review.opendev.org/64256518:57
*** goldyfruit has joined #openstack-ironic19:00
*** itlinux has joined #openstack-ironic19:03
*** priteau has quit IRC19:06
*** priteau has joined #openstack-ironic19:07
openstackgerritVlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support  https://review.opendev.org/64256519:19
TheJuliahjensas: on the actual tap http://logs.openstack.org/69/613169/60/check/ironic-tempest-dsvm-ipxe-ipv6/f657143/controller/logs/tcpdump.pcap.gz <-- gets the dhcpv6 response.... and then nothing. I'm wondering if maybe the second interface is being attempted by iPXE for some crazy absurd reason....19:23
TheJuliahttp://logs.openstack.org/69/613169/60/check/ironic-tempest-dsvm-ipxe-ipv6/f657143/job-output.txt.gz#_2019-06-05_18_52_38_26003619:23
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316919:24
TheJuliahjensas: ^^^ changing the tcpdump to the second interface19:24
openstackgerritMerged openstack/ironic-python-agent master: Software RAID: Add IPA deploy support  https://review.opendev.org/63939019:40
openstackgerritVlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support  https://review.opendev.org/64256519:41
hjensasTheJulia: yes, it's strange that we see no more traffic after that.19:41
hjensasTheJulia: On the console there are two addresses on net0 ? http://logs.openstack.org/69/613169/60/check/ironic-tempest-dsvm-ipxe-ipv6/f657143/controller/logs/ironic-bm-logs/node-0_no_ansi_2019-06-05-18:51:44_log.txt.gz19:41
openstackgerritMerged openstack/ironic-python-agent master: Incorporate bandit support in CI  https://review.opendev.org/66269919:43
*** Emine has joined #openstack-ironic19:45
*** kplant has quit IRC19:48
TheJuliahjensas: yeah, I spotted that. fe80 would be the link layer address19:50
rpiosoTheJulia: In case you haven't seen it, yet,  https://review.opendev.org/#/c/660534/ merged earlier today.19:50
patchbotpatch 660534 - x/driverlog - Add openstack/ironic ipmi and redfish drivers (MERGED) - 4 patch sets19:51
rpiosoTheJulia: Dell EMC wants to propose a change similar to that one which adds another pair of entries for the ipmi and redfish h/w types/drivers. The new pair would differ from the one that just merged by having the vendor set to Dell EMC. The CI information would refer to the Dell EMC third party CI testing.19:56
TheJuliarpioso: that seems reasonable19:57
rpiosoTheJulia: Our intent is to inform the marketplace that those h/w types are tested and supported with Dell EMC servers.19:58
TheJuliarpioso: I gathered that :)19:58
rpiosoTheJulia: Thank you! :-)19:58
openstackgerritVlad Sorokin proposed openstack/networking-generic-switch master: Add Mellanox MLNX-OS Switch support  https://review.opendev.org/64256520:06
*** goldyfruit has quit IRC20:07
*** itlinux has quit IRC20:17
*** itlinux has joined #openstack-ironic20:19
*** Goneri has quit IRC20:30
*** Goneri has joined #openstack-ironic20:44
*** goldyfruit has joined #openstack-ironic20:50
*** jtomasek has quit IRC20:57
*** pcaruana has quit IRC20:59
*** Emine has quit IRC21:00
TheJuliahjensas: http://lists.ipxe.org/pipermail/ipxe-devel/2015-November/004485.html sounds familiar....21:05
TheJuliahjensas: http://lists.ipxe.org/pipermail/ipxe-devel/2016-July/005109.html21:07
openstackgerritPradip Kadam proposed openstack/ironic master: Adding reset_idrac and known_good_state cleaning steps  https://review.opendev.org/65827121:08
*** priteau has quit IRC21:10
openstackgerritPradip Kadam proposed openstack/ironic master: Adding reset_idrac and known_good_state cleaning steps  https://review.opendev.org/65827121:11
*** rh-jelabarre has quit IRC21:12
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 job  https://review.opendev.org/61316921:18
openstackgerritJulia Kreger proposed openstack/ironic master: Add a pxe/uefi/grub2 CI job  https://review.opendev.org/61399821:21
*** rcernin has joined #openstack-ironic21:26
*** gmann has joined #openstack-ironic21:45
*** itlinux has quit IRC21:50
*** itlinux has joined #openstack-ironic21:56
*** whoami-rajat has quit IRC22:00
*** henriqueof has joined #openstack-ironic22:02
*** rcernin has quit IRC22:04
*** e0ne has quit IRC22:13
*** itlinux has quit IRC22:15
*** henriqueof has quit IRC22:15
*** goldyfruit has quit IRC22:40
TheJuliahjensas: changed to stateful http://logs.openstack.org/69/613169/62/check/ironic-tempest-dsvm-ipxe-ipv6/c3d789d/controller/logs/ and we see tftp requests transmitted \o/22:52
TheJuliadtantsur|afk: the inspector stable backports, do they need to go any further back?23:14
*** itlinux has joined #openstack-ironic23:16
*** goldyfruit has joined #openstack-ironic23:16
*** goldyfruit has quit IRC23:41
*** hwoarang has quit IRC23:41
*** hwoarang has joined #openstack-ironic23:42
rpiosoTheJulia: I submitted the change we discussed earlier and invited you to review it -- https://review.opendev.org/#/c/663460/23:42
patchbotpatch 663460 - x/driverlog - Add openstack/ironic ipmi and redfish drivers redux - 1 patch set23:42
*** Goneri has quit IRC23:43
*** itlinux has quit IRC23:46

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