| *** bobh has joined #openstack-mistral | 00:04 | |
| *** bobh has quit IRC | 00:09 | |
| *** livelace has quit IRC | 00:11 | |
| *** livelace has joined #openstack-mistral | 00:11 | |
| *** bobh has joined #openstack-mistral | 00:15 | |
| *** bobh has quit IRC | 00:19 | |
| *** bobh has joined #openstack-mistral | 00:23 | |
| *** bobh has quit IRC | 00:27 | |
| *** bobh has joined #openstack-mistral | 00:47 | |
| *** bobh has quit IRC | 00:51 | |
| *** bobh has joined #openstack-mistral | 01:10 | |
| *** bobh has quit IRC | 01:14 | |
| *** bobh has joined #openstack-mistral | 01:15 | |
| *** bobh has quit IRC | 01:19 | |
| *** bobh has joined #openstack-mistral | 01:26 | |
| *** bobh has quit IRC | 01:31 | |
| *** bobh has joined #openstack-mistral | 01:36 | |
| *** bobh has quit IRC | 02:14 | |
| *** bobh has joined #openstack-mistral | 02:35 | |
| *** bobh has quit IRC | 02:58 | |
| *** gkadam_ has joined #openstack-mistral | 03:24 | |
| *** hardikjasani has joined #openstack-mistral | 03:55 | |
| *** gkadam_ has quit IRC | 04:30 | |
| rakhmerov | apetrich: it's not needed really because it just populates the "session" function argument with the session from thread local storage (or it first creates a new one, if needed) and ends a transaction if the session had to be created specifically for this method | 04:39 |
|---|---|---|
| rakhmerov | it's enough that the most inner method that directly uses the session is wrapped with this decorator | 04:40 |
| rakhmerov | for symmetry though we can add a decorator but it'll be useless really | 04:40 |
| *** apetrich has joined #openstack-mistral | 05:05 | |
| *** shardy has joined #openstack-mistral | 07:39 | |
| *** shardy has quit IRC | 07:40 | |
| *** shardy has joined #openstack-mistral | 07:41 | |
| *** jaosorior has quit IRC | 07:47 | |
| *** zhenguo has joined #openstack-mistral | 07:50 | |
| d0ugal | #startmeeting mistral | 08:06 |
| openstack | Meeting started Fri Aug 17 08:06:53 2018 UTC and is due to finish in 60 minutes. The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:06 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:06 |
| d0ugal | Morning | 08:06 |
| *** openstack changes topic to " (Meeting topic: mistral)" | 08:06 | |
| openstack | The meeting name has been set to 'mistral' | 08:06 |
| d0ugal | Sorry I'm late! | 08:06 |
| d0ugal | https://etherpad.openstack.org/p/mistral-office-hours | 08:07 |
| d0ugal | PING rakhmerov, apetrich, bobh, mcdoker181818, akovi, hardikjasani | 08:07 |
| rakhmerov | I'm here | 08:07 |
| apetrich | o/ | 08:07 |
| d0ugal | Hows everyone doing? | 08:08 |
| d0ugal | Anything specific you'd like to discuss? | 08:08 |
| *** openstackgerrit has joined #openstack-mistral | 08:09 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Removes non needed parameter passed in magnum client creation https://review.openstack.org/592924 | 08:09 |
| rakhmerov | d0ugal: good ) | 08:11 |
| rakhmerov | d0ugal: trying to find out if I can actually go to Denver | 08:11 |
| rakhmerov | checking with Kendall | 08:12 |
| d0ugal | rakhmerov: cool, fingers crossed :) | 08:12 |
| rakhmerov | we'll see | 08:12 |
| rakhmerov | hopefully I'll get a reply today | 08:12 |
| d0ugal | We have four security bugs thanks to some testing at OVH. | 08:12 |
| rakhmerov | yeah | 08:12 |
| * apetrich is deep into mistral-tempest-plugin | 08:12 | |
| d0ugal | rakhmerov: Have you looked at them? | 08:13 |
| rakhmerov | nope | 08:13 |
| d0ugal | :) | 08:13 |
| rakhmerov | well, only one of them | 08:13 |
| rakhmerov | d0ugal: from my side, just a small update. I temporarily switched back to performance work and postponed scheduler as I planned. I want to fix one serious performance problem in the Mistral architecture | 08:14 |
| d0ugal | Which one? :) | 08:14 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1783708 | 08:14 |
| openstack | rakhmerov: Error: malone bug 1783708 not found | 08:14 |
| rakhmerov | ooh, which one you mean what? :) | 08:16 |
| rakhmerov | security bug or performance problem | 08:16 |
| rakhmerov | ? | 08:16 |
| d0ugal | rakhmerov: I actually meant which perf issue | 08:16 |
| d0ugal | but I was also interested in the bug :) | 08:16 |
| rakhmerov | ok, so | 08:17 |
| rakhmerov | performance problem is the following | 08:17 |
| rakhmerov | due to our non-blocking transactional model we have issues with things like completing workflows and triggering joins | 08:18 |
| rakhmerov | we have to use self-scheduling periodic jobs that check some conditions | 08:19 |
| rakhmerov | and the problem with them is that if we do them too often it's a load on DB | 08:19 |
| rakhmerov | if too rarely, then there are big delays between say when a workflow is logically completed and marked as completed | 08:20 |
| d0ugal | makes sense | 08:20 |
| rakhmerov | we kinda try to predict a delay when another iteration of a checking job should run but the algo is very very unreliable | 08:20 |
| d0ugal | I have noticed joins can be a little slow, but we only use them in a couple of places | 08:20 |
| rakhmerov | and probably it's hard to make it reliable, too many things that it depends on | 08:21 |
| rakhmerov | yeah | 08:21 |
| rakhmerov | so | 08:21 |
| rakhmerov | we hit a corner case | 08:21 |
| rakhmerov | we have workflows with 300+ tasks | 08:21 |
| rakhmerov | where every task is a subworkflow! | 08:21 |
| d0ugal | haha, nice | 08:21 |
| rakhmerov | now imagine how much time we lose on delays that occur for each subworkflow | 08:22 |
| rakhmerov | seconds for each | 08:22 |
| rakhmerov | many minutes for the entire thing.. | 08:22 |
| rakhmerov | just on watings | 08:22 |
| rakhmerov | waitings | 08:22 |
| d0ugal | Yeah, that is bad :) | 08:22 |
| rakhmerov | yeah | 08:22 |
| rakhmerov | and so I'm eager to fix that somehow | 08:22 |
| rakhmerov | so thinking now how because it's fundamentally not easy | 08:23 |
| d0ugal | It does sound complicated, but also should be interesting. | 08:24 |
| rakhmerov | yeah | 08:26 |
| rakhmerov | it is ) | 08:26 |
| rakhmerov | d0ugal: did you try to find out how many people are ready to go to Denver for Mistral sessions | 08:31 |
| rakhmerov | I'm just wondering if it makes sense for me to go at all ) | 08:31 |
| d0ugal | rakhmerov: I only checked with the core team, and nobody could go. | 08:31 |
| d0ugal | Other than maybe you :) | 08:31 |
| rakhmerov | I can go alone, of course, and have fun but alone.. ) | 08:31 |
| d0ugal | I know rbrady and thrash|g0ne will be there - but I think they will be focused on other sessions. | 08:31 |
| rakhmerov | and may be participate in other sessions | 08:31 |
| d0ugal | rakhmerov: indeed, I do wonder if it would be useful. | 08:32 |
| rakhmerov | yeah | 08:32 |
| rakhmerov | d0ugal: can you check again may be? | 08:32 |
| d0ugal | Right, it would be good for crossover sessions and maybe you can meetup with some mistral people (maybe make the mistral session very short?) | 08:32 |
| rakhmerov | via ML for example | 08:32 |
| d0ugal | rakhmerov: Sure. | 08:32 |
| rakhmerov | d0ugal: yeah, I still see the point | 08:32 |
| rakhmerov | ok, if they can cover flights/hotel I'll go | 08:33 |
| d0ugal | rakhmerov: did you email Kendall? | 08:33 |
| rakhmerov | yes | 08:33 |
| d0ugal | I hope they have not already started removing Mistrals room. It could get confusing with me sending this email now... | 08:34 |
| d0ugal | rakhmerov: email sent - so hopefully we get some replies :) | 08:36 |
| d0ugal | rakhmerov: Can you take a look at the security bugs? Do you think any of them are urgent and must be fixed in Rocky? | 08:38 |
| d0ugal | https://launchpad.net/mistral/+milestone/rocky-rc2 - you can see them all here | 08:38 |
| rakhmerov | d0ugal: ok | 08:39 |
| d0ugal | I guess they are only really important if Mistral is publicly accessible | 08:39 |
| d0ugal | or at least with users you don't trust :) | 08:39 |
| rakhmerov | yeah, looking.. | 08:41 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1666565 seems very easy to fix | 08:43 |
| openstack | rakhmerov: Error: malone bug 1666565 not found | 08:43 |
| rakhmerov | we could probably backport it to rocky | 08:44 |
| d0ugal | rakhmerov: Yeah, agreed. | 08:46 |
| d0ugal | I can probably do that now actually | 08:46 |
| rakhmerov | :) | 08:46 |
| d0ugal | I got distracted looking at the YAML bug | 08:46 |
| rakhmerov | hah | 08:47 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1699786 is interesting and suprising to me | 08:47 |
| openstack | rakhmerov: Error: malone bug 1699786 not found | 08:47 |
| rakhmerov | I wasn't aware of this asymmetry | 08:47 |
| rakhmerov | shouldn't also be a problem to fix | 08:48 |
| d0ugal | rakhmerov: indeed, that surprised me too :) | 08:48 |
| rakhmerov | yeah | 08:48 |
| rakhmerov | I'm not sure though what is our criteria based on which we decide if a bug needs to be backported | 08:49 |
| rakhmerov | if simplicity, then yes, this one is for rocky too | 08:49 |
| rakhmerov | importance? Well, not sure on that | 08:49 |
| rakhmerov | but I assume for some users it may be quite important | 08:50 |
| rakhmerov | who use auth and many tenants | 08:50 |
| d0ugal | I guess we should backport them if we think it is realistic (not too complicated) and safe (not backwards incompatible risks) | 08:50 |
| rakhmerov | right | 08:50 |
| rakhmerov | hm.. | 08:50 |
| rakhmerov | this is not compatible though.. | 08:50 |
| rakhmerov | but definitely a bug | 08:51 |
| d0ugal | yeah | 08:51 |
| d0ugal | We need to fix them all anyway, so I guess we can do that and then see if the fix can bbe backported. | 08:51 |
| d0ugal | Once we have a fix it will be much easier to judge :) | 08:51 |
| rakhmerov | d0ugal: agreed | 08:56 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590266 | 08:57 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Make Tripleo non-voting on Rocky https://review.openstack.org/592933 | 08:57 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Revert "Make Tripleo non-voting on Rocky" https://review.openstack.org/592934 | 08:57 |
| d0ugal | #endmeeting | 09:00 |
| *** openstack changes topic to "Mistral the Workflow Service for OpenStack. https://docs.openstack.org/mistral/latest/" | 09:00 | |
| openstack | Meeting ended Fri Aug 17 09:00:14 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-08-17-08.06.html | 09:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-08-17-08.06.txt | 09:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-08-17-08.06.log.html | 09:00 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590267 | 09:00 |
| d0ugal | rakhmerov, apetrich: I am disabling TripleO on stable/rocky because it is badly broken. https://review.openstack.org/#/c/592933/ | 09:01 |
| d0ugal | I think that is because we don't have a tripleo Rocky release yet. | 09:01 |
| d0ugal | but I want to get that series of patches in | 09:01 |
| d0ugal | (and need CI green for any incoming bug fixes) | 09:02 |
| rakhmerov | ok | 09:02 |
| d0ugal | rakhmerov: For X-Target-Insecure, would you just call bool() and explicitly convert it? | 09:10 |
| rakhmerov | yes, I guess | 09:10 |
| d0ugal | or maybe they expect me to check for == "False" | 09:11 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Removes non needed parameter passed in magnum client creation https://review.openstack.org/592924 | 09:30 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Explicitly convert X-Target-Insecure to a boolean https://review.openstack.org/592945 | 09:50 |
| *** zhenguo has quit IRC | 10:00 | |
| *** vgvoleg has quit IRC | 10:51 | |
| *** shardy has quit IRC | 11:24 | |
| *** shardy has joined #openstack-mistral | 11:24 | |
| openstackgerrit | Merged openstack/mistral stable/rocky: Make Tripleo non-voting on Rocky https://review.openstack.org/592933 | 12:20 |
| openstackgerrit | Merged openstack/mistral stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590266 | 12:24 |
| openstackgerrit | Merged openstack/mistral stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590267 | 12:24 |
| *** bobh has joined #openstack-mistral | 12:35 | |
| *** bobh has quit IRC | 12:36 | |
| *** bobh has joined #openstack-mistral | 12:36 | |
| *** bobh has quit IRC | 12:41 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/rocky: Removes non needed parameter passed in magnum client creation https://review.openstack.org/592924 | 12:46 |
| *** bobh has joined #openstack-mistral | 13:35 | |
| *** hardikjasani has quit IRC | 13:44 | |
| *** toure|gone is now known as toure | 14:14 | |
| *** shardy has quit IRC | 14:29 | |
| *** nguyenhai_ has quit IRC | 17:39 | |
| *** nguyenhai_ has joined #openstack-mistral | 17:40 | |
| *** bobh has quit IRC | 17:59 | |
| *** bobh has joined #openstack-mistral | 18:38 | |
| *** apetrich has quit IRC | 19:26 | |
| *** bobh has quit IRC | 19:30 | |
| *** bobh_ has joined #openstack-mistral | 19:30 | |
| *** bobh_ has quit IRC | 19:52 | |
| *** bobh has joined #openstack-mistral | 20:29 | |
| *** apetrich has joined #openstack-mistral | 20:33 | |
| *** apetrich has quit IRC | 21:10 | |
| *** bobh has quit IRC | 21:18 | |
| *** bobh has joined #openstack-mistral | 23:04 | |
| *** bobh has quit IRC | 23:34 | |
| *** toure is now known as toure|gone | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!