Friday, 2017-01-13

*** chlong has quit IRC00:01
*** Pavo has quit IRC00:01
*** dfflanders has joined #heat00:02
*** stingaci has joined #heat00:02
*** vijendar1 has quit IRC00:04
*** vijendar_ has quit IRC00:04
*** stingaci has quit IRC00:07
*** Pavo has joined #heat00:11
*** Pavo has quit IRC00:22
*** vijendar has joined #heat00:23
*** vijendar_ has joined #heat00:23
*** Pavo has joined #heat00:24
*** jasond has quit IRC00:25
*** vijendar has quit IRC00:27
*** EricGonczer_ has joined #heat00:29
*** Pavo has quit IRC00:32
*** EricGonczer_ has quit IRC00:34
*** rajpatel has quit IRC00:34
*** Guest29822 has quit IRC00:37
*** Drago has quit IRC00:39
*** jose-phillips has quit IRC00:40
*** hoangcx has joined #heat00:48
*** zhurong has joined #heat00:55
*** om2 has quit IRC00:58
*** Drago has joined #heat01:00
*** om2 has joined #heat01:00
*** vijendar has joined #heat01:02
*** vijendar_ has quit IRC01:02
*** vijendar_ has joined #heat01:02
*** om2 has quit IRC01:03
*** rajpatel has joined #heat01:06
*** vijendar has quit IRC01:06
*** EricGonczer_ has joined #heat01:23
*** Pavo has joined #heat01:27
*** EricGonczer_ has quit IRC01:28
*** ayoung has quit IRC01:28
*** Pavo has quit IRC01:29
*** yanyanhu has joined #heat01:37
*** gongysh has joined #heat01:40
*** om2 has joined #heat01:48
*** adrian_otto has quit IRC01:52
*** yanyanhu has quit IRC01:53
*** yanyanhu has joined #heat01:54
*** Pavo has joined #heat01:56
*** vijendar_ has quit IRC01:57
*** dimtruck is now known as zz_dimtruck01:58
*** zz_dimtruck is now known as dimtruck01:59
*** wllabs has joined #heat02:05
*** rajpatel has quit IRC02:05
*** bana_k has quit IRC02:07
*** Pavo has quit IRC02:09
openstackgerritZhenyu Zheng proposed openstack/python-heatclient: Enable coverage report in console output  https://review.openstack.org/41864902:13
*** Pavo has joined #heat02:19
openstackgerritRussell Bryant proposed openstack/python-heatclient: Support -y for --yes.  https://review.openstack.org/41971802:37
*** dimtruck is now known as zz_dimtruck02:48
*** spzala has joined #heat02:51
*** yuanying has quit IRC02:52
openstackgerritJeremy Liu proposed openstack/python-heatclient: Enable coverage report in console output  https://review.openstack.org/41973102:55
*** om2 has quit IRC03:03
openstackgerritMerged openstack/heat: Ignore dependency errors with conditions  https://review.openstack.org/41958803:10
*** sdake_ has joined #heat03:14
*** sdake has quit IRC03:18
*** tiantian has quit IRC03:19
*** huangtianhua has joined #heat03:19
*** adrian_otto has joined #heat03:22
*** Pavo has quit IRC03:24
*** ramishra has quit IRC03:26
*** zz_dimtruck is now known as dimtruck03:27
*** ramishra has joined #heat03:28
*** gongysh has quit IRC03:29
*** om2 has joined #heat03:31
*** om2 has quit IRC03:31
*** zhurong has quit IRC03:39
*** dfflanders has quit IRC03:40
*** Pavo has joined #heat03:40
*** adrian_otto has quit IRC03:41
*** sdake has joined #heat03:42
*** sdake_ has quit IRC03:45
*** steveg_afk has quit IRC03:50
*** vijendar has joined #heat03:50
*** EricGonczer_ has joined #heat03:51
*** EricGonczer_ has joined #heat03:52
*** yuanying has joined #heat03:52
*** links has joined #heat03:54
*** EricGonczer_ has quit IRC03:54
*** Pavo has quit IRC04:01
*** KanagarajM has joined #heat04:01
*** spzala has quit IRC04:02
*** ramishra has quit IRC04:10
*** rajpatel has joined #heat04:12
*** vijendar has quit IRC04:12
*** ramishra has joined #heat04:12
*** vijendar has joined #heat04:12
*** EricGonczer_ has joined #heat04:13
*** rajpatel1 has joined #heat04:14
*** EricGonczer_ has quit IRC04:15
*** rajpatel has quit IRC04:17
*** vijendar has quit IRC04:26
*** Drago has quit IRC04:51
*** masber has joined #heat04:57
*** rajpatel1 has quit IRC05:02
*** dimtruck is now known as zz_dimtruck05:11
*** adrian_otto has joined #heat05:13
*** prateek has joined #heat05:19
*** gongysh has joined #heat05:30
*** adrian_otto has quit IRC05:40
openstackgerritMerged openstack/heat: Allow to scale when group is in CHECK_COMPLETE  https://review.openstack.org/41871605:47
*** hoangcx has quit IRC05:47
*** KanagarajM has quit IRC05:59
*** KanagarajM has joined #heat06:00
*** stingaci has joined #heat06:04
*** yuanying has quit IRC06:06
*** yuanying has joined #heat06:07
*** stingaci has quit IRC06:08
*** hoangcx has joined #heat06:10
*** yuanying has quit IRC06:11
*** yuanying_ has joined #heat06:11
*** yuanying_ has quit IRC06:12
*** yuanying has joined #heat06:12
*** yuanying has quit IRC06:17
*** hoangcx has quit IRC06:19
*** KanagarajM has quit IRC06:25
*** ramishra has quit IRC06:26
*** vijendar has joined #heat06:26
*** jprovazn has joined #heat06:27
*** ramishra has joined #heat06:31
*** ramishra has quit IRC06:36
*** ramishra has joined #heat06:38
*** om2 has joined #heat06:39
*** vijendar has quit IRC06:44
openstackgerritMerged openstack/heat: Add OS::Neutron::Quota resource  https://review.openstack.org/38047106:51
*** gongysh has quit IRC06:57
openstackgerritRabi Mishra proposed openstack/heat: Don't use MagicMock for stacks in tests  https://review.openstack.org/41978607:05
*** zhurong has joined #heat07:05
openstackgerritMerged openstack/heat: Deprecate designate v1 resource plugin  https://review.openstack.org/41704207:17
openstackgerritMerged openstack/heat: Release notes for OS::Keystone::Domain  https://review.openstack.org/41712407:17
*** yuanying has joined #heat07:19
wllabsLBaaS::member add member, if i want remove or replace member ip address07:22
openstackgerritCrag Wolfe proposed openstack/heat: WIP - Efficient passing of attrs during traversals  https://review.openstack.org/41862807:22
wllabshow to do it?07:22
ramishrawllabs: if you change address of the PoolMember, it would be replaced, the old one removed from the pool and new one added.07:29
ramishraIs not that the behaviour you see?07:29
openstackgerrithuangtianhua proposed openstack/heat: Fix validate_external()  https://review.openstack.org/41979407:30
*** om2 has quit IRC07:30
*** wirehead_ has quit IRC07:30
*** mwhahaha has quit IRC07:30
*** Hazelesque has quit IRC07:30
*** mwhahaha has joined #heat07:31
*** Hazelesque has joined #heat07:31
*** wirehead_ has joined #heat07:36
*** AlexeyAbashkin has joined #heat07:39
*** wllabs has quit IRC07:44
*** wllabs has joined #heat07:48
openstackgerritMerged openstack/heat: Deprecate OS::Glance::Image  https://review.openstack.org/41904307:50
*** stingaci has joined #heat07:51
*** stingaci has quit IRC07:55
*** wirehead_ has quit IRC07:57
*** wllabs has quit IRC08:00
*** wirehead_ has joined #heat08:04
*** stingaci has joined #heat08:10
*** shardy has joined #heat08:12
openstackgerrithuangtianhua proposed openstack/heat: Fix validate_external()  https://review.openstack.org/41979408:34
*** ricolin has joined #heat08:35
*** sorantis has joined #heat08:46
*** jpich has joined #heat08:53
openstackgerrithuangtianhua proposed openstack/heat: Use super method _show_resource() for alarms  https://review.openstack.org/41982408:55
*** zzzeek has quit IRC09:00
*** zzzeek has joined #heat09:00
*** KanagarajM has joined #heat09:02
*** wllabs has joined #heat09:12
*** zhurong has quit IRC09:16
openstackgerrithuangtianhua proposed openstack/heat: No need to check if resource not created  https://review.openstack.org/41563909:19
*** yassine has joined #heat09:23
*** yassine is now known as Guest6001509:23
*** derekh has joined #heat09:25
openstackgerrithuangtianhua proposed openstack/heat: Refactor _show_resource() for neutron resources  https://review.openstack.org/41985109:43
*** mannamne has joined #heat09:44
*** askb has quit IRC09:45
*** jpich has quit IRC09:55
*** jpich has joined #heat09:56
*** manjong has joined #heat10:07
*** manjong has quit IRC10:12
*** openstackgerrit has quit IRC10:18
*** yanyanhu has quit IRC10:19
*** vijendar has joined #heat10:41
*** openstackgerrit has joined #heat10:43
openstackgerritMartin AndrĂ© proposed openstack/heat-agents: Prevent trailing whitespace in rendered JSON  https://review.openstack.org/41987810:43
*** vijendar has quit IRC10:45
*** prazumovsky has joined #heat11:01
*** wllabs has quit IRC11:07
openstackgerritMartin AndrĂ© proposed openstack/heat-agents: Prevent trailing whitespace in rendered JSON  https://review.openstack.org/41987811:08
*** prazumovsky has quit IRC11:10
*** ricolin has quit IRC11:18
openstackgerritKanagaraj Manickam proposed openstack/heat: Release notes for designate v2 support  https://review.openstack.org/41704311:33
*** openstackgerrit has quit IRC11:47
*** openstackgerrit has joined #heat11:48
*** EricGonczer_ has joined #heat11:49
*** ayoung has joined #heat11:57
*** shardy is now known as shardy_lunch12:06
*** thijsferyn has joined #heat12:19
thijsferyn#Is it possible to define an AS Group with a loadbalanced set of servers in a single YAML file?12:20
*** KanagarajM has quit IRC12:23
thijsferynThe difficulty is adding LBaaS::PoolMembers after server creation. Any idea how to do all this in one single file? THX!12:23
*** catintheroof has joined #heat12:30
*** catintheroof has quit IRC12:31
*** catintheroof has joined #heat12:32
*** cmyster has quit IRC12:35
*** sshnaidm is now known as sshnaidm|off12:40
*** cmyster has joined #heat12:42
*** cmyster has joined #heat12:42
*** ccamacho is now known as ccamacho|lunch12:44
*** dprince has joined #heat12:49
*** vijendar has joined #heat13:02
*** shardy_lunch is now known as shardy13:04
*** spotz is now known as spotz_zzz13:05
*** vijendar has quit IRC13:07
openstackgerritMerged openstack/heat: Use correct charset when create a text/* type MIME  https://review.openstack.org/33864913:14
thervethijsferyn, I don't think so. Why do you require a single file?13:36
thijsferynmakes it easier for example purposes: single use case, single file. But I sort of accepted by now that it insn't possible13:37
*** AlexeyAbashkin has quit IRC13:37
thervethijsferyn, You can somewhat if you control the server side, by providing a custom resource in the global environment13:39
therveThat's effectively putting the second file on your server deployment13:40
*** AlexeyAbashkin has joined #heat13:40
thijsferynThx therve13:41
therveNo problem13:41
*** steveg_afk has joined #heat13:41
*** prateek has quit IRC13:43
*** EricGonczer_ has quit IRC13:45
therveramishra, I don't understand https://bugs.launchpad.net/heat/+bug/165619013:48
openstackLaunchpad bug 1656190 in heat "test_neutron.NeutronTest.test_needs_replace_failed failed with TypeError" [Medium,In progress] - Assigned to Rabi Mishra (rabi)13:48
therveIs that an intermittent failure?13:48
*** jcoufal has joined #heat13:49
*** rajpatel has joined #heat13:53
*** steveg_afk has quit IRC13:55
*** rajpatel has quit IRC13:57
*** steveg_afk has joined #heat13:59
*** rajpatel has joined #heat13:59
*** sdake_ has joined #heat14:02
*** sdake has quit IRC14:06
*** jprovazn has quit IRC14:21
*** jprovazn has joined #heat14:21
*** jprovazn has quit IRC14:23
*** jprovazn has joined #heat14:30
*** dyasny has joined #heat14:35
*** crose has joined #heat14:37
ramishratherve: yes, I noticed it once though14:39
*** rajpatel has quit IRC14:40
ramishraWe are using MagicMock for the stack and then doing event_count_all_by_stack https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/api.py#L94014:41
ramishrawhen storing the event for the resource.14:41
ramishraAtleast that's my understanding14:42
*** jasond has joined #heat14:43
therveramishra, I mean, why is it intermittent?14:44
ramishraI don't know14:45
*** chlong has joined #heat14:45
therveHumph14:45
*** vijendar has joined #heat14:50
*** spzala has joined #heat14:51
*** manjong has joined #heat14:52
*** sdake_ is now known as sdake14:54
*** prateek has joined #heat14:54
*** beagles is now known as beagles_afk14:54
*** manjong has quit IRC14:57
*** prateek has quit IRC14:57
*** prateek has joined #heat14:58
*** jaosorior has joined #heat15:05
*** jaosorior has quit IRC15:07
*** sorantis has quit IRC15:07
openstackgerritPeter Razumovsky proposed openstack/heat: Fix using parent_name for Properties  https://review.openstack.org/42001415:08
openstackgerritPeter Razumovsky proposed openstack/heat: Add full path for Property  https://review.openstack.org/42001515:08
openstackgerritPeter Razumovsky proposed openstack/heat: Fix validation in TranslationRule  https://review.openstack.org/42001615:08
openstackgerritPeter Razumovsky proposed openstack/heat: Initial add Translation-in-place mechanism  https://review.openstack.org/42001715:08
openstackgerritPeter Razumovsky proposed openstack/heat: Add translation method for Translation  https://review.openstack.org/42001815:08
openstackgerritPeter Razumovsky proposed openstack/heat: Add "Replace" handling for translation  https://review.openstack.org/42001915:08
openstackgerritPeter Razumovsky proposed openstack/heat: Implement "Add" rule for Translation  https://review.openstack.org/42002015:08
openstackgerritPeter Razumovsky proposed openstack/heat: Add "Resolve" rule for Translation  https://review.openstack.org/42002115:08
openstackgerritPeter Razumovsky proposed openstack/heat: Enable new translation mechanism  https://review.openstack.org/42002215:08
openstackgerritPeter Razumovsky proposed openstack/heat: Add few functional tests for translation  https://review.openstack.org/42002315:08
openstackgerritPeter Razumovsky proposed openstack/heat: Remove unnecessary old translation mechanism  https://review.openstack.org/42002415:08
*** ayoung has quit IRC15:09
*** annegentle has joined #heat15:10
*** vijendar_ has joined #heat15:10
*** manjong has joined #heat15:13
*** vijendar has quit IRC15:13
*** jaosorior has joined #heat15:16
*** annegentle has quit IRC15:16
*** manjong has quit IRC15:18
*** jaosorior has quit IRC15:19
*** zz_dimtruck is now known as dimtruck15:19
*** jaosorior has joined #heat15:22
*** rajpatel has joined #heat15:27
ramishratherve: I think that condition is not checcked most of time15:28
ramishraif check is false15:28
therveramishra, So that's because of a change in your patch?15:29
ramishraI mean https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/api.py#L938-L93915:29
*** chris_hultin|AWA is now known as chris_hultin15:29
*** spotz_zzz is now known as spotz15:30
*** Pavo has joined #heat15:30
ramishrawe randomly compare the event_count15:30
therveAh, ok15:32
zanebtherve: the heat-templates package in RDO appears to make use of http://git.openstack.org/cgit/openstack/heat-templates/tree/hot/software-config/boot-config which is not in the heat-agents repo15:35
zanebtherve: should we just copy them over? or do something else?15:35
thervezaneb, I guess? I only copied the things in elements, I was wondering about the rest15:36
therveI thought those were mostly templates, but I guess we package those15:38
therveHum we only package 2 files15:40
zanebyeah15:40
zanebI wonder if those should just be included as sources in the RDO package, or if they do belong in the repo15:41
*** vijendar_ has quit IRC15:42
openstackgerritMerged openstack/heat: Don't use MagicMock for stacks in tests  https://review.openstack.org/41978615:44
*** vijendar has joined #heat15:45
*** vijendar_ has joined #heat15:46
*** EricGonczer_ has joined #heat15:49
*** Matthias__ has quit IRC15:49
*** vijendar has quit IRC15:49
zanebtherve: actually, it looks like the package installs the entire contents of hot/software_config in /usr/share15:50
*** prateek has quit IRC15:51
*** Drago has joined #heat15:54
*** spzala has quit IRC15:55
*** sdake_ has joined #heat15:56
*** spzala has joined #heat15:56
*** sdake has quit IRC15:57
thervezaneb, In the openstack-heat-templates package though, no?15:58
zanebyes15:59
therveShould we keep that one?15:59
therveI don't think tripleo installs it for instance15:59
*** spzala has quit IRC16:00
zanebgood point, ok16:01
*** spzala has joined #heat16:01
zanebmaybe we shouldn't use Provides then16:01
therveRIght, I would just replace the sub packages16:02
therveWe should probably still copy over those 2 other files though16:04
zanebtherve: copy them to the repo, or add them as source files in the package?16:14
thervezaneb, What's the latter?16:15
therveCould we add source files from another package?16:15
zanebyou add it in the SRPM, like you would a patch file16:15
zanebso RDO would own it and no upstream changes would affect it16:16
*** adrian_otto has joined #heat16:16
zanebwhich seems fine for http://git.openstack.org/cgit/openstack/heat-templates/tree/hot/software-config/boot-config/templates/fragments/20-os-apply-config16:16
zanebmaybe not so much for http://git.openstack.org/cgit/openstack/heat-templates/tree/hot/software-config/boot-config/templates/fragments/os-collect-config.conf though16:16
*** stingaci has quit IRC16:17
therveYeah this file is changed from time to time16:17
therveIt's also referenced from boot-config elsewhere16:17
*** dprince has quit IRC16:19
Pavoanyone in here every deploy csr1000v using heat?16:20
*** dprince has joined #heat16:21
*** stingaci has joined #heat16:28
*** annegentle has joined #heat16:29
*** anton is now known as anton-zzz16:30
zanebtherve: we'll need a LICENSE file in that repo too16:31
*** sdake_ is now known as sdake_torture_ti16:31
*** rajpatel has quit IRC16:32
*** stingaci has quit IRC16:32
openstackgerritMerged openstack/heat-agents: Add 'exec' support to docker cmd  https://review.openstack.org/41752116:38
*** chlong has quit IRC16:38
*** stingaci has joined #heat16:41
*** bnemec is now known as beekneemech16:43
*** chlong has joined #heat16:43
*** lamt has joined #heat16:43
openstackgerritZane Bitter proposed openstack/heat-agents: Reinstate LICENSE file  https://review.openstack.org/42006416:47
openstackgerritZane Bitter proposed openstack/heat-agents: Add boot-config files for os-apply/collect-config  https://review.openstack.org/42006516:47
*** jose-phillips has joined #heat16:47
zanebtherve: ^16:47
*** chlong has quit IRC16:48
*** AlexeyAbashkin has quit IRC16:49
*** bana_k has joined #heat16:50
*** rajpatel has joined #heat16:51
*** EricGonc_ has joined #heat16:52
*** lamt has quit IRC16:52
*** EricGonczer_ has quit IRC16:53
*** stingaci has quit IRC16:59
*** chlong has joined #heat16:59
*** jpich has quit IRC17:02
larsksIf 'heat stack-list' works but 'heat stack-create' fails with "Missing required credential: X-Auth-User", does that mean my client is too new for the server?17:02
*** derekh has quit IRC17:08
*** chlong has quit IRC17:09
*** bana_k has quit IRC17:15
*** lamt has joined #heat17:18
*** dprince_ has joined #heat17:20
*** dprince has quit IRC17:22
*** adrian_otto has quit IRC17:24
*** steveg_afk has quit IRC17:24
*** vijendar_ has quit IRC17:29
*** shardy has quit IRC17:46
*** vijendar has joined #heat17:56
*** vijendar_ has joined #heat17:57
*** ri0 has joined #heat17:58
*** ri0 has quit IRC17:58
*** vijendar has quit IRC18:00
*** rajpatel has quit IRC18:06
*** steveg_afk has joined #heat18:08
*** links has quit IRC18:12
*** rajpatel has joined #heat18:12
*** rajpatel has quit IRC18:15
*** bana_k has joined #heat18:15
*** spzala has quit IRC18:18
*** spzala has joined #heat18:22
*** jprovazn has quit IRC18:24
*** om2 has joined #heat18:28
*** steveg_afk has quit IRC18:36
*** chris_hultin is now known as chris_hultin|AWA18:56
*** openstack has joined #heat19:02
*** jamielennox is now known as jamielennox|away19:12
*** mannamne has quit IRC19:13
*** chlong has joined #heat19:14
*** adrian_otto has joined #heat19:15
*** rajpatel has joined #heat19:26
openstackgerritMerged openstack/heat: Unit tests: Don't mock client_plugin for Zaqar subscription  https://review.openstack.org/41958419:26
*** thijsferyn has quit IRC19:27
*** sdake has joined #heat19:34
*** sdake_ has joined #heat19:40
*** adrian_otto has quit IRC19:43
*** sdake has quit IRC19:44
*** spzala has quit IRC19:45
*** rajpatel has quit IRC19:49
*** dimtruck is now known as zz_dimtruck19:52
*** vijendar_ has quit IRC19:56
*** openstackstatus has quit IRC19:57
*** openstack has joined #heat20:02
*** stingaci has quit IRC20:04
*** liusheng has quit IRC20:11
*** liusheng has joined #heat20:11
*** vijendar has joined #heat20:11
*** vijendar_ has joined #heat20:12
openstackgerritMerged openstack/heat: Fix errors resulting from over-long Zaqar queue names  https://review.openstack.org/41916920:13
*** vijendar has quit IRC20:16
*** iurygregory has quit IRC20:18
*** spzala has joined #heat20:21
*** zz_dimtruck is now known as dimtruck20:24
*** steveg_afk has joined #heat20:26
*** openstackstatus has quit IRC20:27
*** openstack has joined #heat20:32
*** chris_hultin is now known as chris_hultin|AWA20:42
*** jaosorior has quit IRC20:45
*** spotz is now known as spotz_zzz20:45
*** jaosorior has joined #heat20:45
*** Pavo has quit IRC20:47
*** crose has quit IRC20:50
*** annegentle has quit IRC20:55
*** annegentle has joined #heat20:59
zanebcwolferh: re the comment I just left on https://review.openstack.org/#/c/418628/2 ...21:02
*** Pavo has joined #heat21:02
cwolferhzaneb, i was just writing code to avoid always loading the resources :)21:02
zanebI think I'm going to submit a patch that just switches to not loading resources from the DB, independently of the changes you're working on21:02
zanebI want to run the experimental job on that and see how much memory it saves21:03
cwolferhcool21:03
zanebI feel like we need some test to enforce that as well21:03
zanebI can put in an error log for now21:04
*** spotz_zzz is now known as spotz21:06
*** spotz is now known as spotz_zzz21:06
cwolferhzaneb, still needs polish, but my current thinking on the patch: https://github.com/cwolferh/heat-scratch/commit/5519199f647b7220dc14edd29d732e593bd86e1b21:10
cwolferhadded the load_resources classmethod to Resource, and use that to get the relevant set of resources that we are propagating to.21:12
zanebcwolferh: the idea with convergence was that we shouldn't have to load *any* resources except the one the worker is working on though21:13
cwolferhwell, it's either that or always propagate all of your attributes to everybody. i don't think you can have it both ways21:15
*** adrian_otto has joined #heat21:16
cwolferhone thing that does work nicely now is that the attribute data in input_data does populate stack's cache_data. so resource B doing it's attribute lookups on resources it depends on is very cheap.21:18
*** sdake has quit IRC21:19
zanebyes, you can't have it both ways unless the resource name is part of the SyncPoint key (which it isn't, and we certainly aren't doing *that* db migration ;)21:19
*** Pavo has quit IRC21:20
zanebI think we just have to accept that we'll be propagating all of the attribute data to everyone21:20
zanebbut at least we won't be retrieving it in a loop any more21:20
*** adrian_otto has quit IRC21:22
cwolferhzaneb, so are you leaning towards still loading the stack resources to see all the resource's attributes that are referenced elsewhere to pass along, or just pass all of the resources attributes (even ones that may not actually be referenced anywhere)?21:24
zanebwe can definitely still narrow it down to only the ones that are referenced (we have the template loaded, so all we need to do is use the ResourceDefinitions instead of that pass-through method in Resource)21:25
zanebwe just can't know *where* they are referenced21:25
*** jcoufal has quit IRC21:29
*** chris_hultin|AWA is now known as chris_hultin21:31
*** chris_hultin is now known as chris_hultin|AWA21:32
cwolferhstill wondering if we don't have to give up not passing all ref'ed attributes everywhere yet... might be possible to have something like Resource.load_resource not actually load property data resource data (the heavy stuff), since we only care about dep_attrs which uses the resource definition21:32
*** vijendar_ has quit IRC21:34
*** sdake has joined #heat21:35
*** Pavo has joined #heat21:41
*** vijendar has joined #heat21:41
*** sdake has quit IRC21:42
*** sdake has joined #heat21:43
*** Pavo has quit IRC21:50
*** steveg_afk has quit IRC21:52
*** Pavo has joined #heat21:53
*** Pavo has quit IRC21:53
*** rajpatel has quit IRC21:54
*** Pavo has joined #heat21:55
zanebcwolferh: SELECT id, name FROM Resource WHERE id IN [...]21:57
zanebcwolferh: that would do it. you won't get it from sqlalchemy.orm, but it could be done21:58
zanebwhether that trade-off is worth it... only benchmarking can tell21:58
*** jamielennox|away is now known as jamielennox21:59
*** stingaci has joined #heat22:00
openstackgerritZane Bitter proposed openstack/heat: Don't load all resources in convergence check  https://review.openstack.org/42019122:03
cwolferhzaneb yeah, and to be fair it probably has more to do with o.vo baggage than sqlachemy.orm. but, i agree it could be done and the db op should be relatively cheap.22:03
*** stingaci has quit IRC22:04
zanebyeah... it's nice that there's only one op to get all of them. anything O(n) is out of the question imho, but this would be much more likely to be a win22:05
*** rajpatel has joined #heat22:08
*** sdake_ has joined #heat22:08
*** rajpatel has quit IRC22:10
*** sdake has quit IRC22:10
cwolferhwhen building the graph in the first place, it seems like attr names could be included in edge data (not suggesting that short term though)22:13
*** mannamne has joined #heat22:16
zanebcwolferh: the weird thing is that I initially designed it that way but then changed my mind: https://github.com/zaneb/heat-convergence-prototype/commit/3fc02f75167cdc1e250ce5fc7c6b0f0aad0537ac22:22
zanebcwolferh: oh wait, you said attribute names not resource names22:22
cwolferhzaneb, yep, that22:23
cwolferhan extra tuple22:23
cwolferhin the edge definition22:24
cwolferhi kind of like that it has id and not name right now, that keeps a little bit of honesty going. it is too easy to grab the named resource from the wrong stack/template as is right now imo ;)22:25
*** adrian_otto has joined #heat22:25
*** baling-wire is now known as david-lyle22:26
*** dprince_ has quit IRC22:26
zanebagree :D22:27
*** spzala has quit IRC22:29
*** sdake_ has quit IRC22:33
*** sdake has joined #heat22:35
*** catintheroof has quit IRC22:38
*** kraken_ has joined #heat22:39
*** annegentle has quit IRC22:42
*** rajpatel has joined #heat22:48
*** pradk has quit IRC23:00
*** rajpatel has quit IRC23:00
*** rajpatel has joined #heat23:01
*** rajpatel has quit IRC23:03
*** jasond has quit IRC23:09
*** rajpatel has joined #heat23:12
*** kraken_ has quit IRC23:15
*** Pavo has quit IRC23:21
*** mannamne has quit IRC23:27
*** adrian_otto has quit IRC23:31
*** jaosorior has quit IRC23:31
*** EricGonc_ has quit IRC23:37
*** sdake has quit IRC23:38
*** chlong has quit IRC23:38
*** EricGonczer_ has joined #heat23:38
*** harlowja has quit IRC23:56
*** vijendar has quit IRC23:59

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