Friday, 2015-05-15

*** kitch_ has joined #heat00:03
*** kitch_ has quit IRC00:03
asalkeldbug 1455276 is getting hit often (for me) in the gate00:11
openstackbug 1455276 in heat "TemplateResourceUpdateFailedTest.test_update_on_failed_create is failing with ActionInProgress" [High,Triaged] https://launchpad.net/bugs/145527600:11
*** dims has quit IRC00:15
*** dims has joined #heat00:23
*** Drago has quit IRC00:27
openstackgerritMerged openstack/heat: Updated from global requirements  https://review.openstack.org/18191000:43
openstackgerritMerged openstack/heat: Replace self.properties.get on self.properties[]  https://review.openstack.org/17903000:49
*** alexheneveld has quit IRC00:58
*** harlowja has quit IRC00:58
*** pm90_ has quit IRC01:02
*** sdake has quit IRC01:03
*** sdake has joined #heat01:05
*** achanda has quit IRC01:07
openstackgerritMerged openstack/python-heatclient: Add OS_TEST_PATH to testr  https://review.openstack.org/18021601:19
*** erkules_ has joined #heat01:21
*** alanf-mc has quit IRC01:23
*** erkules has quit IRC01:23
*** Qiming has joined #heat01:30
*** Yanyanhu has joined #heat01:38
*** multi_io has quit IRC01:39
*** multi_io has joined #heat01:41
*** Qiming_ has joined #heat01:49
*** sdake_ has joined #heat01:51
*** Qiming has quit IRC01:53
*** zzxwill has joined #heat01:54
*** sdake has quit IRC01:55
*** hdd has quit IRC01:58
openstackgerritAngus Salkeld proposed openstack/heat: Update the stack state as late as possible  https://review.openstack.org/18337402:04
*** signed8bit has quit IRC02:10
openstackgerritRabi Mishra proposed openstack/heat: Fix property validation for TemplateResource during update  https://review.openstack.org/18276302:14
ramishramorning all02:14
Qiming_huangtianhua, about https://review.openstack.org/#/c/180096/, I still believe there needs some improvement to the messages and maybe also the test cases, though it has already been approved02:20
*** alexheneveld has joined #heat02:21
elynnmorning02:22
huangtianhuaQiming_: looking02:22
*** alexheneveld has quit IRC02:32
*** huangtianhua has quit IRC02:37
*** huangtianhua has joined #heat02:37
*** Tross has joined #heat02:43
openstackgerritEthan Lynn proposed openstack/heat: Add CIDR format validation  https://review.openstack.org/18220902:44
*** vijayagurug has quit IRC02:46
*** dims has quit IRC03:12
*** hdd has joined #heat03:18
*** achanda has joined #heat03:21
*** achanda has quit IRC03:27
*** EricGonczer_ has joined #heat03:29
*** sdake has joined #heat03:33
*** hdd has quit IRC03:33
*** EricGonczer_ has quit IRC03:34
*** alexpilotti has quit IRC03:35
*** sdake_ has quit IRC03:36
*** sdake_ has joined #heat03:38
*** sdake has quit IRC03:41
*** tobe4333 has joined #heat03:48
*** coolsvap|afk is now known as coolsvap04:12
*** jaime49 has joined #heat04:19
*** tobe4333 has quit IRC04:20
*** achanda has joined #heat04:22
*** Marga_ has quit IRC04:24
*** rakesh_hs has joined #heat04:27
*** nkhare has joined #heat04:31
*** huangtianhua has quit IRC04:34
*** huangtianhua has joined #heat04:35
*** sdake_ has quit IRC04:35
*** daneyon has joined #heat04:36
*** jaime49 has quit IRC04:40
*** pm90_ has joined #heat04:43
ramishraasalkeld: Hi04:43
*** pm90__ has joined #heat04:47
*** pm90_ has quit IRC04:47
*** noa-koffman has joined #heat04:50
openstackgerrithuangtianhua proposed openstack/heat: Improve cinder volume validation and tests related  https://review.openstack.org/18339204:55
*** Marga_ has joined #heat05:00
*** mwheckmann has joined #heat05:07
*** mwheckmann has quit IRC05:12
*** tspatzier has joined #heat05:21
*** tspatzier has quit IRC05:24
*** gokrokve has joined #heat05:31
*** rakesh_hs has quit IRC05:36
*** inc0 has joined #heat05:44
asalkeldramishra: hi05:47
*** sdake has joined #heat05:52
*** sdake has quit IRC05:53
*** sdake has joined #heat05:54
*** gokrokve has quit IRC05:56
*** gokrokve_ has joined #heat05:56
*** jtomasek has joined #heat05:57
inc0good morning05:57
inc0guys, do we want to schedule heated-beer for Vancouver?:)05:58
*** pm90__ has quit IRC05:58
*** sdake has quit IRC05:58
*** gokrokve_ has quit IRC06:01
*** nihilifer has joined #heat06:03
*** pal has joined #heat06:08
*** Marga_ has quit IRC06:12
ramishraasalkeld: would you be able to review https://review.openstack.org/#/c/182763/ when you've time?06:12
*** sdake has joined #heat06:16
asalkeldsure06:21
*** jprovazn has joined #heat06:21
*** daneyon has quit IRC06:21
ramishrathanks06:22
openstackgerritMerged openstack/heat: Improve cinder volume validation and tests related  https://review.openstack.org/18339206:23
*** achanda has quit IRC06:23
*** daneyon has joined #heat06:23
asalkeldramishra: lgtm06:24
ramishraasalkeld: thanks06:24
*** Murali has joined #heat06:29
*** vijayagurug has joined #heat06:38
*** Marga_ has joined #heat06:43
*** tspatzier has joined #heat06:43
openstackgerritSergey Kraynev proposed openstack/heat: Add functional test for neutron port  https://review.openstack.org/17818906:45
openstackgerritSergey Kraynev proposed openstack/heat: Add scenario test for AWS loadbalancer  https://review.openstack.org/15319206:45
*** Marga_ has quit IRC06:48
*** erkules_ is now known as erkules06:53
*** erkules has joined #heat06:53
*** ishant has joined #heat06:59
openstackgerrithuangtianhua proposed openstack/heat: Should raise error if hook is invalid  https://review.openstack.org/18340907:00
*** daneyon has quit IRC07:03
openstackgerrithuangtianhua proposed openstack/heat: Correct volume restore behavior  https://review.openstack.org/18215207:05
*** sdake_ has joined #heat07:05
*** sdake has quit IRC07:08
*** ifarkas has joined #heat07:13
openstackgerritMerged openstack/heat: Reference the parent stack, not parent resource in Stack  https://review.openstack.org/18321407:18
*** jcoufal has joined #heat07:27
*** chlong has quit IRC07:32
*** inc0 has quit IRC07:32
*** jistr has joined #heat07:33
*** asalkeld has quit IRC07:38
*** Marga_ has joined #heat07:44
*** tobe4333 has joined #heat07:44
*** Marga_ has quit IRC07:49
*** zzxwill has quit IRC07:50
openstackgerritMerged openstack/heat: Encrypt Heat template hidden parameters  https://review.openstack.org/9790007:51
*** tobe4333 has quit IRC08:07
*** derekh has joined #heat08:13
*** blues-man has joined #heat08:30
openstackgerrithuangtianhua proposed openstack/heat: Add and apply cinder backup constraint  https://review.openstack.org/18343308:30
*** smccully has quit IRC08:38
*** vijayagurug has quit IRC08:43
*** inc0 has joined #heat08:44
*** Marga_ has joined #heat08:44
*** Marga_ has quit IRC08:50
*** smccully has joined #heat08:52
*** Trozz has joined #heat08:53
*** BManojlovic has joined #heat08:54
*** higgins has quit IRC08:58
*** higgins has joined #heat09:10
*** vijayagurug has joined #heat09:14
*** bdossant has joined #heat09:30
*** inc0 has quit IRC09:35
*** xuhaiwei has quit IRC09:36
*** tochi has quit IRC09:37
*** shardy has joined #heat09:39
*** dims has joined #heat09:45
*** Tross has quit IRC09:45
*** Tross has joined #heat09:45
*** Marga_ has joined #heat09:45
*** Yanyanhu has quit IRC09:46
*** alexpilotti has joined #heat09:47
*** pas-ha has joined #heat09:47
pas-hamorning all09:48
*** Marga_ has quit IRC09:50
*** e0ne has joined #heat09:55
*** e0ne is now known as e0ne_09:56
*** e0ne_ is now known as e0ne10:01
*** Qiming_ has quit IRC10:02
*** jprovazn has quit IRC10:16
openstackgerritRabi Mishra proposed openstack/heat: Add validation for parameter_groups  https://review.openstack.org/18346610:18
openstackgerritMerged openstack/heat: Resource plug-in for keystone service  https://review.openstack.org/17858410:19
openstackgerritMerged openstack/heat: Resource plug-in for keystone endpoint  https://review.openstack.org/17858510:19
openstackgerritMerged openstack/heat: heat_keystone README update for service and endpoint  https://review.openstack.org/17858610:19
*** tobe4333 has joined #heat10:27
*** e0ne is now known as e0ne_10:31
*** e0ne_ is now known as e0ne10:33
*** Tross has quit IRC10:35
*** Tross has joined #heat10:35
*** ishant has quit IRC10:41
skraynevanybody know should we merge this https://review.openstack.org/#/c/179291/ ?10:44
*** Marga_ has joined #heat10:45
*** Marga__ has joined #heat10:47
*** Marga_ has quit IRC10:47
*** Marga__ has quit IRC10:48
*** Marga_ has joined #heat10:48
*** jprovazn has joined #heat10:50
*** Marga_ has quit IRC10:53
shardyskraynev: honestly I'm not sure - I don't understand why it's changing so much to apply a tag?!10:58
*** pal has quit IRC10:59
shardyI guess it's resolving a merge of the kilo milestone-proposed branch, but I don't really understand why that affects master, given that everything in kilo proposed was backported from master?11:00
openstackgerritThomas Herve proposed openstack/heat: Implement barbican-backed store mechanism  https://review.openstack.org/18095911:03
shardyskraynev: I've pinged ttx to ask about it in #openstack-dev11:03
*** Qiming has joined #heat11:04
*** Marga_ has joined #heat11:05
*** tobe4333 has quit IRC11:07
*** Qiming_ has joined #heat11:23
*** e0ne is now known as e0ne_11:26
*** Qiming has quit IRC11:27
*** blues-man has quit IRC11:31
*** pas-ha has quit IRC11:35
skraynevskraynev: I have no ideas too :) I just met it during review and was surprised, why it's still here.11:35
skraynevshardy: ^ . also thx for the request to ttx11:36
therveIt's a bit weird, it seems to source branch has conflicts11:39
*** alexpilotti has quit IRC11:42
therveAh, maybe got something11:43
thervehttp://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2015-05-05.log11:43
therve"gerrit displays them oddly. checkout that merge commit and diff it against the branch tip"11:43
*** prazumovsky has joined #heat11:45
*** e0ne_ is now known as e0ne11:48
*** saju_m has joined #heat11:50
shardytherve: it's still weird, I don't really know how we can review something that appears to change thousands of lines of stuff, but probably shouldn't do anything11:51
shardytherve: thanks for the link though, sheds some light on it at least :)11:51
*** ifarkas has quit IRC11:52
*** EricGonczer_ has joined #heat11:55
*** Marga_ has quit IRC11:56
*** ifarkas has joined #heat11:58
*** Marga_ has joined #heat12:07
*** blomquisg has joined #heat12:10
*** links has joined #heat12:13
*** Yanyanhu has joined #heat12:13
*** Yanyan has joined #heat12:17
*** Yanyanhu has quit IRC12:20
*** e0ne is now known as e0ne_12:22
*** coolsvap is now known as coolsvap|afk12:23
*** Yanyan has quit IRC12:26
*** e0ne_ is now known as e0ne12:26
*** Marga_ has quit IRC12:30
*** nkhare has quit IRC12:33
*** jtomasek has quit IRC12:38
*** EricGonczer_ has quit IRC12:44
*** derekh_ has joined #heat12:49
*** blues-man has joined #heat12:51
*** blues-man has quit IRC12:52
*** blues-man has joined #heat12:52
*** vijayagurug has left #heat12:52
*** derekh has quit IRC12:52
*** e0ne is now known as e0ne_12:54
*** rpothier has joined #heat12:54
*** blomquisg1 has joined #heat12:55
*** e0ne_ is now known as e0ne12:56
*** blomquisg has quit IRC12:56
*** links has quit IRC12:57
*** sdake has joined #heat12:59
*** sdake_ has quit IRC13:02
*** sdake_ has joined #heat13:03
*** pas-ha has joined #heat13:05
*** sdake_ has quit IRC13:07
*** sdake has quit IRC13:07
openstackgerritOleksii Chuprykov proposed openstack/heat-specs: Add batching to ResGroup  https://review.openstack.org/18350613:08
*** sdake has joined #heat13:11
openstackgerritOleksii Chuprykov proposed openstack/heat-specs: Add batching to ResGroup  https://review.openstack.org/18350613:13
*** dyasny has joined #heat13:22
*** sdake has quit IRC13:23
*** crose has joined #heat13:29
*** sdake has joined #heat13:34
*** pas-ha has quit IRC13:34
*** pas-ha has joined #heat13:35
openstackgerritOleksii Chuprykov proposed openstack/heat-specs: Add batching to ResGroup  https://review.openstack.org/18350613:41
*** pas-ha has quit IRC13:41
*** pshchelo has joined #heat13:42
*** sdake has quit IRC13:42
*** pshchelo has left #heat13:45
*** BManojlovic has quit IRC13:51
*** blomquisg1 has quit IRC13:51
*** pas-ha has joined #heat13:52
*** jasond` has joined #heat13:54
*** pas-ha has quit IRC13:55
openstackgerritOleksii Chuprykov proposed openstack/heat-specs: Add batching to ResGroup  https://review.openstack.org/18350613:56
*** jprovazn is now known as jprovazn_bbl13:56
*** pas-ha has joined #heat13:56
*** pas-ha_ has joined #heat13:59
*** pas-ha_ has quit IRC13:59
*** mwheckmann has joined #heat14:00
*** blomquisg1 has joined #heat14:05
*** andersonvom has joined #heat14:06
*** sdake has joined #heat14:08
*** signed8bit has joined #heat14:11
*** nihilifer has quit IRC14:11
*** Qiming__ has joined #heat14:12
*** Murali_ has joined #heat14:14
*** BManojlovic has joined #heat14:15
*** Qiming_ has quit IRC14:15
zanebskraynev: answered your question on https://review.openstack.org/#/c/183215/ - let me know if that doesn't clear it up14:16
*** dims is now known as dimsum__14:16
*** sdake has quit IRC14:16
*** Murali has quit IRC14:16
*** Murali_ is now known as Murali14:16
*** prazumovsky has quit IRC14:20
skraynevzaneb: thx. looking...14:20
openstackgerritOleksii Chuprykov proposed openstack/heat-specs: Add batching to ResGroup  https://review.openstack.org/18350614:20
*** jcoufal has quit IRC14:21
*** saju_m has quit IRC14:26
*** sdake has joined #heat14:28
*** achanda has joined #heat14:36
*** sabeen1 has joined #heat14:43
*** achanda has quit IRC14:43
*** mragupat has joined #heat14:45
*** mragupat has quit IRC14:46
openstackgerritOleksii Chuprykov proposed openstack/heat-specs: Add batching to ResGroup  https://review.openstack.org/18350614:47
*** vijendar has joined #heat14:49
*** alexpilotti has joined #heat14:50
*** hdd has joined #heat14:51
*** bdossant has quit IRC14:55
*** nihilifer has joined #heat14:57
*** mikal has joined #heat14:57
*** radez_g0n3 is now known as radez15:02
*** sdake_ has joined #heat15:03
*** sabeen2 has joined #heat15:07
*** sdake has quit IRC15:08
*** gus has quit IRC15:08
*** gus has joined #heat15:09
*** sabeen1 has quit IRC15:10
*** mwheckmann has quit IRC15:10
*** mwheckmann has joined #heat15:14
*** jistr is now known as jistr|mtgh15:15
*** jistr|mtgh is now known as jistr|mtg15:15
*** Qiming_ has joined #heat15:17
*** mwheckmann has quit IRC15:19
*** mwheckmann has joined #heat15:19
*** Qiming__ has quit IRC15:20
jasond`skraynev: hello15:23
*** BManojlovic has quit IRC15:23
*** dyasny has quit IRC15:24
*** dyasny has joined #heat15:24
*** Tango has joined #heat15:25
*** jistr|mtg is now known as jistr15:28
*** radez is now known as radez_g0n315:29
*** pm90_ has joined #heat15:33
*** Murali has quit IRC15:33
skraynevzaneb: thx for clarification. it was enough15:41
skraynevjasond`: yeah15:42
skraynevI am here15:43
jasond`skraynev: hey, how's it going with the stack tags filtering in python-heatclient?15:43
*** Drago has joined #heat15:45
*** Drago has quit IRC15:45
*** Drago has joined #heat15:46
zanebskraynev: cool, thanks15:47
skraynevjasond`: ummm. probably I forgot something. could you refresh my memories?15:49
zanebryansb: did some back-of-the-envelope calculations... looks to me like the files dict could be maybe ~0.5MB15:49
*** Qiming_ has quit IRC15:49
zaneb(* no actual envelopes were harmed, for once)15:50
ryansbheh15:50
ryansbyou mean presuming we only had one copy, right?15:50
zanebyeah, each one would be about that much15:50
jasond`skraynev: sure "Apr 17 09:18:58 <skraynev> jasond: I have not seen patch to python-heatclient with commands for tags"  "Apr 17 09:19:36 <skraynev> jasond: do you plan to upload it ? I'd like to try it ;)"15:51
ryansbsounds approximately correct15:51
zanebso if we assume 50 stacks with maybe 4 copies each... that puts us into an awkward range15:51
zanebwhere we can make maybe a 20% (tops) saving, which is worth it15:52
zanebbut really in terms of scaling to a lot more nested stacks... we're still hosed :(15:52
ryansbso 100MB, according to my envelope (total size) and 20MB (saved size)15:52
*** jrist has quit IRC15:52
skraynevjasond`: lol. I meant, another: try = check, test it, when it be on review ;)15:52
skraynevjasond`: sorry for the confused phrase from me.15:53
ryansband the actual 20MB savings would be reduced by mem. fragmentation15:53
ryansb:(15:53
zanebryansb: no, I mean if we're using 550MB and we can reduce ~100MB of files down to 0.5MB, then we've still saved less than 20%15:53
jasond`skraynev: oh, ha ha.  i'm glad i asked about it.  no problem.  i will start working on that then15:54
zanebshardy: merge.py is starting to look like not such a bad idea :D15:55
ryansbzaneb: ah, gotcha15:56
skraynevjasond`: heh. thx for the question it was in time! and my apologize. I really did not want to confuse you.15:56
openstackgerritRabi Mishra proposed openstack/heat: Add validation for parameter_groups  https://review.openstack.org/18346615:56
jasond`skraynev: it's no problem.  thanks for clearing that up15:57
skraynevjasond`: no problem, I glad that you asked about this staff.15:57
*** ifarkas has quit IRC15:58
*** shakamunyi has quit IRC16:00
*** barra204 has quit IRC16:00
*** sdake_ has quit IRC16:00
*** Marga_ has joined #heat16:00
shardyzaneb: Hah, so to clarify, we've gone from one giant stack breaks heat to lots of little stacks break heat? ;D16:00
zanebindeed16:01
*** harlowja has joined #heat16:01
zanebshardy: part of the problem is, everything we do to make Heat more scalable increases memory use, which is a known tradeoff, but the undercloud can only have one node16:02
ryansbzaneb: your ref cycle patch breaks tuskar, btw16:02
shardyzaneb: I don't think it's necessarily true that the undercloud can only have one node16:02
shardyI think we need to support deploying $reasonable number of overcloud nodes with one seed/undercloud node16:03
*** harlowja has quit IRC16:03
shardyif you have a really huge deployment, you're going to want to scale out the undercloud for performance and availability reasons IMO16:03
shardyThe question is the definition of $reasonable I guess16:03
zanebyeah, but we don't support that yet16:04
ryansbAIUI the idea is that once your oc is deployed you can have your uc go down sometimes and be fine16:04
ryansbso HA uc isn't a huge priority16:04
ryansbalso $reasonable for a 3-node OC should probably be equal to 116:04
zaneblol, yes16:05
ryansb"Oh, you want a 3 node oc? Here, deploy a seed VM that will bootstrap 9 nodes for your HA undercloud"16:05
ryansbno thanks16:05
shardyryansb: Yeah, agreed, $reasonable must be at least a few hundred nodes IMO16:05
zanebryansb: are you set up to test the reference loop removal patches and see if that makes any difference?16:05
zanebryansb: in reality, more like 1 node, 24GB RAM ;)16:06
ryansbzaneb: yeah, I'm set up16:06
ryansbbut it breaks tuskar16:06
zanebthe patch breaks tuskar?16:06
shardyeverything breaks tuskar16:06
ryansbso with your ref loop patch, the stack create for tuskar fails because the stack ref disappears16:06
zanebdoh!16:07
zanebwe should stop that patch from merging upstream then16:07
ryansbI'm adding a "if there's no stack ref, go ahead and grab from the DB"16:07
ryansbyeah, well I just found it16:07
ryansbwas testing my own patches earlier16:07
zanebnah, grabbing from the db is the wrong solution. we should make sure we're retaining a ref to the stack all the time16:09
skraynevzaneb: only last patch should be stopped, I hope16:09
zanebskraynev: correct16:09
zanebI just set Workflow -116:09
zanebhopefully that might stop it16:09
ryansboh, you got it before I refreshed16:10
ryansbthe first patch (that's already merged) seems fine16:10
ryansbjust rolled back and tested16:10
zanebryansb: can you send me the stack trace from where it failed?16:10
ryansbzaneb: can you -1 the rest of that series while I do some digging?16:10
zanebI wonder why are functional tests didn't catch this :/16:11
ryansbnot a clue16:11
ryansbit's probably something like "the tests were holding a ref themselves, so it didn't happen"16:11
zanebthe rest of the series should be harmless16:11
ryansbwait, functests should've still caught that16:11
ryansbhrm.16:11
zanebwith the possible exception of the one that's merged already16:12
zanebyeah, functional tests test from the outside16:12
ryansbI tested the merged one independently and it seems fine16:12
ryansbmaybe we should just add tuskar as a submodule for functests16:12
ryansbit seems to break everything pretty thoroughly16:12
zanebI was half-expecting them to fail in the gate, but when they didn't I assumed I had caught everything with the unit tests16:13
zaneb(which found only one problem)16:13
*** jrist has joined #heat16:13
*** jrist has quit IRC16:13
*** jrist has joined #heat16:13
ryansbzaneb: here's your trace, btw16:13
ryansbhttp://fpaste.org/222364/14317064/16:14
zanebthanks16:14
zanebwhat the...16:15
zanebso it logs CREATE: ResourceGroup "Controller" Stack "overcloud" [61a1b847-8d09-448a-b1a9-a2bde05ef012]16:15
*** e0ne is now known as e0ne_16:15
zanebclearly it the Stack exists at that point16:15
*** nihilifer has quit IRC16:15
zaneboh, it might be in stack_resource16:16
zanebryansb: is there another stack trace in the logs by any chance?16:19
ryansblemme look16:20
*** e0ne_ is now known as e0ne16:20
zanebthis one is the local one raised by the parent stack upon receipt of an error over RPC16:20
zanebI suspect that the child one gets turned into a validation error, so the stack doesn't get printed :(16:20
zanebs/stack/traceback/16:20
zanebtoo many overloaded meanings of 'stack'16:21
ryansbhttp://fpaste.org/222366/14317068/16:21
*** blues-man has left #heat16:21
ryansbthere's another16:21
ryansbheh, no joke16:21
ryansb"we use heat stacks to provision our stack, then sometimes read the stacks from errors16:21
ryansb"16:21
zanebsecond one looks like the same thing but on update instead of create16:22
ryansbhere's another http://fpaste.org/222367/31706979/16:23
ryansbk, that's the earliest one16:23
zanebstill in the parent16:23
ryansbyeah, that16:25
*** derekh_ has quit IRC16:25
*** sikor_sxe has quit IRC16:25
*** tspatzier has quit IRC16:26
*** tspatzier has joined #heat16:26
ryansb1 sec, rerunning with a blank logfile16:26
*** tspatzier has quit IRC16:26
*** pm90__ has joined #heat16:28
*** pm90_ has quit IRC16:28
zanebryansb: this should give us the debug we need - http://paste.fedoraproject.org/222368/7074141416:30
*** Marga_ has quit IRC16:35
ryansbhttp://fpaste.org/222372/07750143/16:35
ryansbmsg is angry about the str16:36
ryansbgonna fix that first16:36
*** nihilifer has joined #heat16:36
*** e0ne has quit IRC16:37
zanebbah16:37
zanebI thought that was actually the exception object, but it isn't16:38
ryansbunicode: the natural enemy of all python programmers16:38
*** pm90__ has quit IRC16:39
zanebwe don't even need to print that then16:39
*** pm90_ has joined #heat16:39
zanebproblem is apparently in StackResource.validate_nested_stack16:39
ryansbhttp://fpaste.org/222374/14317079/16:39
zanebwhich seems... inexplicable16:40
ryansb¯\_ツ_/¯16:40
ryansbhttps://dionscher.files.wordpress.com/2015/04/multithreaded-programming.jpeg16:40
zanebah, the root stack nonsense16:40
zanebboy that is awful16:40
zanebthat could be it16:40
openstackgerritMerged openstack/heat: Retain references to stacks in all unit tests  https://review.openstack.org/18321516:41
openstackgerritMerged openstack/heat: Pass stack to thread in resource_signal  https://review.openstack.org/18321616:41
* zaneb goes to investigate root stacks16:41
*** spzala has joined #heat16:41
*** EricGonczer_ has joined #heat16:45
zanebIT BURRRRNS16:50
*** alanf-mc has joined #heat16:51
zanebso the innocent looking property root_stack...16:52
shardylol, that stuff is really evil16:52
zanebloads the seemingly innocent-looking property parent_resource (3 times)...16:52
zanebdespite the comment that "this should only be used by "Fn::ResourceFacade"16:52
zaneband not only recursively loads all the parent stacks back to the root...16:53
zanebbut *also* their resources, which we don't need and which are already equipped with lazy-loading16:53
*** inc0 has joined #heat16:53
zaneb^#&*&#% !*&(#16:54
*** radez_g0n3 is now known as radez16:55
*** radez is now known as radez_g0n316:56
*** radez_g0n3 is now known as radez16:56
zanebryansb: we could save a truckload of memory by just fixing this16:57
*** radez is now known as radez_g0n316:57
*** thedodd has joined #heat17:00
*** alexheneveld has joined #heat17:02
*** dimsum__ has quit IRC17:04
*** Marga_ has joined #heat17:04
*** dimsum__ has joined #heat17:05
ryansbo.O17:06
*** thedodd has quit IRC17:07
zanebryansb: even with a fix for this though... there must be so many more copies of the files loaded than I imagined17:11
zanebso that's sort-of good news17:11
zanebour upside may be higher than we calculated earlier17:12
*** jistr has quit IRC17:12
*** pas-ha has quit IRC17:15
*** e0ne has joined #heat17:15
*** EricGonc_ has joined #heat17:18
ryansbhrm. bbiab, heading to get some lunch17:20
ryansblmk if you need me to test any patches17:20
zanebk, thanks17:20
zanebI might do the same17:21
*** EricGonczer_ has quit IRC17:21
*** nihilifer has quit IRC17:21
ryansbwfm17:22
*** inc0 has quit IRC17:22
zaneboh geez17:23
zanebso next we are calling total_resources() which literally loads every resource for every stack in the entire tree17:23
openstackgerritRob Pothier proposed openstack/heat: Add BayModel and Bay to Magnum client plugin  https://review.openstack.org/17977117:24
*** achanda has joined #heat17:27
*** daneyon has joined #heat17:31
zanebI raised https://bugs.launchpad.net/heat/+bug/145558917:32
openstackLaunchpad bug 1455589 in heat "Memory use is quadratic in the number of stacks" [Critical,New]17:32
*** daneyon_ has joined #heat17:33
mwheckmannIf the objecj nodes are calculating csums, then what if due to some corruption object nodes don't have the same ETag for an object in metadata? I guess I should use the source to really understand the behviour..17:33
*** tlashchova has quit IRC17:34
mwheckmanni.e. if replication factor of 3x spread across 3 nodes and not all nodes agree on the Etag value17:34
*** nihilifer has joined #heat17:35
*** daneyon has quit IRC17:36
openstackgerritMerged openstack/heat: Add CIDR format validation  https://review.openstack.org/18220917:40
zanebmwheckmann: you sure you're in the right channel?17:55
*** jprovazn_bbl has quit IRC17:56
mwheckmannzaneb: nope :) Just noticed that my client was in the wrong one. Wanted to jump in on a conversation in #openstack-swift17:57
mwheckmannseems I failed miserably :) Explains why I got no answers. Thanks for pointing that out!17:58
zanebthat's what I thought :)17:58
*** daneyon has joined #heat18:11
*** daneyon_ has quit IRC18:11
*** saju_m has joined #heat18:11
*** Murali has joined #heat18:15
*** saju_m has quit IRC18:18
*** andersonvom has quit IRC18:21
*** openstackgerrit has quit IRC18:22
*** openstackgerrit has joined #heat18:23
*** tspatzier has joined #heat18:23
*** Marga_ has quit IRC18:26
*** saju_m has joined #heat18:30
*** shakamunyi has joined #heat18:37
*** barra204 has joined #heat18:37
*** Murali has quit IRC18:40
*** saju_m has quit IRC18:43
*** nihilifer has quit IRC18:53
*** MasterPiece has joined #heat18:55
*** Marga_ has joined #heat18:57
*** Marga__ has joined #heat19:01
*** Marga_ has quit IRC19:01
*** nihilifer has joined #heat19:05
*** Marga__ has quit IRC19:05
zanebryansb: it occurs to me that you probably cherry-picked that patch and you don't have 2032548c4e501ebf90604bf60a69fdf9f64b981d in your tree. would I be right?19:07
ryansbhrm19:08
ryansbyou would be right, because I cherry-picked "zaneRefPatch~4" instead of ~519:09
zanebdoh19:10
ryansbworst game of go fish ever "Do you have any hashes beginning with 20325 in your tree?"19:10
zanebso that explains why you're getting errors anyway19:10
zaneblol19:10
ryansbI'll slap that in, thanks for the catch19:10
zanebpresumably you have d29bb1927489d5590f17d2f3ef2904b6f5edd0d4, which you'll also need19:10
ryansbyeah, I have that19:11
ryansbI have 3/4 of your patches, and about to add #419:11
*** Marga_ has joined #heat19:12
zanebcool19:12
ryansbnot sure how you divined that I was missing that patch, but good on you19:13
zanebwell, I looked at the code and it was awful, but I couldn't figure out how it could be losing the reference19:14
zaneband then I realised that was the exact bug I had fixed when I was developing it :)19:14
*** Marga__ has joined #heat19:14
*** Marga_ has quit IRC19:14
zanebthat explains why the tests didn't catch it, anyway19:14
*** andersonvom has joined #heat19:17
*** pm90_ has quit IRC19:18
*** pm90_ has joined #heat19:19
*** andersonvom has quit IRC19:22
*** pm90_ has quit IRC19:23
*** blomquisg1 has quit IRC19:26
*** daneyon has quit IRC19:27
*** alanf-mc has quit IRC19:32
*** alanf-mc has joined #heat19:34
*** alexheneveld has quit IRC19:43
ryansbhah, yeah19:45
*** barra204 has quit IRC19:48
*** shakamunyi has quit IRC19:48
openstackgerritJason Dunsmore proposed openstack/python-heatclient: Add stack tag filtering options to stack-list  https://review.openstack.org/18367919:50
openstackgerritJason Dunsmore proposed openstack/python-heatclient: Add --tags option to stack-create and stack-update  https://review.openstack.org/18368019:50
zanebryansb: working?19:52
ryansbhad to redo discovery19:52
ryansbit'll be a bit19:52
zanebkk19:53
*** crose has quit IRC20:03
ryansbzaneb: it's past where it broke earlier, and mem usage is down tremendously20:04
*** tlashchova has joined #heat20:05
*** Marga__ has quit IRC20:08
*** Marga__ has joined #heat20:11
*** Drago has quit IRC20:15
*** alexpilotti has quit IRC20:17
*** andersonvom has joined #heat20:17
*** achanda has quit IRC20:18
openstackgerritVijendar Komalla proposed openstack/heat: Change user_data_format default value in Rackspace server  https://review.openstack.org/18369020:22
ryansbzaneb: failed, but I don't think it's your patch20:22
*** andersonvom has quit IRC20:22
openstackgerritVijendar Komalla proposed openstack/heat: Change user_data_format default value in Rackspace server  https://review.openstack.org/18369020:23
*** daneyon has joined #heat20:25
*** achanda has joined #heat20:37
*** Marga__ has quit IRC20:39
*** Marga_ has joined #heat20:40
*** vijendar has quit IRC20:41
*** vijendar has joined #heat20:43
*** multi_io has quit IRC20:48
*** Drago has joined #heat20:51
*** Drago has quit IRC20:51
*** Drago has joined #heat20:52
*** multi_io has joined #heat20:53
*** alexheneveld has joined #heat21:06
*** htruta has quit IRC21:15
zanebryansb: think it's safe to reapprove?21:17
ryansbzaneb: I haven't had it work yet21:18
*** andersonvom has joined #heat21:18
ryansbbut this system is drifty, so I'm trying to track down whether the failure is related to your patch21:18
ryansbI should be able to confirm in a bit21:18
ryansbI'll +1 when it works21:18
*** vijendar has quit IRC21:19
ryansbshould be sometime soon. Though I said that an hour ago too21:19
zanebryansb: ok thanks21:20
zanebI'd really like to get this stuff merged into midstream today if I can21:20
zanebso that folks have something to work with next week21:20
ryansbnp. If the state of heat before it ruined everything is indicative, that patch may save our bacon21:20
ryansbyeah, I gotcha21:21
zanebmem usage is looking good? numbers?21:21
ryansbfortunately my wife is in grad school, so I don't have anywhere to be ;)21:21
ryansb133M21:21
ryansbas opposed to 54021:21
zaneboh, that's much better21:21
ryansbyeah21:22
zaneblittle bit of caching, we can get that right down ;)21:22
ryansbheh, yeah21:22
ryansbof course, then it ruined my whole virt setup21:22
ryansbso it may have just loaded some stuff then crashed21:23
*** andersonvom has quit IRC21:23
zaneblol, ok21:28
*** alexheneveld has quit IRC21:32
*** tspatzier has quit IRC21:42
*** alanf-mc_ has joined #heat21:42
*** alanf-mc has quit IRC21:45
*** Marga_ has quit IRC21:47
*** Marga_ has joined #heat21:48
*** Marga_ has quit IRC21:50
*** Marga_ has joined #heat21:52
*** sabeen2 has quit IRC21:53
*** Marga_ has quit IRC21:55
ryansbstill waiting for my images to redeploy :/21:56
*** EricGonc_ has quit IRC21:59
zanebugh. tripleo. :(22:06
*** jasond` has quit IRC22:11
*** hdd has quit IRC22:11
*** hdd has joined #heat22:14
*** Marga_ has joined #heat22:14
*** rpothier has quit IRC22:16
ryansbzaneb: working now, looks like22:18
ryansband sitting at 133M of ram22:18
zanebryansb: w00t!!! thank you!22:18
ryansbno problemo22:19
zanebwe still have work to do to fix bug 1455589, but I think this should get us unblocked again22:20
openstackbug 1455589 in heat "Memory use is quadratic in the number of stacks" [Critical,New] https://launchpad.net/bugs/145558922:20
zanebmaybe I'll downgrade that to 'high'22:21
openstackgerritZane Bitter proposed openstack/heat: Get rid of circular references in Resource and Function  https://review.openstack.org/18321722:26
openstackgerritZane Bitter proposed openstack/heat: Don't swallow AssertionErrors  https://review.openstack.org/18372222:26
*** andersonvom has joined #heat22:34
*** andersonvom has quit IRC22:39
*** spzala has quit IRC22:40
*** sabeen1 has joined #heat22:43
multi_iodo any of you people generate their heat templates, rather than writing them manually?22:47
multi_ioyaml and the heat functions are pretty verbose and limited.22:48
multi_io*seem to be22:48
*** jamielennox|away is now known as jamielennox22:48
*** hdd has quit IRC22:53
*** jimmybee has joined #heat22:54
*** dannycastle has joined #heat22:55
zanebmulti_io: yes, lots of people do that22:56
*** dannycastle has quit IRC22:57
*** signed8bit is now known as signed8bit_ZZZzz22:58
*** jimmybee has quit IRC22:58
multi_iozaneb: any specific tools you use?22:59
*** crystal0079 has joined #heat23:01
*** crystal0079 has left #heat23:01
*** achanda_ has joined #heat23:01
*** e0ne has quit IRC23:04
*** MasterPiece has quit IRC23:05
*** achanda has quit IRC23:05
*** dims_ has joined #heat23:05
*** achanda_ has quit IRC23:06
*** dimsum__ has quit IRC23:07
zanebmulti_io: I don't do it personally, but the TripleO project used to just use python. other folks are using jinja2 templating.23:08
*** Marga_ has quit IRC23:11
multi_iozaneb: ok thanks23:11
*** Marga_ has joined #heat23:18
openstackgerritMerged openstack/heat: Get rid of circular references in Resource and Function  https://review.openstack.org/18321723:27
zanebw00t!23:30
*** Marga_ has quit IRC23:31
*** andersonvom has joined #heat23:35
*** hdd has joined #heat23:38
*** andersonvom has quit IRC23:39
*** sabeen1 has quit IRC23:40
*** Marga_ has joined #heat23:40
ryansbzaneb: \o/23:49
ryansbI'll toss it into midstream in a bit, after create it's stable at 258M23:50
ryansbgoing to test w/ some updates no23:50
ryansb*now23:50
*** Marga_ has quit IRC23:57
*** vahidh has joined #heat23:59

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