Wednesday, 2016-03-23

zanebthe inputs schema specifically00:00
zanebbut really any of them00:00
zanebhttp://git.openstack.org/cgit/openstack/heat/tree/heat/api/openstack/v1/software_configs.py#n86 <- API passes them straight through00:00
zanebhttp://git.openstack.org/cgit/openstack/heat/tree/heat/engine/service_software_config.py#n64 <- engine appears to do the same00:01
zanebare they validated at some point that they match the schema defined in http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/resources/openstack/heat/software_config.py#n65 ?00:02
*** trown|outtypewww is now known as trown00:03
stevebakerzaneb: you're right, at the API level there is none, and this was less important when only the config resource consumed it00:03
zanebthat makes sense00:03
zanebshould we have some now?00:03
*** dims_ has joined #heat00:03
stevebakerthat seems prudent00:03
zaneback00:04
stevebakeralso there is a bug to parse the outputs based on the type of the output schema, also a good idea00:04
*** dims has quit IRC00:04
zanebwould that have backwards compat issues?00:05
stevebakeryeah, potentially. strings becoming dicts would be surprising00:06
stevebakermaybe a new coerce-type property is needed00:07
stevebakeralthough I00:07
stevebaker'd be surprised if anyone is specifying an outputs schema, let alone specifying types on them00:08
*** wangbo has quit IRC00:08
*** Marga_ has quit IRC00:14
*** Marga_ has joined #heat00:15
*** achanda has quit IRC00:18
*** Qiming has quit IRC00:19
*** achanda has joined #heat00:21
*** sridhar_ram has joined #heat00:28
*** achanda has quit IRC00:40
*** krotscheck has joined #heat00:41
*** sdake has quit IRC00:50
*** sdake has joined #heat00:51
*** ricolin has joined #heat00:55
*** daneyon has quit IRC01:00
*** sridhar_ram has quit IRC01:03
*** achanda has joined #heat01:11
*** Qiming has joined #heat01:13
*** EricGonczer_ has quit IRC01:17
jamespdshould heat be able to tear down a stack if some resources are missing?01:26
*** daneyon has joined #heat01:29
*** Marga_ has quit IRC01:30
*** MVenesio has joined #heat01:34
*** daneyon has quit IRC01:35
*** daneyon has joined #heat01:36
*** zz_dimtruck is now known as dimtruck01:38
*** MVenesio has quit IRC01:39
*** dims_ has quit IRC01:40
*** yanyanhu has joined #heat01:40
*** sabeen has joined #heat01:43
*** EricGonczer_ has joined #heat01:47
*** blomquisg is now known as blomquisg_gone01:53
*** elynn has joined #heat01:56
*** elynn has quit IRC02:01
*** elynn has joined #heat02:01
elynnMorning02:02
*** bana_k has quit IRC02:10
*** EricGonczer_ has quit IRC02:21
*** Raj1 has joined #heat02:21
*** neelashah has joined #heat02:22
stevebakerjamespd: yes02:24
jamespdstevebaker: perhaps you can give me advice on why I get heat-engine stacktraces? http://paste.openstack.org/show/491520/02:27
jamespdand 'heat resource-list STACK-ID' reports:02:28
jamespdERROR: Remote error: NetworkNotFoundClient Network a47820d8-ab42-45a4-801c-90aa65e7da60 could not be found02:28
stevebakerjamespd: ah yes, the neutron dependencies code. I believe there has been fixes for that recently02:29
jamespdso, do I have options in kilo without stack-abandon?02:29
*** dimtruck is now known as zz_dimtruck02:31
stevebakerhere we go02:33
stevebakerhttps://bugs.launchpad.net/heat/+bug/155462502:33
openstackLaunchpad bug 1554625 in heat "Exceptions when adding implicit dependencies cause stacks to be unloadable" [Medium,Fix released] - Assigned to Zane Bitter (zaneb)02:33
stevebakerhttps://review.openstack.org/#/c/290027/ needs backporting to kilo and liberty02:34
stevebakerjamespd: until that is backported, manually deleting the remaining resources and abandoning the stack may be your only option02:37
* jamespd reads.02:39
jamespdrad; thanks for the help stevebaker.02:41
stevebakerjamespd: np02:42
*** sabeen has quit IRC02:43
openstackgerritReedip proposed openstack/heat-templates: Remove Py33 support  https://review.openstack.org/26611502:44
*** neelashah has quit IRC02:45
*** sabeen has joined #heat02:47
*** Raj1 has quit IRC02:50
*** wangbo has joined #heat02:53
*** yanyanhu has quit IRC02:55
*** sabeen has quit IRC03:13
*** Qiming has quit IRC03:15
*** yuanying has quit IRC03:21
*** yuanying_ has joined #heat03:21
*** yuanying_ has quit IRC03:24
*** yuanying has joined #heat03:24
*** Marga_ has joined #heat03:26
*** links has joined #heat03:28
*** yuanying has quit IRC03:29
*** Marga_ has quit IRC03:30
*** yanyanhu has joined #heat03:30
*** elynn has quit IRC03:35
*** MVenesio has joined #heat03:35
*** wangbo has quit IRC03:39
*** Marga_ has joined #heat03:39
*** MVenesio has quit IRC03:39
*** Marga_ has quit IRC03:40
*** Marga_ has joined #heat03:40
*** elynn has joined #heat03:58
*** akuznetsov has joined #heat04:01
*** wangbo has joined #heat04:01
*** yuanying has joined #heat04:01
*** elynn_ has joined #heat04:01
*** gberginc has joined #heat04:02
*** elynn has quit IRC04:04
*** akuznetsov has quit IRC04:05
*** achanda has quit IRC04:06
*** KanagarajM has joined #heat04:34
*** achanda has joined #heat04:53
*** elynn_ has quit IRC05:07
*** elynn has joined #heat05:09
*** elynn has quit IRC05:13
*** elynn has joined #heat05:14
*** sridhar_ram has joined #heat05:16
*** Qiming has joined #heat05:18
*** MVenesio has joined #heat05:36
*** MVenesio has quit IRC05:40
*** Qiming_ has joined #heat06:00
*** Qiming has quit IRC06:02
*** ricolin has quit IRC06:03
*** achanda_ has joined #heat06:13
*** wangbo has quit IRC06:13
*** achanda has quit IRC06:14
*** bana_k has joined #heat06:16
*** wangbo has joined #heat06:25
*** akuznetsov has joined #heat06:30
*** sdake_ has joined #heat06:35
*** sdake has quit IRC06:36
*** akuznetsov has quit IRC06:40
*** akuznetsov has joined #heat06:41
*** sdake_ has quit IRC06:44
openstackgerritRabi Mishra proposed openstack/heat: Use region from config in functional tests  https://review.openstack.org/29620606:44
*** sdake has joined #heat06:44
*** sridhar_ram has quit IRC06:46
openstackgerritKanagaraj Manickam proposed openstack/heat-specs: heat template-migrate  https://review.openstack.org/22842406:46
openstackgerritCrag Wolfe proposed openstack/heat: WIP - Move resource.properties_data into separate table  https://review.openstack.org/26795306:48
*** ricolin has joined #heat06:49
*** vgridnev has joined #heat06:53
*** akuznetsov has quit IRC06:53
*** e0ne has joined #heat06:58
*** akuznetsov has joined #heat07:06
openstackgerritRakesh H S proposed openstack/heat: Makes monasca plugin as supported  https://review.openstack.org/25286407:06
*** akuznetsov has quit IRC07:07
*** akuznetsov has joined #heat07:12
*** vishwanathj is now known as vishwanathj_away07:12
*** wangbo has quit IRC07:12
openstackgerritRakesh H S proposed openstack/heat: Makes monasca plugin as supported  https://review.openstack.org/25286407:13
*** ricolin has quit IRC07:16
openstackgerritKanagaraj Manickam proposed openstack/heat-specs: heat template-migrate  https://review.openstack.org/22842407:16
*** aaronmehar has joined #heat07:16
*** sridhar_ram has joined #heat07:19
*** sridhar_ram has quit IRC07:19
*** bana_k has quit IRC07:20
*** akuznetsov has quit IRC07:20
*** rakesh_hs has quit IRC07:23
*** sdake has quit IRC07:23
*** achanda_ has quit IRC07:26
*** e0ne has quit IRC07:28
openstackgerritRabi Mishra proposed openstack/heat: Use region from config in functional tests  https://review.openstack.org/29620607:31
*** ricolin has joined #heat07:32
*** achanda has joined #heat07:35
*** wangbo has joined #heat07:36
*** MVenesio has joined #heat07:37
*** kebray has quit IRC07:38
*** mr has joined #heat07:39
*** kebray has joined #heat07:41
*** MVenesio has quit IRC07:41
*** wangbo has quit IRC07:44
*** ccamacho has joined #heat07:48
*** cwolferh has quit IRC07:49
*** akuznetsov has joined #heat07:49
openstackgerritSergey Kraynev proposed openstack/heat: Fix race condition for WaitCondition with several signals  https://review.openstack.org/23212407:49
openstackgerritSergey Kraynev proposed openstack/heat: Handling concurrent transactions in metadata_set method  https://review.openstack.org/29226807:49
*** vgridnev has quit IRC07:49
*** akuznetsov has quit IRC07:51
*** akuznetsov has joined #heat07:52
*** pt_15 has quit IRC07:55
*** zeih has joined #heat07:55
*** akuznetsov has quit IRC07:56
*** vgridnev has joined #heat07:58
*** cwolferh has joined #heat08:00
openstackgerritMerged openstack/heat: Added constraint for barbican order  https://review.openstack.org/28942408:01
*** wangbo has joined #heat08:01
*** kebray has quit IRC08:04
*** achanda has quit IRC08:06
*** jaosorior has joined #heat08:06
*** achanda has joined #heat08:08
*** kebray has joined #heat08:10
*** achanda has quit IRC08:11
*** achanda has joined #heat08:12
*** jprovazn has joined #heat08:12
*** achanda has quit IRC08:12
*** achanda has joined #heat08:13
*** achanda has quit IRC08:13
*** vgridnev has quit IRC08:15
*** yanyanhu has quit IRC08:21
*** yanyanhu has joined #heat08:21
*** palexster has joined #heat08:27
*** xek has quit IRC08:32
*** xek has joined #heat08:33
openstackgerritAbhishek Kekane proposed openstack/heat: Restrict use of iteritems, iterkeys and itervalues  https://review.openstack.org/26905108:33
*** gberginc has quit IRC08:34
*** tmatsu has quit IRC08:36
*** palexster has quit IRC08:36
*** ricolin has quit IRC08:37
*** shardy has joined #heat08:37
*** DaveJ__ has joined #heat08:38
*** tmatsu has joined #heat08:39
*** permalac has joined #heat08:43
*** tmatsu has quit IRC08:44
*** ricolin has joined #heat08:50
*** akuznetsov has joined #heat08:53
*** akuznetsov has quit IRC08:55
*** gberginc has joined #heat08:56
therveskraynev, Using private methods from other projects is a no-go: https://review.openstack.org/28942408:56
*** ramishra has quit IRC08:57
therveI'm also pretty sure that it's wrong in this case08:57
*** russellb has quit IRC08:58
*** ramishra has joined #heat08:59
*** palexster has joined #heat08:59
openstackgerrithuangtianhua proposed openstack/heat: Add a new HOT template version for Newton  https://review.openstack.org/29627508:59
*** russellb has joined #heat09:00
*** links has quit IRC09:01
*** e0ne has joined #heat09:03
*** openstackgerrit has quit IRC09:03
*** yassine has joined #heat09:04
*** ramishra has quit IRC09:04
*** openstackgerrit has joined #heat09:04
*** yassine is now known as Guest9254209:04
*** Guest92542 is now known as yassou09:05
*** yanyanhu has quit IRC09:05
openstackgerritRabi Mishra proposed openstack/heat: Use domain env variables as defaults  https://review.openstack.org/29537309:06
*** vgridnev has joined #heat09:09
*** ramishra has joined #heat09:09
openstackgerrithuangtianhua proposed openstack/heat: Provides 'equals' intrinsic function  https://review.openstack.org/28336509:10
*** zeih_ has joined #heat09:13
*** zeih has quit IRC09:13
*** achanda has joined #heat09:14
*** sorantis has joined #heat09:15
*** links has joined #heat09:18
openstackgerritDmitriy Uvarenkov proposed openstack/heat: Added constraint for mistral workflow  https://review.openstack.org/29283809:19
*** KanagarajM has quit IRC09:19
*** derekh has joined #heat09:19
*** achanda has quit IRC09:20
openstackgerrithuangtianhua proposed openstack/heat: Support resource/output condition  https://review.openstack.org/28336609:22
*** ricolin has quit IRC09:28
openstackgerrithuangtianhua proposed openstack/heat: Provides 'if' intrinsic function  https://review.openstack.org/28347909:36
openstackgerrithuangtianhua proposed openstack/heat: Support resource/output condition  https://review.openstack.org/28336609:36
openstackgerrithuangtianhua proposed openstack/heat: Provides 'equals' intrinsic function  https://review.openstack.org/28336509:36
*** tlashchova has joined #heat09:36
*** KanagarajM has joined #heat09:37
*** MVenesio has joined #heat09:37
openstackgerritThomas Herve proposed openstack/heat: Tidy up BarbicanClientPlugin  https://review.openstack.org/29634009:39
*** ricolin has joined #heat09:41
skraynevtherve: ok. agree. thx for the spotting09:42
*** MVenesio has quit IRC09:42
*** KanagarajM has quit IRC09:48
*** dims has joined #heat10:00
*** tochi has quit IRC10:02
*** links has quit IRC10:05
openstackgerritAbhishek Kekane proposed openstack/heat: Restrict use of iteritems, iterkeys and itervalues  https://review.openstack.org/26905110:05
*** Qiming_ has quit IRC10:15
*** ricolin has quit IRC10:15
*** vgridnev has quit IRC10:19
*** vgridnev has joined #heat10:20
*** links has joined #heat10:22
*** gmoro has quit IRC10:32
*** jaosorior has quit IRC10:34
*** e0ne has quit IRC10:35
*** jaosorior has joined #heat10:36
*** elynn has quit IRC10:39
*** elynn has joined #heat10:40
*** elynn has quit IRC10:40
*** e0ne has joined #heat10:45
*** zeih_ has quit IRC10:58
*** tlashchova has quit IRC11:01
*** dims_ has joined #heat11:05
*** dims has quit IRC11:06
*** Qiming has joined #heat11:07
*** tlashchova has joined #heat11:08
*** achanda has joined #heat11:19
*** achanda has quit IRC11:24
*** sorantis has quit IRC11:28
*** EricGonczer_ has joined #heat11:28
*** sorantis has joined #heat11:30
*** MVenesio has joined #heat11:38
*** MVenesio has quit IRC11:43
openstackgerritMerged openstack/heat: Added constraint for mistral workflow  https://review.openstack.org/29283811:43
*** gberginc has quit IRC11:51
*** gberginc has joined #heat11:52
*** gberginc has quit IRC11:54
*** blues-man has joined #heat11:55
*** zeih has joined #heat11:57
*** david-lyle_ has joined #heat11:58
*** rakesh_hs has joined #heat12:00
*** david-lyle has quit IRC12:00
*** wangbo has quit IRC12:01
*** wangbo has joined #heat12:02
*** MVenesio has joined #heat12:02
*** blomquisg_gone is now known as blomquisg12:06
*** dprince has joined #heat12:07
openstackgerritMerged openstack/heat: Restrict use of iteritems, iterkeys and itervalues  https://review.openstack.org/26905112:07
*** jcoufal has joined #heat12:11
*** rpothier has joined #heat12:13
*** gberginc has joined #heat12:39
openstackgerritThomas Herve proposed openstack/heat: Tidy up BarbicanClientPlugin  https://review.openstack.org/29634012:39
*** EricGonczer_ has quit IRC12:57
*** zeih has quit IRC12:58
*** wangbo has quit IRC12:59
*** tmatsu has joined #heat13:01
*** links has quit IRC13:01
openstackgerritRabi Mishra proposed openstack/heat: Use region from config in functional tests  https://review.openstack.org/29620613:05
*** david-lyle_ is now known as david-lyle13:06
*** tmatsu has quit IRC13:06
*** jonesbr has joined #heat13:08
*** neelashah has joined #heat13:11
*** zeih has joined #heat13:19
openstackgerritOleksii Chuprykov proposed openstack/heat: Add validation for mistral cron trigger  https://review.openstack.org/29304713:21
*** MVenesio has quit IRC13:21
*** achanda has joined #heat13:22
*** sabeen1 has joined #heat13:26
*** achanda has quit IRC13:27
*** trown is now known as trown|afk13:28
openstackgerritPeter Razumovsky proposed openstack/heat: EncryptedVolumeType get_live_state implementation  https://review.openstack.org/27673013:29
*** vgridnev has quit IRC13:37
*** jdob has quit IRC13:41
*** jdob has joined #heat13:41
*** vgridnev has joined #heat13:48
openstackgerritRabi Mishra proposed openstack/heat: Use domain env variables as defaults  https://review.openstack.org/29537313:48
*** jreeves has joined #heat13:48
*** trown|afk is now known as trown13:51
*** jdob has quit IRC13:51
*** jdob has joined #heat13:51
openstackgerritThomas Herve proposed openstack/heat: Tidy up BarbicanClientPlugin  https://review.openstack.org/29634013:54
*** prazumovsky has joined #heat13:55
*** mspreitz has joined #heat13:58
*** neelashah1 has joined #heat13:59
*** neelashah has quit IRC14:01
*** pradk has joined #heat14:01
*** sorantis_ has joined #heat14:02
openstackgerritCrag Wolfe proposed openstack/heat: Move properties_data into a separate table  https://review.openstack.org/26795314:02
*** dims has joined #heat14:04
*** sorantis has quit IRC14:04
*** sorantis_ is now known as sorantis14:04
*** dims_ has quit IRC14:04
*** achanda has joined #heat14:05
*** spotz_zzz is now known as spotz14:06
*** neelashah has joined #heat14:06
*** achanda has quit IRC14:07
*** neelashah1 has quit IRC14:07
*** vijendar has joined #heat14:08
*** openstack has joined #heat14:25
openstackgerritPeter Razumovsky proposed openstack/heat: EncryptedVolumeType get_live_state implementation  https://review.openstack.org/27673014:25
openstackgerritHongbin Lu proposed openstack/heat: Remove property "ssh_authorized_key" from OS::Magnum::BayModel  https://review.openstack.org/29559814:26
*** sdake has joined #heat14:27
*** dprince has quit IRC14:30
*** crose has joined #heat14:32
*** EricGonczer_ has quit IRC14:41
*** kebray has quit IRC14:45
*** david-lyle_ is now known as david-lyle14:47
*** sdake_ has joined #heat14:48
*** sdake has quit IRC14:49
*** zeih has quit IRC14:50
*** sdake has joined #heat14:52
*** sdake_ has quit IRC14:53
*** prazumovsky has quit IRC14:58
*** trown|meeting is now known as trown15:00
*** EricGonczer_ has joined #heat15:09
*** openstackstatus has joined #heat15:11
*** ChanServ sets mode: +v openstackstatus15:11
*** wangbo has joined #heat15:15
*** Drago1 has joined #heat15:18
*** Drago1 has quit IRC15:18
*** Drago1 has joined #heat15:18
*** mr has left #heat15:20
*** EricGonczer_ has quit IRC15:24
*** EricGonczer_ has joined #heat15:24
*** kebray has joined #heat15:25
*** wangbo has quit IRC15:25
*** jaosorior has quit IRC15:27
*** crose has quit IRC15:27
*** jprovazn has quit IRC15:28
*** Qiming has quit IRC15:31
*** gberginc has quit IRC15:36
*** jaosorior has joined #heat15:44
*** tmatsu has joined #heat15:44
*** zeih has joined #heat15:47
*** tmatsu has quit IRC15:48
openstackgerritDmitriy Uvarenkov proposed openstack/heat: Add validation check for barbican order  https://review.openstack.org/29655415:49
*** jdob has quit IRC15:49
*** sorantis has quit IRC15:52
openstackgerritDmitriy Uvarenkov proposed openstack/heat: Add validation check for barbican order  https://review.openstack.org/29655415:52
*** rpothier has quit IRC15:53
*** mspreitz has quit IRC15:56
*** sridhar_ram1 has joined #heat15:57
*** gberginc has joined #heat16:00
*** tmatsu has joined #heat16:00
*** vgridnev has quit IRC16:00
*** daneyon has quit IRC16:00
*** bana_k has joined #heat16:03
*** tmatsu has quit IRC16:04
*** links has joined #heat16:06
*** aaronmehar has quit IRC16:09
*** zeih has quit IRC16:10
*** zeih has joined #heat16:13
*** kebray has quit IRC16:15
*** aaronmehar has joined #heat16:16
*** kebray has joined #heat16:18
*** trown is now known as trown|lunch16:19
*** LimorStotland has quit IRC16:23
*** vijendar has quit IRC16:23
*** LimorStotland has joined #heat16:23
*** crose has joined #heat16:31
*** david-lyle has quit IRC16:33
*** wangbo has joined #heat16:33
*** david-lyle has joined #heat16:33
*** kebray has quit IRC16:36
*** zz_dimtruck is now known as dimtruck16:37
*** zeih has quit IRC16:37
*** vishwanathj_away is now known as vishwanathj16:39
*** aaronmehar has quit IRC16:41
*** jprovazn has joined #heat16:45
*** jdob_lt has joined #heat16:49
*** links has quit IRC16:49
*** yassou has quit IRC16:50
*** wangbo has quit IRC16:53
*** vgridnev has joined #heat16:56
*** EricGonczer_ has quit IRC16:58
*** achanda has joined #heat16:58
openstackgerritOleksii Chuprykov proposed openstack/heat: wip: yaql function  https://review.openstack.org/19698417:02
*** tmatsu has joined #heat17:02
*** achanda has quit IRC17:03
*** rpothier has joined #heat17:04
*** dims_ has quit IRC17:05
*** zeih has joined #heat17:06
*** tmatsu has quit IRC17:06
*** dims has joined #heat17:06
*** vijendar has joined #heat17:08
*** zeih has quit IRC17:10
*** zeih has joined #heat17:10
*** achanda has joined #heat17:14
shardyHey all, I started a ML thread looking for help with bug #155980717:15
openstackbug 1559807 in heat "Some intrinsic functions fail when referencing get_attr" [Medium,Confirmed] https://launchpad.net/bugs/1559807 - Assigned to Steven Hardy (shardy)17:15
shardyUnfortunately preview updates are also completely broken for the same reason17:15
*** gberginc has quit IRC17:18
*** vijendar has quit IRC17:22
*** palexster has quit IRC17:24
*** derekh has quit IRC17:25
*** rcernin has quit IRC17:29
*** achanda has quit IRC17:29
*** _erick0zcr has quit IRC17:30
*** achanda has joined #heat17:33
*** tmatsu has joined #heat17:33
*** trown|lunch is now known as trown17:36
*** tmatsu has quit IRC17:38
*** aaronmehar has joined #heat17:40
*** achanda has quit IRC17:41
*** blomquisg is now known as blomquisg_gone17:43
*** aaronmehar has quit IRC17:51
*** jdob_lt has quit IRC17:51
*** achanda has joined #heat17:51
*** jonesbr has left #heat17:52
*** daneyon has joined #heat17:54
*** jdob_lt has joined #heat17:58
*** zeih has quit IRC18:00
*** zeih has joined #heat18:01
*** vijendar has joined #heat18:04
*** yuanying_ has joined #heat18:07
*** vgridnev has quit IRC18:08
*** achanda has quit IRC18:09
*** yuanying has quit IRC18:09
*** bana_k has quit IRC18:09
*** jdob_lt has quit IRC18:10
*** jonesbr has joined #heat18:11
*** EricGonczer_ has joined #heat18:13
*** jaosorior has quit IRC18:13
Drago1shardy: I feel like a schema for attributes may help with validation. It wouldn't completely solve it, but validation could be done with more confidence if get_attr knew the type the attribute was supposed to be and returned a placeholder like STRING_ATTRIBUTE. When resolving nested intrinsic functions, it may be easier for them to validate against that placeholder if the placeholder's implementation allowed for certain checks. For example both a real string and18:14
*** krotscheck has quit IRC18:14
*** krotscheck has joined #heat18:15
Drago1whereas that check will fail because value_passed_to_intrinsic is None in the case of get_attr nested inside of another intrinsic18:15
Drago1*fail currently18:15
*** bana_k has joined #heat18:20
shardyDrago1: Yeah, I did think that too, but I think there are too many corner-cases where the dummy value will break just like None does18:20
shardylike nested get_attr's etc where one expects a list of non-zero length so path based indexes work etc18:20
Drago1shardy: I figure you're right18:20
*** vgridnev has joined #heat18:20
shardyI think the validation part can be solved by just not validating values during validation18:21
shardywe already do that for all nested stacks by setting strict_validate to False18:21
shardybut we inconsistently left that as True for parent stacks18:21
shardythe downside of that is our validation becomes somewhat less strict in some cases, so you might fail slower if e.g a wrong parameter type is mapped to a resource via get_param18:22
shardyThat may be an OK compromise tho18:22
shardythe preview case is harder18:22
shardyI'm not yet sure how we solve that at all18:23
shardyDrago1: thanks for the ideas tho :)18:23
Drago1It's probably too late now, but it kind of feels like HOT's declarative nature conflicts with being able to do things like nested get_attr, because the value that it would eventually resolve to can honestly be anything due to the logic inside of resources18:24
*** aaronmehar has joined #heat18:24
*** zeih has quit IRC18:24
shardyDrago1: I'm not sure that's true - we always intended to split structural/syntax validation from runtime value validation18:25
*** jdob_lt has joined #heat18:25
shardyonly, we didn't get it right, evidently ;)18:25
*** krotscheck has quit IRC18:26
Drago1shardy: except runtime values have the ability to affect structure18:26
*** krotscheck has joined #heat18:26
shardyDrago1: They don't (although I was aiming for something closer to that via my proposed abuse of Repeat in bug #1546684)18:28
openstackbug 1546684 in heat "repeat can't reference attributes/parameters" [Medium,In progress] https://launchpad.net/bugs/1546684 - Assigned to Steven Hardy (shardy)18:28
shardyDrago1: we parse the template into distinct objects, including one for every function regardless of the nesting18:28
*** blues-man has quit IRC18:28
shardyBut, I guess I see what you mean, we've conflated validating the schema those objects can parse with the values they generate18:29
shardywhich is my main problem ;)18:29
shardyDrago1: Many intrinsic functions can validate just fine when they see a Function.Function (unresolved), the problem starts when you try to validate the (resolved) value before anything is actually created18:30
Drago1shardy: That makes sense, and like you said, that's structure vs runtime values18:33
shardyhttps://github.com/openstack/heat/blob/master/heat/engine/resource.py#L133318:34
shardyI think we can fix that part just by saying False there18:34
shardybecause with_value=True should probably always be reserved for runtime validation18:34
shardybut that still doesn't resolve how we handle the property comparison for update previews18:34
*** vgridnev has quit IRC18:35
*** tmatsu has joined #heat18:36
Drago1shardy: use_existing? :)18:37
*** sridhar_ram1 is now known as sridhar_ram18:37
*** krotscheck has quit IRC18:38
*** aaronmehar has quit IRC18:38
Drago1shardy: I guess that may not be quite right because I really mean use existing heat resources, not physical ones, but the idea is that for resources that you can already tell have no diff in the template, just the ones that are in the existing stack18:39
*** krotscheck has joined #heat18:39
*** permalac has quit IRC18:39
shardyYeah the problem is the temporary "new" stack we make in ram to compare with the existing one18:39
shardythe resources in that don't actually exist18:39
* shardy wonders off to ponder further over dinner18:41
*** tmatsu has quit IRC18:41
*** sdake_ has joined #heat18:41
*** vgridnev has joined #heat18:42
*** sdake has quit IRC18:44
*** sdake has joined #heat18:47
openstackgerritMerged openstack/heat: Use region from config in functional tests  https://review.openstack.org/29620618:47
*** sdake_ has quit IRC18:49
*** vgridnev has quit IRC18:50
*** eraineri has joined #heat18:50
*** openstack has joined #heat19:24
*** therve has joined #heat19:24
*** tmatsu has quit IRC19:27
*** achanda has joined #heat19:35
*** tmatsu has joined #heat19:38
*** tmatsu has quit IRC19:43
*** dims_ has joined #heat19:43
*** daneyon has quit IRC19:44
*** skraynev_ has joined #heat19:44
*** daneyon has joined #heat19:45
*** dims has quit IRC19:45
*** vijendar1 has quit IRC19:47
*** daneyon_ has joined #heat19:47
*** aaronmehar has quit IRC19:48
*** zeih_ has joined #heat19:49
*** jprovazn has quit IRC19:50
*** daneyon has quit IRC19:50
*** aaronmehar has joined #heat19:51
*** dims_ has quit IRC19:53
*** zeih_ has quit IRC19:54
*** dims has joined #heat19:56
*** vijendar has joined #heat19:57
skraynev_good evening19:57
skraynev_meeting time20:00
*** aaronmehar has quit IRC20:05
*** zeih_ has joined #heat20:07
*** shardy_ has joined #heat20:07
*** shardy has quit IRC20:08
*** EricGonczer_ has quit IRC20:10
*** aaronmehar has joined #heat20:11
*** e0ne has joined #heat20:17
*** ryansb has quit IRC20:17
*** achanda has quit IRC20:18
*** aaronmehar has quit IRC20:18
*** openstack has joined #heat20:34
shardy_zaneb: Hey thanks for the ML response20:34
zanebshardy_: np, I'm looking at your RG thing now too20:35
shardy_I think we can probably work around the validation part, but I'm still struggling with how we can ever make preview results accurate for update20:35
shardy_because we have to compare property values to derive if a resource will be replaced20:35
shardy_(or even updated)20:36
*** e0ne has quit IRC20:37
zanebso I don't think the current approach will work for that20:37
*** tmatsu has joined #heat20:37
zanebwhat you'd need to do is take each resource in the context of the stack (rather than looking at all of the resources outside of the context of a stack)20:37
jreevesskraynev: isn't it like 1:30 am for you?20:38
zaneband for each one say: this is replaced, this is updated and may be replaced, this is unchanged20:38
zanebor this *may* be updated and/or replaced20:39
zaneband pass the results down through the graph as you go20:39
zanebso the results will get progressively more uncertain as you travel down20:39
shardy_zaneb: Yeah, that's what I was thinking, even if you copy the current stack and substitute resources, basically you can't accurately derive anything below a replacement (or even update)20:40
openstackgerritMark Vanderwiel proposed openstack/heat: Add lbaas v2 scenario test  https://review.openstack.org/23760820:41
zanebshardy_: yeah. but there's some stuff you can say20:41
*** tmatsu has quit IRC20:41
zanebyou can quite often say when a resource will be replaced for sure20:41
zaneband you can sometimes say when a resource will be unchanged for sure20:42
shardy_Yeah, but if you say a resource will be updated, you have no way to ever know what will happen to anything it depends on20:43
shardy_So I'm not sure what value the preview really provides20:43
shardy_I suppose we can fix it up somewhat so it's a rough guide of what might happen20:43
*** sabeen has joined #heat20:44
shardy_With the benefit of hindsight I guess it's pretty obvious, but I missed this when working on enabling the update preview stuff20:45
zanebshardy: I think my review of patch set 3 on https://review.openstack.org/#/c/122473/ actually says it all :D20:45
shardy_Yeah, I made the mistake of assuming what we had actually worked, and just made it recurse20:46
shardy_Anyway, dropping off now but thanks for the feedback, will give it some more thought :)20:47
zanebright after that review was when I stopped having time to do reviews :(20:48
jamespdCan someone explain what 'critical data, returned only after commencing a destructive delete..' means, in the context of stack-abandon?20:49
jamespdi.e. what is the critical data?20:49
shardy_jamespd: the entire definition of the thing you're about to delete20:50
shardy_all the resource IDs, stored data associated with resources etc20:50
*** ccamacho has quit IRC20:50
jamespdright; thanks.20:51
shardy_the issue is if you for any reason fail to get that data, you've orphaned a bunch of resources with no way to recover20:52
jamespdcool.  I didn't realize that people expected to be able to recover after an abandon.20:52
*** EricGonczer_ has joined #heat20:52
shardy_jamespd: that was kind of the point, you can abandon, keep the data, then do a stack-adopt to reinstate the resources under the control of heat20:53
shardy_only in practice there were a bunch of bugs which didn't all get fixed20:53
* jamespd nods.20:53
*** jreeves has quit IRC20:55
*** shardy_ has quit IRC20:55
*** david-lyle has quit IRC20:59
*** aaronmehar has joined #heat20:59
*** achanda has joined #heat21:01
*** aaronmeh_ has joined #heat21:01
*** prazumovsky has joined #heat21:01
*** prazumovsky has quit IRC21:01
openstackgerritJason Dunsmore proposed openstack/heat: Don't create an "update task" inside a try block  https://review.openstack.org/29679821:02
*** aaronmehar has quit IRC21:04
*** david-lyle has joined #heat21:05
openstackgerritMerged openstack/heat: Handling concurrent transactions in metadata_set method  https://review.openstack.org/29226821:06
openstackgerritMerged openstack/heat: Fix race condition for WaitCondition with several signals  https://review.openstack.org/23212421:08
*** vijendar has quit IRC21:11
*** neelashah1 has joined #heat21:15
*** EricGonczer_ has quit IRC21:16
*** neelashah has quit IRC21:16
*** EricGonczer_ has joined #heat21:16
*** trown is now known as trown|outtypewww21:17
*** david-lyle has quit IRC21:21
*** david-lyle has joined #heat21:23
*** vijendar has joined #heat21:27
*** sdake has quit IRC21:27
*** charlesrose has quit IRC21:34
*** jonesbr has left #heat21:40
*** achanda has quit IRC21:46
*** zeih_ has quit IRC21:52
*** sdake has joined #heat21:55
*** zeih has joined #heat21:59
*** EricGonczer_ has quit IRC22:01
*** jasond has quit IRC22:05
*** achanda has joined #heat22:16
*** jcoufal has quit IRC22:18
*** zeih has quit IRC22:18
*** dimtruck is now known as zz_dimtruck22:23
*** sabeen has quit IRC22:25
*** sdake_ has joined #heat22:29
*** sdake has quit IRC22:30
*** spotz is now known as spotz_zzz22:32
*** aaronmeh_ has quit IRC22:34
*** vijendar has quit IRC22:34
*** sabeen has joined #heat22:43
*** tristanC has quit IRC22:43
*** tristanC has joined #heat22:44
*** Qiming has joined #heat22:44
*** pradk has quit IRC22:45
*** Qiming has quit IRC22:58
*** sdake_ is now known as sdake22:59
*** neelashah1 has quit IRC22:59
openstackgerritThomas Herve proposed openstack/heat: Replace SD RPC polling by long RPC call  https://review.openstack.org/28122023:00
*** Drago1 has quit IRC23:01
*** DaveJ__ has quit IRC23:09
*** zeih has joined #heat23:12
*** zeih has quit IRC23:18
*** openstack has joined #heat23:26
*** sridhar_ram has quit IRC23:27
*** tochi has joined #heat23:40
*** sabeen has quit IRC23:53
*** sridhar_ram has joined #heat23:53
*** tlashchova has quit IRC23:55
*** tmatsu has joined #heat23:57
*** sridhar_ram has quit IRC23:59

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