| *** harlowja has quit IRC | 01:07 | |
| *** thrash is now known as thrash|g0ne | 01:26 | |
| *** hardikjasani has joined #openstack-mistral | 04:06 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 05:00 | |
| *** AlexeyAbashkin has quit IRC | 05:17 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 05:41 | |
| *** AlexeyAbashkin has quit IRC | 05:44 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 05:44 | |
| openstackgerrit | Merged openstack/mistral master: Replace Chinese quotes with English quotes https://review.openstack.org/570142 | 05:55 |
|---|---|---|
| *** AlexeyAbashkin has quit IRC | 05:59 | |
| *** jaosorior has joined #openstack-mistral | 06:14 | |
| openstackgerrit | Merged openstack/mistral master: Release note for adding YAQL engine options https://review.openstack.org/571481 | 06:31 |
| *** gkadam has quit IRC | 07:02 | |
| *** shardy has joined #openstack-mistral | 07:58 | |
| *** jpich has joined #openstack-mistral | 08:03 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 08:05 | |
| *** gkadam has joined #openstack-mistral | 08:21 | |
| openstackgerrit | Andras Kovi proposed openstack/mistral master: Use register_auth_conf_options API https://review.openstack.org/572300 | 08:31 |
| *** opetrenko_ has quit IRC | 08:59 | |
| *** lxkong has quit IRC | 09:09 | |
| *** opetrenko_ has joined #openstack-mistral | 09:15 | |
| *** lxkong has joined #openstack-mistral | 09:32 | |
| openstackgerrit | Tomer Shtilman proposed openstack/mistral master: Generate default source execution id https://review.openstack.org/572317 | 09:33 |
| openstackgerrit | Oleksiy Petrenko proposed openstack/mistral-specs master: New more user friendly documentation for Mistral https://review.openstack.org/571710 | 09:42 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-dashboard master: Fix the README formatting https://review.openstack.org/572330 | 10:11 |
| d0ugal | apetrich: ^ getting blocked by the mistral dashboard README now :-| ^ | 10:12 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-dashboard master: Ignore the .eggs directory https://review.openstack.org/572332 | 10:12 |
| openstackgerrit | Merged openstack/mistral-dashboard master: Fix the README formatting https://review.openstack.org/572330 | 10:29 |
| openstackgerrit | Tomer Shtilman proposed openstack/mistral master: Generate default source execution id https://review.openstack.org/572317 | 10:43 |
| *** weshay has quit IRC | 10:57 | |
| *** weshay has joined #openstack-mistral | 10:58 | |
| openstackgerrit | Tomer Shtilman proposed openstack/mistral master: Generate default source execution id https://review.openstack.org/572317 | 11:11 |
| *** thrash|g0ne is now known as thrash | 11:17 | |
| *** gkadam has quit IRC | 11:17 | |
| *** gkadam has joined #openstack-mistral | 11:17 | |
| openstackgerrit | Tomer Shtilman proposed openstack/mistral master: Generate default source execution id https://review.openstack.org/572317 | 11:18 |
| *** AlexeyAbashkin has quit IRC | 11:36 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 11:46 | |
| *** katkapilatova has joined #openstack-mistral | 12:16 | |
| *** toure|gone is now known as toure | 13:01 | |
| d0ugal | apetrich: Got a moment to help me figure out a puzzler? | 13:20 |
| apetrich | d0ugal, sure | 13:21 |
| apetrich | shoot | 13:21 |
| d0ugal | apetrich: 1 sec, making a paste | 13:21 |
| d0ugal | apetrich: http://paste.openstack.org/show/722723/ | 13:23 |
| d0ugal | apetrich: The point being that resolve_outputs doesn't seem to work with the mistral action (the output is the same size) | 13:23 |
| d0ugal | btw, don't run "heat stack-show overcloud" without piping, that is a terminal killer :) | 13:24 |
| d0ugal | hmm, actually, I'm not sure why our output is so much smaller tho' | 13:24 |
| d0ugal | "only" 572 thousand | 13:25 |
| d0ugal | :) | 13:25 |
| apetrich | I wouldn't :) | 13:25 |
| d0ugal | wow, I only just calculated how big that is | 13:27 |
| d0ugal | I think the stack show output is ~130 megs | 13:27 |
| apetrich | yeah | 13:29 |
| apetrich | I was trying to scp it | 13:29 |
| apetrich | and it was 1% | 13:29 |
| apetrich | and I was like WHAAA | 13:29 |
| d0ugal | jtomasek: we could just remove the "outputs" key from the result of the stack get | 13:29 |
| d0ugal | as a quick workaround. | 13:29 |
| jtomasek | d0ugal: yeah, sounds good | 13:30 |
| apetrich | but both resolve and no resolve mistral calls give identical outputs | 13:30 |
| d0ugal | apetrich: yeah, it is a bit strange | 13:31 |
| apetrich | I'm unfamiliar with the --resolve-outputs what is it suppose to do? | 13:32 |
| apetrich | supposed | 13:32 |
| d0ugal | apetrich: I believe it includes or excludes the stack output | 13:33 |
| d0ugal | but I am not 100% sure | 13:33 |
| d0ugal | jtomasek: https://review.openstack.org/#/c/572396/ | 13:40 |
| d0ugal | apetrich: ^ | 13:40 |
| apetrich | aye | 13:41 |
| jtomasek | d0ugal: thanks, I'll test it right away | 14:02 |
| therve | d0ugal: Don't you want to remove "outputs" ? | 14:21 |
| d0ugal | therve: Yeah, isn't that what that does? | 14:21 |
| therve | d0ugal: I see .delete("output") | 14:21 |
| d0ugal | oh | 14:21 |
| d0ugal | f | 14:21 |
| d0ugal | ahem | 14:22 |
| d0ugal | jtomasek: ^ update incoming. | 14:22 |
| jtomasek | ack | 14:22 |
| d0ugal | therve: thanks! | 14:22 |
| therve | d0ugal: If you want to investigate the resolve_outputs stuff too... | 14:22 |
| therve | d0ugal: That'd be much better because it would make a quicker request too | 14:23 |
| d0ugal | therve: agreed, I am still looking into it | 14:23 |
| d0ugal | I added some logging lines and it appeared to be passed to heatclient correct, but paused to get a quick fix out first | 14:23 |
| d0ugal | jtomasek: updated | 14:24 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-dashboard stable/queens: Fix the README formatting https://review.openstack.org/572425 | 14:26 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-dashboard stable/pike: Fix the README formatting https://review.openstack.org/572426 | 14:27 |
| apetrich | d0ugal, here https://github.com/openstack/python-heatclient/blob/master/heatclient/v1/stacks.py#L76 | 14:30 |
| apetrich | not passing the parameter forward I think | 14:30 |
| therve | Hum you shouldn't use that | 14:31 |
| d0ugal | We don't use that | 14:31 |
| d0ugal | I believe we use this | 14:31 |
| d0ugal | https://github.com/openstack/python-heatclient/blob/master/heatclient/v1/stacks.py#L270 | 14:31 |
| therve | That makes more sense | 14:31 |
| apetrich | yeah | 14:32 |
| apetrich | sorry | 14:33 |
| d0ugal | np | 14:35 |
| apetrich | I missed the line number going from vi to github | 14:36 |
| * apetrich blames microsoft for that | 14:36 | |
| d0ugal | I logged 'method' and self._kwargs_for_run here to verify what they are | 14:37 |
| d0ugal | https://github.com/openstack/mistral/blob/master/mistral/actions/openstack/base.py#L116 | 14:37 |
| d0ugal | and got http://paste.openstack.org/show/722730/ | 14:37 |
| d0ugal | Which is what I expected | 14:37 |
| d0ugal | I have then been juggling a few other things, but about to continue digging | 14:37 |
| therve | Yeah have a look at heat API logs maybe | 14:40 |
| openstackgerrit | Merged openstack/mistral-dashboard stable/queens: Fix the README formatting https://review.openstack.org/572425 | 14:41 |
| openstackgerrit | Merged openstack/mistral-dashboard stable/pike: Fix the README formatting https://review.openstack.org/572426 | 14:42 |
| d0ugal | therve: I can't see anything obvious in the logs. Comparing the mistral action and calling it directly with heatclient. | 14:52 |
| therve | d0ugal: Do you see the resolve_outputs arg? | 14:53 |
| d0ugal | therve: no | 14:53 |
| therve | d0ugal: Looking at which log? | 14:53 |
| d0ugal | therve: /var/log/heat/heat_api.log | 14:53 |
| therve | d0ugal: Try httpd logs? | 14:54 |
| d0ugal | k | 14:54 |
| d0ugal | therve: hmm, this is trange | 15:09 |
| d0ugal | therve: http://paste.openstack.org/show/722736/ | 15:09 |
| d0ugal | The mistral action seems to make a second request without resolve_outputs | 15:10 |
| therve | That is weird | 15:10 |
| apetrich | d0ugal, I'm debugging the heat.api.openstack.v1.stacks.StackController:show | 15:12 |
| therve | Looks like the issue is client side | 15:12 |
| apetrich | and it shows the resolve_outputs correctly | 15:13 |
| therve | d0ugal: Maybe it does a .get() on the result of stacks.get somehow? | 15:14 |
| therve | Which would point the code apetrich showed earlier | 15:14 |
| apetrich | http://paste.openstack.org/show/722737/ | 15:15 |
| * apetrich is always ashamed of his pastes and debugs :) | 15:15 | |
| apetrich | but here it is sending the value on | 15:15 |
| therve | apetrich: Do you see the extra request too? | 15:16 |
| apetrich | therve, aye | 15:18 |
| openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Use trusts by default in OpenStack action calls https://review.openstack.org/572448 | 15:26 |
| d0ugal | therve: I think you are correct, it does seem to call .get() | 15:29 |
| *** nguyenhai93 has joined #openstack-mistral | 15:29 | |
| *** nguyenhai_ has quit IRC | 15:32 | |
| d0ugal | apetrich, therve: http://paste.openstack.org/show/722739/ | 15:33 |
| d0ugal | seems promising! | 15:33 |
| d0ugal | (but there is no chance this "fix" will be accepted) | 15:33 |
| d0ugal | I almost don't even dare tell you what I did | 15:33 |
| apetrich | NICE | 15:33 |
| * apetrich is curious now | 15:33 | |
| d0ugal | I added 'raise Exception("TESTING")' to the get method you found | 15:34 |
| d0ugal | I wanted to know what called it | 15:34 |
| d0ugal | but aparently whatever calls it also swallows the exception | 15:34 |
| d0ugal | so, umm, fixed! | 15:34 |
| *** katkapilatova has quit IRC | 15:35 | |
| apetrich | LOL | 15:35 |
| apetrich | +2 +W SHIP IT! | 15:35 |
| d0ugal | it must be something in the serialisation code | 15:36 |
| d0ugal | brb | 15:36 |
| *** hardikjasani has quit IRC | 15:46 | |
| d0ugal | I have a massive headache, so I'll continue figuring this out tomorrow | 15:48 |
| d0ugal | but it must be related to serialisation. Maybe we have something "clever" that calls .get() | 15:48 |
| *** jpich has quit IRC | 16:04 | |
| *** harlowja has joined #openstack-mistral | 16:28 | |
| therve | Oh, hum | 16:33 |
| therve | apetrich, d0ugal: http://paste.openstack.org/show/722743/ maybe | 16:34 |
| *** harlowja has quit IRC | 16:45 | |
| *** jaosorior has quit IRC | 16:58 | |
| *** AlexeyAbashkin has quit IRC | 17:28 | |
| *** apetrich has quit IRC | 17:29 | |
| *** apetrich has joined #openstack-mistral | 17:30 | |
| *** thrash is now known as thrash|biab | 17:51 | |
| *** gkadam has quit IRC | 17:52 | |
| apetrich | therve, I see no difference with that change :( | 18:16 |
| *** shardy has quit IRC | 18:23 | |
| *** shardy has joined #openstack-mistral | 18:24 | |
| *** EmilienM is now known as EmilienM|PTO | 18:43 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 19:19 | |
| therve | Too bad :/ | 19:21 |
| *** Alexey_Abashkin has joined #openstack-mistral | 19:22 | |
| *** AlexeyAbashkin has quit IRC | 19:23 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 19:23 | |
| *** thrash|biab is now known as thrash|g0ne | 19:36 | |
| *** AlexeyAbashkin has quit IRC | 19:38 | |
| *** shardy has quit IRC | 22:10 | |
| *** threestrands has joined #openstack-mistral | 23:08 | |
| *** harlowja has joined #openstack-mistral | 23:10 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!