Tuesday, 2018-11-06

*** gyee has quit IRC00:25
*** hshiina has joined #openstack-ironic00:52
*** dsneddon has quit IRC01:08
*** mjturek has quit IRC01:24
TheJuliatonyb:  I suspect an option01:25
TheJuliaI was not aware openbmc implemented ipmi01:25
tonybTheJulia: It does, there is an alternate driver but it isn't ready today01:25
tonybTheJulia: Is theer a similar option I could consult as a template?01:26
TheJulianot really, but it is just the timeout flag right?01:27
TheJuliaactually, there might be in one of the ipmi options01:29
* TheJulia grabs link01:29
TheJuliatonyb: wait, that link is from the irmc driver01:30
tonybTheJulia: there may be another place we send those same bytes let me check01:31
tonybironic/drivers/modules/ipmitool.py:920 is probably the right point01:32
* tonyb did bad grepping01:32
TheJuliasomething kind of like http://git.openstack.org/cgit/openstack/ironic/tree/ironic/drivers/modules/ipmitool.py#n9501:33
tonybTheJulia: Okay so I could add ipmi_dont_do_the_timeout_thing abnd then set that on the node?01:34
TheJuliayeah01:34
TheJuliaor "ipmi_remote_bmc_openbmc" or "ipmi_the_bmc_does_not_like_us"01:35
tonybTheJulia: ... would that be backportable to rocky?01:35
*** rloo has quit IRC01:35
TheJulianein. :(01:35
tonybTheJulia: okay, that was kinda what I expected01:35
TheJuliaNothing really would :\01:35
TheJuliaSince it would be a new option to support openbmc01:36
tonybTheJulia: well support an otherwise fine implenentation of an IPMI based BMC ;p01:36
*** rh-jelabarre has joined #openstack-ironic01:37
TheJuliatonyb: what is it doign to the bmc anyway?01:37
tonybTheJulia: The IPMI spec says: don’t automatically clear boot flag valid bit if Chassis Control command not01:38
tonyband JayF added it in https://review.openstack.org/#/c/134413/ claiming that some BMCs got grumpy it it wasn't sent01:39
patchbotpatch 134413 - ironic - Disable IPMI timeout before setting boot device (MERGED) - 1 patch set01:39
tonybTheJulia: something about not resetting the device boot timeout after 60seconds01:39
TheJuliaThat was a long time ago01:39
tonybTheJulia: Yeah but it'd be next to impossible to verify it isn't needed anymore01:40
TheJuliayup01:40
TheJuliaDoes the bmc tank, or does it just respond poorly?01:40
tonybTheJulia: We could add a flag to disable it and slow transition that from fale -> true -> gone to remove it01:40
tonybTheJulia: ipmitool returns non-zero and what cause the boot to abort with the appropriate exception01:41
TheJuliaany specific error string? anythign we can use to determine it is not horribly fatal for us?01:41
tonybTheJulia: Hmm I'll grep for the logs, I'm seeing it through a tripleo lens01:42
tonybTheJulia: gimme 5-10 to find it01:42
tonybironic-conductor.log:2018-11-05 18:49:21.877 1 ERROR ironic.drivers.modules.ipmitool Stderr: u'Unable to send RAW command (channel=0x0 netfn=0x0 lun=0x0 cmd=0x8 rsp=0x80): Unknown (0x80)\n'01:43
tonybthat was much easier than I expected01:43
TheJuliafun01:43
* TheJulia wonders if we know that... if we could just kind of go "okay" and move along01:44
tonybTheJulia: I was thinking of adding a ok_to_fail=False kwarg to send_raw and then setting that to True for this one call but adding an option seems more correct01:46
*** gabys has joined #openstack-ironic01:51
* tonyb will file a bug and change upon which we can discuss the merits01:54
TheJuliasounds good01:55
TheJuliacould you add it to the meeting discussion list?01:56
*** gabys has quit IRC02:01
*** gabys has joined #openstack-ironic02:02
tonybTheJulia: meeting discussion list?02:06
tonybTheJulia: https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting ?02:07
TheJuliayup, I've not cleaned it up from today, but I'll see it when I edit it next02:08
tonybOkay cool. I wont be there as it's at 0200 for me, this would be for after the summit correct?02:09
TheJuliayes02:10
tonybcool02:10
TheJuliawe've cancelled for the next two weeks, I've just not sent the email02:10
*** ianychoi has quit IRC02:10
TheJuliabecause OMG all the things02:10
tonyb:)02:10
*** ianychoi has joined #openstack-ironic02:10
*** ajya[m] has quit IRC02:17
*** toabctl has quit IRC02:19
*** toabctl has joined #openstack-ironic02:23
*** ajya[m] has joined #openstack-ironic02:50
*** ajya[m] has quit IRC03:39
openstackgerritMerged openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/60998204:17
openstackgerritMerged openstack/ironic master: Improve sushy mocks  https://review.openstack.org/61562104:17
*** ajya[m] has joined #openstack-ironic04:35
*** jtomasek has joined #openstack-ironic05:14
*** rh-jelabarre has quit IRC05:32
*** skazi has quit IRC05:34
*** S4ren has quit IRC05:44
*** S4ren1 has joined #openstack-ironic05:44
openstackgerritIlya Etingof proposed openstack/ironic master: Add configurable Redfish client authentication  https://review.openstack.org/60381805:48
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Support ip6tables for iptables pxe filter  https://review.openstack.org/61572805:51
openstackgerritIlya Etingof proposed openstack/ironic master: Kill misbehaving `ipmitool` process  https://review.openstack.org/61000706:02
*** e0ne has joined #openstack-ironic06:24
*** jtomasek has quit IRC06:39
*** e0ne has quit IRC06:44
*** e0ne has joined #openstack-ironic06:48
*** e0ne has quit IRC07:07
*** iurygregory|home is now known as iurygregory07:19
*** lenka has joined #openstack-ironic07:22
iurygregorygood morning07:25
*** pcaruana has joined #openstack-ironic07:36
*** adrianc has joined #openstack-ironic07:43
*** gkadam_ has joined #openstack-ironic07:45
*** arne_wiebalck_ has joined #openstack-ironic07:54
*** gabys has quit IRC07:58
*** gabys has joined #openstack-ironic07:59
*** iurygregory is now known as iurygregory|mtg08:01
*** lenka has quit IRC08:03
*** gabys has quit IRC08:03
*** tssurya has joined #openstack-ironic08:18
*** priteau has joined #openstack-ironic08:39
*** jtomasek has joined #openstack-ironic08:41
openstackgerritXing Zhang proposed openstack/bifrost stable/rocky: Fix ironic configuration for new oslo messaging  https://review.openstack.org/61578108:43
*** betherly has joined #openstack-ironic09:01
*** lenka has joined #openstack-ironic09:10
*** dougsz has joined #openstack-ironic09:11
openstackgerritNguyen Hai Truong proposed openstack/python-ironic-inspector-client master: Add python 3.6 unit test job  https://review.openstack.org/61579409:15
*** priteau has quit IRC09:20
*** Emine has joined #openstack-ironic09:20
*** priteau has joined #openstack-ironic09:23
jiapeiGood morning all o/09:25
etingofgood morning jiapei, iurygregory|mtg and the rest of ironic o/09:38
openstackgerritIlya Etingof proposed openstack/ironic master: Kill misbehaving `ipmitool` process  https://review.openstack.org/61000709:38
*** derekh has joined #openstack-ironic09:39
jiapeihey, morning etingof09:39
*** hshiina has quit IRC09:59
*** serlex has joined #openstack-ironic10:01
*** iurygregory|mtg is now known as iurygregory10:20
iurygregoryetingof, morning o/10:20
openstackgerritNguyen Van Trung proposed openstack/ironic master: Support firmware upgrade for irmc driver  https://review.openstack.org/59999210:35
*** adrianc has quit IRC10:40
*** iurygregory is now known as iury|lunch10:41
*** mkrai has joined #openstack-ironic10:57
mkraiHi I am getting error while deploy where IPA fails to download image from Swift temp10:58
mkraiI see that there is no container in swift10:59
mkraiCan someone please help me with this?10:59
mkraiHere is the error http://paste.openstack.org/show/734197/11:02
*** iury|lunch is now known as iurygregory11:11
*** e0ne has joined #openstack-ironic11:11
openstackgerritJavier Peña proposed openstack/ironic master: Fix redfish test_get_system_resource_not_found test  https://review.openstack.org/61583311:23
*** dtantsur|afk is now known as dtantsur11:33
dtantsurmorning/afternoon ironic11:33
jrollmorning dtantsur and everyone else11:34
dtantsuretingof: mind joining #rdo? it seems like your sushy test update broke unit tests there11:34
iurygregorymorning guys o/11:34
dtantsurmorning jroll and iurygregory11:34
jrolltonyb: TheJulia: at least one set of hardware that needed that raw commmand is from 2015? 2016? so I suspect we can't just drop it, though +1 for making it optional :)11:35
*** bfournie has joined #openstack-ironic11:39
mkraiGood morning Ironic!!11:40
openstackgerritJavier Peña proposed openstack/ironic master: Fix redfish test_get_system_resource_not_found test  https://review.openstack.org/61583311:40
mkraiHi I am getting error while deploy where IPA fails to download image from Swift temp URL11:40
mkraiHere is the error http://paste.openstack.org/show/734197/11:40
mkraidtantsur: jroll Can you please help?11:41
jrollmkrai: I saw your message earlier, if I could help I would :)11:42
jrollI've never seen a 409 from a temp url11:43
dtantsurditto11:43
dtantsurmaybe ask swift folks?11:43
jrollalthough11:43
jrollmkrai: you say you see no container in swift, are you using swift as your glance backend?11:43
mkraijroll: ok :)11:43
mkraijroll: I just check the container is there11:43
mkraia container named glance11:43
jrollah ok, no clue why the 409 then11:44
mkraiyes swift as glance backend11:44
iurygregorydid you check glance and swift logs and if the services are running ok?11:45
*** gkadam_ has quit IRC11:46
*** gkadam has joined #openstack-ironic11:46
mnasiadkaguys, once this gets merged: https://review.openstack.org/#/c/615781/ - can you plan for a release soon? bifrost in kolla stable/rocky is broken due to that, and we rely on released versions.11:49
patchbotpatch 615781 - bifrost (stable/rocky) - Fix ironic configuration for new oslo messaging - 1 patch set11:49
jrolldtantsur: ^^11:50
* jroll just +2'd11:50
jrollmnasiadka: I'll make sure we get that released soon, thanks11:50
mnasiadkajroll: once a release gets out - I'll be thanking then :)11:51
jroll:)11:52
dtantsur+A11:53
jrollthanks11:54
dtantsurjroll: please check https://review.openstack.org/#/c/615833/11:58
patchbotpatch 615833 - ironic - Fix redfish test_get_system_resource_not_found test - 2 patch sets11:58
jrollwhat happened to our job that installs driver-requirements? :(11:59
dtantsurit has never been created..11:59
jroll+W11:59
jrollI thought we did11:59
*** mkrai has quit IRC11:59
jrollor rather, I thought pas-ha did11:59
dtantsurwe only created a tox target12:00
jrollah12:00
jrollalso, why does my house smell like someone is cooking breakfast when I'm the only person/animal awake? >:(12:01
*** adrianc has joined #openstack-ironic12:06
dtantsurheh12:10
* dtantsur thinks about lunch at this point12:13
etingofyour breakfast is being deployed, jroll o/12:13
openstackgerritMerged openstack/ironic master: Fix redfish test_get_system_resource_not_found test  https://review.openstack.org/61583312:13
jrolletingof: if only!12:13
*** dnuka has joined #openstack-ironic12:16
dtantsurdoes anyone remember from the top of their head if nova uses ironic port API?12:16
dnukaGood morning o/12:16
dnukaGood morning dtantsur :)12:17
jrolldtantsur: I'm about 80% sure it does12:17
dtantsurmorning dnuka12:17
dnukaGood morning iurygregory , etingof :)12:17
dtantsurjroll: sigh, I was hoping to omit it from ironic-proxy prototype :D12:17
openstackgerritMerged openstack/bifrost stable/rocky: Fix ironic configuration for new oslo messaging  https://review.openstack.org/61578112:18
jrollhttps://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L197612:18
iurygregorydnuka, morning o/ sorry for the review some internal things showed up for me yesterday and today12:18
dtantsuroh, even portgroups..12:19
dnukaiurygregory: It's alright :)12:19
dtantsurTheJulia: I'd like to make a bifrost rocky release per mnasiadka's request above. any objections?12:20
dtantsurjroll: right. I'll test with metalsmith for now then. proxying ports/port groups is something I have to think about12:21
* jroll likes it when dtantsur does things like releases that are already on his todo list12:22
etingofdtantsur, oh, I missed your ping :(12:22
dtantsurjroll: I can leave it up to you for sure ;)12:26
*** e0ne has quit IRC12:29
jrolldtantsur: okay :) https://review.openstack.org/61585212:29
patchbotpatch 615852 - releases - Release bifrost stable/rocky at 5.1.1 - 1 patch set12:29
jrollTheJulia: ^ when you're around, please :)12:29
etingofso the sushy@#rdo matter seems to be resolved12:39
*** e0ne has joined #openstack-ironic12:43
dtantsuryup12:53
*** Emine has quit IRC12:59
*** cdearborn has joined #openstack-ironic13:05
*** Emine has joined #openstack-ironic13:15
*** emine__ has joined #openstack-ironic13:19
*** Emine has quit IRC13:20
*** jcoufal has joined #openstack-ironic13:23
*** dnuka has quit IRC13:25
*** rloo has joined #openstack-ironic13:27
sambetts|afkdtantsur: the plan with /nodes/<uuid>/vifs was to remove the requirement on nova to talk the ports API, but currently it requires it to populate the mac addresses for config drive, I wanted to switch it to use the info (mac/binding) on the neutron port we update to populate the mac addresses and other info before I got pulled away from things13:32
dtantsurI see13:34
* TheJulia wipes sleep from eyes13:37
TheJuliajroll: is your house haunted? :)13:38
TheJuliaGood morning everyone!13:38
TheJuliadtantsur: bifrost release ++13:40
dtantsurTheJulia: jroll already requested :)13:40
dtantsuralso g'morning13:40
TheJulia\o/13:40
* TheJulia greatly dislikes daylight savings time changes13:40
*** iurygregory has quit IRC13:41
*** iurygregory has joined #openstack-ironic13:42
TheJuliaI feel like I've seen mkrai's issue when the tempurl shared key does not match13:44
jrollI think that's typically a 401 or 403, but I could be wrong13:44
TheJuliaI think that is only when it can decode it and the time stamps for authorization are no longer valid13:45
TheJuliaI think13:46
TheJuliareally, as suggested, swift logs need to be reviewed13:46
*** zul has joined #openstack-ironic13:46
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: playbooks: Use --upgrade-strategy option for openSUSE distributions  https://review.openstack.org/61263213:47
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [DNM] Add openSUSE Leap 15 jobs  https://review.openstack.org/59952113:47
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Remove paste references on keystone deployments  https://review.openstack.org/61587713:47
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: Use the Connection.baremetal adapter for ironicclient Session  https://review.openstack.org/61587813:47
TheJuliacould I get some eyeballs on https://review.openstack.org/#/c/449425/ today? I'd like to add one or two more things to my "already merged into stein" list13:48
patchbotpatch 449425 - ironic - Add Redfish inspect interface - 16 patch sets13:48
* TheJulia misses the good old reproduce.sh....13:50
*** sthussey has joined #openstack-ironic13:52
*** mkrai has joined #openstack-ironic13:57
*** yolanda has joined #openstack-ironic14:08
openstackgerritNguyen Hai Truong proposed openstack/networking-generic-switch master: Add python 3.6 unit test job  https://review.openstack.org/61588814:10
*** emine__ has quit IRC14:15
dtantsurFailed to deploy: {u'priority': 100, u'interface': u'deploy', u'step': u'deploy', u'argsinfo': None}14:16
dtantsurWUT?14:16
TheJulianothing else?14:18
iurygregorythis just the output from ansible no?14:23
dtantsurnope, this is a deploy step14:25
*** early has quit IRC14:25
dtantsurand this is the full content of last_error14:25
*** early has joined #openstack-ironic14:28
dtantsurnow last_error=None provision_state=deploy failed14:32
dtantsurwith the root cause being outdated ironic-lib...14:33
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: playbooks: Use --upgrade-strategy option for openSUSE distributions  https://review.openstack.org/61263214:34
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [DNM] Add openSUSE Leap 15 jobs  https://review.openstack.org/59952114:34
TheJuliadtantsur: perhaps we should revise how we update the last error on exit of the decorator?14:37
dtantsuryyeaaah14:51
*** baha has joined #openstack-ironic14:53
dtantsur\o/ ironic-proxy is now able to deploy/undeploy (using metalsmith, dunno about nova)14:54
*** munimeha1 has joined #openstack-ironic14:57
*** rh-jelabarre has joined #openstack-ironic14:59
*** skazi has joined #openstack-ironic14:59
*** arne_wiebalck_ has quit IRC15:01
*** arne_wiebalck_ has joined #openstack-ironic15:04
*** e0ne has quit IRC15:06
rloogood morning. dtantsur, TheJulia: did we do a release of rocky or queens? (cuz of the rolling upgrade bug)?15:07
mbeierldumb question time: I was doing a bit of a feature compare of different bare metal deployment tools.  I just want to confirm: it is not possible to use Ironic for something like PXE botting from ISO with a kickstart/preseed file, correct?15:18
mbeierl*booting*15:18
mbeierlI'm thinking forward to edge use cases where there might not be a pre-built qcow cloud image for the bare metal application running at the edge.15:20
dtantsurmbeierl: I don't think it's actually possible. also I don't quite get how this limitation is linked to the Edge case, could you elaborate?15:28
mbeierlI am thinking of a fictitious product: it is a bare metal deployment with its own installer.  It is shipped on an ISO and has the ability to set operating system parameters via a kickstart file.15:29
*** cdearborn has quit IRC15:30
*** gkadam has quit IRC15:30
mbeierldtantsur: an option would be to create a qcow via virt-manager, but if this product decides to install components based on the presence of certain hardware, that would not work.15:31
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: playbooks: Use --upgrade-strategy option for openSUSE distributions  https://review.openstack.org/61263215:41
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [DNM] Add openSUSE Leap 15 jobs  https://review.openstack.org/59952115:41
* TheJulia sighs with devstack15:47
TheJuliarloo: I don't think so, we should.15:47
dtantsur++15:47
dtantsurand ++ to sigh about devstack15:47
TheJulia:)15:47
TheJuliaI think there is a race in start-up :\15:47
TheJuliaof vms15:48
* TheJulia added some ||true to try and get past a point15:48
* dtantsur adds ||true everywhere15:49
*** priteau has quit IRC15:49
rloodtantsur: I assume you or your people :) tested that rolling upgrade fix?15:50
dtantsurrloo: I did not directly test it, but I think our folks did. in any case, we'll learn when they close/reopen that bug :D15:50
rloodtantsur: heh. would be good if they tried it out, before we release!15:51
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Fix libvirt driver to handle domains by UUID  https://review.openstack.org/61431615:54
*** arne_wiebalck_ has quit IRC15:54
*** mkrai has quit IRC15:56
*** arne_wiebalck_ has joined #openstack-ironic16:02
*** hamzy_ has quit IRC16:19
*** gyee has joined #openstack-ironic16:21
*** mjturek has joined #openstack-ironic16:21
mjturektonyb: saw your mesage from yesterday.. why is it hitting IRMC modules on the ppc64le node? It should be using the IPMItool one (at least that's what we use) http://git.openstack.org/cgit/openstack/ironic/tree/ironic/drivers/modules/ipmitool.py#n89516:29
*** arne_wiebalck_ has quit IRC16:31
*** mjturek has quit IRC16:36
*** andymccr has quit IRC16:36
*** mjturek has joined #openstack-ironic16:36
openstackgerritWill Szumski proposed openstack/tenks master: Wait for resources to become available  https://review.openstack.org/61593916:40
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: playbooks: Use --upgrade-strategy option for openSUSE distributions  https://review.openstack.org/61263216:41
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [DNM] Add openSUSE Leap 15 jobs  https://review.openstack.org/59952116:41
*** hamzy_ has joined #openstack-ironic16:44
openstackgerritMerged openstack/metalsmith master: Use the Connection.baremetal adapter for ironicclient Session  https://review.openstack.org/61587816:56
openstackgerritWill Szumski proposed openstack/tenks master: Add an initial noop job to bootstrap zuul  https://review.openstack.org/61594517:05
openstackgerritMerged openstack/tenks master: Add an initial noop job to bootstrap zuul  https://review.openstack.org/61594517:06
openstackgerritMerged openstack/tenks master: Add a .gitreview file  https://review.openstack.org/61521817:08
TheJuliaeureka!17:11
openstackgerritWill Szumski proposed openstack/tenks master: Add an initial set of zuul jobs  https://review.openstack.org/61522417:16
*** S4ren1 has quit IRC17:18
*** skazi has quit IRC17:27
*** gabys has joined #openstack-ironic17:29
openstackgerritWill Szumski proposed openstack/tenks master: Wait for resources to become available  https://review.openstack.org/61593917:31
*** e0ne has joined #openstack-ironic17:32
*** gabys has quit IRC17:32
*** gabys has joined #openstack-ironic17:33
*** gabys has quit IRC17:37
rpiosoGood morning, ironicers17:55
rpiosoTheJulia: o/17:56
rpiosoTheJulia: Recall the issue I encountered with iscsi deploy on Friday.17:56
*** dougsz has quit IRC17:56
TheJuliayes17:56
rpiosoTheJulia: Installation of libguestfs-tools installs iscsi-initiator-utils-iscsiuio.x86_64 0:6.2.0.874-7.el717:58
rpiosoI did that after having installed the tripleo undercloud via "openstack undercloud install".17:59
TheJuliaafaik iscsi-initiator-utils depends on iscsi-initiator-utils-iscsiuio17:59
rpiosoRight, it was installed along with iscsi-initiator-utils.x86_64 0:6.2.0.874-7.el718:00
*** derekh has quit IRC18:01
rpiosoAfter installing the undercloud, but before installing those tools, iscsiadm is not found on the undercloud's CentOS.18:01
rpiosoWhere does the ironic conductor container get it from?18:02
rpiosoI'm thinking we may have been looking in the wrong place.18:02
*** e0ne has quit IRC18:03
TheJuliait should be built into the ironic conductor container18:04
rpiosoHow can I examine its version and command output?18:05
*** skazi has joined #openstack-ironic18:05
TheJuliaafaik, you should be able to run commands directly against the container18:05
rpiosoSuch as "docker exec -ti $CONTAINER_ID_OR_NAME /bin/bash"18:06
TheJuliayup18:06
rpiosoI'll look18:06
*** dtantsur is now known as dtantsur|afk18:06
dtantsur|afko/18:06
TheJuliagoodnight dtantsur|afk18:06
rpiosodtantsur|afk: \o18:06
*** e0ne has joined #openstack-ironic18:06
* TheJulia makes frowny faces at devstack18:08
*** adrianc has quit IRC18:17
rpiosohttps://www.irccloud.com/pastebin/tIEhJF4R/18:17
*** jtomasek has quit IRC18:18
TheJuliait needs to be executed as root18:18
rpiosohttps://www.irccloud.com/pastebin/cgwTR6k6/18:23
* rpioso needs to grab lunch before the cafeteria closes18:24
*** rpioso is now known as rpioso|afk18:24
*** e0ne has quit IRC18:24
*** tssurya has quit IRC18:26
openstackgerritMerged openstack/tenks master: Only set Ironic port attributes when changed  https://review.openstack.org/61525418:26
openstackgerritMerged openstack/tenks master: Use extend-ignore in preference to ignore  https://review.openstack.org/61525218:26
openstackgerritMerged openstack/tenks master: Fix lint warning concerning task names  https://review.openstack.org/61525318:26
*** gyee has quit IRC18:34
openstackgerritMerged openstack/tenks master: Add an initial set of zuul jobs  https://review.openstack.org/61522418:39
TheJuliarpioso|afk: so interestingly enough, that doesn't match the errors that were appearing in the logs for the same command, granted there was an ip address, but it was throwing a syntax error if memory serves18:39
*** hamzy_ is now known as hamzy18:53
openstackgerritMerged openstack/ironic master: Change BFV job to use ipxe interface  https://review.openstack.org/61453519:06
*** brandor5 has joined #openstack-ironic19:08
*** serlex has quit IRC19:11
*** cxc99 has quit IRC19:12
*** rpioso|afk is now known as rpioso19:15
hwoarangTheJulia: hello. about a month ago we were looking at some opensuse150 problems with bifrost on https://review.openstack.org/#/c/599521/ The VM log is now available there. My understanding is that is_whole_disk_image: True is the problem (http://logs.openstack.org/21/599521/28/check/bifrost-integration-tinyipa-opensuse-150/88b2eb2/logs/baremetal.txt)19:16
patchbotpatch 599521 - bifrost - [DNM] Add openSUSE Leap 15 jobs - 28 patch sets19:16
hwoarangTheJulia: i have no other debug or investigation info to share at this point :/ if you have any idea please let me know19:17
TheJuliahwoarang: agent deploy logs or maybe the console log could help...19:17
hwoarangthe VM console is here http://logs.openstack.org/21/599521/28/check/bifrost-integration-tinyipa-opensuse-150/88b2eb2/logs/testvm1_console.log19:18
TheJuliayup, I see the issue19:18
TheJulialooks like the kernel is now wanting a root=/dev/ram0 entry19:19
* TheJulia looks to see where things fail19:21
hwoaranglooks like as if no disk is attached19:21
*** cxc99 has joined #openstack-ironic19:22
rpiosoTheJulia: The output just above was immediately following the installation of the undercloud. libguestfs-tools had not been installed.19:23
rpiosoPlaying forward to the snapshot following installation of libguestfs-tools:19:23
rpiosohttps://www.irccloud.com/pastebin/zVRnjgpW/19:23
rpiosoLook familiar?19:23
TheJuliawow19:24
TheJuliaso installing libguestfs-tools breaks it?19:24
TheJuliaor replaces the iscsiadm binary?19:24
TheJuliaor ?replaces? it19:24
rpiosoOr something19:24
TheJuliawtf19:24
rpiosoIt was installed on the base CentOS 7.5.19:24
TheJuliahwoarang: still looking, looks like it did inspect19:24
TheJuliarpioso: give me a few and I'll fire up my centos machine and take a look19:25
rpiosoPrior to the installation, the base didn't have iscsiadm.19:25
rpiosoI did a minimal installation of CentOS 7.5.19:25
TheJuliaoh jeeze, that seems really bad :(19:26
*** brandor5 has left #openstack-ironic19:29
TheJuliahwoarang: you win an award for the most cryptic failure of the week19:31
TheJulia:(19:31
hwoarangthat makes me so proud :)19:31
hwoaranghow can you tell if the image was flushed to the disk?19:32
TheJuliaThat is a great question19:32
TheJuliaThe logs should indicate the deploy was done19:32
TheJuliahmmm19:35
hwoarangmaybe we should also capture the logs during the initial disk flushing by the ironic agent19:37
TheJuliapossibly19:38
TheJulialooks like the ramdisk does see the inspection ramdisk contents...19:38
TheJuliaerr19:38
hwoaranganyway not getting anywhere right now. i am just curious why the bootlog does not say anything about a 'vda' disk compared to suse 42 or centos logs19:38
TheJuliaso discoveyr looks broken19:38
TheJuliasanboot failing seems... problematic19:39
hwoarangi suspect i might be missing a package to make virtio disks work19:39
TheJulia_or_ virtio disks no longer work with ipxe19:39
hwoarango_O19:39
*** gabys has joined #openstack-ironic19:40
hwoarangok i will leave it here for today. I will try more things tomorrow :/ thank you for your help so far!19:40
*** mjturek has quit IRC19:41
*** mjturek has joined #openstack-ironic19:43
TheJuliaokay19:45
TheJuliaI'm looking, this is intruiging19:45
tonybmjturek: Yeah I used the wrong link  I opened a Story with the logs and the correct link.19:45
mjturektonyb: cool, I'll take a look19:46
tonybmjturek, TheJulia, jroll: I plan on pushing a change today to make it optional19:46
TheJuliatonyb: most excellent19:46
mjturekwill review19:46
jroll👍19:46
tonyb:)19:46
*** dsneddon has joined #openstack-ironic19:46
* tonyb is blocked on $other_things so it'll be nice to do somethign upstream for a bit :)19:47
*** luzC has quit IRC19:47
openstackgerritJulia Kreger proposed openstack/bifrost master: Sudo is needed to copy logs  https://review.openstack.org/61598319:52
*** luzC has joined #openstack-ironic19:52
*** arne_wiebalck_ has joined #openstack-ironic19:54
openstackgerritJulia Kreger proposed openstack/bifrost master: Mark discovery ramdisk to have an initrd  https://review.openstack.org/61598419:55
*** ptx0 has quit IRC19:55
*** gabys has quit IRC19:56
*** gabys has joined #openstack-ironic19:56
*** gabys has quit IRC20:01
*** ptx0 has joined #openstack-ironic20:02
TheJuliahwoarang: I approved two of your changes to land, and I've got another change and your change rebased on top. It might fix things, it won't solve the entire issue, but it should get things moving and from there we might want to explore alternatives to virtio if the rom loading doesn't grok the disks.....20:09
TheJuliarpioso: md5sum that iscsiadm file for me20:18
TheJuliasame behavior after installing libguestfs-tools on my centos machine20:19
jrollblurgh, this is fun: https://storyboard.openstack.org/#!/story/200427920:21
*** jtomasek has joined #openstack-ironic20:21
TheJuliaugh20:21
* TheJulia steps away to go check the pizza dough before pivoting back to the frustration of devstack20:22
jrollmmmm pizza.20:22
jrollthat reminds me, this is awesome: https://polls.pizza/20:23
TheJuliaThat is awesome20:24
*** jtomasek has quit IRC20:24
openstackgerritMerged openstack/bifrost master: Remove paste references on keystone deployments  https://review.openstack.org/61587720:25
openstackgerritMerged openstack/bifrost master: playbooks: Use --upgrade-strategy option for openSUSE distributions  https://review.openstack.org/61263220:26
rpiosoTheJulia: Post installation of libguestfs-tools:20:26
rpiosohttps://www.irccloud.com/pastebin/wXhu2LXU/20:26
rpiosoI'm glad you reproduced it :)20:27
TheJuliano, I didn't reproduce it20:28
TheJuliaI get connection refused errors which is what we woudl expect20:28
TheJuliarpioso: `strace -f -F -o strace.out iscsiadm -m discovery -t st -p 127.0.0.2:3260` and then email me strace.out20:30
TheJuliasomething doesn't align, but I _DO_ have the exact same iscsiadm binary20:30
TheJuliayou may need to install strace20:30
rpiosoTheJulia: I had started the revert to before libguestfs-tools process. I'll grab the md5sum value there.20:32
rpiosoWould strace be helpful at that point in time?20:32
rpiosoI assume strace should be run within the ironic-conductor container.20:32
rpiosoTheJulia: Before installationof libguestfs-tools:20:34
rpiosohttps://www.irccloud.com/pastebin/4RFOcEAm/20:35
*** mjturek has quit IRC20:38
*** mmethot_ is now known as mmethot20:38
*** mjturek has joined #openstack-ironic20:39
*** mjturek has quit IRC20:46
*** munimeha1 has quit IRC20:46
TheJuliahmm rpioso that makes me think that they are unrelated20:49
TheJuliasince the contents and the behavior of the container really hsouldn't be impacted by the host os on this level20:50
*** mjturek has joined #openstack-ironic20:51
*** mjturek has quit IRC20:54
rpiosoTheJulia: In the container, the version of iscsiadm and the md5sum are identical before and after installation of libguestfs-tools.21:06
TheJuliaindeed, and your installing it outside of the container, which should not impact container contents21:06
TheJuliamaybe something is getting pulled in dependency wise that is changing the appearance of the underlying host holding the undercloud controller container21:07
*** arne_wiebalck_ has quit IRC21:07
TheJuliainstalling did seem to activate the iscsi tcp transport class, but that could have just been my first exec of iscsiadm21:08
rpiosoPerhaps one of more of its dependencies were upgraded by the installation of those tools.21:09
rpioso“maybe something is getting pulled in ...” That’s what I was thinking, too :)21:11
*** jcoufal has quit IRC21:25
*** mjturek has joined #openstack-ironic21:26
*** pcaruana has quit IRC21:31
*** mjturek has quit IRC21:31
*** dsneddon_ has quit IRC21:43
*** mjturek has joined #openstack-ironic21:44
*** hamzy has quit IRC21:45
rpiosoTheJulia: I sent you the strace from before and after.21:53
jrolldtantsur|afk: I feel like we have had conversations about things this would solve in the past. in case you hadn't seen it: https://docs.openstack.org/oslo.config/latest/reference/locations.html21:54
*** baha has quit IRC22:10
*** mjturek has quit IRC22:12
*** rh-jelabarre has quit IRC22:14
* TheJulia just bangs her head into the dsk22:17
TheJuliadesk22:17
*** lenka has quit IRC22:22
openstackgerritJulia Kreger proposed openstack/bifrost master: [DNM] Add openSUSE Leap 15 jobs  https://review.openstack.org/59952122:34
openstackgerritJulia Kreger proposed openstack/bifrost master: Set default_boot_option for bifrost  https://review.openstack.org/61602122:34
*** lenka has joined #openstack-ironic22:50
openstackgerritJim Rollenhagen proposed openstack/ironic master: Allow disabling instance image cache  https://review.openstack.org/61603123:31
jroll^ still needs tests and a release note, but that's my thoughts on that bug23:32
*** lenka has quit IRC23:40
TheJuliajroll: ack ack23:50
TheJuliarpioso: sent you an email with my findings23:50
TheJuliatl;dr the reply over the unix socket to iscsid is different. Why... no clue23:50
*** MattMan_ has quit IRC23:51
*** MattMan_ has joined #openstack-ironic23:52
jrollTheJulia: no rush btw, we'll be cherry-picking it downstream23:52
rpiosoTheJulia: Thank you so much. I'll need to dig in further.23:54
*** rpioso is now known as rpioso|afk23:54
rpioso|afko/23:54
TheJuliarpioso|afk: thanks, there really is not much else I can do right now since I can't reproduce it locally... and libguestfs-tools has an insane number of dependencies23:55
TheJuliarpioso|afk: let me know what you find23:55
TheJuliaAnyway... it is 4pm local. I've been at it since sometime between 5-6 AM this morning. I think it is time to go do things like laundry and maybe start dinner23:56
* jroll passes TheJulia a pint23:57
jrollhave a good night!23:57
* TheJulia really wonders where the day went23:57

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