| *** threestrands has joined #openstack-mistral | 00:23 | |
| *** threestrands has quit IRC | 00:23 | |
| *** threestrands has joined #openstack-mistral | 00:23 | |
| *** livelace2 has quit IRC | 00:28 | |
| *** livelace2 has joined #openstack-mistral | 00:28 | |
| *** harlowja has quit IRC | 01:09 | |
| *** thrash is now known as thrash|g0ne | 01:22 | |
| openstackgerrit | wangxiyuan proposed openstack/mistral master: Use register_session_conf_options API https://review.openstack.org/572300 | 01:23 |
|---|---|---|
| *** threestrands has quit IRC | 02:54 | |
| *** threestrands has joined #openstack-mistral | 03:10 | |
| *** threestrands has quit IRC | 03:10 | |
| *** threestrands has joined #openstack-mistral | 03:10 | |
| *** hardikjasani has joined #openstack-mistral | 04:19 | |
| *** harlowja has joined #openstack-mistral | 04:30 | |
| *** threestrands has quit IRC | 04:40 | |
| *** harlowja has quit IRC | 04:52 | |
| *** akovi has joined #openstack-mistral | 05:03 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 06:15 | |
| rakhmerov | d0ugal: hi | 06:23 |
| rakhmerov | https://review.openstack.org/#/c/581059/ has 2 +2 but Andras wants you to address my comment ) | 06:23 |
| rakhmerov | I think we can merge it as is but please send a follow up patch | 06:24 |
| rakhmerov | (more patches is always good!) | 06:24 |
| rakhmerov | apetrich, mcdoker181818, akovi: hey, can you please review https://review.openstack.org/#/c/579592/ and https://review.openstack.org/#/c/579570/ ? | 06:24 |
| openstackgerrit | Merged openstack/mistral-lib stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/584126 | 06:26 |
| openstackgerrit | Merged openstack/mistral-lib stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/584127 | 06:26 |
| openstackgerrit | Merged openstack/mistral-lib master: Update reno for stable/rocky https://review.openstack.org/584128 | 06:26 |
| openstackgerrit | Merged openstack/mistral-specs master: Switch to use stestr for unit test https://review.openstack.org/583837 | 06:26 |
| akovi | rakhmerov: done :) | 06:27 |
| rakhmerov | ok | 06:37 |
| rakhmerov | thnx | 06:37 |
| *** AlexeyAbashkin has quit IRC | 06:41 | |
| pgaxatte | hello | 06:52 |
| pgaxatte | can someone have a look at https://review.openstack.org/#/c/579477/ please? | 06:52 |
| *** AlexeyAbashkin has joined #openstack-mistral | 06:58 | |
| *** Alexey_Abashkin has joined #openstack-mistral | 07:01 | |
| *** AlexeyAbashkin has quit IRC | 07:02 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 07:02 | |
| *** jtomasek has joined #openstack-mistral | 07:05 | |
| *** xavierhardy has joined #openstack-mistral | 07:10 | |
| *** xavierhardy1 has joined #openstack-mistral | 07:11 | |
| *** xavierhardy has quit IRC | 07:15 | |
| openstackgerrit | Merged openstack/mistral master: Fix testenv cover in tox.ini https://review.openstack.org/583850 | 07:21 |
| openstackgerrit | Merged openstack/python-mistralclient master: Add missing oslo.serialization requirement https://review.openstack.org/579592 | 07:21 |
| *** gkadam has joined #openstack-mistral | 07:37 | |
| d0ugal | rakhmerov: sure, I'll try and do that follow up today | 07:52 |
| rakhmerov | ok | 07:52 |
| d0ugal | rakhmerov: btw, apetrich isn't around until August 1st | 07:52 |
| rakhmerov | ooh, I didn't know | 07:55 |
| rakhmerov | thanks | 07:55 |
| d0ugal | He probably should have told everyone :) | 07:56 |
| d0ugal | He is moving house to Berlin | 07:56 |
| rakhmerov | hah, interesting.. | 07:56 |
| rakhmerov | ok | 07:56 |
| d0ugal | #startmeeting mistral | 08:04 |
| openstack | Meeting started Fri Jul 20 08:04:13 2018 UTC and is due to finish in 60 minutes. The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:04 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:04 |
| *** openstack changes topic to " (Meeting topic: mistral)" | 08:04 | |
| openstack | The meeting name has been set to 'mistral' | 08:04 |
| d0ugal | Friday office hour! | 08:04 |
| d0ugal | https://etherpad.openstack.org/p/mistral-office-hours | 08:04 |
| d0ugal | rakhmerov, apetrich, bobh, mcdoker181818 ^ | 08:04 |
| d0ugal | Add your nick to line 16 for future pings! | 08:04 |
| d0ugal | I don't have any agenda for today | 08:09 |
| d0ugal | Oh, actually, I do. mistral-lib was released for Rocky today | 08:09 |
| d0ugal | There were very little changes really | 08:10 |
| *** shardy has joined #openstack-mistral | 08:10 | |
| d0ugal | Other than that, no news from me :) | 08:10 |
| d0ugal | https://bugs.launchpad.net/mistral/+bugs?search=Search&field.status=New&orderby=id&start=0 | 08:10 |
| d0ugal | We do have three new bugs! | 08:10 |
| akovi | Regarding #1782076 this is a documented (in code) limitation | 08:11 |
| akovi | We could not transfer the cert file to the server to be used | 08:11 |
| d0ugal | akovi: Do you have a link to the code comment? | 08:12 |
| akovi | Especially because it would have to be present for the executor | 08:12 |
| d0ugal | So the problem is that mistral server doesn't have the cert file? Is there a way to workaround this or is it not possible? | 08:13 |
| rakhmerov | hey | 08:22 |
| mcdoker181818 | As I know we get a cert as parameter and cache it | 08:22 |
| mcdoker181818 | I mean a my company openstack actions | 08:23 |
| mcdoker181818 | rakhmerov: Hi! How "Transitions with expressions" must work with the join policy? | 08:24 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1782305 was filed by one of my colleagues | 08:24 |
| openstack | Launchpad bug 1782305 in Mistral "unable to alter env variable" [Undecided,New] | 08:24 |
| rakhmerov | hi | 08:24 |
| rakhmerov | mcdoker181818: not sure I understand your question | 08:24 |
| d0ugal | rakhmerov: I am actually surprised there isn't a bug for that already - we (tripleo) wanted to do that for a while and I have tried to do it previously too :) | 08:24 |
| akovi | sorry, I had to get out for a little while | 08:24 |
| d0ugal | akovi: np | 08:24 |
| rakhmerov | d0ugal: yeah, it's rather a BP though IMO | 08:25 |
| mcdoker181818 | https://thepasteb.in/p/O7h5jv9KgjOSq | 08:25 |
| rakhmerov | d0ugal: should be easy to implement a YAQL function to do that | 08:25 |
| d0ugal | rakhmerov: Agreed, a bp function would be better. I also thought a YAQL function made sense :) | 08:25 |
| rakhmerov | yep | 08:25 |
| rakhmerov | it's a low hanging fruit actually | 08:25 |
| d0ugal | Yeah, so I am happy to treat it as a bug for that reason. | 08:26 |
| rakhmerov | we can give it to someone who wants to learn | 08:26 |
| rakhmerov | :) | 08:26 |
| akovi | hmm, seems like we lost the target_cacert somewhere in the history or refactoring | 08:26 |
| d0ugal | Good idea. I might have somebody in mind. | 08:26 |
| rakhmerov | mcdoker181818: ok, so in this case there's no difference for "tj" if the transition is conditional or not | 08:26 |
| rakhmerov | both edges are considered preconditions for "tj" | 08:27 |
| rakhmerov | "tj" will be in WAITING state as long as both transitions are still possible and not evaluated yet | 08:27 |
| rakhmerov | and "tj" will be in ERROR state if at least one of those routes will become impossible | 08:28 |
| rakhmerov | mcdoker181818: makes sense? | 08:28 |
| mcdoker181818 | then this workflow will be failed, yes? Yep, make sense | 08:30 |
| akovi | I took #1782076, it is a real bug | 08:39 |
| akovi | but the target cacert works only on the client side | 08:40 |
| akovi | the mistral server still needs to be able to communicate with the target cloud with SSL as the cert is not transferred from the client | 08:40 |
| akovi | Can we talk about these patches: https://review.openstack.org/#/q/topic:service-catalog-issue+(status:open+OR+status:merged) | 08:42 |
| mcdoker181818 | rakhmerov: I think we need to update docs for this case | 08:42 |
| mcdoker181818 | If anybody has time, please review https://review.openstack.org/#/c/583030/ | 08:43 |
| rakhmerov | mcdoker181818: ok | 08:43 |
| akovi | mcdoker181818: what is idp? | 08:45 |
| mcdoker181818 | identity provider | 08:46 |
| akovi | can we write it out? I like self-descriptive names :) | 08:47 |
| d0ugal | +1 | 08:48 |
| mcdoker181818 | problemo. I think it is a common abbreviation :) | 08:49 |
| rakhmerov | IDP yes, in the context of security it's usually a known thing ) | 08:50 |
| akovi | yes, TLWs are common everywhere, meaning everything haha | 08:50 |
| d0ugal | Does that change mean keycloak is enabled by default in the container or it is just an option? | 08:52 |
| * d0ugal reads the README change :) | 08:52 | |
| mcdoker181818 | Noop, it's disabled by default | 08:54 |
| mcdoker181818 | Also, I answered on the comment https://review.openstack.org/#/c/499790/14 | 08:55 |
| d0ugal | mcdoker181818: Are you going to change idp? I'm trying to decide if I should merge it or not :) | 08:57 |
| rakhmerov | mcdoker181818: ok | 08:57 |
| rakhmerov | +2ed | 08:57 |
| mcdoker181818 | identity-provider? | 08:57 |
| mcdoker181818 | Wait a moment | 08:57 |
| d0ugal | Thanks | 08:57 |
| d0ugal | I really wish docker compose had a way to mark services as disabled by default | 08:58 |
| mcdoker181818 | +++ | 08:59 |
| d0ugal | I only started learning how to write a docker-compose file 2 weeks ago. I was surprised how limiting it is | 09:00 |
| d0ugal | I kinda expected more from it. | 09:00 |
| d0ugal | but anyway, it is also quite useful at times | 09:00 |
| d0ugal | Oh, we are at the end of the hour. I am not going anywhere, but I'll stop the meeting bot | 09:01 |
| d0ugal | #endmeeting | 09:01 |
| *** openstack changes topic to "Mistral the Workflow Service for OpenStack. https://docs.openstack.org/mistral/latest/" | 09:01 | |
| openstack | Meeting ended Fri Jul 20 09:01:13 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-07-20-08.04.html | 09:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-07-20-08.04.txt | 09:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-07-20-08.04.log.html | 09:01 |
| mcdoker181818 | You can contribute , docker-compose is a python project unlike docker :) | 09:01 |
| d0ugal | I could... but I am not sure I have the time to navigate the politics :) | 09:02 |
| d0ugal | (I read lots of github issues that were rejected with features that I wanted) | 09:02 |
| mcdoker181818 | Do you mean in the docker-compose issues ? | 09:03 |
| d0ugal | yup | 09:03 |
| openstackgerrit | Merged openstack/python-mistralclient master: Add the root execution ID to the CLI output https://review.openstack.org/579570 | 09:04 |
| openstackgerrit | Vitalii Solodilov proposed openstack/mistral master: Keycloak and Docker https://review.openstack.org/583030 | 09:14 |
| mcdoker181818 | d0ugal: akovi: Done, sorry for delay | 09:16 |
| d0ugal | np | 09:16 |
| mcdoker181818 | * d0ugal reads the README change :) | 09:17 |
| mcdoker181818 | How did you do this? :) | 09:17 |
| d0ugal | "/me message" | 09:17 |
| * mcdoker181818 lol | 09:17 | |
| mcdoker181818 | Thanks! | 09:17 |
| d0ugal | np | 09:17 |
| rakhmerov | 👌 | 09:27 |
| *** xavierhardy1 has quit IRC | 09:28 | |
| mcdoker181818 | (ಥ﹏ಥ) | 09:28 |
| *** xavierhardy has joined #openstack-mistral | 09:29 | |
| *** mcdoker181818 has quit IRC | 09:38 | |
| d0ugal | rakhmerov: https://github.com/openstack/mistral-lib/blob/master/mistral_lib/actions/types.py#L21 | 10:03 |
| d0ugal | Does the base class mean it is automatically serialised over the RPC calls? | 10:04 |
| d0ugal | oh, the base class and the ResultSerializer | 10:04 |
| d0ugal | I guess that is what does it | 10:04 |
| d0ugal | I think I need to add the same thing to the Context - so the RPM call to the notifier works correctly. | 10:05 |
| d0ugal | If that is correct, I wish I had realised this before the release yesterday :( | 10:05 |
| *** AlexeyAbashkin has quit IRC | 10:08 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 10:11 | |
| openstackgerrit | Merged openstack/mistral-tempest-plugin master: Use id of workflow instead of name to keep track of what is created https://review.openstack.org/583550 | 10:12 |
| *** AlexeyAbashkin has quit IRC | 10:18 | |
| *** xavierhardy has quit IRC | 10:27 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 10:31 | |
| *** xavierhardy has joined #openstack-mistral | 10:32 | |
| *** xavierhardy1 has joined #openstack-mistral | 10:32 | |
| *** xavierhardy has quit IRC | 10:36 | |
| *** jenner_ has joined #openstack-mistral | 10:39 | |
| * jenner_ waves | 10:40 | |
| jenner_ | guys, is there a nice web ui for mistral except the mistral-dashboard? | 10:41 |
| *** xavierhardy1 has quit IRC | 10:42 | |
| *** xavierhardy has joined #openstack-mistral | 10:42 | |
| akovi | jenner_: shameless self advertisement https://github.com/nokia/CloudFlow :) | 10:52 |
| *** shardy has quit IRC | 10:56 | |
| *** xavierhardy has quit IRC | 10:57 | |
| *** mcdoker181818 has joined #openstack-mistral | 11:00 | |
| jenner_ | akovi: nice! | 11:09 |
| d0ugal | jenner_: What do you use for auth? | 11:14 |
| mcdoker181818 | jenner_: you can use docker-compose https://github.com/openstack/mistral/blob/master/tools/docker/DOCKER_README.rst#running-mistral-using-docker-compose to try a Cloudflow on it | 11:15 |
| mcdoker181818 | d0ugal: Why aren't there a bulk deletion of workflow? | 11:17 |
| akovi | The DB API supports it mistral.db.v2.api.delete_workflow_definitions there's just no REST API correspondent | 11:18 |
| jenner_ | d0ugal: no auth atm | 11:22 |
| jenner_ | mcdoker181818: we're runing mistral in openshift | 11:23 |
| mcdoker181818 | akovi: So there is no problem to add? I looked a bit at the other openstack components, and didn't find a bulk deletion api too | 11:24 |
| mcdoker181818 | jenner_: I am too. Maybe need to contribute kubernetes chart \ openshift tempaltes ... | 11:25 |
| d0ugal | mcdoker181818: bulk delete doesn't seem that common in the APIs, I'm not really sure why not | 11:25 |
| d0ugal | mcdoker181818: The CLI does allow you to delete many resources in one command, but I guess it does many HTTP requests? | 11:26 |
| mcdoker181818 | Do you mean the mistral cli? | 11:26 |
| d0ugal | Yeah | 11:26 |
| d0ugal | mcdoker181818: I run commands like this quite often... | 11:28 |
| d0ugal | mistral workflow-delete (mistral workflow-list -c ID -f value) | 11:28 |
| akovi | I think the problem with bulk deletion is the overload it can cause. Take a look at the expiration policy implementation mistral.services.expiration_policy._delete_executions | 11:29 |
| d0ugal | (That is fish syntax tho') | 11:29 |
| d0ugal | akovi: good point, it is easier to limit/control single delete requests | 11:29 |
| d0ugal | if we allow bulk deleting then we move the problem inside Mistral | 11:29 |
| mcdoker181818 | deletion entity one by one is a pain | 11:33 |
| d0ugal | mcdoker181818: write a workflow to do it ;) | 11:33 |
| mcdoker181818 | :D | 11:34 |
| akovi | d0ugal, mcdoker181818: Once I left a test run for several hours that created like 30K executions. In the end I went to the DB and deleted them manually in 1sec instead of waiting 30k * 3-4 sec :) | 11:36 |
| d0ugal | lol | 11:37 |
| *** xavierhardy has joined #openstack-mistral | 11:40 | |
| mcdoker181818 | :D | 11:46 |
| openstackgerrit | Merged openstack/mistral master: Return the result of the MistralHTTPAction https://review.openstack.org/581059 | 11:47 |
| openstackgerrit | Merged openstack/mistral master: Use on-clause and retry_policy get_spec for validation https://review.openstack.org/499790 | 11:47 |
| d0ugal | mcdoker181818: We could make deleting a workbook remove the workflows in it - that would be a sort-of bulk delete :) | 11:48 |
| mcdoker181818 | d0ugal. It will not help: o(n)/c=o(n) :) | 11:50 |
| akovi | d0ugal: with really big numbers, yes. But I happily take a 10 fold speed up from 10mins to 1min. | 11:52 |
| *** thrash|g0ne is now known as thrash | 12:18 | |
| *** AlexeyAbashkin has quit IRC | 12:31 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 12:35 | |
| *** hardikjasani has quit IRC | 12:39 | |
| *** d0ugal has quit IRC | 12:45 | |
| *** d0ugal has joined #openstack-mistral | 12:45 | |
| *** d0ugal has quit IRC | 12:45 | |
| *** d0ugal has joined #openstack-mistral | 12:45 | |
| *** AlexeyAbashkin has quit IRC | 12:55 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 12:56 | |
| *** d0ugal has quit IRC | 12:57 | |
| *** d0ugal has joined #openstack-mistral | 13:10 | |
| *** rbrady_ has joined #openstack-mistral | 13:16 | |
| *** rbrady_ has joined #openstack-mistral | 13:16 | |
| *** rbrady has quit IRC | 13:17 | |
| *** EmilienM is now known as EvilienM | 13:41 | |
| *** jtomasek has quit IRC | 14:28 | |
| *** xavierhardy has quit IRC | 14:58 | |
| openstackgerrit | Kevin Pouget proposed openstack/python-mistralclient master: Add missing `--public` option to workbook api https://review.openstack.org/584415 | 15:03 |
| *** xavierhardy has joined #openstack-mistral | 15:08 | |
| *** thrash is now known as thrash|bbl | 15:37 | |
| *** AlexeyAbashkin has quit IRC | 16:13 | |
| *** akovi has quit IRC | 16:24 | |
| *** gkadam has quit IRC | 16:29 | |
| *** thrash|bbl is now known as thrash | 16:45 | |
| *** mcdoker181818 has quit IRC | 16:51 | |
| *** jtomasek has joined #openstack-mistral | 17:28 | |
| *** jtomasek has quit IRC | 17:47 | |
| *** jtomasek has joined #openstack-mistral | 17:48 | |
| *** xavierhardy has quit IRC | 17:54 | |
| *** jtomasek has quit IRC | 17:54 | |
| *** rbrady__ has joined #openstack-mistral | 21:08 | |
| *** rbrady_ has quit IRC | 21:11 | |
| *** toure is now known as toure|gone | 21:12 | |
| *** rbrady__ has quit IRC | 21:42 | |
| *** weshay is now known as weshay_PTO | 22:15 | |
| *** bobh has joined #openstack-mistral | 23:18 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!