| *** threestrands has quit IRC | 00:21 | |
| *** yangyapeng has quit IRC | 00:22 | |
| *** yangyapeng has joined #openstack-mistral | 00:55 | |
| *** yangyapeng has quit IRC | 00:59 | |
| *** yangyapeng has joined #openstack-mistral | 01:04 | |
| *** daidv has joined #openstack-mistral | 01:34 | |
| *** zhurong has joined #openstack-mistral | 02:48 | |
| *** threestrands has joined #openstack-mistral | 02:55 | |
| *** thrash is now known as thrash|g0ne | 02:57 | |
| *** bobh has quit IRC | 03:23 | |
| *** bobh has joined #openstack-mistral | 03:31 | |
| *** zhurong has quit IRC | 03:46 | |
| *** d0ugal_ has joined #openstack-mistral | 03:51 | |
| *** d0ugal has quit IRC | 03:52 | |
| *** bobh has quit IRC | 03:58 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 04:15 | |
| *** AlexeyAbashkin has quit IRC | 04:20 | |
| *** threestrands has quit IRC | 06:50 | |
| *** gkadam_ has quit IRC | 07:24 | |
| *** jtomasek has joined #openstack-mistral | 07:27 | |
| *** jtomasek has quit IRC | 07:28 | |
| *** jtomasek has joined #openstack-mistral | 07:28 | |
| *** oidgar has joined #openstack-mistral | 07:38 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 08:06 | |
| *** jaosorior has quit IRC | 08:13 | |
| *** jaosorior has joined #openstack-mistral | 08:34 | |
| *** AlexeyAbashkin has quit IRC | 08:41 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 08:42 | |
| *** gkadam_ has joined #openstack-mistral | 08:46 | |
| *** jpich has joined #openstack-mistral | 08:55 | |
| *** d0ugal_ has quit IRC | 09:17 | |
| *** d0ugal has joined #openstack-mistral | 09:18 | |
| d0ugal | jtomasek: Hey, I just had an idea I wanted to run past you | 09:18 |
|---|---|---|
| jtomasek | d0ugal: hi | 09:19 |
| d0ugal | jtomasek: The main reason you like direct action calls is due to the faster response, right? | 09:19 |
| d0ugal | jtomasek: I was wondering if we could add priority to workflow executions, so that you could start a workflow execution (like the new get_horizon_url one) with a high priority and then it would run before any other workflows that are scheduled | 09:19 |
| jtomasek | d0ugal: probably biggest impact is that it basically matches ordinary request-response api call | 09:20 |
| d0ugal | at the moment it would be added to the end of the queue iiuc | 09:20 |
| d0ugal | jtomasek: oh okay, then maybe this idea doesn't help with that | 09:20 |
| d0ugal | jtomasek: so the problem is that you want to use Mistral for something that it isn't :) | 09:20 |
| d0ugal | I'll keep thinking about it. | 09:21 |
| jtomasek | d0ugal: it's a good idea as it would probably help with cases where there are validations triggered which can potentially cause some request to take long | 09:21 |
| d0ugal | Yeah, I think it could be generally useful - I am surprised I can't see any bugs or previous discussions about priority. | 09:22 |
| jtomasek | d0ugal: I understand, thing is that one of the reasons we chose it as a generic API was that it was also capable of running actions directly in cases when the logic does not really need a workflow | 09:22 |
| d0ugal | jtomasek: yeah, well, I was always in favour of a simple API infront of Mistral :) | 09:23 |
| d0ugal | I guess we will just have to live with this awkward setup :) | 09:24 |
| jtomasek | d0ugal: on the other hand, directly calling actions should not be required very often as at that case we could usually just plain api call to openstack api | 09:24 |
| d0ugal | Right | 09:24 |
| jtomasek | d0ugal: it basically just applies to tripleo specific usecases (such as roles management) | 09:25 |
| d0ugal | jtomasek: is there a way for me to easily see all the direct action calls? | 09:25 |
| d0ugal | jtomasek: one issue is that we don't really know which you use (or test using them directly anywhere in CI) | 09:25 |
| jtomasek | d0ugal: I don't have strong oppinion and I am fine with calling workflows as long as there are no performance problems or us creating workflow just for a sake of having workflow | 09:25 |
| d0ugal | right | 09:26 |
| * jtomasek searches | 09:27 | |
| jtomasek | d0ugal: https://github.com/openstack/tripleo-ui/search?utf8=%E2%9C%93&q=runAction&type= | 09:27 |
| jtomasek | (ignore the tests and you should be able to see direct calls to mistral actions) | 09:28 |
| jtomasek | as you can see, it is basically just listing stuff | 09:28 |
| d0ugal | k, thanks | 09:29 |
| jtomasek | d0ugal: btw. I've added comments to https://review.openstack.org/#/c/521624/1 | 09:31 |
| d0ugal | jtomasek: I spotted - it is so close to merging I was trying to decide what to do | 09:32 |
| d0ugal | jtomasek: if you think it is worth blocking, maybe workflow -1 | 09:33 |
| d0ugal | or we can change it after | 09:33 |
| jtomasek | d0ugal: it would be really nice to make the zaqar message 'message' contents match the workflow output. I think it is quite important from consumer point of view, as GUI/CLI always knows that the data it requested is found under payload.message | 09:33 |
| d0ugal | Right, it isn't always the case tho' | 09:34 |
| d0ugal | The introspection workflows use introspected_nodes iirc | 09:34 |
| d0ugal | Moving to #tripleo | 09:36 |
| *** mfedosin_ has joined #openstack-mistral | 09:38 | |
| *** mfedosin has quit IRC | 09:39 | |
| *** EmilienM has quit IRC | 09:39 | |
| *** mfedosin_ is now known as mfedosin | 09:39 | |
| *** EmilienM has joined #openstack-mistral | 09:41 | |
| *** Kevin_Zheng has quit IRC | 10:09 | |
| *** kong has quit IRC | 10:09 | |
| *** zhenguo has quit IRC | 10:09 | |
| *** dtantsur|afk has quit IRC | 10:09 | |
| *** dtantsur has joined #openstack-mistral | 10:15 | |
| *** Kevin_Zheng has joined #openstack-mistral | 10:15 | |
| *** kong has joined #openstack-mistral | 10:15 | |
| *** zhenguo has joined #openstack-mistral | 10:15 | |
| *** dtantsur has quit IRC | 10:15 | |
| *** dtantsur has joined #openstack-mistral | 10:15 | |
| *** daidv has quit IRC | 10:35 | |
| *** katkapilatova has joined #openstack-mistral | 11:28 | |
| *** jkilpatr has quit IRC | 11:37 | |
| *** jkilpatr has joined #openstack-mistral | 11:56 | |
| *** oidgar has quit IRC | 12:03 | |
| *** thrash|g0ne is now known as thrash | 12:08 | |
| *** jkilpatr has quit IRC | 12:14 | |
| *** jkilpatr has joined #openstack-mistral | 12:17 | |
| *** oidgar has joined #openstack-mistral | 12:18 | |
| *** bobh has joined #openstack-mistral | 12:26 | |
| *** bobh has quit IRC | 12:30 | |
| *** oidgar has quit IRC | 12:37 | |
| *** oidgar has joined #openstack-mistral | 12:51 | |
| *** thrash is now known as thrash|biab | 12:51 | |
| *** dprince has joined #openstack-mistral | 12:59 | |
| *** rbrady has quit IRC | 13:01 | |
| *** dtantsur is now known as dtantsur|brb | 13:13 | |
| *** rbrady has joined #openstack-mistral | 13:13 | |
| *** rbrady has quit IRC | 13:13 | |
| *** rbrady has joined #openstack-mistral | 13:13 | |
| *** thrash|biab is now known as thrash | 13:23 | |
| *** yangyapeng has quit IRC | 13:26 | |
| *** yangyapeng has joined #openstack-mistral | 13:27 | |
| *** yangyapeng has quit IRC | 13:36 | |
| thrash | d0ugal: check it out... this gets the non-apache devstack job passing! https://review.openstack.org/#/c/522023/ | 14:01 |
| thrash | d0ugal: and just one test not passing in the apache-wsgi version | 14:02 |
| *** d0ugal has quit IRC | 14:03 | |
| *** d0ugal has joined #openstack-mistral | 14:18 | |
| *** oidgar has quit IRC | 14:31 | |
| *** dtantsur|brb is now known as dtantsur | 14:42 | |
| *** oidgar has joined #openstack-mistral | 14:50 | |
| *** brunograz has quit IRC | 14:52 | |
| *** bobh has joined #openstack-mistral | 14:52 | |
| *** brunograz has joined #openstack-mistral | 14:53 | |
| *** oidgar has quit IRC | 15:07 | |
| *** jrist has quit IRC | 15:08 | |
| *** jrist has joined #openstack-mistral | 15:09 | |
| *** oidgar has joined #openstack-mistral | 15:15 | |
| *** thrash is now known as thrash|biab | 15:48 | |
| *** dprince has quit IRC | 15:54 | |
| *** dprince has joined #openstack-mistral | 16:08 | |
| *** thrash|biab is now known as thrash | 16:15 | |
| openstackgerrit | Andras Kovi proposed openstack/mistral master: Add id field to db query if no sorting order is provided https://review.openstack.org/523926 | 16:23 |
| *** katkapilatova has quit IRC | 16:50 | |
| *** d0ugal has quit IRC | 16:55 | |
| *** thrash is now known as thrash|biab | 17:01 | |
| *** AlexeyAbashkin has quit IRC | 17:01 | |
| *** dtantsur is now known as dtantsur|afk | 17:03 | |
| *** d0ugal has joined #openstack-mistral | 17:09 | |
| *** gkadam_ has quit IRC | 17:22 | |
| *** d0ugal has quit IRC | 17:24 | |
| *** d0ugal has joined #openstack-mistral | 17:35 | |
| *** jpich has quit IRC | 17:44 | |
| *** yangyapeng has joined #openstack-mistral | 17:49 | |
| *** yangyapeng has quit IRC | 17:54 | |
| *** jkilpatr has quit IRC | 18:23 | |
| *** jkilpatr has joined #openstack-mistral | 18:39 | |
| *** openstackgerrit has quit IRC | 18:48 | |
| *** oidgar has quit IRC | 18:49 | |
| *** gkadam has joined #openstack-mistral | 18:58 | |
| *** thrash|biab is now known as thrash | 19:04 | |
| *** brunograz has quit IRC | 19:20 | |
| *** brunograz has joined #openstack-mistral | 19:20 | |
| *** brunograz has quit IRC | 19:21 | |
| *** brunograz has joined #openstack-mistral | 19:22 | |
| *** jkilpatr has quit IRC | 19:39 | |
| *** yangyapeng has joined #openstack-mistral | 19:45 | |
| *** yangyapeng has quit IRC | 19:50 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 19:52 | |
| *** AlexeyAbashkin has quit IRC | 19:59 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 20:12 | |
| *** jtomasek has quit IRC | 20:13 | |
| *** AlexeyAbashkin has quit IRC | 20:17 | |
| *** gkadam has quit IRC | 21:01 | |
| *** jkilpatr has joined #openstack-mistral | 21:09 | |
| *** threestrands has joined #openstack-mistral | 21:12 | |
| *** threestrands has quit IRC | 21:12 | |
| *** threestrands has joined #openstack-mistral | 21:12 | |
| *** brunograz has quit IRC | 21:21 | |
| *** bobh has quit IRC | 23:01 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 23:12 | |
| *** dprince has quit IRC | 23:16 | |
| *** AlexeyAbashkin has quit IRC | 23:16 | |
| *** thrash is now known as thrash|g0ne | 23:26 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!