| *** d0ugal has joined #openstack-mistral | 00:01 | |
| *** weshay has quit IRC | 01:30 | |
| *** weshay has joined #openstack-mistral | 01:36 | |
| *** EmilienM has quit IRC | 03:22 | |
| *** EmilienM has joined #openstack-mistral | 03:23 | |
| *** hardikjasani has joined #openstack-mistral | 04:35 | |
| *** openstackgerrit has joined #openstack-mistral | 07:14 | |
| openstackgerrit | Kupai József proposed openstack/mistral master: A mechanism to close stuck running action executions https://review.openstack.org/527085 | 07:14 |
|---|---|---|
| *** jtomasek has joined #openstack-mistral | 07:15 | |
| *** shardy has quit IRC | 07:19 | |
| *** shardy has joined #openstack-mistral | 07:19 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 07:35 | |
| *** jistr is now known as jistr|mtgs | 07:38 | |
| *** gkadam has joined #openstack-mistral | 07:44 | |
| openstackgerrit | Kupai József proposed openstack/mistral master: A mechanism to close stuck running action executions https://review.openstack.org/527085 | 07:47 |
| *** manuelstein has joined #openstack-mistral | 07:49 | |
| *** jaosorior has joined #openstack-mistral | 08:05 | |
| *** gkadam has quit IRC | 08:05 | |
| openstackgerrit | Oleksiy Petrenko proposed openstack/mistral-specs master: New more user friendly documentation for Mistral https://review.openstack.org/571710 | 08:09 |
| *** gkadam has joined #openstack-mistral | 08:10 | |
| *** AlexeyAbashkin has quit IRC | 08:27 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 08:27 | |
| opetrenko_ | d0ugal: are you here? | 08:44 |
| d0ugal | opetrenko_: Hi, yes I am! | 08:44 |
| d0ugal | I see you have been busy :) | 08:44 |
| opetrenko_ | ? | 08:44 |
| d0ugal | https://review.openstack.org/571710 | 08:45 |
| opetrenko_ | Yeah, I wanted to discuss this "skeleton" | 08:46 |
| *** manuelstein has quit IRC | 08:46 | |
| d0ugal | Sure | 08:46 |
| d0ugal | I still need to take a look at it :) | 08:46 |
| opetrenko_ | thats just an idea what should be in readme of mistral | 08:46 |
| opetrenko_ | ok, then I go take a smoke for a while | 08:47 |
| opetrenko_ | here | 09:02 |
| d0ugal | opetrenko_: I am juggling a few things, so I wont get a chance to read it right now. I'll get to it today. | 09:09 |
| opetrenko_ | ok | 09:10 |
| d0ugal | This is a busy week :) | 09:10 |
| *** thrash has joined #openstack-mistral | 10:18 | |
| *** thrash has quit IRC | 10:18 | |
| *** thrash has joined #openstack-mistral | 10:18 | |
| *** thrash|g0ne has quit IRC | 10:20 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-mistralclient master: Migrate python-mistralclient jobs to Zuul v3 https://review.openstack.org/509184 | 12:00 |
| *** apetrich has quit IRC | 12:23 | |
| *** apetrich has joined #openstack-mistral | 12:24 | |
| *** openstackgerrit has quit IRC | 12:34 | |
| *** toure|gone is now known as toure | 13:01 | |
| *** goutham has joined #openstack-mistral | 13:01 | |
| *** rbrady has joined #openstack-mistral | 13:23 | |
| *** rbrady has quit IRC | 13:23 | |
| *** rbrady has joined #openstack-mistral | 13:23 | |
| *** goutham has left #openstack-mistral | 13:30 | |
| *** openstackgerrit has joined #openstack-mistral | 13:59 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-mistralclient master: Revert "Adding environment-get-definition for easy environment-update" https://review.openstack.org/572111 | 13:59 |
| d0ugal | rakhmerov: Hey - you around today? | 14:00 |
| *** hardikjasani has quit IRC | 14:04 | |
| pgaxatte | hello | 14:27 |
| pgaxatte | is there a way to stop a running workflow (which apparently is not willing to end on itself) | 14:27 |
| d0ugal | Hey folks! | 15:01 |
| d0ugal | It is office hour time! | 15:01 |
| d0ugal | #startmeeting mistral | 15:01 |
| openstack | Meeting started Mon Jun 4 15:01:14 2018 UTC and is due to finish in 60 minutes. The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| *** openstack changes topic to " (Meeting topic: mistral)" | 15:01 | |
| openstack | The meeting name has been set to 'mistral' | 15:01 |
| d0ugal | Who is around today? | 15:01 |
| opetrenko_ | here | 15:01 |
| d0ugal | opetrenko_: Hey | 15:01 |
| d0ugal | pgaxatte: did you manage to answer your question? | 15:01 |
| opetrenko_ | what question? | 15:01 |
| pgaxatte | d0ugal: I think so :) I used "openstack workflow execution update -s CANCELLED" | 15:02 |
| d0ugal | opetrenko_: How to stop a running execution | 15:02 |
| d0ugal | pgaxatte: yup, that is correct | 15:02 |
| d0ugal | We should document that and/or make a more obvious way to do it. | 15:03 |
| pgaxatte | yeah it feels like I'm just setting a flag somewhere in DB and that this command will not actively stop anything | 15:04 |
| pgaxatte | but it seems it does | 15:04 |
| opetrenko_ | New topic for new documentation | 15:04 |
| opetrenko_ | Probably we should ask as much people as we can, what do they need in docs | 15:05 |
| pgaxatte | also I noticed the execution delete <id> does not work in this case | 15:05 |
| pgaxatte | and the --force just returns an error | 15:05 |
| d0ugal | pgaxatte: In what case? What was the error? | 15:05 |
| d0ugal | opetrenko_: Agreed, it would be good to get feedback. | 15:06 |
| pgaxatte | openstack workflow execution delete --force 5022e724-e582-4394-9b06-2271b0ea1fa5 | 15:06 |
| pgaxatte | Unknown argument: "force" | 15:06 |
| pgaxatte | Unable to delete the specified execution(s). | 15:06 |
| pgaxatte | the workflow was still running | 15:06 |
| d0ugal | pgaxatte: Which version are you using? --force is quite new. | 15:06 |
| opetrenko_ | Yeah, because now There is only my opinion about what should be in it, but we can find minimal required topic for everyone | 15:07 |
| d0ugal | opetrenko_: I added the Mistral core reviewers to your patch to try and get more eyes on it. | 15:07 |
| pgaxatte | 3.14.0 for openstack cli, 3.5.0 for mistralclient and 5.2.2 for mistral infra | 15:07 |
| opetrenko_ | well, thats cool but not really | 15:07 |
| opetrenko_ | we need opinion from other users | 15:08 |
| d0ugal | pgaxatte: So you have --force in mistralclient, but not in mistral server. | 15:08 |
| pgaxatte | d0ugal, reading back my terminal, I got the exact same error without --force | 15:08 |
| d0ugal | opetrenko_: Yes, of course, but it is a start. | 15:08 |
| d0ugal | pgaxatte: Right, I guess you have hit a bug somewhere. I would check the mistral logs for errors | 15:08 |
| d0ugal | opetrenko_: I don't have a good way to contact users and get their feedback. | 15:10 |
| pgaxatte | d0ugal: the bug is client-side since I don't give the --force but it still appears in the URL called | 15:12 |
| d0ugal | pgaxatte: aha, damn | 15:12 |
| d0ugal | that is bad | 15:12 |
| d0ugal | pgaxatte: If you downgrade to mistralclient 3.4.0 that will solve it | 15:13 |
| d0ugal | I'm going to open a bug for this, we need a fix. Other people will hit this soon | 15:13 |
| pgaxatte | by the way, the execution delete, is just supposed to remove it from the database like the expiration processus would do, right? | 15:14 |
| d0ugal | pgaxatte: yup | 15:14 |
| pgaxatte | if the workflow is still running, would it kill it? | 15:14 |
| d0ugal | pgaxatte: but the expiration wont delete running workflows | 15:14 |
| d0ugal | pgaxatte: yes | 15:14 |
| pgaxatte | ok | 15:14 |
| pgaxatte | there is no automatic expiration of any kind for a running workflow? | 15:15 |
| d0ugal | pgaxatte: No, I don't believe so | 15:15 |
| d0ugal | https://bugs.launchpad.net/mistral/+bug/1775027 | 15:15 |
| openstack | Launchpad bug 1775027 in Mistral "Execution deletion doesn't work if you use mistralclient 3.5.0 with Queens or earlier" [Critical,Confirmed] | 15:15 |
| d0ugal | I'll take a look into this bug tomorrow. | 15:16 |
| d0ugal | I suspect some of my colleagues will hit it soon :) | 15:17 |
| d0ugal | pgaxatte: ^ | 15:17 |
| pgaxatte | d0ugal, ok great :) | 15:17 |
| pgaxatte | I think I found it | 15:18 |
| openstackgerrit | Toure Dunnon proposed openstack/mistral-extra master: Add Ansible actions to Mistral https://review.openstack.org/514360 | 15:18 |
| pgaxatte | d0ugal: https://git.openstack.org/cgit/openstack/python-mistralclient/tree/mistralclient/api/v2/executions.py#n122 | 15:19 |
| d0ugal | pgaxatte: yup, exactly | 15:20 |
| d0ugal | pgaxatte: If you like you can propose a fix - we just need to make sure the parameter isn't included unless it is set to true. | 15:21 |
| pgaxatte | force is always included in the url parameters | 15:21 |
| pgaxatte | exactly | 15:21 |
| pgaxatte | I can try to submit that :) | 15:21 |
| d0ugal | Great | 15:21 |
| d0ugal | If you don't get a chance I can do it tomorrow | 15:21 |
| openstackgerrit | Pierre Gaxatte proposed openstack/python-mistralclient master: Fix force parameter being always added to URL in execution delete https://review.openstack.org/572136 | 15:30 |
| pgaxatte | d0ugal: let me know if the fix is ok with you, if needed I can take another look tomorrow | 15:32 |
| d0ugal | pgaxatte: I think that looks pretty good. | 15:33 |
| pgaxatte | i tried running tox but it fails even on master | 15:33 |
| d0ugal | I'll try and get apetrich to review it too - he worked on this feature. | 15:33 |
| d0ugal | pgaxatte: oh, what error? | 15:33 |
| pgaxatte | oh nevermind, I wasn't on master, just a test branch :D | 15:34 |
| d0ugal | lol | 15:35 |
| pgaxatte | sometimes your cool prompt on zsh is just too long and you don't see the important info :D | 15:36 |
| d0ugal | I am running the tests locally. | 15:36 |
| pgaxatte | d0ugal: gotta go, I'll read your comments tomorrow if any. Bye! | 15:38 |
| d0ugal | Sure | 15:42 |
| *** AlexeyAbashkin has quit IRC | 15:45 | |
| d0ugal | I'm going to end things a little early | 15:49 |
| d0ugal | Thanks all, hopefully we can continue discussing the documentation and get more input over time. | 15:50 |
| d0ugal | #endmeeting | 15:50 |
| *** openstack changes topic to " (Meeting topic: test)" | 15:50 | |
| openstack | Meeting ended Mon Jun 4 15:50:03 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:50 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-06-04-15.01.html | 15:50 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-06-04-15.01.txt | 15:50 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-06-04-15.01.log.html | 15:50 |
| *** gkadam has quit IRC | 16:08 | |
| *** gkadam has joined #openstack-mistral | 16:08 | |
| *** harlowja has joined #openstack-mistral | 16:28 | |
| *** harlowja has quit IRC | 17:10 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 17:33 | |
| *** Alexey_Abashkin has joined #openstack-mistral | 17:36 | |
| *** AlexeyAbashkin has quit IRC | 17:37 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 17:37 | |
| *** harlowja has joined #openstack-mistral | 17:49 | |
| *** shardy has quit IRC | 17:55 | |
| *** AlexeyAbashkin has quit IRC | 18:09 | |
| *** jaosorior has quit IRC | 19:19 | |
| *** harlowja has quit IRC | 19:29 | |
| *** harlowja has joined #openstack-mistral | 19:59 | |
| *** toure is now known as toure|gone | 20:25 | |
| *** harlowja has quit IRC | 21:04 | |
| *** harlowja has joined #openstack-mistral | 21:36 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!