Monday, 2015-04-20

*** naohirot has joined #openstack-ironic00:07
*** dttocs has joined #openstack-ironic00:52
*** zhenguo has joined #openstack-ironic00:54
*** dttocs has quit IRC00:57
*** alex_xu has quit IRC01:18
*** alex_xu has joined #openstack-ironic01:23
*** saripurigopi has quit IRC01:30
*** Haomeng has joined #openstack-ironic01:43
*** Haomeng|2 has quit IRC01:45
*** chenglch has joined #openstack-ironic01:47
*** achanda has joined #openstack-ironic01:53
*** dttocs has joined #openstack-ironic01:53
*** dttocs has quit IRC01:57
*** kan__ has joined #openstack-ironic02:29
*** kan__ has quit IRC02:47
*** dttocs has joined #openstack-ironic02:54
*** david-lyle has quit IRC02:55
*** jamielennox is now known as jamielennox|away02:58
*** dttocs has quit IRC02:58
*** achanda has quit IRC03:00
*** jamielennox|away is now known as jamielennox03:03
*** kan__ has joined #openstack-ironic03:31
Haomenggood morning Ironic:)03:32
pshigeHaomeng: morining03:33
Haomengpshige: morning:)03:33
*** coolsvap|afk is now known as coolsvap03:39
*** saripurigopi has joined #openstack-ironic03:50
*** dttocs has joined #openstack-ironic03:55
*** dttocs has quit IRC04:00
*** rameshg87 has joined #openstack-ironic04:14
rameshg87good morning ironic04:14
pshigerameshg87: morining :)04:15
rameshg87morning pshige04:20
*** subscope has joined #openstack-ironic04:25
zhenguomorning :)04:33
rameshg87zhenguo: o/04:34
zhenguorameshg87: thanks for +2 my patch:)04:35
rameshg87zhenguo: :)04:35
openstackgerritNisha Agarwal proposed openstack/ironic: Update ilo documentation for capabilities  https://review.openstack.org/17525704:36
*** subscope has quit IRC04:47
openstackgerritYuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials  https://review.openstack.org/17078204:54
*** dttocs has joined #openstack-ironic04:55
*** dttocs has quit IRC05:00
*** chenglch has quit IRC05:06
*** chenglch|2 has joined #openstack-ironic05:12
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic  https://review.openstack.org/13486505:33
*** achanda has joined #openstack-ironic05:37
rameshg87naohirot: hello05:52
rameshg87naohirot: you might want to look at https://review.openstack.org/#/c/174514/305:53
naohirotrameshg87: Hi good morning05:53
rameshg87naohirot: morning05:53
pshigenaohirot: good afternoon :)05:54
naohirotpshige: good afternoon :)05:54
naohirotrameshg87: I got a comment from john https://review.openstack.org/#/c/163738/2/ironic/tests/drivers/irmc/test_deploy.py05:55
*** dttocs has joined #openstack-ironic05:56
rameshg87naohirot: yeah, jlvillal is making changes to ensure every mock is created with 'spec'05:57
rameshg87naohirot: it turned out that we have many mistakes in code with respect to assertions05:57
rameshg87naohirot: i mean in test case assertions05:57
rameshg87naohirot: so he is modifying the drivers now and the in https://review.openstack.org/#/c/174514/3 he is changing irmc driver05:58
naohirotrameshg87: John change iRMC code in https://review.openstack.org/#/c/174514/3, thanks for letting me know, rameshg87!05:58
rameshg87naohirot: yeah, just wanted you to have a look too05:58
naohirotrameshg87: I'll review his proposal05:58
rameshg87thanks05:59
*** rameshg87 is now known as rameshg87-away05:59
*** dttocs has quit IRC06:01
*** ukalifon has joined #openstack-ironic06:06
*** saripurigopi has quit IRC06:07
openstackgerritNaohiro Tamura proposed openstack/ironic: Fixed is_glance_image(image_href) predicate logic  https://review.openstack.org/17521706:10
*** saripurigopi has joined #openstack-ironic06:13
*** zz_jgrimm is now known as jgrimm06:19
*** lazy_prince has joined #openstack-ironic06:25
*** gillllllliard is now known as gilliard06:48
*** jamielennox is now known as jamielennox|away06:52
*** dttocs has joined #openstack-ironic06:57
*** dttocs has quit IRC07:01
*** rameshg87-away is now known as rameshg8707:06
*** subscope has joined #openstack-ironic07:10
*** rsalevsky has joined #openstack-ironic07:11
openstackgerritNisha Agarwal proposed openstack/ironic: Update ilo documentation for capabilities  https://review.openstack.org/17525707:26
naohirotrameshg87: Hi, I looked at John's mock spec for ilo https://review.openstack.org/#/c/174174/07:26
rameshg87naohirot: yes07:28
rameshg87naohirot: we will have to create a new spec entry to ilo library07:28
naohirotrameshg87: Why doesn't proliantutils need spec set file?07:28
rameshg87naohirot: yeah, i will do that. planning to do that today07:28
naohirotrameshg87: I see.07:29
rameshg87naohirot: that got merged before jlvillal introduced the spec files07:29
*** dlpartain has joined #openstack-ironic07:29
*** chlong has quit IRC07:29
naohirotrameshg87: Okay, I understood that all external lib are in the same situation.07:30
rameshg87naohirot: yeah07:30
*** dlpartain has left #openstack-ironic07:30
naohirotrameshg87: thanks!07:30
*** kalpase has joined #openstack-ironic07:43
*** jistr has joined #openstack-ironic07:47
*** achanda has quit IRC07:47
*** dttocs has joined #openstack-ironic07:58
kejiahumorning ironic!07:58
*** dttocs has quit IRC08:02
*** dtantsur has joined #openstack-ironic08:06
*** dtantsur has quit IRC08:14
*** dtantsur has joined #openstack-ironic08:14
*** derekh has joined #openstack-ironic08:19
*** lucasagomes has joined #openstack-ironic08:21
*** dlpartain has joined #openstack-ironic08:25
*** dlpartain has left #openstack-ironic08:25
*** ndipanov has joined #openstack-ironic08:28
*** dmellado has joined #openstack-ironic08:30
dtantsurfolks, devananda, I'm proposing backport of client retries for Kilo: https://review.openstack.org/#/c/175301/08:32
dtantsurthe reason is mainly that inspection is too troublesome without it, and we (at Red Hat) will have to carry this patch downstream for Kilo release anyway.08:33
dtantsurand good (actually not at all) morning to everyone08:34
*** saripurigopi has quit IRC08:37
*** andreykurilin__ has joined #openstack-ironic08:39
openstackgerritMerged openstack/ironic: Rearrange some code in PXEDeploy.prepare  https://review.openstack.org/17392508:41
*** MattMan has joined #openstack-ironic08:41
*** chlong has joined #openstack-ironic08:50
*** kbyrne has joined #openstack-ironic08:54
*** saripurigopi has joined #openstack-ironic08:57
pshigekejiahu, dtantsur, morning :)08:57
dtantsuro/08:57
*** dttocs has joined #openstack-ironic08:59
*** rameshg87 is now known as rameshg87-away09:01
*** dttocs has quit IRC09:03
*** a1exhughe5 has joined #openstack-ironic09:03
*** edcragg has joined #openstack-ironic09:06
*** sambetts has joined #openstack-ironic09:06
*** foexle has joined #openstack-ironic09:13
openstackgerritZhenguo Niu proposed openstack/ironic: Log attempts while trying to sync power state  https://review.openstack.org/16712209:17
*** alexpilotti has joined #openstack-ironic09:17
*** romcheg has joined #openstack-ironic09:18
*** chlong has quit IRC09:20
*** kalpase has quit IRC09:23
*** pelix has joined #openstack-ironic09:23
*** mgoddard1 has joined #openstack-ironic09:25
*** pcaruana has quit IRC09:27
*** mgoddard has quit IRC09:27
*** pcaruana has joined #openstack-ironic09:30
*** chlong has joined #openstack-ironic09:33
*** naohirot has quit IRC09:36
*** dtantsur has quit IRC09:37
openstackgerritVictor Sergeyev proposed openstack/ironic: Run tests in py34 environment  https://review.openstack.org/15619209:44
*** rameshg87-away has quit IRC09:50
lucasagomesvdrok, hi there09:50
lucasagomesvdrok, when you worked on the non-glance images spec, this bit here https://review.openstack.org/#/c/175217/2/ironic/common/glance_service/service_utils.py09:50
lucasagomesis it due we can use the image name in glance instead of the UUID?09:50
*** coolsvap is now known as coolsvap|afk09:51
vdroklucasagomes, morning09:52
yuriyzmorning Ironic lucasagomes dtantsur09:52
lucasagomesvdrok, yuriyz good morning09:52
yuriyzmorning vdrok09:52
vdrokmorning yuriyz09:52
yuriyzI'm not sure about usecase of this09:52
pshigevdrok, yuriyz, lucasagomes, morining :)09:52
lucasagomesI looked at the spec and it says UUID only, so seems the code there is correct09:52
lucasagomespshige, hi there good morning09:53
vdroklucasagomes, at the moment we can not use glance image names, it is just some pre-check09:53
*** ifarkas has joined #openstack-ironic09:53
vdroklucasagomes, yup, only uuid09:53
lucasagomesvdrok, gotcha. I thouht that was the reason why you were not checking if it's UUID09:53
lucasagomesvdrok, thanks for clarifying, I will approve that patch then09:53
vdroklucasagomes, I had a patch adding uuid check, but then it had to throw some exceptions that seemed to be unnecessary09:54
pshigeUUID is one of href-s?09:55
vdrokmorning pshige09:55
lucasagomesvdrok, oh...09:56
lucasagomesvdrok, people were asking it to throw some exceptions you didn't agree with?09:56
pshigeUUID is not a hypertext reference from the definition, I think.09:57
vdroklucasagomes, JoshNang and yuriyz suggested that we should throw an exception in that method if href provided was not uuid like09:57
lucasagomesI see09:58
pshigeIt digresses from the main point, but ...09:58
*** saripurigopi has quit IRC09:58
vdroklucasagomes, but it appeared that because some logic would have to be added to the conductor09:58
lucasagomesI'm happy with that patch from naohiro09:58
*** saripurigopi has joined #openstack-ironic09:58
*** dttocs has joined #openstack-ironic09:59
*** andreykurilin__ has quit IRC10:00
yuriyzvdrok agree because add mentioned code to conductor looks like level violation10:00
*** rameshg87 has joined #openstack-ironic10:02
*** yuriyz has quit IRC10:02
*** yuriyz has joined #openstack-ironic10:02
vdroklucasagomes, yes, that patch looks ok, but it does not really introduce a bug I think, it will fail anyway when image will be downloaded10:03
lucasagomesyeah, that's what I thought. It will check if it's glance, is file, is http... none will be found so it will fail10:03
lucasagomeswhich seems like the correct behavior10:03
*** dttocs has quit IRC10:04
rameshg87lucasagomes: in my opinion that patch was bettering the approach there10:04
rameshg87instead of just checking if '/' existed10:05
lucasagomesrameshg87, yeah10:05
lucasagomesrameshg87, at first I thought that was because we could use the image names instead of UUID10:05
lucasagomesso I just +1 and clarified with vdrok before approving10:05
rameshg87lucasagomes: i think we can't, glance can have 2 images with same name :)10:06
lucasagomesrameshg87, yup yeah we can't10:06
vdroklucasagomes, rameshg87 here is the old patch https://review.openstack.org/#/c/151951/10:06
rameshg87lucasagomes: yeah i had +1ed that :)10:06
lucasagomes:-)10:07
*** jcoufal has joined #openstack-ironic10:07
vdrokrameshg87, theoretically we can have image names, just ask the user to provide uuid if multiple images with same name found10:07
rameshg87vdrok: so, will we fail validate if multiple images are found with same name ?10:07
vdrokrameshg87, yup, I think in that case we should, with some meaningful message :)10:08
vdrokrameshg87, but it's not supported anyway now10:08
rameshg87vdrok: yeah, may be just like ironic has logical names and uuid10:09
kan__Hi, lucas10:12
kan__lucasagomes: the bug 1443003, do you have some time to take a look?10:14
openstackbug 1443003 in Ironic "port-list --detail using 'Node UUID' as sort key and node-list --detail using 'Chassis UUID' as sort key fail" [Undecided,In progress] https://launchpad.net/bugs/1443003 - Assigned to Kan (kansks)10:15
kan__Please just comment in that bug, thx :-)10:15
lucasagomeskan__, oh man I totally forgot! I will do10:19
* lucasagomes writes it down10:19
lucasagomeskan__, so are you looking for a fix on that?10:20
*** ifarkas has quit IRC10:20
*** ifarkas has joined #openstack-ironic10:21
*** mgoddard1 has quit IRC10:34
*** jcoufal_ has joined #openstack-ironic10:50
openstackgerritMerged openstack/ironic: Fixed is_glance_image(image_href) predicate logic  https://review.openstack.org/17521710:50
*** yuanying has quit IRC10:52
*** jcoufal has quit IRC10:53
*** dttocs has joined #openstack-ironic11:00
*** mgoddard has joined #openstack-ironic11:01
*** romcheg has quit IRC11:01
*** afaranha has quit IRC11:02
*** afaranha_ has quit IRC11:03
*** dttocs has quit IRC11:04
* rameshg87 goes home11:06
*** rameshg87 has quit IRC11:06
openstackgerritMerged openstack/ironic: Add cleaning network docs  https://review.openstack.org/17462011:16
*** jcoufal_ has quit IRC11:28
*** romcheg has joined #openstack-ironic11:29
*** jcoufal has joined #openstack-ironic11:29
*** dlpartain has joined #openstack-ironic11:31
*** thrash|g0ne is now known as thrash11:37
*** chenglch|2 has quit IRC11:42
kan__<lucasagomes>, yes, I think not use this 'Node UUID' as the sort key might solve this11:43
kan__Or is there other solutions?11:44
lucasagomeskan__, apparently this bug was introduced due https://review.openstack.org/#/c/141737/11:52
lucasagomeskan__, I wonder if we should translate the node_uuid to node_ident (that's how it's called now)11:52
lucasagomesso that we keep the same interface working11:53
*** jistr has quit IRC11:57
*** chlong has quit IRC11:58
*** saripurigopi has quit IRC11:59
*** dttocs has joined #openstack-ironic12:01
*** lucasagomes is now known as lucas-hungry12:01
*** jistr has joined #openstack-ironic12:03
*** saripurigopi has joined #openstack-ironic12:04
*** dttocs has quit IRC12:05
*** zhenguo has quit IRC12:07
*** trown|outttypeww is now known as trown12:13
*** lazy_prince has quit IRC12:14
*** Haomeng|2 has joined #openstack-ironic12:21
*** Haomeng has quit IRC12:24
*** jamielennox|away is now known as jamielennox12:31
*** dprince has joined #openstack-ironic12:31
*** jamielennox is now known as jamielennox|away12:32
*** dlpartain has left #openstack-ironic12:50
*** dencaval has joined #openstack-ironic12:53
*** lucas-hungry is now known as lucasagomes12:55
*** rloo has joined #openstack-ironic12:55
*** dttocs has joined #openstack-ironic13:02
*** clett has quit IRC13:03
*** lifeless has quit IRC13:05
*** dttocs has quit IRC13:06
*** saripurigopi has quit IRC13:08
*** kkoski has joined #openstack-ironic13:10
NobodyCamgood monday morning Ironic13:29
lucasagomesNobodyCam, good morning13:33
lucasagomesNobodyCam, btw, I won't be able to make to the meeting today13:34
lucasagomesNobodyCam, there's anything we should discuss before?13:34
NobodyCamhey hey lucasagomes :)13:35
NobodyCamlet me take a look at the agenda13:35
NobodyCamhope you had a good weekend13:35
*** jistr_ has joined #openstack-ironic13:35
NobodyCamlooks like just a item from rloo :)13:37
rloohello ironic'ers. on this rainy monday.13:38
*** jistr has quit IRC13:38
NobodyCamahh raining :(13:38
NobodyCamsunny down here today :/13:39
NobodyCamand good morning rloo :)13:39
rlooNobodyCam: lucky you. Well, April showers bring May flowers so we'll see. Morning!13:39
NobodyCam:)13:39
rloohi lucasagomes. that agenda item isn't important; just wanted us/someone to decide :)13:43
lucasagomesNobodyCam, it was good :-) thanks13:43
lucasagomesrloo, good morning13:43
* lucasagomes looks at the agenda13:43
*** jistr_ has quit IRC13:44
*** jistr has joined #openstack-ironic13:44
*** zhenguo has joined #openstack-ironic13:44
lucasagomesrloo, yeah that item seems not so important. IMO if you want to proxy my opnion13:46
*** viktors has joined #openstack-ironic13:46
lucasagomesI think it's nice to have the enconding on the file13:46
rloolucasagomes: I don't really have an opinion. I mean, since the client has it, we should be consistent. but if the client didn't have it, i wouldn't bother adding that line.13:47
lucasagomesthat's just to prevent SyntaxError just because say, some person or a company that has non utf-8 characters in their names add a NOTE or a copyright line13:47
lucasagomesin the code13:47
rloolucasagomes: so i guess the 'right' solution is to add it, in case it is ever needed.13:48
lucasagomesrloo, IMHO, yes I would add it13:50
* lucasagomes adds it by default in his template13:51
rloolucasagomes: good to know. I'll add your vote :)13:51
lucasagomesrloo, :-) ack13:52
lucasagomesbtw there's a PEP about it13:52
lucasagomesI dunno the number, lemme check it out13:52
lucasagomes263 https://www.python.org/dev/peps/pep-0263/13:53
rloolucasagomes: thx. I guess it makes sense, if it'll help our eg Asian buddies.13:56
*** tiagogomes_ has joined #openstack-ironic13:58
*** dttocs has joined #openstack-ironic14:02
*** jgrimm is now known as zz_jgrimm14:04
*** dttocs has quit IRC14:07
*** mtanino has joined #openstack-ironic14:08
NobodyCambrb14:12
*** pas-ha has joined #openstack-ironic14:20
*** Party_Bear is now known as BadCub14:24
NobodyCammorning BadCub :)14:24
BadCubMorning NobodyCam :)14:25
BadCubMorning folks :)14:26
vdrokmorning BadCub, NobodyCam, rloo14:29
rloohi vdrok. Morning BadCub.14:30
NobodyCammorning vdrok14:30
BadCubmorning rloo , vdrok14:30
jroll\o morning all14:31
NobodyCammorning jroll :)14:31
BadCubmornin jroll14:31
TheJuliagood morning everyone14:31
BadCubmorning TheJulia :)14:32
rlooG'morning jroll, TheJulia14:32
vdrokmorning jroll, TheJulia14:32
NobodyCammorning TheJulia14:32
*** lifeless has joined #openstack-ironic14:32
*** achanda has joined #openstack-ironic14:32
*** achanda has quit IRC14:39
NobodyCambrb14:47
*** tiagogomes_ has left #openstack-ironic14:57
devanandamorning, all14:59
NobodyCammorning devananda15:00
BadCubmorning devananda15:00
jroll\o deva15:01
TheJuliagood morning15:01
*** rwsu has joined #openstack-ironic15:02
lucasagomesdevananda, morning15:04
lucasagomesBadCub, jroll  TheJulia  morning!15:04
*** zz_jgrimm is now known as jgrimm15:04
jrollheya lucas :)15:05
*** achanda has joined #openstack-ironic15:05
*** rainya has left #openstack-ironic15:06
*** achanda has quit IRC15:07
BadCubmorning lucasagomes15:08
* NobodyCam luvs our morning roll call it always reminds him https://www.youtube.com/watch?v=YEEkNG9OrVA15:09
*** beekneemech is now known as bnemec15:10
*** jistr has quit IRC15:14
*** mgoddard has quit IRC15:14
*** mgoddard has joined #openstack-ironic15:15
*** lucasagomes_ has joined #openstack-ironic15:17
*** lucasagomes has quit IRC15:18
*** lucasagomes_ is now known as lucasagomes15:19
NobodyCamcall for any agenda items?15:21
BadCubI got nothing for agena right now15:24
*** dttocs has joined #openstack-ironic15:25
*** foexle has quit IRC15:27
*** jistr has joined #openstack-ironic15:28
vdrokdevananda, Shrews, morning, got some questions about tempest scenario for ironic without nova, if you have a couple of minutes15:30
vdrokseems that I need a couple of things from our client there - configdrive building for injecting ssh key and retrying on having conflicts, like node locked15:31
lucasagomesdevananda, hi there, since L is now open mind lifting this -2 here https://review.openstack.org/#/c/171672/ ?15:32
* NobodyCam is jumping on conf call15:33
vdrokironicclient is not in the reqirements at the moment in tempest, and it seems I'll need the latest version of it15:33
vdrokand I don't really understand how to use it (client) there, I haven't found any direct usage of client libraries15:35
vdrokit seems that everything is done via api, and if so, I'll need to copypaste configdrive building inside scenario15:36
jrollvdrok: yeah, tempest doesn't use the client libs15:36
vdrokjroll, if so, should I copypaste it, how do you think?15:37
jrollmmm, I don't know15:38
jrollmaybe you could bring the client in just for the configdrive stuff?15:38
vdrokI need configdrive with ssh key to ssh into created vm and check that I can do something there, like create a file15:38
vdrokjroll, yes, but then comes retrying15:38
vdroksometimes I get NodeLocked locally15:39
jrollright, I mean, that's not a lot of code15:39
vdrokjroll, ok, I'll try to add it there15:40
jrollvdrok: though it's odd to expect errors in API tests :P15:41
vdrokjroll, well, it's not an api test, it's integration test15:41
vdrokI use nova for managing keypairs and neutron for dhcp15:42
jrollvdrok: right, kind of joking, tempest was meant to be an api test framework15:42
devanandatempest intentionally does not use any of the python-*client libs15:42
devanandabecause what we're supposed to be testing is the REST API, not the client15:42
devanandaso yes, you'll need to build the configdrive separately (without python-ironicclient)15:42
devanandaif you are testing ironic's API15:42
vdrokdevananda, ok, I see15:42
jrolldevananda: I don't think it would be bad to use the client's configdrive builder, it doesn't touch the api15:43
jrollafaik15:43
vdrokone more thing - seems that there should be one more option introduced to know which image_source to be used15:43
jrollmaybe it does, maybe that's just a path argument to set-provision-state15:43
vdrokas they're different for agent and pxe15:44
devanandajroll: I'm inclined to disagree -- using a different method to build the config drive ehlps ensure that our server isn't bound to the client's particular configdrive output15:44
*** viktors is now known as viktors|afk15:44
devanandaactually15:44
vdrokso, maybe devstack should write to config whether we're using agent_ssh or pxe_ssh15:44
jrolldevananda: sure, that's fine15:44
jrolldevananda: though the client just makes an image out of the directory you give it15:45
devanandai could make a case either way. configdrive should be an opaque blob from ironic's POV, right?15:45
jrollright15:45
jrollI'm fine with either way, I just like code re-use15:46
jrollthe client builder thing just takes a directory, stuffs it in an iso, gzips it, and base64-encodes it15:46
devanandais that something we could put in ironic-lib?15:47
jrollyes, though I don't think the server or agent would use it15:47
*** pas-ha has quit IRC15:47
jrollbut the client could, bifrost could15:47
*** pas-ha has joined #openstack-ironic15:47
jrollthe nova driver does the gzip/base64 step15:48
jrollso could have ironic_lib.{build_iso,pack_iso}15:48
vdrokdevananda, jroll, sorry for asking twice, but should it be inside tempest or imported from somewhere else? :)15:49
jrollvdrok: I don't think we've decided that yet :P15:50
vdrokok, then I'll add it in scenario and see what tempest people think15:50
devanandavdrok: it'd be best to get the folks in #openstack-qa to weigh in. I could see it going either way15:50
* jroll bbiab15:50
vdrokok, devananda, 2 more questions - first about determining the driver we're using to get the correct image written to instance_info/image_source15:51
*** jlvillal has quit IRC15:52
vdrokis it ok to add it to tempest config15:52
vdroksomething like baremetal_deploy_driver15:52
*** jlvillal has joined #openstack-ironic15:52
lucasagomesfolks I will have to leave a tad earlier today15:53
vdrokdevstack creates CONF.compute.image_uuid config option with glance uuid of an image that's going to be used15:53
*** jistr_ has joined #openstack-ironic15:53
lucasagomesso have a great night everyone!15:53
vdroknight lucasagomes15:53
BadCubg'night lucas15:53
lucasagomesI won't be able to go to the meeting as I said before, so have a good meeting I will read the logs later15:53
lucasagomesdevananda, if you have some time, mind lifting this -2 here https://review.openstack.org/#/c/171672/ ?15:53
vdrokand second question - how should we select the node for deployment?15:54
NobodyCamgood night lucasagomes15:54
*** pradipta has joined #openstack-ironic15:54
vdrokshould we create one from scratch? or use one that devstack provides, like nodes[0] or something like that, assuming devstack creates one vm15:54
*** lucasagomes is now known as lucas-afk15:55
*** Marga_ has joined #openstack-ironic15:55
*** jistr has quit IRC15:56
*** ukalifon has quit IRC15:57
openstackgerritJohn L. Villalovos proposed openstack/ironic: ironic/tests/drivers/drac: Add spec= & autospec=True  https://review.openstack.org/17414516:01
devanandalucas-afk: done, and g'night16:03
*** jistr_ is now known as jistr16:05
*** Marga_ has quit IRC16:05
*** Marga_ has joined #openstack-ironic16:06
*** a1exhughe5 has quit IRC16:08
*** jcoufal has quit IRC16:11
*** absubram has joined #openstack-ironic16:19
*** Marga_ has quit IRC16:22
*** Marga_ has joined #openstack-ironic16:24
*** jistr has quit IRC16:24
* BadCub needs food16:24
*** ijw has joined #openstack-ironic16:32
*** ijw_ has joined #openstack-ironic16:36
*** ijw__ has joined #openstack-ironic16:37
*** romcheg has quit IRC16:37
openstackgerritJosh Gachnang proposed openstack/ironic: Remove API reboot from cleaning docs  https://review.openstack.org/17547716:38
*** ijw has quit IRC16:38
*** Nisha has joined #openstack-ironic16:39
*** trown is now known as trown|lunch16:39
*** ijw_ has quit IRC16:40
*** Nisha_away has joined #openstack-ironic16:44
*** Nisha has quit IRC16:44
openstackgerritNisha Agarwal proposed openstack/ironic: Update iLO documentation for capabilities  https://review.openstack.org/17525716:45
*** ifarkas has quit IRC16:51
*** derekh has quit IRC16:52
*** dtantsur has joined #openstack-ironic16:54
dtantsurgood evening/morning/afternoon, says man arriving for meeting from his sick day16:55
dtantsur(also no IRC history available, so if you pinged - try again now please)16:56
rloodtantsur: you are one dedicated ironic'er :-)16:56
dtantsur:)16:56
NobodyCammorning dtantsur16:57
NobodyCamdtantsur: if your sick you should be sleeping16:57
pshigedtantsur: morning :)16:57
NobodyCammorning pshige16:58
dtantsurNobodyCam, can't sleep anyway16:58
NobodyCam:/ :(16:58
pshigeNobodyCam: morning :)16:58
rlooNobodyCam, dtantsur: maybe the meeting will lull you to sleep :)16:58
dtantsurlol we'll see16:58
NobodyCamwe did get a couple more topics on the agenda :)16:58
*** Nisha_away is now known as Nisha17:00
*** ijw has joined #openstack-ironic17:01
*** ijw__ has quit IRC17:04
*** Guest36304 is now known as mgagne17:16
*** mgagne has joined #openstack-ironic17:16
*** romcheg has joined #openstack-ironic17:18
*** davideagnello has joined #openstack-ironic17:20
*** achanda has joined #openstack-ironic17:24
*** edcragg has quit IRC17:24
*** zhenguo has quit IRC17:27
rloodtantsur: lucas' example was what if someone wanted to add their name/company, and it had non-ascii chars in it17:31
dtantsurfair17:31
*** romcheg has quit IRC17:34
*** ukalifon1 has joined #openstack-ironic17:38
*** romcheg has joined #openstack-ironic17:38
*** romcheg has left #openstack-ironic17:38
*** harlowja_away is now known as harlowja17:38
*** vdrok is now known as vdrok_afk17:40
*** ukalifon1 has quit IRC17:48
*** dttocs has quit IRC17:56
*** ijw has quit IRC17:58
*** pelix has quit IRC17:58
*** ijw has joined #openstack-ironic17:59
NobodyCamthankyou all great meeting18:00
rloohi devananda. wrt the migration scripts in kilo. we don't know if those scripts will work with nova bm juno -> ironic kilo?18:00
devanandarloo: ooooh.18:01
* dtantsur is going back to bed18:01
devanandarloo: I think I was thinking of something else18:01
rloonight dtantsur. hope you feel better soon18:01
devanandadtantsur: g'night! feel better18:01
devanandarloo: the supported path should be: juno nova-bm -> juno ironic -> kilo ironic18:01
rloodtantsur: sorry the meeting wasn't boring enough to lull you to sleep18:01
devanandarloo: not juno nova-bm -> kilo ironic18:02
dtantsur:)18:02
*** dtantsur has quit IRC18:02
rloodevananda: yeah, that's what i thought. so we shouldn't have the migration scripts in kilo, or we leave them there, but make it clear it is for use with ironic juno. hmm, that doesn't make sense I don't think.18:02
devanandarloo: and they've already been deleted from master?18:03
*** ijw has quit IRC18:03
NobodyCambrb18:04
rloodevananda: no, not yet: https://review.openstack.org/#/c/174593/18:04
devanandathat should have been dropped last cycle, right?18:05
rloodevananda: yeah, that's what I think anyway. dropped from kilo.18:05
devanandarloo: mind filing a bug, tagging kilo-rc-potential, and proposing a backport of ^ ?18:05
rloodevananda: will do. yee haw.18:05
*** ijw has joined #openstack-ironic18:06
BadCubbrb18:06
*** trown|lunch is now known as trown18:08
*** Nisha_away has joined #openstack-ironic18:10
*** Nisha has quit IRC18:10
openstackgerritRuby Loo proposed openstack/ironic: Remove scripts for migrating nova baremetal  https://review.openstack.org/17459318:14
jlvillalrloo: Were you going to remove un-needed 'utf-8' lines?18:32
rloojlvillal: yup.18:32
jlvillalrloo: Not sure if this is of any use:  http://paste.openstack.org/show/204905/18:32
rloojlvillal: thx, it might be. I haven't gotten around to doing it. since it is your script, you're welcome to do it instead of me.18:33
jlvillalrloo: Okay.  So 'ironic' and 'python-ironicclient'?18:34
rloojlvillal: well, I wasn't going to bother with ironic. the ones there are cuz people added them. we aren't going to prevent folks from adding them, are we? but most? of the ones in client were due to a patch I think.18:34
rloojlhttps://review.openstack.org/#/c/96109/18:35
rloojlvillal: https://review.openstack.org/#/c/96109/18:35
jlvillalrloo: Okay.  I will only do python-ironicclient then.  Only six files in ironic actually seem to need the utf-8 line.18:35
jlvillalrloo: Many more have the line.18:35
rloojlvillal: I think lucas-afk mentioned to me that he adds it when he adds a new file.18:36
jlvillalrloo: Okay18:36
rloojlvillal: i don't want to have to deal with patches that remove those lines. i don't mind doing it now to undo that other patch, but it will become ridiculous. I think.18:37
jlvillalrloo: Okay.  I will try just a revert then.18:37
rloothx jlvillal. I'm fine with a revert or with a global remove. It is a one-time thing, and who knows if anyone copy/pasted/added the utf-8 line.18:38
jlvillalrloo: Okay.  I'm leaning towards a global one-time remove then.18:38
rloojlvillal: crap. just thought of something. well, guess it won't matter that much. it might back backports harder cuz of the file diff. oh well.18:39
jlvillalrloo: Uh, I wouldn't think so.  As only the top two lines of the file.  So should be removed from most code that would be backported, I would think.18:39
rloojlvillal: I see some new patches with new files, with that line in it. Sigh.18:40
jlvillalrloo: Oh :(18:41
rloojlvillal: that's cuz we're movers and shakers. Let's not worry about that.18:41
rloojlvillal: also, my suggestion is to use the original bug, to indicate there that we're undoing the change.18:42
jlvillalrloo: will do18:43
rloothx jlvillal!18:43
openstackgerritJosh Gachnang proposed openstack/ironic: Remove API reboot from cleaning docs  https://review.openstack.org/17547718:53
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Implement FRU inventory (WIP)  https://review.openstack.org/17460118:54
*** pradipta has quit IRC18:54
*** andreykurilin__ has joined #openstack-ironic18:55
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Remove uneeded 'utf-8' coding lines  https://review.openstack.org/17553418:56
jlvillalrloo: ^^^   and I'm off to lunch.  Thanks.18:56
rloothx jlvillal. will look soon...18:57
*** dttocs has joined #openstack-ironic18:57
*** ukalifon has joined #openstack-ironic18:58
*** ukalifon has quit IRC19:00
*** dttocs has quit IRC19:02
*** Nisha_away has quit IRC19:02
*** ukalifon has joined #openstack-ironic19:03
*** ukalifon has quit IRC19:09
*** ijw has quit IRC19:10
*** cri has joined #openstack-ironic19:11
*** cri has left #openstack-ironic19:12
rloohi NobodyCam, would you like me to clear the agenda/etc of our meetings, after we have a meeting? (I clear the subteam stuff on the etherpad, so ...)19:17
NobodyCamrloo: that would be super awesome!!!19:18
*** ijw has joined #openstack-ironic19:18
*** ukalifon has joined #openstack-ironic19:20
*** ukalifon has quit IRC19:20
*** ukalifon has joined #openstack-ironic19:22
rlooNobodyCam: ok, done :)19:23
NobodyCamThank you rloo :)19:25
*** ukalifon has quit IRC19:41
* BadCub steps away for a while19:41
* NobodyCam will brb19:42
*** dprince has quit IRC19:50
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Implement FRU inventory (WIP)  https://review.openstack.org/17460119:55
*** dencaval has quit IRC19:56
*** andreykurilin__ has quit IRC19:59
*** andreykurilin__ has joined #openstack-ironic20:00
*** dttocs has joined #openstack-ironic20:04
*** andreykurilin__ has quit IRC20:16
*** andreykurilin__ has joined #openstack-ironic20:17
lucas-afkrloo, thanks for voting on my behalf heh20:20
rloolucas-afk: no worries. Are you OK with that?20:20
lucas-afkrloo, FYI I would vote "no" for all files, I think we should not have a hacking rules forbidding it for all files20:20
*** jjohnson2 has joined #openstack-ironic20:20
lucas-afkbecause some people may need to add a non ascii character in a file, such as their own names on a NOTE or their company names20:20
rloolucas-afk: well, we can't have a hacking rule forbidding it cuz sometimes you need it.20:21
lucas-afkyup20:21
lucas-afkI'm ok having files without it, but I'm not ok if we ban it20:21
lucas-afk:-)20:21
rloolucas-afk: yeah, I noticed that some red hat person uses a symbol in some of the files :-)20:21
lucas-afkI put it by default, just becuase my edit is configured to do it20:21
lucas-afkyeah20:21
*** achanda has quit IRC20:22
rloolucas-afk: so I think we're good then. jlvillal has a patch to undo the changes in the client cuz jroll wanted it, but after that, we shouldn't be mucking around with it.20:22
lucas-afkrloo, any thanks much for bringing the idea to the meeting20:22
lucas-afkrloo, cool! fair enuff20:22
lucas-afkand thanks much*20:22
rloolucas-afk: :)20:22
rloolucas-afk: enjoy your evening/night. go away :D20:22
lucas-afkrloo, :-) hah I will. Yeah I was away20:23
lucas-afkwill be afk again now20:23
lucas-afkthanks!20:23
rloook, bye again lucas-afk!20:23
*** andreykurilin__ has quit IRC20:24
*** spandhe has joined #openstack-ironic20:34
*** achanda has joined #openstack-ironic20:35
*** kevinbenton has joined #openstack-ironic20:43
*** kevinbenton has quit IRC20:44
*** kevinbenton has joined #openstack-ironic20:44
* NobodyCam is back20:45
TheJuliawelcome back20:46
NobodyCamThank you TheJulia :)20:46
*** kkoski has quit IRC21:04
*** trown is now known as trown|outttypeww21:04
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Implement FRU inventory (WIP)  https://review.openstack.org/17460121:05
*** jjohnson2 has quit IRC21:14
*** kkoski has joined #openstack-ironic21:19
*** pas-ha has quit IRC21:24
*** jgrimm is now known as zz_jgrimm21:25
*** romcheg has joined #openstack-ironic21:40
*** romcheg has quit IRC21:40
mrdaMorning Ironic21:43
NobodyCammorning mrda21:43
jlvillalmrda: Good morning21:43
BadCubmorning mrda21:43
mrdaNobodyCam, jlvillal, BadCub: \o21:44
NobodyCam:)21:44
pshigemrda: morning :)21:50
*** kkoski has quit IRC21:51
mrdapshige: :)21:58
*** absubram has quit IRC22:00
rloomorning mrda!22:04
rlooNobodyCam: why does IPMIPower need to use temporary files? (https://review.openstack.org/#/c/160383/26/ironic/drivers/modules/ipmitool.py)22:08
NobodyCamrloo: password22:09
rlooNobodyCam: Oh, because we put the password in a file. right. sigh.22:09
NobodyCam:)22:09
NobodyCamyeppers22:09
mrdarloo: thanks for the review of 173674, I'll respond during my day today22:10
rloomrda: yw22:10
*** ijw has quit IRC22:11
*** ijw has joined #openstack-ironic22:11
*** ijw has quit IRC22:16
*** lucas-afk has quit IRC22:17
*** ijw has joined #openstack-ironic22:23
*** kbs has joined #openstack-ironic22:26
*** achanda has quit IRC22:31
openstackgerritChris Krelle proposed openstack/ironic: Add check for usable tftp directory to PXE driver  https://review.openstack.org/16745122:32
*** wuhg has joined #openstack-ironic22:47
NobodyCamoh rloo you are correct I do need to add that dir check to the other ipmi interfaces :-p good catch ! thank you22:53
*** ijw has quit IRC22:54
*** ijw has joined #openstack-ironic23:00
*** pas-ha has joined #openstack-ironic23:02
*** yuanying has joined #openstack-ironic23:06
openstackgerritShilla Saebi proposed openstack/ironic: changes to upgrade-guide.rst  https://review.openstack.org/17406523:09
*** achanda has joined #openstack-ironic23:10
devanandaI have posted a first pass at Kilo release notes: https://wiki.openstack.org/wiki/Ironic/ReleaseNotes/Kilo23:10
mrdacool devananda!23:10
devanandarloo, NobodyCam, anyone else ... feedback welcome ^23:10
* NobodyCam looks23:11
* BadCub ganders23:11
*** Haomeng has joined #openstack-ironic23:15
BadCubdevananda: Looks pretty darn good for a draft23:17
NobodyCamdevananda: added the driver links and removed the TODO23:17
*** Haomeng|2 has quit IRC23:18
*** achanda_ has joined #openstack-ironic23:18
*** achanda_ has quit IRC23:18
*** achanda_ has joined #openstack-ironic23:19
*** achanda has quit IRC23:19
devanandacheers23:20
devanandai'm about to add some more things too23:20
BadCubdevananda: cool. I will gander at those tomorrow and see if I can lend a hand to the effort23:21
mrdadevananda: lgtm. We could reword the "one nova-compute process" bit.  Would it be better to say that communicating to Ironic from more than one nova-compute is not officially supported?23:21
devanandamrda: that's a copy-paste from the juno notes. so, yes, we can change it :)23:22
mrdaDo you want me to do that? Happy to help :)23:22
jrollI'm currently digging on a potential CCM thing right now :(23:22
* mrda hands jroll a shovel23:23
jroll:/23:23
jrolltl;dr instance cache.23:24
JayFdevananda: under additional features; is cleaning eligible for that?23:24
JayFor should it be, I mean23:24
jrollJayF: that's in key features23:24
JayFOf course it is; I read right over it23:24
jroll:D23:25
JayFNoteworthy Improvements over Juno: Any agent documentation at all23:25
JayFlol23:25
devananda:)23:32
* devananda tries to merge his changes with nobodycam's23:32
NobodyCamoh wipe mine out23:33
NobodyCamjust three links easy to re add23:33
NobodyCam:-p23:33
*** arif-ali has quit IRC23:34
*** chlong has joined #openstack-ironic23:34
devanandadone23:38
NobodyCamahh more links to add23:38
NobodyCam:)23:38
devanandaheh23:38
*** david-ly_ has joined #openstack-ironic23:44
*** jamielennox|away is now known as jamielennox23:45
*** david-ly_ is now known as david-lyle23:46
*** mtanino has quit IRC23:46
*** openstackgerrit has quit IRC23:58
*** openstackgerrit has joined #openstack-ironic23:58

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