Thursday, 2018-07-12

*** bobh has joined #openstack-mistral00:42
*** hardikjasani has joined #openstack-mistral05:00
*** bobh has quit IRC05:30
openstackgerritRenat Akhmerov proposed openstack/mistral master: Add the config option for Oslo Messaging executor type  https://review.openstack.org/58167105:37
*** bobh has joined #openstack-mistral05:43
*** bobh has quit IRC05:47
*** bobh has joined #openstack-mistral05:54
*** d0ugal_ has joined #openstack-mistral05:55
*** d0ugal has quit IRC05:56
*** bobh has quit IRC05:58
*** bobh has joined #openstack-mistral06:04
*** bobh has quit IRC06:08
*** bobh has joined #openstack-mistral06:13
*** bobh has quit IRC06:17
*** jtomasek has joined #openstack-mistral06:18
*** bobh has joined #openstack-mistral06:24
*** bobh has quit IRC06:29
*** bobh has joined #openstack-mistral06:34
*** bobh has quit IRC06:38
*** bobh has joined #openstack-mistral06:44
*** bobh has quit IRC06:49
*** bobh has joined #openstack-mistral06:56
*** bobh has quit IRC07:00
*** pgaxatte has joined #openstack-mistral07:03
*** bobh has joined #openstack-mistral07:04
*** bobh has quit IRC07:08
*** gkadam has joined #openstack-mistral07:14
*** bobh has joined #openstack-mistral07:20
*** bobh has quit IRC07:24
openstackgerritRenat Akhmerov proposed openstack/mistral master: Add the config option for Oslo Messaging executor type  https://review.openstack.org/58167107:36
*** kimamisa has joined #openstack-mistral07:42
*** bobh has joined #openstack-mistral07:57
*** bobh has quit IRC08:01
*** d0ugal_ has quit IRC08:03
*** d0ugal has joined #openstack-mistral08:03
*** d0ugal has quit IRC08:03
*** d0ugal has joined #openstack-mistral08:03
mcdoker181818Hi, team!08:04
mcdoker181818What 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
mcdoker181818https://thepasteb.in/p/AnhrXNQl8pDtv08:05
mcdoker181818Currently 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
mcdoker181818There are some hacks to achieve this behavior, for example mergeWith yaql function but it's not convenient for the user.08:05
mcdoker181818Yep, 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-mistral08:13
*** bobh has quit IRC08:17
openstackgerritayyob proposed openstack/mistral master: add docs for states. add docs explaining what each state mean.  https://review.openstack.org/58038908:34
d0ugalmcdoker181818: I think I would find merging by default confusing08:44
d0ugalIt would be good to open a blueprint so we can discuss it08:44
d0ugalIt would be good to get opinions from others08:44
d0ugalControlling the behaviour with a config setting would make workflows less portable. I am not sure that is a good idea.08:45
pgaxattemaybe add a merge strategy option to workflows? at the same level as the type direct/indirect08:48
pgaxattea merge behavior like in ansible08:49
openstackgerritayyob proposed openstack/mistral master: add docs for states. add docs explaining what each state mean.  https://review.openstack.org/58038908:51
mcdoker181818I like that idea `option to workflows`08:51
mcdoker181818d0ugal: My users don't like manipulating a flat parameters structure, when they have around 50 parameters :)08:52
*** bobh has joined #openstack-mistral08:53
pgaxattethe other alternative i can see would be a different publish option like publish_merge08:53
pgaxattebut then you'd need to check that publish and publish_merge are not used at the same time08:54
mcdoker181818hmm, then we need the "output_merge"08:57
*** bobh has quit IRC08:57
pgaxatteyes, not quite as intuitive as a merge strategy on the workflow08:58
*** bobh has joined #openstack-mistral09:04
d0ugalDoing something in the workflow dsl does sound good.09:07
d0ugalYou 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 now09:08
d0ugalbecause at the moment there is no way to know if a Workflow will work on rocky/queens/pike09:08
d0ugalWe should really start doing that09:08
d0ugalrakhmerov: ^09:08
*** bobh has quit IRC09:09
*** bobh has joined #openstack-mistral09:16
*** bobh has quit IRC09:20
mcdoker181818d0ugal: You are right09:20
*** bobh has joined #openstack-mistral09:24
*** bobh has quit IRC09:28
*** bobh has joined #openstack-mistral09:47
*** pgaxatte has left #openstack-mistral09:50
*** bobh has quit IRC09:51
*** bobh has joined #openstack-mistral09:55
*** bobh has quit IRC09:59
*** bobh has joined #openstack-mistral10:27
*** bobh has quit IRC10:31
*** bobh has joined #openstack-mistral10:34
*** bobh has quit IRC10:38
*** bobh has joined #openstack-mistral11:01
*** bobh has quit IRC11:05
*** bobh has joined #openstack-mistral11:15
*** bobh has quit IRC11:19
*** bobh has joined #openstack-mistral11:34
*** bobh has quit IRC11:38
*** bobh has joined #openstack-mistral11:49
*** bobh has quit IRC11:53
*** bobh has joined #openstack-mistral11:54
*** bobh has quit IRC11:58
*** bobh has joined #openstack-mistral12:05
*** bobh has quit IRC12:10
*** bobh has joined #openstack-mistral12:16
*** bobh has quit IRC12:21
*** bobh has joined #openstack-mistral12:23
*** bobh has quit IRC12:27
*** hardikjasani has quit IRC12:29
*** geb has quit IRC12:30
*** apetrich has quit IRC12:32
*** geb has joined #openstack-mistral12:34
*** bobh has joined #openstack-mistral12:39
*** bobh has quit IRC12:43
*** bobh has joined #openstack-mistral12:45
*** bobh has quit IRC12:48
*** bobh has joined #openstack-mistral12:49
*** bobh has quit IRC12:50
*** pgaxatte has joined #openstack-mistral13:01
*** bobh has joined #openstack-mistral13:23
*** apetrich has joined #openstack-mistral14:05
*** gkadam has quit IRC15:47
openstackgerritDougal Matthews proposed openstack/mistral master: Return the result of the MistralHTTPAction  https://review.openstack.org/58105915:59
*** rbrady is now known as rbrady-afk16:01
*** shardy has quit IRC16:25
*** kimamisa has quit IRC16:41
openstackgerritMerged openstack/mistral master: add docs for states. add docs explaining what each state mean.  https://review.openstack.org/58038917:30
*** rbrady-afk is now known as rbrady18:12
*** harlowja has joined #openstack-mistral18:24
*** jistr has quit IRC18:50
*** jistr has joined #openstack-mistral19:25
*** mcdoker181818 has quit IRC20:35
*** bobh has quit IRC21:26
*** bobh has joined #openstack-mistral22:34
*** bobh has quit IRC22:39

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!