| *** jamielennox|away is now known as jamielennox | 00:02 | |
| *** kbaegis has quit IRC | 00:17 | |
| *** bobh has quit IRC | 00:38 | |
| *** bobh has joined #openstack-mistral | 00:41 | |
| *** rbrady-afk has quit IRC | 00:44 | |
| *** bobh has quit IRC | 00:56 | |
| *** rbrady-afk has joined #openstack-mistral | 01:00 | |
| *** bobh has joined #openstack-mistral | 01:16 | |
| *** bobh has quit IRC | 01:32 | |
| *** jamielennox is now known as jamielennox|away | 02:00 | |
| *** Kevin_Zheng has joined #openstack-mistral | 02:07 | |
| *** rbrady-afk has quit IRC | 02:53 | |
| *** rbrady-afk has joined #openstack-mistral | 03:05 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: Optimize API layer https://review.openstack.org/466575 | 03:08 |
|---|---|---|
| *** kbaegis has joined #openstack-mistral | 03:09 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Optimize API layer: using from_db_model() instead of from_dict() https://review.openstack.org/466575 | 03:18 |
| *** bobh has joined #openstack-mistral | 03:22 | |
| *** bobh has quit IRC | 03:37 | |
| *** kbaegis has quit IRC | 03:51 | |
| *** jamielennox|away is now known as jamielennox | 03:53 | |
| *** sharatss has joined #openstack-mistral | 04:34 | |
| openstackgerrit | Merged openstack/mistral master: Adding mistral_lib actions to mistral https://review.openstack.org/464186 | 04:39 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: refactor rest_utils https://review.openstack.org/466587 | 04:39 |
| *** jaosorior has joined #openstack-mistral | 04:52 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Refactor rest_utils https://review.openstack.org/466587 | 04:56 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Refactor rest_utils https://review.openstack.org/466587 | 04:58 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Refactor rest_utils https://review.openstack.org/466587 | 05:04 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Optimize API layer: using from_db_model() instead of from_dict() https://review.openstack.org/466575 | 05:04 |
| rakhmerov | d0ugal: hi, can you please look at our discussion in https://review.openstack.org/#/c/457184 ? | 05:27 |
| rakhmerov | it's an important change and there are pros and cons for making it | 05:28 |
| *** abhishekk has quit IRC | 06:34 | |
| *** shardy has joined #openstack-mistral | 06:55 | |
| d0ugal | rakhmerov: will do! | 07:11 |
| rakhmerov | d0ugal: thanks | 07:17 |
| d0ugal | The discussion about on-success and on-complete calling the action only once? | 07:20 |
| d0ugal | rakhmerov: ^ | 07:21 |
| rakhmerov | yes | 07:21 |
| rakhmerov | right | 07:21 |
| rakhmerov | It was probably a mistake to mix two different things within the same commit | 07:21 |
| rakhmerov | although they are related | 07:21 |
| rakhmerov | but yes, nobody is against of adding "triggered_by". Behavior of "on-" clauses is the only question | 07:22 |
| d0ugal | right | 07:23 |
| *** jpich has joined #openstack-mistral | 07:23 | |
| d0ugal | I think on-success: task1 on-complete: task1 should call it twice (unless there is an error) | 07:24 |
| d0ugal | but I've still not read it all | 07:24 |
| d0ugal | so I'll do that - then I'll reply to the patch | 07:24 |
| rakhmerov | ok ) | 07:35 |
| rakhmerov | d0ugal: my colleagues are all for the same behavior | 07:35 |
| rakhmerov | basically as it is now | 07:35 |
| rakhmerov | ok, I'll split those two changes | 07:35 |
| rakhmerov | well, I won't make this specific one for now | 07:36 |
| rakhmerov | d0ugal: btw, a couple of cool patches that I wanted to make for a long time: https://review.openstack.org/#/c/466575/ and https://review.openstack.org/#/c/466587/ | 07:37 |
| rakhmerov | once you have a chance.. | 07:37 |
| rakhmerov | it reduces memory footprint and improves performance of API controllers | 07:38 |
| d0ugal | rakhmerov: having mistral automatically check that an action is in on-success and on-complete is a bit too "magic" for me | 07:44 |
| rakhmerov | yeah, may be | 07:44 |
| d0ugal | rakhmerov: I think we could come up with a better way to split up expressions into multiple lines | 07:58 |
| rakhmerov | :) | 07:58 |
| rakhmerov | yeah, right | 07:58 |
| rakhmerov | you know, I've been thinking about this for so long that I'm already really confused :) | 07:58 |
| rakhmerov | that's why I'm asking people to participate | 07:58 |
| rakhmerov | I need a fresh look | 07:58 |
| d0ugal | rakhmerov: we could actually make expressions more powerful if we supported some basic operations like AND/OR etc. | 07:59 |
| d0ugal | I am not a huge fan of this, but maybe we could do something like how mongodb represents them in JSON | 07:59 |
| d0ugal | https://docs.mongodb.com/manual/reference/operator/query/or/ | 07:59 |
| d0ugal | Obviously we would use the YAML equivilent. | 08:00 |
| rakhmerov | yes, there've been some discussions internally here about it | 08:00 |
| rakhmerov | the general idea is to make branching more powerful in Mistral | 08:00 |
| rakhmerov | the goal | 08:00 |
| d0ugal | yeah | 08:00 |
| rakhmerov | I'm generally ok with that but I'd like to keep simplicity too, at least so that for something stupid and simple we could continue to do what we're doing now | 08:01 |
| rakhmerov | but if someone wants to use more advanced capabilities, - go ahead, here they are | 08:02 |
| d0ugal | yeah | 08:02 |
| d0ugal | We should aim to make the language simple, but allow people to go deeper if they need something more advanced. | 08:02 |
| rakhmerov | so, for simple use cases workflows should still look simple | 08:02 |
| d0ugal | agreed | 08:02 |
| rakhmerov | yes | 08:02 |
| rakhmerov | we're on the same page | 08:03 |
| d0ugal | :) | 08:03 |
| rakhmerov | I'm already thinking on the background about the starting a new language spec | 08:03 |
| rakhmerov | that would account for such things | 08:03 |
| rakhmerov | but this is going to be huge :) it'll take long to land it | 08:04 |
| rakhmerov | so at some point we could expose new lang (e.g. v3.0) and still keep supporting v2.0 | 08:04 |
| rakhmerov | although it's not trivial | 08:04 |
| d0ugal | rakhmerov: yeah, that would be interesting but I don't think we should rush into it | 08:10 |
| rakhmerov | sure, no rush | 08:10 |
| d0ugal | v2 mostly works well and I don't feel we have hit the limitations of it yet - so we can still improve it and learn from it for a bit longer | 08:10 |
| rakhmerov | it will be evolution, not revolution :) | 08:10 |
| d0ugal | good :) | 08:10 |
| rakhmerov | yeah | 08:11 |
| d0ugal | I commented on the patch btw | 08:11 |
| rakhmerov | ok, thanks | 08:14 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add 'runtime_context' to task execution REST resource https://review.openstack.org/458000 | 09:06 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add 'triggered_by' into task execution runtime context https://review.openstack.org/457184 | 09:06 |
| rakhmerov | d0ugal: thanks for the quick review! | 09:14 |
| d0ugal | np! | 09:14 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add 'runtime_context' to task execution REST resource https://review.openstack.org/458000 | 09:19 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add 'triggered_by' into task execution runtime context https://review.openstack.org/457184 | 09:19 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add 'runtime_context' to task execution REST resource https://review.openstack.org/458000 | 10:06 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add 'triggered_by' into task execution runtime context https://review.openstack.org/457184 | 10:06 |
| openstackgerrit | Merged openstack/mistral master: Optimize API layer: using from_db_model() instead of from_dict() https://review.openstack.org/466575 | 10:08 |
| openstackgerrit | Merged openstack/mistral master: Refactor rest_utils https://review.openstack.org/466587 | 10:08 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add one more test for task() function used in on-success https://review.openstack.org/465858 | 10:11 |
| *** jkilpatr has quit IRC | 10:31 | |
| *** kbaegis has joined #openstack-mistral | 10:48 | |
| *** jkilpatr has joined #openstack-mistral | 10:57 | |
| *** shardy has quit IRC | 11:27 | |
| *** kbaegis has left #openstack-mistral | 12:00 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: [Regions] Fixing determining keystone for actions https://review.openstack.org/466704 | 12:00 |
| *** dprince has joined #openstack-mistral | 12:04 | |
| *** chlong has joined #openstack-mistral | 12:29 | |
| *** catintheroof has joined #openstack-mistral | 12:41 | |
| *** chlong has quit IRC | 12:55 | |
| *** fultonj has joined #openstack-mistral | 12:58 | |
| *** bobh has joined #openstack-mistral | 13:04 | |
| *** sharatss has quit IRC | 13:22 | |
| *** rbrady-afk is now known as rbrady | 13:33 | |
| *** rbrady is now known as Guest29035 | 13:34 | |
| *** fultonj has quit IRC | 13:35 | |
| *** fultonj has joined #openstack-mistral | 13:36 | |
| *** Guest29035 is now known as rbrady-foo | 13:37 | |
| openstackgerrit | Toure Dunnon proposed openstack/mistral-specs master: Workflow Error Analysis https://review.openstack.org/443217 | 13:49 |
| *** rbrady-foo is now known as rbrady_ | 13:49 | |
| *** rbrady_ has quit IRC | 13:50 | |
| *** rbrady_ has joined #openstack-mistral | 13:50 | |
| *** rbrady_ is now known as rbrady | 13:50 | |
| *** catinthe_ has joined #openstack-mistral | 13:53 | |
| *** catinth__ has joined #openstack-mistral | 13:53 | |
| *** catintheroof has quit IRC | 13:54 | |
| *** catinthe_ has quit IRC | 13:57 | |
| *** mgershen has joined #openstack-mistral | 13:59 | |
| *** mbrennan has joined #openstack-mistral | 14:50 | |
| *** d0ugal has quit IRC | 14:57 | |
| openstackgerrit | Merged openstack/mistral master: Add 'triggered_by' into task execution runtime context https://review.openstack.org/457184 | 15:00 |
| openstackgerrit | Merged openstack/mistral master: Add 'runtime_context' to task execution REST resource https://review.openstack.org/458000 | 15:00 |
| rakhmerov | hi all, meeting at #openstack-meeting-3 | 15:01 |
| rakhmerov | thrash, rbrady, d0ugal: ^ | 15:02 |
| rakhmerov | apetrich_: ^ | 15:03 |
| rakhmerov | kong: | 15:03 |
| *** d0ugal has joined #openstack-mistral | 15:07 | |
| *** jpich has quit IRC | 15:22 | |
| *** rbrady is now known as rbrady-afk | 15:24 | |
| *** mgershen has quit IRC | 15:33 | |
| openstackgerrit | Stephen Finucane proposed openstack/python-mistralclient master: Use Sphinx 1.5 warning-is-error https://review.openstack.org/457941 | 15:37 |
| *** chlong has joined #openstack-mistral | 15:39 | |
| openstackgerrit | luong tuan proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 15:46 |
| *** mbrennan has quit IRC | 15:49 | |
| *** jaosorior is now known as jaosorior_away | 16:45 | |
| *** chlong has quit IRC | 16:59 | |
| *** dprince has quit IRC | 17:00 | |
| *** dprince has joined #openstack-mistral | 17:14 | |
| *** chlong has joined #openstack-mistral | 17:15 | |
| *** chlong has quit IRC | 17:15 | |
| *** chlong has joined #openstack-mistral | 17:55 | |
| prometheanfire | does anyone mind responding to my email to -dev about projects holding back requirements updates? | 17:56 |
| prometheanfire | [requirements] Projects holding back requirements updates | 17:57 |
| prometheanfire | is it possible to get a pip published rc or beta from master? | 17:57 |
| *** thrash is now known as thrash|biab | 17:58 | |
| *** rbrady-afk is now known as rbrady | 18:01 | |
| *** thrash|biab is now known as thrash | 18:23 | |
| *** catinth__ has quit IRC | 18:28 | |
| *** bobh has quit IRC | 18:29 | |
| *** catintheroof has joined #openstack-mistral | 18:30 | |
| *** harlowja has joined #openstack-mistral | 18:37 | |
| *** harlowja has quit IRC | 18:43 | |
| *** dprince has quit IRC | 18:47 | |
| *** dprince has joined #openstack-mistral | 18:57 | |
| *** fultonj has quit IRC | 19:34 | |
| *** fultonj has joined #openstack-mistral | 19:38 | |
| *** bobh has joined #openstack-mistral | 19:49 | |
| *** chlong has quit IRC | 20:07 | |
| *** rbrady is now known as rbrady-biab | 20:09 | |
| *** bobh has quit IRC | 20:09 | |
| *** chlong has joined #openstack-mistral | 20:31 | |
| *** dprince has quit IRC | 20:32 | |
| *** bobh has joined #openstack-mistral | 20:35 | |
| *** chlong has quit IRC | 20:41 | |
| *** mattybre_ has joined #openstack-mistral | 20:48 | |
| *** mattybre_ has quit IRC | 20:48 | |
| *** mattybrennan has joined #openstack-mistral | 20:48 | |
| *** thrash is now known as thrash|g0ne | 21:06 | |
| *** toure is now known as toure|gone | 21:10 | |
| *** rbrady-biab is now known as rbrady-afk | 21:15 | |
| *** catintheroof has quit IRC | 21:22 | |
| *** mattybrennan has quit IRC | 21:23 | |
| *** catintheroof has joined #openstack-mistral | 21:29 | |
| *** mattybrennan has joined #openstack-mistral | 21:44 | |
| *** jkilpatr has quit IRC | 22:08 | |
| *** mattybrennan has quit IRC | 22:17 | |
| *** jkilpatr has joined #openstack-mistral | 22:27 | |
| *** bobh has quit IRC | 22:42 | |
| *** catintheroof has quit IRC | 23:40 | |
| *** catintheroof has joined #openstack-mistral | 23:48 | |
| *** catintheroof has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!