Thursday, 2017-06-29

*** bobh has joined #openstack-mistral00:31
*** bobh has quit IRC00:53
*** bobh has joined #openstack-mistral00:55
*** bobh has quit IRC01:04
*** thrash is now known as thrash|g0ne01:34
*** bobh has joined #openstack-mistral01:36
*** bobh has quit IRC02:09
*** bobh has joined #openstack-mistral02:24
*** gfidente has quit IRC02:51
*** bobh has quit IRC03:26
*** sharatss has joined #openstack-mistral05:41
*** sharatss has quit IRC05:55
*** sharatss has joined #openstack-mistral06:11
*** sharat has joined #openstack-mistral06:31
*** shardy has joined #openstack-mistral07:06
*** jpich has joined #openstack-mistral07:20
d0ugalrakhmerov: yay https://review.openstack.org/#/c/477450/07:31
d0ugal:)07:31
rakhmerovd0ugal: excellent!07:32
rakhmerovno need to change the release model then )07:32
d0ugalIndeed07:32
d0ugalUnless you want to change it :)07:32
*** jkilpatr has quit IRC07:33
*** sharatss_ has joined #openstack-mistral07:37
*** sharatss has quit IRC07:37
*** amoralej|off is now known as amoralej07:38
*** openstackgerrit has quit IRC07:47
*** sharatss_ has quit IRC07:49
rakhmerovd0ugal: no, I don't for now )07:53
apetrichd0ugal, woot!07:56
*** sharat has quit IRC08:10
*** dtantsur|afk is now known as dtantsur08:11
rakhmerovkong: hi08:16
rakhmerovyou here?08:16
*** mgershen has joined #openstack-mistral08:32
d0ugalrakhmerov: do you understand the "--filter" option on the CLI?08:38
d0ugalHow can I find the list of valid names.08:38
d0ugalrakhmerov: Foudn it, it seems you can't do "mistral task-list --filter execution_id=ID" - that would be useful.08:42
* d0ugal might write a quick patch.08:42
rakhmerovyep, right08:44
rakhmerovd0ugal: but in case of task-list you don't even need to use --filter08:45
rakhmerovjust do: mistral task-list <execution_id>08:45
d0ugaloh, wow08:45
rakhmerovyep08:45
d0ugalTIL!08:45
rakhmerovsame with action-execution-list08:45
rakhmerovyou can pass it with a task execution ID08:45
d0ugalamazing08:46
d0ugalthanks08:46
rakhmerovbut yes, --filter can be used for all other fields08:46
rakhmerovname, state etc.08:46
d0ugalright, I have used it before and thought it was weird that it didn't allow execution_id08:46
d0ugalNow I know why :)08:47
d0ugalUntil now I have just grepped08:47
*** sharat has joined #openstack-mistral08:49
rakhmerovhaha :)08:51
rakhmerovyou suffered, it's bad )08:52
rakhmerovdocumentation!! We need a good documentation!08:52
d0ugalhaha, yeah09:00
d0ugalrakhmerov: I would like "mistral execution-list EXECUTION_ID"09:16
d0ugalto get the execution and it's sub-workflow executions09:17
d0ugalIs there a relation in the db for sub-workflow to parent workflow?09:17
d0ugalI don't see one :(09:18
rakhmerovd0ugal: actually it's a good idea09:18
rakhmerovbut we can't use EXECUTION_ID09:18
rakhmerovwe rather have to use TASK_EXECUTION_ID09:18
d0ugaloh09:18
rakhmerovbecause workflow may have multiple subworkflows09:18
d0ugalright09:19
rakhmerovyes, the relation is "task_execution_id"09:19
rakhmerovin workflow execution09:19
d0ugalbut they also might have multiple task executions with sub-workflows?09:19
d0ugalhmm09:19
d0ugalThis would be hard to do with one db query :)09:19
rakhmerovnot sure I understood your last question..09:21
d0ugalrakhmerov: so, given this example: http://paste.openstack.org/show/614015/09:22
rakhmerovyep09:22
d0ugalI only started one workflow, the others were started by it09:22
rakhmerovyes09:22
rakhmerovooh, you just want to list ALL subworkflows of your original workflow?09:22
d0ugalif there were other workflow executions, how would I find only the executions related to 03fcdef4-1701-4661-9049-c944b8a8776009:22
d0ugalrakhmerov: yeah09:22
rakhmerovyep, I see09:23
rakhmerovit'd be good09:23
d0ugalso at the moment I only have that workflow, because I delete all executions before running to make it easier09:23
d0ugalbut it would be really useful if I could say "mistral execution-list 03fcdef4-1701-4661-9049-c944b8a87760" and it gave me all the sub-workflows (and the sub-sub-workflows etc)09:23
rakhmerovwe can probably have some pseudo filter for this09:23
d0ugalyeah09:23
mgershend0ugal: you can try and run a workflow that gets all tasks of the execution recursively and them get the execution_id from there distinctly using yaql.09:24
d0ugalmgershen: oh, that is a cool idea.09:24
d0ugalit also sounds fun :)09:24
rakhmerovyes, it is there now09:24
d0ugalOkay, I'll try this!09:24
rakhmerovwe can expose it in the API too somehow09:24
rakhmerovthat would be nice09:24
rakhmerovbtw, what Toure is doing can probably help with this too09:25
rakhmerovit should show the structure of workflow/subworkflows09:25
d0ugalyeah, I was just reviewing his patch which is what made me think of it :)09:25
rakhmerovaha09:25
mgershenit will take work to get it done.09:26
mgershenI mean even with the workflow09:27
d0ugalyup09:29
mgershenI belive something like this will get you the execution IDs: <% tasks(TOP_LVL_EXEC, true).workflow_execution_id.distinct() %>09:31
mgershend0ugal: you using the http://yaqluator.com/ ?09:34
d0ugalmgershen: no, I use the YAQL repl sometimes.09:34
d0ugalmgershen: I did try it once but it has an old version of YAQL and didn't have the features I wanted09:35
mgershenok, it now has yaql 1.1.0 if you want to try it again. I like it because of the auto-completion based on content.09:39
d0ugalmgershen: oh cool, I don't remember auto-completion. I'll try it again.09:41
mgershend0ugal: maybe you are typing too fast haha. see here: https://pasteboard.co/1PfMFMUd.png09:55
d0ugalmgershen: very cool :)09:56
*** shardy has quit IRC10:49
*** openstackgerrit has joined #openstack-mistral10:54
openstackgerritSharat Sharma proposed openstack/python-mistralclient master: Add pagination options for workflows and actions  https://review.openstack.org/45851510:54
*** jkilpatr has joined #openstack-mistral10:56
*** jkilpatr has quit IRC10:56
*** jkilpatr has joined #openstack-mistral10:57
openstackgerritbhavani proposed openstack/mistral-specs master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47888611:45
*** amoralej is now known as amoralej|lunch11:56
*** thrash|g0ne is now known as thrash11:57
openstackgerritbhavani proposed openstack/mistral-specs master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47888612:03
*** abhishekk has quit IRC12:28
*** bobh has joined #openstack-mistral12:29
*** bobh has quit IRC12:32
*** chlong_ has joined #openstack-mistral12:35
*** dmellado has quit IRC12:39
*** amoralej|lunch has quit IRC12:39
*** dmellado has joined #openstack-mistral12:42
*** bobh has joined #openstack-mistral12:43
openstackgerritSharat Sharma proposed openstack/python-mistralclient master: Add pagination options for workflows and actions  https://review.openstack.org/45851512:44
*** thrash is now known as thrash|biab12:46
*** dprince has joined #openstack-mistral12:47
*** jkilpatr has quit IRC12:49
*** jkilpatr has joined #openstack-mistral12:49
*** bobh has quit IRC12:53
*** shardy has joined #openstack-mistral12:54
*** sharat has quit IRC12:58
*** brunograz has joined #openstack-mistral12:59
*** jkilpatr has quit IRC13:26
*** jkilpatr has joined #openstack-mistral13:28
*** toure|gone is now known as toure13:30
*** jkilpatr_ has joined #openstack-mistral13:34
*** jkilpatr_ has quit IRC13:36
*** jkilpatr_ has joined #openstack-mistral13:36
*** jkilpatr has quit IRC13:37
*** bobh has joined #openstack-mistral13:39
*** livelace has joined #openstack-mistral13:46
*** mgershen has quit IRC13:58
toured0ugal ping14:01
*** mgershen has joined #openstack-mistral14:01
d0ugaltoure: hey14:02
tourequestion in regards to the sqlalchemy structure in my proposal14:02
toured0ugal would a join solve that problem14:03
tourecombined with the filter14:03
d0ugaltoure: with sub workflows?14:03
d0ugalno, I don't think do14:03
toureyeah14:03
tourewhat did you have in mind in regards?14:03
d0ugalI don't have a solution :)14:03
tourehehe14:03
toureok, I will play around with the database and see if I can come up with something14:04
toured0ugal would each sub_workflow be considered a unique workflow_Execution with a parent?14:06
d0ugaltoure: I don't think it will be possible with one query14:06
d0ugaltoure: yes14:06
* toure accepts d0ugal's challenge 14:07
toure:)14:07
d0ugalhaha, good luck14:07
* toure will bend sqlalchemy to his will 14:07
tourelol14:07
d0ugaltoure: the workflow has a reference to the current task exectution, that task execution links back to the execution of the sub-workflow14:07
d0ugalIIUC14:07
touredo you have a quick example of a sub_workflow yaml14:08
d0ugaltoure: baremetal.yaml?14:08
d0ugalin tripleo-common14:08
d0ugal but there it isn't really a feature of the workflow language14:09
toureah14:09
d0ugalany workflow can call any other workflow14:09
tourek14:09
*** thrash|biab is now known as thrash14:09
toured0ugal I think I got it figured out in my mind at least, so if we take the approach of starting with the parent workflow query then extract the execution id(s) we should capture everything14:15
toureso we can get a count from the parent in regards to execution_id then traverse each id14:16
d0ugalright14:16
d0ugalHaving to do N queries is a pain, but getting that working is a good starting point.14:16
* toure sqlalchemy-Fu will be raised a notch 14:16
tourewell you wouldn't have to do N queries as the count returned can load a loop to perform the task14:17
toureif I am thinking clearly :)14:18
d0ugalI'm not sure14:18
d0ugalI'd have to try it and see14:18
toureI am working on it14:19
*** jkilpatr_ has quit IRC14:26
*** jkilpatr has joined #openstack-mistral14:27
*** zhurong has joined #openstack-mistral14:31
*** zhurong has quit IRC14:40
*** brunograz has quit IRC15:07
openstackgerritAndras Kovi proposed openstack/mistral master: Update docker build  https://review.openstack.org/47387615:10
*** jkilpatr has quit IRC15:16
d0ugal15:16
*** jistr is now known as jistr|afk15:20
*** jkilpatr has joined #openstack-mistral15:29
*** jistr|afk is now known as jistr16:07
*** jpich has quit IRC16:28
*** dtantsur is now known as dtantsur|afk16:50
*** bobh has quit IRC17:04
*** dprince has quit IRC18:32
*** dprince has joined #openstack-mistral18:48
*** livelace has quit IRC18:59
*** bobh has joined #openstack-mistral19:04
*** bobh has quit IRC19:25
*** bobh has joined #openstack-mistral19:33
openstackgerritToure Dunnon proposed openstack/mistral master: Workflow Error Analysis  https://review.openstack.org/45544719:54
*** bobh has quit IRC19:55
*** toure is now known as toure|gone20:03
*** dprince has quit IRC20:44
openstackgerritMichal Gershenzon proposed openstack/mistral-specs master: Create and run workflows within a namespace  https://review.openstack.org/47099621:41
*** shardy has quit IRC22:09
*** jamielennox has quit IRC22:57
*** jamielennox has joined #openstack-mistral23:03
*** dougshelley66 has quit IRC23:50

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