| *** bobh has joined #openstack-mistral | 00:44 | |
| *** zhurong has joined #openstack-mistral | 01:16 | |
| *** pengdake has joined #openstack-mistral | 01:45 | |
| *** zhurong has quit IRC | 02:36 | |
| *** zhurong has joined #openstack-mistral | 02:37 | |
| *** bobh has quit IRC | 04:11 | |
| *** bobh has joined #openstack-mistral | 04:20 | |
| *** bobh has quit IRC | 04:25 | |
| *** bobh has joined #openstack-mistral | 04:34 | |
| *** bobh has quit IRC | 04:39 | |
| *** itlinux has quit IRC | 04:39 | |
| *** bobh has joined #openstack-mistral | 04:43 | |
| *** bobh has quit IRC | 04:48 | |
| *** bobh has joined #openstack-mistral | 04:51 | |
| *** bobh has quit IRC | 04:57 | |
| *** bobh has joined #openstack-mistral | 05:00 | |
| *** zhurong has quit IRC | 05:04 | |
| *** bobh has quit IRC | 05:04 | |
| *** bobh has joined #openstack-mistral | 05:10 | |
| *** bobh has quit IRC | 05:10 | |
| *** bobh has joined #openstack-mistral | 05:21 | |
| *** bobh has quit IRC | 05:25 | |
| *** openstackgerrit has quit IRC | 05:49 | |
| *** bobh has joined #openstack-mistral | 05:53 | |
| *** bobh has quit IRC | 05:58 | |
| *** bobh has joined #openstack-mistral | 06:00 | |
| *** bobh has quit IRC | 06:05 | |
| *** openstackgerrit has joined #openstack-mistral | 06:07 | |
| openstackgerrit | Qi Peng proposed openstack/mistral master: Update test-requirements https://review.openstack.org/550363 | 06:07 |
|---|---|---|
| *** bobh has joined #openstack-mistral | 06:12 | |
| *** bobh has quit IRC | 06:17 | |
| *** bobh has joined #openstack-mistral | 06:20 | |
| *** bobh has quit IRC | 06:25 | |
| *** bobh has joined #openstack-mistral | 06:31 | |
| *** bobh has quit IRC | 06:37 | |
| *** bobh has joined #openstack-mistral | 06:41 | |
| *** bobh has quit IRC | 06:46 | |
| *** d0ugal has joined #openstack-mistral | 07:00 | |
| *** d0ugal has quit IRC | 07:00 | |
| *** d0ugal has joined #openstack-mistral | 07:00 | |
| *** zhurong has joined #openstack-mistral | 07:00 | |
| *** bobh has joined #openstack-mistral | 07:03 | |
| *** bobh has quit IRC | 07:08 | |
| *** bobh has joined #openstack-mistral | 07:11 | |
| *** bobh has quit IRC | 07:16 | |
| *** bobh has joined #openstack-mistral | 07:23 | |
| *** bobh has quit IRC | 07:27 | |
| openstackgerrit | Merged openstack/mistral master: Fixed the broken MySQL job https://review.openstack.org/547808 | 07:28 |
| openstackgerrit | Merged openstack/mistral master: Clarify usage of break-on and continue-on https://review.openstack.org/553386 | 07:28 |
| openstackgerrit | Merged openstack/mistral-tempest-plugin master: Updated from global requirements https://review.openstack.org/553197 | 07:28 |
| apetrich | Morning | 07:39 |
| *** bobh has joined #openstack-mistral | 07:42 | |
| *** bobh has quit IRC | 07:47 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 07:54 | |
| *** zhurong has quit IRC | 07:54 | |
| *** bobh has joined #openstack-mistral | 07:55 | |
| *** bobh has quit IRC | 07:59 | |
| openstackgerrit | Andras Kovi proposed openstack/mistral master: Ensure workflow check is retried on DB error https://review.openstack.org/547069 | 08:12 |
| openstackgerrit | Andras Kovi proposed openstack/mistral master: WIP: Enable delayed call rescheculing before execution on execution integrity check https://review.openstack.org/552911 | 08:14 |
| openstackgerrit | Merged openstack/mistral master: Fix server info string https://review.openstack.org/552703 | 08:19 |
| openstackgerrit | Merged openstack/mistral master: Add a release note for execution events noitifications https://review.openstack.org/550779 | 08:19 |
| openstackgerrit | Merged openstack/mistral master: Remove a redundant initializer of InvalidUnicodeAction in the tests https://review.openstack.org/552834 | 08:19 |
| openstackgerrit | Andras Kovi proposed openstack/mistral master: WIP: Enable delayed call rescheculing before execution on execution integrity check https://review.openstack.org/552911 | 08:23 |
| *** jaosorior has joined #openstack-mistral | 08:40 | |
| *** bobh has joined #openstack-mistral | 08:51 | |
| *** bobh has quit IRC | 08:56 | |
| *** shardy has joined #openstack-mistral | 08:59 | |
| openstackgerrit | Merged openstack/mistral master: Rename task_id to task_execution_id https://review.openstack.org/514688 | 09:01 |
| *** jtomasek has joined #openstack-mistral | 09:02 | |
| *** bobh has joined #openstack-mistral | 09:14 | |
| *** bobh has quit IRC | 09:19 | |
| *** chkumar|off is now known as chandankumare | 09:21 | |
| *** chandankumare is now known as chandankumar | 09:21 | |
| *** bobh has joined #openstack-mistral | 09:21 | |
| *** bobh has quit IRC | 09:26 | |
| *** bobh has joined #openstack-mistral | 09:30 | |
| openstackgerrit | Adriano Petrich proposed openstack/mistral master: Ensure workflow check is retried on DB error https://review.openstack.org/547069 | 09:34 |
| *** bobh has quit IRC | 09:35 | |
| openstackgerrit | Merged openstack/mistral master: Retry DB operation on deadlock https://review.openstack.org/550864 | 09:38 |
| *** bobh has joined #openstack-mistral | 09:57 | |
| *** bobh has quit IRC | 10:01 | |
| *** bobh has joined #openstack-mistral | 10:05 | |
| *** bobh has quit IRC | 10:10 | |
| *** bobh has joined #openstack-mistral | 10:12 | |
| *** bobh has quit IRC | 10:16 | |
| *** bobh has joined #openstack-mistral | 10:22 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: fixing 'pause' engine command https://review.openstack.org/552455 | 10:26 |
| *** bobh has quit IRC | 10:28 | |
| *** pengdake has quit IRC | 10:33 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: fixing 'pause' engine command https://review.openstack.org/552455 | 10:38 |
| *** bobh has joined #openstack-mistral | 10:45 | |
| *** bobh has quit IRC | 10:51 | |
| *** bobh has joined #openstack-mistral | 10:51 | |
| *** bobh has quit IRC | 10:56 | |
| *** bobh has joined #openstack-mistral | 11:01 | |
| *** bobh has quit IRC | 11:06 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix 'pause' engine command https://review.openstack.org/552455 | 11:11 |
| *** bobh has joined #openstack-mistral | 11:13 | |
| rakhmerov | d0ugal: hi, can you please approve https://review.openstack.org/#/c/552870/? | 11:13 |
| rakhmerov | it's a backport to queens | 11:13 |
| rakhmerov | I also would like to release a new version of mistral this week from stable/queens (we need to promote this to RDO) | 11:15 |
| d0ugal | rakhmerov: sure | 11:17 |
| *** bobh has quit IRC | 11:18 | |
| *** bobh has joined #openstack-mistral | 11:20 | |
| *** katkapilatova has joined #openstack-mistral | 11:23 | |
| *** bobh has quit IRC | 11:25 | |
| openstackgerrit | Merged openstack/mistral-dashboard master: Updated from global requirements https://review.openstack.org/553195 | 11:26 |
| *** bobh has joined #openstack-mistral | 11:30 | |
| *** bobh has quit IRC | 11:36 | |
| *** bobh has joined #openstack-mistral | 11:40 | |
| *** pengdake has joined #openstack-mistral | 11:44 | |
| *** bobh has quit IRC | 11:45 | |
| apetrich | rakhmerov, oh I'm happy with that being merged and backported :) | 11:51 |
| *** bobh has joined #openstack-mistral | 11:53 | |
| *** pengdake has quit IRC | 12:09 | |
| *** thrash|g0ne is now known as thrash | 12:20 | |
| *** pengdake has joined #openstack-mistral | 12:20 | |
| *** bobh has quit IRC | 12:23 | |
| *** pengdake has quit IRC | 12:31 | |
| *** dprince has joined #openstack-mistral | 12:57 | |
| *** toure|gone is now known as toure | 13:20 | |
| *** shardy is now known as shardy_mtg | 13:28 | |
| openstackgerrit | Merged openstack/mistral stable/queens: Make sure not to swallow original exceptions on handling remote errors https://review.openstack.org/552870 | 14:03 |
| openstackgerrit | Merged openstack/mistral master: Improve the Docker integration https://review.openstack.org/539048 | 14:03 |
| *** bobh has joined #openstack-mistral | 14:13 | |
| *** itlinux has joined #openstack-mistral | 15:00 | |
| *** josecastroleon has joined #openstack-mistral | 15:47 | |
| bobh | anybody here for office hours? | 16:07 |
| d0ugal | bobh: I am around | 16:15 |
| bobh | d0ugal: hello | 16:16 |
| d0ugal | Hey :) | 16:16 |
| bobh | d0ugal: question for you | 16:16 |
| d0ugal | I keep meaning to add an IRC bot to announce when office hours start - it would help remind me :-D | 16:16 |
| d0ugal | Sure | 16:16 |
| bobh | d0ugal: the latest mistralclient added a default LIMIT=100 on task-list, execution-list and action-execution-list commands to reduce the load on the server | 16:16 |
| d0ugal | Yup | 16:17 |
| bobh | d0ugal: that causes problems when you have more than 100 tasks/executions/action-executions because it only returns the first 100 | 16:17 |
| bobh | so you have to specify limit on every command to see the data you want | 16:17 |
| d0ugal | bobh: right, or you could filter the result? | 16:18 |
| d0ugal | How often do you really need to look at more than 100 at a time? | 16:18 |
| bobh | I have been trying to find a way to get it to return the latest 100 rows instead of the first 100 but I can't find a way to do it in oslo_db/sqlalchemy | 16:18 |
| bobh | Never | 16:18 |
| bobh | but I need the latest 100 not the earliest 100 | 16:18 |
| d0ugal | ah | 16:18 |
| d0ugal | You can sort it with the CLI | 16:19 |
| bobh | so sort created_at desc ? | 16:19 |
| bobh | but then I want that result reversed | 16:19 |
| d0ugal | Let me get an example... | 16:21 |
| d0ugal | bobh: mistral execution-list --sort_key="updated_at" --sort_dir="asc" --limit=1 | 16:21 |
| d0ugal | That will get you the most recent one :) | 16:21 |
| d0ugal | It would maybe make sense for us to change the default sort order, because I agree, showing the oldest 100 isn't intuative | 16:21 |
| bobh | hm | 16:22 |
| bobh | I think the defaults are created_at and asc so it "should" work as expected | 16:23 |
| bobh | also sort_ket and sort_dir are not mentioned in the cli help - so another thing to fix :-) | 16:23 |
| bobh | only when you put invalid inputs into task-list | 16:24 |
| bobh | there does not seem to be a way to specify sort-dir | 16:25 |
| bobh | client sends sort_keys=['created_at'] and sort_dir=['asc'] by default | 16:27 |
| bobh | and limit=100 | 16:27 |
| bobh | so you get the oldest 100 | 16:27 |
| bobh | looks like tasks is defined in the client different than executions, which has sort_keys and sort_dir | 16:29 |
| bobh | I'll open a bug to make them consistent and see if I can get the default to be what would be most useful | 16:30 |
| d0ugal | bobh: I see it in the help for "$ mistral help execution-list" | 16:31 |
| bobh | but not in task-list or action-execution-list | 16:31 |
| d0ugal | oh, odd | 16:32 |
| toure | bobh I don't see that option in the client side | 16:32 |
| toure | the cli switches that is | 16:32 |
| bobh | I think it will be easy to get the latest 100 entries by sort_dir=desc, but then need to sort again to reverse the list so it is in the expected order | 16:32 |
| umbSublime | since most cli's are built with cliff now IIRC they all commands should have those available | 16:33 |
| umbSublime | s/they/then/ | 16:33 |
| bobh | it's inheriting from lister.Lister so it should be there | 16:34 |
| d0ugal | yup | 16:34 |
| bobh | I'll dig into it some more | 16:34 |
| toure | yeah, but it isn't defined in the base module | 16:34 |
| toure | each module has to define their own cli switches | 16:35 |
| toure | but bobh you are corrent the mistral server side has the api support exposed | 16:36 |
| bobh | cliff DisplayCommandBase provides --column and --format | 16:37 |
| bobh | toure - right but the server side will only do one sort - the desc to get the latest 100 rows | 16:38 |
| bobh | toure: I need another sort to reverse the result into ascending order | 16:38 |
| bobh | seems clunky to have the client do that, but even more clunky to have the server do it | 16:38 |
| toure | bobh I agree I think the cli should have the option to define a customized output especially since the api is there to support it | 16:39 |
| toure | I think there should be a unified client implementation which matches the api | 16:39 |
| bobh | someone must have solved this problem before but I can't find any evidence of it | 16:40 |
| bobh | I think the default output for the task-list, execution-list and action-execution-list commands should be the most recent 100 entries, in ascending order | 16:42 |
| toure | bobh so do you mean implement something like ABC for client models which make sure there is a matching api implementations | 16:42 |
| toure | ^from base. | 16:42 |
| bobh | toure: I think that's part of it but I think there is a larger problem which is hard to solve even in SQL, which is the "get me the latest N records" | 16:43 |
| bobh | toure: it requires a desc sort followed by an asc sort | 16:43 |
| bobh | toure: which neither oslo_db not sqlalchemy support | 16:44 |
| toure | oh I get it now you want a list which give you a chunk of data which sorted via desc then have the ability to sort that list via asc | 16:45 |
| * toure thinks custom sqlalchmey query is needed | 16:45 | |
| bobh | right | 16:46 |
| bobh | I agree with putting the limit on the task/execution/action-execution list command, since I never need to see all 20000 executions anyway | 16:46 |
| toure | bobh if you open the bug I will take a look at it, I have been spending time in sqlalchemy world for a bit | 16:46 |
| bobh | toure: ok, I wasnt sure that this should be a server-side fix, since we can ask the server for the latest 100 rows (updated_at, desc) and it will give them to us | 16:47 |
| bobh | toure: but then the client would need to do as asc sort or reversed() on the result to make it look like the user expects | 16:48 |
| toure | make sense | 16:49 |
| *** AlexeyAbashkin has quit IRC | 16:51 | |
| *** shardy_mtg has quit IRC | 16:52 | |
| *** weshay is now known as weshay_brb | 17:05 | |
| *** shardy_mtg has joined #openstack-mistral | 17:08 | |
| *** toure is now known as toure|biab | 17:10 | |
| *** thrash is now known as thrash|biab | 17:13 | |
| *** d0ugal has quit IRC | 17:21 | |
| *** shardy_mtg has quit IRC | 17:26 | |
| *** weshay_brb is now known as weshay | 17:50 | |
| *** thrash|biab is now known as thrash | 17:51 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 18:20 | |
| *** AlexeyAbashkin has quit IRC | 18:24 | |
| *** toure|biab is now known as toure | 18:30 | |
| *** rbrady is now known as rbrady-afk | 18:36 | |
| *** openstackgerrit has quit IRC | 18:48 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 19:20 | |
| *** AlexeyAbashkin has quit IRC | 19:24 | |
| *** dprince has quit IRC | 19:33 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 20:21 | |
| *** livelace has joined #openstack-mistral | 20:25 | |
| *** AlexeyAbashkin has quit IRC | 20:26 | |
| *** jtomasek has quit IRC | 21:00 | |
| *** toure is now known as toure|gone | 21:04 | |
| *** itlinux has quit IRC | 21:06 | |
| *** livelace has quit IRC | 21:08 | |
| *** bobh has quit IRC | 22:19 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 22:20 | |
| *** AlexeyAbashkin has quit IRC | 22:25 | |
| *** katkapilatova has quit IRC | 22:36 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 23:20 | |
| *** AlexeyAbashkin has quit IRC | 23:24 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!