Friday, 2017-08-04

openstackgerritMerged openstack/ironic master: Add hardware type for HPE OneView  https://review.openstack.org/48579800:00
openstackgerritMerged openstack/ironic master: Update ironic.conf.sample due to non-ironic code  https://review.openstack.org/49022900:03
*** Goneri has joined #openstack-ironic00:03
*** rama_y has quit IRC00:15
*** yangyapeng has quit IRC00:20
*** yangyapeng has joined #openstack-ironic00:20
*** Sukhdev_ has quit IRC00:20
*** yangyapeng has quit IRC00:24
*** slagle has quit IRC00:53
*** slagle has joined #openstack-ironic00:54
*** baoli has joined #openstack-ironic01:01
*** phuongnh has joined #openstack-ironic01:05
*** Goneri has quit IRC01:16
*** yangyapeng has joined #openstack-ironic01:19
*** ijw has quit IRC01:28
*** mat128 has joined #openstack-ironic01:35
*** phuongnh has quit IRC01:40
*** phuongnh has joined #openstack-ironic01:41
*** pmannidi has joined #openstack-ironic01:43
*** sasha2 has quit IRC01:44
*** sasha2 has joined #openstack-ironic01:51
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-pxe driver  https://review.openstack.org/48047701:57
*** diablo_rojo has joined #openstack-ironic01:58
*** phuongnh has quit IRC02:02
*** phuongnh has joined #openstack-ironic02:03
*** mat128 has quit IRC02:10
*** mat128 has joined #openstack-ironic02:11
*** http_GK1wmSU has joined #openstack-ironic02:12
*** http_GK1wmSU has left #openstack-ironic02:14
openstackgerritTao Li proposed openstack/ironic-specs master: Add service management API for ironic  https://review.openstack.org/47121702:14
openstackgerritMerged openstack/ironic master: Remove WARNING from pin_release_version's help  https://review.openstack.org/49027002:15
*** Sukhdev has quit IRC02:18
openstackgerritMerged openstack/ironic master: Remove SSH-based driver interfaces and drivers  https://review.openstack.org/48168802:20
*** mat128 has quit IRC02:21
*** rloo has quit IRC02:23
*** Nisha_Agarwal has joined #openstack-ironic02:27
*** rbudden has quit IRC02:30
*** diablo_rojo has quit IRC02:36
*** udesale has joined #openstack-ironic02:51
*** litao__ has joined #openstack-ironic02:58
openstackgerritRuby Loo proposed openstack/ironic master: SIGHUP restarts service with updated config values  https://review.openstack.org/47433103:00
*** sasha2 has quit IRC03:03
*** sasha2 has joined #openstack-ironic03:03
*** mat128 has joined #openstack-ironic03:10
*** sasha2 has quit IRC03:12
*** Nisha_Agarwal has quit IRC03:21
*** baoli has quit IRC03:23
*** mat128 has quit IRC03:25
*** mat128 has joined #openstack-ironic03:37
*** racedo_ has quit IRC03:40
*** mat128 has quit IRC03:43
*** ijw has joined #openstack-ironic03:44
*** ijw has quit IRC03:49
*** yangyapeng has quit IRC03:51
*** crushil has joined #openstack-ironic03:55
*** yangyapeng has joined #openstack-ironic03:58
*** ijw has joined #openstack-ironic04:00
*** Sukhdev has joined #openstack-ironic04:01
*** yangyapeng has quit IRC04:03
*** links has joined #openstack-ironic04:03
*** yangyapeng has joined #openstack-ironic04:07
*** ijw has quit IRC04:08
*** jinxing has joined #openstack-ironic04:09
*** vmud213 has joined #openstack-ironic04:09
*** hshiina has joined #openstack-ironic04:10
*** yangyapeng has quit IRC04:12
*** crushil has quit IRC04:12
*** crushil has joined #openstack-ironic04:13
*** yangyapeng has joined #openstack-ironic04:16
*** ihrachys_ has quit IRC04:18
*** yangyapeng has quit IRC04:21
*** jaianshu has joined #openstack-ironic04:21
*** crushil has quit IRC04:22
*** yangyapeng has joined #openstack-ironic04:22
*** yangyapeng has quit IRC04:27
*** Sukhdev has quit IRC04:30
*** yangyapeng has joined #openstack-ironic04:31
*** hshiina has quit IRC04:35
*** yangyapeng has quit IRC04:35
*** yangyapeng has joined #openstack-ironic04:40
*** tuanluong has joined #openstack-ironic04:41
*** yangyapeng has quit IRC04:41
*** yangyapeng has joined #openstack-ironic04:41
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Put tests in correct location for ironic/api/controllers/v1/  https://review.openstack.org/46265804:53
jlvillaldtantsur|afk, TheJulia vdrok vsaienk0 sambetts|afk: Patch https://review.openstack.org/#/c/490305/ merged, which is hoped to fix the Neutron / oslo.messaging issue we have been seeing.04:54
patchbotpatch 490305 - neutron - Sinkhole workaround for old topics (MERGED)04:54
*** Nisha_Agarwal has joined #openstack-ironic04:58
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add hardware type for SNMP powered systems  https://review.openstack.org/48504105:07
*** phuongnh has quit IRC05:18
*** phuongnh has joined #openstack-ironic05:19
*** livelace has joined #openstack-ironic05:28
*** mjura has joined #openstack-ironic05:40
*** marios has joined #openstack-ironic05:58
*** vishwana_ has quit IRC06:00
*** vishwanathj has joined #openstack-ironic06:00
*** oomichi has quit IRC06:09
*** oomichi has joined #openstack-ironic06:10
*** ijw has joined #openstack-ironic06:11
openstackgerritVasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase  https://review.openstack.org/48941806:11
*** ijw has quit IRC06:16
*** rcernin has joined #openstack-ironic06:30
*** ijw has joined #openstack-ironic06:31
openstackgerritBéla Vancsics proposed openstack/ironic master: Use more specific asserts in tests  https://review.openstack.org/43294106:42
*** ijw has quit IRC06:57
*** milan has joined #openstack-ironic07:00
*** tesseract has joined #openstack-ironic07:16
*** phuongnh has quit IRC07:29
*** phuongnh has joined #openstack-ironic07:29
*** ccamacho has joined #openstack-ironic07:34
*** pcaruana has joined #openstack-ironic07:38
milanmorning Ironic! :)07:39
*** http_GK1wmSU has joined #openstack-ironic07:42
*** http_GK1wmSU has left #openstack-ironic07:43
vdrokmorning ironic and milan07:44
milanmorning vdrok! :)07:44
vdrokjlvillal: good to know!07:44
etingofmorning everyone!07:45
milan#pixiesay -mflexing Good morning etingof!07:45
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: Good morning etingof!07:45
*** racedo has joined #openstack-ironic07:46
*** jtomasek has joined #openstack-ironic07:48
*** racedo has quit IRC07:48
*** ralonsoh has joined #openstack-ironic07:49
*** ijw has joined #openstack-ironic07:57
*** racedo has joined #openstack-ironic08:01
vdrokmorning etingof :)08:02
vdrokoh, TGIF08:02
*** ralonsoh_ has joined #openstack-ironic08:03
*** ralonsoh has quit IRC08:03
etingofgood Friday morning vdrok ;)08:03
*** ijw has quit IRC08:04
openstackgerritMerged openstack/python-ironicclient master: Skip warning when changing target_raid_config  https://review.openstack.org/48016108:09
*** aarefiev_afk is now known as aarefiev08:10
aarefievmorning all!08:10
*** phuongnh has quit IRC08:11
*** oomichi has quit IRC08:15
tuanluongMorning ironic08:15
*** oomichi has joined #openstack-ironic08:15
pas-hamorning ironic :)08:17
*** ChubYann has quit IRC08:17
*** mgoddard has joined #openstack-ironic08:20
openstackgerritMerged openstack/python-ironicclient master: Create port with specific port group UUID in OSC  https://review.openstack.org/41184008:24
*** early has quit IRC08:24
*** derekh has joined #openstack-ironic08:25
*** early has joined #openstack-ironic08:31
*** pmannidi has quit IRC08:32
openstackgerritHironori Shiina proposed openstack/ironic master: Add a guide for Devstack configuration for boot-from-volume  https://review.openstack.org/46656808:33
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add bindep.txt file  https://review.openstack.org/44420108:33
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-pxe driver  https://review.openstack.org/48047708:41
openstackgerritMateusz Kowalski proposed openstack/ironic-python-agent master: BMC can be configured using different lan channel  https://review.openstack.org/48067908:49
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-pxe driver  https://review.openstack.org/48047708:49
openstackgerritMateusz Kowalski proposed openstack/ironic-python-agent master: BMC can be configured using different lan channel  https://review.openstack.org/48067908:50
*** amarao has joined #openstack-ironic08:53
*** lucas-afk is now known as lucasagomes08:55
makowalsHello everyone08:58
*** amarao has quit IRC08:59
makowalsI have noticed when doing 'openstack baremetal node show' I don't see anything about raid config; I can see this and target_raid_config only by going directly into ironic database. Is it related to the version of my client or is it really the behaviour for now ?08:59
milanmorning aarefiev makowals pas-ha tuanluong! :)09:00
milanvdrok, TGIF! :D09:00
aarefievmorning milan :)09:00
*** amarao has joined #openstack-ironic09:01
*** ijw has joined #openstack-ironic09:01
*** ijw has quit IRC09:06
openstackgerritShangXiao proposed openstack/ironic master: Update and replace http with https for doc links in ironic  https://review.openstack.org/49080609:21
vdrokmakowals: you need to specify a more recent than default api version09:25
vdroklike openstack --os-baremetal-api-version latest baremetal node show node-009:26
vdrokthe reason being, I think openstack client only renders what is being returned in the response to the http request. otoh ironic command has a hardcoded list of fields09:27
vmud213Morning Ironicers09:27
vdrokso it will show target_raid_config field in the table no matter what api version you've used. but if the version is lower than the one ttarget_raid_config was introduced, this row will be empty09:28
vdrokmorning tuanluong vmud21309:28
tuanluongmorning vdrok vmud21309:29
vmud213vdrok,tuanluong o/09:30
*** etingof has quit IRC09:36
*** aparnav has joined #openstack-ironic09:37
*** mikal has quit IRC09:39
*** mikal has joined #openstack-ironic09:41
*** zhangfei has joined #openstack-ironic09:44
*** sambetts|afk is now known as sambetts09:46
sambettsThanks jlvillal for tracking that neutron patch!09:46
*** mgoddard has quit IRC09:46
*** etingof has joined #openstack-ironic09:54
*** dtantsur|afk is now known as dtantsur09:55
dtantsurmorning ironic, happy Friday09:55
dtantsurjlvillal: \o/09:55
*** stendulker has joined #openstack-ironic09:55
*** hw_wutianwei has quit IRC09:56
*** amarao1 has joined #openstack-ironic09:58
*** amarao1 has quit IRC09:59
*** amarao1 has joined #openstack-ironic10:04
*** tuanluong has quit IRC10:08
*** openstackgerrit has quit IRC10:18
*** mgoddard has joined #openstack-ironic10:19
*** jinxing has quit IRC10:21
vdrokmorning sambetts dtantsur10:25
sambettso/ dtantsur, vdrok10:25
dtantsuro/10:25
*** jinxing has joined #openstack-ironic10:29
*** openstackgerrit has joined #openstack-ironic10:37
openstackgerritDmitry Tantsur proposed openstack/ironic master: [tempest] also catch BadRequest in negative tests with physical_network in old API  https://review.openstack.org/49082610:37
dtantsuranother attempt to fix stable/ocata ^^^10:37
dtantsurjlvillal: ^^^10:37
*** jinxing has quit IRC10:44
*** Nisha_Agarwal has quit IRC10:45
*** tellesnobrega has quit IRC10:46
*** udesale has quit IRC10:48
sambettsdtantsur: I think you can probably remove the lib_exc.UnexpectedResponseCode from that call, I think that was just wrong to begin with, it should have always been a bad request10:50
sambettsalthough maybe its just because we aren't catching bad request add turning it into the lib_exc.UnexpectedResponseCode thing?10:51
dtantsursambetts: HTTP 406 is unexpected from tempest point of view.. I didn't bother to figure out why, but I can try10:53
*** jtomasek has quit IRC10:54
sambettsdtantsur: we shouldn't be getting a 406 at any point right? the response should always be BadRequest because otherwise we've broken the API10:54
dtantsursambetts: this is how microversioning works, yes :) one of the reasons I don't like the *implementation*10:54
dtantsurwhen we hide features we use HTTP 406, except for when we use HTTP 40410:54
sambettsits was only a 406 before because we requested a version number that Ironic doesn't support right?10:55
dtantsurone may argue it's good, because users get a better hint on what they're doing wrong10:55
dtantsursambetts: I don't think so, lemme double check10:55
*** e0ne has joined #openstack-ironic10:55
openstackgerritSam Betts proposed openstack/ironic master: Add hardware types to support Cisco UCS Servers  https://review.openstack.org/48610010:56
*** mgoddard has quit IRC10:56
dtantsursambetts: see all these NotAcceptable? https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/utils.py#L276-L35210:56
dtantsurwe've been doing it since long ago :(10:57
* sambetts voms... those should definatly be InvalidParameterValue errors... 10:58
dtantsur#pixiesay -m sad Microversioning...11:00
PixieBootsʕ•͡ᴥ•ʔ: Microversioning...11:00
*** Nisha_Agarwal has joined #openstack-ironic11:01
sambettsits not microversionings fault... its a massive derp on our part for not making the API return the right response for old API versions...11:01
*** jkilpatr has joined #openstack-ironic11:02
sambettsits just inconsistent with the case when you ask for a field that really doesn't exist11:02
sambettsthis is the right request response http://logs.openstack.org/05/484405/1/check/gate-tempest-dsvm-ironic-pxe_ipmitool-postgres-ubuntu-xenial-nv/a8d630c/console.html#_2017-08-03_14_45_22_14971311:03
aparnavhey dtantsur, We had a internal discussion about your concern on the patch https://review.openstack.org/#/c/422572/. There is a comment from wan-yen on the same patch. Just wondering does it address your concern? could you have a look and let me know your thoughts?11:04
patchbotpatch 422572 - ironic - Support hpsum based `update_firmware` for iLO drivers11:04
*** mrtenio has quit IRC11:04
dtantsuraparnav: it does not say anything new to address my concerns, sorry. if you want to keep the name, please make sure that both methods at least accept the same arguments (incl. accepting a list of hpsum)11:05
*** mgoddard has joined #openstack-ironic11:12
*** athomas has quit IRC11:13
*** athomas has joined #openstack-ironic11:14
aparnavdtantsur, anyway I have thought over your comment. Will update the rfe and code patch accordingly. thanks.11:17
*** Nisha_Agarwal has quit IRC11:18
*** Nisha_Agarwal has joined #openstack-ironic11:19
*** jaianshu has quit IRC11:21
*** Nisha_Agarwal has quit IRC11:23
*** Nisha_Agarwal has joined #openstack-ironic11:24
TheJuliagood morning everyone11:27
*** yangyapeng has quit IRC11:30
dtantsurmorning TheJulia11:30
*** lucasagomes is now known as lucas-hungry11:31
*** http_GK1wmSU has joined #openstack-ironic11:37
*** http_GK1wmSU has left #openstack-ironic11:38
* TheJulia feels like it is an appropriate day of some metal or industrial11:38
dtantsur+++++11:39
*** stendulker has quit IRC11:46
*** tellesnobrega has joined #openstack-ironic11:50
*** Nisha_Agarwal has quit IRC11:50
*** MrChao has joined #openstack-ironic11:51
*** vmud213 has quit IRC11:54
*** rbudden has joined #openstack-ironic11:58
*** litao__ has quit IRC11:58
*** dprince has joined #openstack-ironic12:00
*** jlabarre has joined #openstack-ironic12:06
dtantsuranyone (*quick look at TheJulia*) up for reviewing the docs to upgrade to hw types? https://review.openstack.org/#/c/486696/12:21
patchbotpatch 486696 - ironic - Guide on upgrading to hardware types12:21
dtantsuralso TheJulia lemme know if you want me to approve https://review.openstack.org/#/c/482734/ and then follow-up with everything there12:23
patchbotpatch 482734 - bifrost - Begin breaking out installation documentation12:23
*** dprince has quit IRC12:24
*** edleafe is now known as figleaf12:26
*** weshay_pto is now known as weshay12:26
openstackgerritDao Cong Tien proposed openstack/ironic master: Add bookmarks for sections of enabling-drivers.rst  https://review.openstack.org/49084112:29
openstackgerritDao Cong Tien proposed openstack/ironic master: Add bookmarks for sections of enabling-drivers.rst  https://review.openstack.org/49084112:29
*** xavierr has joined #openstack-ironic12:31
xavierrgood morning ironic12:31
xavierrdo we have a date for our cut and release?12:32
*** jtomasek has joined #openstack-ironic12:33
dtantsurxavierr: morning. the hard deadline is Aug 24th. I would be good to get everything major in at least a week before.12:34
*** dprince has joined #openstack-ironic12:37
*** Goneri has joined #openstack-ironic12:43
*** baoli has joined #openstack-ironic12:43
xavierrdtantsur: hey, understood. we are working on the migration of removing python-oneviewclient and use python-hpOneView instead https://review.openstack.org/#/q/topic:bug/1693788.12:45
openstackgerritSam Betts proposed openstack/ironic master: Add hardware types to support Cisco UCS Servers  https://review.openstack.org/48610012:45
TheJuliadtantsur: shortly, very shortly.  Just had to put some thoughts down in writing12:45
xavierrNot sure we have enough time to merge all patches but we need reviews on https://review.openstack.org/#/c/468129/ and https://review.openstack.org/#/c/466872/ to start to make our cycle nice :)12:46
patchbotpatch 468129 - ironic - Introduce hpOneView and ilorest to OneView12:46
patchbotpatch 466872 - ironic - Migrate oneview driver to use python-hpOneView12:46
dtantsurxavierr: yeah, I'm aware of that. do you think it's worth changing the dependency so late in the cycle?12:46
dtantsurdoes it overweight the risks of regressions?12:46
xavierrour, I mean, oneview project team :)12:46
xavierrI think so, dtantsur. it is a important step for oneview drivers internal demands. if we don't get all patches merged we could keep both clients on the driver an work in backports on python-oneviewclient if needed12:49
*** jkilpatr has quit IRC12:49
*** yangyapeng has joined #openstack-ironic12:49
dtantsurxavierr: what is the problem you're trying to solve with the switch at this point?12:49
xavierrdtantsur: all our efforts are concentrated on those patches :)12:49
* dtantsur quick tea, brb12:50
*** lucas-hungry is now known as lucasagomes12:50
xavierrdtantsur: API compatibility between drivers and new oneview versions, mainly12:50
openstackgerritDao Cong Tien proposed openstack/ironic master: Documentation for irmc hardware type  https://review.openstack.org/49048312:53
*** yangyapeng has quit IRC12:53
*** jaypipes is now known as leakypipes12:54
*** gcb has quit IRC12:56
openstackgerritMerged openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes  https://review.openstack.org/47932612:57
*** sasha2 has joined #openstack-ironic12:58
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882513:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617113:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617013:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772813:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617313:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617213:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Introduce keystoneauth adapters for clients  https://review.openstack.org/46962713:00
TheJuliadtantsur: reviewed hw type upgrade doc. I left some feedback/thoughts that might be worth considering, including possibly adding a warning for those with custom drivers that they have assembled.13:00
openstackgerritMerged openstack/ironic master: Use more specific asserts in tests  https://review.openstack.org/43294113:00
*** aparnav has quit IRC13:00
TheJuliadtantsur: re approval on the bifrost docs, *shrug* I'm hoping to get back to that today13:01
dtantsurokay, thanks13:03
dtantsurxavierr: ah, so the new libraries support more hardware?13:03
*** jkilpatr has joined #openstack-ironic13:05
*** aarefiev is now known as aarefiev_afk13:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] use resource classes by default  https://review.openstack.org/47696813:10
*** Goneri has quit IRC13:11
xavierrdtantsur: HPE folks are planning the new API version, but probably yes. And if so, to support that new hardware on oneview drivers we need they new client. that client is updated when the API change. so, adopting they cleint makes us free of evolving drivers and our current python-oneviewclient.13:12
leifmadsenTheJulia: ping re: bifrost13:12
TheJuliao/ leifmadsen13:12
dtantsurxavierr: ack, understood13:12
dtantsurthanks for clarification13:12
leifmadsenI've been working on trying to make it easier to deploy, and I've come up with some non-virtualenv notes that seems to work, and I'm wondering if it makes sense to add a bootstrap.yaml to the bifrost code so that it could properly install the deps, etc13:12
xavierrty dtantsur :)13:12
leifmadsenI would think that would then lead into a nicer deployment experience, and better quickstart documentation13:12
TheJulialeifmadsen: I'm missing context, bootstrap.yaml?13:13
leifmadsenI was going to build my own bifrost-ansible role that wrapped the install.yaml bifrost repo, but it seems like maybe it would just be better to add the playbook to the bifrost repo proper. What do you think?13:13
leifmadsenTheJulia: here, let me link you13:13
TheJulialeifmadsen: feel free to propose whatever makes life easier :)13:13
leifmadsenTheJulia: so before I can install bifrost on a new server (VM), I have to run these commands: https://paste.fedoraproject.org/paste/wpbP3W7ipRz8K-6j6NF3nQ13:14
leifmadsenso I'm thinking, adding a bootstrap.yaml that accomplishes that from a brand new VM would be useful13:14
leifmadsenI was originally doing it with virtualenv and a wrapper playbook/role, but it was a serious pain in the ass working with virtualenv13:14
*** dougbtv_laptop has joined #openstack-ironic13:15
leifmadsenyolanda: ^^^ (thoughts?)13:16
TheJulialeifmadsen: Ahh, part of that should be covered by existing scripting/processes, just depends on how you enter into it I guess. We tend to expect most users to use the setup-env script, but making the experience better for those that do not is also a good idea13:16
leifmadsenI mean, worst case, I write it up, propose it, everyone hates it, and I take my ball home :)13:16
*** yangyapeng has joined #openstack-ironic13:16
TheJulialeifmadsen: heh13:16
leifmadsenTheJulia: yea, honestly, it was a lot easier to just run those few commands, and that would be a very simple playbook to add13:16
TheJulialeifmadsen: I'm always for making people's lives easier13:17
leifmadsenI think it would help with documentation too (because I would circle around and propose a set of docs to go along with it)13:17
leifmadsenanyways, if you don't think it's a dumb idea, I'll work on it today13:17
leifmadsenshoudl have something in a couple hours to review13:17
*** jcoufal has joined #openstack-ironic13:17
TheJuliaso 11-14 should be in the install process, it might just be your version of fedora13:18
yolandaah leifmadsen yep, those bits are not normally covered by the projects, just left for users to install properly13:18
leifmadsenI'll go through and just add where it makes sense, but I think you should be able to bootstrap and deploy from a single command ideally13:20
leifmadsenthat's basically my goal, I shouldn't have to worry about all this extra stuff manually13:20
leifmadsenalso yea, I kind of assumed the DB might get setup as well, but when I put in the password for root, it basically skips all that part, but it's probably not necessary13:21
leifmadsenI'll see where I get today, but it'll be useful to reproduce this easily locally so that I can rebuild the baremetal management13:21
leifmadsenthanks for listening :)13:22
TheJuliaI need to fix https://review.openstack.org/#/c/471750/5  :(13:23
patchbotpatch 471750 - bifrost - Fedora 25 support13:23
leifmadsenI'm deploying on CentOS 7 so haven't run into any Fedora issues :)13:24
TheJuliaahh13:24
leifmadsenalthough I see in the comments, this might address my setuptools issue13:24
* dtantsur looks at "sudo pip install" in horror13:24
leifmadsendtantsur: yea, I need to run through again and make it non-root13:25
leifmadsenand if I wanted to be eager I should actually run through SElinux debugging so that it could actually work with it on13:25
TheJuliadtantsur: if you treat every day like a horror movie... ;)13:25
*** Goneri has joined #openstack-ironic13:26
dtantsurheh. anything with 'pip' in it is a strong "nope" in the world I exist in :)13:26
dtantsurditto for epel, essentially13:26
* TheJulia should consider centos whens he rebuilds her lab machine in the basement13:26
*** aparnav has joined #openstack-ironic13:26
TheJuliashe13:27
dtantsurdamn, 'tox -edocs' still does not work.. what's so wrong with me??13:27
* TheJulia glares at the spacebar13:27
openstackgerritDmitry Tantsur proposed openstack/ironic master: Guide on upgrading to hardware types  https://review.openstack.org/48669613:28
dtantsurI'll have to check ^^ in the gate13:28
TheJuliadtantsur: nuke your .tox and friends folders?13:28
dtantsurTheJulia: tried everything13:28
TheJuliadtantsur: don't worry, I have 7 unit tests that consistently fail when I run locally13:28
dtantsur\o/13:28
TheJuliaAnything to do with hash comparison13:28
*** cdearborn has joined #openstack-ironic13:28
TheJuliaOthers have seen it, but it is totally a python thing13:29
dtantsurI think milan spent substantial time figuring out race condition in ironic-inspector tests that only happens on his mac :)13:29
milanlol13:29
TheJuliaon his mac in OS X? or in a linux vm in osx?13:29
milanTheJulia, in OS X13:29
* TheJulia just shakes her head13:29
* dtantsur makes a grumpy cat face at the refarch patch13:30
milandtantsur, which one?13:30
dtantsurmilan: https://review.openstack.org/#/c/487410/ 43 comments, w00t!! :D13:31
patchbotpatch 487410 - ironic - Reference architecture: common bits13:31
TheJuliawow13:31
milanomg!13:31
* TheJulia is not sure she wants to look right now13:31
dtantsurjust wanted to ask :)13:31
TheJuliathen again we very clearly have had disagreements regarding what a "reference" install should look like.13:31
TheJuliaand those differences come down to implementation details at the hardware level, and maybe that should be left out........13:32
dtantsurright. we may not come to a clear agreement, but some guidence is better than none IMO13:32
dtantsurmaybe we should drop "reference" and call it a different way..13:32
milanTheJulia, my case was the tests assumed execution order instead of mocking i.e relying on something.init() assert something.run() assert something.shutdown()13:32
dtantsurmgoddard wanted to take a look (or rather I dragged him into wanting to take a look) too13:32
TheJuliamaybe restrict it to the controller side and explicitly state that we're not covering hardware side as there is really no one true way that people will all agree upon due to varying requirements13:32
*** racedo has quit IRC13:32
*** racedo has joined #openstack-ironic13:33
dtantsurTheJulia: yeah, I'm not talking about hardware too much, more of "make sure you can find a working driver before you buy stuff and start deploying"13:33
TheJuliadtantsur: totally inappropriate, but perhaps embedding http://i.imgur.com/vdLE8dJ.gif as the diagram ;)13:34
milan...took me some time to track this13:34
dtantsurTheJulia: yes, yes, YESSS (this is officially my most favourite gif on the internet fwiw)13:34
TheJuliadtantsur: good deal, I'll try to take a look monday morning when I have a fresh brain13:35
dtantsurgood, then I'll update the current comments today13:36
TheJuliamilan: ugh, yeah, execution order stuff is a pain :(13:36
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Put tests in correct location for ironic/api/controllers/v1/  https://review.openstack.org/46265813:37
openstackgerritDmitry Tantsur proposed openstack/ironic master: Guide on upgrading to hardware types  https://review.openstack.org/48669613:39
*** jkilpatr_ has joined #openstack-ironic13:41
*** mat128 has joined #openstack-ironic13:41
*** jkilpatr has quit IRC13:42
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall  https://review.openstack.org/47183113:43
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Allow periodics to terminate inspector  https://review.openstack.org/48790913:43
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644813:43
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871213:43
milanTheJulia, yup, also @mock.patch.objec() sometimes doesn't work properly as it's 'compile-time' while some tests better use fixture to perform a 'runtime' mock in the setUp() method13:45
milannot to mention that some tests mock things in wrong places :-/13:45
milantechnical debt anyone? ;)13:45
*** jistr is now known as jistr|mtg13:46
*** dansmith is now known as superdan13:47
*** Dinesh_Bhor has quit IRC13:47
*** baoli has quit IRC13:49
*** crushil has joined #openstack-ironic13:53
*** mat128 has quit IRC13:55
pas-hafwiw, we might soon get another way to provide 'reference deployment' info - as Ansible playbooks for Zuul V3 in our repo :)13:58
*** baoli has joined #openstack-ironic14:01
openstackgerritMateusz Kowalski proposed openstack/ironic-python-agent master: BMC can be configured using different lan channel  https://review.openstack.org/48067914:03
*** marios has quit IRC14:03
dtantsurpas-ha: I'd still have a human-readable text for those not using ansible for some reason :)14:03
*** marios has joined #openstack-ironic14:04
dtantsuralso the ansible playbooks won't tell *why* we recommend it14:04
pas-hasure, was just a reminder :)14:04
*** racedo has quit IRC14:05
*** mjura has quit IRC14:05
*** bnemec is now known as beekneemech14:07
* dtantsur is rarely against replacing things with ansible14:11
*** links has quit IRC14:12
*** Nisha_Agarwal has joined #openstack-ironic14:13
jlvillal makowals Are you the author of https://review.openstack.org/#/c/480679/ ?14:13
patchbotpatch 480679 - ironic-python-agent - BMC can be configured using different lan channel14:13
makowalsjlvillal: Yep, that's me14:13
jlvillalmakowals, Thanks. I think the unit test might need to be updated. Not sure. I'm guessing it will fail now.14:14
jlvillalmakowals, Besides that it looks good to me.14:14
jlvillalmakowals, Not sure if you ran the unit tests on patch set 10 or not.14:14
makowalsYes, they just failed, I see why, stupid mistake14:14
jlvillalMy brain and the computer often disagree on correct code ;)14:14
jlvillalmakowals, Okay. I'll wait for update. But thanks for the comments. Made it much clearer to me!14:15
makowalsNo problem, that's the goal of this whole process14:15
*** mjturek has joined #openstack-ironic14:17
openstackgerritAparna proposed openstack/ironic master: Support hpsum based firmware update as clean step for iLO drivers  https://review.openstack.org/42257214:22
*** mjturek has quit IRC14:22
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add negative tests for VIF attach/detach operations  https://review.openstack.org/43595214:27
*** tellesnobrega has quit IRC14:36
*** mjturek has joined #openstack-ironic14:36
openstackgerritDmitry Tantsur proposed openstack/ironic master: Reference architecture: common bits  https://review.openstack.org/48741014:39
dtantsurthat was difficult ^^^14:40
openstackgerritMateusz Kowalski proposed openstack/ironic-python-agent master: BMC can be configured using different lan channel  https://review.openstack.org/48067914:41
NobodyCamGood Morning Ironic'ers ... and ....14:41
NobodyCamTGIF!14:41
mguzhey ironic folks. I'm seeing an issue with ironic on mitaka. I know we're out of date, however the problem we're seeing is ironic-conductor will periodically suck up all the ram on it's node14:41
mguzand stop responding, plus all the shellinabox daemons will die14:41
mguzoccasionally the it gets so stuck ironic sets all the baremetal systems to maintenance mode14:42
mguzwe only have 60 baremetal nodes and the ironic-conductor node has 64GB ram and 32 Cores14:43
dtantsurmorning NobodyCam, omg TGIF (and it finally starts being colder here)14:43
NobodyCam:) Good morning dtantsur :)14:43
mguzwe are working on plans to upgrade to newton, but it's a relatively big undertaking and wont happen for a month or two yet14:44
*** MrChao has quit IRC14:44
mguzi'm desperately looking for a temporary fix as we are using ironic quite extensively in our labs here14:45
mguzanyone got any suggestions other than upgrade?14:45
dtantsurmguz: hi! I don't think we can help much other than ask you to profile the service and see what takes RAM14:46
dtantsurMitaka was... long ago :)14:46
* mgoddard takes a look at dtantsur's reference architecture review14:47
dtantsurmgoddard: cool! please make sure to fetch the latest version, updated a few minutes ago14:47
NobodyCamhey hey dtantsur PTL type question: what are your thought on adding a SystemHealth property to the node object?14:47
dtantsurNobodyCam: I don't have inherent problems with having new properties, it all boils down to how it's going to be used and/or populated :)14:48
mguzdtantsur: well mitaka was only last year! and you guys move so fast, we barely get something up and running before it's out of date14:48
mguzhow do i profile the service?14:48
NobodyCamdtantsur: ack I'll put up a spec in the coming weeks as I get the time!14:49
NobodyCammguz: I know!14:49
dtantsurmguz: this is a common complaint, I hear your pain14:49
dtantsurNobodyCam: start with an RFE, I think. Just to get your idea.14:49
mguzdtantsur: i think the openstack community really needs to think about alternating LTS relases with bleeding edge releases14:50
NobodyCamdtantsur: ++14:50
dtantsurmguz: the problem will be upgrades between LTS version14:50
mguzdtantsur: indeed14:50
dtantsurmguz: we don't support N->N+2 upgrades now, it's going to be quite an effort to start doing it14:50
dtantsurotherwise I don't disagree with that at all14:50
dtantsuranother alternative is to have more rare releases14:51
mguzyeah. which would give a chance for more stability14:51
NobodyCamlol I just had to upgrade a client in prod.. went from 0.3.1 to 0.8.1 :p14:51
dtantsurnice14:52
NobodyCam:p14:52
dtantsurmguz: as you can imagine, that's going to be a huge openstack-wide decision.. but you're not the first to raise this issue, if it makes you feel better14:52
dtantsurderekh: what did you use to tackle that memory leak in the driver code?14:53
mguzdtantsur: hehe yeah. anyway, how do I profile the service? is it just strace or is there some other trick ?14:55
*** baoli has quit IRC14:55
dtantsurmguz: I'm trying to this of something, but nothing comes to my mind. this is why I ask derekh above what he used recently14:57
derekhdtantsur: luckily I was chasing a increase in cpu usage (and time to process a particular piece of code),14:57
openstackgerritSam Betts proposed openstack/ironic master: Add detail=True query string to API list endpoints  https://review.openstack.org/43149714:57
derekhdtantsur: so I inserted lots of timers in certain parts of the code, and looked for one increased over time14:57
derekhdtantsur: then narrowed it down until I got a result14:57
derekhdtantsur: it was a very manual and involved process14:58
dtantsurmguz: something like that may help. if you see CPU increasing too (which is not unlikely). standard cProfile module can help as well14:59
NobodyCamderekh: is there a patch you could share with the fix?15:00
*** jistr|mtg is now known as jistr15:01
*** etingof has quit IRC15:01
derekhNobodyCam: this was the fix https://review.openstack.org/#/c/449577/  is that what you wanted??15:03
patchbotpatch 449577 - ironic - Copy and append to static lists (MERGED)15:03
NobodyCamderekh: awesome Thank you... :)15:04
*** anupn has quit IRC15:07
*** aNuposic has joined #openstack-ironic15:07
*** jtomasek has quit IRC15:07
*** jtomasek has joined #openstack-ironic15:10
*** jtomasek has quit IRC15:12
*** mjturek has quit IRC15:12
*** mjturek has joined #openstack-ironic15:13
*** dprince has quit IRC15:13
*** rcernin has quit IRC15:14
*** mjturek1 has joined #openstack-ironic15:16
*** etingof has joined #openstack-ironic15:17
*** zhangfei has quit IRC15:17
mguzdtantsur: thanks! i'll give it a try15:17
*** MattMan has quit IRC15:18
*** MattMan has joined #openstack-ironic15:19
*** rpioso has joined #openstack-ironic15:20
*** mjturek1 has quit IRC15:20
*** mjturek has quit IRC15:21
*** jtomasek has joined #openstack-ironic15:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882515:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617115:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617015:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772815:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617315:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617215:22
*** jaosorior has quit IRC15:23
*** jtomasek has quit IRC15:26
*** jtomasek has joined #openstack-ironic15:28
*** crushil has quit IRC15:32
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644815:32
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871215:32
*** jtomasek has quit IRC15:32
milan#pixiesay -mrnr TGIF NobodyCam!15:42
PixieBoots\m/ʕ>ᴥ<ʔ\m/: TGIF NobodyCam!15:42
milanmorning NobodyCam! :P15:42
NobodyCam:) hehehe good morning milan15:42
*** etingof has quit IRC15:43
dtantsurjlvillal: this will hopefully fix stable/ocata (again!) https://review.openstack.org/#/c/490826/ mind taking a look?15:43
patchbotpatch 490826 - ironic - [tempest] also catch BadRequest in negative tests ...15:43
milanfinally PixieBoots is back! thanks dtantsur and lucasagomes ;)15:43
dtantsurmilan: now in a container! :D15:43
NobodyCam:) ++++15:43
lucasagomesmilan, haha o/15:44
dtantsurhttps://hub.docker.com/r/dtantsur/pixiebot/15:44
lucasagomesit's been run for a while actually :D15:44
milan#pixiesay -mlikeaboss dtantsur!15:44
PixieBootsʕ▀̿ᴥ▀̿ʔ: dtantsur!15:44
lucasagomes#pixiesay -m yay niceeeeee15:44
PixieBootsʕ•͡ᴥ•ʔ: niceeeeee15:44
lucasagomesthere's no yay :-(15:44
* lucasagomes can't remember his own code15:44
NobodyCam:O15:44
dtantsurhehe15:44
milanwhere to post pull requests? :D15:44
dtantsurwe need a help command for him15:44
dtantsurPixieBoots: help!15:45
dtantsuranyway, wine'o'clock. have a great weekend everyone!15:45
NobodyCamhave a great weekend dtantsur :)15:45
*** dtantsur is now known as dtantsur|afk15:46
milannight dtantsur|afk !15:46
*** [1]rpioso has joined #openstack-ironic15:48
*** rpioso has quit IRC15:48
*** [1]rpioso is now known as rpioso15:49
*** e0ne has quit IRC15:50
aNuposicGood Morning Ironic'ers :)15:51
*** aNuposic is now known as anupn15:51
NobodyCamgood morning anupn15:51
lucasagomesNobodyCam, (-: hi there15:52
NobodyCam:)15:52
lucasagomesdtantsur|afk, see ya15:52
anupnNobodyCam, morning :)15:52
NobodyCamhey hey lucasagomes :)15:54
NobodyCamnice to pixiebot back in action :)15:54
lucasagomesyeah15:54
lucasagomes#findspec volume15:54
PixieBoots2 spec(s) found: "boot from volume reference drivers": http://specs.openstack.org/openstack/ironic-specs/specs/approved/boot-from-volume-reference-drivers.html  | "volume connection information": http://specs.openstack.org/openstack/ironic-specs/specs/approved/volume-connection-information.html15:54
lucasagomeso/ :D15:54
NobodyCamnice :)15:54
*** etingof has joined #openstack-ironic15:55
*** aparnav has quit IRC15:56
*** krtaylor has joined #openstack-ironic16:00
*** racedo has joined #openstack-ironic16:01
openstackgerritPeter Piela proposed openstack/ironic-ui master: Refactor graph class into a separate service  https://review.openstack.org/49095916:03
*** marios has quit IRC16:07
*** amarao has quit IRC16:07
*** amarao1 has quit IRC16:08
vdrokhey lucasagomes anupn NobodyCam :)16:09
*** mrtenio has joined #openstack-ironic16:09
lucasagomesvdrok, hey there :D16:10
*** pcaruana has quit IRC16:10
anupnhi vdrok, :)16:12
NobodyCamGood Morning vdrok :)16:13
*** Nisha_Agarwal has quit IRC16:13
TheJuliaWell, I think i finally figured out a long annoying dib bug16:15
openstackgerritJulia Kreger proposed openstack/bifrost master: DNM: Test CI with DIB fixes  https://review.openstack.org/49096316:17
* milan calls it a day16:18
NobodyCamhave a good weekend milan16:18
* jlvillal is hopeful that he can retire his 'recheck bot' :)16:18
milanNobodyCam, same to you! Thanks! :)16:18
milannight Ironic!16:18
*** milan has quit IRC16:18
jlvillalciao milan16:18
TheJuliaNobodyCam: can I get a review on https://review.openstack.org/#/c/490056/3 ?16:21
patchbotpatch 490056 - bifrost - Ignore DIB failures that generate output files16:21
vdrokgood morning jlvillal , yeah, ci seems pretty stable today16:21
jlvillalvdrok: Hi, and woo hoo :)16:22
vdrokand it's time to leave for /me16:22
jlvillalvdrok: Paka!16:22
vdrokgood night everyone!16:22
vdrokjlvillal: :)16:22
*** derekh has quit IRC16:26
*** lucasagomes is now known as lucas-afk16:28
NobodyCamTheJulia: very minor but would would you think about splitting the failed check in to separate image and ramdisk checks?16:30
TheJuliahmm.16:31
TheJuliathe conundrum is one can request an image, and still get a ramdisk due to the elements being requested16:31
TheJuliahence why I was thinking generic16:31
*** tellesnobrega has joined #openstack-ironic16:32
NobodyCammy thought was with that error one would have to dig thru the logs to find what actually failed to build16:33
NobodyCamand dib outputs can be long.. :p16:34
*** srart has joined #openstack-ironic16:34
openstackgerritMerged openstack/ironic master: Prevent tests from using utils.execute()  https://review.openstack.org/46519716:36
*** baoli has joined #openstack-ironic16:42
*** dprince has joined #openstack-ironic16:43
TheJuliaYeah....16:47
TheJuliait is like the most headachy part to troubleshoot16:47
JayFdtantsur|afk: feel free to ping me here re: removing -2s and such16:48
NobodyCamwould you like a comment on the review?16:49
TheJuliaNobodyCam: sure, I'm just not sure the best path to make that happen since the logic kind of explodes then16:49
TheJuliabecause it is a "detect if ramdisk was actual expected result" conundrum then as well16:50
*** crushil has joined #openstack-ironic16:54
leifmadsenTheJulia: you going to work on https://review.openstack.org/#/c/471750 anytime today, or could I submit just the setuptools changes? That part blocks some of my automation work"16:54
patchbotpatch 471750 - bifrost - Fedora 25 support16:54
NobodyCamI added comment, not sure it would involve that much. Just split the test in to two parts one checking *_image_* vars and one checking *[_ramdisk_|_initramfs_]* vars16:55
NobodyCam-1 may have been a little harsh.. I am happy to s/-/+/16:55
*** amarao has joined #openstack-ironic16:56
*** ElCoyote_ has joined #openstack-ironic16:58
*** ralonsoh_ has quit IRC17:00
*** mgoddard has quit IRC17:01
TheJuliaNobodyCam: so how does that work if I have ironic-agent on the element command line?17:02
TheJulialeifmadsen: dunno, kind of leaning towards I might not get to that today, I need to focus on docs17:04
leifmadsennp17:04
leifmadsendocs++17:04
*** amarao has quit IRC17:05
NobodyCamhttps://www.irccloud.com/pastebin/jgDj98wW/17:09
* sambetts --> home 17:11
sambettsnight all o/17:11
*** sambetts is now known as sambetts|afk17:11
NobodyCamhave a good weekend sambetts17:11
sambetts|afkcya NobodyCam, TGIF17:12
NobodyCam:)17:12
NobodyCamTheJulia: its not much but could help narrow down the search when trouble shooting... unless I've missed something17:13
TheJuliaNobodyCam:  I see, and I'm thinking of the case of disk-image-create making a ramdisk for ironic-agent plus if you define the extension, it gets stripped out if memory serves :(17:16
NobodyCambut would that cause the current check to also fail?17:17
TheJuliano, because it considers the possible variations17:22
TheJuliait just doesn't have a customized output message based upon perceived output type17:22
NobodyCamI changed my vote17:24
NobodyCamwill look more closely later today...17:25
*** openstackstatus has quit IRC17:27
*** openstack has joined #openstack-ironic17:29
NobodyCam:)17:29
*** priteau has joined #openstack-ironic17:34
*** racedo has quit IRC17:41
*** e0ne has joined #openstack-ironic17:46
*** mrtenio has quit IRC17:48
*** athomas has quit IRC18:06
*** e0ne has quit IRC18:09
*** mrtenio has joined #openstack-ironic18:11
*** makowals has quit IRC18:12
mrtenioHi Ironic'ers!18:16
mrtenioTheJulia, question, in the patch https://review.openstack.org/#/c/483430. It is important to note that one doesn't need to put a node in manageable to add the flag, does that mean we should remove the step for changing the node to manageable? Thank you18:16
patchbotpatch 483430 - ironic - Add a flag to always perform persistent boot on PX...18:16
*** sambetts|afk has quit IRC18:19
*** slagle has quit IRC18:19
TheJuliamrtenio: in my mind, that it shouldn't need to be in manageable, or that the user might need to be aware that they will incure cleaning again moving back to manageable18:20
*** sambetts_ has joined #openstack-ironic18:21
mrtenioTheJulia, if we remove the first step and state that after setting the flag, the next cleaning/deploy will be persistent, would that be more accurate? wdyt?18:25
*** openstackstatus has quit IRC18:26
*** openstack has joined #openstack-ironic18:28
mrteniounhum, 'will send using that approach then. tnx.18:32
*** slagle has joined #openstack-ironic18:51
*** rbudden has quit IRC18:52
*** cdearborn_ has joined #openstack-ironic18:58
*** sambetts_ has quit IRC19:06
*** cdearborn__ has joined #openstack-ironic19:07
*** sambetts_ has joined #openstack-ironic19:08
*** livelace has quit IRC19:11
openstackgerritJulia Kreger proposed openstack/bifrost master: DNM: Test CI with DIB fixes  https://review.openstack.org/49096319:13
*** cdearborn has quit IRC19:14
openstackgerritXavier proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView  https://review.openstack.org/46812919:14
openstackgerritXavier proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687219:14
*** marlinc has quit IRC19:17
*** amarao has joined #openstack-ironic19:17
*** cdearborn_ has quit IRC19:18
*** cdearborn has joined #openstack-ironic19:18
openstackgerritMerged openstack/ironic master: [tempest] also catch BadRequest in negative tests with physical_network in old API  https://review.openstack.org/49082619:24
openstackgerritStenio Araujo proposed openstack/ironic master: Add a flag to always perform persistent boot on PXE interface  https://review.openstack.org/48343019:28
*** krtaylor has quit IRC19:29
*** cdearborn__ has quit IRC19:34
openstackgerritPeter Piela proposed openstack/ironic-ui master: Create driver-property service  https://review.openstack.org/49106719:38
*** krtaylor has joined #openstack-ironic19:43
openstackgerritXavier proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687219:48
mrtenioTheJulia, could you review this patch for us? https://review.openstack.org/#/c/468129/ Thank you19:51
patchbotpatch 468129 - ironic - Introduce hpOneView and ilorest to OneView19:51
TheJuliamrtenio: sure, in a little bit, fixing documentation at the moment, so maybe a half hour19:52
mrtenioIf you need any help, just let me know :P19:53
*** ChubYann has joined #openstack-ironic19:58
*** alineb has joined #openstack-ironic19:59
openstackgerritXavier proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687220:01
*** dougbtv_laptop has quit IRC20:01
openstackgerritXavier proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687220:08
*** e0ne has joined #openstack-ironic20:16
*** amarao has quit IRC20:19
openstackgerritPeter Piela proposed openstack/ironic-ui master: Refactor the PostfixExpr class into a separate service  https://review.openstack.org/49108020:29
jlvillalTheJulia: Any patches that should be added to the BFV priority list?20:32
jlvillalTheJulia: I just reviewed the one open one on the Whiteboard20:32
-openstackstatus- NOTICE: Gerrit is being restarted to pick up CSS changes and should be back momentarily20:36
*** jkilpatr_ has quit IRC20:41
*** e0ne has quit IRC20:53
*** krtaylor has quit IRC20:54
*** ijw has joined #openstack-ironic20:56
*** ijw has quit IRC20:57
*** ijw has joined #openstack-ironic20:57
*** fxpester has quit IRC20:58
openstackgerritJulia Kreger proposed openstack/bifrost master: Begin breaking out installation documentation  https://review.openstack.org/48273421:01
TheJuliajlvillal: looking21:01
*** sambetts_ has quit IRC21:02
TheJuliajlvillal:  updated the whiteboard to add the devstack docs in.  I _think_ from a core driver point of view, that is mainly it, but vendor drivers should also get a look at.21:04
TheJuliajlvillal: next week :)21:05
jlvillalTheJulia: Thanks21:05
*** sambetts_ has joined #openstack-ironic21:06
zer0c00l_I made a review to fix the devstack issue with RHEL7 https://review.openstack.org/#/c/491032/21:08
patchbotpatch 491032 - requirements (stable/ocata) - Bump libvirt-python to 3.5.021:08
zer0c00l_Also i am getting into issues with IRONIC_BASIC_OPS21:08
konetzedAnyone doing uefi booting local disk that has issues with 'linuxefi chain.c32 mbr:0x00000000' spitting out something like bad magic number? Seems like linuxefi doesnt like the chain.c32 version in ubuntu 16.04.  I have tried a few other versions on the web and cannot seem to find one it likes. I have a working work around for now21:08
openstackgerritJulia Kreger proposed openstack/bifrost master: Begin breaking out installation documentation  https://review.openstack.org/48273421:09
zer0c00l_Devstack is failing to create ironic nodes, when i am using 'agent_ssh' driver21:09
zer0c00l_Fails trying to ssh21:09
*** krtaylor has joined #openstack-ironic21:09
zer0c00l_using the generated ssh keypair21:09
zer0c00l_2017-08-04 19:48:54.069 | ++/opt/stack/ironic/devstack/lib/ironic:ironic_ssh_check:1674  ssh -p 22 -o BatchMode=yes -o ConnectTimeout=15 -o StrictHostKeyChecking=no -i /opt/stack/data/ironic/ssh_keys/ironic_key stack@127.0.0.1 exit21:09
zer0c00l_2017-08-04 19:48:54.097 | Warning: Permanently added '127.0.0.1' (ECDSA) to the list of known hosts.21:09
zer0c00l_2017-08-04 19:48:54.212 | Permission denied (publickey).21:09
zer0c00l_Any one see this before?21:09
zer0c00l_what driver does gate use ?21:09
TheJuliakonetzed: never seen that, using a whole disk image built to be uefi, or using a partition image?21:13
*** krtaylor has quit IRC21:14
TheJuliazer0c00l_: the gate now uses agent_ipmitool with virtualbmc which emulates an ipmi bmc for the virtual machines.21:14
zer0c00l_I see21:14
zer0c00l_i will try and use that one21:14
TheJuliazer0c00l_: I'm fairly sure we switched the majority of the gate jobs to use that during the ocata cycle21:15
zer0c00l_Thanks TheJulia21:15
zer0c00l_okay21:15
zer0c00l_Does that need any additional RPMs?21:15
zer0c00l_The virtualBMC stuff?21:15
TheJuliaI don't think so, the devstack plugin installs virtualbmc and pyghmi library (which has the ipmi support built in) from python pypi21:16
*** etingof has quit IRC21:16
konetzedTheJulia: tried both, premade images from ubuntu both version 16.04.  Right now I have it working for both an efi ready image and a nonefi image using 'configfile (hd0,msdos1)/boot/grub/grub.cfg' and 'configfile (hd0,gpt15)/boot/grub/grub.cfg'.  Tells the pxe grub to run the config file off disk and everything seems to boot just fine21:17
konetzedTheJulia: what base os do you use when doing efi, maybe i need to grab that distros chain.c32 and efi-grub?21:18
TheJuliakonetzed: Last time I touched refi, I think i was just doing whole disk images21:19
TheJuliabut it has been about a year21:19
TheJulias/refi/uefi21:19
konetzedah sorry i missed that part yes they are both images are whole disk images21:20
*** actatux has joined #openstack-ironic21:20
actatuxhi21:20
*** crushil has quit IRC21:20
konetzedjust using prebuilt ubuntu cloud images that are efi and non-efi21:20
TheJuliakonetzed: if they are whole boot, why is pxe involved?21:20
TheJuliaactatux: hi21:21
TheJuliakonetzed: (i realize there is a knob to change it from being netbook default to instance, I just don't remember offhand)21:21
konetzedoh, because the moonshot m510 seems to have no legacy boot so it always pxe boots.  Could be im not finding the right docs but even installing from iso on a cartridge it wouldnt boot unless i told efi exactly where to boot from21:22
TheJuliakonetzed: does it boot into IPAA at all?21:24
TheJuliaiPA21:24
konetzedI have also tried setting " properties/capabilities='boot_mode:uefi, boot_option:local'" but I had to go hardcode this line to 'local' as for now it seems to always think the node is set for 'netboot' https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L314 . Still need to debug why that option isnt being set correctly21:25
konetzedTheJulia: sorry not sure what iPA is?21:26
TheJuliaironic-python-agent21:26
NobodyCamfyi: The HPE ProLiant m510 Server defaults to UEFI and does not currently support Legacy BIOS Boot Mode.21:26
konetzedTheJulia: doh yea that makes sense, yes it does boot up, goes through full deploy this is just erroring on first boot after the deploy is finished21:27
konetzedNobodyCam: thanks for confirming, so many docs to read through :D21:27
NobodyCamoh yea21:27
TheJuliaNobodyCam: so, that hardware goes on julia's never buy list21:27
*** krtaylor has joined #openstack-ironic21:27
NobodyCam:)21:28
NobodyCamthat was taken from page four of https://www.hpe.com/h20195/v2/GetPDF.aspx/c05069172.pdf21:28
TheJuliakonetzed: so I wonder what the BMC reports the default boot device as....21:28
konetzedit seems to be correct bootdev accoring to checking on an instance that is fully booted up21:29
TheJuliakonetzed: but you said it tried pxe booting as well?21:29
*** Goneri has quit IRC21:30
TheJuliathe chaining makes me think it is not starting with the whole disk image, but is instead trying to chain load via pxe21:30
*** jkilpatr_ has joined #openstack-ironic21:30
*** cdearborn has quit IRC21:30
konetzedTheJulia: let me start over, so right now ironic can power on, load iPA, dd the whole disk image on and then it reboots, the m510 gets stuck in pxe booting the whole disk image because 'linuxefi chain.c32 mbr:0x00000000' spits out bad magic number21:31
TheJuliawhich pxe->ipxe in non-uefi mode might be occurring, which means a chain load beyond that to refi wouldn't work21:31
* TheJulia looks up the uefi pxe template21:31
konetzedTheJulia: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe_grub_config.template21:32
*** priteau has quit IRC21:34
*** dprince has quit IRC21:35
TheJuliakonetzed: so bootx64.efi is from grub?21:37
*** baoli has quit IRC21:38
konetzedgrub-efi-amd64-signed pkg in ubuntu21:38
konetzedgot that from directions here, https://docs.openstack.org/project-install-guide/baremetal/draft/configure-pxe.html21:39
* TheJulia kind of wonders if http://boot.ipxe.org/ipxe.efi would work, but suspects the bmc is going to demand signed images21:41
*** xavierr has quit IRC21:41
konetzedhumm i could test it21:42
TheJuliayou would have to change the template out for the ipxe one, I seem to remember seeing deep in the ipxe code that it groks chain loading efi21:43
TheJuliabut, not 100% sure since I've never actually tried it21:43
TheJuliathe last hp test machines I had that supported UEFI wouldn't actually boot ubuntu.... :(21:43
TheJuliaat least in eefi mode21:44
TheJuliaufi21:44
* TheJulia gives up, tosses spellcheck into the rubbish bin, and thinks it is time for a beer21:44
konetzedTheJulia: thanks, I still have to debug code and figure out why when a nodes has "properties/capabilities='boot_mode:uefi, boot_option:local'" this value is always set to 'netboot', https://github.com/openstack/ironic/blob/stable/newton/ironic/drivers/modules/pxe.py#L25521:48
tellesnobregahi, have you guys seen issues with keystone starting when using ironic on devstack plugin?21:48
*** leakypipes has quit IRC21:48
tellesnobregaI'm trying on centos7, stack started without ironic21:48
konetzedTheJulia: i will see if the ipxe stuff works on monday one i figure out how to set it up.  I will let you know how it goes :D21:49
*** jcoufal has quit IRC21:50
TheJuliakonetzed: that sounds like a bug :(21:51
TheJuliabecause I bet it might be happier if it defaults to local and asserts that to ipmi21:51
TheJuliatellesnobrega: no, I think you might need to file a bug on bugs.launchpad.net/ironic21:52
TheJuliaI'm literally about to call it a night. :(21:52
konetzedmy work around for now is to hard code it to local :D21:52
TheJuliajlvillal: maybe you've seen something ^^^21:52
TheJuliakonetzed: and that works?21:52
tellesnobregaTheJulia, no worries, me too21:52
tellesnobregahave a good weekend21:52
TheJuliakonetzed: because if it does, then yeah, bug :)21:52
konetzedTheJulia: yep, i added in a line right below 255 that sets it to boot_option = 'local'21:53
jlvillalTheJulia: tellesnobrega: I saw this on a stable/ocata run: http://logs.openstack.org/05/484405/1/check/gate-grenade-dsvm-ironic-ubuntu-xenial/17ca54d/logs/new/screen-n-cpu.txt.gz#_2017-08-04_21_34_38_06421:53
jlvillalNot sure if that is what people are seeing.21:53
konetzedTheJulia: added log statements before and after to verify.21:53
tellesnobregajlvillal, I got a 565: keystone did not start21:54
jlvillaltellesnobrega: I don't know what that means. Missing a lot of context for me.21:54
TheJuliakonetzed: totally a bug then, seems silly not to fix that :)21:54
konetzedTheJulia: best place to file a bug on that?  I have been out of the openstack dev game for a while.21:54
tellesnobregajlvillal, basically trying to set up ironic with devstack on centos721:54
TheJuliakonetzed: bugs.launchpad.net/ironic21:54
jlvillaltellesnobrega: But anyway besides the one error I showed, I have not seen anything.21:55
TheJuliakonetzed: and please remind me monday :)21:55
jlvillaltellesnobrega: But I don't run devstack much locally21:55
konetzedTheJulia: will try and file bug now.  Thanks, have a good weekend and thanks for all your help21:55
* TheJulia engages her cloaking device and go finds dinner21:55
tellesnobregajlvillal, I will keep digging and file a bug if that seems to be the case21:55
TheJuliakonetzed: if you try ipxe, let me know what you encounter21:55
TheJulia:)21:55
konetzedTheJulia: will do21:55
*** rbudden has joined #openstack-ironic22:04
*** pooja has joined #openstack-ironic22:04
poojaHi.. I would like to know if anyone has tried running Ironic on Ocata on an existing Openstack environment at Newton (nova, neutron)?22:05
poojaIs is okay to start with Ocata release just for Ironic or I should set it up on Newton and then upgrade to Ocata alongwith Nova and Neutron services?22:06
TheJuliapooja: in theory, that should work for the most part, but not exactly supported for long term operation22:07
TheJuliawe support nova to be one release behind our release, I don't know how neutron support has tracked, if we have any interdependencies between the two releases22:08
TheJuliapooja: should work just fine for flat networking, might not work for neutron networking22:08
poojaoh okay, thanks @TheJulia.. looks like hardware discovery support was only added in Ocata so was interested in trying that out22:09
poojaOh great, I do intend to use flat networking with neutron for DHCP, don't need tenant network isolation for now22:09
*** ijw has quit IRC22:21
*** krtaylor has quit IRC22:26
*** marlinc has joined #openstack-ironic22:27
*** figleaf is now known as edleafe22:41
*** etingof has joined #openstack-ironic22:45
*** pooja has quit IRC22:45
*** baoli has joined #openstack-ironic22:52
*** Goneri has joined #openstack-ironic22:52
*** superdan is now known as dansmith22:52
*** ijw has joined #openstack-ironic23:12
*** ijw has quit IRC23:17
*** ijw has joined #openstack-ironic23:33
*** ElCoyote_ has quit IRC23:43
*** jinxing has joined #openstack-ironic23:52

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