Thursday, 2016-09-22

« Wednesday, 2016-09-21 Index Friday, 2016-09-23 »
*** bdeschenes has joined #openstack-ironic00:00
*** rama_y has quit IRC00:02
*** bdeschenes has quit IRC00:09
*** rloo has quit IRC00:20
*** causten has joined #openstack-ironic00:48
*** causten has quit IRC00:54
*** causten has joined #openstack-ironic00:55
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE  https://review.openstack.org/36651800:58
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE  https://review.openstack.org/36651800:59
*** causten has quit IRC00:59
*** ijw has quit IRC01:00
*** phuongnh has joined #openstack-ironic01:04
*** sdake has quit IRC01:06
*** baoli has joined #openstack-ironic01:16
*** ijw has joined #openstack-ironic01:19
*** baoli has quit IRC01:21
*** r-daneel has quit IRC01:42
*** ijw has quit IRC01:48
*** baoli has joined #openstack-ironic01:52
*** tuanla has joined #openstack-ironic02:02
*** vsaienko has joined #openstack-ironic02:10
*** wajdi has joined #openstack-ironic02:11
*** wajdi_ has joined #openstack-ironic02:12
*** wajdi has quit IRC02:16
*** ijw has joined #openstack-ironic02:20
*** ijw has quit IRC02:25
openstackgerritzhangyanxian proposed openstack/ironic: Fix typos in rootwrap.conf  https://review.openstack.org/37449302:26
openstackgerritzhangyanxian proposed openstack/ironic: Fix typos in rootwrap.conf  https://review.openstack.org/37449302:27
openstackgerritzhangyanxian proposed openstack/ironic: Fix typos in rootwrap.conf  https://review.openstack.org/37449302:38
openstackgerritzhangyanxian proposed openstack/ironic: Fix typos in rootwrap.conf & ironic-images.filters & ironic-lib.filters  https://review.openstack.org/37449302:39
*** wuhg has joined #openstack-ironic02:39
openstackgerritzhangyanxian proposed openstack/ironic: Fix typos in rootwrap.conf &ironic-images.filters &ironic-lib.filters  https://review.openstack.org/37449302:40
*** vsaienko has quit IRC02:47
*** syed_ has quit IRC03:03
*** david-lyle has quit IRC03:03
*** sdake has joined #openstack-ironic03:08
*** baoli has quit IRC03:21
*** vikrant has joined #openstack-ironic03:35
*** hoangcx has joined #openstack-ironic03:41
*** Sukhdev has joined #openstack-ironic03:45
openstackgerritShivanand Tendulker proposed openstack/ironic: Fix iLO drivers to not clear local_gb if its not detected  https://review.openstack.org/34175203:52
*** vikrant has quit IRC03:53
*** vikrant has joined #openstack-ironic03:54
*** vikrant is now known as vikrant|brb03:57
*** vikrant|brb is now known as vikrant04:02
*** vmud213 has joined #openstack-ironic04:09
*** ijw has joined #openstack-ironic04:25
*** vmud213 has quit IRC04:26
*** ijw has quit IRC04:29
*** jaianshu has joined #openstack-ironic04:49
*** vmud213 has joined #openstack-ironic04:50
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE  https://review.openstack.org/36651804:51
*** bks has joined #openstack-ironic04:57
*** jaianshu_ has joined #openstack-ironic05:06
*** jaosorior has joined #openstack-ironic05:07
*** jaianshu has quit IRC05:10
*** ianw has quit IRC05:23
*** ianw has joined #openstack-ironic05:27
*** vsaienko has joined #openstack-ironic05:29
*** jaianshu__ has joined #openstack-ironic05:34
*** jaianshu_ has quit IRC05:37
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE  https://review.openstack.org/36651805:39
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Add warning and add setting of $LOGFILE  https://review.openstack.org/36651805:39
*** rajinir has quit IRC05:45
*** jaianshu_ has joined #openstack-ironic05:51
*** jaianshu__ has quit IRC05:54
*** mjura has joined #openstack-ironic05:58
*** vmud213 has quit IRC05:58
*** vmud213 has joined #openstack-ironic06:02
*** mjura has quit IRC06:05
*** mjura has joined #openstack-ironic06:05
*** rcernin has joined #openstack-ironic06:07
*** ccamacho has joined #openstack-ironic06:09
*** moshele has joined #openstack-ironic06:21
*** vmud213 has quit IRC06:21
*** Guest40699 has quit IRC06:22
*** vmud213 has joined #openstack-ironic06:22
*** vishwanathj has quit IRC06:25
*** vsaienko has quit IRC06:27
pas-hamorning Ironic06:27
*** vsaienko has joined #openstack-ironic06:28
*** vsaienko has quit IRC06:29
*** vsaienko has joined #openstack-ironic06:33
*** rbartal has joined #openstack-ironic06:35
*** vmud213 has quit IRC06:36
*** vmud213 has joined #openstack-ironic06:38
*** AlexeyAbashkin has quit IRC06:44
*** AlexeyAbashkin has joined #openstack-ironic06:46
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses  https://review.openstack.org/34492106:57
*** Sukhdev has quit IRC07:02
openstackgerritzhangyanxian proposed openstack/ironic: Fix a typo in helper.py  https://review.openstack.org/37299407:03
*** vsaienko has quit IRC07:24
*** tuanla_ has joined #openstack-ironic07:30
*** sdake has quit IRC07:30
*** tuanla has quit IRC07:31
*** narasimha_SV has joined #openstack-ironic07:33
narasimha_SVhttp://paste.openstack.org/show/582509/07:33
narasimha_SVwhen I am doing pxe boot it is stopping with above messages07:34
narasimha_SVhttp://paste.openstack.org/show/582511/07:35
narasimha_SVimages which I have uploaded07:35
*** jpich has joined #openstack-ironic07:39
narasimha_SVhttp://paste.openstack.org/show/582512/07:40
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228907:52
*** ohamada has joined #openstack-ironic07:55
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** athomas has joined #openstack-ironic08:03
*** Romanenko_K has joined #openstack-ironic08:15
*** tuanla has joined #openstack-ironic08:23
*** hoangcx_ has joined #openstack-ironic08:23
*** hoangcx has quit IRC08:25
*** phuongnh has quit IRC08:25
*** phuongnh has joined #openstack-ironic08:25
*** tuanla_ has quit IRC08:25
*** jaianshu has joined #openstack-ironic08:32
pas-hanarasimha_SV: could you also paste result of "ironic node-show <your-node>"?08:32
*** jaianshu__ has joined #openstack-ironic08:34
*** jaianshu_ has quit IRC08:35
tuanlanarasimha_SV, I think it good, you have to waiting for load image in /tftp/08:35
tuanlanarasimha_SV, I think it good, you have to waiting from load image in /tftp/08:36
tuanlanarasimha_SV, I think it good, you have to waiting load image from /tftp/ server08:36
tuanlamorning pas-ha08:38
narasimha_SVtuanla: it is getting stopped there itself08:38
*** jaianshu has quit IRC08:38
narasimha_SVthe process is not moving08:38
narasimha_SVaftrer some time request is failed and ironic making that server to be in powerr off mode08:39
tuanlanarasimha_SV, please check you network08:39
tuanlaand restart you ironic node08:39
narasimha_SVi am not using neutron , I am using an existing dhcp server for getting the ip08:39
tuanlahi narasimha_SV, maybe i don't know to08:40
*** derekh has joined #openstack-ironic08:41
narasimha_SVjaianshu__: http://paste.openstack.org/show/582526/08:41
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228908:43
*** milan has joined #openstack-ironic09:10
milanmorning Ironic! :)09:11
tangchenmorning milan09:11
milanhey tangchen :)09:11
* milan has a review day ;)09:11
tangchenmilan: would you please take a look the functional test patch ?  :)   https://review.openstack.org/#/c/349421/1909:12
patchbotpatch 349421 - ironic - Prepare for functional tests09:12
tangchenmilan: I have posted a patch to enable it. https://review.openstack.org/#/c/356233/09:12
patchbotpatch 356233 - openstack-infra/project-config - Enable ironic functional test09:12
milantangchen, yeah, I shall do it finally today :)09:12
tangchenmilan: Sure. Thanks a lot :)09:13
milanhad some design issues with my patch yesterday that completely consumed me09:13
milanhttps://review.openstack.org/#/c/344921/09:13
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses09:13
* milan needs to ping aarefiev about it actually ;) ^09:13
milanand dtantsur|afk too ;)09:14
aarefievmilan: hey, I'll be here in 1 h09:14
milanaarefiev, sure :)09:14
*** ig0r_ has joined #openstack-ironic09:22
*** stendulker has joined #openstack-ironic09:27
*** milan is now known as milan|f00d09:28
openstackgerritShivanand Tendulker proposed openstack/ironic: Fix iLO drivers to not clear local_gb if its not detected  https://review.openstack.org/34175209:29
mandrelucas-afk, dtantsur|afk: I still haven't gone though a successful deployment with my atomic-host image09:33
mandreit's different from overcloud-full, because one it has is_whole_disk_image set to true09:34
mandreapparently the root cause is partprobe failing on my system with "Device or resource busy" and that triggers the int cast error I've sent yesterday09:35
mandrepartprobe failing: http://paste.openstack.org/show/582540/09:35
mandreint cast error: http://paste.openstack.org/show/582545/09:35
*** bdeschenes has joined #openstack-ironic09:50
mgouldmorning Ironic!09:50
tuanlamorning ironic09:56
aarefievmilan: I'm looking09:59
*** vsaienko has joined #openstack-ironic10:03
openstackgerritVadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA  https://review.openstack.org/37414710:06
*** jaianshu_ has joined #openstack-ironic10:12
*** jaianshu__ has quit IRC10:14
*** jaianshu__ has joined #openstack-ironic10:15
*** hoangcx_ has quit IRC10:16
*** jaianshu_ has quit IRC10:17
vdrokmorning everyone!10:20
*** bdeschenes has quit IRC10:24
*** dtantsur|afk is now known as dtantsur10:26
dtantsurmorning ironic, mandre, mgould, tuanla, aarefiev, milan|f00d, tangchen10:26
dtantsurmandre, ugh.. lemme take a look10:27
aarefievmorning dtantsur10:28
mandredtantsur: thanks, brb lunch10:28
dtantsuryeah, I'll get some food too before jumping into things (really late today, heh)10:28
aarefievdtantsur: When do you plan cut stable branche?10:29
aarefievI'am wondering if we land introspection status list10:30
openstackgerritVadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty  https://review.openstack.org/37076410:32
dtantsuraarefiev, 1. when jroll presses the button to release ironic-inspector, 2. I don't think so10:32
aarefievget it10:33
openstackgerritMerged openstack/ironic: Fix DRAC passthru 'list_unfinished_jobs' desc  https://review.openstack.org/37433110:41
*** dprince has joined #openstack-ironic10:42
jrolldtantsur: s/jroll/release team/ :)10:50
jrollmorning btw10:50
jrollhttps://review.openstack.org/#/c/374367/10:50
patchbotpatch 374367 - releases - Release final newton versions for ironic services10:50
dtantsurmorning jroll, I didn't notice you've already submitted the release request :)10:50
dtantsurso yeah, we're past the cut off point now10:51
*** ohamada has quit IRC10:55
*** ohamada has joined #openstack-ironic10:55
stendulkerdtantsur, jroll: Hi10:56
stendulkerDoes this review https://review.openstack.org/#/c/374367/ have any chance of making it to Newton?10:56
jrollmorning stendulker :)10:56
patchbotpatch 374367 - releases - Release final newton versions for ironic services10:56
jrollwrong link?10:57
jrollstendulker: I assume you mean https://review.openstack.org/34175210:57
patchbotpatch 341752 - ironic - Fix iLO drivers to not clear local_gb if its not d...10:57
stendulkerhttps://review.openstack.org/#/c/341752/10:57
patchbotpatch 341752 - ironic - Fix iLO drivers to not clear local_gb if its not d...10:57
stendulkerHas one +210:57
jrollstendulker: well, final release is already proposed, but we could backport it when the branch is cut and release 6.2.110:58
stendulkerjroll: ok. Thank you10:58
jrollstendulker: no problem, feel free to bug me when the backport is up10:59
stendulkerjroll: sure.10:59
jrollstendulker: one comment on the reno there, when that's fixed I'm +211:02
tangchenmorning dtantsur11:02
*** thrash|g0ne is now known as thrash11:02
stendulkerjroll: fixing it right away11:02
jrollthanks11:03
*** hoonetorg has joined #openstack-ironic11:06
stendulkerjroll: thank you :)11:06
*** dprince has quit IRC11:10
vdrokmorning dtantsur jroll stendulker tangchen11:11
stendulkermorning vdrok :)11:11
*** e0ne has joined #openstack-ironic11:13
tangchenmorning vdork11:14
openstackgerritMerged openstack/python-ironic-inspector-client: Switch stable/newton to Newton upper-constraints  https://review.openstack.org/37402311:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-python-agent: Add info on tinyipa to docs  https://review.openstack.org/37471411:16
openstackgerritShivanand Tendulker proposed openstack/ironic: Fix iLO drivers to not clear local_gb if its not detected  https://review.openstack.org/34175211:16
jroll\o vdrok11:16
mgouldhey everyone11:17
mgouldcan someone explain to me how https://review.openstack.org/#/c/373194 helps with the problem it's meant to address?11:17
patchbotpatch 373194 - ironic - Add 10s sleep for iPXE boot.11:17
mgouldie, how does sleeping for 10s between initrd and boot stop PXE booting problems?11:18
jrollmgould: the commit message should explain it :)11:18
jroll(it doesn't, but, ya know)11:18
*** lucas-afk is now known as lucasagomes11:18
mgouldjroll: heh, good point11:19
lucasagomesmorning all11:19
narasimha_SVhttp://paste.openstack.org/show/582557/11:19
narasimha_SVthose are images i have created using diskimage-builder11:19
narasimha_SVwhat values will be deploy_kernel and deploy_ramdisk of them11:19
mgouldjroll: added a comment asking for an explanation in the commit message11:19
openstackgerritHimanshu Kumar proposed openstack/python-ironicclient: Changes in osc-plugin-ff0d897d8441a9e1.yaml for maintenance node  https://review.openstack.org/37297311:20
jrollmgould: thanks :)11:20
mgouldbut I kinda suspect the answer is "I tried this at random, and it seemed to help"11:20
jrollditto11:20
narasimha_SVI am confused as my pxe booting is getting stopped like this http://paste.openstack.org/show/582509/11:20
mgould[no disrespect to the author, I'm sure we've all done that more than we'd like to admit...]11:20
jrollmgould: btw, om == lazy_prince if you want to chat in irc11:20
mgouldjroll, lazy_prince thanks!11:20
*** jistr is now known as jistr|mtg11:20
mgouldlazy_prince: how does your patch work?! I am confused.11:20
vdrokmorning mgould lucasagomes11:21
openstackgerritVadim Hmyrov proposed openstack/python-ironicclient: Fix display of chassis UUID field if empty  https://review.openstack.org/37076411:21
vdroklucasagomes: jroll dtantsur I'm looking at this https://review.openstack.org/#/c/370764/7 and wonder maybe we should add yet another if here? https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L638-L65211:23
patchbotpatch 370764 - python-ironicclient - Fix display of chassis UUID field if empty11:23
*** stendulker has quit IRC11:24
vdrokso that chassis_uuid is always returned by api11:24
tangchenvdork: Hi Vladyslav, sorry that I have never configured a jenkins job before. I don't quite understand how to make the functional tests run in the job. Would you please tell me more about it ?11:24
vdroktangchen: it's vdrok not vdork :D11:24
tangchenvdrok: Oh, so sorry. :)11:25
vdroktangchen: well, that's what the ironicclient hook does IIUC11:25
vdrokor not, let me check11:25
jrollvdrok: is it not returned as null in the api if it's null?11:25
vdrokjroll: nope :(11:26
jrollthat seems crazy :)11:26
vdrokyeah, it is, and seems like we were not paying attention before as it was hardcoded in client in the list of fields to rpint11:27
*** bdeschenes has joined #openstack-ironic11:29
jrollah, I see11:29
jrollyeah, doing that makes sense, dunno if that's the best place to put it, but if it is then go for it11:29
lucasagomesvdrok, sorry was making some coffee. I agree it should be returned by the API always :-/11:30
vdrok++ will cook a patch shortly11:30
vdroktangchen: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L266-L304 and https://github.com/openstack/python-ironicclient/blob/master/ironicclient/tests/functional/hooks/post_test_hook.sh11:31
vdrokplease take a look at those, we need something similar in ironic functional job11:31
tangchenvdrok: Sure, looking at it. Thank you very much for your patient and nice help. :)11:32
vdrokiirc that was added by aarefiev to the client, he might be able to help if there are any questions11:32
vdroktangchen: np :)11:32
aarefievwhats up?11:32
vdrokaarefiev: tangchen was looking into making the ironic functional job to run functional tests instead of the tempest ones, just pointed him to your work :)11:33
aarefievok, what the problem with ironic job?11:37
dtantsurmorning vdrok, lucasagomes11:39
mgouldmorning dtantsur11:39
tangchenvdrok, aarefiev: Hi guys, IIUC, I think we have already had a functional job config just above the ironicclient one in ironic.yaml11:39
dtantsurand mgould :)11:39
tangchenvdrok, aarefiev: Is that correct ?11:39
* dtantsur is in process of caffeinating11:39
vdroktangchen: well yeah, but it runs tempest tests, not functional11:39
aarefievyeah we have for ironicclient and ^11:40
aarefievalso we have '{pipeline}-ironic-dsvm-functional{job-suffix}'11:40
vdroktangchen: this is what causes tempest tests to run - https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L251-L25211:40
aarefievvdrok: do you what job ^ is for11:41
aarefievKnow11:41
*** rbudden has joined #openstack-ironic11:41
vdrokaarefiev: it is for functional tests which are being added here - https://review.openstack.org/#/c/349421/ and I asked about changing the job to properly these tests instead of tempest ones11:42
patchbotpatch 349421 - ironic - Prepare for functional tests11:42
*** bfournie has quit IRC11:42
vdrokproperly run11:42
aarefievvdrok: aha, ok, I see the problem now11:42
vdrokbasically adding the hooks and changing the template11:43
aarefievI didn't see that functional tests yet, so they setup ironic services by themself11:45
aarefievright?11:46
*** vsaienko has quit IRC11:46
aarefievall we need to run tox11:46
tangchenaarefiev: the functional test patch is here. https://review.openstack.org/#/c/349421/11:46
patchbotpatch 349421 - ironic - Prepare for functional tests11:46
*** ccamacho is now known as ccamacho|lunch11:47
jrolloh hey, did anyone look into fixing the random migration test failures?11:49
jrollI'm about to read that thread in the ML so just curious11:49
tangchenvdrok, aarefiev: IIUC, we need to write a hook in Ironic and then call it in jenkins job template, right ?11:50
tangchen:q11:50
vdrokjroll: I did not in detail, but lucasagomes found this one yesterday https://review.openstack.org/29139711:51
patchbotpatch 291397 - manila - Fix unstable DB migration tests (MERGED)11:51
vdrokin neutron and nova they did basically the same11:51
TheJuliaGood morning11:51
lucasagomesyeah I think worth waiting see if was just a random failures for us or if that really happens more often11:51
lucasagomesthat was the first time I've seem that error in ironic11:52
lucasagomesTheJulia, morning11:52
vdroklucasagomes: happened twice yesterday on another patch, so seems it's getting worse11:52
vdrokmorning TheJulia11:52
lucasagomesvdrok, oh damn, right yeah11:52
jrollvdrok: ah, cool, does someone want to propose that?11:52
jrolllucasagomes: you've seen the thread yeah?11:53
lucasagomesjroll, not really ? ML thread ?11:53
aarefievtangchen: looking on your patch11:53
jrolllucasagomes: yeah, http://lists.openstack.org/pipermail/openstack-dev/2016-September/103664.html11:54
jrolllucasagomes: warning, it's super long, but tl;dr it is a real problem for a bunch of people11:54
*** zigo has quit IRC11:54
lucasagomesjroll, gotcha, I will skim trough it11:55
vdrokjroll: I can do that, but in 3-4 hours11:55
vdrokso if anyone gets faster to that, go ahead :)11:56
jrollvdrok: okay, I'll bug you if I get to it11:56
* jroll has a long list today11:56
mat128morning ironic11:57
jrollheya mat12811:57
*** srobert has joined #openstack-ironic11:58
lucasagomesmat128, good morning11:58
*** baoli has joined #openstack-ironic11:58
*** zigo has joined #openstack-ironic11:58
*** phuongnh has quit IRC11:59
*** zigo is now known as Guest8360111:59
*** baoli_ has joined #openstack-ironic11:59
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228911:59
lucasagomesdtantsur, re virtio disk + uefi, I can't find a proper bug about it. It was a lot of trial and error. The closest I found googling was http://feishare.com/efimail/messages/20130123-0325-Re__edk2__Virtio_disk_not_bootable_in_OVMF-Laszlo_Ersek.html12:00
mandredtantsur: any idea what can cause my partition to be busy at the time ironic is calling partprobe?12:00
lucasagomesdtantsur, it may be fixed on newer versions but the packaged one in ubuntu 16.04 at least doesn't work12:00
lucasagomes(ovmf versions)12:00
dtantsurmandre, sorry, got distracted, not hints yet12:01
dtantsurlucasagomes, ok, fine, I'm just trying to understand myself12:01
mandrewhat is the 'is_whole_disk_image' flag anyway? how does it get set?12:02
*** baoli has quit IRC12:03
dtantsurmandre, based on your Glance image12:03
*** trown|outtypewww is now known as trown12:03
lucasagomesmandre, it means the image you are trying to deploy is a "whole disk image" (an image that contains a partition table)12:03
dtantsurmandre, btw, is this error persistent? I'm confused why is has problems with /dev/sda1 while we're probing an iSCSI share12:03
*** Guest83601 has quit IRC12:03
mandredtantsur: oh, I see, that's because there is no initrd or kernel to fetch12:03
lucasagomesmandre, yeah as dtantsur, in ironic if you don't associate a kernel + ramdisk wiht the image in glance ironic will consider it a whole disk image12:04
dtantsurmandre, correct12:04
dtantsurreleases are on their way \o/12:04
lucasagomeso/12:04
mandredtantsur: yeah, the error is persistent... I've rebooted my undercloud and I still get the same error12:05
*** tuanla has quit IRC12:06
dtantsurlucasagomes, I wonder if we need retries for partprobe...12:09
lucasagomesdtantsur, hm yeah, we could check if the disk is being held by something before using it as well (like lsof)12:10
lucasagomesmay worth trying falling back to kpartx as well12:11
*** joprovost has joined #openstack-ironic12:11
*** vsaienko has joined #openstack-ironic12:11
lucasagomesmandre, is it possible to you hack in the code and check some of ^ ?12:11
*** vsaienko has quit IRC12:11
*** cdearborn has joined #openstack-ironic12:11
openstackgerritMerged openstack/bifrost: Updated from global requirements  https://review.openstack.org/37374112:12
*** zigo_ has joined #openstack-ironic12:12
dtantsurmandre, you can try hacking https://github.com/openstack/ironic-lib/blob/c724cbf368398277071f5b2c735eca32cca22646/ironic_lib/disk_utils.py#L588 to add attempts=10 argument there12:13
dtantsurthis will make this command repeat 10 times before failing12:13
mandredtantsur: will do12:13
*** joprovost1 has joined #openstack-ironic12:14
*** joprovost has quit IRC12:16
*** joprovost1 is now known as joprovost12:16
*** zigo_ has quit IRC12:17
tangchenvdrok, aarefiev: I think I can figure out how to do it, but it is really late for me now. I'll update the patch tomorrow. Please leave comments on my patch, And thanks for the help. :)12:17
vdroktangchen: sure, thank you12:18
*** zigo_ has joined #openstack-ironic12:18
vdrokmorning mat12812:18
milan|f00dmorning jroll, mat128, vdrok! :)12:20
*** milan|f00d is now known as milan12:20
vdrokmorning milan :)12:20
milanand morning lucasagomes  mgould and TheJulia! :) long scroll already ;)12:21
*** joprovost has quit IRC12:21
TheJuliagood morning :)12:21
dtantsurmorning TheJulia12:22
*** jaianshu has joined #openstack-ironic12:23
jrollmorning milan, TheJulia12:23
lucasagomesmilan, good morning12:25
* dtantsur is looking at the final release notes12:25
dtantsurgreat release as usual, thanks everyone!12:25
lucasagomesdtantsur, using "sata" as controller kinda works: http://paste.openstack.org/show/582571/, it does works and boots but the disk doesn't appear in the OS (at least on cirros)12:25
* dtantsur facepalms12:26
lucasagomes¯\_(ツ)_/¯12:26
dtantsurlucasagomes, ok, leave whatever works for you12:26
lucasagomesdtantsur, hah ack12:26
lucasagomesI think it's still a bit immature, that's why we see all this oddities12:26
*** jaianshu__ has quit IRC12:27
jrolldtantsur: +1000, newton was amazing12:27
dtantsurnow we need stable/newton's for $everything (and then fix reno, fix u-c, ..)12:28
jrollyeah, lots to do there12:28
* milan lands just tests in Newton :D but otherwise, lot of good work guys! 12:30
*** vsaienko has joined #openstack-ironic12:30
* dtantsur realized he no longer has excuses for not hacking on the driver composition..12:31
jrollhehehe12:31
mandredtantsur: alright, so I didn't have the partprobe issue this time, so it looks like it's not consistent after all. I don't think it had to retry, but I guess it doesn't hurt to leave the attempts=10 in the code12:31
mandredtantsur: bad news, I'm still hitting the int cast issue12:32
*** vsaienko has quit IRC12:34
*** bfournie has joined #openstack-ironic12:35
dtantsurugh12:35
*** causten has joined #openstack-ironic12:35
dtantsurlemme finish a thing here, I'll get back to it12:35
openstackgerritDavanum Srinivas (dims) proposed openstack/bifrost: Update .gitreview for stable/newton  https://review.openstack.org/37476412:35
openstackgerritDavanum Srinivas (dims) proposed openstack/bifrost: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37476512:35
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic-inspector: Update .gitreview for stable/newton  https://review.openstack.org/37476612:36
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic-inspector: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37476812:36
*** jistr|mtg is now known as jistr12:36
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic-python-agent: Update .gitreview for stable/newton  https://review.openstack.org/37477012:37
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic-python-agent: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37477112:37
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic-python-agent: Update reno for stable/newton  https://review.openstack.org/37477212:37
* jroll goes through and approves there12:37
* mat128 was going through those for IPA, but github doesnt even have stable/newton yet :(12:37
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: Update .gitreview for stable/newton  https://review.openstack.org/37477312:37
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37477412:38
*** vsaienko has joined #openstack-ironic12:38
*** srobert has quit IRC12:38
*** vnogin has joined #openstack-ironic12:39
*** causten has quit IRC12:39
jrolldims: you rock12:40
dtantsurjroll, don't forget to check reno, sometimes it's off12:40
openstackgerritMerged openstack/bifrost: Updated from global requirements  https://review.openstack.org/36597012:40
TheJuliadims: jroll: thank you both!12:41
jrolldtantsur: yeah, wip'd it and made a todo to check all the projects12:41
dimsthanks TheJulia jroll, just pushing buttons like dhellmann taught me to :)12:41
jrollhehe12:41
*** vikrant has quit IRC12:42
TheJulia:)12:42
dtantsurmandre, in the meanwhile, could you please file both your problems as bugs on launchpad?12:47
dtantsur(I guess they're independent, so 2 bugs)12:47
mandredtantsur: sure12:47
jrollhttps://review.openstack.org/#/c/373701/5 https://review.openstack.org/#/c/373747/3 https://review.openstack.org/#/c/373749/212:48
patchbotpatch 373701 - ironic - Updated from global requirements12:48
patchbotpatch 373747 - ironic-inspector - Updated from global requirements12:48
patchbotpatch 373749 - ironic-python-agent - Updated from global requirements12:48
jroll^ ocata requirements that need merged12:48
*** zigo_ has quit IRC12:48
dtantsurdone12:49
*** vmud213 has quit IRC12:49
aarefievtangchen, vdrok, IIUC  it should be something like https://review.openstack.org/#/c/374785/112:50
patchbotpatch 374785 - openstack-infra/project-config - Add lightweight ironic functional job12:50
*** zigo_ has joined #openstack-ironic12:51
vdrokjroll: hm, on the ironic requirements patch *ipmitool jobs fail with No matching distribution found for pyghmi===1.0.1112:51
vdrokaarefiev: for now it's possible, but when we'll have real functional tests we'll need other services12:52
jrollJayF: I've unblocked the install guide tree, but also unapproved a bunch because rloo has -1s, probably want to talk to her first before we start approving https://review.openstack.org/#/q/topic:bug/161227812:52
mat128jroll: I havent read them yet but rloo told me she would poke me when shes done reviewing12:53
aarefievvdrok: so devstack is needed12:53
jrollvdrok: it exists, maybe mirror issues? https://pypi.python.org/pypi/pyghmi/1.0.1112:53
vdrokyeah, might be12:53
*** Goneri has joined #openstack-ironic12:53
* mat128 just noticed my bouncer buffer isnt long enough12:54
aarefievDo we have agreement how functional tests should look like12:54
*** vsaienko has quit IRC12:56
*** david-lyle has joined #openstack-ironic12:57
*** rloo has joined #openstack-ironic12:59
*** vsaienko has joined #openstack-ironic12:59
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228912:59
*** ccamacho|lunch is now known as ccamacho12:59
mandredtantsur: created https://bugs.launchpad.net/ironic/+bug/1626540 and https://bugs.launchpad.net/ironic/+bug/162654412:59
openstackLaunchpad bug 1626540 in Ironic "partprobe sometime fails with Device or resource busy" [Undecided,New]12:59
vdrokaarefiev: I don't remember this being discussed12:59
openstackLaunchpad bug 1626544 in Ironic "int cast error when deploying whole disk image" [Undecided,New]12:59
*** amotoki has joined #openstack-ironic13:00
jrollvdrok: pinged infra about the pyghmi thing13:00
jroll"we had a stuck mirror and only resynced it this morning."13:00
vdrokaha, thank you!13:01
jrollnp :)13:01
dtantsurmandre, thanks! lemme try to figure out the int() thing first, I guess...13:02
mandreAc2llMap13:07
mandrewoops13:07
*** jcoufal has joined #openstack-ironic13:08
dtantsurmandre, could you please hack https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L224 to say LOG.exception? we should have done it for real...13:09
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/37370113:09
* dtantsur fixes it13:09
openstackgerritMerged openstack/bifrost: Update .gitreview for stable/newton  https://review.openstack.org/37476413:10
openstackgerritMerged openstack/bifrost: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37476513:10
xavierr|afkdtantsur: when the InstanceDeployFailure is raised does not it log the exception already?13:11
*** xavierr|afk is now known as xavierr13:11
dtantsurxavierr|afk, exception message; but not the traceback13:11
dtantsurthere is a rule (nearly): every time you catch generic Exception, you use LOG.exception13:12
dtantsurmessage only can be extremely useless (think KeyError)13:12
xavierrI'll take a note13:12
*** jaosorior has quit IRC13:12
xavierrthank you dtantsur :)13:12
xavierrgood morning Ironic13:12
*** jaosorior has joined #openstack-ironic13:13
vdrokmorning xavierr13:13
dtantsurmorning xavierr13:13
*** jaianshu_ has joined #openstack-ironic13:14
mandredtantsur: ack, I'm testing13:14
sergekMorning all13:16
*** vsaienko has quit IRC13:17
*** jheroux has joined #openstack-ironic13:17
*** jaianshu has quit IRC13:18
milanmorning xavierr, sergek! :)13:20
xavierrmilan: hi there13:20
openstackgerritVladyslav Drok proposed openstack/ironic: Always return chassis UUID in node's API representation  https://review.openstack.org/37481113:21
vdrokhope we don't need api version bump for that ^13:22
vdrokmorning sergek13:22
*** baoli_ has quit IRC13:22
*** baoli has joined #openstack-ironic13:22
*** hshiina has joined #openstack-ironic13:23
openstackgerritLucas Alvares Gomes proposed openstack/ironic: DevStack: Support for creating UEFI VMs  https://review.openstack.org/37434613:23
mandredtantsur: we got the culprit, it's disksize_mb = int(disksize_bytes) // 1024 // 102413:24
mandrein /usr/lib/python2.7/site-packages/ironic_lib/disk_utils.py:57113:24
*** causten has joined #openstack-ironic13:24
dtantsurmandre, good find. and what is disksize_mb for you?13:25
openstackgerritXu Ao proposed openstack/ironic: Make ironic console supporting the nokeepalive mode  https://review.openstack.org/37283013:25
*** wajdi_ has quit IRC13:27
*** baoli has quit IRC13:27
rloohi and good morning everyone, including vdork (er vdrok, chuckle)13:27
*** jaianshu_ has quit IRC13:27
vdrok:D morning rloo13:28
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/36599913:28
lucasagomesrloo, g'morning13:28
rloomat128: I pinged you yesterday aft but I think you were gone by then. Anyway, I finished reviewing the install guide. unless i missed anything (i hope not) i think if you address (some of) my concerns, I'll be happy.13:28
rloomat128: the only thing I'm not sure about is the organization/wording maybe of the install& configure OS parts, but we can refine those after looking at them.13:29
xavierrmorning rloo :)13:29
mat128rloo: I'm already on it :) I missed your ping but found it on eavesdrop.o.o. Also adjusted my IRC bouncer buffer, was set way too low13:29
rloomorning lucasagomes, xavierr13:29
rloothx mat128!13:29
mat128rloo: the only thing I might not be able to address in this iteration is the per-OS instructions13:29
mandredtantsur: moment... I added more debug13:30
mat128this requires a lot of work in it's current form (duplicate sections) or support for tags, which requires -infra's help and project-config modifications13:30
*** ayoung has quit IRC13:30
rloomat128: yeah, understood. it only matters for the docs in the install&config OS sections.13:30
mat128rloo: but I'm going through all the wording and things you and everyone else pointed out13:30
mat128we're going to make this install guide a great install guide :)13:30
rloothx mat128! I would hold off any editorial changes to the existing content though.13:31
rloomat128: cuz there are things in the content that can be improved upon. let's just get it moved to the new location first, then improve on it!13:32
mat128=)13:32
mat128yes13:32
openstackgerritVladyslav Drok proposed openstack/ironic: Always return chassis UUID in node's API representation  https://review.openstack.org/37481113:32
rloovdrok: thx for fixing the db migration/random timeout issue. I saw it happening a few times yesterday.13:32
vdrokrloo: I have not yet, going to do that after a lunch13:33
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/37374913:39
dtantsurhuh, I find it pretty confusing that we have two IronicException: in ironic-lib and in ironic13:39
vdrok++ we should switch to ironic-lib's one I think13:40
mgoulddtantsur: did we get part-way through moving it from one repo to the other?13:40
dtantsurprobably13:41
dtantsurnow our code does not treat ironic_lib.IronicException as proper IronicException...13:41
* dtantsur should probably fix this too13:41
mandredtantsur: the return of utils.execute is not properly chomped, I got disksize_bytes ('53687091200\n', '')13:41
* dtantsur facepalms13:42
*** joprovost has joined #openstack-ironic13:42
mgouldaaargh13:42
pesterhm, I need to rebuild KVM qcow2 image for Ironic, looks like it is possible with disk-image-builder, but image I got consist of 7 LVM Volumes - is there any exmples for such a case ?13:42
dtantsurmandre, thanks for investigating, looking into a fix now..13:42
mandredtantsur: I'm testing now13:43
*** mgoddard__ has joined #openstack-ironic13:43
dtantsurdamn, was this code even tested?13:43
*** syed_ has joined #openstack-ironic13:44
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/37374713:44
dtantsurI guess our gate runs without config drive, right?13:44
*** mtanino has joined #openstack-ironic13:46
*** mgoddard_ has quit IRC13:46
*** narasimha_SV has quit IRC13:49
openstackgerritDmitry Tantsur proposed openstack/ironic-lib: Fix creating config drive for whole disk images  https://review.openstack.org/37483213:52
dtantsurmandre, ^^^13:52
dtantsuralso morning rloo, I haven't noticed you in this bug fixing drill...13:52
*** srobert has joined #openstack-ironic13:52
openstackgerritBharath kumar proposed openstack/ironic-python-agent: Moving Reboot bashscript to python  https://review.openstack.org/37247813:54
*** makowals_ has joined #openstack-ironic13:59
*** mgoddard__ has quit IRC13:59
rloodtantsur: what drill? (am I missing) :)13:59
dtantsurrloo, just too many bugs today :)13:59
rloodtantsur: oh. Murphy's law, right after a release? :-(14:00
*** makowals has quit IRC14:01
*** r-daneel has joined #openstack-ironic14:01
*** bks has quit IRC14:02
*** joprovost1 has joined #openstack-ironic14:02
*** joprovost has quit IRC14:03
dtantsurof course14:03
mandredtantsur: that did the trick, I now see my server as active14:03
openstackgerritRicardo Araújo Santos proposed openstack/ironic: Fixes wrong steps to perform migration of nodes  https://review.openstack.org/37484114:03
dtantsurmandre, awesome14:04
*** joprovost has joined #openstack-ironic14:05
*** sdake has joined #openstack-ironic14:06
*** joprovost1 has quit IRC14:07
dtantsurmandre, I'll backport the patch to newton as soon as it merges to master14:07
openstackgerritMerged openstack/ironic-python-agent: Update .gitreview for stable/newton  https://review.openstack.org/37477014:08
openstackgerritMerged openstack/ironic-python-agent: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37477114:08
rloojroll, dtantsur, anyone -- do we put release notes when documentation is corrected? https://review.openstack.org/#/c/374841/1/releasenotes/notes/bug-1626548-b49ca6ce1f84da79.yaml14:10
patchbotpatch 374841 - ironic - Fixes wrong steps to perform migration of nodes14:10
dtantsurhmm, I'm not sure14:11
mandredtantsur: I spotted another error in my logs, not sure if this is critical since it was apparently able to recover from it http://paste.openstack.org/show/582586/14:12
rloodtantsur: if anything, i would think the documentation itself would be revisioned14:12
rloodtantsur: with its own 'release notes'...14:13
dtantsurmandre, this is suspicious14:14
*** rajinir has joined #openstack-ironic14:17
*** vsaienko has joined #openstack-ironic14:17
dtantsurjroll, FYI https://bugs.launchpad.net/ironic/+bug/162657814:19
openstackLaunchpad bug 1626578 in Ironic "[RFE] Switch to using IronicException from ironic-lib" [Wishlist,Confirmed]14:19
*** ralonsoh_ has joined #openstack-ironic14:19
*** vishwanathj has joined #openstack-ironic14:19
*** wuhg has quit IRC14:21
jrollrloo: I don't think that's terribly useful14:21
jrolldtantsur: configdrive is on in the gate14:22
*** daemontool has joined #openstack-ironic14:22
dtantsurcool!14:22
rloojroll: that's what i thought. thx.14:22
jrolldtantsur: commented14:22
dtantsurjroll, then how comes that we haven't noticed an obvious error?14:22
jrolldtantsur: hmm, good question, maybe our gate doesn't actually do whole disk images with iscsi?14:23
dtantsurI hope you're wrong :)14:24
jrolldtantsur: or we haven't used a version of ironic-lib with that bug in the gate?14:24
* jroll wonders how new/old that is14:24
dtantsurjroll, launchpad won't let me comment, so here: Maybe, but IPA exceptions are different, while ironic and ironic-lib ones are copy-pastes of each other.14:24
*** athomas has quit IRC14:24
jrollsure14:25
jrollapproved14:25
dtantsurthanks14:25
jrollnp14:25
rloojroll: https://review.openstack.org/#/c/230924/ the configdrive stuff14:26
patchbotpatch 230924 - ironic-lib - Support configdrive in iscsi deploy for whole disk... (MERGED)14:26
rloojroll: fairly recent, july 2814:27
jrollhuh, that's in 2.1.0, which is in u-c for newton14:27
jrollshould've seen it14:27
jroll:|14:27
*** zzzeek has quit IRC14:27
dtantsurjroll, whole disk gate is doing partitions http://logs.openstack.org/28/366828/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-nv/61fbe9b/logs/screen-ir-cond.txt.gz?level=DEBUG#_2016-09-13_13_35_51_04214:27
jrolloh good!14:28
rloojroll: am thinking i might have made that change to .execute() prior to that configdrive patch merging.14:28
dtantsurso yeah, we're not testing it14:28
* dtantsur will dive into project-config after a quick snack14:29
*** jaosorior has quit IRC14:29
rloojroll: nope, execute has always returned out, err14:29
*** joprovost has quit IRC14:30
jrollrloo: yeah, we missed it in CI. the downside of mocking the world :(14:30
*** joprovost has joined #openstack-ironic14:30
rloojroll: yeah, it'd be fine, if our mocks were correct. i've seen that happen before too. unless we all spend even more time reviewing, hard to catch those.14:30
lucasagomesdid we break whole disk image ?14:31
*** sabeen has joined #openstack-ironic14:31
* lucasagomes reads the scrollback14:31
*** zzzeek has joined #openstack-ironic14:31
*** wajdi has joined #openstack-ironic14:31
dtantsurlucasagomes, and probably long time ago, provided that it's not tested in gate...14:31
*** baoli has joined #openstack-ironic14:31
lucasagomes*sigh* :-/14:32
dtantsuranyway, https://review.openstack.org/#/c/374832/ seems valid, I'd appreciate +A on it14:32
patchbotpatch 374832 - ironic-lib - Fix creating config drive for whole disk images14:32
jrolldtantsur: sure, just a minute14:32
openstackgerritJim Rollenhagen proposed openstack/ironic: Update devstack IPA_DOWNLOAD_BRANCH to stable/newton  https://review.openstack.org/37486314:33
openstackgerritJim Rollenhagen proposed openstack/ironic: Link to newton version of old install guide  https://review.openstack.org/37486414:33
jrollthese are for newton, first might fail CI until something lands in IPA14:33
jrolldtantsur: +A14:34
*** mat128 is now known as mat128|bbl14:35
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Change setting of $LOGFILE  https://review.openstack.org/36651814:35
openstackgerritTuan Luong-Anh proposed openstack/ironic: Docs: Remove setting of $LOGFILE  https://review.openstack.org/36651814:37
*** athomas has joined #openstack-ironic14:37
*** rpioso has joined #openstack-ironic14:41
openstackgerritMerged openstack/ironic-inspector: Update .gitreview for stable/newton  https://review.openstack.org/37476614:45
openstackgerritMerged openstack/ironic-inspector: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37476814:45
*** ayoung has joined #openstack-ironic14:49
openstackgerritDmitry Tantsur proposed openstack/ironic: Fix exception handling in iscsi_deploy.continue_deploy  https://review.openstack.org/37488114:49
*** catintheroof has joined #openstack-ironic14:50
dtantsurhmm, localrc looks correct http://logs.openstack.org/28/366828/5/check/gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-nv/61fbe9b/logs/localrc.txt.gz14:51
dtantsur(even though we duplicate IRONIC_VM_EPHEMERAL_DISK)14:51
*** moshele has quit IRC14:52
dtantsurjroll, vdrok, do we need to merge https://review.openstack.org/329625 to actually test whole disk images?14:53
patchbotpatch 329625 - ironic - Change the logic of selecting image for tests14:53
openstackgerritJim Rollenhagen proposed openstack/ironic: Link to newton version of old install guide  https://review.openstack.org/37486414:53
jrolldtantsur: I thought that was only needed for IPA + partition, but maybe we do :/14:54
vdrokdtantsur: yeah, that was supposed to do ipa+partition14:54
* dtantsur checks14:54
vdroknot sure when whole disk gone wrong :(14:54
*** daemontool has quit IRC14:54
openstackgerritRicardo Araújo Santos proposed openstack/ironic: Fixes wrong steps to perform migration of nodes  https://review.openstack.org/37484114:54
dtantsurseems like with this change we do get wholedisk tested: http://logs.openstack.org/25/329625/13/check/gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-nv/3a3c7f5/logs/screen-ir-cond.txt.gz?level=DEBUG#_2016-09-21_11_56_40_28314:55
*** daemontool has joined #openstack-ironic14:55
dtantsurso it will break as soon as we get newer ironic-lib in u-c ^_^14:56
dtantsurjroll, I think we should merge https://review.openstack.org/#/c/329625 as then figure out how to fix the job, as it's non-voting anyway14:56
patchbotpatch 329625 - ironic - Change the logic of selecting image for tests14:56
jrolldtantsur: yeah, I've wanted to merge that for some time anyway :)14:57
dtantsurand probably backport it to newton...14:57
*** e0ne has quit IRC14:57
jrollyep14:57
dtantsurwe need all jobs running on ironic-lib too.. but I guess infra will shot it down because of "migrate to xenial, finally!"14:58
jrolldtantsur: merged14:58
jrollthough, why isn't that broken now?14:58
jroller14:58
dtantsurjroll, because older ironic-lib in u-c?14:58
jrolldtantsur: the breaking change was in 2.1.0 which is in u-c14:59
* dtantsur applies his head to this desk14:59
dtantsurI don't know, let us fix one thing at a time.....14:59
jrollhttps://github.com/openstack/ironic-lib/commit/7e926fd3fbb15b2e247e5bbc8c490d4c85857fbc14:59
jrollsure14:59
jrolljust confused, I did approve it, we need it anyway14:59
openstackgerritMerged openstack/ironic-lib: Fix creating config drive for whole disk images  https://review.openstack.org/37483215:00
openstackgerritDmitry Tantsur proposed openstack/ironic-lib: Fix creating config drive for whole disk images  https://review.openstack.org/37489715:00
jrolldtantsur: oh, that has a depends-on for https://review.openstack.org/#/c/326061/15:00
patchbotpatch 326061 - openstack-infra/devstack-gate - Do not set ephemeral size based on driver15:00
dtantsurgrrrr15:01
dtantsurjroll, vdrok, let's drop this dependency... out project-config setting anyway override whatever devstack-gate forces on us (see generated localrc)15:01
jroll+A on the backport15:01
dtantsurand anyway, Depends-On does not work for devstack-gate and project-config IIRC15:02
vdrokok will edit this15:02
jrollit does work for dsg15:02
vdrokit works for devstack-gate15:02
jrolland it still helps block with p-c15:02
openstackgerritVladyslav Drok proposed openstack/ironic: Change the logic of selecting image for tests  https://review.openstack.org/32962515:03
*** mjura has quit IRC15:03
jroll+2 there15:03
dtantsur+2 as well, we can approve after it passes15:04
jrollsure15:05
openstackgerritMerged openstack/ironic: Update .gitreview for stable/newton  https://review.openstack.org/37477315:07
openstackgerritMerged openstack/ironic: Update UPPER_CONSTRAINTS_FILE for stable/newton  https://review.openstack.org/37477415:07
*** rcernin has quit IRC15:07
dtantsurjroll, have we had reno updates proposed for newton?15:08
jrolldtantsur: no, I'm doing it now15:08
dtantsurawesome15:08
jrolldo we usually update the stable branch too? can't recall15:09
dtantsurjroll, usually no15:09
jrollthought so, thanks15:09
*** arahal_ has joined #openstack-ironic15:09
jrollright there ^15:10
openstackgerritJim Rollenhagen proposed openstack/ironic: Update reno index for Newton  https://review.openstack.org/37490415:10
jroller ^15:10
* jroll does other projectd15:10
rybridges1Hey guys. I was just having a look at _check_deploying_status() here -> https://review.openstack.org/#/c/197141/9/ironic/conductor/manager.py and I see that {'id': node_id} is being passed as the filter to fail_if_in_state(). This filter then gets passed to iter_nodes() from fail_if_in_state(). From iter_nodes(), it looks like the filter eventually makes its way to in _add_nodes_filters() in15:11
patchbotpatch 197141 - ironic - Periodically checks the status of nodes in DEPLOYI... (MERGED)15:11
rybridges1ironic/db/sqalchemy/api.py. However, I do not see a check for an 'id' filter in add_node_filters(). Doesn't this mean that the 'id' filter is effectively doing nothing?15:11
dtantsurjroll, I think you need origin/stable/newton, not just stable/newton15:11
*** e0ne has joined #openstack-ironic15:11
openstackgerritRuby Loo proposed openstack/ironic: Add i18n _() to string  https://review.openstack.org/37490515:11
jrolldtantsur: oops, thanks15:12
openstackgerritJim Rollenhagen proposed openstack/ironic: Update reno index for Newton  https://review.openstack.org/37490415:12
dtantsurjroll, lgtm, waiting for rendered version before voting15:13
*** r-daneel has quit IRC15:13
jrollsure15:13
*** causten has quit IRC15:13
*** causten has joined #openstack-ironic15:13
*** makowals_ has quit IRC15:14
*** zigo_ is now known as zigo15:14
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Update reno for stable/newton  https://review.openstack.org/37477215:14
jrollfixed the IPA one15:14
dtantsurrybridges1, looking15:15
*** causten_ has joined #openstack-ironic15:15
jrolland inspector15:16
dtantsurrybridges1, it looks like that indeed... have you experienced any problems due to that?15:16
openstackgerritJim Rollenhagen proposed openstack/ironic-inspector: Update reno for Newton  https://review.openstack.org/37491015:16
*** makowals has joined #openstack-ironic15:17
*** causten has quit IRC15:18
jrolland bifrost15:19
openstackgerritJim Rollenhagen proposed openstack/bifrost: Update reno for Newton  https://review.openstack.org/37491215:19
openstackgerritVladyslav Drok proposed openstack/ironic: Increase timeout for migration-related tests  https://review.openstack.org/37491315:19
jrollin all this excitement I've forgotten ironic-ui15:19
jrollbetherly: do you have a final release of ironic-ui happening in the next few days?15:19
vdrokjroll: ^ timeout increase15:19
jrollbetherly: and remind me, do you do stable branches there?15:19
jrollvdrok: thanks!15:19
dtantsurjroll, https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1246 looks suspicious indeed (see rybridges1's comment above)15:20
vdrokI'm not sure if the decorator is necessary, as the reason is very specific, maybe just adding usefixture is ok15:20
jrollvdrok: hm, I think the problem exists for postgres too, no?15:20
jrollvdrok: yeah, agree15:20
vdrokjroll: ah, right, i'll rename the variable :)15:21
jrollvdrok: thanks :D15:21
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/37370115:21
vdrokit is used for both mysql and postgres tho15:21
jrollah good15:21
*** jcoufal has quit IRC15:21
*** rbartal has quit IRC15:22
openstackgerritVladyslav Drok proposed openstack/ironic: Increase timeout for migration-related tests  https://review.openstack.org/37491315:22
*** causten_ has quit IRC15:23
rybridges1@dtantsur Yes I have been extending that check_deploying_status() method to also work for nodes in the CLEANING state. And in the midst of writing more robust unit tests, I noticed that there was actually no filtering being done at all15:23
dtantsurugh15:23
betherlyjroll: early next week should be. finishing touches happening15:24
dtantsurrybridges1, mind reporting it as a bug, so that we don't forget (of course feel free to fix it as well)?15:24
jrollbetherly: thanks15:24
jrolldtantsur: that's super weird, but it must be doing the right thing, right? otherwise we're failing every node there15:25
jrollor, every deploying node15:25
rybridges1@dtantsur I actually already have a fix for it. Thing is... I don't know what the process is for making upstream pull requests. If you guys have a doc of some sort that outlines the process, I can make a gerrit review with the fix today15:25
dtantsuryeah, I'm thinking why we don't face it..15:25
rybridges1@jroll my thoughts exactly, it seems very strange...15:25
*** causten has joined #openstack-ironic15:26
jrollrybridges1: http://docs.openstack.org/infra/manual/developers.html15:26
dtantsurmaybe because offline_conductors is usually empty?15:26
jrollit's a bit of work15:26
jrolldtantsur: ah, you're probably right15:27
*** jcoufal has joined #openstack-ironic15:27
dtantsuralso, this requires a lock, which won't work for normal nodes15:27
dtantsurhttps://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L34615:28
betherlyjroll: think its just the master branch15:28
dtantsurmeh, this code is confusing...15:28
jrollbetherly: hm, there's a stable/mitaka15:28
jrolldtantsur: why wouldn't locks work for normal nodes?15:28
jrolloh15:28
jrollbecause they're in deploying15:28
jrollso should be locked15:28
dtantsurjroll, we remove locks only for "bad" nodes15:29
jrollso it'll probably just hang eh15:29
jrollweird15:29
dtantsurhere filter is still correct: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L122815:29
dtantsurso we only forcibly unlock correct nodes15:29
dtantsurthen we try to fail all nodes, but it requires a lock15:29
jrollmmm, right15:29
rybridges1@dtantsur right, we will only remove the lock from nodes that were associated with an offline conductor15:29
dtantsurso it eventually does the right thing15:29
betherlyjroll: ah ok. likely. my brains a bit broken atm sooo...15:29
dtantsurstill, this needs fixing15:30
jrollbetherly: anyway, let me know when you have a release and we'll do the branch and whatnot15:30
jrolldtantsur: yeah agree15:30
dtantsurthanks for spotting it, rybridges115:30
jroll++15:30
*** zackf has quit IRC15:30
*** lucasagomes is now known as lucas-hungry15:30
*** jcoufal_ has joined #openstack-ironic15:31
jrollvdrok: remind me when your talk with john and vasyl is?15:31
vdroklemme check15:31
*** zackf has joined #openstack-ironic15:31
jrollour schedule looks nice https://docs.google.com/spreadsheets/d/1TQ-RSlbiBBEclkonIbfUP7R1ExZSJylF1uiEKV2G_Cw/pubhtml?gid=1107826458&single=true15:32
dtantsurI guess everything conflicting with everything is inavoidable :)15:32
jrollheh15:33
*** jcoufal has quit IRC15:33
vdrokjroll: Thu 27  3:30pm-4:10pm15:33
jrollthanks!15:33
vdrokyw :)15:34
jrollwoo, no conflict15:34
*** bks has joined #openstack-ironic15:34
rybridges1no problem. the fix i have in mind is to add a check for uuid in _add_nodes_filters(). Then just pass in the uuid to fail_if_in_state() as a filter. That way we ensure that the exact node that we just unlocked is actually the one that we fail. we can even return immediately after the uuid check in _add_nodes_filters() since uuid is unique and the rest of the checks wont matter at that point.15:35
jrollrybridges1: might be able to just use dbapi.get_node_by_id() there, but I guess I do like the idea of fixing _add_nodes_filters so this doesn't happen again15:36
*** hshiina has quit IRC15:36
openstackgerritCharlle Daniel proposed openstack/ironic: Adds to the documentation a note about the state changing of a node by Ironic  https://review.openstack.org/37273515:37
*** marlinc has quit IRC15:37
*** marlinc has joined #openstack-ironic15:38
openstackgerritMerged openstack/ironic-ui: Updated to ironic API v1.11 and added manageable state  https://review.openstack.org/36109815:39
*** ccamacho is now known as ccamacho|afk15:39
*** marlinc has quit IRC15:42
*** marlinc has joined #openstack-ironic15:43
*** arahal_ has quit IRC15:46
*** BadCub has quit IRC15:47
*** jjohnson2 has joined #openstack-ironic15:47
rybridges1jroll: I was considering that. The thing is, i think that would also require a rewrite to fail_if_in_state() because it is currently asking for filters in its parameters to lookup nodes that need to be failed via iter_nodes(), rather than simply looking for a concrete reference to a single node that has already been looked up. I think it was written this way because that logic makes sense in the15:47
rybridges1case where we need to fail nodes when a conductor comes back online and it still has nodes in CLEANING/DEPLOYING. In that case, we just want to lookup ALL nodes in the CLEANING/DEPLOYING state and fail them immediately since after coming back online, it is impossible for the conductor to complete the deployment process for a node that got stuck in one of those states.15:47
*** BadCub has joined #openstack-ironic15:47
jrollrybridges1: ah yeah, good point. forgot it ran through there for a moment15:47
*** MattMan has quit IRC15:50
*** MattMan has joined #openstack-ironic15:50
*** bks has quit IRC15:51
rybridges1Personally, I think it might be best to simply add id and uuid to add_node_filters() because it is so intuitive. Whoever wrote tried to pass 'id' in as the filter to fail_if_in_state probably assumed that 'id' would be something that you can filter by since it is so basic and simple. Having it there would almost certainly prevent future bugs of this nature15:52
jrollyeah, that makes sense to me :)15:53
dtantsurrybridges1, we should also raise an exception if unknown filters are passed15:53
*** arahal_ has joined #openstack-ironic16:01
*** Nisha_away has joined #openstack-ironic16:03
rybridges1dtantsur: that's actually a really great point.16:04
rybridges1dtantsur: we should definitely add that as well16:04
rybridges1would have prevented this whole thing16:04
dtantsuryeah16:04
jroll+116:04
*** jpich has quit IRC16:05
openstackgerritCharlle Daniel proposed openstack/ironic: Implements the validation to verify if the node is free or in use by Ironic  https://review.openstack.org/37276216:07
*** Nisha_away has quit IRC16:18
dtantsursee you tomorrow, g'night16:18
*** dtantsur is now known as dtantsur|afk16:19
*** rcernin has joined #openstack-ironic16:20
vdrokgood night dta16:23
vdrokgood night dtantsur|afk :)16:23
rajinirkrtaylor: MoltenIron, is there any instructions how to add this to the build template?16:24
openstackgerritRicardo Araújo Santos proposed openstack/ironic: Fixes wrong steps to perform migration of nodes  https://review.openstack.org/37484116:26
rybridges1jroll: so to raise an exception if unknown filters are passed, I was thinking of just creating a list called supported_filters which contains all of the filters that are checked by _add_nodes_filters(). Then just check in a for loop is every filter that is passed in is in the list of supported filters, and if not, raise an exception. My question is, where is the best place to define this16:28
rybridges1supported_filters list? do we have a place to define constants somewhere? I want to avoid initializing a new list every time the method is called16:28
jrollrybridges1: I'd probably define it in all caps at the top of the module16:29
jrollSUPPORTED_FILTERS = ['foo', 'bar']16:29
rybridges1okay sounds good16:30
*** baoli has quit IRC16:31
*** bks has joined #openstack-ironic16:32
openstackgerritMerged openstack/ironic: Add validation of 'ilo_deploy_iso' in deploy.validate()  https://review.openstack.org/33477416:34
*** baoli has joined #openstack-ironic16:34
*** lucas-hungry is now known as lucasagomes16:35
*** vmud213 has joined #openstack-ironic16:37
openstackgerritMerged openstack/ironic-lib: Fix creating config drive for whole disk images  https://review.openstack.org/37489716:37
*** kumarb has joined #openstack-ironic16:37
*** vmud213 has quit IRC16:37
*** vmud213 has joined #openstack-ironic16:37
* jlvillal wonders if https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L84-L89 is also wrong16:38
* jlvillal realizes that no it isn't. He sees the [0] at the end16:38
NobodyCamGood Morning Ironic'ers16:39
jlvillalGood morning NobodyCam :)16:39
NobodyCammorning jlvillal :)16:39
lucasagomesNobodyCam, jlvillal morning16:39
jlvillallucasagomes: Oi!16:39
lucasagomes:D16:39
openstackgerritMichael Turek proposed openstack/python-ironicclient: Refactor provision state so all actions can use inherited take_action  https://review.openstack.org/36455216:40
NobodyCammorning lucasagomes :)16:40
*** derekh has quit IRC16:40
*** bks has quit IRC16:40
*** zigo has quit IRC16:41
vdrokgood morning NobodyCam and jlvillal !16:41
NobodyCammorning vdrok :)16:41
jlvillalvdrok: Privyet!16:41
vdrokjlvillal: how many languages do you know??? :D16:42
rloojlvillal: feel free to update that code in ironic-lib to make it clearer.16:42
* jlvillal always in Rosetta Stone when he tries to say Zdravstvuyte. It always says, "try again" :(16:42
jlvillals/always in/always fails in/16:42
NobodyCammorning rloo :)16:42
krtaylorrajinir, you can add MoltenIron to your pre test hook script, right after getting a VM from nodepool16:43
jlvillalvdrok: I know how to say hello in a few languages. I only speak one language.16:43
rloomorning NobodyCam16:43
krtaylorrajinir, a template, hm...16:43
vdrokjlvillal: :)16:43
rajinirkrtaylor: Do you have a sample of the pre_hook?16:43
*** mjturek has joined #openstack-ironic16:44
jlvillalrloo: This might be wrong: https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L622-L63316:44
jlvillalThe return at the end16:45
jlvillalBecause I'm thinking output is a tuple16:45
openstackgerritPeter Piela proposed openstack/ironic-ui: Minor state-machine fixes/updates  https://review.openstack.org/37497416:45
rloojlvillal: yup, its always going to return false16:45
krtaylorrajinir, I can get that - we should add that to the README also, that would be good info16:45
jlvillalrloo: I think so16:45
rajinirkrtaylo: Thanks, if you get that, I can add it to readme16:46
rloojlvillal: good catch. which is why i think we should change the usage of the return value from utils.execute().16:46
krtaylormjturek, hamzy - rajinir is looking for info on how to use molteniron in a test environment - how to add it in a prehook script16:46
rloojlvillal: to indicate (out, err) is returned16:47
jlvillalrloo: Update the doc string?16:47
mjtureksure, rajinir do you have a pre-test-hook for your ironic job?16:47
manjeetshello ironic folks I have a devstack one node setup with ironic16:48
rloojlvillal: oh, i didn't look at the doc string. i mean wherever a utils.execute() is called.16:48
manjeetsi am trying to boot nova instances and having a issue16:48
jlvillalrloo: Ah. Okay16:48
*** vsaienko has quit IRC16:48
rloojlvillal: but if docstring isn't clear, should fix that too.16:48
rajinirmjturek: I have a pre-test-hook, yes16:48
jlvillalrloo: The unit tests pass on that because the test value is:  ('gpt', '')16:49
manjeetspreviously i thought it may be when ironic wait for neutron dhcp port to get ready but i increased wait time to 30 and found no error trace in neutron log16:49
mjturekcool, rajinir let me get you a paste of how we use it in ours16:49
jlvillalrloo: I don't think it will pass with ('gpt hello', '')16:49
manjeetsfound one error trace in ironic log16:49
manjeetshttp://paste.openstack.org/show/582624/16:49
rloojlvillal: yup, it won't pass if 'gpt' is a substring... i have no idea what is returned from that call.16:50
rloojlvillal: but doing 'gpt' in output indicates to me that they were looking for it as a substring or in a list.16:50
jlvillalrloo: I think so. Testing the blkid command to see what it returns16:51
*** zigo has joined #openstack-ironic16:51
*** zigo is now known as Guest1865616:52
jlvillalrloo: It might actually work. On my system it outputs: 'dos'16:52
manjeetsis that something going wrong with image ?16:52
rloojlvillal: then it should be 'gpt' == output[0]16:53
jlvillalrloo: Agreed. I'll fix. But luckily I don't think it was broken. More like accidentally correct :)16:53
*** vmud213 has quit IRC16:54
vdrokmanjeets: it seems like the image cannot be downloaded from glance/swift16:54
mjturekrajinir: http://paste.openstack.org/show/582628/ here's how we clone MI, call it, then extract the info into the ipmi_info file and localrc16:54
vdrokare you sure that swift account is correctly set?16:54
*** vmud213 has joined #openstack-ironic16:54
*** ohamada has quit IRC16:54
vdrokin the ironic config file16:54
rloojlvillal: hard to say whether it was accidentally correct or not; the person that coded may have known it was (output, err) and coded it that way.16:54
jlvillalrloo: Yeah16:55
rloojlvillal: which reminds me of folks saying 'but it works' when commenting that their code isn't that great ...16:55
vdrokmanjeets: and is swift running at all?16:55
manjeetsI think I got the issue swift end point is not discoverable16:56
manjeetsvdrok, thanks16:56
*** rama_y has joined #openstack-ironic16:56
*** Guest18656 has quit IRC16:56
mjturekrajinir: feel free to ping me if you have any questions about it (or MI in general :) )16:56
vdrokaha, np :)16:56
gmmahaTheJulia: jlvillal: cinerama: created a RFE request for the bifrost release lockdown https://bugs.launchpad.net/bifrost/+bug/162665316:56
openstackLaunchpad bug 1626653 in bifrost "[RFE]: Lock down all components of bifrost on release branches" [Undecided,New]16:56
gmmahaplease let me know your thoughts16:56
rajinirmjturek: Thank you , will look at it and get back if I have any trouble16:56
jlvillalvdrok: How did you figure out it is swift from the pastebin? I'm looking and seeing glance in the paste. Just curious to improve my logfile parsing :)16:57
vdrokjlvillal: port 8080 and the structure of the url, it is a swift url16:57
jlvillalvdrok: Okay. Thanks :)16:57
vdrokglance is the default swift container for images iirc16:58
vdrokname of the container I mean16:58
vdrokyw :)16:58
jlvillalAh16:58
*** zigo_ has joined #openstack-ironic16:59
manjeetsvdrok, thanks It'll help me in parsing next time as well16:59
vdrok:)16:59
xavierrhey vdrok, could you look at https://review.openstack.org/#/c/367065/ ? :)17:01
patchbotpatch 367065 - ironic - Add inspection feature for the OneView drivers17:01
*** causten has quit IRC17:01
TheJuliagmmaha: So I've been pondering this and I think partially the thing to do is to provide a "override and default to x" functionality, which gets auto-set by whatever branch zulu says is being worked on17:01
vdrokxavierr: aw, right, I meant to and forgot, will do today.17:02
openstackgerritMerged openstack/bifrost: Explain how to pronounce Bifrost  https://review.openstack.org/37157917:02
xavierrhahaha awesome vdrok17:02
gmmahaTheJulia: the mechanism is independent of zuul right?17:03
gmmahathat way anyone who wants to use an older version of bifrost in their production env can do the same17:04
TheJuliagmmaha: would be independent, but we could also trigger off it for CI automatically17:05
*** causten has joined #openstack-ironic17:06
gmmahaTheJulia: that sounds workable. Guessing it will be an env variable that ansible parses to choose one or the other? for all the source based dependencies, we can definitely set the branch name and use that for choosing them.17:07
*** ralonsoh_ has quit IRC17:07
gmmahafor cases will the coreos ipa pre-built image...17:07
*** zigo_ has quit IRC17:07
gmmahawell, we can do the same and parse the data to change the file it looks for17:07
*** causten has quit IRC17:08
TheJuliagmmaha: one slight problem, dib, at least git wise, does not stable branch afaik17:08
jrolljlvillal: rloo:17:08
jroll$ sudo blkid -p -o value -s PTTYPE /dev/sda17:08
jrollgpt17:08
jrollseems right :)17:08
gmmahaTheJulia: neither does shade, glean etc. :P17:08
rloojroll: ok, then the code should be changed to something like 'gpt' == output[0]17:08
jrollyeah, that's possible17:09
jlvillaljroll: Yeah...  I might make it more explicit17:09
jrollyeah, that's fine17:09
rloojroll: otherwise, it is confusing. if stderr was 'gpt' for whatever reason... :-(17:09
jlvillaljroll: Is there a trailing linefeed on it?17:09
*** rbudden has quit IRC17:09
rloojroll: i thought maybe they did that cuz sometimes the return value (depending on os) might be eg 'gpt and foobar'17:10
*** e0ne has quit IRC17:10
* jlvillal is unsure if execute strips the ending linefeed or not.17:10
jrolljlvillal: I mean, in the cli, yes but idk about execute()17:10
*** zigo_ has joined #openstack-ironic17:11
rloojlvillal: in this case, assuming it works, it must have stripped any linefeed. otherwise it would have been 'false', not 'true'.17:11
rloojlvillal: i'm fine if we just change it to 'gpt' in output[0]17:11
jlvillalI might just do:  if 'gpt' in stdout:17:11
jlvillalrloo: Jinx :P17:12
rloojlvillal: yeah, that :)17:12
*** causten has joined #openstack-ironic17:12
jrolljlvillal: rloo: I'd ask lucasagomes or TheJulia about it, they'll probably have answers for you17:12
* lucasagomes reads17:13
lucasagomesrloo, hi there17:13
jrolle.g. they'll be able to tell you if there's a mystical 'gptplus' or something17:13
jrollwhich won't work :P17:13
TheJuliaanswers?17:13
rlooTheJulia: 42?17:13
*** vsaienko has joined #openstack-ironic17:13
jrollTheJulia: https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L622-L63317:13
jrollshould that be "gpt in stdout" or "gpt == stdout"17:13
jlvillalTheJulia: lucasagomes :  https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L622-L63317:13
jrollit is currently the latter due to that return value being a tuple17:14
* jlvillal realizes jroll was too fast...17:14
jrollso would 'gpt' in output[0] break things or whatever17:14
* jroll not an fs expert17:14
jlvillalAlso we weren't sure when execute runs does it return 'gpt\n' or just 'gpt'17:14
lucasagomesjlvillal, the command seems alright17:15
jlvillalOn the CLI when I redirect to a file I get: 'gpt\n'17:15
* TheJulia moves the cat to type a proper reply17:15
lucasagomeshttp://paste.openstack.org/show/582631/17:15
lucasagomesoh17:15
lucasagomesjroll, ++ indeed due the tuple17:15
lucasagomesperhaps we just need a output, err = execute(...)17:16
*** ijw has joined #openstack-ironic17:16
jlvillalYeah. and then maybe output.strip() == 'gpt' ??17:16
*** causten has quit IRC17:16
lucasagomesjlvillal, that works as well17:16
jlvillalOr: 'gpt' in output17:16
lucasagomesfwiw "in" will also work even if it's gpt\n17:16
TheJuliaso my understanding is that in will search for 'gpt' in output, so it would always be false, and a \n shouldn't matter17:16
TheJuliawell, always be false when not present17:16
jlvillalTheJulia: In the current case 'output' is a tuple of (stdout, stderr)17:17
*** ChubYann has joined #openstack-ironic17:17
TheJuliaugh17:17
TheJuliayeah, so yeah, we need to collect error separately17:17
TheJulia:(17:17
* jlvillal foolishly started yak-shaving this morning17:18
jrollI guess what I was wondering is if it'd be possible for blkid to return something with gpt IN the output, but also something else in the output, where we didn't want to run that code17:18
lucasagomesgood catch who saw that btw17:18
jrolle.g. a gptplus or something I'm not aware of17:18
jlvillallucasagomes: I was motivated by dtantsur|afk fix to check the usage of execute.17:18
TheJuliajroll: I guess that would require digging through the blkid code :\17:19
jrollTheJulia: well, was thinking about obscure filesystems more than auditing any possible output :)17:20
lucasagomesjroll, not sure if there's any other that matches GPT but, as a safe-guard it would be resonable to lower().strip() and check if it's "gpt"17:21
*** trown is now known as trown|lunch17:21
jrollyeah, I think so too17:21
jlvillalOkay. I'll run with that. Thanks.17:21
jrollthank YOU!17:21
jlvillalGoing to just do a quick test for my curiosity if we have a trailing \n or not.17:21
TheJuliaI think it is relatively safe doing that, of course we could always try opening the block device and reading the magical bits out... and I'll be quiet before you guys start tossing wet cats at me17:22
jlvillalTheJulia: Just add water to your cat for us ;)17:22
TheJuliajlvillal: ugh, the claw marks...17:23
openstackgerritCharlle Daniel proposed openstack/ironic: --amend  https://review.openstack.org/37498517:23
jlvillalThat doesn't look right...17:23
TheJuliaNo... it does not look right17:24
mat128|bbllmao17:24
*** mat128|bbl is now known as mat12817:24
mat128git commit -m '--amend' ?17:24
* jlvillal is sure he has done much worse :)17:24
TheJuliamat128: likely :)17:25
*** causten has joined #openstack-ironic17:26
jlvillal ./tt.py17:26
jlvillalStdoutStderr(stdout='dos\n', stderr='')17:26
jlvillalIt was broken :(17:26
*** sabeen has quit IRC17:26
*** vmud213 has quit IRC17:27
*** rbudden has joined #openstack-ironic17:28
*** joprovost has quit IRC17:33
openstackgerritRicardo Araújo Santos proposed openstack/ironic: Fixes wrong steps to perform migration of nodes  https://review.openstack.org/37484117:33
openstackgerritLucas Alvares Gomes proposed openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode  https://review.openstack.org/37498817:33
lucasagomeso/17:33
openstackgerritVladyslav Drok proposed openstack/ironic: Avoid name errors in oneview periodics  https://review.openstack.org/37498917:34
lucasagomesthe only missing bits is the cirros release, then I thin we are good with UEFI in the gate17:34
vdrokxavierr: ^^ you might want to look at that one17:34
*** mgould is now known as mgould|afk17:35
*** causten has quit IRC17:36
openstackgerritCharlle Daniel proposed openstack/ironic: Implements the validation to verify if the node is free or in use by Ironic  https://review.openstack.org/37276217:38
*** causten has joined #openstack-ironic17:42
kumarbhttps://review.openstack.org/#/c/37247817:42
patchbotpatch 372478 - ironic-python-agent - Moving Reboot bashscript to python17:42
*** aNuposic has joined #openstack-ironic17:43
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Fix check for GPT partioned device  https://review.openstack.org/37499517:44
jlvillalrloo: jroll lucasagomes TheJulia: ^^^  In regards to earlier discussion17:44
lucasagomesjlvillal, cheers will take a look17:45
vdrokjroll: lucasagomes https://review.openstack.org/374913 seems to pass, rechecked a couple of times17:45
patchbotpatch 374913 - ironic - Increase timeout for migration-related tests17:45
jrollvdrok: +217:46
lucasagomesvdrok, looking17:46
lucasagomesjlvillal, looks good, 1 nit17:46
*** causten has quit IRC17:46
lucasagomesbut not needed to fix unless you submit a new patch-set17:46
jlvillallucasagomes: looking...17:46
*** causten has joined #openstack-ironic17:47
rajinirmjturek:Basic question on MI, how to add the nodes? Is it via command line?17:47
jlvillallucasagomes: I know they aren't needed, but I like them :)  I like parenthesis to make things explicit at times. Not sure what others think.17:47
jlvillallucasagomes: If I re-spin I can remove17:47
lucasagomesjlvillal, yeah no worries, just pointing that they are useless there17:48
lucasagomesbut it's ok :-)17:48
jroll+2 for jlvillal as well17:48
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: Add py35 to tox environments  https://review.openstack.org/37500017:48
jlvillalThanks!17:48
lucasagomesvdrok, looks good... I see you rechecked twice and it passed. I mean, 2x doesn't guarantee it's fixed because it's a tricky problem17:48
lucasagomesbut I guess we will only find out with time17:48
jrolland another +217:49
jrollwhee17:49
jrollthis is fun17:49
*** causten_ has joined #openstack-ironic17:49
vdrokyup, I can't reproduce it locally :(17:49
vdrokthank you!17:49
lucasagomessomeone wants to look at #374913 before I +a it ?17:49
rloojlvillal: so it was actually broken? if so, do we need a release note in ironic ?17:49
jrolllucasagomes: gopher it17:50
jlvillalrloo: Good question. I'm not sure. I feel pretty confident it didn't work based on my test17:50
lucasagomesjroll, done!17:50
jlvillal>>> a = ('gpt\n', '')17:50
jlvillal>>> 'gpt' in a17:50
jlvillalFalse17:50
jrollwoo17:50
jlvillaljroll, rloo: Not sure about a release note. What do you think?17:51
jrolljlvillal: can't hurt, kind of a big bug17:51
lucasagomesrloo, I don't think we have reno for ironic-lib17:51
rloojlvillal: i guess it depends on what 'feature' broke cuz of that.17:51
jlvillaljroll: I assume it goes into openstack/ironic17:51
*** causten has quit IRC17:51
*** causten__ has joined #openstack-ironic17:51
jrolloh right17:51
lucasagomesit's not a user facing lib remember ?17:51
rloolucasagomes: yeah, no reno for ironic-lib, that's fine, BUT presumably we have ironic code using that.17:51
lucasagomesrloo, true17:52
rloolucasagomes: so i'm guessing? that something is broke in ironic cuz of that. something to do with gpt...17:52
lucasagomesrloo, indeed, a release note in ironic wouldn't hurt17:52
rloolucasagomes: if nothing is broken in ironic or ipa cuz of that, then no reno is needed.17:52
rloolucasagomes, jlvillal: so the reno can't be added until the requirements is up'd to use the new lib, but we could open a bug or add ironic project to the bug that jlvillal opened17:53
*** causten_ has quit IRC17:53
mjturekrajinir: yep! 'molteniron add'17:53
jlvillalOh yeah. I was thinking, I'll just do a depends-on. But that isn't enough.17:54
lucasagomesrloo, looking at the code apparently it iwll affect whole disk image with GPT17:54
jlvillalHave to wait for ironic-lib to be released and requirements updated...17:54
mjturekrajinir: that'll give you a set of args to use17:54
rajinirmjturek>: thanks17:54
*** e0ne has joined #openstack-ironic17:54
rajinirmjturek>: What happens if all nodes are in use, is there a wait?17:54
rloojlvillal: right17:55
*** [1]rpioso has joined #openstack-ironic17:55
mjturekrajinir: it'll return a 400 error  and you can handle accordingly. Currently we fail if all nodes are in use, but theoretically that shouldn't happen as we allow N concurrent jobs for N nodes. Make sense?17:56
rajinirmjturek> Yep that's what I think will work. Makes sense17:56
xavierrvdrok: thank you! want some help on that?17:57
vdrokxavierr: feel free to take over if you want :) I can maintain that too, but it's not a top priority for me17:58
*** moshele has joined #openstack-ironic17:58
jlvillalvdrok: Thanks for fixing the migration unit test error thingy :)18:00
vdrokjlvillal: we'll need some time to ensure that it's actually fixed :)18:00
*** vsaienko has quit IRC18:00
*** rpioso has quit IRC18:01
*** [1]rpioso has quit IRC18:02
jlvillalvdrok: Yeah, that too :)18:02
rloovdrok: do you know what the original time out was?18:03
vdrokrloo: the default timeout for all our unittests is 60 seconds18:03
*** kumarb has quit IRC18:03
rloovdrok: ok (I suppose I should have asked to put that in the commit, too late now). So we should keep an eye after that merges, to see if we get timeouts18:03
vdrokyep18:04
rloothx vdrok!18:04
vdrokyou're all welcome, that was not that hard to fix :)18:05
*** xhku_ has quit IRC18:06
*** xhku has joined #openstack-ironic18:07
*** Romanenko_K has quit IRC18:09
*** ig0r_ has quit IRC18:09
jrollfwiw, seems there's also a sqlalchemy fix coming for it18:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses  https://review.openstack.org/37504518:17
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Add py35 to tox environments  https://review.openstack.org/37504618:18
milanGood night Ironic! /me gone18:18
lucasagomesmilan, have a great evening18:19
jlvillallucasagomes: milan Good night18:19
jlvillalOh, lucasagomes is still here :)18:19
lucasagomesjlvillal, 15:00 pm for me yet heh18:20
lucasagomesyeah18:20
xavierrnight milan18:21
*** milan has quit IRC18:23
rajinirmjturek: I see this error when I try to add {"status": 400, "message": "No module named MySQLdb"}. Am I missing soomething?18:24
*** trown|lunch is now known as trown18:24
*** ayoung has quit IRC18:25
mjturekrajinir: looks like you need MySQL-python18:25
rajinirmjturek> worked, that was it. Thx18:28
*** causten__ has quit IRC18:29
*** causten has joined #openstack-ironic18:29
openstackgerritVladyslav Drok proposed openstack/ironic: Add more details to MIGRATIONS_TIMEOUT note  https://review.openstack.org/37504818:30
*** causten_ has joined #openstack-ironic18:31
openstackgerritVladyslav Drok proposed openstack/ironic: Add more details to MIGRATIONS_TIMEOUT note  https://review.openstack.org/37504818:31
vdroktime to leave for me, good night everyone!18:32
openstackgerritJulia Kreger proposed openstack/bifrost: DNM: Test timeout extension for inspection  https://review.openstack.org/37505018:33
*** causten has quit IRC18:34
xavierrgood night vdrok :)18:36
xavierrty18:36
lucasagomesvdrok, g'night!18:41
NobodyCamnight vdrok18:41
NobodyCammorning xavierr18:41
NobodyCammorning lucasagomes :)18:41
lucasagomesNobodyCam, hi there :D18:42
NobodyCam:) late day for you lucasagomes :)18:42
lucasagomesNobodyCam, not really, I'm in brazil at the moment18:43
lucasagomes15:43pm yet :-)18:43
NobodyCamahh :) nice :)18:43
rajinirmjturek: Can you explain how allocation_pool works18:44
*** baoli has quit IRC18:44
openstackgerritMerged openstack/ironic: Change the logic of selecting image for tests  https://review.openstack.org/32962518:45
*** baoli has joined #openstack-ironic18:45
*** baoli_ has joined #openstack-ironic18:46
openstackgerritMerged openstack/ironic: Increase timeout for migration-related tests  https://review.openstack.org/37491318:46
openstackgerritMerged openstack/ironic: Update reno index for Newton  https://review.openstack.org/37490418:46
openstackgerritDmitry Tantsur proposed openstack/ironic: Change the logic of selecting image for tests  https://review.openstack.org/37505518:46
xavierrNobodyCam: morning/afternoon :)18:47
NobodyCam:)18:48
xavierrlucasagomes: sao paulo?18:48
*** baoli has quit IRC18:50
openstackgerritCharlle Daniel proposed openstack/ironic: Implements the validation to verify if the node is free or in use by  https://review.openstack.org/37276218:50
rajinirmjturek: can I specify like this for allocation pool start=x.x.x.x,end=x.x.x.x.x?18:51
rajinir:q18:51
*** e0ne has quit IRC18:53
lucasagomesxavierr, yes são paulo state. The city is called Bauru18:53
lucasagomesxavierr, it's the city I grew up in18:54
rajinirmjturek; I see it is comma separated18:54
rajinirthanks18:54
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438118:55
*** _mbelanger has joined #openstack-ironic18:55
mjtureksorry rajinir was afk :) all good?18:55
xavierrlucasagomes: awesome and welcome back :)18:56
lucasagomesxavierr, cheers! Yeah, it's good to be here18:56
rajinirmjturek: good. figured it out by looking at the code18:56
*** e0ne has joined #openstack-ironic18:57
*** e0ne has quit IRC18:57
*** arahal_ has quit IRC18:58
*** [1]rpioso has joined #openstack-ironic18:59
*** vsaienko has joined #openstack-ironic18:59
*** joprovost has joined #openstack-ironic19:00
*** absubram has joined #openstack-ironic19:01
*** absubram has quit IRC19:03
*** zackf has quit IRC19:03
*** [1]cdearborn has joined #openstack-ironic19:05
openstackgerritCharlle Daniel proposed openstack/ironic: Implements the validation to verify if the node is free or in use by  https://review.openstack.org/37276219:06
*** absubram has joined #openstack-ironic19:06
*** e0ne has joined #openstack-ironic19:09
*** e0ne has quit IRC19:11
*** daemontool has quit IRC19:12
mjturekgreat :)19:12
*** daemontool has joined #openstack-ironic19:13
*** cdearborn has quit IRC19:15
openstackgerritCharlle Daniel proposed openstack/ironic: Adds to the documentation a note about the state changing of a node by  https://review.openstack.org/37273519:16
*** daemontool_ has joined #openstack-ironic19:18
*** daemontool has quit IRC19:19
*** sabeen has joined #openstack-ironic19:19
*** devananda is now known as devananda|lunch19:20
*** daemontool_ has quit IRC19:24
*** daemontool_ has joined #openstack-ironic19:25
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438119:25
*** bdeschenes has quit IRC19:29
*** zigo_ is now known as zigo19:30
*** jvgrant has quit IRC19:31
*** jvgrant has joined #openstack-ironic19:31
openstackgerritHugo Nicodemos proposed openstack/ironic: Reusing oneview_client when possible  https://review.openstack.org/35804119:32
*** moshele has quit IRC19:33
*** cdearborn has joined #openstack-ironic19:35
xavierris possible to delete an account from gerrit?19:47
TheJuliaThat is a really good question...19:48
*** [1]cdearborn has quit IRC19:49
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438119:49
xavierrI have a colleaguethat created it twice, so we want to delete one...19:49
xavierrs/colleaguethat/colleague that/19:49
*** srobert has quit IRC19:53
*** ijw has quit IRC19:54
*** ijw has joined #openstack-ironic19:58
openstackgerritJulia Kreger proposed openstack/bifrost: Add new lines to improve pre-install yaml files  https://review.openstack.org/37508420:01
lucasagomesour jobs does run on ubuntu 16.04 yet ?20:02
lucasagomesgate jobs*20:02
*** ijw has quit IRC20:03
*** jrcloud has joined #openstack-ironic20:05
TheJulialucasagomes: I think I saw a change merge in infra earlier for one of the nv jobs20:07
lucasagomesTheJulia, I see, strange yeah it seems to be 16.04 but http://logs.openstack.org/88/374988/1/check/gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-xenial-nv/e9c264d/logs/devstacklog.txt.gz#_2016-09-22_18_03_07_20420:08
lucasagomesovmf should be there hmm20:08
lucasagomeshttp://packages.ubuntu.com/xenial/ovmf20:09
lucasagomesmaybe that multiverse repo is disabled ?20:09
TheJulialucasagomes: at least on my xenial virtual box machine, it is not enabled by default20:11
lucasagomesTheJulia, yeah, that could be it I guess20:11
lucasagomesTheJulia, it fails when you try to apt-get install ovmf right?20:12
openstackgerritHugo Nicodemos proposed openstack/ironic: Add support for the OneView audit logging  https://review.openstack.org/33775120:12
TheJuliaone moment20:12
lucasagomescause I don't remember enabling this repo on mine ... strange20:12
TheJulialucasagomes: nope... it worked20:12
lucasagomesouch...20:12
TheJuliainteresting20:12
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/37374820:13
lucasagomesTheJulia, will ask on -infra20:14
openstackgerritAnnie Lezil proposed openstack/ironic: Add description for vendor passthru methods  https://review.openstack.org/37046920:15
TheJuliaI kind of wonder if it is just missing on the vanilla infra cloud mirror20:16
openstackgerritXavier proposed openstack/python-oneviewclient: Remove cpus validation due inconsistency with inspection cpus count  https://review.openstack.org/37508920:17
lucasagomesTheJulia, indeed, it should be something silly like that20:18
lucasagomesTheJulia, http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/files/sources.list.xenial20:23
lucasagomesseems to be enabled... hmm20:24
*** Goneri has quit IRC20:28
*** ijw has joined #openstack-ironic20:28
*** absubram_ has joined #openstack-ironic20:32
*** bfournie has quit IRC20:33
*** absubram has quit IRC20:33
*** absubram_ is now known as absubram20:33
rajinirhttps://www.irccloud.com/pastebin/OlAAbDzu/MIError20:33
rajinirmjturek: https://www.irccloud.com/pastebin/OlAAbDzu/MIError20:34
rajinirmjturek: may be I don't have the dsvm_uuid set20:34
*** hamzy has quit IRC20:36
*** baoli_ has quit IRC20:40
*** rpioso has joined #openstack-ironic20:41
lucasagomesgoing to call it a day and continue to investigate tomorrow20:41
lucasagomeshave a great evening all, talk to you tomorrow!20:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Local boot with partition images"  https://review.openstack.org/36977620:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections  https://review.openstack.org/36977720:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import Install and configure sections  https://review.openstack.org/36798820:41
mat128o/ lucasagomes20:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Bare Metal service overview"  https://review.openstack.org/36632920:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Image requirements"  https://review.openstack.org/36975420:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "integration with other OpenStack components"  https://review.openstack.org/36951020:41
openstackgerritMathieu Mitchell proposed openstack/ironic: Update developer's guide "Installation Guide" link  https://review.openstack.org/37429620:41
openstackgerritMathieu Mitchell proposed openstack/ironic: Fix absolute links to install-guide.rst in developer docs  https://review.openstack.org/37430320:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Flavor creation"  https://review.openstack.org/36976120:41
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service"  https://review.openstack.org/37427220:41
openstackgerritMathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide  https://review.openstack.org/36980220:42
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image"  https://review.openstack.org/36980020:42
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Trusted boot with partition image"  https://review.openstack.org/36980120:42
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import configdrive  https://review.openstack.org/36979820:42
*** lucasagomes is now known as lucas-afk20:42
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances"  https://review.openstack.org/36979920:42
openstackgerritMathieu Mitchell proposed openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints  https://review.openstack.org/36979720:42
lucas-afkmat128, heh! See ya20:42
*** [1]rpioso has quit IRC20:44
*** ijw has quit IRC20:48
openstackgerritJohn L. Villalovos proposed openstack/ironic: Have bashate run for entire project  https://review.openstack.org/37426920:49
openstackgerritJohn L. Villalovos proposed openstack/ironic: Have bashate run for entire project  https://review.openstack.org/37426920:56
*** rama_y has quit IRC20:58
*** wajdi_ has joined #openstack-ironic21:08
*** vsaienko has quit IRC21:10
*** wajdi has quit IRC21:11
jlvillaljroll: TheJulia rloo: So this has four +2 votes. Does someone want to approve it?  https://review.openstack.org/#/c/374995/21:13
patchbotpatch 374995 - ironic-lib - Fix check for GPT partioned device21:13
*** wajdi_ has quit IRC21:13
jlvillalNobodyCam: ^^ :)21:13
jlvillalYou could be #5 :)21:13
*** wajdi has joined #openstack-ironic21:13
* TheJulia pushed the button :)21:14
jrollheh21:14
NobodyCamahh21:14
jlvillal:)21:14
* TheJulia hopes it was not a red button21:14
NobodyCam:p21:14
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/37374821:17
*** wajdi has quit IRC21:18
*** jheroux has quit IRC21:25
*** rcernin has quit IRC21:34
mrdaMorning Ironic21:34
*** rpioso has quit IRC21:36
*** bradjones has quit IRC21:37
*** vsaienk0 has quit IRC21:37
*** strigazi has quit IRC21:37
*** amitkqed has quit IRC21:37
*** aries_ has quit IRC21:37
*** jhesketh has quit IRC21:37
*** daemontool_ has quit IRC21:37
*** rbudden has quit IRC21:37
*** amotoki has quit IRC21:37
*** mjturek has quit IRC21:37
*** jlvillal has quit IRC21:37
*** mikal has quit IRC21:37
*** _mbelanger has quit IRC21:37
*** athomas has quit IRC21:37
*** AlexeyAbashkin has quit IRC21:37
*** aslezil has quit IRC21:37
*** slagle has quit IRC21:37
*** agordeev has quit IRC21:37
*** kozhukalov has quit IRC21:37
*** greghaynes has quit IRC21:37
*** boris-42 has quit IRC21:37
*** izaakk has quit IRC21:37
*** ildikov has quit IRC21:37
*** skazi has quit IRC21:37
*** kragniz has quit IRC21:37
*** purp has quit IRC21:37
*** mgould|afk has quit IRC21:37
*** melwitt has quit IRC21:37
*** hwoarang has quit IRC21:37
*** sergek has quit IRC21:37
*** betherly has quit IRC21:37
*** vdrok has quit IRC21:37
*** morgabra has quit IRC21:37
*** UForgotten has quit IRC21:37
*** sabeen has quit IRC21:37
*** vishwanathj has quit IRC21:37
*** caiobo has quit IRC21:37
*** stevemar has quit IRC21:37
*** mandre has quit IRC21:37
*** kirankv has quit IRC21:37
*** ishant|away has quit IRC21:37
*** lazy_prince has quit IRC21:37
*** jcook has quit IRC21:37
*** eglute has quit IRC21:37
*** alineb has quit IRC21:37
*** gmmaha has quit IRC21:37
*** causten_ has quit IRC21:37
*** zzzeek has quit IRC21:37
*** rajinir has quit IRC21:37
*** syed_ has quit IRC21:37
*** rloo has quit IRC21:37
*** ccamacho|afk has quit IRC21:37
*** krtaylor has quit IRC21:37
*** ppiela has quit IRC21:37
*** jroll has quit IRC21:37
*** TheJulia has quit IRC21:37
*** yarkot has quit IRC21:37
*** sturivnyi_ has quit IRC21:37
*** clif_h has quit IRC21:37
*** amit213 has quit IRC21:37
*** zhenguo has quit IRC21:37
*** pshige______ has quit IRC21:37
*** mgkwill has quit IRC21:37
*** evgenyl has quit IRC21:37
*** yonglihe has quit IRC21:37
*** fyxim has quit IRC21:37
*** sirushti has quit IRC21:37
*** clenimar has quit IRC21:37
*** Mr_T has quit IRC21:37
*** notmorgan has quit IRC21:37
*** HenryG has quit IRC21:37
*** tdurakov has quit IRC21:37
*** natorious has quit IRC21:37
*** adreznec has quit IRC21:37
*** mgagne has quit IRC21:37
*** _mbelanger has joined #openstack-ironic21:38
*** athomas has joined #openstack-ironic21:38
*** AlexeyAbashkin has joined #openstack-ironic21:38
*** aslezil has joined #openstack-ironic21:38
*** slagle has joined #openstack-ironic21:38
*** agordeev has joined #openstack-ironic21:38
*** kozhukalov has joined #openstack-ironic21:38
*** greghaynes has joined #openstack-ironic21:38
*** boris-42 has joined #openstack-ironic21:38
*** izaakk has joined #openstack-ironic21:38
*** ildikov has joined #openstack-ironic21:38
*** skazi has joined #openstack-ironic21:38
*** kragniz has joined #openstack-ironic21:38
*** purp has joined #openstack-ironic21:38
*** mgould|afk has joined #openstack-ironic21:38
*** melwitt has joined #openstack-ironic21:38
*** hwoarang has joined #openstack-ironic21:38
*** sergek has joined #openstack-ironic21:38
*** betherly has joined #openstack-ironic21:38
*** vdrok has joined #openstack-ironic21:38
*** morgabra has joined #openstack-ironic21:38
*** UForgotten has joined #openstack-ironic21:38
*** causten_ has joined #openstack-ironic21:40
*** zzzeek has joined #openstack-ironic21:40
*** rajinir has joined #openstack-ironic21:40
*** rloo has joined #openstack-ironic21:40
*** ccamacho|afk has joined #openstack-ironic21:40
*** krtaylor has joined #openstack-ironic21:40
*** ppiela has joined #openstack-ironic21:40
*** jroll has joined #openstack-ironic21:40
*** TheJulia has joined #openstack-ironic21:40
*** yarkot has joined #openstack-ironic21:40
*** sturivnyi_ has joined #openstack-ironic21:40
*** clif_h has joined #openstack-ironic21:40
*** amit213 has joined #openstack-ironic21:40
*** zhenguo has joined #openstack-ironic21:40
*** pshige______ has joined #openstack-ironic21:40
*** yonglihe has joined #openstack-ironic21:40
*** fyxim has joined #openstack-ironic21:40
*** sirushti has joined #openstack-ironic21:40
*** Mr_T has joined #openstack-ironic21:40
*** clenimar has joined #openstack-ironic21:40
*** notmorgan has joined #openstack-ironic21:40
*** HenryG has joined #openstack-ironic21:40
*** natorious has joined #openstack-ironic21:40
*** adreznec has joined #openstack-ironic21:40
*** mgagne has joined #openstack-ironic21:40
*** ildikov has quit IRC21:40
*** zhenguo has quit IRC21:40
*** fyxim has quit IRC21:40
*** rpioso has joined #openstack-ironic21:40
*** bradjones has joined #openstack-ironic21:40
*** vsaienk0 has joined #openstack-ironic21:40
*** strigazi has joined #openstack-ironic21:40
*** amitkqed has joined #openstack-ironic21:40
*** aries_ has joined #openstack-ironic21:40
*** serverascode has quit IRC21:41
*** cppforlife_ has quit IRC21:41
*** jhesketh has joined #openstack-ironic21:41
*** daemontool_ has joined #openstack-ironic21:41
*** rbudden has joined #openstack-ironic21:41
*** mjturek has joined #openstack-ironic21:41
*** amotoki has joined #openstack-ironic21:41
*** jlvillal has joined #openstack-ironic21:41
*** mikal has joined #openstack-ironic21:41
*** agordeev has quit IRC21:41
*** boris-42 has quit IRC21:41
*** daemontool_ has quit IRC21:42
*** sabeen has joined #openstack-ironic21:42
*** vishwanathj has joined #openstack-ironic21:42
*** caiobo has joined #openstack-ironic21:42
*** mandre has joined #openstack-ironic21:42
*** stevemar has joined #openstack-ironic21:42
*** ishant|away has joined #openstack-ironic21:42
*** kirankv has joined #openstack-ironic21:42
*** lazy_prince has joined #openstack-ironic21:42
*** jcook has joined #openstack-ironic21:42
*** eglute has joined #openstack-ironic21:42
*** alineb has joined #openstack-ironic21:42
*** gmmaha has joined #openstack-ironic21:42
*** vdrok has quit IRC21:43
*** TheJulia has quit IRC21:43
*** amit213 has quit IRC21:44
*** rajinir has quit IRC21:44
*** kozhukalov has quit IRC21:44
*** betherly has quit IRC21:44
*** yonglihe has quit IRC21:45
*** agordeev has joined #openstack-ironic21:49
*** tdurakov has joined #openstack-ironic21:49
*** mgkwill has joined #openstack-ironic21:49
*** jcoufal_ has quit IRC21:49
*** jcoufal has joined #openstack-ironic21:50
*** jcoufal has quit IRC21:50
*** zhenguo has joined #openstack-ironic21:51
*** jrcloud has quit IRC21:52
openstackgerritMerged openstack/ironic-lib: Fix check for GPT partioned device  https://review.openstack.org/37499521:52
*** bdeschenes has joined #openstack-ironic21:53
*** syed_ has joined #openstack-ironic21:54
*** cppforlife_ has joined #openstack-ironic21:57
zer0c00lDoes ironic node-list by default displays 1000 nodes at a time?22:03
*** TheJulia has joined #openstack-ironic22:04
jlvillalzer0c00l: Not sure.22:05
*** _mbelanger has quit IRC22:06
*** evgenyl has joined #openstack-ironic22:07
*** boris-42 has joined #openstack-ironic22:07
*** BadCub has quit IRC22:08
*** yonglihe has joined #openstack-ironic22:08
*** ildikov has joined #openstack-ironic22:08
zer0c00lBecause it is fishy, always returns 1000 on some clusters where i know for sure that there are more nodes22:09
*** rajinir has joined #openstack-ironic22:10
*** kozhukalov has joined #openstack-ironic22:10
*** vdrok has joined #openstack-ironic22:13
*** serverascode has joined #openstack-ironic22:13
*** fyxim has joined #openstack-ironic22:13
*** amit213 has joined #openstack-ironic22:14
*** bdeschenes has quit IRC22:14
*** betherly has joined #openstack-ironic22:15
*** vishwanathj has quit IRC22:18
*** cdearborn has quit IRC22:19
*** jrist has quit IRC22:19
*** zackf has joined #openstack-ironic22:26
*** zackf has quit IRC22:31
*** causten_ has quit IRC22:34
*** aNuposic has quit IRC22:57
*** rbudden has quit IRC23:01
*** rama_y has joined #openstack-ironic23:03
xavierrzer0c00l: https://github.com/openstack/python-ironicclient/blob/5b1b332cd34cbf45ccd19a1dbc06e63e453577db/ironicclient/v1/node.py#L7023:07
xavierrzer0c00l: use this limit param to set the pagination size23:08
openstackgerritXavier proposed openstack/ironic: Add inspection feature for the OneView drivers  https://review.openstack.org/36706523:08
openstackgerritXavier proposed openstack/ironic: Add inspection feature for the OneView drivers  https://review.openstack.org/36706523:11
*** BadCub has joined #openstack-ironic23:15
*** hamzy has joined #openstack-ironic23:20
rybridges1Hey, do any of you know at what point during 'nova boot' the 'reservation' gets set?23:22
*** HenryG has quit IRC23:28
*** HenryG has joined #openstack-ironic23:28
zer0c00lxavierr: Thanks23:30
zer0c00lrybridges1: reservation of conductors?23:30
*** baoli has joined #openstack-ironic23:32
TheJuliarybridges1: reservation is used by the conductor as a lock.  Periodic tasks, such as those checking power state can cause a conflict and thus nova (if memory serves) to reschedule and try on a new host23:36
*** hugonicodemos has joined #openstack-ironic23:46
*** hugonicodemos has quit IRC23:49
rybridges1zer0c00l: thanks for that! I am actually wondering at what point during the nova boot does the field get set in the database23:50
rybridges1oops, last message was meant for TheJulia23:52
TheJuliarybridges1: it was a team effort :)23:54
*** zackf has joined #openstack-ironic23:55
« Wednesday, 2016-09-21 Index Friday, 2016-09-23 »

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