*** chlong has quit IRC | 00:01 | |
*** Pavo has quit IRC | 00:01 | |
*** dfflanders has joined #heat | 00:02 | |
*** stingaci has joined #heat | 00:02 | |
*** vijendar1 has quit IRC | 00:04 | |
*** vijendar_ has quit IRC | 00:04 | |
*** stingaci has quit IRC | 00:07 | |
*** Pavo has joined #heat | 00:11 | |
*** Pavo has quit IRC | 00:22 | |
*** vijendar has joined #heat | 00:23 | |
*** vijendar_ has joined #heat | 00:23 | |
*** Pavo has joined #heat | 00:24 | |
*** jasond has quit IRC | 00:25 | |
*** vijendar has quit IRC | 00:27 | |
*** EricGonczer_ has joined #heat | 00:29 | |
*** Pavo has quit IRC | 00:32 | |
*** EricGonczer_ has quit IRC | 00:34 | |
*** rajpatel has quit IRC | 00:34 | |
*** Guest29822 has quit IRC | 00:37 | |
*** Drago has quit IRC | 00:39 | |
*** jose-phillips has quit IRC | 00:40 | |
*** hoangcx has joined #heat | 00:48 | |
*** zhurong has joined #heat | 00:55 | |
*** om2 has quit IRC | 00:58 | |
*** Drago has joined #heat | 01:00 | |
*** om2 has joined #heat | 01:00 | |
*** vijendar has joined #heat | 01:02 | |
*** vijendar_ has quit IRC | 01:02 | |
*** vijendar_ has joined #heat | 01:02 | |
*** om2 has quit IRC | 01:03 | |
*** rajpatel has joined #heat | 01:06 | |
*** vijendar has quit IRC | 01:06 | |
*** EricGonczer_ has joined #heat | 01:23 | |
*** Pavo has joined #heat | 01:27 | |
*** EricGonczer_ has quit IRC | 01:28 | |
*** ayoung has quit IRC | 01:28 | |
*** Pavo has quit IRC | 01:29 | |
*** yanyanhu has joined #heat | 01:37 | |
*** gongysh has joined #heat | 01:40 | |
*** om2 has joined #heat | 01:48 | |
*** adrian_otto has quit IRC | 01:52 | |
*** yanyanhu has quit IRC | 01:53 | |
*** yanyanhu has joined #heat | 01:54 | |
*** Pavo has joined #heat | 01:56 | |
*** vijendar_ has quit IRC | 01:57 | |
*** dimtruck is now known as zz_dimtruck | 01:58 | |
*** zz_dimtruck is now known as dimtruck | 01:59 | |
*** wllabs has joined #heat | 02:05 | |
*** rajpatel has quit IRC | 02:05 | |
*** bana_k has quit IRC | 02:07 | |
*** Pavo has quit IRC | 02:09 | |
openstackgerrit | Zhenyu Zheng proposed openstack/python-heatclient: Enable coverage report in console output https://review.openstack.org/418649 | 02:13 |
---|---|---|
*** Pavo has joined #heat | 02:19 | |
openstackgerrit | Russell Bryant proposed openstack/python-heatclient: Support -y for --yes. https://review.openstack.org/419718 | 02:37 |
*** dimtruck is now known as zz_dimtruck | 02:48 | |
*** spzala has joined #heat | 02:51 | |
*** yuanying has quit IRC | 02:52 | |
openstackgerrit | Jeremy Liu proposed openstack/python-heatclient: Enable coverage report in console output https://review.openstack.org/419731 | 02:55 |
*** om2 has quit IRC | 03:03 | |
openstackgerrit | Merged openstack/heat: Ignore dependency errors with conditions https://review.openstack.org/419588 | 03:10 |
*** sdake_ has joined #heat | 03:14 | |
*** sdake has quit IRC | 03:18 | |
*** tiantian has quit IRC | 03:19 | |
*** huangtianhua has joined #heat | 03:19 | |
*** adrian_otto has joined #heat | 03:22 | |
*** Pavo has quit IRC | 03:24 | |
*** ramishra has quit IRC | 03:26 | |
*** zz_dimtruck is now known as dimtruck | 03:27 | |
*** ramishra has joined #heat | 03:28 | |
*** gongysh has quit IRC | 03:29 | |
*** om2 has joined #heat | 03:31 | |
*** om2 has quit IRC | 03:31 | |
*** zhurong has quit IRC | 03:39 | |
*** dfflanders has quit IRC | 03:40 | |
*** Pavo has joined #heat | 03:40 | |
*** adrian_otto has quit IRC | 03:41 | |
*** sdake has joined #heat | 03:42 | |
*** sdake_ has quit IRC | 03:45 | |
*** steveg_afk has quit IRC | 03:50 | |
*** vijendar has joined #heat | 03:50 | |
*** EricGonczer_ has joined #heat | 03:51 | |
*** EricGonczer_ has joined #heat | 03:52 | |
*** yuanying has joined #heat | 03:52 | |
*** links has joined #heat | 03:54 | |
*** EricGonczer_ has quit IRC | 03:54 | |
*** Pavo has quit IRC | 04:01 | |
*** KanagarajM has joined #heat | 04:01 | |
*** spzala has quit IRC | 04:02 | |
*** ramishra has quit IRC | 04:10 | |
*** rajpatel has joined #heat | 04:12 | |
*** vijendar has quit IRC | 04:12 | |
*** ramishra has joined #heat | 04:12 | |
*** vijendar has joined #heat | 04:12 | |
*** EricGonczer_ has joined #heat | 04:13 | |
*** rajpatel1 has joined #heat | 04:14 | |
*** EricGonczer_ has quit IRC | 04:15 | |
*** rajpatel has quit IRC | 04:17 | |
*** vijendar has quit IRC | 04:26 | |
*** Drago has quit IRC | 04:51 | |
*** masber has joined #heat | 04:57 | |
*** rajpatel1 has quit IRC | 05:02 | |
*** dimtruck is now known as zz_dimtruck | 05:11 | |
*** adrian_otto has joined #heat | 05:13 | |
*** prateek has joined #heat | 05:19 | |
*** gongysh has joined #heat | 05:30 | |
*** adrian_otto has quit IRC | 05:40 | |
openstackgerrit | Merged openstack/heat: Allow to scale when group is in CHECK_COMPLETE https://review.openstack.org/418716 | 05:47 |
*** hoangcx has quit IRC | 05:47 | |
*** KanagarajM has quit IRC | 05:59 | |
*** KanagarajM has joined #heat | 06:00 | |
*** stingaci has joined #heat | 06:04 | |
*** yuanying has quit IRC | 06:06 | |
*** yuanying has joined #heat | 06:07 | |
*** stingaci has quit IRC | 06:08 | |
*** hoangcx has joined #heat | 06:10 | |
*** yuanying has quit IRC | 06:11 | |
*** yuanying_ has joined #heat | 06:11 | |
*** yuanying_ has quit IRC | 06:12 | |
*** yuanying has joined #heat | 06:12 | |
*** yuanying has quit IRC | 06:17 | |
*** hoangcx has quit IRC | 06:19 | |
*** KanagarajM has quit IRC | 06:25 | |
*** ramishra has quit IRC | 06:26 | |
*** vijendar has joined #heat | 06:26 | |
*** jprovazn has joined #heat | 06:27 | |
*** ramishra has joined #heat | 06:31 | |
*** ramishra has quit IRC | 06:36 | |
*** ramishra has joined #heat | 06:38 | |
*** om2 has joined #heat | 06:39 | |
*** vijendar has quit IRC | 06:44 | |
openstackgerrit | Merged openstack/heat: Add OS::Neutron::Quota resource https://review.openstack.org/380471 | 06:51 |
*** gongysh has quit IRC | 06:57 | |
openstackgerrit | Rabi Mishra proposed openstack/heat: Don't use MagicMock for stacks in tests https://review.openstack.org/419786 | 07:05 |
*** zhurong has joined #heat | 07:05 | |
openstackgerrit | Merged openstack/heat: Deprecate designate v1 resource plugin https://review.openstack.org/417042 | 07:17 |
openstackgerrit | Merged openstack/heat: Release notes for OS::Keystone::Domain https://review.openstack.org/417124 | 07:17 |
*** yuanying has joined #heat | 07:19 | |
wllabs | LBaaS::member add member, if i want remove or replace member ip address | 07:22 |
openstackgerrit | Crag Wolfe proposed openstack/heat: WIP - Efficient passing of attrs during traversals https://review.openstack.org/418628 | 07:22 |
wllabs | how to do it? | 07:22 |
ramishra | wllabs: if you change address of the PoolMember, it would be replaced, the old one removed from the pool and new one added. | 07:29 |
ramishra | Is not that the behaviour you see? | 07:29 |
openstackgerrit | huangtianhua proposed openstack/heat: Fix validate_external() https://review.openstack.org/419794 | 07:30 |
*** om2 has quit IRC | 07:30 | |
*** wirehead_ has quit IRC | 07:30 | |
*** mwhahaha has quit IRC | 07:30 | |
*** Hazelesque has quit IRC | 07:30 | |
*** mwhahaha has joined #heat | 07:31 | |
*** Hazelesque has joined #heat | 07:31 | |
*** wirehead_ has joined #heat | 07:36 | |
*** AlexeyAbashkin has joined #heat | 07:39 | |
*** wllabs has quit IRC | 07:44 | |
*** wllabs has joined #heat | 07:48 | |
openstackgerrit | Merged openstack/heat: Deprecate OS::Glance::Image https://review.openstack.org/419043 | 07:50 |
*** stingaci has joined #heat | 07:51 | |
*** stingaci has quit IRC | 07:55 | |
*** wirehead_ has quit IRC | 07:57 | |
*** wllabs has quit IRC | 08:00 | |
*** wirehead_ has joined #heat | 08:04 | |
*** stingaci has joined #heat | 08:10 | |
*** shardy has joined #heat | 08:12 | |
openstackgerrit | huangtianhua proposed openstack/heat: Fix validate_external() https://review.openstack.org/419794 | 08:34 |
*** ricolin has joined #heat | 08:35 | |
*** sorantis has joined #heat | 08:46 | |
*** jpich has joined #heat | 08:53 | |
openstackgerrit | huangtianhua proposed openstack/heat: Use super method _show_resource() for alarms https://review.openstack.org/419824 | 08:55 |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #heat | 09:00 | |
*** KanagarajM has joined #heat | 09:02 | |
*** wllabs has joined #heat | 09:12 | |
*** zhurong has quit IRC | 09:16 | |
openstackgerrit | huangtianhua proposed openstack/heat: No need to check if resource not created https://review.openstack.org/415639 | 09:19 |
*** yassine has joined #heat | 09:23 | |
*** yassine is now known as Guest60015 | 09:23 | |
*** derekh has joined #heat | 09:25 | |
openstackgerrit | huangtianhua proposed openstack/heat: Refactor _show_resource() for neutron resources https://review.openstack.org/419851 | 09:43 |
*** mannamne has joined #heat | 09:44 | |
*** askb has quit IRC | 09:45 | |
*** jpich has quit IRC | 09:55 | |
*** jpich has joined #heat | 09:56 | |
*** manjong has joined #heat | 10:07 | |
*** manjong has quit IRC | 10:12 | |
*** openstackgerrit has quit IRC | 10:18 | |
*** yanyanhu has quit IRC | 10:19 | |
*** vijendar has joined #heat | 10:41 | |
*** openstackgerrit has joined #heat | 10:43 | |
openstackgerrit | Martin André proposed openstack/heat-agents: Prevent trailing whitespace in rendered JSON https://review.openstack.org/419878 | 10:43 |
*** vijendar has quit IRC | 10:45 | |
*** prazumovsky has joined #heat | 11:01 | |
*** wllabs has quit IRC | 11:07 | |
openstackgerrit | Martin André proposed openstack/heat-agents: Prevent trailing whitespace in rendered JSON https://review.openstack.org/419878 | 11:08 |
*** prazumovsky has quit IRC | 11:10 | |
*** ricolin has quit IRC | 11:18 | |
openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Release notes for designate v2 support https://review.openstack.org/417043 | 11:33 |
*** openstackgerrit has quit IRC | 11:47 | |
*** openstackgerrit has joined #heat | 11:48 | |
*** EricGonczer_ has joined #heat | 11:49 | |
*** ayoung has joined #heat | 11:57 | |
*** shardy is now known as shardy_lunch | 12:06 | |
*** thijsferyn has joined #heat | 12: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 IRC | 12:23 | |
thijsferyn | The 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 #heat | 12:30 | |
*** catintheroof has quit IRC | 12:31 | |
*** catintheroof has joined #heat | 12:32 | |
*** cmyster has quit IRC | 12:35 | |
*** sshnaidm is now known as sshnaidm|off | 12:40 | |
*** cmyster has joined #heat | 12:42 | |
*** cmyster has joined #heat | 12:42 | |
*** ccamacho is now known as ccamacho|lunch | 12:44 | |
*** dprince has joined #heat | 12:49 | |
*** vijendar has joined #heat | 13:02 | |
*** shardy_lunch is now known as shardy | 13:04 | |
*** spotz is now known as spotz_zzz | 13:05 | |
*** vijendar has quit IRC | 13:07 | |
openstackgerrit | Merged openstack/heat: Use correct charset when create a text/* type MIME https://review.openstack.org/338649 | 13:14 |
therve | thijsferyn, I don't think so. Why do you require a single file? | 13:36 |
thijsferyn | makes it easier for example purposes: single use case, single file. But I sort of accepted by now that it insn't possible | 13:37 |
*** AlexeyAbashkin has quit IRC | 13:37 | |
therve | thijsferyn, You can somewhat if you control the server side, by providing a custom resource in the global environment | 13:39 |
therve | That's effectively putting the second file on your server deployment | 13:40 |
*** AlexeyAbashkin has joined #heat | 13:40 | |
thijsferyn | Thx therve | 13:41 |
therve | No problem | 13:41 |
*** steveg_afk has joined #heat | 13:41 | |
*** prateek has quit IRC | 13:43 | |
*** EricGonczer_ has quit IRC | 13:45 | |
therve | ramishra, I don't understand https://bugs.launchpad.net/heat/+bug/1656190 | 13:48 |
openstack | Launchpad bug 1656190 in heat "test_neutron.NeutronTest.test_needs_replace_failed failed with TypeError" [Medium,In progress] - Assigned to Rabi Mishra (rabi) | 13:48 |
therve | Is that an intermittent failure? | 13:48 |
*** jcoufal has joined #heat | 13:49 | |
*** rajpatel has joined #heat | 13:53 | |
*** steveg_afk has quit IRC | 13:55 | |
*** rajpatel has quit IRC | 13:57 | |
*** steveg_afk has joined #heat | 13:59 | |
*** rajpatel has joined #heat | 13:59 | |
*** sdake_ has joined #heat | 14:02 | |
*** sdake has quit IRC | 14:06 | |
*** jprovazn has quit IRC | 14:21 | |
*** jprovazn has joined #heat | 14:21 | |
*** jprovazn has quit IRC | 14:23 | |
*** jprovazn has joined #heat | 14:30 | |
*** dyasny has joined #heat | 14:35 | |
*** crose has joined #heat | 14:37 | |
ramishra | therve: yes, I noticed it once though | 14:39 |
*** rajpatel has quit IRC | 14:40 | |
ramishra | We 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#L940 | 14:41 |
ramishra | when storing the event for the resource. | 14:41 |
ramishra | Atleast that's my understanding | 14:42 |
*** jasond has joined #heat | 14:43 | |
therve | ramishra, I mean, why is it intermittent? | 14:44 |
ramishra | I don't know | 14:45 |
*** chlong has joined #heat | 14:45 | |
therve | Humph | 14:45 |
*** vijendar has joined #heat | 14:50 | |
*** spzala has joined #heat | 14:51 | |
*** manjong has joined #heat | 14:52 | |
*** sdake_ is now known as sdake | 14:54 | |
*** prateek has joined #heat | 14:54 | |
*** beagles is now known as beagles_afk | 14:54 | |
*** manjong has quit IRC | 14:57 | |
*** prateek has quit IRC | 14:57 | |
*** prateek has joined #heat | 14:58 | |
*** jaosorior has joined #heat | 15:05 | |
*** jaosorior has quit IRC | 15:07 | |
*** sorantis has quit IRC | 15:07 | |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Fix using parent_name for Properties https://review.openstack.org/420014 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Add full path for Property https://review.openstack.org/420015 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Fix validation in TranslationRule https://review.openstack.org/420016 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Initial add Translation-in-place mechanism https://review.openstack.org/420017 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Add translation method for Translation https://review.openstack.org/420018 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Add "Replace" handling for translation https://review.openstack.org/420019 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Implement "Add" rule for Translation https://review.openstack.org/420020 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Add "Resolve" rule for Translation https://review.openstack.org/420021 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Enable new translation mechanism https://review.openstack.org/420022 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Add few functional tests for translation https://review.openstack.org/420023 | 15:08 |
openstackgerrit | Peter Razumovsky proposed openstack/heat: Remove unnecessary old translation mechanism https://review.openstack.org/420024 | 15:08 |
*** ayoung has quit IRC | 15:09 | |
*** annegentle has joined #heat | 15:10 | |
*** vijendar_ has joined #heat | 15:10 | |
*** manjong has joined #heat | 15:13 | |
*** vijendar has quit IRC | 15:13 | |
*** jaosorior has joined #heat | 15:16 | |
*** annegentle has quit IRC | 15:16 | |
*** manjong has quit IRC | 15:18 | |
*** jaosorior has quit IRC | 15:19 | |
*** zz_dimtruck is now known as dimtruck | 15:19 | |
*** jaosorior has joined #heat | 15:22 | |
*** rajpatel has joined #heat | 15:27 | |
ramishra | therve: I think that condition is not checcked most of time | 15:28 |
ramishra | if check is false | 15:28 |
therve | ramishra, So that's because of a change in your patch? | 15:29 |
ramishra | I mean https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/api.py#L938-L939 | 15:29 |
*** chris_hultin|AWA is now known as chris_hultin | 15:29 | |
*** spotz_zzz is now known as spotz | 15:30 | |
*** Pavo has joined #heat | 15:30 | |
ramishra | we randomly compare the event_count | 15:30 |
therve | Ah, ok | 15:32 |
zaneb | therve: 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 repo | 15:35 |
zaneb | therve: should we just copy them over? or do something else? | 15:35 |
therve | zaneb, I guess? I only copied the things in elements, I was wondering about the rest | 15:36 |
therve | I thought those were mostly templates, but I guess we package those | 15:38 |
therve | Hum we only package 2 files | 15:40 |
zaneb | yeah | 15:40 |
zaneb | I wonder if those should just be included as sources in the RDO package, or if they do belong in the repo | 15:41 |
*** vijendar_ has quit IRC | 15:42 | |
openstackgerrit | Merged openstack/heat: Don't use MagicMock for stacks in tests https://review.openstack.org/419786 | 15:44 |
*** vijendar has joined #heat | 15:45 | |
*** vijendar_ has joined #heat | 15:46 | |
*** EricGonczer_ has joined #heat | 15:49 | |
*** Matthias__ has quit IRC | 15:49 | |
*** vijendar has quit IRC | 15:49 | |
zaneb | therve: actually, it looks like the package installs the entire contents of hot/software_config in /usr/share | 15:50 |
*** prateek has quit IRC | 15:51 | |
*** Drago has joined #heat | 15:54 | |
*** spzala has quit IRC | 15:55 | |
*** sdake_ has joined #heat | 15:56 | |
*** spzala has joined #heat | 15:56 | |
*** sdake has quit IRC | 15:57 | |
therve | zaneb, In the openstack-heat-templates package though, no? | 15:58 |
zaneb | yes | 15:59 |
therve | Should we keep that one? | 15:59 |
therve | I don't think tripleo installs it for instance | 15:59 |
*** spzala has quit IRC | 16:00 | |
zaneb | good point, ok | 16:01 |
*** spzala has joined #heat | 16:01 | |
zaneb | maybe we shouldn't use Provides then | 16:01 |
therve | RIght, I would just replace the sub packages | 16:02 |
therve | We should probably still copy over those 2 other files though | 16:04 |
zaneb | therve: copy them to the repo, or add them as source files in the package? | 16:14 |
therve | zaneb, What's the latter? | 16:15 |
therve | Could we add source files from another package? | 16:15 |
zaneb | you add it in the SRPM, like you would a patch file | 16:15 |
zaneb | so RDO would own it and no upstream changes would affect it | 16:16 |
*** adrian_otto has joined #heat | 16:16 | |
zaneb | which seems fine for http://git.openstack.org/cgit/openstack/heat-templates/tree/hot/software-config/boot-config/templates/fragments/20-os-apply-config | 16:16 |
zaneb | maybe not so much for http://git.openstack.org/cgit/openstack/heat-templates/tree/hot/software-config/boot-config/templates/fragments/os-collect-config.conf though | 16:16 |
*** stingaci has quit IRC | 16:17 | |
therve | Yeah this file is changed from time to time | 16:17 |
therve | It's also referenced from boot-config elsewhere | 16:17 |
*** dprince has quit IRC | 16:19 | |
Pavo | anyone in here every deploy csr1000v using heat? | 16:20 |
*** dprince has joined #heat | 16:21 | |
*** stingaci has joined #heat | 16:28 | |
*** annegentle has joined #heat | 16:29 | |
*** anton is now known as anton-zzz | 16:30 | |
zaneb | therve: we'll need a LICENSE file in that repo too | 16:31 |
*** sdake_ is now known as sdake_torture_ti | 16:31 | |
*** rajpatel has quit IRC | 16:32 | |
*** stingaci has quit IRC | 16:32 | |
openstackgerrit | Merged openstack/heat-agents: Add 'exec' support to docker cmd https://review.openstack.org/417521 | 16:38 |
*** chlong has quit IRC | 16:38 | |
*** stingaci has joined #heat | 16:41 | |
*** bnemec is now known as beekneemech | 16:43 | |
*** chlong has joined #heat | 16:43 | |
*** lamt has joined #heat | 16:43 | |
openstackgerrit | Zane Bitter proposed openstack/heat-agents: Reinstate LICENSE file https://review.openstack.org/420064 | 16:47 |
openstackgerrit | Zane Bitter proposed openstack/heat-agents: Add boot-config files for os-apply/collect-config https://review.openstack.org/420065 | 16:47 |
*** jose-phillips has joined #heat | 16:47 | |
zaneb | therve: ^ | 16:47 |
*** chlong has quit IRC | 16:48 | |
*** AlexeyAbashkin has quit IRC | 16:49 | |
*** bana_k has joined #heat | 16:50 | |
*** rajpatel has joined #heat | 16:51 | |
*** EricGonc_ has joined #heat | 16:52 | |
*** lamt has quit IRC | 16:52 | |
*** EricGonczer_ has quit IRC | 16:53 | |
*** stingaci has quit IRC | 16:59 | |
*** chlong has joined #heat | 16:59 | |
*** jpich has quit IRC | 17:02 | |
larsks | If '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 IRC | 17:08 | |
*** chlong has quit IRC | 17:09 | |
*** bana_k has quit IRC | 17:15 | |
*** lamt has joined #heat | 17:18 | |
*** dprince_ has joined #heat | 17:20 | |
*** dprince has quit IRC | 17:22 | |
*** adrian_otto has quit IRC | 17:24 | |
*** steveg_afk has quit IRC | 17:24 | |
*** vijendar_ has quit IRC | 17:29 | |
*** shardy has quit IRC | 17:46 | |
*** vijendar has joined #heat | 17:56 | |
*** vijendar_ has joined #heat | 17:57 | |
*** ri0 has joined #heat | 17:58 | |
*** ri0 has quit IRC | 17:58 | |
*** vijendar has quit IRC | 18:00 | |
*** rajpatel has quit IRC | 18:06 | |
*** steveg_afk has joined #heat | 18:08 | |
*** links has quit IRC | 18:12 | |
*** rajpatel has joined #heat | 18:12 | |
*** rajpatel has quit IRC | 18:15 | |
*** bana_k has joined #heat | 18:15 | |
*** spzala has quit IRC | 18:18 | |
*** spzala has joined #heat | 18:22 | |
*** jprovazn has quit IRC | 18:24 | |
*** om2 has joined #heat | 18:28 | |
*** steveg_afk has quit IRC | 18:36 | |
*** chris_hultin is now known as chris_hultin|AWA | 18:56 | |
*** openstack has joined #heat | 19:02 | |
*** jamielennox is now known as jamielennox|away | 19:12 | |
*** mannamne has quit IRC | 19:13 | |
*** chlong has joined #heat | 19:14 | |
*** adrian_otto has joined #heat | 19:15 | |
*** rajpatel has joined #heat | 19:26 | |
openstackgerrit | Merged openstack/heat: Unit tests: Don't mock client_plugin for Zaqar subscription https://review.openstack.org/419584 | 19:26 |
*** thijsferyn has quit IRC | 19:27 | |
*** sdake has joined #heat | 19:34 | |
*** sdake_ has joined #heat | 19:40 | |
*** adrian_otto has quit IRC | 19:43 | |
*** sdake has quit IRC | 19:44 | |
*** spzala has quit IRC | 19:45 | |
*** rajpatel has quit IRC | 19:49 | |
*** dimtruck is now known as zz_dimtruck | 19:52 | |
*** vijendar_ has quit IRC | 19:56 | |
*** openstackstatus has quit IRC | 19:57 | |
*** openstack has joined #heat | 20:02 | |
*** stingaci has quit IRC | 20:04 | |
*** liusheng has quit IRC | 20:11 | |
*** liusheng has joined #heat | 20:11 | |
*** vijendar has joined #heat | 20:11 | |
*** vijendar_ has joined #heat | 20:12 | |
openstackgerrit | Merged openstack/heat: Fix errors resulting from over-long Zaqar queue names https://review.openstack.org/419169 | 20:13 |
*** vijendar has quit IRC | 20:16 | |
*** iurygregory has quit IRC | 20:18 | |
*** spzala has joined #heat | 20:21 | |
*** zz_dimtruck is now known as dimtruck | 20:24 | |
*** steveg_afk has joined #heat | 20:26 | |
*** openstackstatus has quit IRC | 20:27 | |
*** openstack has joined #heat | 20:32 | |
*** chris_hultin is now known as chris_hultin|AWA | 20:42 | |
*** jaosorior has quit IRC | 20:45 | |
*** spotz is now known as spotz_zzz | 20:45 | |
*** jaosorior has joined #heat | 20:45 | |
*** Pavo has quit IRC | 20:47 | |
*** crose has quit IRC | 20:50 | |
*** annegentle has quit IRC | 20:55 | |
*** annegentle has joined #heat | 20:59 | |
zaneb | cwolferh: re the comment I just left on https://review.openstack.org/#/c/418628/2 ... | 21:02 |
*** Pavo has joined #heat | 21:02 | |
cwolferh | zaneb, i was just writing code to avoid always loading the resources :) | 21:02 |
zaneb | I 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 on | 21:02 |
zaneb | I want to run the experimental job on that and see how much memory it saves | 21:03 |
cwolferh | cool | 21:03 |
zaneb | I feel like we need some test to enforce that as well | 21:03 |
zaneb | I can put in an error log for now | 21:04 |
*** spotz_zzz is now known as spotz | 21:06 | |
*** spotz is now known as spotz_zzz | 21:06 | |
cwolferh | zaneb, still needs polish, but my current thinking on the patch: https://github.com/cwolferh/heat-scratch/commit/5519199f647b7220dc14edd29d732e593bd86e1b | 21:10 |
cwolferh | added the load_resources classmethod to Resource, and use that to get the relevant set of resources that we are propagating to. | 21:12 |
zaneb | cwolferh: the idea with convergence was that we shouldn't have to load *any* resources except the one the worker is working on though | 21:13 |
cwolferh | well, it's either that or always propagate all of your attributes to everybody. i don't think you can have it both ways | 21:15 |
*** adrian_otto has joined #heat | 21:16 | |
cwolferh | one 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 IRC | 21:19 | |
zaneb | yes, 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 IRC | 21:20 | |
zaneb | I think we just have to accept that we'll be propagating all of the attribute data to everyone | 21:20 |
zaneb | but at least we won't be retrieving it in a loop any more | 21:20 |
*** adrian_otto has quit IRC | 21:22 | |
cwolferh | zaneb, 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 |
zaneb | we 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 |
zaneb | we just can't know *where* they are referenced | 21:25 |
*** jcoufal has quit IRC | 21:29 | |
*** chris_hultin|AWA is now known as chris_hultin | 21:31 | |
*** chris_hultin is now known as chris_hultin|AWA | 21:32 | |
cwolferh | still 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 definition | 21:32 |
*** vijendar_ has quit IRC | 21:34 | |
*** sdake has joined #heat | 21:35 | |
*** Pavo has joined #heat | 21:41 | |
*** vijendar has joined #heat | 21:41 | |
*** sdake has quit IRC | 21:42 | |
*** sdake has joined #heat | 21:43 | |
*** Pavo has quit IRC | 21:50 | |
*** steveg_afk has quit IRC | 21:52 | |
*** Pavo has joined #heat | 21:53 | |
*** Pavo has quit IRC | 21:53 | |
*** rajpatel has quit IRC | 21:54 | |
*** Pavo has joined #heat | 21:55 | |
zaneb | cwolferh: SELECT id, name FROM Resource WHERE id IN [...] | 21:57 |
zaneb | cwolferh: that would do it. you won't get it from sqlalchemy.orm, but it could be done | 21:58 |
zaneb | whether that trade-off is worth it... only benchmarking can tell | 21:58 |
*** jamielennox|away is now known as jamielennox | 21:59 | |
*** stingaci has joined #heat | 22:00 | |
openstackgerrit | Zane Bitter proposed openstack/heat: Don't load all resources in convergence check https://review.openstack.org/420191 | 22:03 |
cwolferh | zaneb 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 IRC | 22:04 | |
zaneb | yeah... 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 win | 22:05 |
*** rajpatel has joined #heat | 22:08 | |
*** sdake_ has joined #heat | 22:08 | |
*** rajpatel has quit IRC | 22:10 | |
*** sdake has quit IRC | 22:10 | |
cwolferh | when 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 #heat | 22:16 | |
zaneb | cwolferh: the weird thing is that I initially designed it that way but then changed my mind: https://github.com/zaneb/heat-convergence-prototype/commit/3fc02f75167cdc1e250ce5fc7c6b0f0aad0537ac | 22:22 |
zaneb | cwolferh: oh wait, you said attribute names not resource names | 22:22 |
cwolferh | zaneb, yep, that | 22:23 |
cwolferh | an extra tuple | 22:23 |
cwolferh | in the edge definition | 22:24 |
cwolferh | i 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 #heat | 22:25 | |
*** baling-wire is now known as david-lyle | 22:26 | |
*** dprince_ has quit IRC | 22:26 | |
zaneb | agree :D | 22:27 |
*** spzala has quit IRC | 22:29 | |
*** sdake_ has quit IRC | 22:33 | |
*** sdake has joined #heat | 22:35 | |
*** catintheroof has quit IRC | 22:38 | |
*** kraken_ has joined #heat | 22:39 | |
*** annegentle has quit IRC | 22:42 | |
*** rajpatel has joined #heat | 22:48 | |
*** pradk has quit IRC | 23:00 | |
*** rajpatel has quit IRC | 23:00 | |
*** rajpatel has joined #heat | 23:01 | |
*** rajpatel has quit IRC | 23:03 | |
*** jasond has quit IRC | 23:09 | |
*** rajpatel has joined #heat | 23:12 | |
*** kraken_ has quit IRC | 23:15 | |
*** Pavo has quit IRC | 23:21 | |
*** mannamne has quit IRC | 23:27 | |
*** adrian_otto has quit IRC | 23:31 | |
*** jaosorior has quit IRC | 23:31 | |
*** EricGonc_ has quit IRC | 23:37 | |
*** sdake has quit IRC | 23:38 | |
*** chlong has quit IRC | 23:38 | |
*** EricGonczer_ has joined #heat | 23:38 | |
*** harlowja has quit IRC | 23:56 | |
*** vijendar has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!