Wednesday, 2015-11-04

*** Raj has quit IRC00:02
*** Kennan has joined #heat00:07
*** steveg_afk has joined #heat00:09
*** daneyon has joined #heat00:12
*** daneyon_ has quit IRC00:15
*** manishg has quit IRC00:16
*** manishg_ has joined #heat00:17
*** pratikma_ has quit IRC00:17
*** pt_15 has quit IRC00:18
*** slagle has quit IRC00:19
*** manishg_ has quit IRC00:20
*** Drago has quit IRC00:20
*** shadower has quit IRC00:23
*** shadower has joined #heat00:23
*** vishwanathj has joined #heat00:27
*** Drago has joined #heat00:28
*** dims has joined #heat00:28
*** sdake_ has quit IRC00:34
*** jobewan has quit IRC00:35
*** zengyingzhe__ has joined #heat00:36
*** ricolin has joined #heat00:39
*** sdake has joined #heat00:44
*** bapalm has quit IRC00:46
*** slagle has joined #heat00:47
*** bapalm has joined #heat00:48
*** manishg has joined #heat00:50
*** manishg has quit IRC00:50
*** manishg has joined #heat00:51
*** slagle has quit IRC00:52
*** Drago has quit IRC00:53
*** slagle has joined #heat00:54
*** zhenguo has joined #heat01:00
*** achanda has quit IRC01:02
*** liusheng has joined #heat01:05
*** daneyon_ has joined #heat01:06
*** daneyon has quit IRC01:08
*** jaime49 has quit IRC01:11
*** chlong has joined #heat01:18
zanebrowagner: I suspect that UrbanCodeDeploy adds its own implicit dependencies that aren't defined in the template like Heat dependencies are supposed to be. you should probably talk to the UCD developers; Heat developers can't really help you because it's not open source01:21
openstackgerritRico Lin proposed openstack/heat-specs: Create resources for environment  https://review.openstack.org/24143201:27
*** signed8bit has quit IRC01:28
*** Pavo has quit IRC01:30
*** bana_k has quit IRC01:32
openstackgerritYingzhe Zeng proposed openstack/heat: Improve parameters obtaining logic from adopt stack data  https://review.openstack.org/24078001:33
*** Pavo has joined #heat01:37
*** Yanyanhu has joined #heat01:40
*** manishg has quit IRC01:40
openstackgerritRico Lin proposed openstack/heat-specs: Create resources for environment  https://review.openstack.org/24143201:46
*** huangtianhua has quit IRC02:03
*** huangtianhua_ has joined #heat02:03
*** elynn has joined #heat02:09
*** Yanyanhu has quit IRC02:09
*** dims has quit IRC02:13
*** elynn has quit IRC02:15
*** pratikmallya has joined #heat02:17
*** manishg has joined #heat02:17
*** jasond has joined #heat02:20
*** chlong has quit IRC02:25
*** cwolferh has quit IRC02:26
*** elynn has joined #heat02:27
*** Yanyanhu has joined #heat02:27
elynnmorning02:28
*** chlong has joined #heat02:31
*** Qiming has joined #heat02:35
*** jasond has quit IRC02:38
*** chlong has quit IRC02:42
*** jasond has joined #heat02:46
*** jasond has quit IRC02:57
*** zengyingzhe__ has quit IRC02:59
*** zengyingzhe_ has joined #heat03:04
*** Qiming has quit IRC03:06
*** huangtianhua_ has quit IRC03:10
*** huangtianhua_ has joined #heat03:11
*** sileht has joined #heat03:19
*** jasond has joined #heat03:23
*** lkarm has joined #heat03:25
*** lkarm has quit IRC03:28
*** jdandrea has quit IRC03:29
*** ricolin has quit IRC03:29
*** dims has joined #heat03:33
*** jasond has quit IRC03:33
*** Qiming has joined #heat03:34
*** jamielennox is now known as jamielennox|away03:38
*** jamielennox|away is now known as jamielennox03:40
*** achanda has joined #heat03:44
*** masco__ has joined #heat03:50
cmystermorning03:51
*** pratikmallya has quit IRC03:51
*** Raj has joined #heat03:57
*** liusheng has quit IRC03:58
*** liusheng has joined #heat03:59
*** dims has quit IRC04:03
openstackgerrithuangtianhua proposed openstack/heat: Catch all exceptions to move stack to UPDATE_FAILED  https://review.openstack.org/23979004:04
*** achanda has quit IRC04:04
*** vishwana_ has joined #heat04:07
*** huangtianhua has joined #heat04:08
*** Michalik_ has joined #heat04:08
*** daneyon has joined #heat04:08
*** manishg_ has joined #heat04:09
*** bapalm_ has joined #heat04:10
*** gus_ has joined #heat04:10
*** arlion_ has joined #heat04:10
*** arlion_ has joined #heat04:10
*** KanagarajM has joined #heat04:11
*** liusheng has quit IRC04:11
*** ameade has quit IRC04:11
*** huangtianhua_ has quit IRC04:11
*** Michalik has quit IRC04:11
*** fbo has quit IRC04:11
*** manishg has quit IRC04:11
*** daneyon_ has quit IRC04:11
*** dsneddon has quit IRC04:11
*** bapalm has quit IRC04:11
*** vishwanathj has quit IRC04:11
*** gus has quit IRC04:11
*** arlion has quit IRC04:11
*** Michalik_ is now known as Michalik04:11
*** liusheng has joined #heat04:12
*** fbo has joined #heat04:15
*** gus_ is now known as gus04:18
*** asalkeld has quit IRC04:19
*** dsneddon has joined #heat04:20
*** ameade has joined #heat04:21
*** boris-42 has quit IRC04:28
*** asalkeld has joined #heat04:30
*** chlong has joined #heat04:33
*** spzala has quit IRC04:43
*** chlong has quit IRC04:45
*** chlong has joined #heat04:49
*** achanda has joined #heat04:54
*** links has joined #heat05:02
*** chlong has quit IRC05:10
*** Raj has quit IRC05:35
*** jraju has joined #heat05:37
*** links has quit IRC05:37
*** jaosorior has joined #heat05:40
*** eliqiao has quit IRC05:42
*** vijayagurug has joined #heat05:45
*** ramishra has quit IRC05:46
*** ramishra has joined #heat05:46
*** eliqiao1 has joined #heat05:51
*** lkarm has joined #heat05:52
openstackgerritDi XiaoLi proposed openstack/heat-specs: Online Schema Migrations  https://review.openstack.org/24148405:55
*** eliqiao1 has quit IRC05:55
*** lkarm has quit IRC05:56
openstackgerritDi XiaoLi proposed openstack/heat-specs: Online Schema Migrations  https://review.openstack.org/24148406:06
openstackgerritRabi Mishra proposed openstack/heat: Skip scenario test 'test_autoscaling_loadbalancer_neutron'  https://review.openstack.org/24148906:10
*** eliqiao has joined #heat06:11
*** gberginc has quit IRC06:14
*** jraju has quit IRC06:15
*** hdd has joined #heat06:16
*** sdake has quit IRC06:18
*** metral has quit IRC06:22
*** metral has joined #heat06:23
*** hdd has quit IRC06:27
*** e0ne has joined #heat06:40
*** links has joined #heat06:42
*** hdd has joined #heat06:45
*** ananta has joined #heat06:46
*** e0ne has quit IRC06:50
*** gberginc has joined #heat06:56
*** jprovazn has joined #heat06:57
*** jamielennox is now known as jamielennox|away07:02
*** ricolin has joined #heat07:07
*** tsufiev has quit IRC07:12
*** tsufiev has joined #heat07:12
*** zengyingzhe_ has quit IRC07:26
openstackgerritDi XiaoLi proposed openstack/heat-specs: Online Schema Migrations  https://review.openstack.org/24148407:54
*** jaosorior has quit IRC08:00
*** jaosorior has joined #heat08:01
*** jaosorior has quit IRC08:01
*** jaosorior has joined #heat08:02
openstackgerritRico Lin proposed openstack/heat-specs: Shared Scope Environment  https://review.openstack.org/24151608:03
openstackgerritRico Lin proposed openstack/heat-specs: Create resources for environment  https://review.openstack.org/24143208:05
*** kodokuu has joined #heat08:06
*** achanda has quit IRC08:07
kodokuuHi, somebody can tell me how heat-cfn generate signature for ec2 credential ?08:13
kodokuuBecause I guess my keystone doesn't recognize my signature because they don't take same URI08:14
therveananta, Hi, you around? I wanted to chat about remaining failure in convergence tests08:18
anantatherve: Hi08:19
anantayes08:19
therveIs someone looking at those? The 2 remaining are interesting08:19
kodokuutherve Hi ;)08:19
therveThey don't seem related to race conditions, more some logic somewhere in delete that doesn't work08:19
anantayes, I am looking into it08:19
anantaonly one for delete ... the autoscaling one08:19
thervekodokuu, Look at heat_waitcondition_server_url in the conf08:20
therveananta, Any idea what's going on?08:20
kodokuutherve So after more test I guess heat and keystone not use same endpoint URI for generate signature08:20
*** shardy has joined #heat08:20
kodokuutherve heat_waitcondition_server_url =https://controler:8000/v1/waitcondition08:20
*** yuanying has quit IRC08:20
anantanone of the AS resources are getting deleted... (it fails with error The parameter is not provided)08:21
therveYeah I noticed that08:21
thervekodokuu, Is that how you talked to it?08:21
anantacreated a etherpad for the failing tests https://etherpad.openstack.org/p/convergence-failing-tests08:22
therveananta, I wonder if it has anything to do with the empty template. But then it I guess it would fail for other tests08:22
therveAh awesome08:22
anantatherve: few are intermittently failing08:23
therveThe first one fails because of a lock08:23
anantait fails because the delete waits for update to complete...08:24
anantasirushti was working on cancel update implemetation which we use to cancel the update and delete the stack08:25
therveRight! OK I understand, this is definitely an issue08:25
therveSo yeah that delete in autoscaling is one remaining mystery08:26
anantatherve: the last two are bit annoying, they are intermittent and only sometime fail08:26
therveI'm surprised with the notifications one08:26
kodokuutherve Hum I don't know but I guess signature generate by heat is not good08:26
*** pratikmallya has joined #heat08:27
anantatherve:  yeah, don't know how it could be possible though08:27
anantatherve: stack update rollback seems simple but it fails occasionally :)08:28
openstackgerrithuangtianhua proposed openstack/heat: Remove the redundant spaces in function doc  https://review.openstack.org/24152108:28
therveananta, Yeah that's a strange one too08:30
therveI'd focus on the autoscaling issue, once that one is fixed we may get a better idea08:31
therveI'll quickly look at the rollback one to see if I find something08:31
*** tspatzier has joined #heat08:33
anantatherve: sure, the autoscaling is hit by another one https://bugs.launchpad.net/bugs/151293708:33
openstackLaunchpad bug 1512937 in neutron "CREATE_FAILED status due to 'Resource CREATE failed: NotFound: resources.pool: No eligible backend for pool" [Undecided,In progress] - Assigned to Rabi Mishra (rabi)08:33
therveHum yeah that one is just a neutron issue though08:34
*** jtomasek has quit IRC08:37
*** Guest98556 is now known as d0ugal08:37
*** d0ugal has quit IRC08:38
*** d0ugal has joined #heat08:38
*** jtomasek has joined #heat08:38
*** sorantis has joined #heat08:39
*** jcoufal has joined #heat08:40
*** tspatzier has quit IRC08:41
*** vgridnev has joined #heat08:44
*** manishg_ has quit IRC08:45
*** palexster1 is now known as palexster08:47
*** yuanying has joined #heat08:53
*** sorantis has quit IRC08:56
*** tspatzier has joined #heat08:56
*** e0ne has joined #heat08:59
*** tochi has quit IRC09:03
*** jistr has joined #heat09:08
*** sorantis has joined #heat09:12
kodokuutherve After debug keystone I have that09:21
kodokuusignature (send by heat): tEnIPQxhbDlcR1MIuQCyzZ92UtHrDcVITlG/rWfqPQA=        signature(use by keystone for check): FTh0hLTk3PnH18YTTBDm88+e1DlnUR4kkiSK+rLEDMo=09:21
kodokuuso heat generate wrong signature ?09:21
*** yassine__ has joined #heat09:21
thervekodokuu, So one of the elements must be different09:25
therveThe URL is an obvious candidate, that's why I mentioned it09:25
kodokuuyes because I use proxy for keystone09:25
kodokuumaybe keystone use local URI and heat use other URI09:26
kodokuuI need to debug heat for see what URI is used09:26
*** rakesh_hs has joined #heat09:26
*** elynn has quit IRC09:27
*** pratikmallya has quit IRC09:28
thervekodokuu, It uses heat_waitcondition_server_url09:29
therveAnd whatever it receives when it's hit09:30
*** vgridnev has quit IRC09:34
*** vgridnev has joined #heat09:34
*** dkusidlo has joined #heat09:37
*** manishg has joined #heat09:42
*** pratikmallya has joined #heat09:45
*** stannie has joined #heat09:46
openstackgerritThomas Herve proposed openstack/heat: Add caching to neutron extensions  https://review.openstack.org/24155109:53
*** alex_xu has quit IRC09:54
*** jamielennox|away is now known as jamielennox09:55
*** derekh has joined #heat09:55
*** alex_xu has joined #heat09:57
kodokuutherve heat use  heat_waitcondition_server_url but keystone doesn't know this URI. So how keystone can generate the good signature ?09:59
*** elynn has joined #heat10:02
*** kodokuu has quit IRC10:03
*** zhenguo has quit IRC10:03
*** pratikmallya has quit IRC10:05
*** elynn has quit IRC10:07
*** Qiming has quit IRC10:08
*** khomkrit has joined #heat10:18
*** khomkrit has left #heat10:18
*** metral is now known as metral_zzz10:23
*** metral_zzz is now known as metral10:23
*** alex_xu has quit IRC10:23
*** alex_xu has joined #heat10:24
*** ricolin has quit IRC10:35
*** Yanyanhu has quit IRC10:45
*** jistr_ has joined #heat10:48
*** jistr has quit IRC10:48
*** vgridnev has quit IRC10:49
*** liusheng has quit IRC10:50
*** liusheng has joined #heat10:51
*** tspatzier has quit IRC10:51
*** jistr_ has quit IRC10:53
*** Kennan has quit IRC10:53
*** khomkrit has joined #heat10:56
*** Kennan has joined #heat10:56
*** sorantis has quit IRC11:00
*** Qiming has joined #heat11:01
*** ochuprykov has joined #heat11:04
*** jistr_ has joined #heat11:05
*** vgridnev has joined #heat11:14
*** sorantis has joined #heat11:20
*** masco__ has quit IRC11:22
*** dkusidlo has quit IRC11:22
*** pratikmallya has joined #heat11:30
*** jaosorior has quit IRC11:40
*** jaosorior has joined #heat11:41
shardyJust added a section for our meeting later today, feel free to add any topics:11:42
shardyhttps://wiki.openstack.org/wiki/Meetings/HeatAgenda11:42
*** jaosorior has quit IRC11:49
*** jaosorior has joined #heat11:50
*** pratikmallya has quit IRC11:50
*** Kennan has quit IRC11:51
*** Kennan has joined #heat11:54
*** sorantis_ has joined #heat12:07
*** sorantis has quit IRC12:09
*** sorantis_ is now known as sorantis12:09
*** dims has joined #heat12:18
ramishrashardy: hi!  it seems gate is broken due a neutron-lbaas issue.  I've a patch up for review https://review.openstack.org/#/c/241481/. Not sure how long it would take to land.12:21
shardyramishra: Hi!  Thanks, looking12:21
ramishraI've also pushed a patch to disable the specific scenrio test if required https://review.openstack.org/#/c/241489/.12:21
*** Kennan has quit IRC12:22
*** masco__ has joined #heat12:23
shardyramishra: is there any workaround we can do on the heat side, to un-break the gate, and enable working with older lbaas versions, e.g liberty?12:24
*** Kennan has joined #heat12:25
*** tspatzier has joined #heat12:25
shardyramishra: if that patch isn't going to land immediately, I'd say we have to temporarily land a skip for the failing test12:25
ramishrashardy: yeah, it seems there are too many issues on the lbaas side.12:25
ramishrashardy: I've not checked if/how we can use older version with devstack. Though we can skip the test by landing the second patch.12:27
ramishrashardy:  https://review.openstack.org/#/c/241489/12:28
shardyramishra: thanks, approved12:28
*** tspatzier has quit IRC12:30
*** kodokuu has joined #heat12:30
*** ananta has quit IRC12:36
*** crose has joined #heat12:40
*** achanda has joined #heat12:40
*** achanda has quit IRC12:42
*** dims has quit IRC12:43
*** vijayagurug has left #heat12:43
*** chlong has joined #heat12:44
*** jistr_ is now known as jistr12:44
*** Qiming has quit IRC12:45
*** jamielennox is now known as jamielennox|away12:48
*** crose has quit IRC12:49
*** jasond has joined #heat12:49
*** cody-somerville has joined #heat12:49
openstackgerritBertrand Lallau proposed openstack/heat: Update instructions to generate GMR error reports  https://review.openstack.org/24158812:58
*** chlong has quit IRC13:01
*** chlong has joined #heat13:02
*** masco__ has quit IRC13:05
*** pratikmallya has joined #heat13:06
*** dprince has joined #heat13:09
*** dims has joined #heat13:09
*** jasond has quit IRC13:13
*** ricolin has joined #heat13:14
*** sorantis has quit IRC13:15
openstackgerritMerged openstack/heat: Skip scenario test 'test_autoscaling_loadbalancer_neutron'  https://review.openstack.org/24148913:16
*** jasond has joined #heat13:18
*** dims has quit IRC13:20
*** sorantis has joined #heat13:21
*** arlion_ is now known as arlion13:25
*** pratikmallya has quit IRC13:26
*** jasond has quit IRC13:26
*** KanagarajM has quit IRC13:29
openstackgerritRakesh H S proposed openstack/heat: DNM: Convergence: Assign params in special stack  https://review.openstack.org/24159813:32
*** kodokuu has quit IRC13:33
*** rakesh_hs has quit IRC13:35
*** pt_15 has joined #heat13:37
*** crose has joined #heat13:37
*** inc0 has joined #heat13:38
*** ricolin has quit IRC13:38
*** ochuprykov has quit IRC13:39
*** links has quit IRC13:41
*** crose has quit IRC13:42
*** elynn has joined #heat13:42
*** inc0 has quit IRC13:49
*** spzala has joined #heat13:49
*** elynn_ has joined #heat13:55
*** neelashah has joined #heat13:55
*** sergmelikyan has joined #heat13:56
*** elynn has quit IRC13:56
*** jdob has quit IRC14:00
*** jdob has joined #heat14:01
*** Piet has quit IRC14:02
*** jonesbr has joined #heat14:04
*** jaime49 has joined #heat14:10
*** dspano has joined #heat14:14
*** miguelgrinberg has quit IRC14:17
*** links has joined #heat14:17
*** miguelgrinberg has joined #heat14:19
dspanoGood morning all. I was wondering if there's anything in the heat roadmap for converting the integration tests into a plugin for Tempest?14:19
dspanohttp://docs.openstack.org/developer/tempest/plugin.html#14:19
dspanoI've got something almost finished for our internal use, and wasn't sure if someone is already working on something upstream.14:21
thervedspano, I believe it was somewhat discussed, but I don't know if anything happened14:31
therveAt stevebaker mentioned using tempest-lib14:31
*** tlashchova has quit IRC14:40
*** lkarm has joined #heat14:40
*** vijayagurug has joined #heat14:41
*** openstackgerrit has quit IRC14:47
*** openstackgerrit has joined #heat14:47
dspanotherve: I was able to get working fairly easily last night without changing much. Would it need a blueprint, or just commit?14:49
*** tlashchova has joined #heat14:49
thervedspano, I'd put the patch up for review and see if anyone complains :)14:50
therveNot sure why a spec would be useful14:50
dspanotherve: Cool. I was just double-checking. I didn't want to ruffle feathers.14:50
thervePeople like make others write things that we'll never read14:50
*** jaosorior has quit IRC14:51
*** jaime49 has quit IRC14:52
*** pratikmallya has joined #heat14:52
LimorStotlandHi all14:53
LimorStotlandi have problem... i have a big stack with nested stack inside and when i run the command : heat resource-list ${stack} -n 4 i keep getting MessagingTimeout: Timed out waiting for a reply to message ID14:54
LimorStotlandI changed the rpc_response_timeout to be 2 minutes but i still keep getting this error after 1 min14:55
LimorStotlandany idea what can change it to 1 min?14:56
therveMaybe you have an exception in the engine?14:56
therveHow big are we talking about?14:56
*** links has quit IRC14:56
LimorStotlandthe engine log seams clean14:57
LimorStotlandits have about 1000+ resources15:01
LimorStotlandbut it's weird because it like completely ignored from the new configuration of rpc_response_timeout15:02
therveEven a 1000 resources should be doable, I think?15:05
therveDepends on your setup of course, is it cpu bound?15:05
dspanotherve: I had one last question. Other projects put their tests in an api directory, is having them in a directory named functional over api important for a particular reason?15:08
dspanoI would just leave it as is if that's the case.15:09
LimorStotlandtherve  I don't think so...15:10
LimorStotlandtherve  is there anything else that can change the timeout configuration15:10
*** manishg has quit IRC15:11
*** vijendar has joined #heat15:13
*** signed8bit has joined #heat15:13
pratikmallyatherve: hey, thanks for replying to my email. A related question is: how is it that the notification_driver option is not present in the code for config? https://github.com/openstack/heat/blob/master/heat/common/config.py15:15
*** ochuprykov has joined #heat15:20
*** jaime49 has joined #heat15:20
*** Piet has joined #heat15:21
*** jaime49 has quit IRC15:22
*** jaime49 has joined #heat15:22
thervepratikmallya, It's an oslo messaging option15:24
therveLimorStotland, No idea15:24
LimorStotlandtherve : thanks15:25
thervedspano, Not sure which tests you're talking about15:25
thervedspano, heat_integrationtests/functional/ ?15:25
dspanotherve: heat/heat_integrationtests/functional15:25
therveI'd surprised if it mattered much :)15:26
therveI hope it could stay the same, though. Randomly renaming things is annoying.15:26
dspanotherve: I'm used to seeing one dir for api and another for scenario.15:26
pratikmallyatherve: ah, danke15:26
dspanotherve: I agree15:26
dspanotherve: I'll leave it as is for now.15:26
dspanotherve: Shouldn't be hard to change to api if it were desired in the future.15:28
thervedspano, Sure, but it can break patches in progress, changes stack traces and logs, etc15:29
pratikmallyatherve: so that option is to be specified in the oslo config and not the heat config? e.g. is it specified in /etc/heat/heat.conf?15:30
thervepratikmallya, It's a library used by heat, so yes in heat.conf15:31
*** jasond has joined #heat15:31
pratikmallyai see15:31
*** rpothier has joined #heat15:34
*** sorantis has quit IRC15:41
*** EricGonczer_ has joined #heat15:42
*** ochuprykov has quit IRC15:49
openstackgerritJay Dobies proposed openstack/heat: Hook into environment merging on server  https://review.openstack.org/24166215:50
*** crose has joined #heat15:51
openstackgerritJay Dobies proposed openstack/python-heatclient: Changes to support server-side environment resolution  https://review.openstack.org/23950415:51
*** kebray has quit IRC15:52
*** elynn_ has quit IRC15:59
*** randallburt has joined #heat16:04
*** EricGonczer_ has quit IRC16:08
*** randallburt has quit IRC16:10
*** randallburt has joined #heat16:10
*** kebray has joined #heat16:11
*** zz_gondoi is now known as gondoi16:15
openstackgerritRico Lin proposed openstack/heat-specs: Shared Scope Environment  https://review.openstack.org/24151616:17
*** ricolin has joined #heat16:18
*** ricolin has quit IRC16:22
*** neelashah1 has joined #heat16:26
*** neelashah has quit IRC16:29
*** dprince has quit IRC16:30
*** Drago has joined #heat16:33
*** Drago has quit IRC16:34
*** Drago has joined #heat16:34
*** achanda has joined #heat16:44
openstackgerritSteven Hardy proposed openstack/heat: Special-case for in-place TemplateResource update  https://review.openstack.org/23819416:45
*** dprince has joined #heat16:50
*** lkarm has quit IRC16:53
*** lkarm has joined #heat16:54
shardyzaneb: ^^ Hey FYI that's my attempt at making TemplateResource updates less destructive, ref the TripleO VIP replacement problem16:56
*** cwolferh has joined #heat16:57
shardyfeedback welcome, it appears to work and the code isn't too awful16:57
zanebshardy: kk16:57
*** EricGonczer_ has joined #heat16:57
zanebwithout having looked at the code, I think this is absolutely the Right Thing to do16:57
*** lkarm has quit IRC16:59
*** sergmelikyan has quit IRC17:00
zanebshardy: one question: did you consider moving the type check into Resource._needs_update()?17:00
shardyzaneb: Hmm, actually no I didn't, let me take a look17:01
*** sergmelikyan has joined #heat17:01
zanebthere'd be a certain logic to that17:02
*** manishg has joined #heat17:02
*** jcoufal has quit IRC17:02
shardyI guess I was thinking that by the time we get to _process_new_resource_update we know it needs an update, it's just a question of if we attempt to do it in place or not17:03
*** hdd has quit IRC17:04
*** pratikma_ has joined #heat17:04
zanebthe more logic we move out of the update code, the less bugs we will have in convergence :)17:04
*** sergmelikyan has quit IRC17:06
shardySo you mean raise UpdateReplace in _needs_update if we find the type comparison unacceptable, then _process_new_resource_update will fall through to the create case17:06
zanebyes, exactly17:06
shardyOk, yeah maybe that would be cleaner, thanks, I'll give it a try17:06
* zaneb is trying to figure out where this gets checked in convergence17:07
*** pratikmallya has quit IRC17:07
*** jistr has quit IRC17:08
*** pratikmallya has joined #heat17:11
*** pratikma_ has quit IRC17:11
*** pratikma_ has joined #heat17:11
*** gondoi is now known as zz_gondoi17:12
*** Raj has joined #heat17:12
*** vijayagurug has quit IRC17:12
zanebok, it doesn't get checked at all in convergence :)17:14
*** pratikmallya has quit IRC17:16
*** tspatzier has joined #heat17:16
*** daneyon_ has joined #heat17:16
*** derekh has quit IRC17:18
*** daneyon has quit IRC17:19
zanebshardy: added a more detailed comment on the review17:20
*** manishg has quit IRC17:20
shardyzaneb: thanks17:20
*** manishg has joined #heat17:21
shardyOne side-effect of moving the test is it's in StackResource not TemplateResource (unless I also override _needs_update there17:25
shardymaybe that's OK, but it's a bigger scope than I originally intended17:26
*** e0ne has quit IRC17:27
zanebhmm, yeah, it might be better to override it there too...17:27
* zaneb thinks17:27
shardyYeah I think I'll overrided it only in TemplateResource for now17:28
zanebdoing it in StackResource would certainly make it easier to convert a scaling group to a plain old nested stack17:29
shardyYeah I was wondering about that use-case too, but was planning to check it out in a subsequent pass w/more tests17:30
*** Drago has quit IRC17:34
shardyzaneb: If I move the type comparision to _needs_update, I'm comparing ResourceDefinitions instead of actual resources17:37
shardyis there any way I can derive from the definition that something is a TemplateResource, vs just the resource_type?17:37
zanebah, tricky17:37
zanebI guess you'd have to resolve it from the environment17:37
shardyAh, self._get_resource_info17:38
*** Raj has quit IRC17:40
*** pratikma_ has quit IRC17:41
*** tspatzier has quit IRC17:42
*** tspatzier has joined #heat17:44
*** Raj has joined #heat17:44
*** pt_15 has quit IRC17:48
*** hdd has joined #heat17:52
*** EricGonczer_ has quit IRC17:53
*** Raj has quit IRC17:55
*** hdd_ has joined #heat18:00
*** eliqiao has quit IRC18:00
*** jprovazn is now known as jprovazn_bbl18:00
*** tspatzier has quit IRC18:01
*** hdd has quit IRC18:01
*** jdandrea has joined #heat18:02
*** zz_gondoi is now known as gondoi18:04
*** EricGonczer_ has joined #heat18:04
*** hdd_ has quit IRC18:07
*** kebray has quit IRC18:11
*** crose has quit IRC18:14
*** hdd_ has joined #heat18:14
*** e0ne has joined #heat18:17
*** bana_k has joined #heat18:17
*** pratikmallya has joined #heat18:25
openstackgerritRandall Burt proposed openstack/heat: Always process node addresses  https://review.openstack.org/24171918:26
*** dyasny has quit IRC18:37
*** manishg has quit IRC18:47
*** stannie has quit IRC18:48
*** manishg has joined #heat18:48
*** Marga_ has joined #heat18:52
*** Marga_ has quit IRC18:55
*** Marga_ has joined #heat18:55
*** jprovazn_bbl is now known as jprovazn18:58
pratikmallyarandallburt: what did you mean by " only encrypt default values rather than storing them "?18:59
randallburtpratikmallya:  so instead of storing the original value, just skip the defaults for now given that this changes behavior for hidden vs non-hidden defaults.19:00
pratikmallyarandallburt: so https://review.openstack.org/#/c/217377/19:02
*** jonesbr has quit IRC19:03
randallburtpratikmallya:  I guess, yeah.19:05
mfischdoes this error that I get on heat stack-delete after upgrading to 5.0.0 look familiar to anyone?  "Unknown column 'resource.root_stack_id' in 'field list'"19:05
pratikmallyamfisch: looks like a db migration issue19:06
mfischI agree19:06
mfischbut we were on 5.0.0.0b2 and there's only 1 db migration in between IIRC19:07
*** jonesbr has joined #heat19:07
*** lkarm has joined #heat19:07
mfisch065_root_resource.py is the only new migration19:08
mfischthe 65_ migration is what's supposed to add that field19:14
mfischdb claims to be at 6519:15
mfischbut that column didnt show up19:15
*** crose has joined #heat19:15
*** tspatzier has joined #heat19:20
jasondmfisch: there have been two 065 db migrations19:20
mfischmaybe 5.0.0b2 has a busted one19:21
jasondmfisch: 065_update_at_fsp.py was reverted in 90abc6a2f0b19:21
mfischlooking19:21
jasondmfisch: then, 065_root_resource.py was added19:22
*** tspatzier has quit IRC19:22
*** clayton has joined #heat19:23
mfischactually sorry we were on 5.0.0.0rc2 which == 5.0.019:23
jasondmfisch: it seems like you haven't run the 065_root_resource.py migration script.  you might try "heat-manage db_sync"19:27
jasondmfisch: select version from migrate_version; will show your current db version19:27
mfischconfirmed I have the right one19:27
*** vishwana_ has quit IRC19:27
mfisch[DEV] root@dev01-control-001:/var/log/heat# heat-manage db_version19:27
mfisch6519:27
mfischI dont see a heat-manage log file anywhere, is it logged?19:27
jasondmfisch: only to stdout as far as i know19:29
mfischI'm going to force the db down to 64 and re-run db_sync to see what happens, I can get back to the start to retry stuff later19:29
jasondmfisch: that should work fine, if you did run the old migration https://github.com/openstack/heat/blob/f8f4307f66a7aaaed7bde21b0fdb23df6f3fbd5f/heat/db/sqlalchemy/migrate_repo/versions/065_update_at_fsp.py19:30
mfischjasond: so I poked into my container to confirm but I do have the 65 root resource19:30
mfischokay column showed up now19:30
mfischthe only good news for me is that I've now seen this twice19:31
mfischnow heat delete doesn't blow up, it just returns a stack-list and does nothing at all19:31
jasondmfisch: i haven't seen that one19:32
mfischthis happened in our real dev last time I tried this, so I'm in a virtual environment trying to repro19:32
mfischwe may be stuck on 5.0.0.0b2 forever :(19:33
mfischrc2 I mean19:33
jasondmfisch: git bisect may help19:33
claytonmfisch worst case we can probably locally patch it to fix the migration if we have to19:34
mfischnot sure what to patch yet though19:34
mfischI wish I knew what the migration did the first time, why it thinks its on 065 but its not really19:34
*** Drago has joined #heat19:35
*** Drago has quit IRC19:35
*** Drago has joined #heat19:35
claytonwell, can you can restore the db and run the heat-manage migrations again19:35
mfischclayton: I wonder if our base point is the issue19:35
claytonI don't know about heat-manage, but most of them have a verbose mode that will tell you what migrations are running19:35
mfisch--verbose19:36
mfischthats about all it does19:36
mfischI'll restore and disable puppet19:36
*** kebray has joined #heat19:37
*** randallburt has quit IRC19:44
*** achanda has quit IRC19:49
*** Raj has joined #heat19:49
*** randallburt has joined #heat19:51
*** achanda has joined #heat19:52
*** Raj has quit IRC19:53
jdobthe meeting is in 5 minutes, right?19:55
shardyjdob: yup19:56
jdobkk, just double checking I didnt screw up daylight savings time :)19:57
*** tspatzier has joined #heat20:01
*** jprovazn has quit IRC20:04
*** hdd_ has quit IRC20:06
*** tspatzier has quit IRC20:06
*** neelashah1 has quit IRC20:10
*** khomkrit has quit IRC20:26
*** crose has quit IRC20:29
*** dyasny has joined #heat20:31
*** EricGonc_ has joined #heat20:33
openstackgerritJason Dunsmore proposed openstack/heat-specs: Add spec for updatable parameters blueprint  https://review.openstack.org/24175520:34
*** Raj has joined #heat20:34
jdandreajdob: https://www.youtube.com/watch?v=k4EUTMPuvHo20:35
*** EricGonczer_ has quit IRC20:36
*** jamielennox|away is now known as jamielennox20:36
*** Raj1 has joined #heat20:37
*** Raj has quit IRC20:39
*** dyasny has quit IRC20:48
*** vgridnev has quit IRC20:50
*** Raj1 has quit IRC20:59
*** achanda has quit IRC21:03
*** hdd_ has joined #heat21:04
*** Raj has joined #heat21:05
*** shardy has quit IRC21:06
*** e0ne has quit IRC21:06
*** yassine__ has quit IRC21:07
*** zeih has joined #heat21:14
*** daneyon_ has quit IRC21:15
*** rowagner has quit IRC21:18
*** pratikmallya has quit IRC21:20
*** e0ne has joined #heat21:26
*** vgridnev has joined #heat21:27
*** e0ne has quit IRC21:29
*** jamielennox is now known as jamielennox|away21:30
*** hdd_ has quit IRC21:31
*** jaime491 has joined #heat21:34
*** jaime49 has quit IRC21:34
*** jaime491 has quit IRC21:35
*** jaime49 has joined #heat21:35
*** jamielennox|away is now known as jamielennox21:37
*** zeih has quit IRC21:39
*** e0ne has joined #heat21:40
*** e0ne has quit IRC21:41
*** jtomasek has quit IRC21:41
*** pratikmallya has joined #heat21:46
*** Piet has quit IRC21:47
*** Raj has left #heat21:50
*** achanda has joined #heat22:00
*** jaime49 has quit IRC22:01
*** e0ne has joined #heat22:07
*** e0ne has quit IRC22:08
*** pratikmallya has quit IRC22:13
*** huangtianhua has quit IRC22:15
*** lkarm has quit IRC22:17
*** sdake has joined #heat22:17
*** lkarm has joined #heat22:18
*** lkarm has quit IRC22:22
openstackgerritBryan Jones proposed openstack/heat: LBaaS v2 Pool  https://review.openstack.org/24004922:24
openstackgerritBryan Jones proposed openstack/heat: LBaaS v2 Loadbalancer  https://review.openstack.org/23861222:24
openstackgerritBryan Jones proposed openstack/heat: LBaaS v2 custom constraints  https://review.openstack.org/23270922:24
openstackgerritBryan Jones proposed openstack/heat: LBaaS v2 Listener  https://review.openstack.org/23975522:24
*** jonesbr has left #heat22:25
*** yassine__ has joined #heat22:25
*** Raj has joined #heat22:26
*** achanda has quit IRC22:26
*** achanda has joined #heat22:30
*** sdake_ has joined #heat22:30
*** sdake has quit IRC22:31
openstackgerritJason Dunsmore proposed openstack/python-heatclient: Rename "stack-update --dry-run" to "stack-update-preview"  https://review.openstack.org/24179322:34
*** dspano has quit IRC22:37
*** vgridnev has quit IRC22:41
*** lkarm has joined #heat22:47
*** EricGonc_ has quit IRC22:47
*** pratikmallya has joined #heat22:49
*** pratikma_ has joined #heat22:50
*** lkarm has quit IRC22:51
*** rpothier has quit IRC22:52
*** jasond has quit IRC22:53
*** Raj has quit IRC22:53
*** pratikmallya has quit IRC22:55
*** Raj has joined #heat22:57
*** dprince has quit IRC22:58
*** randallburt has quit IRC23:04
*** vijendar has quit IRC23:05
*** manishg has quit IRC23:05
*** yassine__ has quit IRC23:06
*** manishg has joined #heat23:07
*** gondoi is now known as zz_gondoi23:10
*** manishg has quit IRC23:11
*** Raj has quit IRC23:15
*** manishg has joined #heat23:17
*** asalkeld has quit IRC23:26
*** asalkeld has joined #heat23:29
*** spzala has quit IRC23:33
*** spzala has joined #heat23:33
*** signed8bit is now known as signed8bit_ZZZzz23:35
*** signed8bit_ZZZzz is now known as signed8bit23:39
*** manishg has quit IRC23:51

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