*** itlinux has quit IRC | 00:51 | |
*** AlexeyAbashkin has joined #openstack-mistral | 00:58 | |
*** AlexeyAbashkin has quit IRC | 01:03 | |
openstackgerrit | Merged openstack/mistral master: Update cut_dict() to return no more than specified by length https://review.openstack.org/557952 | 01:05 |
---|---|---|
*** harlowja has quit IRC | 01:19 | |
*** yangyapeng has joined #openstack-mistral | 01:48 | |
*** yangyape_ has joined #openstack-mistral | 01:50 | |
*** yangyapeng has quit IRC | 01:50 | |
*** zhurong has joined #openstack-mistral | 02:29 | |
*** threestrands has quit IRC | 02:47 | |
*** threestrands has joined #openstack-mistral | 02:47 | |
*** threestrands has quit IRC | 02:47 | |
*** threestrands has joined #openstack-mistral | 02:47 | |
*** threestrands has quit IRC | 02:48 | |
*** threestrands has joined #openstack-mistral | 02:49 | |
*** threestrands has quit IRC | 02:49 | |
*** threestrands has joined #openstack-mistral | 02:49 | |
*** yangyape_ has quit IRC | 02:50 | |
*** yangyapeng has joined #openstack-mistral | 02:50 | |
openstackgerrit | Qi Peng proposed openstack/mistral master: Update SenlinAction https://review.openstack.org/549729 | 02:52 |
*** yangyapeng has quit IRC | 02:55 | |
openstackgerrit | Merged openstack/mistral master: Change unreasonable title https://review.openstack.org/555609 | 02:56 |
*** yangyapeng has joined #openstack-mistral | 03:19 | |
*** yangyapeng has quit IRC | 03:24 | |
*** hardikjasani has joined #openstack-mistral | 03:34 | |
*** yangyapeng has joined #openstack-mistral | 03:35 | |
*** yangyapeng has quit IRC | 03:40 | |
*** yangyapeng has joined #openstack-mistral | 04:01 | |
*** yangyapeng has quit IRC | 04:06 | |
*** akuznetsova has quit IRC | 04:12 | |
*** akuznetsova has joined #openstack-mistral | 04:12 | |
*** yangyapeng has joined #openstack-mistral | 04:30 | |
*** threestrands has quit IRC | 04:38 | |
*** zhurong has quit IRC | 04:55 | |
*** harlowja has joined #openstack-mistral | 05:16 | |
*** yangyapeng has quit IRC | 05:18 | |
*** harlowja has quit IRC | 05:20 | |
*** jtomasek has joined #openstack-mistral | 05:39 | |
*** yangyapeng has joined #openstack-mistral | 05:43 | |
*** yangyapeng has quit IRC | 05:48 | |
*** yangyapeng has joined #openstack-mistral | 06:02 | |
openstackgerrit | Qi Peng proposed openstack/mistral master: Support actions for zun https://review.openstack.org/558699 | 06:04 |
*** yangyapeng has quit IRC | 06:07 | |
*** yangyapeng has joined #openstack-mistral | 06:23 | |
*** jaosorior has quit IRC | 06:25 | |
*** yangyapeng has quit IRC | 06:28 | |
*** shardy has quit IRC | 06:36 | |
*** shardy has joined #openstack-mistral | 06:36 | |
*** jaosorior has joined #openstack-mistral | 06:58 | |
*** yangyapeng has joined #openstack-mistral | 07:34 | |
*** AlexeyAbashkin has joined #openstack-mistral | 07:44 | |
*** shardy has quit IRC | 07:45 | |
*** shardy has joined #openstack-mistral | 07:45 | |
openstackgerrit | Merged openstack/mistral master: Fix WF execution getting stuck on null description https://review.openstack.org/557336 | 07:45 |
*** d0ugal_ has quit IRC | 07:46 | |
*** d0ugal has joined #openstack-mistral | 07:47 | |
*** d0ugal has quit IRC | 07:47 | |
*** d0ugal has joined #openstack-mistral | 07:47 | |
*** gkadam has joined #openstack-mistral | 07:49 | |
*** jpich has joined #openstack-mistral | 08:02 | |
*** yangyapeng has quit IRC | 08:50 | |
*** yangyapeng has joined #openstack-mistral | 09:09 | |
*** yangyapeng has quit IRC | 09:14 | |
d0ugal | rakhmerov: Hey, curious to know what you think of https://review.openstack.org/#/c/558505/ | 09:20 |
d0ugal | I am undecided :) | 09:20 |
*** yangyapeng has joined #openstack-mistral | 09:26 | |
*** yangyapeng has quit IRC | 09:31 | |
*** yangyapeng has joined #openstack-mistral | 09:53 | |
*** yangyapeng has quit IRC | 09:58 | |
*** jtomasek_ has joined #openstack-mistral | 10:07 | |
*** weshay_ has joined #openstack-mistral | 10:14 | |
*** jtomasek has quit IRC | 10:15 | |
*** jrist has quit IRC | 10:15 | |
*** weshay has quit IRC | 10:15 | |
*** jrist has joined #openstack-mistral | 10:19 | |
openstackgerrit | Qi Peng proposed openstack/mistral master: Support actions for zun https://review.openstack.org/558699 | 10:20 |
*** weshay_ has quit IRC | 10:28 | |
*** weshay_ has joined #openstack-mistral | 10:28 | |
*** katkapilatova has joined #openstack-mistral | 11:33 | |
*** katkapilatova has quit IRC | 11:40 | |
*** katkapilatova has joined #openstack-mistral | 11:51 | |
*** dprince has joined #openstack-mistral | 11:55 | |
*** yangyapeng has joined #openstack-mistral | 12:02 | |
*** yangyapeng has quit IRC | 12:06 | |
*** katkapilatova has quit IRC | 12:09 | |
*** yangyapeng has joined #openstack-mistral | 12:14 | |
*** yangyapeng has quit IRC | 12:25 | |
*** yangyapeng has joined #openstack-mistral | 12:42 | |
*** yangyapeng has quit IRC | 12:47 | |
*** shardy is now known as shardy_mtg | 12:58 | |
*** weshay_ is now known as weshay | 13:06 | |
*** yangyapeng has joined #openstack-mistral | 13:30 | |
*** yangyapeng has quit IRC | 13:31 | |
*** hardikjasani has quit IRC | 13:34 | |
*** bobh has joined #openstack-mistral | 14:11 | |
*** d0ugal has quit IRC | 14:39 | |
*** d0ugal has joined #openstack-mistral | 14:41 | |
pgaxatte | hi | 14:44 |
pgaxatte | is there a way to add some context when failing a workflow | 14:45 |
pgaxatte | when I look at an execution I see there is a "State info" field | 14:45 |
pgaxatte | is it possible to fill it with some info when I make workflow fail based on some conditions? | 14:46 |
*** zhurong has joined #openstack-mistral | 15:14 | |
toure | d0ugal ping | 15:31 |
toure | d0ugal so thrash just sent me the wf-report github link you put together should I just incorporate that in the mistral client as I think I am at the point of abandoning my workflow report patch | 15:32 |
*** toure is now known as toure|biab | 15:38 | |
d0ugal | toure|biab: Not sure. I just threw that together because I wanted something "now" - it is far from ideal | 15:39 |
d0ugal | toure|biab: it sort-of follows the approach I suggeted a bunch of times - get something working and see how well it helps etc. | 15:40 |
d0ugal | vs starting with a new API | 15:40 |
*** thrash is now known as thrash|biab | 15:43 | |
d0ugal | pgaxatte: You can use the fail engine command and pass a messasge | 15:48 |
d0ugal | pgaxatte: https://github.com/openstack/mistral/blob/master/mistral/tests/unit/engine/test_commands.py#L379 | 15:48 |
pgaxatte | d0ugal: thanks, I just noticed that :) | 15:48 |
pgaxatte | d0ugal I have an additional question though but regarding the "switch" use case with a noop action | 15:49 |
d0ugal | switch use case? | 15:49 |
pgaxatte | when doing something like this http://paste.openstack.org/show/718396/ | 15:50 |
pgaxatte | i'd like to use to different fails to identify the source of the error | 15:50 |
pgaxatte | but both fail condition are evaluated | 15:51 |
pgaxatte | on a more realistic example, i'd like to first check the existence of an object and then test one property for instance | 15:51 |
pgaxatte | but i would have to repeat the existence test in every fail | 15:52 |
pgaxatte | shouldn't the first fail condition be evaluated, then fail everything? | 15:53 |
d0ugal | pgaxatte: looking | 15:53 |
pgaxatte | it feels like the second fail condition should never be evaluated but it is | 15:53 |
d0ugal | pgaxatte: no, they should always be evaluated | 15:54 |
d0ugal | pgaxatte: it is a bit strange in this case - you wouldn't want to fall fail twice | 15:54 |
d0ugal | but you might want to call two other tasks conditionally | 15:54 |
pgaxatte | its a bit odd I agree but it would be useful to provide a meaningful error | 15:55 |
pgaxatte | maybe this would be a bit easier to understand why I'd like to do this : http://paste.openstack.org/show/718397/ | 15:58 |
d0ugal | pgaxatte: Yeah, I understand - you want a series of conditions that must all pass | 16:00 |
d0ugal | but that isn't how it is designed to work :) | 16:00 |
pgaxatte | d0ugal: exactly :) | 16:01 |
pgaxatte | do you see a proper way to do this? | 16:01 |
pgaxatte | cut the series in multiple tasks with only one failing condition each time? | 16:02 |
d0ugal | pgaxatte: yeah, multiple tasks is probably best - or combine them into one YAQL expression | 16:04 |
pgaxatte | d0ugal yeah I'll do this it'll more readable | 16:06 |
pgaxatte | thanks :) | 16:06 |
d0ugal | np | 16:07 |
*** zhurong has quit IRC | 16:29 | |
*** toure|biab is now known as toure | 16:36 | |
*** AlexeyAbashkin has quit IRC | 16:38 | |
*** dprince has quit IRC | 16:59 | |
*** jpich has quit IRC | 17:11 | |
*** dprince has joined #openstack-mistral | 17:26 | |
*** d0ugal has quit IRC | 17:42 | |
*** shardy_mtg has quit IRC | 18:06 | |
*** AlexeyAbashkin has joined #openstack-mistral | 18:08 | |
*** AlexeyAbashkin has quit IRC | 18:12 | |
*** d0ugal has joined #openstack-mistral | 18:12 | |
*** AlexeyAbashkin has joined #openstack-mistral | 18:58 | |
openstackgerrit | John Eckersberg proposed openstack/mistral master: Use eventlet-aware threading events https://review.openstack.org/557487 | 18:58 |
*** gkadam has quit IRC | 19:00 | |
*** AlexeyAbashkin has quit IRC | 19:02 | |
*** harlowja has joined #openstack-mistral | 19:10 | |
openstackgerrit | Bob Haddleton proposed openstack/mistral master: Update cut_list() to return the specified number of characters https://review.openstack.org/558902 | 19:20 |
openstackgerrit | Bob Haddleton proposed openstack/mistral stable/queens: Update cut_dict() to return no more than specified by length https://review.openstack.org/558905 | 19:27 |
*** gkadam has joined #openstack-mistral | 19:42 | |
*** yangyapeng has joined #openstack-mistral | 20:05 | |
*** yangyapeng has quit IRC | 20:09 | |
*** jtomasek_ has quit IRC | 20:44 | |
*** d0ugal has quit IRC | 21:21 | |
*** gkadam has quit IRC | 21:36 | |
*** d0ugal has joined #openstack-mistral | 21:51 | |
*** yangyapeng has joined #openstack-mistral | 21:53 | |
*** yangyapeng has quit IRC | 21:58 | |
*** AlexeyAbashkin has joined #openstack-mistral | 21:59 | |
*** AlexeyAbashkin has quit IRC | 22:04 | |
*** bobh has quit IRC | 22:57 | |
*** d0ugal has quit IRC | 23:03 | |
*** Guest98296 has joined #openstack-mistral | 23:31 | |
*** thrash|biab is now known as thrash|g0ne | 23:54 | |
*** thrash|g0ne is now known as thrash | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!