Friday, 2014-02-28

*** jcru has quit IRC00:00
zanebSpamapS: the ones that have succeeded should be in UPDATE_COMPLETE; they aren't hard to spot?00:01
SpamapSwe could probably even glean that from the event-list00:01
SpamapSzaneb: we'd have to go back and change them to something other than UPDATE_COMPLETE after the update finished, or the next update would be fooled.00:01
zaneboh right00:02
SpamapSbut the event-list ...00:02
*** arbylee has joined #heat00:02
zanebyou need to walk through the graph until you find one that is UPDATE_FAILED and start there00:02
zaneband after that ignore the status00:02
SpamapSzaneb: there will be others on the same level of the graph that want to happen in parallel though.00:02
SpamapSunless we don't do updates in parallel00:02
SpamapSat which point, we could in fact do just that.00:03
*** andrew_plunk has joined #heat00:03
zanebSpamapS: right, you have to do what I said above for every path in the graph00:03
*** arbylee has quit IRC00:04
SpamapSzaneb: also sometimes the resource that caused the issue is CREATE_FAILED .. but I think that it still works if you take action out of it.00:05
zanebyes, agreed, the action is unimportant00:06
zanebjust the status00:06
zanebThings I Don't Want to Read in a Bug Report Under "Reproduction"00:07
zaneb#1: "Set your DIB environment variables appropriately"00:07
zanebsdake: ^ any idea which environment variables he meant?00:09
*** david-lyle has quit IRC00:13
*** topol has joined #heat00:15
stevebakerzaneb: probably ELEMENTS_PATH to point at your elements collection http://git.openstack.org/cgit/openstack/diskimage-builder/tree/README.md00:16
zanebthanks, I think it's actually this: https://github.com/openstack/diskimage-builder/blob/master/elements/rhel/README.md00:16
zanebfound it in another bug report00:16
*** zns has quit IRC00:17
*** shakayumi has joined #heat00:20
*** shakayumi has quit IRC00:21
*** shakayumi has joined #heat00:21
*** shakayumi has quit IRC00:21
*** tspatzier has joined #heat00:21
*** openstackgerrit has joined #heat00:24
*** arbylee has joined #heat00:31
*** cadenzajon_ has quit IRC00:31
*** Tross has quit IRC00:31
*** Tross has joined #heat00:34
*** Tross has quit IRC00:34
*** erkules has quit IRC00:38
*** pvaneck has quit IRC00:38
*** tspatzier has quit IRC00:39
*** maxskew__ has joined #heat00:39
*** erkules has joined #heat00:39
*** pvaneck has joined #heat00:39
*** shortstop has quit IRC00:40
*** bnemec has quit IRC00:40
*** pasquier-s has quit IRC00:40
*** openstack has joined #heat00:48
*** _jmp_ has joined #heat00:49
*** gokrokve has quit IRC00:49
*** _jmp_ is now known as Guest1510200:49
*** gokrokve has joined #heat00:49
*** maxskew has joined #heat00:50
*** nati_uen_ has joined #heat00:50
*** chandankumar_ has joined #heat00:51
*** bnemec has quit IRC00:54
*** pasquier-s has quit IRC00:54
*** arbylee has quit IRC00:54
*** topol has quit IRC00:54
*** Guest76806 has quit IRC00:54
*** cody-somerville has quit IRC00:54
*** gokrokve has quit IRC00:54
*** tspatzier has quit IRC00:57
*** bada has quit IRC00:58
*** maxskew has quit IRC01:00
*** bada has joined #heat01:00
*** alexpilotti has quit IRC01:00
*** openstack has joined #heat01:00
*** giulivo has quit IRC01:00
SpamapSzaneb: this appears to work for simple cases.. Writing tests for the more complex ones requires careful thought though.01:03
*** dims has quit IRC01:03
SpamapSas with all concurrent things.. testing how it handles all of the things that can happen in all of the orders is the challenge. :-P01:03
*** julienvey1 has joined #heat01:05
*** maxskew has joined #heat01:05
*** swygue has quit IRC01:05
*** maxskew__ has quit IRC01:06
*** jamieh has quit IRC01:06
*** skraynev has joined #heat01:06
*** cody-somerville has joined #heat01:06
*** cody-somerville has quit IRC01:06
*** cody-somerville has joined #heat01:06
*** bnemec has joined #heat01:06
*** pvaneck has quit IRC01:06
*** jamieh has joined #heat01:07
*** nati_ueno has quit IRC01:08
*** chandankumar_ has quit IRC01:08
*** chandankumar_ has joined #heat01:08
*** pasquier-s has joined #heat01:10
*** achampion has joined #heat01:14
*** dims has joined #heat01:18
*** WinnieTsang has quit IRC01:20
*** ramishra has joined #heat01:21
SpamapSwell my brain has melted.. :-P01:21
*** openstack has quit IRC01:26
*** openstack has joined #heat01:29
*** randallburt is now known as 23LAACT9B01:29
*** randallburt has joined #heat01:29
*** therve` has joined #heat01:29
*** cody-somerville has quit IRC01:30
*** daneyon has quit IRC01:30
*** andersonvom has quit IRC01:30
*** randallburt has quit IRC01:30
*** 23LAACT9B has quit IRC01:30
*** 23LAACT9B has joined #heat01:30
*** larsks has joined #heat01:31
*** IgorYozhikov has joined #heat01:32
*** gokrokve has joined #heat01:35
openstackgerrithuangtianhua proposed a change to openstack/heat: Fix InternalException raised on stack-show  https://review.openstack.org/7642901:35
*** lazy_prince is now known as killer_prince01:36
openstackgerritA change was merged to openstack/heat: Convert AccessKey resource to attributes_schema  https://review.openstack.org/7627501:44
tangoSpamapS: maybe I can help with writing the unit tests01:44
SpamapStango: I've split it up a bit, and I think I'm on track now.. just have to keep brain from overheating and melting again ;)01:45
*** nosnos has joined #heat01:45
tangoSpamapS: pace yourself :)   I have been staring at the update code all day to see how to merge the code for create fail in the stack update code01:47
*** cody-somerville has joined #heat01:47
SpamapStango: the thing I wrote before for resuming failed creates still works fine I think01:48
sdakezaneb sorry I was out01:50
tangoSpamapS: that's what I am thinking also, it seems I need to do some preparation to reuse the same template and keeping the action as CREATE instead of UPDATE01:50
sdakezaneb it is that readme01:50
sdakealso you need to overrride IMAGE_something01:50
zanebyep, I got it working, thanks01:50
sdakethe default is old in dib01:50
sdakeya sorry for delay was at folks eating01:50
*** alexpilotti has quit IRC01:50
zanebnow if I can get mariadb to start I will be able to test it :|01:50
*** alexpilotti has joined #heat01:51
*** pablosan_ has quit IRC01:53
*** tango has quit IRC01:57
*** pablosan has joined #heat01:58
*** guohliu has joined #heat02:00
*** fandi has joined #heat02:05
*** Tross has joined #heat02:09
*** cody-somerville has quit IRC02:11
*** erkules_ has joined #heat02:20
*** zhiyan_ is now known as zhiyan02:21
*** erkules has quit IRC02:22
openstackgerrithuangtianhua proposed a change to openstack/heat: Ensure property 'name' can be updated  https://review.openstack.org/7462402:29
openstackgerrithuangtianhua proposed a change to openstack/heat: Fix InternalException raised on stack-show  https://review.openstack.org/7642902:37
*** arbylee1 has quit IRC02:37
*** david-lyle has joined #heat02:37
*** yogesh has joined #heat02:37
*** gokrokve has quit IRC02:42
*** gokrokve has joined #heat02:43
*** gokrokve has quit IRC02:43
*** dtalton has joined #heat02:44
*** spzala_ has quit IRC02:44
*** yogesh has quit IRC02:45
*** dtalton has quit IRC02:46
*** andersonvom has joined #heat02:49
*** pablosan has quit IRC02:49
*** nati_uen_ has quit IRC02:51
sdakenew bitcoin miner on tuesday \o/ ;)02:58
sdakeand off to bed, night all02:58
*** killer_prince is now known as lazy_prince03:02
*** andersonvom has quit IRC03:02
*** daneyon has joined #heat03:03
sdakelifeless ping03:04
sdakelifeless nm I'm heading off for the night, I'll catch up with you tomorrow03:05
sdakewanted to chat about concoord03:05
sdakeor perhaps monday since tomorrow is saturday for you :)03:06
lifelesssdake: monday i'm in the tripleo sprint in sunnyvale03:06
lifelesssdake: but yes, lets03:06
sdakereal quick, do you mean use concoord in heat, or use concoord in place of heat03:06
lifelessthe former03:06
*** daneyon_ has quit IRC03:07
lifelessexpress [much of] heat's internals in it though03:07
sdakewouldn't mistral be a better fit for that long term?03:08
sdakeI guess I don't knokw what concoord does, need to do some reading :)03:08
sdakeok thanks for not giving me a panic attack :)03:09
sdakenight03:09
lifelesssdake: mistral is a workflow engine03:11
lifelesssdake: concoord is a consensus + replicated state machine engine03:11
sdake__lifeless - switched computers to nerd-surf03:11
sdake__i see, so like zookeeper ?03:11
lifelessexcept not java , not based on writing files-with-zomg-locking, and not sucking03:12
sdake__does it work ? :)03:12
lifelessyes03:12
lifelessits based on paxos03:12
lifelesswhich is old and well nown03:12
lifelessknown03:12
lifelessso for instance, here is one of the ideas I have03:13
lifelesstake multi-engine heat03:13
lifelessas a small step03:13
lifelessif we stored the state machine for stack updates in concoord03:13
lifelessand the engine that was 'master' goes down03:13
lifelessthe stack update will proceed automatically03:13
lifelessbecause another engine will become master03:14
sdake__I am familiar with paxos03:14
lifelessand get called from the local statemachine to carry on03:14
lifelesswe could have a state machine per stack that is being updated concurrently, delete them when the update is complete03:14
lifelesssdake__: [paxos - good, wasn't explaining paxos here - assumed you knew it :)03:15
sdake__I think paxos rocks but it has serious IP issues03:15
lifelessoh?03:15
sdake__everytime microsoft writes a paper about it they put their 10+ patents in the paper description03:16
* sdake__ ughs03:16
lifelessoh03:17
lifelessso, I think we'd probably ask the board then if that is an issue03:17
lifelessI rather suspect we have enough to balance things03:17
lifeless(HP)03:17
sdake__;)03:17
sdake__well its the board's job to protect promote blah blah :)03:18
lifelessso I guess I see a migration strategy of - start by using concoord to select which engine does a given piece of work, then start storing state machines in there to let work be handed off across engine shutdowns etc03:18
sdake__I'd probably be better off if I hadn't read the research papers :(03:18
lifelessI see checkpointing workflow kindof things in a DB as orthogonal to this03:19
lifelesswhich is more a way of writing fairly straightforward python that will be replicated and applied once and only once across a cluster for us03:19
lifelesssdake__: did you see the http://openreplica.org/ link ?03:28
lifelesswhich is where I found concoord03:28
lifelesssdake__: so the thing I *started* looking for was a nice way to initialise the rabbit and galera clusters for TripleO03:28
sdake__looking03:28
sdake__ya there is a thread on our internal list about that topic one of the engineers asked folks about03:29
lifelessJan ?03:29
sdake__yup03:29
sdake__asking how we sort it out in rhel ha03:29
lifelessso in the short term we're going to do a hack03:29
sdake__I saw the review03:29
lifelessbut in the medium/long term I'd like to do something nice03:29
sdake__looks fragile :)03:29
sdake__have you thought about synchronizing via iscsi03:30
*** arbylee has joined #heat03:30
lifelesse.g. elect a leader from a zero-knowledge-other-than-peers-list and then do the critical section stuff under its direction03:30
lifelesssdake__: I hadn't. Now I can't get the image out of my head.03:30
lifelesssdake__: eek03:30
sdake__don't like it? :)03:30
lifelesssdake__: so, chicken and egg. How do we init the cluster fs the data needs to be on03:31
lifelesssdake__: if we don't cluster the fs, we have failure modes if the server it is on glitches on deploy03:31
sdake__don't run an fs on it03:32
sdake__cluster fs requires a dlm in most cases anyway - just use the iscsi block device as a data blob for synchronizing servers03:32
sdake__block 1 = machine 1, block 2 = machine 203:33
sdake__init on start03:33
sdake__some cats use storage for cluster membership - its not that hard, the performance just sucks03:34
sdake__even more daring cats use shared storage (eg iscsi device) for messaging03:35
sdake__that is really not ideal :)03:35
sdake__best part is no daemon :)03:36
sdake__shared storage membership is pretty easy peasy with 2pc03:37
sdake__anyway just an idea - maybe crackheaded ;)03:38
*** 23LAACT9B has quit IRC03:38
*** erkules_ is now known as erkules03:44
*** chandan_kumar has joined #heat03:46
*** radez_g0n3 is now known as radez03:47
*** jamieh has quit IRC03:48
*** ramishra_ has joined #heat03:53
openstackgerritA change was merged to openstack/heat: Raise NotFound in software_config_get/software_deployment_get  https://review.openstack.org/7636903:53
openstackgerritA change was merged to openstack/heat: RPC method to fetch deployments metadata  https://review.openstack.org/7420203:53
*** ramishra has quit IRC03:53
openstackgerritA change was merged to openstack/heat: REST deployment metadata method  https://review.openstack.org/7420303:54
*** harlowja is now known as harlowja_away03:55
openstackgerritA change was merged to openstack/heat: SignalResponder move signed URL deleting to its own method  https://review.openstack.org/7420503:55
*** cody-somerville has joined #heat03:57
*** chandan_kumar has quit IRC03:57
*** achampion has quit IRC04:08
openstackgerritSteve Baker proposed a change to openstack/heat: Change software_config storage of config and io columns  https://review.openstack.org/7706104:08
openstackgerritSteve Baker proposed a change to openstack/heat: Resource type implementations for structured software config  https://review.openstack.org/7420604:09
openstackgerritSteve Baker proposed a change to openstack/heat: OS::Nova::Server support for software config  https://review.openstack.org/6762504:09
openstackgerritSteve Baker proposed a change to openstack/heat: Resource type implementation for software deployment  https://review.openstack.org/6762404:09
openstackgerritSteve Baker proposed a change to openstack/heat: Nova server to ref cloud-config resources in user_data  https://review.openstack.org/6923804:09
*** radez is now known as radez_g0n304:10
*** lnxnut has joined #heat04:15
*** radez_g0n3 is now known as radez04:18
*** radez is now known as radez_g0n304:22
*** arbylee has quit IRC04:24
*** openstack has joined #heat04:38
*** erkules has joined #heat04:39
*** dims has joined #heat04:39
*** jasond`` has joined #heat04:39
*** IYozhikov_away has joined #heat04:41
*** IgorYozhikov has quit IRC04:41
*** larsks has joined #heat04:42
*** daneyon_ has joined #heat04:42
*** larsks|alt has quit IRC04:42
*** daneyon has quit IRC04:42
openstackgerritMitsuru Kanabuchi proposed a change to openstack/heat: Implement OS::Neutron::ExtraRoute as /contrib  https://review.openstack.org/7489904:43
*** DandyPandy_ has quit IRC04:43
*** IYozhikov_away has quit IRC04:48
*** openstack has quit IRC04:48
*** openstack has joined #heat04:50
*** vaibhav has joined #heat04:51
*** chandan_kumar has joined #heat04:54
*** openstack has joined #heat05:02
*** david_lyle_ has joined #heat05:02
*** ramishra has joined #heat05:03
*** bnemec_ has joined #heat05:03
*** Tross has joined #heat05:03
*** akuznets_ has joined #heat05:03
*** john-n-s- has joined #heat05:05
*** zhiyan- has joined #heat05:06
*** tspatzier has joined #heat05:06
*** openstack has quit IRC05:10
*** openstack has joined #heat05:17
*** openstack has quit IRC05:22
*** openstack has joined #heat05:29
*** dickson.freenode.net sets mode: +ns 05:29
*** dickson.freenode.net sets mode: -o openstack05:30
-dickson.freenode.net- *** Notice -- TS for #heat changed from 1393565383 to 133236478105:30
*** dickson.freenode.net sets mode: +cpt-s 05:30
*** akuznets_ has joined #heat05:30
*** zhiyan has joined #heat05:30
*** bnemec has joined #heat05:30
*** liang has joined #heat05:30
*** tspatzier has joined #heat05:30
*** john-n-s- has joined #heat05:30
*** Tross has joined #heat05:30
*** ramishra has joined #heat05:30
*** david_lyle_ has joined #heat05:30
*** dims_ has joined #heat05:30
*** IYozhikov_away has joined #heat05:30
*** DandyPandy has joined #heat05:30
*** daneyon_ has joined #heat05:30
*** larsks has joined #heat05:30
*** jasond`` has joined #heat05:30
*** erkules has joined #heat05:30
*** nosnos_ has joined #heat05:30
*** fandi has joined #heat05:30
*** guohliu has joined #heat05:30
*** therve` has joined #heat05:30
*** scroiset1 has joined #heat05:30
*** saurabhs1 has joined #heat05:30
*** pasquier-s has joined #heat05:30
*** skraynev has joined #heat05:30
*** maxskew has joined #heat05:30
*** julienvey1 has joined #heat05:30
*** bada has joined #heat05:30
*** chandankumar_ has joined #heat05:30
*** Guest15102 has joined #heat05:30
*** zpatten has joined #heat05:30
*** dteselkin_ has joined #heat05:30
*** topol_ has joined #heat05:30
*** glucas has joined #heat05:30
*** sirushti has joined #heat05:30
*** openstackgerrit has joined #heat05:30
*** SpamapS has joined #heat05:30
*** sgran_ has joined #heat05:30
*** sdake__ has joined #heat05:30
*** rpothier has joined #heat05:30
*** Michalik has joined #heat05:30
*** dmakogon_ has joined #heat05:30
*** funzo has joined #heat05:30
*** blamar has joined #heat05:30
*** sballe has joined #heat05:30
*** jdag_ has joined #heat05:30
*** asalkeld has joined #heat05:30
*** jrist has joined #heat05:30
*** lazy_prince has joined #heat05:30
*** varora has joined #heat05:30
*** Slower has joined #heat05:30
*** mestery has joined #heat05:30
*** sbadia has joined #heat05:30
*** grncdr has joined #heat05:30
*** FL1SK has joined #heat05:30
*** ruhe has joined #heat05:30
*** aignatov_ has joined #heat05:30
*** sdake has joined #heat05:30
*** pscheie_ has joined #heat05:30
*** stevebaker has joined #heat05:30
*** jpeeler has joined #heat05:30
*** rwsu has joined #heat05:30
*** sergmelikyan has joined #heat05:30
*** ccorrigan has joined #heat05:30
*** SergeyLukjanov has joined #heat05:30
*** tteggel has joined #heat05:30
*** zaneb has joined #heat05:30
*** scollier has joined #heat05:30
*** mkerrin has joined #heat05:30
*** nijaba has joined #heat05:30
*** lsmola has joined #heat05:30
*** harlowja_away has joined #heat05:30
*** wirehead_ has joined #heat05:30
*** christophk_ has joined #heat05:30
*** Ng has joined #heat05:30
*** shardy_afk has joined #heat05:30
*** ChanServ has joined #heat05:30
*** aliljet has joined #heat05:30
*** kevinbenton has joined #heat05:30
*** rockstar has joined #heat05:30
*** mikal has joined #heat05:30
*** radix has joined #heat05:30
*** gondoi has joined #heat05:30
*** cyli has joined #heat05:30
*** samkottler has joined #heat05:30
*** russellb has joined #heat05:30
*** wendar has joined #heat05:30
*** lekha has joined #heat05:30
*** zigo has joined #heat05:30
*** fungi has joined #heat05:30
*** chmouel has joined #heat05:30
*** clarkb has joined #heat05:30
*** abramley has joined #heat05:30
*** pleia2 has joined #heat05:30
*** Daviey has joined #heat05:30
*** dickson.freenode.net sets mode: +o ChanServ05:30
*** mordred has joined #heat05:30
*** BLZbubba_ has joined #heat05:30
*** lifeless has joined #heat05:30
*** lon|gone has joined #heat05:30
*** radez_g0n3 has joined #heat05:30
*** mnaser has joined #heat05:30
*** anteaya has joined #heat05:30
*** shadower has joined #heat05:30
*** aedocw has joined #heat05:30
*** EmilienM has joined #heat05:30
*** plomakin_ has joined #heat05:30
*** sileht has joined #heat05:30
*** dklyle has joined #heat05:30
*** PhilK has joined #heat05:30
*** brint has joined #heat05:30
*** 1JTAAFJP3 has joined #heat05:30
*** q3k has joined #heat05:30
*** dickson.freenode.net changes topic to "support @ https://ask.openstack.org | developer wiki @ https://wiki.openstack.org/wiki/Heat | development @ https://launchpad.net/heat | logged @ http://eavesdrop.openstack.org/irclogs/%23heat/"05:30
*** openstack has quit IRC05:34
*** openstack has joined #heat05:44
*** IYozhikov_away has quit IRC05:49
*** DandyPandy has quit IRC05:49
*** jasond`` has quit IRC05:49
*** q3k has quit IRC05:49
*** bada has quit IRC05:49
*** chandankumar_ has quit IRC05:49
*** Guest15102 has quit IRC05:49
*** zpatten has quit IRC05:49
*** dteselkin_ has quit IRC05:49
*** glucas has quit IRC05:49
*** sirushti has quit IRC05:49
*** SpamapS has quit IRC05:49
*** sgran_ has quit IRC05:49
*** rpothier has quit IRC05:49
*** dmakogon_ has quit IRC05:49
*** funzo has quit IRC05:49
*** sballe has quit IRC05:49
*** lazy_prince has quit IRC05:49
*** ruhe has quit IRC05:49
*** aignatov_ has quit IRC05:49
*** sdake has quit IRC05:49
*** pscheie_ has quit IRC05:49
*** blamar has quit IRC05:49
*** jdag_ has quit IRC05:49
*** sbadia has quit IRC05:49
*** varora has quit IRC05:49
*** FL1SK has quit IRC05:49
*** harlowja_away has quit IRC05:49
*** mkerrin has quit IRC05:49
*** dklyle has quit IRC05:49
*** PhilK has quit IRC05:49
*** brint has quit IRC05:49
*** 1JTAAFJP3 has quit IRC05:49
*** zhiyan has quit IRC05:49
*** tspatzier has quit IRC05:49
*** john-n-s- has quit IRC05:49
*** dims_ has quit IRC05:49
*** daneyon_ has quit IRC05:49
*** erkules has quit IRC05:49
*** nosnos_ has quit IRC05:49
*** guohliu has quit IRC05:49
*** scroiset1 has quit IRC05:49
*** saurabhs1 has quit IRC05:49
*** maxskew has quit IRC05:49
*** julienvey1 has quit IRC05:49
*** sergmelikyan has quit IRC05:49
*** scollier has quit IRC05:49
*** ccorrigan has quit IRC05:49
*** topol_ has quit IRC05:49
*** openstackgerrit has quit IRC05:49
*** sdake__ has quit IRC05:49
*** asalkeld has quit IRC05:49
*** jrist has quit IRC05:49
*** Slower has quit IRC05:49
*** mestery has quit IRC05:49
*** grncdr has quit IRC05:49
*** jpeeler has quit IRC05:49
*** rwsu has quit IRC05:49
*** SergeyLukjanov has quit IRC05:49
*** tteggel has quit IRC05:49
*** zaneb has quit IRC05:49
*** nijaba has quit IRC05:49
*** lsmola has quit IRC05:49
*** plomakin_ has quit IRC05:49
*** aedocw has quit IRC05:49
*** shadower has quit IRC05:49
*** mnaser has quit IRC05:49
*** radez_g0n3 has quit IRC05:49
*** lon|gone has quit IRC05:49
*** lifeless has quit IRC05:49
*** BLZbubba_ has quit IRC05:49
*** mordred has quit IRC05:49
*** Daviey has quit IRC05:49
*** Michalik has quit IRC05:49
*** stevebaker has quit IRC05:49
*** wirehead_ has quit IRC05:49
*** sileht has quit IRC05:49
*** EmilienM has quit IRC05:49
*** clarkb has quit IRC05:49
*** chmouel has quit IRC05:49
*** fungi has quit IRC05:49
*** zigo has quit IRC05:49
*** lekha has quit IRC05:49
*** wendar has quit IRC05:49
*** russellb has quit IRC05:49
*** gondoi has quit IRC05:49
*** radix has quit IRC05:49
*** mikal has quit IRC05:49
*** rockstar has quit IRC05:49
*** kevinbenton has quit IRC05:49
*** ChanServ has quit IRC05:49
*** akuznets_ has quit IRC05:49
*** bnemec has quit IRC05:49
*** liang has quit IRC05:49
*** Tross has quit IRC05:49
*** ramishra has quit IRC05:49
*** david_lyle_ has quit IRC05:49
*** larsks has quit IRC05:49
*** fandi has quit IRC05:49
*** therve` has quit IRC05:49
*** pasquier-s has quit IRC05:49
*** skraynev has quit IRC05:49
*** anteaya has quit IRC05:49
*** pleia2 has quit IRC05:49
*** abramley has quit IRC05:49
*** samkottler has quit IRC05:49
*** cyli has quit IRC05:49
*** shardy_afk has quit IRC05:49
*** aliljet has quit IRC05:49
*** Ng has quit IRC05:49
*** christophk_ has quit IRC05:49
*** zns has joined #heat05:51
*** IgorYozhikov has joined #heat05:51
*** cody-somerville has joined #heat05:51
*** akuznets_ has joined #heat05:51
*** zhiyan has joined #heat05:51
*** bnemec has joined #heat05:51
*** liang has joined #heat05:51
*** tspatzier has joined #heat05:51
*** john-n-s- has joined #heat05:51
*** Tross has joined #heat05:51
*** ramishra has joined #heat05:51
*** david_lyle_ has joined #heat05:51
*** dims_ has joined #heat05:51
*** DandyPandy has joined #heat05:51
*** daneyon_ has joined #heat05:51
*** larsks has joined #heat05:51
*** jasond`` has joined #heat05:51
*** erkules has joined #heat05:51
*** nosnos_ has joined #heat05:51
*** fandi has joined #heat05:51
*** guohliu has joined #heat05:51
*** therve` has joined #heat05:51
*** scroiset1 has joined #heat05:51
*** saurabhs1 has joined #heat05:51
*** pasquier-s has joined #heat05:51
*** skraynev has joined #heat05:51
*** maxskew has joined #heat05:51
*** julienvey1 has joined #heat05:51
*** bada has joined #heat05:51
*** chandankumar_ has joined #heat05:51
*** Guest15102 has joined #heat05:51
*** zpatten has joined #heat05:51
*** dteselkin_ has joined #heat05:51
*** topol_ has joined #heat05:51
*** glucas has joined #heat05:51
*** sirushti has joined #heat05:51
*** openstackgerrit has joined #heat05:51
*** SpamapS has joined #heat05:51
*** sgran_ has joined #heat05:51
*** sdake__ has joined #heat05:51
*** rpothier has joined #heat05:51
*** Michalik has joined #heat05:51
*** dmakogon_ has joined #heat05:51
*** funzo has joined #heat05:51
*** blamar has joined #heat05:51
*** sballe has joined #heat05:51
*** jdag_ has joined #heat05:51
*** asalkeld has joined #heat05:51
*** jrist has joined #heat05:51
*** lazy_prince has joined #heat05:51
*** varora has joined #heat05:51
*** Slower has joined #heat05:51
*** mestery has joined #heat05:51
*** sbadia has joined #heat05:51
*** grncdr has joined #heat05:51
*** FL1SK has joined #heat05:51
*** ruhe has joined #heat05:51
*** aignatov_ has joined #heat05:51
*** sdake has joined #heat05:51
*** pscheie_ has joined #heat05:51
*** stevebaker has joined #heat05:51
*** jpeeler has joined #heat05:51
*** rwsu has joined #heat05:51
*** sergmelikyan has joined #heat05:51
*** ccorrigan has joined #heat05:51
*** SergeyLukjanov has joined #heat05:51
*** tteggel has joined #heat05:51
*** zaneb has joined #heat05:51
*** scollier has joined #heat05:51
*** mkerrin has joined #heat05:51
*** nijaba has joined #heat05:51
*** lsmola has joined #heat05:51
*** harlowja_away has joined #heat05:51
*** wirehead_ has joined #heat05:51
*** christophk_ has joined #heat05:51
*** Ng has joined #heat05:51
*** shardy_afk has joined #heat05:51
*** ChanServ has joined #heat05:51
*** aliljet has joined #heat05:51
*** kevinbenton has joined #heat05:51
*** rockstar has joined #heat05:51
*** mikal has joined #heat05:51
*** radix has joined #heat05:51
*** gondoi has joined #heat05:51
*** cyli has joined #heat05:51
*** samkottler has joined #heat05:51
*** russellb has joined #heat05:51
*** wendar has joined #heat05:51
*** lekha has joined #heat05:51
*** zigo has joined #heat05:51
*** fungi has joined #heat05:51
*** chmouel has joined #heat05:51
*** clarkb has joined #heat05:51
*** abramley has joined #heat05:51
*** dickson.freenode.net sets mode: +o ChanServ05:51
*** pleia2 has joined #heat05:51
*** Daviey has joined #heat05:51
*** mordred has joined #heat05:51
*** BLZbubba_ has joined #heat05:51
*** lifeless has joined #heat05:51
*** lon|gone has joined #heat05:51
*** radez_g0n3 has joined #heat05:51
*** mnaser has joined #heat05:51
*** anteaya has joined #heat05:51
*** shadower has joined #heat05:51
*** aedocw has joined #heat05:51
*** EmilienM has joined #heat05:51
*** plomakin_ has joined #heat05:51
*** sileht has joined #heat05:51
*** dklyle has joined #heat05:51
*** PhilK has joined #heat05:51
*** brint has joined #heat05:51
*** 1JTAAFJP3 has joined #heat05:51
*** q3k has joined #heat05:51
*** maxskew_ has joined #heat06:02
*** maxskew_ has quit IRC06:09
*** openstack has quit IRC06:09
*** openstack has joined #heat06:11
-dickson.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp06:11
*** sirushti is now known as shortstop06:18
*** amritanshu_RnD has joined #heat06:21
*** zns has quit IRC06:21
*** amritanshu_RnD is now known as Guest40006:21
*** tspatzier has quit IRC06:22
*** zhiyan is now known as zhiyan_06:29
*** asalkeld_ has quit IRC06:32
openstackgerrithuangtianhua proposed a change to openstack/heat: I18N support for log message and attributes schema  https://review.openstack.org/6911106:35
openstackgerritA change was merged to openstack/heat-cfntools: cfn-signal provides a unique default id  https://review.openstack.org/6482606:41
openstackgerritRabi Mishra proposed a change to openstack/heat: Allows vip from a different subnet for lbaas resource  https://review.openstack.org/7708406:51
*** yogesh has joined #heat06:52
*** ramishra has joined #heat06:55
*** asalkeld has joined #heat06:56
ramishratherve: hello06:57
ramishrag'morning all06:59
*** yogesh_ has joined #heat07:02
*** yogesh has quit IRC07:03
*** saurabhs1 has left #heat07:04
*** nkhare has joined #heat07:06
*** jistr has joined #heat07:09
openstackgerritKiyohiro Adachi proposed a change to openstack/heat: Remove unused variable  https://review.openstack.org/7708807:15
*** zhiyan_ is now known as zhiyan07:17
*** ramishra has quit IRC07:20
*** che-arne has joined #heat07:23
*** yogesh_ has quit IRC07:26
*** yogesh has joined #heat07:27
skraynev,orning07:28
skraynevs/,orning/morning07:29
*** saju_m has joined #heat07:32
*** sgran_ is now known as sgran07:36
*** e0ne has joined #heat07:39
*** david_lyle_ has quit IRC07:45
*** daneyon_ has quit IRC07:47
openstackgerritJenkins proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/7668907:53
*** ifarkas has joined #heat07:54
therve`'morning08:01
*** zhiyan is now known as zhiyan_08:07
*** topol_ has quit IRC08:12
*** shardy_afk is now known as shardy08:17
shardymorning all08:17
*** Guest400 has quit IRC08:18
*** bvandenh_ has joined #heat08:28
openstackgerritSteven Hardy proposed a change to openstack/heat: migrate User/AccessKey resources to StackUser base class  https://review.openstack.org/7276308:29
openstackgerritSteven Hardy proposed a change to openstack/heat: StackUser add _delete_keypair function  https://review.openstack.org/7276208:29
openstackgerritSteven Hardy proposed a change to openstack/heat: engine: allow stack_user_project users to retrieve stack  https://review.openstack.org/7130008:29
openstackgerritSteven Hardy proposed a change to openstack/heat: Add test for StackUser._create_keypair  https://review.openstack.org/7276108:29
openstackgerritSteven Hardy proposed a change to openstack/heat: Add config options to specify stack domain admin  https://review.openstack.org/7603508:29
openstackgerritSteven Hardy proposed a change to openstack/heat: StackUser add suspend/resume support  https://review.openstack.org/7193008:29
openstackgerritSteven Hardy proposed a change to openstack/heat: migrate StackUser base class to stack domain users  https://review.openstack.org/7121008:29
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient add delete_stack_domain_user_keypair  https://review.openstack.org/7192908:29
openstackgerritSteven Hardy proposed a change to openstack/heat: Modify stack_user_domain config option to take domain ID  https://review.openstack.org/7397808:29
*** Guest400 has joined #heat08:46
*** yogesh has quit IRC09:01
*** jprovazn has joined #heat09:03
*** xuhanp has joined #heat09:03
*** xuhanp has left #heat09:05
*** derekh has joined #heat09:12
*** nati_ueno has joined #heat09:15
*** sballe_ has joined #heat09:19
*** mkollaro has joined #heat09:21
*** giulivo has joined #heat09:21
*** sballe has quit IRC09:21
*** andersonvom has joined #heat09:23
*** stannie has joined #heat09:24
*** sballe has joined #heat09:26
*** jprovazn has quit IRC09:27
*** sballe_ has quit IRC09:27
*** jprovazn has joined #heat09:28
*** jamieh has joined #heat09:32
*** andersonvom has quit IRC09:34
*** tomek_adamczewsk has joined #heat09:35
*** sballe_ has joined #heat09:40
*** sballe has quit IRC09:40
*** jprovazn has quit IRC09:45
*** sballe_ has quit IRC09:48
*** saju_m has quit IRC09:49
*** saju_m has joined #heat09:50
sdake__for folks on fedora + devstack, take note of https://bugzilla.redhat.com/show_bug.cgi?id=102710409:51
shardysdake__: You're up late!09:55
shardysdake__: and thanks :)09:55
sdake__i slept for about 6 hours09:55
sdake__up early unfortunately09:55
openstackgerritA change was merged to openstack/python-heatclient: "version" section should be required in template  https://review.openstack.org/6980810:07
openstackgerritA change was merged to openstack/heat: Check that 'heat_template_version' is a HOT version  https://review.openstack.org/7302110:07
*** liang has quit IRC10:08
*** bada has quit IRC10:08
*** bada has joined #heat10:09
*** jistr has quit IRC10:12
*** bvandenh_ has quit IRC10:22
*** jufeng has joined #heat10:26
openstackgerritA change was merged to openstack/heat: Fix typo and remove unused code in nova_utils.py  https://review.openstack.org/7674010:30
openstackgerritRabi Mishra proposed a change to openstack/heat: Allows vip from a different subnet for lbaas resource  https://review.openstack.org/7708410:32
*** e0ne_ has joined #heat10:33
*** jistr has joined #heat10:34
*** guohliu has quit IRC10:36
*** e0ne has quit IRC10:38
*** ramishra has joined #heat10:48
*** ramishra_ has joined #heat10:49
*** ramishra has quit IRC10:49
*** nkhare has quit IRC10:57
*** fandi has quit IRC11:02
sdake__shardy do you think the ordering of :11:02
sdake__    cfg.StrOpt('deferred_auth_method',11:02
sdake__               choices=['password', 'trusts'],11:02
sdake__               default='password',11:02
sdake__               help=_('Select deferred auth method, '11:02
sdake__choices vs default would make any difference?11:02
*** jamieh has quit IRC11:03
shardysdake__: choices allows oslo to validate the value, it doesn't set the default11:04
sdake__i get that11:04
shardyOh, can you rephrase your query then pls? :)11:04
sdake__sec let me try something11:04
*** lsmola has quit IRC11:05
*** jamieh has joined #heat11:06
sdake__so basically I changed the order of those two parameters11:06
sdake__the original code generated a backtrace11:06
shardyit did?11:07
sdake__changing the order fixed the problem11:07
shardyweird, what version of oslo.cfg do you have?11:07
sdake__1.2.111:07
sdake__I changed the order back to try to reproduce the original11:07
sdake__unfortunately my test cases are locking up11:08
sdake__master did work properly when I ran it after i re-devstacked11:09
sdake__maybe it was some pyc thing11:09
sdake__I don't see how changing the parameters would have any effect11:10
shardyme neither, like you say stale pyc maybe11:11
*** ramishra_ has quit IRC11:11
sdake__best guess11:11
sdake__when i saved the file it would trigger a pyc regeneration11:11
sdake__shardy there was a keystone v2 client review11:16
sdake__when you get a chance would you mind taking a look11:16
sdake__it looks pretty solid to me11:16
*** aignatov_ is now known as aignatov11:16
sdake__but it doesn't implement trusts and I'm not sure what the implications of that are11:16
shardysdake__: sure, IIRC I already reviewed it once11:17
sdake__oh must have missed that11:17
shardybeen meaning to get back to it :)11:17
shardysooo many reviews..11:17
sdake__ya our review queue is packed11:17
sdake__I spent 3 hours today going through reviews11:17
shardyThey can't implement trusts because the OS-TRUST extension is v3 only11:17
sdake__yes i know, I'm just wondering what that breaks from a heat perspective11:18
shardythe implications are that you have to use deferred_auth_method=password, which is the default right now, but I'd like to change that soon11:18
shardythe main advantage of moving to trusts (other than not storing passwords) is that users won't have to pass a password e.g in the horizon stack-create interface11:19
shardyalso it gives you much more granular control of what is delegated to the heat service user11:19
sdake__I thought the main advantage was to be able to create users and wcs without admin ?11:19
shardysdake__: No, that's what the instance-users BP implements11:19
shardybut that's not using trusts11:19
sdake__so rax will get user based wcs and users with v2?11:20
shardytrusts turned out to be a poor solution to the in-instance users problem11:20
shardybut it's a really good solution to the deferred auth requirement11:20
shardysdake__: Yes, it should just work the same as what we have in Havana11:21
sdake__ok, I guess I dont have time at the moment to discuss, maybe we can speak after the deadline about it in more detail11:21
shardysdake__: I'm planning a couple of blog posts explaining it all in more detail11:21
sdake__I would just like rax to have a clear picture of what they are getting/not getting with v211:21
sdake__cool11:21
sdake__I think your the only cat that understands it in detail :)11:22
sdake__imo makes sense for them to document it in their readme11:22
shardysdake__: I've discussed it with andersonvom and I believe they understand the problems with sticking with v2, but because they're using $not_keystone they don't have any choice atm11:22
sdake__but I was at a loss as to what they should actually put in it11:22
sdake__understood11:22
shardyapparently the issue of needing admin is less of an issue with their auth solution, but I'm not aware of why11:23
sdake__I think when we spoke with kebray about it a month or so ago we weren't quite sure what they would be giving up11:23
sdake__at the time I think we thought it was wc, autoscaling and users11:24
*** ramishra has joined #heat11:24
shardysdake__: tbh when I started moving everything to v3 I did not realize it would be an issue for rax, otherwise I would have reached out to them and discussed it11:24
sdake__it all worked out fine11:24
sdake__rax had plenty of time to work out a solution11:24
shardyThe v2 shim should be fine, if they are happy with the Havana auth stuff11:24
sdake__its not like we told em a week before deadline "this isn't going to happen" :)11:25
sdake__or scramble and make a shim11:25
*** nosnos_ has quit IRC11:27
*** ramishra has quit IRC11:28
sdake__shardy any quick primer on what mock_enforce_setup does11:31
shardysdake__: it stubs the stuff called by the @util.policy_enforce decorator in the API controllers, so you can control the policy enforcement in the test11:34
shardysdake__: basically you just pass the controller method, which should match what's in the policy.json, and the return value for the policy enforcement11:35
*** jufeng has quit IRC11:35
sdake__it takes a mock_enforce parameter11:35
sdake__I have changed mock.patch.object to act as a context11:36
shardysdake__: That's provided via the class-level mock.patch11:36
sdake__(eg with mock.patch.object(..)11:36
shardy@mock.patch.object(policy.Enforcer, 'enforce')11:36
shardyEach test gets the function level mocks then the class level mocks in the argument list11:37
sdake__https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L33411:38
shardyYup, and mock_enforce comes from:11:39
shardyhttps://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L31611:39
shardyjust saves decorating every function11:39
sdake__in order to get this test case to work, I have to do something like this:11:39
sdake__    def test_index(self, mock_enforce):11:39
sdake__        with mock.patch.object(self.controller.rpc_client._client,11:39
sdake__                               'call') as mock_call:11:39
sdake__            self._mock_enforce_setup(mock_enforce, 'index', True)11:39
sdake__11:39
sdake__sorry for spam11:39
sdake__hopefully its late a nd nbody is around :)11:40
*** ramishra has joined #heat11:40
*** tomek_adamczewsk has quit IRC11:40
sdake__I guess I dont totally get how that gets connected in with the context manager11:40
*** tomek_adamczewsk has joined #heat11:41
shardycan't you use the mock.patch.object decorator, like in the existing tests?11:41
sdake__the reason I have to use a context is ._client is created on init of the object11:41
sdake__it is not a function11:42
shardysdake__: Sorry, I'm not quite clear what the question is tbh11:43
sdake__the existing mock.patch.object only works on functions not constructed objects11:43
sdake__how do I get enforce_setup to work with a context manager11:43
sdake__or should it just work as is?11:44
sdake__shardy give me a moment i'll post an updated patch11:47
openstackgerritSteven Dake proposed a change to openstack/heat: Use olso.messaging  https://review.openstack.org/7279811:51
sdake__shardy https://review.openstack.org/#/c/72798/28/heat/tests/test_api_openstack_v1.py11:52
sdake__line 33111:52
sdake__so as you can see, I use a context manager to patch call11:53
sdake__I cannot use the decorator, because the decorator doesn't work with stuff created in __init__ of the rpc_client object11:53
*** rpothier has quit IRC11:54
sdake__.client is not a function but a constructed object11:54
*** rpothier has joined #heat11:55
sdake__https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L316 mocks polciy.Enforcer, I assume the enforce call11:57
sdake__and it passed that to the second argument of every stackcontrollertest object?11:57
sdake__rather function11:57
ramishrahello eveyone11:58
sdake__morning ramishra11:58
ramishrasdake: good morning11:58
ramishrasdake: I have a question, what is our approach composite resource properties11:59
ramishrasdake: I don't see any validation done for nested peoperties.. it just accepts anything..12:01
sdake__ramishra I'd like to get into it, but we have 3 days including the weeeknd to get our work done for the i3 blueprints12:01
sdake__and I get the feeling I'm going to need every moment of it12:01
sdake__can you ping on the 4th :)12:01
ramishrasdake: np12:01
ramishrasdake: sometime later12:01
*** jufeng has joined #heat12:02
sdake__plate is empty after the 4th, happy to talk then12:02
sdake__shardy should that test case at line 333 enforce the policy without further changes?12:03
shardyramishra: Have you seen https://bugs.launchpad.net/heat/+bug/1265862 ?12:03
shardysdake__: AFAICS yes, but I'm not a mock expert :)12:04
shardydoes it work?12:04
sdake__shardy cool - it doesn't work but i wanted to possibly eliminate that part of the problem as I don't understand how it works12:05
sdake__it actually locks up12:05
sdake__shardy so just to be clear, mock_enforce (the 2nd parameter on line 331) comes from https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L31612:06
shardysdake__: yup, exactly, it's just shorthand for decorating every test function12:06
sdake__cool12:07
sdake__shardy thanks i'll dig around in the test case more, I have a clearer picture of how it works12:07
sdake__I actually ahve alot of these enforce test cases failing12:07
sdake__shardy thanks again - hopefully I wont bug you for awhile again :)12:08
ramishrashardy: Hello. this is similar bug around validation of constraints.. but I saw this issue with nested properties.. I was working on the neutron/loadbalancer.py, and the embeded 'vip' from within 'pool' is not validated at all. It just accepts anything..12:08
ramishrashardy: That's why I had suggested to rewrite it.. however, it seems most of the group is not for it in icehouse12:09
*** sballe has joined #heat12:09
shardysdake__: tbh I'd be tempted to just put a call function in the EngineClient then you don't have to mess with all the mocking as you can just stub the wrapper function12:11
*** jistr has quit IRC12:11
*** jistr has joined #heat12:12
sdake__shardy seems unoptimal but if i get jammed up I may do it as a short term thing12:13
sdake__thanks for the suggestion12:13
shardyramishra: if your issue is not resolved by https://review.openstack.org/#/c/69655/ then please raise a bug describing the issue so we can track working out a solution12:13
* shardy -> lunch12:14
sdake__shardy I fixed that test case - spacing error12:18
sdake__thanks for helping explain the enforce, now I've got it and understand how it works at a rudimentary level :)12:19
openstackgerritChangBo Guo(gcb) proposed a change to openstack/heat: Update Oslo wiki link in README  https://review.openstack.org/7715312:20
openstackgerritA change was merged to openstack/heat: Don't install cloud-init on Rackspace images  https://review.openstack.org/7699312:22
ramishrashardy: will do thanx12:23
*** aignatov is now known as aignatov_12:30
*** topol has joined #heat12:30
*** jamieh has quit IRC12:30
*** jufeng has quit IRC12:33
*** aignatov_ is now known as aignatov12:36
*** jamieh has joined #heat12:36
sdake__shardy any idea what mock_format is here: https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L85012:38
*** eglynn has joined #heat12:42
eglynnhi folks12:42
sdake__morning eglynn12:42
sdake__or evening in your case I think :)12:42
eglynnsdake__: afternoon-ish ;)12:42
eglynnsdake__: is this BP expected to make it for icehouse? https://blueprints.launchpad.net/heat/+spec/oslo-messaging12:43
sdake__absolutely12:43
eglynnsdake__: cool12:43
sdake__may be a 3 day coding binge but its going to happen12:43
*** dmakogon_ is now known as denis_makogon12:43
sdake__beating the test cases into submission now12:43
sdake__I have a working implementation12:43
sdake__have tested all api operations and multiple engine support12:44
*** ramishra has quit IRC12:44
sdake__eglynn https://review.openstack.org/72798 if you want to take a look12:45
eglynnsdake__: reason I ask is to gauge how much of an outlier ceilometer would be if our switch didn't make it for icehouse12:45
sdake__i think you want to try to make it if possible12:45
eglynnsdake__: ... debate is ongoing as to whether to beg an FFE for this12:45
*** jufeng has joined #heat12:47
*** faramir has joined #heat12:53
*** jminter has joined #heat12:53
*** alexpilotti has joined #heat12:54
*** lsmola has joined #heat12:55
*** tomek_adamczewsk has quit IRC12:56
jminterhello, could anyone help me with the recommended way to reference pre-existing openstack architectural components in a heat template?12:59
jminterexample: I want to create a new server, but in an existing network & subnet12:59
jminterobviously I can reference the network and subnet by ID within the heat template12:59
jminterbut if I send the heat template to someone else who has networks with the same friendly name, but different IDs, the template won't work12:59
jminteris there a recommended neat way that gets round this - e.g., either is there a heat resource which represents an existing network element which I can instantiate by name, not ID12:59
jminteror is there a commonly used script that preprocesses a heat template substituting friendly names for IDs?12:59
openstackgerritA change was merged to openstack/heat: Ensure that the NoCloud data source is loaded  https://review.openstack.org/7699413:04
openstackgerritA change was merged to openstack/python-heatclient: Expand get_file using to template scope  https://review.openstack.org/7622313:07
*** tomek_adamczewsk has joined #heat13:07
*** jufeng has quit IRC13:08
shardyjminter: hi13:08
jminterhi shardy13:08
shardyjminter: the short answer is atm you have to pass the ID as a parameter into the template13:08
shardyjminter: what we need to do is make all the resources capable of looking up the ID by name13:09
jmintermakes sense13:09
shardylike the external gateway in OS::Neutron::Router13:09
shardyjminter: feel free to raise bugs upstream in launchpad for each resource you'd like to see lookup by name, and we will look at adding that capability13:10
jminterok13:10
jminterin the meantime, is this the sort of thing environment files can help with too - prepopulating name/id mappings for commonly used items?13:10
shardyjminter: yes, exactly, you can have an environment file which specifies the ID's13:11
jmintere.g. public_net = UUID, private_net = UUID, private_subnet = UUID13:11
openstackgerritA change was merged to openstack/heat: Remove unused output section from test template  https://review.openstack.org/7581713:12
jmintershardy: bug per resource, or one bug listing the resources?13:12
shardyjminter: one bug is probably fine if that's easier13:13
jminterok, will do!13:13
jminterthanks steve13:13
shardyjminter: thanks! :)13:13
sdake__shardy sorry to interrupt, any idea where the mock_format parameter originates here: https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L85013:15
shardysdake__: from the format_stack patch the line above :)13:15
shardythe decorator just appends an argument to the list13:16
*** jdob has joined #heat13:18
*** wchrisj has joined #heat13:19
*** radez_g0n3 is now known as radez13:19
*** Guest15102 is now known as _jmp_13:21
*** faramir has quit IRC13:31
*** aweiteka has joined #heat13:32
*** rbuilta has joined #heat13:33
shardyjminter: Are you testing on Havana?13:35
jminteryes - rdo13:35
shardyjminter: digging into it a bit, we added support for looking up network by name for OS::Nova::Server in early icehouse:13:35
shardyhttps://bugs.launchpad.net/heat/+bug/124701913:35
shardyjminter: there are other places which expect a UUID, but that seems to be the main one you're missing?13:36
jminternot clear to me that that enables the floating IP use case13:36
shardyhttps://review.openstack.org/#/c/59796/10/heat/engine/resources/server.py13:37
*** tspatzier has joined #heat13:38
shardyjminter: Yep, you're right looks like floating_network_id only takes an id13:39
jminteratm I'm creating a OS::Neutron::FloatingIP and a OS::Neutron::Port as well as a OS::Nova::Server, which is pretty verbose for a pretty standard use case!13:39
shardyjminter: I'll update the bug and hopefully we can work up a todolist of those resources which need updating13:39
jminter(although I guess that can be fixed with a custom template resource13:39
jminterso as you say, floating_network_id in OS::Neutron::FloatingIP, but also network_id and subnet_id in OS::Neutron::Port13:40
shardyyeah abstracting it into a provider template is probably the way to go13:40
jminterseems like a pain to have to go around all these resources working out which params need to change13:41
jminterwouldn't a model whereby heat "knows" that Neutron::FloatingIP floating_network_id and Neutron::Port network_id both refer to a Neutron::Net object, and having a method on Neutron::Net which resolves an incoming name or uuid be better?13:42
shardyjminter: possibly, but in this case there won't be a Neutron::Net resource, because the user is providing the network by name13:43
shardyjminter: probably we need to update the neutron base-class with a common lookup helper or something13:43
jminterwell, there wouldn't be a Neutron::Net resource in the template and created by heat, but it might be a useful feature that heat could build a proxy Neutron::Net resource for pre-existing networks even if they weren't created this time aroudn13:45
*** ramishra has joined #heat13:45
jminterthen you could have a proxy object that you could get parameters from to pass back out of your template13:45
jminterregardless of whether you created it or not13:45
shardyjminter: the problem with that is if we persist the resource, we'd need to keep track of those we actually own, so we know what to do on update/delete13:46
shardywe could create a transient resource just to help do the lookup but that's kinda backwards from how we do everything atm :)13:46
shardyjminter: if you actually wanted the network to be picked up and managed by heat, we're working on a feature called stack-adopt, which would make that possible13:47
*** tspatzier has quit IRC13:47
jminternot quite the same thing, but it sounds good13:48
*** arbylee has joined #heat13:49
*** ramishra has quit IRC13:49
*** andersonvom has joined #heat13:51
*** arbylee has quit IRC13:52
*** blomquisg has joined #heat13:54
radixSpamapS: I just posted a reply to the AutoScalingGroup review14:00
radixSpamapS: just wanted to make sure you see it: you can scale Stacks just fine14:00
*** cody-somerville has quit IRC14:00
*** rbuilta has quit IRC14:01
*** rbuilta has joined #heat14:04
*** jmckind has joined #heat14:05
*** dave79 has joined #heat14:14
dave79Hello14:14
*** cadenzajon has joined #heat14:18
*** jamieh has quit IRC14:20
*** saju_m has quit IRC14:21
*** jamieh has joined #heat14:22
*** sgordon has joined #heat14:23
*** sgordon has quit IRC14:23
*** sgordon has joined #heat14:23
sdake__shardy so teh format_stack, how does that actually get converted into mock_format?  Should I make the funtion defintion14:25
shardysdake__: each decorator just passes a value to the argument, you could name the arg whatever you want AFAIK14:26
sdake__   @mock.patch.object(stacks.stacks_view, 'format_stack')14:26
sdake__    def test_preview_stack(self, mock_format, mock_enforce):14:26
sdake__        with mock.patch.object(self.controller.rpc_client._client,14:26
sdake__                               'call') as mock_call:14:26
sdake__shardy thianks tha tis helpful14:26
sdake__does it pass it in the order the decorators are delcared?14:27
shardysdake__: Yes, well kinda reversed, the decorator nearest the function is the first arg14:29
sdake__sweet that helps alot14:29
sdake__shardy you are teaching me all kinds of stuff about test cases14:29
shardyand the class decorator is the same as putting a function decorator at the top of the other decorators for every function14:29
shardysdake__: I can only just use mock myself tbh14:30
sdake__so in this particular case, the ordering of the parameters will come out correclty14:30
sdake__so the class decorator gets executed last then?14:31
*** Linz has joined #heat14:32
sdake__shardy I really appreciate the hand holding :)14:33
shardysdake__: yup and np14:33
*** arbylee has joined #heat14:34
*** jasond has joined #heat14:35
*** rbuilta1 has joined #heat14:38
sdake__I keep fixing test cases and the failed test case doesn't go down14:39
sdake__yet the failed test is actually fixed14:39
*** alexheneveld has joined #heat14:40
sdake__test case count doesn't go down14:40
sdake__is that normal shardy?14:40
*** rbuilta has quit IRC14:41
*** jcru has joined #heat14:42
*** jcru has quit IRC14:42
*** jcru has joined #heat14:43
jasondsdake__: it helps if you run the test individually while trying to fix it.  eg. python -m testtools.run heat.tests.test_server.ServersTest.test_server_status_resume_no_resource_id14:44
sdake__jasond thanks14:45
jasondnp14:45
*** jamieh has quit IRC14:45
*** ramishra has joined #heat14:46
*** pablosan has joined #heat14:46
*** arbylee has quit IRC14:46
*** jcru_ has joined #heat14:46
*** arbylee has joined #heat14:46
sdake__jasond ya good suggestion14:49
*** jcru has quit IRC14:50
jasondpdb even works when you run it with testtools14:50
*** ramishra has quit IRC14:50
jasondsdake__: can you re-approve this? https://review.openstack.org/#/c/69111/14:51
jasondwasn't sure what the process is for re-approving rebases14:51
jasonddoes it still need 2 +2s?14:52
*** jamieh has joined #heat14:52
SpamapSradix: ah right, thanks. :)14:52
sdake__jasond needs a +2 and a second +2/+a14:52
sdake__jasond reviewing now14:52
SpamapSradix: it was the end of a long string of reviews :)14:52
SpamapSradix: which is why I kept it at +014:52
jasondsdake__: thanks14:52
radixhehe14:52
radixalrighty14:52
*** scroiset1 is now known as scroiset14:53
radixwhat's mike spreitzer's nick again, is it mspreitzer?14:53
*** jcru_ has quit IRC14:55
openstackgerritRichard Lee proposed a change to openstack/heat: Restructure contrib/ directories  https://review.openstack.org/7341714:55
openstackgerritRichard Lee proposed a change to openstack/heat: Generate docs for contrib plugins  https://review.openstack.org/6875114:55
*** alexheneveld has quit IRC14:55
shardySpamapS: Hi!  You have a minute to talk tripleo and trusts?14:56
*** pablosan is now known as ZZpablosan14:56
SpamapSshardy: I have very few minutes just now.. children are about to awaken. But we can start.. I may go afk abrubtly at any time. :)14:57
shardySpamapS: np ;)14:57
shardyhttps://bugs.launchpad.net/heat/+bug/128615714:57
SpamapSIt is raining heavily though, so that usually keeps them in bed a bit longer.14:57
sdake__jasond enjoy14:57
shardySpamapS: basically I'd like to flip deferred_auth_method to trusts by default, primarily so we can kill the password box in the heat horizon ui (along with it being much more secure)14:58
SpamapSshardy: wonderful14:58
jasondsdake__: thanks14:58
shardystevebaker pointed out that it may impact tripleo so I wanted to work out what's needed to minimise any breakage :)14:58
SpamapSshardy: sounds like we'd just need to assert heat_stack_owner exists at the same time or before that happens.14:58
shardySpamapS: yup, and all users creating heat stacks will need that role14:59
SpamapSshardy: how do you handle the situation where stacks exist and are being managed without a trust?14:59
shardySpamapS: atm we don't, my expectation is that you choose one or the other then stick with it15:00
SpamapSupgraders from havana might be.. upset at you for that.. ;)15:00
shardyIf that's impossible, then I'll have to look at options to support passwords for backwards compatibility15:00
shardyugh, yeah of course you are right15:00
SpamapSand for instance, our undercloud has stacks that have been alive for months, and need to stay alive ideally forever...15:00
SpamapSshardy: should be a small matter of requesting a trust using the auth details you already have right?15:01
shardySpamapS: I actually think it should mostly work, since heat_keystoneclient will just use whatever we find in user_creds15:01
SpamapSso you can go userpass->trusts, but not the other way15:01
shardySpamapS: I think probably just use user/pass until the stack is deleted15:01
SpamapSshardy: could you even just make a migration tool?15:01
SpamapSoh if it still uses user/pass forever that is fine15:01
*** zns has joined #heat15:03
shardySpamapS: Ok, cool, well sounds like if I do some upgrade testing tripleo should be good with the change right?15:04
*** zns has quit IRC15:04
SpamapSshardy: right. :)15:05
*** zns has joined #heat15:05
*** vijendar has joined #heat15:07
*** daneyon has joined #heat15:07
*** dave79 has quit IRC15:08
*** spzala has joined #heat15:08
*** Guest400 has quit IRC15:10
*** dims_ has quit IRC15:11
*** daneyon has quit IRC15:15
*** daneyon has joined #heat15:16
*** sabeen has joined #heat15:18
jminterhello again, am I right in thinking that in havana, the only way to synchronise elements of a heat template implies: use waitconditions, have admin user, don't use HOT templates as they don't have dependson?15:23
*** lnxnut has joined #heat15:24
*** dims_ has joined #heat15:24
*** mestery has quit IRC15:25
*** mestery has joined #heat15:26
shardyjminter: basically yes15:28
jminterthanks again shardy.  I can see ways in which this improves in icehouse for myself, I just wanted to check I wasn't missing something currently15:29
shardyjminter: although I thought you could use depends_on, will have to check15:29
jmintersaw a bug to implement depends_on in icehouse15:29
shardyjminter: the main reason not to use HOT is that we've not declared a first stable version yet15:29
jminterpity, it's great!15:29
shardyjminter: we're planning to for Icehouse, and to be fair plent of folks are already using it15:30
shardywe just didn't want to promise to maintain stability before it was reasonably complete15:30
shardyjminter: if stability is paramount I'd say use yaml rendered CFN on Havana, or if minor tweaks on upgrade are OK, feel free to use HOT :)15:31
shardy(HeatTemplateFormatVersion == yaml rendered CFN json)15:32
jminterhttps://bugs.launchpad.net/heat/+bug/1271008 for hot having no depends_on yet15:32
jmintershardy: got it for all of the above, thanks again15:33
*** chandan_kumar has joined #heat15:35
shardyjminter: np, checkout the hot-software-config and instance-users BP's for details on expected Icehouse improvements :)15:35
*** david_lyle_ has joined #heat15:37
*** jcru has joined #heat15:42
*** lindsayk has joined #heat15:45
*** Linz has quit IRC15:46
*** ramishra has joined #heat15:47
*** cfriesen_ has joined #heat15:50
*** ramishra has quit IRC15:51
*** topol has quit IRC15:54
andersonvomzaneb: you around?15:56
*** DandyPandy has quit IRC15:57
*** tomek_adamczewsk has quit IRC15:59
*** ZZpablosan is now known as pablosan16:02
zanebandersonvom: yes16:04
andersonvomzaneb: morning!16:04
*** skraynev is now known as skraynev_afk16:05
andersonvomzaneb: we had to rebase a patch you approved yesterday (https://review.openstack.org/#/c/68751/). do you have a min to take a look at it?16:05
zanebok, you should be good to go16:07
*** funzo has quit IRC16:12
*** funzo has joined #heat16:13
sdake__zaneb question re mock16:14
sdake__is there any way to do the following:16:14
sdake__https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L38116:14
zanebfor the record, I have never written a single mock test in my life :)16:14
sdake__shardy?16:14
sdake__I dont want to use mock.patch.object16:15
sdake__I want to use StubOutWithMock16:15
zanebah, that I know about16:15
sdake__https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L33316:16
shardysdake__: You're going to convert test_api_openstack_v1.py to mox?16:16
zanebso yeah, you do StubOutWithMock, then you make the (mock) call that you're expecting later, then you put everything into replay mode16:17
sdake__StubOutWithMock is mox right?16:17
zanebyes16:17
shardysdake__: yeah16:17
sdake__i didn't see https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L381 in the documentation16:17
zanebso I'd have to wonder why, since we're supposed to be moving away from mox towards mock16:17
sdake__what the test does is make sure the function is only executed once by the client16:17
sdake__well the main reason is I want to mock out a function in a constructed object16:18
sdake__and I dont want to use a context manager to do it16:18
shardyhttp://docs.python.org/dev/library/unittest.mock16:18
sdake__is there a stuboutwithmock thing that works for mock?16:19
sdake__reading16:19
sdake__i read the mox documentation i think i understand it at a high level16:19
zanebsdake__: so the object is only constructed in the test case16:19
sdake__zaneb precisely16:20
shardysdake__: I would prefer you didn't convert that test from mock to mox if possible16:20
SpamapSsdake__: StubOutWithMock is just a handy monkey patcher.16:20
zaneband you don't want to use a context manager because it is an intrusive change to the existing code?16:20
SpamapSsdake__: so, fixtures.MonkeyPatch may be all you need.16:20
sdake__i dont like the context manager approach because it makes the test look evil16:21
zanebSpamapS: still need cleanup though presumably? and that probably means a context manager16:22
sdake__the entire test has to be spaced over 4 lines16:22
zanebsdake__: paste? that sounds like a really bad reason on the face of it ;)16:23
sdake__http://paste.fedoraproject.org/81280/3604562116:23
sdake__line 33216:23
andersonvomzaneb: thanks!16:24
sdake__I guess I need mock to deal with the fact that we want to verify the server only gets called once16:24
sdake__I just want a StubOutwithMock function instead of a context manager :)16:24
* zaneb doesn't see a problem with that code16:24
sdake__is there such a beast16:24
zanebsdake__: note that you only need the context manager to cover the part where it will actually be called16:25
sdake__zaneb I guess if you will accept it I wont change it around16:25
sdake__mock_call is used all the way to the end of the function16:25
zanebi.e. you could move line 332 down to line 36016:25
zanebor 358, rather16:26
zanebthe expected = {...} part could go outside too16:26
sdake__I dont really want to refactor the tests more then necessary16:27
sdake__makes it harder to review16:28
*** randallburt has joined #heat16:28
sdake__regarding moving line 332 down that isn't possible, self._get('/stacks') calls the wsgi stack controller which triggers an rpc call16:28
*** chandan_kumar has quit IRC16:29
sdake__the test verifies that the stack controller only calls into rpc once16:29
sdake__the rpc call is which needs to be mocked16:29
*** tomek_adamczewsk has joined #heat16:29
sdake__it also makes sure the wsgi calls the right method in the rpc system16:30
zaneboh, ok16:30
sdake__I guess what I'm looking for is a mock analog to StubOutWithMock :)16:30
zanebbut still, you could move a lot of the setup up outside the context manager if you wanted16:31
sdake__zaneb agree, although I dont want to significantly reorganize the test cases16:31
sdake__i'd like people not to have to think "ok this logic got moved aroudn now I have to figure out if its still correct"16:31
zanebsdake__: that exists and it looks like this http://paste.openstack.org/show/70679/16:33
zanebI don't see how it could be considered better than a context manager though16:33
sdake__zaneb your right that is not better16:34
sdake__ok well I guess I'll stop working on making is beutiful and focus on making it work :)16:35
zanebthe difference here is that mox retains a global list of things that it has patched16:35
sdake__i see and mock does not16:35
zanebso you can clean up in TearDown() without knowing anything about the particular test16:35
zanebthat's right, mock does not so you have to clean up after each one16:36
sdake__so mock is probably better16:36
zanebyes. and a context manager is the easiest way to make sure that happens16:36
sdake__ok i'm convinced16:36
zanebalternative is to set it up in setUp() and assign mock_call to an instance variable16:37
zanebthen clean it up in tearDown()16:37
sdake__context manager seems better then that16:37
zanebthat will work assuming all your tests in one test case are mocking the same thing16:37
zanebwhich should probably be the case here16:38
sdake__yes it is the case16:38
sdake__(I think)16:38
sdake__the test cases also mock policy.Enforcer16:39
sdake__but that is done trough:16:39
sdake__@mock.patch.object(policy.Enforcer, 'enforce')16:39
sdake__class StackControllerTest(ControllerTest, HeatTestCase):16:39
zanebhttp://paste.openstack.org/show/70682/16:40
sdake__zaneb thanks for explaining the advantages of the conteext manager16:40
*** jistr has quit IRC16:40
sdake__ya that is pretty good but I think I would trust a context manager more then setup and teardown16:41
sdake__but I don't know the specifics of either implementation16:41
*** lsmola has quit IRC16:41
*** pasquier-s has quit IRC16:41
sdake__atlesat you have given me some ammo to use in the reviews :)16:41
sdake__ta :)16:41
zanebiirc tearDown always gets called (regardless of test result) unless setUp raised an exception16:41
sdake__is it official that all openstack projects should migrate to mock?16:42
shardysdake__: all the threads I've read said new tests should be mock and existing tests can be migrated when convenient16:44
shardywe've not really enforced the former though16:44
sdake__we should so we have one system to maintain :)16:44
shardyHa, by the time all the tests get migrated I expect there will be something else folks want to use ;)16:45
zaneblol16:45
zanebI know I wouldn't be pumped about having to rewrite some of the hairier unit tests16:45
zanebthey were bad enough the first time16:46
shardyzaneb: agree, it's hard enough just keeping them working ;)16:46
sdake__when I get some free time i'll convert the rpc and api tests to mock - seems easy enough16:46
radixshardy: and there, you've found the problem with mocks16:46
shardyI want to change one config option default and it breaks 600 tests16:46
radixwell, one of them :)16:46
*** alexheneveld has joined #heat16:46
zanebshardy: yah, and I suspect that mock isn't going to make that any easier16:46
*** ramishra has joined #heat16:47
zanebwe do rely far too heavily on StubOutWithMock though16:47
zanebit's meant to be a last resort16:47
zanebwe use it for everything16:47
radixit's kind of a disease on the dynamic languages community lately16:48
shardyzaneb: how should we be doing it, setting up fixtures?16:48
*** rbuilta1 has quit IRC16:49
zanebshardy: in theory you're supposed to make a whole mock object, and pass it in using dependency injection16:49
shardykeystone has a much more fixture orientated approach and it's sometimes really hard to figure out where stuff is getting set16:49
zanebbut the reality is that we're not really writing unit tests here, more integration tests16:49
radixyeah, being able to reuse testing objects is nice.16:49
radixinstead of having imperative code creating ad-hoc mocks everywhere16:49
zaneband Java-style dependency injection where every method takes 500 parameters is not nice either16:49
sdake__is there a way to print in mock16:50
sdake__or in testtools16:50
sdake__or whatever the hell this thing uses16:50
radixsdake__: side effects, probably?16:50
*** david_lyle_ is now known as david-lyle16:50
zanebradix: I think you're talking about fakes, but I mean just calling mox.Mock(SomeClass)16:50
sdake__I just want to print out some debugging16:50
radixzaneb: oh.16:50
radixzaneb: ok. :)16:50
*** alexheneveld has quit IRC16:50
radixsdake__: you can assign a side effect to a mock call so that it runs a function16:51
zanebsdake__: I usually use a logger, those work16:51
radixlike, .side_effect = lambda: print("hello")16:51
shardysdake__: I still use nosetests -sx for exactly that reason16:51
shardyall the other test running solutions seem to eat stdout16:52
zanebyeah, it would be really nice if testr captured stdout/stderr the same way it does with the logs16:52
*** rbuilta has joined #heat16:52
zanebI don't know of any technical reason why it couldn't16:52
shardyprinting actual output takes away all the magic16:53
shardy;)16:53
*** rbuilta has quit IRC16:54
*** aignatov is now known as aignatov_16:55
sdake__shardy how do you select a specific test with nosetests16:55
shardynosetest -sx mytest.py16:55
sdake__that runs every tes tin the file16:55
sdake__I want to run only one test16:55
therve`pacakge.module:class.method16:57
zanebisn't it nosetests heat.tests.test_something.TestCase:test_name16:58
therve`I think16:58
sdake__i tried that16:58
sdake__I guess I'll try again16:58
zanebtherve` could be right, can't remember where the colon goes16:59
therve`Yeah me neither :)16:59
therve`I use trial, which takes fqdn16:59
zanebit's either before or after the class :)16:59
*** rbuilta has joined #heat16:59
*** rbuilta has quit IRC17:00
*** rbuilta has joined #heat17:00
sdake__before the class17:01
*** dtalton has joined #heat17:01
*** rcleere has joined #heat17:01
sdake__thanks therve`17:01
therve`No problem17:02
*** dtalton has left #heat17:03
*** tomek_adamczewsk has quit IRC17:04
*** cadenzajon has quit IRC17:06
*** e0ne_ has quit IRC17:08
*** e0ne has joined #heat17:09
sdake__thanks for the hand holding guys I think I'm up to speed on writing tests now :)17:09
SpamapSzaneb: so.. thoughts on automatically resuming using the status as I'm poking at it today..17:09
*** pablosan has quit IRC17:10
*** e0ne has quit IRC17:13
SpamapSzaneb: I'm finding it difficult to determine what resources, at the same level as the failed resource, were completed. It is entirely possible that one went to FAILED before some went to IN_PROGRESS. :-/17:14
zanebas soon as something fails everything in progress should be cancelled (which will also put it into the failed state)17:15
zanebhttps://github.com/openstack/heat/blob/master/heat/engine/scheduler.py#L34617:15
SpamapSzaneb: but there may be things, on the same level, that will not have been IN_PROGRESS.17:16
sdake__what does _ mean on the left side of an assignment?17:16
SpamapSzaneb: they'll show as UPDATE_COMPLETE if a previous update finished..17:16
*** eglynn has quit IRC17:17
SpamapSzaneb: but we'll skip them...17:17
SpamapSsdake__: "throw this away" .. but sometimes it means "oops I just screwed up gettext"17:17
zaneblol17:17
sdake__SpamapS the example is17:17
sdake__            rpc_call_args, _ = mock_call.call_args17:17
zanebsdake__: it's just a variable name17:17
SpamapSit's a convention which collides with use of gettext .. and thus is likely a bad idea.17:18
zanebpeople use it when they want to throw away the value17:18
sdake__got it17:18
zanebwhat SpamapS said17:18
radixsdake__: basically, that example is equivalent to: "self.assertEqual(len(mock_call.call_args), 2); rpc_call_args = mock_call.call_args[0]"17:18
*** saurabhs has joined #heat17:19
radixnot exactly, of course, a different error will be raised if the number of args is not 2, but that's probably the intention of that test code.17:19
zanebrpc_call_args, who_cares = mock_call.call_args; _ = who_cares; del who_cares17:19
radixo.O17:20
*** Linz has joined #heat17:20
zanebSpamapS: assuming for the moment that chains of dependencies are completely separate... each chain either completes successfully or has one resource that has failed (on its own or due to cancellation)17:21
*** Linz has quit IRC17:21
zanebSpamapS: so you want to ignore the successful ones *until* you find the failed one, and then update everything from there17:22
*** Linz has joined #heat17:22
zanebSpamapS: and this can be generalised to an arbitrary DAG, where chains join and split17:22
zanebin theory ;D17:23
*** randallburt has quit IRC17:26
*** arbylee has quit IRC17:28
SpamapSzaneb: so I think you're assuming that the status will go from IN_PROGRESS -> FAILED in the same order that they go from COMPLETED -> IN_PROGRESS .. the DAG is just controlling what we can do next, but the DAG will have chains that have never gone to IN_PROGRESS, and say they are COMPLETED17:29
SpamapSzaneb: one possibility is to mark EVERYTHING as having an update in progress first..which would make a failure explicitly fail everything not done yet..17:30
*** Tross has quit IRC17:31
zanebhmmm, I was thinking that anything without dependencies would be started at the same time17:31
zanebbut you make a good point, that isn't actually true17:31
SpamapSyeah I'm saying in the process of starting them, you may fail17:31
*** rbuilta has quit IRC17:34
*** jamieh has quit IRC17:37
*** sgordon has quit IRC17:39
*** jdob has quit IRC17:39
*** david-lyle has quit IRC17:39
*** openstack has joined #heat22:51
*** alexpilotti has joined #heat22:53
kfox1111_cool. :)22:54
*** lnxnut has joined #heat22:54
*** zns has joined #heat22:55
zanebkfox1111_: do you have a bug id? I'm gonna fix it right now23:00
*** asalkeld has joined #heat23:01
kfox1111_https://bugs.launchpad.net/heat-templates/+bug/128638023:03
kfox1111_thanks. :)23:03
*** openstackgerrit has quit IRC23:07
*** jmckind has quit IRC23:07
*** openstackgerrit has joined #heat23:07
sdakewhy do we depend on python-heatclient in heat?23:09
zanebsdake: because soon we will do multi-region stacks23:10
*** spzala has quit IRC23:11
*** arbylee has quit IRC23:13
*** zns has quit IRC23:18
*** jcru has quit IRC23:20
*** kfox1111_ is now known as kfox111123:21
*** daneyon has joined #heat23:21
openstackgerritSteven Dake proposed a change to openstack/heat: Use olso.messaging  https://review.openstack.org/7279823:22
sdakezaneb got it23:22
sdakemakes sense23:22
zanebw00t23:24
*** daneyon has quit IRC23:24
sdake__only 59 more failures to go23:25
sdake__down from 65 :)23:25
sdake__difference being now I know what I'm doing :)23:25
sdake__i feel so confident in my understanding i could actually help others write test cases :)23:25
sdake__learn how that is23:25
*** zns has joined #heat23:27
sdake__btw zaneb, don't upgrade f20 setuptools23:27
sdake__its broke23:27
sdake__I think devstack will do this automatically if your prereqs file times out23:28
sdake__yum downgrade fixes it23:28
zanebmy laptop is still on F19, so I should be OK I think23:29
sdake__i think they updated f19 too23:29
sdake__for no good reason23:29
zaneboh, good23:29
zaneb:/23:30
sdake__i bitched in bugzilla so hopefully they will fix it23:30
*** yogesh_ has quit IRC23:31
sdake__is kargs a special word in python23:31
zanebI installed openstack on F20 yesterday with packstack, and it seems quite broken with selinux on :(23:31
zanebsdake__: no23:31
sdake__try installing with foreman23:31
*** openstack has quit IRC23:36
*** openstack has joined #heat23:42
*** giulivo has quit IRC23:43
zanebhow does that make my life _easier_? ;)23:43
*** lnxnut has quit IRC23:43
sdake__you will get to see what our customers see :)23:43
zanebRDO/packstack instructions were actually great23:43
zanebexcept for it not working :-P23:43
*** lnxnut has joined #heat23:43
sdake__packstack is on its way out23:43
sdake__probably can just boot that image on master that dib kicks out23:43
sdake__only need to test dib works not rhos works23:43
*** zns has quit IRC23:43
sdake__but if you have problems fiel bugs or they wont get fixed23:43
sdake__packstack is focused around puppet - and all that is reused in foreman23:43
zanebyeah, planning on trying it again in VM23:43
*** erkules has quit IRC23:43
zanebfirst time through broke because of another SELinux issue with mariadb, which I already added info on in Bugzilla23:43
sdake__we arepulling the trigger on dib today so knowing if it works soon is good :)23:43
zanebso by the 2nd or third time it was a bit messy23:43
zanebsdake__: I already moved it to VERIFIED23:43
sdake__cool23:43
sdake__i have totally ignored all email today23:43
sdake__not even at my workstation on my laptop23:43
*** jasond has quit IRC23:43
sdake__normally bugzilla is first thing i read every day23:43
sdake__say why is  type bad23:43
sdake__type()23:43
sdake__the string compare?23:43
zanebit doesn't catch subclasses23:43
sdake__got it23:43
zaneband, worse, in this case, it doesn't catch unicode23:43
sdake__thanks23:43
zanebunicode and str are both subclasses of basestring23:43
*** dims_ has quit IRC23:44
sdake__and dict/list are subclass of collections?23:44
zanebtechnically, no23:46
zanebbut they will respond to isinstance as if they were23:47
zanebhttp://docs.python.org/2/library/collections.html#collections-abstract-base-classes23:47
zanebthanks to http://docs.python.org/2/library/abc.html#abc.ABCMeta.register23:48
sdake__zaneb you know python like I know C23:48
zanebI used to know C23:49
sdake__I am really pro with C23:49
zanebsometimes I wonder how much I would remember23:49
sdake__even if I didn't use it for 10 years, i'd remember everything23:49
zaneblol23:49
sdake__learning python doesn't like semicolons was especially challenging :)23:50
zanebthere wasn't much I couldn't tell you about C, say 3 years ago23:50
*** lindsayk has joined #heat23:50
zanebI don't know much about trigraphs23:50
zanebeverything else23:50
sdake__I am not a C++ wizard23:50
sdake__C++ is a terribly designed language imo :)23:50
zanebI thoroughly agree, and plan never to touch it again23:51
sdake__pyton is actually pretty cool but it has a steep learning curve23:51
zanebI don't think it's steep so much as long23:51
sdake__I struggle to do basic things23:51
zanebit's *very* easy to get started23:51
zanebcompared to other langs23:51
sdake__but moving around all kinds of stuff to do oslo messaging is easy - that architecture work23:52
sdake__thats23:52
*** lnxnut has quit IRC23:52
zanebbut there's a huge amount of stuff you can, and eventually have, to learn23:52
*** lindsayk has quit IRC23:52
*** lindsayk has joined #heat23:53
*** dims_ has joined #heat23:59

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