*** bobh has joined #openstack-mistral | 00:42 | |
*** hardikjasani has joined #openstack-mistral | 05:00 | |
*** bobh has quit IRC | 05:30 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add the config option for Oslo Messaging executor type https://review.openstack.org/581671 | 05:37 |
---|---|---|
*** bobh has joined #openstack-mistral | 05:43 | |
*** bobh has quit IRC | 05:47 | |
*** bobh has joined #openstack-mistral | 05:54 | |
*** d0ugal_ has joined #openstack-mistral | 05:55 | |
*** d0ugal has quit IRC | 05:56 | |
*** bobh has quit IRC | 05:58 | |
*** bobh has joined #openstack-mistral | 06:04 | |
*** bobh has quit IRC | 06:08 | |
*** bobh has joined #openstack-mistral | 06:13 | |
*** bobh has quit IRC | 06:17 | |
*** jtomasek has joined #openstack-mistral | 06:18 | |
*** bobh has joined #openstack-mistral | 06:24 | |
*** bobh has quit IRC | 06:29 | |
*** bobh has joined #openstack-mistral | 06:34 | |
*** bobh has quit IRC | 06:38 | |
*** bobh has joined #openstack-mistral | 06:44 | |
*** bobh has quit IRC | 06:49 | |
*** bobh has joined #openstack-mistral | 06:56 | |
*** bobh has quit IRC | 07:00 | |
*** pgaxatte has joined #openstack-mistral | 07:03 | |
*** bobh has joined #openstack-mistral | 07:04 | |
*** bobh has quit IRC | 07:08 | |
*** gkadam has joined #openstack-mistral | 07:14 | |
*** bobh has joined #openstack-mistral | 07:20 | |
*** bobh has quit IRC | 07:24 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add the config option for Oslo Messaging executor type https://review.openstack.org/581671 | 07:36 |
*** kimamisa has joined #openstack-mistral | 07:42 | |
*** bobh has joined #openstack-mistral | 07:57 | |
*** bobh has quit IRC | 08:01 | |
*** d0ugal_ has quit IRC | 08:03 | |
*** d0ugal has joined #openstack-mistral | 08:03 | |
*** d0ugal has quit IRC | 08:03 | |
*** d0ugal has joined #openstack-mistral | 08:03 | |
mcdoker181818 | Hi, team! | 08:04 |
mcdoker181818 | What do you think about object model inside workflow execution context? Currently variables are replaced inside context when we publish task result. But I want that they are merged. For example, workflow: | 08:04 |
mcdoker181818 | https://thepasteb.in/p/AnhrXNQl8pDtv | 08:05 |
mcdoker181818 | Currently the workflow will return {'aa': {'bb': 'task2_res'}}. But i want that it returns {'aa': {'cc': 'task1_res', 'dd': u'task1_res', 'bb': u'task2_res', 'zz': 'wf_ex_input'}} | 08:05 |
mcdoker181818 | There are some hacks to achieve this behavior, for example mergeWith yaql function but it's not convenient for the user. | 08:05 |
mcdoker181818 | Yep, this behavior is not backward compatible. I think to create some option inside config. Can I create a blueprint? :) | 08:05 |
*** bobh has joined #openstack-mistral | 08:13 | |
*** bobh has quit IRC | 08:17 | |
openstackgerrit | ayyob proposed openstack/mistral master: add docs for states. add docs explaining what each state mean. https://review.openstack.org/580389 | 08:34 |
d0ugal | mcdoker181818: I think I would find merging by default confusing | 08:44 |
d0ugal | It would be good to open a blueprint so we can discuss it | 08:44 |
d0ugal | It would be good to get opinions from others | 08:44 |
d0ugal | Controlling the behaviour with a config setting would make workflows less portable. I am not sure that is a good idea. | 08:45 |
pgaxatte | maybe add a merge strategy option to workflows? at the same level as the type direct/indirect | 08:48 |
pgaxatte | a merge behavior like in ansible | 08:49 |
openstackgerrit | ayyob proposed openstack/mistral master: add docs for states. add docs explaining what each state mean. https://review.openstack.org/580389 | 08:51 |
mcdoker181818 | I like that idea `option to workflows` | 08:51 |
mcdoker181818 | d0ugal: My users don't like manipulating a flat parameters structure, when they have around 50 parameters :) | 08:52 |
*** bobh has joined #openstack-mistral | 08:53 | |
pgaxatte | the other alternative i can see would be a different publish option like publish_merge | 08:53 |
pgaxatte | but then you'd need to check that publish and publish_merge are not used at the same time | 08:54 |
mcdoker181818 | hmm, then we need the "output_merge" | 08:57 |
*** bobh has quit IRC | 08:57 | |
pgaxatte | yes, not quite as intuitive as a merge strategy on the workflow | 08:58 |
*** bobh has joined #openstack-mistral | 09:04 | |
d0ugal | Doing something in the workflow dsl does sound good. | 09:07 |
d0ugal | You know what I just realised, we have been adding features to the dsl - but we have never one increased the version. We should be on someting like version 2.1 or 2.2 by now | 09:08 |
d0ugal | because at the moment there is no way to know if a Workflow will work on rocky/queens/pike | 09:08 |
d0ugal | We should really start doing that | 09:08 |
d0ugal | rakhmerov: ^ | 09:08 |
*** bobh has quit IRC | 09:09 | |
*** bobh has joined #openstack-mistral | 09:16 | |
*** bobh has quit IRC | 09:20 | |
mcdoker181818 | d0ugal: You are right | 09:20 |
*** bobh has joined #openstack-mistral | 09:24 | |
*** bobh has quit IRC | 09:28 | |
*** bobh has joined #openstack-mistral | 09:47 | |
*** pgaxatte has left #openstack-mistral | 09:50 | |
*** bobh has quit IRC | 09:51 | |
*** bobh has joined #openstack-mistral | 09:55 | |
*** bobh has quit IRC | 09:59 | |
*** bobh has joined #openstack-mistral | 10:27 | |
*** bobh has quit IRC | 10:31 | |
*** bobh has joined #openstack-mistral | 10:34 | |
*** bobh has quit IRC | 10:38 | |
*** bobh has joined #openstack-mistral | 11:01 | |
*** bobh has quit IRC | 11:05 | |
*** bobh has joined #openstack-mistral | 11:15 | |
*** bobh has quit IRC | 11:19 | |
*** bobh has joined #openstack-mistral | 11:34 | |
*** bobh has quit IRC | 11:38 | |
*** bobh has joined #openstack-mistral | 11:49 | |
*** bobh has quit IRC | 11:53 | |
*** bobh has joined #openstack-mistral | 11:54 | |
*** bobh has quit IRC | 11:58 | |
*** bobh has joined #openstack-mistral | 12:05 | |
*** bobh has quit IRC | 12:10 | |
*** bobh has joined #openstack-mistral | 12:16 | |
*** bobh has quit IRC | 12:21 | |
*** bobh has joined #openstack-mistral | 12:23 | |
*** bobh has quit IRC | 12:27 | |
*** hardikjasani has quit IRC | 12:29 | |
*** geb has quit IRC | 12:30 | |
*** apetrich has quit IRC | 12:32 | |
*** geb has joined #openstack-mistral | 12:34 | |
*** bobh has joined #openstack-mistral | 12:39 | |
*** bobh has quit IRC | 12:43 | |
*** bobh has joined #openstack-mistral | 12:45 | |
*** bobh has quit IRC | 12:48 | |
*** bobh has joined #openstack-mistral | 12:49 | |
*** bobh has quit IRC | 12:50 | |
*** pgaxatte has joined #openstack-mistral | 13:01 | |
*** bobh has joined #openstack-mistral | 13:23 | |
*** apetrich has joined #openstack-mistral | 14:05 | |
*** gkadam has quit IRC | 15:47 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Return the result of the MistralHTTPAction https://review.openstack.org/581059 | 15:59 |
*** rbrady is now known as rbrady-afk | 16:01 | |
*** shardy has quit IRC | 16:25 | |
*** kimamisa has quit IRC | 16:41 | |
openstackgerrit | Merged openstack/mistral master: add docs for states. add docs explaining what each state mean. https://review.openstack.org/580389 | 17:30 |
*** rbrady-afk is now known as rbrady | 18:12 | |
*** harlowja has joined #openstack-mistral | 18:24 | |
*** jistr has quit IRC | 18:50 | |
*** jistr has joined #openstack-mistral | 19:25 | |
*** mcdoker181818 has quit IRC | 20:35 | |
*** bobh has quit IRC | 21:26 | |
*** bobh has joined #openstack-mistral | 22:34 | |
*** bobh has quit IRC | 22:39 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!