Friday, 2014-03-14

adam_ghttp://paste.ubuntu.com/7087553/  <- is this trace in the nova driver anything obvious?00:00
adam_gjust about got devstack rolling on fedora00:00
devanandaadam_g: yep - saw taht earlier too00:00
devanandaadam_g: it's trying to remove neutron VIF info from ironic, but it wasn't set to begin with00:01
adam_gdevananda, is that expected to be set during ironic node/port creation? or by nova/etc later?00:01
devanandaby nova during deploy00:01
devananda* during spawn()00:01
devanandathere's a lot of poor handling of exceptions in teh nova driver right now. i'm trying to clean some of it up00:01
NobodyCamback in a bit.. after 5 walkies00:02
devanandaor -- put another way, things that maybe could be idempotent are raising exceptions if nova does them twice00:02
devanandalike deleting an instance00:02
adam_gah00:02
devanandanova expects taht to be idepotent, but ironic rreturns an error if you try to unset the node.instance_uuid when it wasn't set to begin with00:02
devanandai suspect you just hit the same thing with ports00:03
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Better instance-not-found handling within IronicDriver  https://review.openstack.org/8042500:18
devanandaadam_g: ^ is a bit sloppy but may alleviate some issues everyoe is having00:18
*** rloo has quit IRC00:27
*** rloo has joined #openstack-ironic00:27
devanandalifeless: reviewed00:27
devanandanow i must go do administriva00:27
*** rloo_ has joined #openstack-ironic00:30
*** rloo has quit IRC00:30
*** matsuhashi has joined #openstack-ironic00:31
lifelessdevananda: thanks00:32
*** Lotus907efi is now known as Lotus907efi-00:36
*** rloo_ has quit IRC00:46
*** rloo has joined #openstack-ironic00:47
*** openstack has joined #openstack-ironic00:53
*** openstack has joined #openstack-ironic00:57
devanandaadam_g: http://paste.openstack.org/show/73472/00:57
*** openstack has quit IRC00:57
*** openstack has joined #openstack-ironic00:59
*** rpodolyaka has joined #openstack-ironic00:59
devanandathe host itself may be a bit old, but all packages and projects should be no older than this morning01:02
*** jgrimm has quit IRC01:02
devanandamay be a permission issue on rootwrap.d/* ?01:02
*** blamar_ has joined #openstack-ironic01:02
adam_gdevananda, yeah, looks like a rootwrap config issue of some kind.. have not seen that since i started using devstack again last week01:02
*** SpamapS has quit IRC01:04
*** blamar has quit IRC01:06
*** rpodolyaka1 has joined #openstack-ironic01:07
*** matsuhashi has quit IRC01:08
*** SpamapS_ has joined #openstack-ironic01:08
adam_gdevananda, any chance i can hop on that instance to poke really before i EOD?01:08
devanandaadam_g: it's inside my laptop01:08
adam_goh :)01:08
adam_gdevananda, you should be able to just manually test rootwrap01:08
adam_gsudo ironic-rootwrap /etc/ironic/rootwrap.conf iscsiadm01:08
devanandayea. repeats the error01:08
*** rpodolyaka has quit IRC01:08
adam_gdevananda, do /etc/ironic/rootwrap.conf + /etc/ironic/rootwrap.d/* look sane?  they just get copied there straight from the git repo01:08
devanandachecking alrady01:08
devanandacomparing to nova01:08
*** blamar_ is now known as blamar01:08
devanandaodd01:10
devanandathere's a rootwrap.d/rootwrap.d/01:10
devanandaremoving that seems to fix it01:12
devanandaperhaps a left over from an old devstack run01:12
adam_gwtf?01:12
devanandayea01:12
devanandano files were different in taht dir01:12
*** matsuhashi has joined #openstack-ironic01:12
devanandajust its mere presence broke rootwrap :)01:12
*** rpodolyaka1 has quit IRC01:14
*** openstack has joined #openstack-ironic01:15
*** openstack has joined #openstack-ironic01:19
*** nosnos has joined #openstack-ironic01:19
*** openstack has quit IRC01:24
*** openstack has joined #openstack-ironic01:25
-dickson.freenode.net- [freenode-info] if you're at a conference and other people are having trouble connecting, please mention it to staff: http://freenode.net/faq.shtml#gettinghelp01:25
*** openstack has joined #openstack-ironic15:09
devanandaShrews: some comments on 80022 -- rloo spotted some issues. could be fixed in that patch or a follow in15:12
devanandaNobodyCam: want to merge with 80425?15:12
*** yuriyz has quit IRC15:14
*** rpodolyaka has joined #openstack-ironic15:15
*** dwalleck_ has joined #openstack-ironic15:16
NobodyCam80022 w/ 8042515:17
NobodyCamoh fyi: starting to push up tripleo patches again: https://review.openstack.org/#/c/80601 http://review.openstack.org/#/c/80595/ https://review.openstack.org/#/c/7296915:20
NobodyCammore to come15:21
*** dwalleck has joined #openstack-ironic15:21
*** dwalleck has quit IRC15:22
*** dwalleck_ has quit IRC15:22
NobodyCamdevananda: I didn't quite understand the merge w/ comment? .. most likely due to lack of coffee15:27
*** killerVJT has quit IRC15:29
devanandaNobodyCam: "i think i have a patch to ..."15:31
devanandaNobodyCam: i meant add your changes on _retry_if_service to my patch. or something. or not, that's fine too :)15:32
NobodyCamoh yes deff lack of coffee there15:32
NobodyCam:)15:32
NobodyCamya I can add to your patch :)15:33
Shrewsdevananda: to make sure i understand your comments in 80022, the 2 deploy.validate() calls should catch InvalidParameterValue and reraise InstanceDeployFailure. Not sure what you want done with the power.validate() call... do you want it wrapped by the decorator as well?15:36
devanandaShrews: what line for power.validate?15:37
Shrews23915:37
devanandaah15:37
devanandaShrews: InvalidState?15:40
devanandaShrews: or InvalidParameterValue, which it's already raising... i would prefer not to invent a new exception15:40
devanandaalternatively, we could remove L239-241 and just compare to the node.power_state in the db15:41
devanandathat's probably best15:41
devanandaseparate patch, though15:41
devanandabrb15:42
*** rpodolyaka has quit IRC15:42
Shrewsdevananda: these suggested changes go beyond the purpose of the original patch. let's do them all in a separate patch15:42
*** drumkilla is now known as rustlebee15:47
NobodyCamoh wait its casual nick Friday15:48
*** NobodyCam is now known as BlartVersenwald15:49
*** romcheg is now known as anyone_else15:51
anyone_elseY not :)15:51
BlartVersenwaldlol morning anyone_else :)15:52
BlartVersenwaldhow are things going over there?15:52
anyone_elseMorning BlartVersenwald15:52
BlartVersenwaldlol the news did a show on the ex prez's house. wow I want to take the tour!15:53
anyone_elseBlartVersenwald: Me too :)15:54
BlartVersenwald:) if you take lots of pics15:55
anyone_elseIt's quite here, at least now. However we have to be ready to evacuate any moment15:55
BlartVersenwald:(15:55
anyone_elseRussia moves more and more military facilities to the borders near Kharkiv (about 40 km)15:56
BlartVersenwaldthats really a tuff day to day. ya I hear 150K troops..15:56
BlartVersenwald:(15:56
BlartVersenwaldwill any of this impact your travel plans for the may summit?15:57
anyone_elseThe main problem is the visa. My interview is scheduled to the end of March.15:57
BlartVersenwald:)15:58
anyone_elseHope they won't manage to get to Kyiv :)15:58
BlartVersenwaldya,15:58
devanandaheading down town, bbiab16:06
BlartVersenwaldgoing into the office?16:06
devanandayep16:07
*** anyone_else has left #openstack-ironic16:07
BlartVersenwald:) say Hi ti folks from /me16:07
*** anyone_else has joined #openstack-ironic16:07
*** eghobo has joined #openstack-ironic16:07
anyone_elseBlartVersenwald: We got lots of photos like this http://www.taminfo.ru/uploads/posts/2014-03/1394692176_sostav.jpg for the last two or three days. That's Belgorod, near the border with Kharkiv16:08
*** matty_dubs is now known as matty_dubs|lunch16:09
*** ndipanov has quit IRC16:11
BlartVersenwaldoh man, thats does not look like a good sign :(16:11
lucasagomesdevananda, left some comments at 8042516:15
BlartVersenwaldmorning lucasagomes :)16:18
*** comstud is now known as bearhands16:19
lucasagomesBlartVersenwald, morning :)16:19
*** rpodolyaka has joined #openstack-ironic16:25
*** vkozhukalov_ has joined #openstack-ironic16:26
BlartVersenwaldbbt...brb16:29
*** jistr has quit IRC16:31
*** martyntaylor has left #openstack-ironic16:38
*** openstackstatus has joined #openstack-ironic16:38
BlartVersenwaldpost bbt walkies ... brb16:40
*** blamar has quit IRC16:41
*** blamar has joined #openstack-ironic16:42
lucasagomesBlartVersenwald, oh now I know ur NobodyCam hah16:42
lucasagomesouch16:42
openstackgerritA change was merged to openstack/ironic: Fix typo tenet -> tenant  https://review.openstack.org/8040316:44
*** ifarkas has quit IRC16:47
*** ifarkas has joined #openstack-ironic16:47
*** jgrimm has quit IRC16:50
*** rpodolyaka has quit IRC16:51
*** anyone_else has quit IRC16:53
*** romcheg has joined #openstack-ironic16:54
jroll07:07:15      @devananda | jroll: you guys had some arch diagrams from the sprint, IIRC. are they posted anywhere? <- I'll try to find them for you and put them in the wiki16:55
BlartVersenwaldlucasagomes: LOL :) it casual nic friday16:56
lucasagomeshah I see16:56
lucasagomesnice16:56
*** rpodolyaka has joined #openstack-ironic16:58
*** harlowja_away is now known as harlowja17:02
jrolldevananda: that diagram: https://www.lucidchart.com/publicSegments/view/532335f8-edc4-40c2-804e-11060a009433/image.pdf17:03
lucasagomesjroll, Ironic neutron api?! Ironic neutron db?!17:06
lucasagomesjroll, or that's just neutron api/neutron db17:06
*** matty_dubs|lunch is now known as matty_dubs17:06
jrolllucasagomes: right, it's api/db for "the neutron running for ironic's purposes"17:07
lucasagomesjroll, ah, ack17:07
devanandaback17:07
BlartVersenwaldwb devananda :)17:07
lucasagomeswb :)17:08
*** max_lobur1 has quit IRC17:09
*** tatyana has quit IRC17:26
* devananda is watching jenkins do its first run of the new dsvm-virtual-ironic job17:30
BlartVersenwald:) nie17:30
BlartVersenwaldnice even17:30
BlartVersenwaldhumm my undercloud appered to deploy but heat is still saying CREATE_IN_PROGRESS17:32
*** dshulyak has quit IRC17:34
*** athomas has quit IRC17:38
*** rpodolyaka has quit IRC17:45
*** rpodolyaka has joined #openstack-ironic17:48
*** jdob has quit IRC17:51
*** jdob has joined #openstack-ironic17:52
*** romcheg has quit IRC17:56
devanandaok, jenkins didn't set up devstack with nova-ironic driver17:57
*** derekh has quit IRC17:58
BlartVersenwald:(18:06
lucasagomesdevananda, BlartVersenwald I gotta run18:07
devanandalucasagomes: have a godo weekend!18:08
lucasagomesnext week I will be travelling so I won't be online that often18:08
lucasagomesI might miss the meeting as well  :(18:08
devanandaack18:08
lucasagomescause I will travel on monday18:08
lucasagomesdevananda, BlartVersenwald have a good weeked as well!18:08
lucasagomesg'night :)18:08
BlartVersenwaldnight lucasagomes18:10
*** romcheg has joined #openstack-ironic18:12
*** killerVJT has joined #openstack-ironic18:18
*** killerVJT has quit IRC18:20
devanandafixing up 70348 now... getting very close to landing it :)18:24
*** jdob has quit IRC18:25
BlartVersenwaldheat is up with heat this morning...grrrr18:26
devanandaadam_g: is BM_BUILD_DEPLOY_RAMDISK=True even necessary? it looks like that should be the default ...18:27
*** rwsu has quit IRC18:30
*** jgrimm has joined #openstack-ironic18:31
BlartVersenwaldbrb18:36
openstackgerritDavid Shrewsbury proposed a change to openstack/ironic: Suppress conductor logging of expected exceptions  https://review.openstack.org/8002218:37
*** rpodolyaka has quit IRC18:39
*** rpodolyaka has joined #openstack-ironic18:47
Shrewsdevananda: so, i was working on a separate patch to catch the deploy.validate() InvalidParameterValue and instead raise InstanceDeployFailure. Turns out we have 2 test cases (test_do_node_deploy_validate_fail and test_do_node_tear_down_validate_fail) that depend on the InvalidParameterValue exception being thrown. So, question is, is this a change we really want to make?18:53
devanandaShrews: i think deploy() and tear_down() should raise InstanceDeployFailure exceptions, and thus the unit tests should be changed.18:56
Shrewsdevananda: roger that. just making sure  :)18:56
devananda:)18:56
*** jistr has joined #openstack-ironic19:04
*** rpodolyaka has quit IRC19:14
*** rpodolyaka has joined #openstack-ironic19:17
BlartVersenwaldbrb19:23
*** rpodolyaka has quit IRC19:25
*** rpodolyaka has joined #openstack-ironic19:26
openstackgerritChris Behrens proposed a change to openstack/ironic: Remove redundant nova virt driver test  https://review.openstack.org/8066819:36
openstackgerritChris Behrens proposed a change to openstack/ironic: Add admin_auth_token option for nova driver  https://review.openstack.org/8066919:36
bearhandsdevananda, jroll: ^^ this is what I was thinking for what we discussed yesterday19:37
bearhandshm, rest of my commit message is gone19:39
* bearhands addresses19:39
openstackgerritChris Behrens proposed a change to openstack/ironic: Add admin_auth_token option for nova driver  https://review.openstack.org/8066919:41
BlartVersenwaldwoo hoo heat detected the under cloud depoly19:44
openstackgerritDavid Shrewsbury proposed a change to openstack/ironic: Change deploy validation exception handling  https://review.openstack.org/8067119:45
openstackgerritDavid Shrewsbury proposed a change to openstack/ironic: Change deploy validation exception handling  https://review.openstack.org/8067119:47
*** rpodolyaka has quit IRC19:47
*** rpodolyaka has joined #openstack-ironic19:49
devanandaheading out for lunch, bbiaw19:57
*** Alexei_9871 has quit IRC19:57
BlartVersenwald:)19:58
BlartVersenwaldenjoy19:58
*** jistr has quit IRC20:05
jrollbearhands: nice, thanks20:05
*** rpodolyaka has quit IRC20:10
BlartVersenwaldwait call-back20:12
BlartVersenwalddeploy failed20:12
BlartVersenwaldcloser though20:12
*** romcheg has quit IRC20:13
*** rpodolyaka has joined #openstack-ironic20:15
*** rpodolyaka has quit IRC20:16
BlartVersenwaldoh now thats a strange error20:17
BlartVersenwaldironic-rootwrap: command not found\n20:17
*** rpodolyaka has joined #openstack-ironic20:18
*** romcheg has joined #openstack-ironic20:25
* BlartVersenwald rebuilds undercloud and goes afk to look for some food20:26
*** rwsu has joined #openstack-ironic20:28
lifelesslucasagomes: thanks for investigating that patch!20:38
*** romcheg has quit IRC20:47
*** rpodolyaka has quit IRC20:48
openstackgerritA change was merged to openstack/ironic: Remove redundant nova virt driver test  https://review.openstack.org/8066820:49
*** vkozhukalov_ has quit IRC20:53
openstackgerritlifeless proposed a change to openstack/ironic: Permit passing SSH keys into the Ironic API  https://review.openstack.org/8037620:53
*** lazy_prince has quit IRC20:57
BlartVersenwaldcd ..21:01
BlartVersenwaldgha21:01
*** killer_prince has joined #openstack-ironic21:05
openstackgerritlifeless proposed a change to openstack/ironic: Permit passing SSH keys into the Ironic API  https://review.openstack.org/8037621:07
*** datajerk has joined #openstack-ironic21:09
*** datajerk1 has quit IRC21:09
*** datajerk1 has joined #openstack-ironic21:12
*** datajerk has quit IRC21:13
*** matty_dubs is now known as matty_dubs|gone21:13
BlartVersenwalddevananda: you back ?21:18
*** datajerk has joined #openstack-ironic21:18
*** datajerk1 has quit IRC21:19
*** openstackgerrit has quit IRC21:21
*** openstackgerrit has joined #openstack-ironic21:21
lifelesslucasagomes: should be all fixed21:27
*** openstackgerrit has quit IRC21:32
*** openstackgerrit has joined #openstack-ironic21:32
*** eguz has joined #openstack-ironic21:37
*** vkozhukalov_ has joined #openstack-ironic21:37
BlartVersenwaldbrb21:39
*** eghobo has quit IRC21:41
*** rpodolyaka has joined #openstack-ironic21:47
*** eguz has quit IRC22:18
*** eghobo has joined #openstack-ironic22:19
*** jrist has quit IRC22:20
*** jrist has joined #openstack-ironic22:20
*** rpodolyaka has quit IRC22:26
*** eguz has joined #openstack-ironic22:26
*** eghobo has quit IRC22:29
devanandaback22:42
BlartVersenwaldwb22:43
* BlartVersenwald builds another undercloud22:43
*** linggao has quit IRC22:47
*** rpodolyaka has joined #openstack-ironic22:49
BlartVersenwaldany other core what to approve https://review.openstack.org/#/c/80365 !!!!!23:01
BlartVersenwaldif not I will :-p23:02
*** Lotus907efi is now known as Lotus907efi-23:27
*** rpodolyaka has quit IRC23:32
*** openstackstatus has quit IRC23:36
*** Lotus907efi- has quit IRC23:36
*** rwsu has quit IRC23:38
*** openstackgerrit has quit IRC23:39
*** killer_prince has quit IRC23:42
*** eguz has quit IRC23:45
*** yongli has quit IRC23:46
*** SpamapS has quit IRC23:46
*** Hefeweizen has quit IRC23:46
*** jb11211 has quit IRC23:46
*** rainya has quit IRC23:46
*** JoshNang has quit IRC23:46
*** dtantsur has quit IRC23:46
*** jrist has quit IRC23:46
*** vkozhukalov_ has quit IRC23:46
*** harlowja has quit IRC23:46
*** antonym has quit IRC23:46
*** mrda has quit IRC23:46
*** Madasi has quit IRC23:46
*** shortstop has quit IRC23:46
*** datajerk has quit IRC23:46
*** JayF has quit IRC23:46
*** EmilienM has quit IRC23:46
*** lsmola has quit IRC23:46
*** dividebin has quit IRC23:46
*** jgrimm has quit IRC23:46
*** blamar has quit IRC23:46
*** lynxman has quit IRC23:46
*** sbadia has quit IRC23:46
*** dhellmann has quit IRC23:46
*** pradipta_away has quit IRC23:46
*** morgabra has quit IRC23:46
*** ewindisch has quit IRC23:46
*** Shrews has quit IRC23:46
*** mdurnosvistov has quit IRC23:46
*** mikal has quit IRC23:46
*** davidlenwell has quit IRC23:46
*** Slower has quit IRC23:46
*** aignatov has quit IRC23:46
*** ifarkas has quit IRC23:46
*** matty_dubs|gone has quit IRC23:46
*** dkehn_ has quit IRC23:46
*** early has quit IRC23:46
*** pleia2 has quit IRC23:46
*** bearhands has quit IRC23:46
*** lifeless has quit IRC23:46
*** mordred has quit IRC23:46
*** Haomeng has quit IRC23:46
*** lucasagomes has quit IRC23:46
*** bigjools has quit IRC23:46
*** zigo has quit IRC23:46
*** GheRivero has quit IRC23:46
*** openstack has joined #openstack-ironic23:46
*** rpodolyaka1 has joined #openstack-ironic23:47
*** openstackstatus has joined #openstack-ironic23:47
*** Lotus907efi1 has joined #openstack-ironic23:48

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