Thursday, 2018-06-28

*** liuzz has quit IRC00:00
*** markguz has quit IRC00:03
*** alex_xu has joined #openstack-ironic00:04
*** rpioso is now known as rpioso|afk00:08
openstackgerritRuby Loo proposed openstack/ironic master: include all versions of Node in release_mappings  https://review.openstack.org/57861000:11
openstackgerritSergii Golovatiuk proposed openstack/ironic master: Reboot server after 10 tries  https://review.openstack.org/57851200:11
*** cjloader has joined #openstack-ironic00:20
*** gyee has quit IRC00:39
*** kzaitsev_pi has joined #openstack-ironic00:42
*** kzaitsev1pi has quit IRC00:45
*** kzaitsev1pi has joined #openstack-ironic00:54
*** rh-jelabarre has joined #openstack-ironic00:55
*** kzaitsev_pi has quit IRC00:55
*** hshiina|afk is now known as hshiina01:00
hshiinadtantsur|afk: thank you for fixing ironic-lib ci01:01
*** phuongnh has joined #openstack-ironic01:03
*** spotz has quit IRC01:04
*** yolanda_ has joined #openstack-ironic01:05
*** yolanda has quit IRC01:08
*** yolanda has joined #openstack-ironic01:09
*** yolanda_ has quit IRC01:11
*** yangyapeng has joined #openstack-ironic01:11
*** yolanda_ has joined #openstack-ironic01:13
*** yolanda has quit IRC01:16
*** cjloader has quit IRC01:18
*** tiendc has joined #openstack-ironic01:18
*** yolanda__ has joined #openstack-ironic01:19
*** spotz has joined #openstack-ironic01:20
*** yolanda_ has quit IRC01:22
TheJuliahshiina: mind taking a look at https://review.openstack.org/#/c/578566/ ?01:30
patchbotpatch 578566 - ironic-python-agent - Enable Auto VM engine for CoreOS CI jobs01:30
*** zshi has joined #openstack-ironic01:32
openstackgerritHironori Shiina proposed openstack/ironic master: DevStack: Tiny changes following iRMC classic driver removal  https://review.openstack.org/57862801:36
*** rloo has quit IRC01:36
hshiinaTheJulia: sure, i'll look at it01:36
*** liuzz has joined #openstack-ironic01:37
*** phuongnh has quit IRC01:37
*** phuongnh has joined #openstack-ironic01:38
TheJuliahshiina: given it addresses gate performance/test failures, I'm all for you +2+a'ing it if you want01:38
*** liuzz_ has joined #openstack-ironic01:40
*** liuzz has quit IRC01:40
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update boot-from-volume feature docs  https://review.openstack.org/57863301:46
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update boot-from-volume feature docs  https://review.openstack.org/57863301:50
*** AlexeyAbashkin has joined #openstack-ironic02:10
*** links has joined #openstack-ironic02:30
*** links has quit IRC02:30
*** kzaitsev1pi has quit IRC02:32
*** phuongnh has quit IRC02:32
*** kzaitsev_pi has joined #openstack-ironic02:32
*** phuongnh has joined #openstack-ironic02:33
*** AlexeyAbashkin has quit IRC02:39
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update boot-from-volume feature docs  https://review.openstack.org/57863302:48
openstackgerritMerged openstack/ironic-python-agent master: Enable Auto VM engine for CoreOS CI jobs  https://review.openstack.org/57856603:10
*** zhangfei has joined #openstack-ironic03:10
*** zhangfei has quit IRC03:26
*** zhangfei has joined #openstack-ironic03:27
*** spartacloud has joined #openstack-ironic03:28
*** rh-jelabarre has quit IRC03:52
*** hshiina2 has joined #openstack-ironic04:03
openstackgerritDao Cong Tien proposed openstack/ironic master: Follow-up to update doc for irmc driver  https://review.openstack.org/57864404:03
*** hshiina has quit IRC04:07
*** mvpnitesh has joined #openstack-ironic04:26
*** nizam037 has joined #openstack-ironic04:26
openstackgerritwu.chunyang proposed openstack/python-ironic-inspector-client master: Add release note link in README  https://review.openstack.org/57864604:26
*** AlexeyAbashkin has joined #openstack-ironic04:32
*** jaganathan has joined #openstack-ironic04:39
openstackgerritRuby Loo proposed openstack/ironic master: [WIP] Deploy steps Unabridged version  https://review.openstack.org/57864904:46
*** hshiina2 has quit IRC04:47
*** hshiina has joined #openstack-ironic04:49
*** hshiina has quit IRC04:53
*** zshi has quit IRC04:55
*** nizam037 has quit IRC05:01
*** hshiina has joined #openstack-ironic05:03
*** spartacloud has quit IRC05:21
*** yangyapeng has quit IRC05:25
*** AlexeyAbashkin has quit IRC05:25
*** yangyapeng has joined #openstack-ironic05:25
*** yangyapeng has quit IRC05:29
*** openstackgerrit has quit IRC05:34
*** bdodd has quit IRC05:46
*** openstackstatus has quit IRC05:51
*** openstackstatus has joined #openstack-ironic05:52
*** ChanServ sets mode: +v openstackstatus05:52
*** yangyapeng has joined #openstack-ironic05:59
*** yolanda__ is now known as yolanda05:59
*** rbartal has joined #openstack-ironic06:00
*** zhangfei has quit IRC06:01
*** rbartal_ has joined #openstack-ironic06:02
*** fanzhang has quit IRC06:04
*** rbartal has quit IRC06:04
*** zshi has joined #openstack-ironic06:11
*** zhangfei has joined #openstack-ironic06:13
*** mjura has joined #openstack-ironic06:16
*** liuzz_ has quit IRC06:19
*** openstackgerrit has joined #openstack-ironic06:19
openstackgerritDao Cong Tien proposed openstack/ironic master: Lack import of i18n _() function  https://review.openstack.org/57869606:19
*** liuzz has joined #openstack-ironic06:19
*** AlexeyAbashkin has joined #openstack-ironic06:29
*** bdodd has joined #openstack-ironic06:30
*** fanzhang has joined #openstack-ironic06:35
*** fanzhang has left #openstack-ironic06:36
*** trungnv has quit IRC06:39
*** retr0h has quit IRC06:39
*** AlexeyAbashkin has quit IRC06:41
olivierbourdon38good morning ironicers06:41
olivierbourdon38TheJulia sure with great pleasure06:41
TheJuliagood morning06:42
* TheJulia blames insomnia for still being up06:42
olivierbourdon38TheJulia I understand, very sorry for you06:45
TheJulia:)06:45
* TheJulia is not sure there is anything to be sorry for06:45
TheJuliaAnyway, I'm going to actually try and sleep and let my brain ponder the ipa gate and its dislike of me06:46
olivierbourdon38have a nice rest then06:46
TheJuliaThanks, have a wonderful day06:47
*** retr0h has joined #openstack-ironic06:48
*** trungnv has joined #openstack-ironic06:53
*** serlex has joined #openstack-ironic07:01
*** jcoufal has joined #openstack-ironic07:01
*** pcaruana has joined #openstack-ironic07:10
*** tesseract has joined #openstack-ironic07:11
*** cjloader has joined #openstack-ironic07:25
*** spartacloud has joined #openstack-ironic07:25
*** cjloader has quit IRC07:29
*** spartacloud has quit IRC07:29
*** jiapei has joined #openstack-ironic07:34
*** yolanda_ has joined #openstack-ironic07:35
*** liuzz_ has joined #openstack-ironic07:37
*** yolanda is now known as Guest6344807:37
*** yolanda_ is now known as yolanda07:37
*** mgoddard has joined #openstack-ironic07:37
*** Guest63448 has quit IRC07:38
*** liuzz has quit IRC07:39
*** racedo has joined #openstack-ironic07:47
*** ajya has joined #openstack-ironic08:05
*** threestrands has joined #openstack-ironic08:06
*** d0ugal has joined #openstack-ironic08:08
*** threestrands has quit IRC08:13
*** pmannidi has quit IRC08:13
*** racedo has quit IRC08:18
*** athomas has joined #openstack-ironic08:19
*** dansmith has quit IRC08:20
*** mgoddard has quit IRC08:20
*** dansmith has joined #openstack-ironic08:21
*** dansmith is now known as Guest8882308:21
*** racedo has joined #openstack-ironic08:24
*** mvpnitesh has quit IRC08:26
*** mvpnitesh has joined #openstack-ironic08:26
*** jcoufal has quit IRC08:31
openstackgerritNguyen Van Trung proposed openstack/ironic master: Improve pep8 checking along with hacking  https://review.openstack.org/57869608:32
*** cjloader has joined #openstack-ironic08:37
*** derekh has joined #openstack-ironic08:40
*** cjloader has quit IRC08:41
*** _milan_ has joined #openstack-ironic08:45
*** MrBerns has joined #openstack-ironic08:49
MrBernsMorning all08:50
trungnvmoring MrBerns08:50
*** alexpilotti has quit IRC08:50
*** ajya has quit IRC08:59
*** ajya has joined #openstack-ironic08:59
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Remove support for creating and loading classic drivers  https://review.openstack.org/57845709:06
*** alexpilotti has joined #openstack-ironic09:08
trungnvplease review and give me comments in following patches: https://review.openstack.org/#/c/512979/09:09
patchbotpatch 512979 - ironic - Support RAID configuration for BM via iRMC driver09:09
*** racedo_ has joined #openstack-ironic09:12
*** hjensas|afk is now known as hjensas09:13
*** athomas has quit IRC09:14
*** athomas has joined #openstack-ironic09:14
*** racedo has quit IRC09:15
*** d0ugal has quit IRC09:16
openstackgerritIlya Etingof proposed openstack/ironic master: Add BayTech MRP27 snmp driver type  https://review.openstack.org/57806809:16
*** d0ugal has joined #openstack-ironic09:16
*** kiseok7 has quit IRC09:18
*** _milan_ is now known as miln09:18
openstackgerritSam Betts proposed openstack/ironic master: [DNM] Testing Cisco Third Party CI  https://review.openstack.org/51431209:28
openstackgerritwu.chunyang proposed openstack/python-ironic-inspector-client master: Add release note link in README  https://review.openstack.org/57864609:42
*** quiquell|rover has joined #openstack-ironic09:46
quiquell|roverHello09:47
quiquell|roverWe see this riverLoadError: Driver, hardware type or interface pxe_ipmitool could not be loaded. Reason: [Errno 13] Permission denied: '/var/lib/ironic/httpboot/boot.ipxe'.09:47
quiquell|roverIt's a red herring ?09:47
quiquell|roverhttps://logs.rdoproject.org/openstack-periodic/periodic-tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset002-master-upload/db23aa3/undercloud/var/log/extra/errors.txt.gz09:47
quiquell|roverintrospections times out09:47
*** dtantsur|afk is now known as dtantsur09:49
dtantsurmorning ironic09:50
quiquell|roverdtantsur: gm sir09:50
trungnvmorning dtantsur09:51
dtantsurquiquell|rover: ironic must be able to write to /var/lib/ironic/httpboot09:51
dtantsurif it's broken - tripleo needs fixing09:51
trungnvplease review for following patches: https://review.openstack.org/#/c/512979/09:52
patchbotpatch 512979 - ironic - Support RAID configuration for BM via iRMC driver09:52
quiquell|roverdtantsur: I have found that we copy /httpboot to /var/lib/ironic/09:53
quiquell|roverdtantsur: Maybe changing permissions there should work ?09:53
quiquell|roveris /httpboot read onnly ?09:53
dtantsurquiquell|rover: should not be09:54
dtantsurI guess something was lost on migration to containers09:54
*** rcernin has quit IRC09:56
*** trungnv has quit IRC10:00
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Implement iRMC BIOS configuration  https://review.openstack.org/53459510:01
etingofgood morning everyone o/10:05
quiquell|roverdtantsur: https://logs.rdoproject.org/openstack-periodic/periodic-tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset002-master-upload/db23aa3/undercloud/var/log/containers/nova/nova-compute.log.txt.gz#_2018-06-28_00_33_22_05210:05
quiquell|roverdtantsur: Looks like it's not able to find No compute node record for undercloud.localdomain:26e85889-10a9-4ad3-8392-0ee341b2f657: ComputeHostNotFound_Remote: Compute host undercloud.localdomain could not be found.10:06
dtantsurquiquell|rover: do you have problems with nova? I seem to recall it showing similar warnings for long time10:06
quiquell|roverdtantsur: Nova uses ironic under the hood to discover nodes ?10:07
dtantsurquiquell|rover: well, yes, nova calls ironic to get the list of nodes (and does it periodically)10:07
quiquell|roverdtantsur: ok10:08
openstackgerritMerged openstack/ironic master: include all versions of Node in release_mappings  https://review.openstack.org/57861010:10
openstackgerritMerged openstack/ironic-lib master: Switch to using stestr  https://review.openstack.org/57852210:10
openstackgerritMerged openstack/ironic master: Follow-up to update doc for irmc driver  https://review.openstack.org/57864410:10
*** jcoufal has joined #openstack-ironic10:11
openstackgerritDao Cong Tien proposed openstack/ironic master: Follow-up to update doc for ilo driver  https://review.openstack.org/57875310:14
*** jcoufal has quit IRC10:14
*** jcoufal has joined #openstack-ironic10:15
openstackgerritDao Cong Tien proposed openstack/ironic master: Follow-up to update doc for oneview driver  https://review.openstack.org/57875610:33
*** skazi has quit IRC10:34
openstackgerritMerged openstack/python-ironic-inspector-client master: Support passing manage_boot argument in Python API  https://review.openstack.org/33947910:37
*** yangyapeng has quit IRC10:51
*** yangyapeng has joined #openstack-ironic10:52
*** racedo_ has quit IRC10:59
*** hshiina has quit IRC11:01
*** alexpilotti has quit IRC11:02
*** phuongnh has quit IRC11:04
*** zhangfei has quit IRC11:10
*** yangyapeng has quit IRC11:13
*** yangyapeng has joined #openstack-ironic11:14
*** yangyapeng has quit IRC11:14
*** yangyapeng has joined #openstack-ironic11:15
*** alexpilotti has joined #openstack-ironic11:17
*** alexpilo_ has joined #openstack-ironic11:17
*** MattMan has quit IRC11:20
*** MattMan has joined #openstack-ironic11:20
*** alexpilotti has quit IRC11:21
*** alexpilo_ has quit IRC11:22
openstackgerrityolanda.robla proposed openstack/ironic-tempest-plugin master: Add bios testing to plugin  https://review.openstack.org/56855111:22
*** alexpilotti has joined #openstack-ironic11:24
jrollmorning11:25
*** bfournie has quit IRC11:26
*** bfournie has joined #openstack-ironic11:26
*** bfournie has quit IRC11:26
*** bfournie has joined #openstack-ironic11:27
*** marios has quit IRC11:30
*** marios has joined #openstack-ironic11:30
*** bfournie has quit IRC11:31
*** cjloader has joined #openstack-ironic11:31
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Remove support for creating and loading classic drivers  https://review.openstack.org/57845711:31
dtantsurmorning jroll11:31
*** dtantsur is now known as dtantsur|brb11:32
*** niteshmvp has joined #openstack-ironic11:34
*** cjloader has quit IRC11:36
*** mvpnitesh has quit IRC11:37
*** zul has joined #openstack-ironic11:39
*** dprince has joined #openstack-ironic11:45
*** jcoufal has quit IRC11:48
*** zul has quit IRC11:49
*** serlex has quit IRC11:49
*** jiapei has quit IRC11:50
*** pcaruana has quit IRC11:56
*** zshi has quit IRC11:58
*** racedo has joined #openstack-ironic12:03
*** yangyapeng has quit IRC12:09
*** alexpilotti has quit IRC12:10
*** zhangfei has joined #openstack-ironic12:11
*** alexpilotti has joined #openstack-ironic12:15
*** markguz has joined #openstack-ironic12:17
*** tiendc has quit IRC12:17
*** markguz has quit IRC12:22
*** skazi has joined #openstack-ironic12:23
*** trown|outtypewww is now known as trown12:28
*** yangyapeng has joined #openstack-ironic12:28
*** alexpilotti has quit IRC12:29
*** zhangfei has quit IRC12:30
*** rbartal has joined #openstack-ironic12:31
*** cjloader has joined #openstack-ironic12:32
*** rbartal_ has quit IRC12:33
*** cjloader has quit IRC12:36
*** zul has joined #openstack-ironic12:37
*** pcaruana has joined #openstack-ironic12:44
*** rloo has joined #openstack-ironic12:45
*** rcernin has joined #openstack-ironic12:52
*** dtantsur|brb is now known as dtantsur12:55
*** bfournie has joined #openstack-ironic12:55
*** serlex has joined #openstack-ironic12:56
*** yangyapeng has quit IRC12:56
*** mjturek has joined #openstack-ironic13:00
*** namnh has joined #openstack-ironic13:02
*** namnh has quit IRC13:02
openstackgerritChuck Short proposed openstack/ironic-inspector master: Switch to using stestr  https://review.openstack.org/57852413:07
*** rcernin has quit IRC13:08
*** rh-jelabarre has joined #openstack-ironic13:09
openstackgerritChuck Short proposed openstack/ironic-python-agent master: Switch to using stestr  https://review.openstack.org/57852113:09
*** markguz has joined #openstack-ironic13:12
*** niteshmvp has quit IRC13:15
*** quiquell|rover is now known as quique|rover|lch13:18
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Reduce CoreOS CPU count to 1  https://review.openstack.org/57879413:20
TheJuliagood morning13:20
TheJuliaipa is going to literally drive me bonkers13:20
*** markguz has quit IRC13:24
dtantsurof course it is..13:25
dtantsurmorning TheJulia13:25
TheJuliadtantsur: am I already bonkers?13:25
TheJuliadtantsur: out of curiosity, do you know if there is a devstack means to change the instance build timeout in nova?13:26
dtantsurTheJulia: no idea13:27
TheJuliait doesn't look like it :\13:28
jrollTheJulia: there's a mechanism to specify any config option in local.conf13:29
* jroll looks13:29
jrollhttps://docs.openstack.org/devstack/latest/configuration.html#local-conf13:30
jroll[[post-config|$NOVA_CONF]]\n[DEFAULT]\nuse_syslog = True13:30
jrollas an example13:30
*** rbartal_ has joined #openstack-ironic13:30
TheJuliaokay, looks like the most recent recheck failed for different reasons, digging into why13:33
*** baha has joined #openstack-ironic13:33
*** rbartal has quit IRC13:33
TheJuliaat least coreos ipa boots in 60 seconds instead of 6-8 minutes13:33
TheJuliaugh, timeout while waiting for callback13:35
openstackgerritMichael Turek proposed openstack/ironic-lib master: Add logic to create PReP partition for ppc64* arch  https://review.openstack.org/57210213:43
*** quique|rover|lch is now known as quiquell|rover13:43
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client master: Provide proper error message if interface name is invalid  https://review.openstack.org/57855213:45
*** markguz has joined #openstack-ironic13:52
*** edleafe- has joined #openstack-ironic13:52
*** rbartal has joined #openstack-ironic13:52
*** rbartal_ has quit IRC13:53
*** edleafe has quit IRC13:54
*** edleafe- is now known as edleafe13:54
openstackgerritMerged openstack/ironic-inspector master: fix tox python3 overrides  https://review.openstack.org/57380313:54
*** rbartal_ has joined #openstack-ironic13:55
*** rbartal has quit IRC13:58
*** Guest88823 is now known as dansmith14:01
*** e0ne has joined #openstack-ironic14:02
*** yangyapeng has joined #openstack-ironic14:02
*** rbartal_ has quit IRC14:04
openstackgerritMerged openstack/bifrost master: fix tox python3 overrides  https://review.openstack.org/57370014:04
*** yangyapeng has quit IRC14:06
NobodyCamGood Morning Ironic'ers14:08
*** lbragstad has quit IRC14:10
*** mjura has quit IRC14:12
rloomorning NobodyCam :)14:13
NobodyCam:) Good Morning rloo :)14:13
*** zhangfei has joined #openstack-ironic14:17
*** quiquell|rover is now known as quiquell|off14:21
*** yangyapeng has joined #openstack-ironic14:21
openstackgerritRuby Loo proposed openstack/ironic master: [WIP] Deploy steps Unabridged version  https://review.openstack.org/57864914:24
dtantsurmorning NobodyCam, rloo14:27
rloohi dtantsur!14:27
NobodyCamMorning dtantsur :)14:27
*** bfournie has quit IRC14:27
*** lhinds has quit IRC14:27
*** rnoriega has quit IRC14:27
*** weshay|ruck has quit IRC14:27
*** bfournie has joined #openstack-ironic14:27
*** rnoriega has joined #openstack-ironic14:28
*** weshay has joined #openstack-ironic14:28
*** lhinds has joined #openstack-ironic14:31
*** e0ne has quit IRC14:31
*** yangyapeng has quit IRC14:32
*** cjloader has joined #openstack-ironic14:32
*** weshay has quit IRC14:37
*** weshay has joined #openstack-ironic14:38
*** lbragstad has joined #openstack-ironic14:38
*** markguz has quit IRC14:40
*** e0ne has joined #openstack-ironic14:40
*** markguz has joined #openstack-ironic14:41
*** hkominos has joined #openstack-ironic14:42
hkominosHi all. Is there anyone here who has tried ironic on aarch64 ?14:43
*** markguz has quit IRC14:45
openstackgerritAija Jaunteva proposed openstack/sushy master: Remove etag from Bios  https://review.openstack.org/57882714:46
*** markguz has joined #openstack-ironic14:46
dtantsurhkominos: I know a guy in Red Hat who had some reasonable success, he does not seem to be here though14:47
*** spartacloud has joined #openstack-ironic14:48
hkominosdtantsur: radez ?14:49
dtantsurhkominos: precisely14:49
openstackgerritMatthew Thode proposed openstack/virtualbmc master: pycrypto is not used by virtualbmc  https://review.openstack.org/57883214:50
hkominosdtansur: Hi is on some sort of leave at the moment and I cannot reach him. Do you by any chance now pxe or ipxe is the way to go if you have an uefi system ?14:50
*** racedo_ has joined #openstack-ironic14:52
*** racedo has quit IRC14:53
*** spartacloud has quit IRC14:53
dtantsurhkominos: IIRC iPXE did not quite work for him, so he went with UEFI grub214:55
hkominosdtantsur: That is what I am tryign but not matter what I do the node always tries to find pxelinux.0 instead of grubefi14:57
*** spartacloud has joined #openstack-ironic14:58
*** alex_xu has quit IRC14:58
dtantsurhkominos: have you configured the ironic nodes with boot_mode: uefi?14:58
hkominosyes15:00
*** alex_xu has joined #openstack-ironic15:00
hkominosdtantsur: let me share a paste15:01
hkominosu might see something that I dont15:01
* dtantsur brb sorry15:01
*** v12aml has quit IRC15:02
*** v12aml has joined #openstack-ironic15:03
*** spartacloud has quit IRC15:03
yolandahi TheJulia , i tried to address the timeout ci problems. I hit the 600ms limit lots of time in my standalone tests, and i proposed https://review.openstack.org/#/c/578090/15:04
patchbotpatch 578090 - ironic - WIP: Increase timeouts for standalone tests15:04
yolandabut maybe we can just increase timeouts not generally, but just for the tests that we know that take longer?15:04
TheJuliayolanda: outside of standalone I've seen us hitting the timeout with nova, we don't explicitely pass it. Those timeouts should be good for standalone testing15:05
TheJuliawell, pass a timeout value for nova, so we're relying upon the defaults15:06
TheJuliathey are moderately reasonable, problem is we're using qemu by default15:06
*** rpioso|afk is now known as rpioso15:08
rpiosoGood morning15:08
*** racedo_ has quit IRC15:09
jroll\o rpioso15:10
yolandaok i just have been testing standalone so much. Let me run the other testing and i can modify the patch15:10
*** hamzy has quit IRC15:11
*** jaganathan has quit IRC15:11
rpiosojroll: :)15:12
*** jaganathan has joined #openstack-ironic15:12
*** e0ne has quit IRC15:15
hkominosdtantsur: https://paste.fedoraproject.org/paste/glXpYA9rWcV9qf0KEWZDpw this looks like cpu_aarch is set properly15:15
*** gyee has joined #openstack-ironic15:24
*** dougsz has joined #openstack-ironic15:30
openstackgerritRuby Loo proposed openstack/ironic master: [WIP] Deploy steps Unabridged version  https://review.openstack.org/57864915:32
* TheJulia gives zuul cookies hoping for no random failures15:35
*** zhangfei has quit IRC15:35
*** hamzy has joined #openstack-ironic15:40
*** hkominos has quit IRC15:46
*** pbourke has quit IRC15:50
*** pbourke has joined #openstack-ironic15:52
*** racedo has joined #openstack-ironic15:53
dtantsurby the way, folks, does anyone here can change the topic? we have two incorrect links here, it seems15:59
dtantsurTheJulia: ^^15:59
TheJuliaThat is a really good question...16:00
TheJuliaI'm guessing it is some infra config file change, I just asked16:02
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75"16:08
TheJulia\o/16:08
*** baha has quit IRC16:11
TheJuliayolanda: If you have a moment to confirm or deny my concern re gpt partitions: https://review.openstack.org/#/c/568518/416:14
patchbotpatch 568518 - ironic-lib - Expose GPT partitioning fixing method16:14
*** d0ugal has quit IRC16:17
yolandaTheJulia, looking16:19
yolandaTheJulia, can you clarify that? i'm not understanding sorry16:23
TheJuliamjturek: two really minor pep8 errors on https://review.openstack.org/#/c/572102/, otherwise I think it looks good16:24
patchbotpatch 572102 - ironic-lib - Add logic to create PReP partition for ppc64* arch16:24
yolandaTheJulia, you mean... we are not passing config drive externally, but we already are providing a partition for config drive into our image?16:24
TheJuliayolanda: yes, I think you were doing that at one point in time16:25
TheJuliadue to partition limitations16:25
yolandaTheJulia, for whole disk images16:25
TheJuliayes16:25
yolandaso yep, i can see the problem, i think... you mean then, we are not considering the size of the config drive that is already present into our partitions16:25
TheJuliayolanda: more so, you write the whole disk image with a gpt table, at present we wouldn't fix the partition mapping.. I think16:26
TheJuliawell16:26
yolandai was doing that a long time in the past, i'd say for the bifrost/glean on infra16:26
*** raopajay has quit IRC16:26
TheJulianot parrtition mapping, but secondary disk copy16:26
TheJuliaoh, so way back in time16:27
TheJuliaokay16:27
yolandawe dont' do it for the security hardened images now16:27
yolandaTheJulia, so the fix will be to detect if the partitions have a config drive present there, even if the is_config_drive is not present16:27
yolandaotherwise it will skip that code16:28
TheJuliayolanda: yeah, but there is the other side of the same conditional16:29
yolandaTheJulia, but looking at the code... "disk_is_gpt_partitioned" will be true. Then it will execute the "fix_gpt_structs" part. But it will skip the sgdisk for the config drive one16:29
yolandathat's not bad16:29
TheJuliaokay16:31
yolandamm, None and False should behave the same right? So we should have None when called from ipa, and False when calling from config drive case16:32
* TheJulia lets yolanda ponder and goes and finds some lunch16:33
yolandai'm thinking it will be ok, as the code will just skip the config drive creation, that's expected16:34
yolandalet me answer you16:34
TheJuliaI'm worried that hte backup gpt table missing16:35
TheJuliamaybe I shouldn't be though16:35
yolandaah, i think i understand you now, you mean we won't be even arriving to that code, because we will never reach create_config_drive_parittion16:36
TheJuliacore reviewers: https://review.openstack.org/#/c/578794/ drops the ipa coreos core count down to 1, going to recheck it since... since IPA jobs seem moderately unreliable and time constrained I'm unsure if it is a good idea, or if we can just handle the occassional hypervisor that doesn't crash our VMs.16:37
patchbotpatch 578794 - ironic-python-agent - Reduce CoreOS CPU count to 116:37
TheJuliayolanda: yeah, I think16:37
*** dougsz has quit IRC16:37
TheJuliare IPA, I guess I'm unsure which is worse, the pain of underlying VMs crashing or the pain of jobs timing out due to emulation16:38
*** tesseract has quit IRC16:38
TheJuliayolanda: I guess I'm just worried someone is doing it and if we should try and head off the possible issue or not.16:40
*** miln has quit IRC16:40
yolandathat code cannot be called from another place?16:41
openstackgerritChuck Short proposed openstack/ironic-inspector master: Switch to using stestr  https://review.openstack.org/57852416:42
yolandaTheJulia, why not extract it, and call it from ironic - deploy_disk_image directly?16:42
yolandawe could split that method in 2 : fix_gpt_partition and create_config_drive_partition, and call both from the ironic method16:43
TheJuliayolanda: that is a really good idea16:44
yolandai don't see really any dependency from config drive that cannot be sorted16:44
yolandaok, let me add the comment there16:44
openstackgerritChuck Short proposed openstack/ironic-python-agent master: Switch to using stestr  https://review.openstack.org/57852116:44
markguzhi. is it possible to use boot_mode:uefi in combination with noop network?16:45
TheJuliamarkguz: it should be16:45
TheJuliamarkguz: local boot or network booting?16:45
markguzTheJulia: i switched a node to uefi to try it and now deploy fails due to it not finding an IP Address16:46
TheJuliamarkguz: thats a bug16:46
TheJuliamarkguz: can we get a backtrace?16:46
markguzTheJulia: installing via network then local boot16:46
markguzthere's no backtrace in the logs16:47
TheJuliayolanda: thanks!16:47
yolandagood catch there :)16:47
TheJuliamarkguz: the error and maybe a few lines around it for the same request or uuid16:47
markguzjust the ERROR16:47
TheJulialovely...16:47
TheJuliamarkguz: in that case... can we get a look at your `openstack baremetal node show <uuid>` output so we can see what your passing in field wise, because we're going to have to walk through a number of conditionals16:48
TheJuliamarkguz: redacted naturally :)16:48
markguzhttp://paste.openstack.org/show/724538/ is the logs around the error16:48
*** trown is now known as trown|lunch16:49
TheJuliaweird, that should only be for network booting16:49
markguzTheJulia: http://paste.openstack.org/show/724539/16:51
* TheJulia sighs because the CI gods do not shine upon me today16:51
markguzis boot_mode for installing or "booting" from the network?16:51
*** dsneddon has quit IRC16:52
*** dsneddon has joined #openstack-ironic16:52
TheJuliamarkguz: have you tried adding "boot_option:local"  to the properties?16:52
TheJuliaboot mode is the expeted mode of operation for the host, some drivers will use it to set the actual bios or efi boot mode16:53
markguzTheJulia: let me try that16:53
*** rh-jelabarre has quit IRC16:53
TheJuliayeah, it thinks it has been deploying whole disk images, but not explicitly boot option'ed. I bet we have an extra check that does not apply that is executing :\16:54
TheJuliaAnyway, I'm really going to go grab some lunch, bbiab16:54
markguzit has been deploying whole disk images and writing them to disk. I wanted to start doing that via UEFI rather than BIOS coz the new servers we get all default to UEFI booting16:55
*** baha has joined #openstack-ironic16:57
markguzmaybe i don't need to configure that in ironic, just set the dhcp boot file to the uefi file16:57
mjturekThanks TheJulia - yeah pep8 is broken on my dev vm so I rolled the dice I'll fix it right now16:57
TheJuliamarkguz: your likely going to have to do both, UEFI has a different flag16:58
*** baoli has joined #openstack-ironic16:58
TheJuliaand should use a different ipxe loader16:58
markguzTheJulia: I use regular pxe not ipxe16:59
TheJuliaoh!16:59
TheJuliawell, that is the exception on a lot of the servers....16:59
TheJuliahmmmmm16:59
TheJuliaAnywya, really really really actually going to go get some lunch, I'll dig into that when I get back17:00
*** derekh has quit IRC17:00
markguzTheJulia: thx17:00
*** baoli has quit IRC17:02
markguzTheJulia: i think the problem arises from uefi boot needing the ip address to create the /tftp/bootfile... but I'm using noop so it has no knowledge of the IP address17:02
*** athomas has quit IRC17:03
markguzTheJulia: The traditional bios boot mode used the mac address which makes more sense17:10
markguzTheJulia: to me at least17:10
openstackgerritMichael Turek proposed openstack/ironic-lib master: Add logic to create PReP partition for ppc64* arch  https://review.openstack.org/57210217:10
openstackgerritMerged openstack/ironic master: Do not disable inspector periodic tasks if [inspector]enabled is False  https://review.openstack.org/57840017:12
*** MrBerns has quit IRC17:14
*** ajya has quit IRC17:17
*** ajya has joined #openstack-ironic17:18
TheJuliamarkguz: I thought that was only for when uefi booting via the network for instances..17:19
markguzTheJulia: the server has to boot from the network to deploy the image17:19
TheJuliaindeed17:20
markguzTheJulia: ironic has to create a unique bootfile in /tftpboot17:20
* TheJulia goes and looks up the uefi pxe standard17:20
markguzas far as i can see for uefi, ironic wants to create a /tftpboot/ip_address.conf file17:21
*** dtantsur is now known as dtantsur|afk17:22
*** spartacloud has joined #openstack-ironic17:23
TheJuliamarkguz: what is your dhcp server sending out as bootfile name/bootfile server ?17:24
markguzTheJulia: bootx64.efi17:24
markguzwhich looks for /tftpboot/boot/grub/grub.cfg17:25
openstackgerritMerged openstack/ironic master: Deprecate [inspector]enabled option  https://review.openstack.org/57841117:25
TheJuliawhich then looks for an ip based file I believe17:25
markguzwhich according to the official ironic docs should be configure to look like this: http://paste.openstack.org/show/724540/17:25
markguzTheJulia: yeah. but with ye olde bios boot, it creates a file named with the mac addresses17:26
markguzi always configure the mac address and port in ironic.17:27
markguzbut i think it wants the ip address from neutron...17:27
*** spartacloud has quit IRC17:27
*** e0ne has joined #openstack-ironic17:28
*** e0ne has quit IRC17:28
markguzTheJulia: so unless i can define the ip address as driver_info then i'm probably SOL...17:29
TheJuliaI'm not 100% sure about that, but I've only ever thought of people really using ipxe in this case17:30
TheJuliabut it is also not signed17:30
* TheJulia raises an eyebrow and wonders if this was because of elilo17:32
markguzTheJulia: well we had an existing pxe and dhcp env here. I used that which worked pretty well with bios boots17:33
TheJuliayeah, makes sense not to chanage it unless you really have to17:34
markguzbut now the uefi stuff is becoming hard to ignore :-)17:34
TheJuliayeah17:37
TheJuliawhat, another 2 years for intel chips?17:37
markguzTheJulia: so i guess i can't do what i want to do with out some changes to my env or hacking the ironci code?17:41
TheJuliaseems like we could change out the IP address use17:41
TheJuliayeah, I think we were using ip addresses because of elilo17:42
TheJuliathere is a net_default_mac variable17:44
markguzoooh...17:44
markguzlet me try that17:44
markguzwait.. how will ironic know to use that...17:45
TheJuliaironic doesn't17:47
TheJuliawell17:47
TheJuliathe check would need to know that it could be17:47
markguzwell something has to know which variable to use to create the /tftpboot/ip.file17:48
markguzunless conductor reads the /tftpboot/boot/grub/grub.cfg file17:48
TheJuliait doesn't look like we create the grub config17:49
markguzuhm. confused now. with bios boot it creates the mac address pxelinux file17:49
TheJuliayeah, I think we explicitly chose to use the ip address17:50
markguzbut you're saying something else must create the /tftpboot/ip_address.conf ?17:50
*** trown|lunch has quit IRC17:50
markguzi see. where is that in the code ?17:50
TheJuliatrying to close some tabs to find it correctly17:51
openstackgerritMerged openstack/ironic master: Remove the ipmitool classic drivers  https://review.openstack.org/57809817:53
openstackgerritMerged openstack/ironic master: DevStack: Tiny changes following iRMC classic driver removal  https://review.openstack.org/57862817:53
TheJuliait looks like a fairly minimal patch and changing the temp file17:56
TheJuliaerr17:56
TheJuliatemplate file17:56
* TheJulia ponders17:56
*** markguz has quit IRC17:56
*** markguz has joined #openstack-ironic17:57
*** trown has joined #openstack-ironic17:57
*** mjturek has quit IRC17:57
*** markguz has quit IRC18:01
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Remove support for creating and loading classic drivers  https://review.openstack.org/57845718:07
*** mjturek has joined #openstack-ironic18:10
*** pcaruana has quit IRC18:14
*** mjturek has quit IRC18:15
*** mjturek has joined #openstack-ironic18:16
*** mbeierl has joined #openstack-ironic18:20
mbeierlQuick question: has anyone created parameterized unit tests in Ironic?18:21
TheJuliambeierl: do you mean like having a helper method with the tests just passing parameters in?18:23
mbeierlTheJulia: or like nose-parameterized, yes18:24
jrollwe have a bunch of tests like julia said, typically the method name starts with _test_18:24
*** yolanda_ has joined #openstack-ironic18:25
jrollwe don't use nose, so probably don't do anything "nose-parameterized"18:25
TheJuliayeah, the latter you speak of we've shyed away from due to "magic" factor18:25
mbeierlgotcha, thanks.  I'll take a look at the _test_ pattern and see if that fits for the changes I am going to propose to python-dracclient :)18:26
TheJuliaindividual vendor libraries can do whatever they would like as long as the interface remains stable18:27
*** yolanda has quit IRC18:27
mbeierlya, just want to play nicely with others, so if there is already a general feel for the way parameterized tests are done, I'd prefer to emulate that18:27
*** markguz has joined #openstack-ironic18:28
TheJuliaokay, generally we encourage keeping with-in the existing style for reading sanity, but we're a flexible group :)18:28
TheJuliamarkguz: I almost has a patch for you18:29
markguzTheJulia: Great. I appreciate that. was going to hack it myself !18:32
openstackgerritJulia Kreger proposed openstack/ironic master: WIP/POC: provide mac usage option with uefi grub  https://review.openstack.org/57895918:34
TheJuliamarkguz: give ^^ that a spin. hopefully the config option I put in makes enough sense as to how you need to edit your template.18:34
*** markguz_ has joined #openstack-ironic18:37
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Refuse secure erase if ATA command does not work  https://review.openstack.org/57520618:38
markguzTheJulia: ok thanks will try it now18:38
*** markguz has quit IRC18:38
*** markguz has joined #openstack-ironic18:39
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Provide knob to disable ata secure erase  https://review.openstack.org/57520718:42
*** markguz has quit IRC18:43
*** e0ne has joined #openstack-ironic18:47
*** markguz_ is now known as markguz18:47
markguzTheJulia: just checking, does the fact that patch failed CI, should that worry me?18:48
TheJuliahmmm18:48
TheJuliaoh, ibm's ci18:49
TheJuliaafaik it is broken at the moment18:49
TheJuliaI ave unit tests running on that patch now18:49
TheJuliaalthough, I didn't write any tests, it should solve the issue18:50
markguzTheJulia: Oh, i c IBM Powerkvm...18:50
markguznever mind18:50
*** e0ne has quit IRC18:50
openstackgerritSergii Golovatiuk proposed openstack/ironic master: Reboot server after 10 tries  https://review.openstack.org/57851218:58
rloowheee, one BIG deploy step! Bet no one wants to review it as is :) https://review.openstack.org/#/c/578649/19:16
patchbotpatch 578649 - ironic - [WIP] Deploy steps Unabridged version19:16
TheJuliarloo: that sounds like a challenge!19:19
rlooTheJulia: do you dare? :) I will split it up, but this was easier for me to test. fewer CI resources were used to test this :D19:20
TheJuliaI do dare, just not today19:21
markguzTheJulia: after applying the patch, ironic-conductor appears to no longer work.  doesnt conduct, doesn't start up the shellinabox daemons....19:21
TheJulia\o/19:22
markguzTheJulia: I probably shoudl have mentioned I'm on Ocata19:22
TheJuliaOH!19:22
*** rh-jelabarre has joined #openstack-ironic19:23
markguzTheJulia: sorry19:23
*** cjloader has quit IRC19:26
*** cjloader_ has joined #openstack-ironic19:26
*** cjloader_ has quit IRC19:27
*** cjloader has joined #openstack-ironic19:27
TheJuliamarkguz: with http://paste.openstack.org/show/724545/ being the resolution to the rebase conflict?19:28
markguzTHeJulia: i manually hacked that and it still didn't work.19:28
TheJulia:(19:29
markguzalthough i reverted everything now. and it's still broke so something else must be b0rked19:29
TheJuliawow, I can't even run unit tests on ocata19:30
TheJuliaany hints in the conductor log?19:30
*** jcoufal has joined #openstack-ironic19:31
*** jcoufal has quit IRC19:31
markguzTheJulia: I think it was  a false alarm. I was concerned that the shellinaboxes weren't auto starting, but i just set all the consoles to false then true again and tehy all came up19:47
*** dprince has quit IRC19:49
*** hkominos has joined #openstack-ironic19:50
hkominosHi all. Can i ask a question about ironic-conductor ?19:51
hkominosI think this is the right channel19:51
*** cjloader has quit IRC20:05
TheJuliahkominos: Hi, yes this is. whats up?20:06
openstackgerritIlya Etingof proposed openstack/ironic master: Migrate ironic `snmp` driver to the latest pysnmp API  https://review.openstack.org/57898720:09
*** ajya has quit IRC20:09
*** zul has quit IRC20:10
hkominosTheJulia: Hi I would like to know something about the file permissions because i suspect some of the scripts are messing with permissions . For example my ironic.conf is owned by root:ironic20:11
*** mjturek has quit IRC20:12
hkominosTheJulia: or for example stuff in /tftpboot which are supposed to be used to pxe boot are also owned by root. Is that so?20:12
*** baha has quit IRC20:12
*** cjloader has joined #openstack-ironic20:13
TheJuliahkominos: I think that would all depend on your environment settings, and how you installed it20:28
TheJuliafor ironic.conf root/ironic kind of makes sense in terms of providing write access on the group, read/write for root its self, blocking other users. for /tftpboot ironic will need to write to it, which likely should be as the group privilage or an extended attribute20:29
*** mjturek has joined #openstack-ironic20:30
*** baha has joined #openstack-ironic20:32
TheJuliamarkguz: since that seems to have been a false alarm, let me know if that works with updating the template. I'm happy to carry that patch along and write unit tests since it will generally improve things20:38
*** d0ugal has joined #openstack-ironic20:38
markguzTheJulia: Thanks will try it out shortly20:41
TheJuliacool20:41
TheJulianow... where was my brain20:41
*** d0ugal has quit IRC20:57
*** trown is now known as trown|outtypewww21:03
hkominosTheJulia: Thx for the info. I will continue tmr.21:20
*** rwsu has joined #openstack-ironic21:20
TheJuliahkominos: okay, have a wonderful evening!21:20
hkominosTheJulia. One more thing before I go. does this https://paste.fedoraproject.org/paste/iZeDNlt~1T6vr59K~r~SbA look normal to you? I have been trying to edit my pxe settings to provision aarch64 and the conductor keeps ignoring my config in ironic.conf. ANd then I saw that the process does not have a config-file attribute with it.21:23
TheJuliahmmm21:23
TheJuliaaarch6421:23
hkominosI am editing the pxe_per_architecture option but ironic keeps serving pxelinux.021:24
TheJuliawhere is this "pxe_per_architecture" setting you speak of?21:25
*** mjturek has quit IRC21:25
*** bfournie has quit IRC21:26
hkominosone min21:26
TheJuliaI'm curious because I'm not familiar with it, and a... at least direct string matching search had zero lines returned21:27
hkominoshttps://docs.openstack.org/ironic/latest/configuration/sample-config.html21:28
hkominospxe_config_template_by_arch =21:28
hkominosand pxe_bootfile_name_by_arch =21:28
hkominosAccording to the manual I expected to give aarch64:grubaa64.efi  and get the right bootloader for my aarch64 system. (I think ironic filters for a cpu_arch capability )21:30
*** serlex has quit IRC21:33
*** rcernin has joined #openstack-ironic21:42
TheJuliaokay, different parameter, that explains it21:45
TheJuliahkominos: it does filter by cpuarch, we don't store that data any other way21:46
TheJuliais the capability set?21:46
TheJuliaBTW, folks: https://etherpad.openstack.org/p/ironic-stein-ptg21:46
TheJuliaI put the two topics down21:46
TheJuliafrom earlier in the week21:47
hkominosbut what about the process itselft ?should it have a --config-file associated? otherwise how one would edit the config of the conductor. like this for heat https://paste.fedoraproject.org/paste/K7lCFN~7OnUS9W2nrszzSA/21:48
hkominosTheJulia: yes it is set21:48
TheJuliahkominos: by default we load /etc/ironic/ironic.conf21:49
*** cjloader has quit IRC21:49
*** cjloader_ has joined #openstack-ironic21:49
hkominosTheJulia: The node looks like this https://paste.fedoraproject.org/paste/v0g~acs94aIgdw7ISdinxA21:50
TheJuliathat was my next questtion21:50
hkominoscpu_arch:aarch6421:50
hkominosit is an aarch64 Vm btw. In case it matters21:51
TheJuliahmm21:53
*** cjloader_ has quit IRC21:54
TheJuliahkominos: what version of ironic?21:55
hkominosAll my ironic packages are herehttps://paste.fedoraproject.org/paste/YaetdlqIMXPxJqSZGhDCMw/21:57
hkominosI think this means 10.1.421:57
*** mjturek has joined #openstack-ironic21:58
TheJuliaI have a theory22:01
*** mjturek has quit IRC22:01
hkominosI am listening22:02
TheJuliaif you set debug=true in ironic.conf and restart the conductor process, do you see debug entries in the log?22:02
hkominosI have no access to the infra atm.22:04
TheJuliawhat does the flavor look like?22:04
hkominosCant answer that . What was your theory?22:05
hkominosthat the flavor has some arch specific config ?22:05
TheJuliaor none22:06
TheJuliawe refactored/updated the code heavily this cycle, so I'm looking at the stable branch22:06
*** baha has quit IRC22:09
hkominosTheJulia: Thx for the help.I will continue failing tmr. gn22:10
TheJuliahkominos: we log a line that starts with "Deploy boot mode is" with debug info22:10
TheJuliawhen debugging is enabled22:10
TheJuliathat would at least provide some insight to where it is going off the rails22:11
hkominosok. thx!22:13
*** bnemec has quit IRC22:19
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Provide knob to disable ata secure erase  https://review.openstack.org/57520722:20
TheJuliaAny cores around to review/approve https://review.openstack.org/#/c/578794/ ? It has passed CI 3x times with one dependent patch that has failed quite a bit :\22:21
patchbotpatch 578794 - ironic-python-agent - Reduce CoreOS CPU count to 122:21
*** hamzy has quit IRC22:30
*** hamzy has joined #openstack-ironic22:30
*** hamzy has quit IRC22:35
*** hamzy has joined #openstack-ironic22:37
*** linkmark has quit IRC22:38
JayFTheJulia: for #575207, is it intended that in the case where agent_enable_ata_secure_erase=false, you aren't even trying to run *shred*?22:48
JayFI guess it still gets the metadata...22:49
TheJuliacorrect, turns out there are enough fun cases where disks can get trashed22:49
TheJuliabecause pass-through buses will pass the security mode changes, then freak on the disk not honoring other commands22:50
TheJuliawait22:51
TheJuliawe should try to shred22:52
* TheJulia pulls it back up22:52
TheJuliayeah, we should fall through to shredding22:53
*** pmannidi has joined #openstack-ironic22:54
*** rwsu has quit IRC22:54
*** rwsu has joined #openstack-ironic22:55
*** etingof has quit IRC22:59
*** cjloader has joined #openstack-ironic23:03
*** threestrands has joined #openstack-ironic23:05
*** threestrands has quit IRC23:05
*** threestrands has joined #openstack-ironic23:05
*** yolanda__ has joined #openstack-ironic23:07
*** yolanda_ has quit IRC23:11
*** yolanda__ has quit IRC23:18
*** etingof has joined #openstack-ironic23:37
*** threestrands has quit IRC23:38
*** masber has quit IRC23:48

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