| *** sdake has quit IRC | 00:00 | |
| *** sdake has joined #heat | 00:02 | |
| *** sdake has quit IRC | 00:03 | |
| *** jistr has quit IRC | 00:03 | |
| *** sdake has joined #heat | 00:04 | |
| *** jistr has joined #heat | 00:07 | |
| *** Pavo has joined #heat | 00:23 | |
| *** sdake has quit IRC | 00:28 | |
| *** adrian_otto has joined #heat | 00:30 | |
| *** ramishra has quit IRC | 00:45 | |
| *** ramishra has joined #heat | 00:47 | |
| *** hoangcx has joined #heat | 00:50 | |
| *** dimtruck is now known as zz_dimtruck | 00:50 | |
| *** zz_dimtruck is now known as dimtruck | 00:50 | |
| *** adrian_otto has quit IRC | 00:52 | |
| *** ramishra has quit IRC | 00:58 | |
| *** dimtruck is now known as zz_dimtruck | 01:00 | |
| *** ramishra has joined #heat | 01:00 | |
| *** ramishra has quit IRC | 01:05 | |
| *** ramishra has joined #heat | 01:07 | |
| *** liusheng has quit IRC | 01:10 | |
| *** liusheng has joined #heat | 01:18 | |
| *** elynn has joined #heat | 01:26 | |
| *** ramishra has quit IRC | 01:30 | |
| *** ramishra has joined #heat | 01:32 | |
| *** yanyanhu has joined #heat | 01:34 | |
| *** ramishra has quit IRC | 01:37 | |
| *** ramishra has joined #heat | 01:39 | |
| openstackgerrit | huangtianhua proposed openstack/heat: No need to check if resource not created https://review.openstack.org/415639 | 01:44 |
|---|---|---|
| *** steveg_afk has joined #heat | 01:49 | |
| *** EricGonczer_ has quit IRC | 01:52 | |
| *** liusheng has quit IRC | 01:54 | |
| openstackgerrit | Merged openstack/heat: Set content-type when signaling deployment https://review.openstack.org/419244 | 02:01 |
| *** steveg_afk has quit IRC | 02:03 | |
| *** ramishra has quit IRC | 02:09 | |
| *** zhihui has joined #heat | 02:10 | |
| *** ramishra has joined #heat | 02:11 | |
| *** liusheng has joined #heat | 02:12 | |
| *** vijendar has joined #heat | 02:17 | |
| *** EricGonczer_ has joined #heat | 02:19 | |
| *** stevemar has quit IRC | 02:21 | |
| *** zz_dimtruck is now known as dimtruck | 02:21 | |
| *** stevemar has joined #heat | 02:21 | |
| *** vijendar has quit IRC | 02:22 | |
| *** links has joined #heat | 02:25 | |
| *** ramishra has quit IRC | 02:32 | |
| *** ramishra has joined #heat | 02:34 | |
| *** links has quit IRC | 02:38 | |
| *** links has joined #heat | 02:46 | |
| *** ramishra has quit IRC | 03:00 | |
| *** ramishra has joined #heat | 03:02 | |
| *** wangbo has joined #heat | 03:05 | |
| *** bana_k has joined #heat | 03:16 | |
| *** bana_k has quit IRC | 03:29 | |
| *** wllabs has joined #heat | 03:32 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Refactor _show_resource() for neutron resources https://review.openstack.org/419851 | 03:44 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for neutron resources https://review.openstack.org/420262 | 03:44 |
| *** KanagarajM has joined #heat | 03:53 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Release notes for designate v2 support https://review.openstack.org/417043 | 03:54 |
| *** EricGonczer_ has quit IRC | 03:57 | |
| *** ramishra has quit IRC | 04:05 | |
| *** ramishra has joined #heat | 04:07 | |
| *** ramishra has quit IRC | 04:08 | |
| *** elynn has quit IRC | 04:11 | |
| *** vijendar has joined #heat | 04:18 | |
| wllabs | scaling policy has a propoerties number is 2 | 04:19 |
| wllabs | create a receiver-create -P count=10 | 04:20 |
| wllabs | if triggering reciver, num of node what the cluster scaling node | 04:21 |
| *** manjong has joined #heat | 04:22 | |
| *** adrian_otto has joined #heat | 04:23 | |
| *** vijendar has quit IRC | 04:23 | |
| *** adrian_otto has quit IRC | 04:24 | |
| *** adrian_otto has joined #heat | 04:25 | |
| *** adrian_otto has quit IRC | 04:33 | |
| *** adrian_otto has joined #heat | 04:46 | |
| *** dimtruck is now known as zz_dimtruck | 04:49 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Refactor _show_resource() for neutron resources https://review.openstack.org/419851 | 05:02 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for neutron resources https://review.openstack.org/420262 | 05:02 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for senlin resources https://review.openstack.org/420500 | 05:02 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for senlin resources https://review.openstack.org/420500 | 05:04 |
| *** adrian_otto has quit IRC | 05:11 | |
| *** KanagarajM has quit IRC | 05:20 | |
| *** elynn has joined #heat | 05:24 | |
| *** prateek has joined #heat | 05:28 | |
| openstackgerrit | Merged openstack/python-heatclient: Enable coverage report in console output https://review.openstack.org/418649 | 05:29 |
| *** sgordon has quit IRC | 05:31 | |
| *** sgordon has joined #heat | 05:39 | |
| *** fandi has joined #heat | 05:49 | |
| *** manjong has quit IRC | 06:02 | |
| *** manjong has joined #heat | 06:05 | |
| *** adrian_otto has joined #heat | 06:06 | |
| *** manjong has quit IRC | 06:10 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/heat: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/420509 | 06:11 |
| *** adrian_otto has quit IRC | 06:14 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/heat: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/420509 | 06:15 |
| *** adrian_otto has joined #heat | 06:16 | |
| *** adrian_otto has quit IRC | 06:16 | |
| *** vijendar has joined #heat | 06:19 | |
| *** adrian_otto has joined #heat | 06:21 | |
| *** vijendar has quit IRC | 06:25 | |
| *** manjong has joined #heat | 06:26 | |
| *** adrian_otto has quit IRC | 06:33 | |
| *** manjong has quit IRC | 06:34 | |
| *** manjong has joined #heat | 06:35 | |
| *** vijendar has joined #heat | 06:42 | |
| *** vijendar has quit IRC | 06:47 | |
| *** manjong has quit IRC | 06:50 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for senlin resources https://review.openstack.org/420500 | 06:51 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for neutron resources https://review.openstack.org/420262 | 06:51 |
| *** Drago has quit IRC | 06:51 | |
| *** afazekas has quit IRC | 06:53 | |
| *** afazekas has joined #heat | 06:57 | |
| *** afazekas has quit IRC | 07:02 | |
| *** afazekas has joined #heat | 07:13 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Don't override _show_resource() for image https://review.openstack.org/420532 | 07:27 |
| *** jprovazn has joined #heat | 07:27 | |
| *** fandi has quit IRC | 07:35 | |
| *** ricolin has joined #heat | 07:44 | |
| *** zhurong has joined #heat | 08:05 | |
| *** stingaci has joined #heat | 08:07 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Improve resource.NoActionRequired https://review.openstack.org/420558 | 08:11 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for neutron resources https://review.openstack.org/420262 | 08:13 |
| openstackgerrit | huangtianhua proposed openstack/heat: Apply super _show_resource() for senlin resources https://review.openstack.org/420500 | 08:15 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/heat-templates: Replace yaml.load() with yaml.safe_load() https://review.openstack.org/420562 | 08:20 |
| *** shardy has joined #heat | 08:24 | |
| *** trananhkma has joined #heat | 08:30 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Avoid use internal attribut '_info' https://review.openstack.org/420577 | 08:40 |
| openstackgerrit | huangtianhua proposed openstack/heat: Avoid to use internal attribute '_info' https://review.openstack.org/420577 | 08:42 |
| *** wllabs_ has joined #heat | 08:48 | |
| *** jpich has joined #heat | 08:49 | |
| *** wllabs has quit IRC | 08:49 | |
| *** wllabs_ is now known as wllabs | 08:50 | |
| *** sdake has joined #heat | 08:52 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #heat | 09:00 | |
| *** sdake_ has joined #heat | 09:00 | |
| *** sdake has quit IRC | 09:02 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Remove handle_delete() for keystone group https://review.openstack.org/420601 | 09:26 |
| *** dbecker has joined #heat | 09:29 | |
| *** yassine has joined #heat | 09:32 | |
| *** yassine is now known as Guest7011 | 09:32 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Improve resource.NoActionRequired https://review.openstack.org/420558 | 09:32 |
| robcresswell | Hey guys; when can we expect a new heatclient release? | 09:34 |
| *** sshnaidm|off is now known as sshnaidm | 09:35 | |
| *** zhurong has quit IRC | 09:37 | |
| *** derekh has joined #heat | 09:40 | |
| *** sgordon has quit IRC | 09:41 | |
| *** sdake_ has quit IRC | 09:41 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Allow admins to get resource by physical resource id https://review.openstack.org/411696 | 09:42 |
| *** sgordon has joined #heat | 09:49 | |
| *** ccamacho|lunch is now known as ccamacho | 09:57 | |
| *** nilles has joined #heat | 10:06 | |
| *** zhurong has joined #heat | 10:09 | |
| *** hoangcx has quit IRC | 10:14 | |
| *** yanyanhu has quit IRC | 10:34 | |
| *** Guest7011 has quit IRC | 10:37 | |
| *** roughiz has joined #heat | 10:43 | |
| *** vijendar has joined #heat | 10:43 | |
| *** roughiz has quit IRC | 10:44 | |
| *** roughiz has joined #heat | 10:45 | |
| *** blues-man has joined #heat | 10:45 | |
| *** Guest7011 has joined #heat | 10:45 | |
| openstackgerrit | Rico Lin proposed openstack/heat: Allow multi-layer global environment/template directory https://review.openstack.org/298054 | 11:09 |
| openstackgerrit | Rico Lin proposed openstack/heat: Allow load global template for template resouce https://review.openstack.org/209439 | 11:09 |
| *** ricolin has quit IRC | 11:11 | |
| *** vijendar has quit IRC | 11:12 | |
| *** sshnaidm is now known as sshnaidm|afk | 11:24 | |
| *** blues-man has quit IRC | 11:36 | |
| openstackgerrit | Giulio Fidente proposed openstack/heat: Add initial support for execution of Mistral workflows via Heat resource https://review.openstack.org/420664 | 11:40 |
| openstackgerrit | Giulio Fidente proposed openstack/heat: Add initial support for execution of Mistral workflows via Heat resource https://review.openstack.org/420664 | 11:44 |
| openstackgerrit | Giulio Fidente proposed openstack/heat: Add initial support for execution of Mistral workflows via Heat resource https://review.openstack.org/420664 | 11:45 |
| *** winggundamth has joined #heat | 11:51 | |
| winggundamth | hi all | 11:51 |
| winggundamth | I just wonder how can I do something like this port_id: { get_attr: [instance, addresses, { get_param: external_network_name }, 0, port] } | 11:51 |
| winggundamth | can we use get_param inside get_attr? | 11:52 |
| *** om2 has quit IRC | 11:55 | |
| *** wllabs_ has joined #heat | 12:00 | |
| *** wllabs has quit IRC | 12:01 | |
| *** wllabs_ is now known as wllabs | 12:01 | |
| *** jprovazn has quit IRC | 12:06 | |
| *** jprovazn has joined #heat | 12:07 | |
| therve | winggundamth, Yes | 12:11 |
| winggundamth | therve: is this the correct syntax? port_id: { get_attr: [instance, addresses, { get_param: external_network_name }, 0, port] } | 12:12 |
| therve | Looks like it | 12:12 |
| winggundamth | therve: thanks. so maybe I have done something wrong on the others | 12:13 |
| therve | winggundamth, What's happening? Maybe you should start with that :) | 12:13 |
| *** steveg_afk has joined #heat | 12:13 | |
| winggundamth | therve: grrrr. I don't know why but I just deleted everything and build the stack again. it works -_- | 12:14 |
| therve | Hooray | 12:14 |
| winggundamth | therve: check the whole template file here https://github.com/opsta/openstack-heat-workshop/blob/master/workshop/05-parameters.yml | 12:17 |
| *** winggundamth has quit IRC | 12:34 | |
| *** gfidente has joined #heat | 12:37 | |
| therve | gfidente, Thanks | 12:38 |
| therve | So I had a quick look | 12:38 |
| gfidente | therve hey thanks | 12:38 |
| therve | I think it doesn't answer our main concern, which is being able to trigger a workflow based on stack/resource operations | 12:38 |
| therve | gfidente, Are you aware of https://review.openstack.org/267770 ? | 12:38 |
| gfidente | going through that now | 12:40 |
| gfidente | oh so I basically implemented what was described as phase of in the blueprint it seems | 12:41 |
| therve | Your implementation is totally correct, there is no issue about that | 12:41 |
| therve | That's possible yeah | 12:41 |
| therve | The problem is that the spec is not what we want yet :) | 12:42 |
| gfidente | sec, reading the comments now | 12:42 |
| therve | gfidente, http://lists.openstack.org/pipermail/openstack-dev/2016-December/109089.html and the rest of the thread highlights that, too | 12:44 |
| *** blues-man has joined #heat | 12:47 | |
| *** vijendar has joined #heat | 12:48 | |
| gfidente | therve so you're suggesting that | 12:49 |
| gfidente | we should rather have a list of actions for the existing workflow resource and trigger execution when action is matched? | 12:50 |
| therve | gfidente, No I think it deserves a new resource. But that resource needs some kind of actions | 12:50 |
| gfidente | ok so to clear things up for me a bit | 12:52 |
| *** vijendar has quit IRC | 12:53 | |
| gfidente | if in the template an execution resource refers to a workflow via get_resource, the execution will be deleted/recreated if the parent workflow is updated | 12:53 |
| gfidente | is the above correct? | 12:53 |
| therve | That depends on your resource implementation | 12:56 |
| gfidente | because that was my goal | 12:57 |
| therve | Yeah that may be interesting | 13:02 |
| therve | But I don't think that's the main issue we have | 13:02 |
| therve | What we want is being to control whether the workflow executes on stack update | 13:02 |
| therve | Regardless of the changes on the workflow itself | 13:02 |
| *** zhurong has quit IRC | 13:03 | |
| gfidente | okay and to get that, looking at software_deployment, seems like I could add a generic _handle_action and use that matching against the desired actions for all handle_ requests? | 13:05 |
| *** beagles_afk is now known as beagles | 13:08 | |
| therve | Right that ought to look very similar | 13:08 |
| gfidente | ok thanks | 13:08 |
| gfidente | that sounds cool to me | 13:08 |
| therve | It is! Ultimately it will be nice | 13:09 |
| gfidente | I will ping the blueprint author to see if they want to coop | 13:09 |
| gfidente | maybe we should revive that to describe the intents | 13:10 |
| gfidente | better | 13:10 |
| gfidente | (and the implementation) | 13:10 |
| therve | That'd be nice, yeah. We haven't seen him around for a while, though | 13:10 |
| *** ccamacho is now known as ccamacho|lunch | 13:10 | |
| *** Guest7011 has quit IRC | 13:13 | |
| *** Guest7011 has joined #heat | 13:20 | |
| *** KanagarajM has joined #heat | 13:22 | |
| dims | therve : ramishra : https://review.openstack.org/#/c/420443/ should now show more of the heat related (py35) failures. ive fixed quite a bit of the nova failures etc. | 13:30 |
| *** sshnaidm|afk is now known as sshnaidm | 13:30 | |
| therve | dims, There are none? | 13:32 |
| therve | oh duh | 13:32 |
| therve | sorry didn't spot | 13:33 |
| dims | therve : was looking at "... FAILED" in console.html | 13:33 |
| therve | Yeah you're right | 13:34 |
| therve | dims, It's still that weird locale error in sqlalchemy | 13:35 |
| dims | therve : which file are you looking at? so i can tell someone else to dig into the same spot | 13:36 |
| therve | dims, http://logs.openstack.org/43/420443/1/check/gate-heat-dsvm-functional-orig-mysql-lbaasv2-py35-ubuntu-xenial-nv/d858544/logs/screen-h-eng.txt.gz | 13:36 |
| therve | (warning, big) | 13:36 |
| dims | thanks therve | 13:36 |
| *** jdandrea has joined #heat | 13:39 | |
| *** jprovazn has quit IRC | 13:39 | |
| *** sdake has joined #heat | 13:39 | |
| openstackgerrit | Dan Prince proposed openstack/heat-agents: Fix: all containers are being set privileged https://review.openstack.org/420721 | 13:42 |
| *** EricGonczer_ has joined #heat | 13:43 | |
| openstackgerrit | Flavio Percoco proposed openstack/heat-agents: Run non-detached containers first https://review.openstack.org/420723 | 13:46 |
| therve | dims, It seems that the issue is in pymysql | 13:49 |
| therve | Or possibly oslo.i18n | 13:49 |
| dims | therve : y, going to pass this traceback around (http://paste.openstack.org/show/595050/) | 13:50 |
| *** EricGonczer_ has quit IRC | 13:50 | |
| openstackgerrit | Giulio Fidente proposed openstack/heat: Implements execution of Mistral workflows using a Heat resource https://review.openstack.org/420664 | 13:56 |
| *** dprince has joined #heat | 13:59 | |
| *** jcoufal has joined #heat | 14:02 | |
| openstackgerrit | Dan Prince proposed openstack/heat-agents: Add exit_codes option to docker-cmd hook https://review.openstack.org/420729 | 14:04 |
| *** KanagarajM has quit IRC | 14:06 | |
| *** crose has joined #heat | 14:06 | |
| *** prateek has quit IRC | 14:11 | |
| *** paw has quit IRC | 14:13 | |
| *** ccamacho|lunch is now known as ccamacho | 14:18 | |
| *** pradk has joined #heat | 14:18 | |
| *** jcoufal_ has joined #heat | 14:18 | |
| *** jcoufal has quit IRC | 14:20 | |
| *** jprovazn has joined #heat | 14:21 | |
| *** Guest7011 has quit IRC | 14:28 | |
| *** links has quit IRC | 14:30 | |
| *** jcoufal has joined #heat | 14:38 | |
| *** KanagarajM has joined #heat | 14:39 | |
| *** jcoufal_ has quit IRC | 14:39 | |
| *** Guest7011 has joined #heat | 14:42 | |
| *** KanagarajM has quit IRC | 14:43 | |
| *** manjong has joined #heat | 14:52 | |
| *** vijendar has joined #heat | 14:53 | |
| *** dyasny has quit IRC | 14:56 | |
| *** dyasny has joined #heat | 14:57 | |
| *** KanagarajM has joined #heat | 14:57 | |
| *** paw has joined #heat | 14:57 | |
| *** beagles is now known as beagles_brb | 14:57 | |
| *** vijendar has quit IRC | 14:59 | |
| *** wllabs has quit IRC | 14:59 | |
| *** KanagarajM has quit IRC | 14:59 | |
| *** KanagarajM has joined #heat | 14:59 | |
| *** wllabs has joined #heat | 14:59 | |
| *** spotz_zzz is now known as spotz | 15:01 | |
| *** chris_hultin|AWA is now known as chris_hultin | 15:11 | |
| *** zz_dimtruck is now known as dimtruck | 15:14 | |
| *** KanagarajM has quit IRC | 15:20 | |
| *** KanagarajM has joined #heat | 15:20 | |
| *** jaosorior has joined #heat | 15:25 | |
| *** beekneemech is now known as bnemec | 15:26 | |
| *** frippe75 has joined #heat | 15:27 | |
| *** EricGonczer_ has joined #heat | 15:28 | |
| *** blues-man has quit IRC | 15:28 | |
| *** KanagarajM has quit IRC | 15:32 | |
| *** blomquisg_gone is now known as blomquisg | 15:32 | |
| *** KanagarajM has joined #heat | 15:33 | |
| *** sshnaidm has quit IRC | 15:36 | |
| *** sshnaidm has joined #heat | 15:36 | |
| *** jdandrea has quit IRC | 15:39 | |
| *** KanagarajM has quit IRC | 15:40 | |
| *** blues-man has joined #heat | 15:40 | |
| *** blues-man has quit IRC | 15:40 | |
| *** blues-man has joined #heat | 15:40 | |
| *** KanagarajM has joined #heat | 15:40 | |
| *** beagles_brb is now known as beagles | 15:41 | |
| *** nilles has quit IRC | 15:43 | |
| *** KanagarajM has quit IRC | 15:43 | |
| *** rajpatel has joined #heat | 15:44 | |
| *** KanagarajM has joined #heat | 15:44 | |
| *** jdandrea has joined #heat | 15:45 | |
| *** jasond has joined #heat | 15:48 | |
| *** stingaci has quit IRC | 15:48 | |
| *** stingaci has joined #heat | 15:48 | |
| *** Drago has joined #heat | 15:49 | |
| *** KanagarajM has quit IRC | 15:52 | |
| *** KanagarajM has joined #heat | 15:53 | |
| *** spotz is now known as spotz_zzz | 15:55 | |
| *** Drago has quit IRC | 15:55 | |
| *** vijendar has joined #heat | 15:55 | |
| *** randallburt has joined #heat | 15:55 | |
| *** crose has quit IRC | 15:56 | |
| *** spotz_zzz is now known as spotz | 15:57 | |
| *** randallburt has quit IRC | 16:00 | |
| *** Drago has joined #heat | 16:01 | |
| *** randallburt has joined #heat | 16:09 | |
| *** jasond has quit IRC | 16:10 | |
| *** crose has joined #heat | 16:11 | |
| *** rajpatel has quit IRC | 16:14 | |
| *** manjong has quit IRC | 16:14 | |
| openstackgerrit | Flavio Percoco proposed openstack/heat-agents: Run non-detached containers first https://review.openstack.org/420723 | 16:17 |
| *** rajpatel has joined #heat | 16:18 | |
| *** blues-man has quit IRC | 16:21 | |
| *** jdandrea has quit IRC | 16:23 | |
| *** crose has quit IRC | 16:23 | |
| *** manjong has joined #heat | 16:25 | |
| *** manjong has quit IRC | 16:30 | |
| *** adrian_otto has joined #heat | 16:31 | |
| *** sdake has quit IRC | 16:33 | |
| *** KanagarajM has quit IRC | 16:34 | |
| *** KanagarajM has joined #heat | 16:34 | |
| *** jdandrea has joined #heat | 16:36 | |
| *** jose-phillips has joined #heat | 16:52 | |
| *** jose-phi_ has joined #heat | 16:54 | |
| *** jose-phillips has quit IRC | 16:54 | |
| cwolferh | looking for reviews for https://review.openstack.org/#/c/363415/ . i have patches ready for related patch sets that have merge conflicts once that one is merged | 16:56 |
| therve | cwolferh, Why does it move away from transaction in purge? | 17:01 |
| cwolferh | therve, off the top of my head, too many operations in one transaction, will likely end up db deadlock issues | 17:02 |
| cwolferh | i know its ironic, since i tried hard to get the transaction working in the previous patch, but now it is too much :/ | 17:03 |
| *** dprince has quit IRC | 17:04 | |
| *** dprince has joined #heat | 17:05 | |
| therve | OK :) | 17:05 |
| openstackgerrit | Rico Lin proposed openstack/heat: Brings Magnum resource up to date with Cluster https://review.openstack.org/374321 | 17:06 |
| *** jcoufal_ has joined #heat | 17:07 | |
| cwolferh | therve, for reference, when it was in a transaction i tried adding this to deal with it: https://review.openstack.org/#/c/397938/ . but probably better to avoid it | 17:08 |
| * therve nods | 17:08 | |
| *** dave-johnston has joined #heat | 17:09 | |
| *** jcoufal has quit IRC | 17:09 | |
| *** gfidente is now known as gfidente|afk | 17:17 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/heat: [WIP] python3 experiment - ignore me https://review.openstack.org/420443 | 17:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/420833 | 17:18 |
| *** stingaci has quit IRC | 17:19 | |
| *** stingaci has joined #heat | 17:23 | |
| *** jose-phi_ has quit IRC | 17:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-heatclient: Updated from global requirements https://review.openstack.org/420854 | 17:27 |
| *** stingaci has quit IRC | 17:28 | |
| *** jdandrea has quit IRC | 17:28 | |
| *** jpich has quit IRC | 17:29 | |
| *** stingaci has joined #heat | 17:29 | |
| *** jose-phillips has joined #heat | 17:29 | |
| *** stingaci has quit IRC | 17:30 | |
| *** stingaci has joined #heat | 17:30 | |
| *** rajpatel has quit IRC | 17:32 | |
| *** KanagarajM has quit IRC | 17:33 | |
| *** stingaci has quit IRC | 17:35 | |
| *** jose-phillips has quit IRC | 17:35 | |
| *** jose-phillips has joined #heat | 17:36 | |
| *** jose-phi_ has joined #heat | 17:40 | |
| *** jose-phillips has quit IRC | 17:43 | |
| *** roughiz has quit IRC | 17:44 | |
| *** adrian_otto1 has joined #heat | 17:47 | |
| *** derekh has quit IRC | 17:48 | |
| *** adrian_otto has quit IRC | 17:51 | |
| *** dprince has quit IRC | 17:53 | |
| *** dprince has joined #heat | 17:56 | |
| *** spzala has joined #heat | 17:59 | |
| *** jdandrea has joined #heat | 18:02 | |
| *** jcoufal_ has quit IRC | 18:03 | |
| *** bana_k has joined #heat | 18:05 | |
| *** jcoufal has joined #heat | 18:05 | |
| *** dbecker has quit IRC | 18:09 | |
| *** crose has joined #heat | 18:12 | |
| *** bank_ has joined #heat | 18:17 | |
| *** bana_k has quit IRC | 18:19 | |
| *** shardy has quit IRC | 18:24 | |
| *** bank_ has quit IRC | 18:29 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/heat: [WIP] python3 experiment - ignore me https://review.openstack.org/420443 | 18:29 |
| *** dprince has quit IRC | 18:36 | |
| *** elynn has quit IRC | 19:02 | |
| *** elynn_ has joined #heat | 19:02 | |
| *** dprince has joined #heat | 19:04 | |
| *** rajpatel has joined #heat | 19:06 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/heat: [WIP] python3 experiment - ignore me https://review.openstack.org/420443 | 19:10 |
| *** bana_k has joined #heat | 19:11 | |
| openstackgerrit | Crag Wolfe proposed openstack/heat: Don't load all resources in convergence check https://review.openstack.org/420191 | 19:25 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/heat: [WIP] python3 experiment - ignore me https://review.openstack.org/420443 | 19:36 |
| *** paw has quit IRC | 19:40 | |
| *** om2 has joined #heat | 19:49 | |
| zaneb | ha | 19:50 |
| zaneb | cwolferh: so get this... | 19:51 |
| zaneb | the input data from the sync point is stored in the stack, and it gets substituted in when you call e.g. Resource.FnGetRefId() http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/resource.py#n2039 | 19:52 |
| zaneb | so in order to use the cached data, you have to load all of the resources from the database, then call a method on them so it can ignore the data it just loaded from the database and use the cached data you had already | 19:53 |
| zaneb | I have a plan to attack this and also fix some other longstanding architecture problems, but it's going to take a few days at least | 19:55 |
| cwolferh | zaneb, ah, hmm. i was pretty sure i saw cases in Resource.FnGetAtt where loading the cached data worked without loading all the resources | 19:56 |
| *** sdake has joined #heat | 19:57 | |
| cwolferh | not sure why .FnGetRefId would be different | 19:58 |
| zaneb | cwolferh: by the time you're calling Resource.FnGetAtt/Resource.FnGetRefId (in fact, Resource.anything), you've already lost | 19:59 |
| cwolferh | zaneb, that seems likely :) | 20:05 |
| *** bana_k has quit IRC | 20:07 | |
| cwolferh | zaneb, actually, i don't think it is that bad. http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/resource.py#n263 | 20:08 |
| zaneb | cwolferh: ah, you're right | 20:09 |
| zaneb | cwolferh: but that's only for resources where we have the data? so it will load all the *rest* (the ones you'll never use) from the DB when you do stack.resources... | 20:10 |
| *** bana_k has joined #heat | 20:10 | |
| zaneb | I think? | 20:10 |
| *** chris_hultin is now known as chris_hultin|AWA | 20:11 | |
| cwolferh | zaneb, ah, i think you are right | 20:11 |
| *** spzala has quit IRC | 20:14 | |
| larsks | Is there a way to generate a signed url for externally checking the state of a waitcondition (rather than setting it)? | 20:16 |
| zaneb | cwolferh: took that check out, now getting KeyErrors. I wonder if that was masking some other issues as well | 20:24 |
| zaneb | larsks: no but, more importantly, why? | 20:25 |
| larsks | zaneb, mostly curiosity. | 20:26 |
| zaneb | cwolferh: oh, it's because of the resource actually being checked, never mind | 20:31 |
| * cwolferh deletes what i was writing :-) | 20:31 | |
| cwolferh | zaneb, i suspect we could get away with just not loading any resources from the db if cache_data is not none | 20:32 |
| zaneb | cwolferh: yeah, that's what I'm trying | 20:33 |
| *** jose-phi_ has quit IRC | 20:33 | |
| *** jose-phillips has joined #heat | 20:35 | |
| *** adrian_otto1 has quit IRC | 20:40 | |
| *** Guest7011 has quit IRC | 20:46 | |
| *** spzala has joined #heat | 20:49 | |
| *** charlesr has joined #heat | 20:49 | |
| *** jcoufal has quit IRC | 20:50 | |
| zaneb | cwolferh: ok, fixed it I think. I'll run some tests + then upload patches | 20:55 |
| *** rajpatel has quit IRC | 20:55 | |
| *** chris_hultin|AWA is now known as chris_hultin | 20:58 | |
| *** jamielennox|away is now known as jamielennox | 20:59 | |
| *** chris_hultin is now known as chris_hultin|AWA | 21:07 | |
| *** dimtruck is now known as zz_dimtruck | 21:10 | |
| *** jose-phillips has quit IRC | 21:12 | |
| *** jose-phillips has joined #heat | 21:13 | |
| *** crose has quit IRC | 21:15 | |
| *** jprovazn has quit IRC | 21:15 | |
| *** charlesr has quit IRC | 21:20 | |
| *** sdake_ has joined #heat | 21:23 | |
| *** sdake has quit IRC | 21:24 | |
| *** paw has joined #heat | 21:25 | |
| *** jdandrea has quit IRC | 21:27 | |
| *** sdake_ has quit IRC | 21:27 | |
| *** sdake has joined #heat | 21:28 | |
| *** jdandrea has joined #heat | 21:31 | |
| *** randallburt has quit IRC | 21:31 | |
| *** zz_dimtruck is now known as dimtruck | 21:32 | |
| *** adrian_otto has joined #heat | 21:33 | |
| *** frippe75 has quit IRC | 21:36 | |
| *** sdake has quit IRC | 21:41 | |
| *** randallburt has joined #heat | 21:43 | |
| *** blomquisg is now known as blomquisg_gone | 21:43 | |
| *** blomquisg_gone is now known as blomquisg | 21:45 | |
| *** dfflanders has joined #heat | 21:52 | |
| *** vijendar has quit IRC | 21:57 | |
| *** sdake has joined #heat | 21:57 | |
| *** dprince has quit IRC | 21:57 | |
| openstackgerrit | Zane Bitter proposed openstack/heat: Don't load non-referenced resources from DB https://review.openstack.org/420971 | 21:58 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Don't load resources in dep_attrs https://review.openstack.org/420972 | 21:58 |
| zaneb | cwolferh: ^ first one there should fix it | 21:59 |
| zaneb | cwolferh: although it won't help memory usage as much as I was hoping, because we still create all of the Resource objects | 22:00 |
| cwolferh | zaneb, first one looks good to me | 22:01 |
| zaneb | second one doesn't make any meaningful difference to the bug, it's just a tidy-up really | 22:02 |
| cwolferh | OK, was staring at that wondering if it was fixing something :) | 22:03 |
| zaneb | I also abandoned the one from Friday, which was in the same boat, since it would have made things worse in the short term | 22:05 |
| cwolferh | of course, the other approach would not to be to not instantiate a resource at all in Stack.resources if cache_data is not None and the resource is not in cache_data, but i wouldn't be surprised if that breaks something else. anyways, first one is a lot better than nothing | 22:07 |
| *** gfidente|afk has quit IRC | 22:07 | |
| *** jdandrea has quit IRC | 22:08 | |
| zaneb | cwolferh: that might actually work | 22:08 |
| zaneb | I can't think of a reason we would be accessing any other resources during check_resource | 22:09 |
| cwolferh | me neither | 22:10 |
| zaneb | the main place I was seeing asserts was when resolving the definition of the resource being checked, so that only has references to the resources its referencing | 22:10 |
| *** sdake_ has joined #heat | 22:12 | |
| zaneb | cwolferh: my long term plan is to write a StackDefinition class, and pass that object to intrinsic functions instead of the Stack object | 22:13 |
| *** spzala has quit IRC | 22:13 | |
| zaneb | and it can do something TBD but much lighter-weight when functions try to access resources through it | 22:13 |
| *** sdake has quit IRC | 22:14 | |
| zaneb | in the short term though definitely feel free to try out not instantiating resources that aren't in cache_data | 22:15 |
| cwolferh | makes sense, probably better to avoid passing around stack so much | 22:16 |
| cwolferh | ok, will see what happens :) | 22:16 |
| zaneb | yeah, it's a sucky interface because 3rd party plugin authors have no idea what's supposed to be stable and what's not | 22:17 |
| *** dimtruck is now known as zz_dimtruck | 22:17 | |
| zaneb | tbh *we* have no clear idea | 22:17 |
| *** zz_dimtruck is now known as dimtruck | 22:18 | |
| cwolferh | glad i'm not the only one | 22:19 |
| *** stingaci has joined #heat | 22:20 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/heat: [WIP] python3 experiment - ignore me https://review.openstack.org/420443 | 22:20 |
| cwolferh | zaneb, fwiw, we still don't care about stack output-related attrs being passed around since check_resource doesn't care for its particular resource at it needs to resolve attributes for. but, we would want to pass them once we start caching stack outputs. | 22:22 |
| cwolferh | part of the reason i'm mentioning that is if not instantiating all resources in stack.resource breaks, it'll probably be because of stack outputs (not sure yet) | 22:22 |
| zaneb | it shouldn't break because we're not actually doing *anything* with that data when it comes to calculating outputs afaict | 22:23 |
| zaneb | I agree we should just leave that data out for now | 22:24 |
| *** stingaci has quit IRC | 22:24 | |
| zaneb | later we'll want to ensure any attributes used by outputs are included in the set of attributes whose values we request from the resource, but we'll not want to pass all of those values to other resources, just to wherever we're storing data for outputs | 22:25 |
| zaneb | which is looking increasingly like it should be in the resource itself, not in the sync point | 22:26 |
| cwolferh | i think the sync_point makes sense until the stack is COMPLETE | 22:26 |
| cwolferh | at which point copy it | 22:26 |
| *** dimtruck is now known as zz_dimtruck | 22:28 | |
| *** zz_dimtruck is now known as dimtruck | 22:29 | |
| cwolferh | i'm worried of some race about two traversals trying to write to the same resource which i don't have to worry about with sync_point, but maybe i worry too much. need to think about it more | 22:30 |
| *** vijendar has joined #heat | 22:35 | |
| *** chris_hultin|AWA is now known as chris_hultin | 22:36 | |
| *** vijendar has quit IRC | 22:39 | |
| *** jaosorior has quit IRC | 22:41 | |
| *** vijendar has joined #heat | 22:43 | |
| *** vijendar has quit IRC | 22:44 | |
| *** vijendar has joined #heat | 22:44 | |
| *** vijendar has quit IRC | 22:55 | |
| *** spzala has joined #heat | 23:14 | |
| *** spzala has quit IRC | 23:18 | |
| *** jose-phillips has quit IRC | 23:23 | |
| *** jose-phillips has joined #heat | 23:29 | |
| *** chris_hultin is now known as chris_hultin|AWA | 23:31 | |
| zaneb | cwolferh: in phase 1 resources are locked by a traversal, so there's no possibility of getting 2 writes simultaneously. plus we're already writing to that table so we get it ~for free | 23:33 |
| *** dimtruck is now known as zz_dimtruck | 23:35 | |
| cwolferh | zaneb, ok, so doesn't that mean all sync_point input_data could just be written to resources right now instead of sync_point.input_data? | 23:36 |
| *** spzala has joined #heat | 23:36 | |
| cwolferh | eh, nevermind | 23:36 |
| zaneb | it would, but that would require the resources that the sync points are for to load their dependencies from the DB | 23:37 |
| *** chris_hultin|AWA is now known as chris_hultin | 23:39 | |
| cwolferh | ok, so supposed resA and resB have attributes used by stackA's outputs, and resB has an attribute used by resC... we still want resB to send its input_data to resC. but stackA can just go and look at resA and resB db attribute data whenever it wants to reference it for the sake of calculating its outputs | 23:40 |
| *** randallburt has quit IRC | 23:44 | |
| *** dave-johnston has quit IRC | 23:55 | |
| *** vijendar has joined #heat | 23:56 | |
| *** elynn_ has quit IRC | 23:57 | |
| *** adrian_otto has quit IRC | 23:58 | |
| *** steveg_afk has quit IRC | 23:59 | |
| *** EricGonczer_ has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!