Friday, 2018-10-19

*** bobh has joined #openstack-mistral02:45
*** bobh has quit IRC02:46
*** bobh has joined #openstack-mistral02:46
*** bobh has quit IRC02:51
rakhmerovd0ugal: I thought they had said that they would think :)03:44
rakhmerovd0ugal: I was playing with all that stuff all day yesterday03:45
rakhmerovseems like "threading" works too when we set executor_thread_pool_size to 103:46
*** akovi has joined #openstack-mistral04:15
*** openstackgerrit has joined #openstack-mistral04:17
openstackgerritRenat Akhmerov proposed openstack/mistral stable/rocky: Make Rally job non-voting till its failures are fixed  https://review.openstack.org/61177104:17
*** jaosorior has joined #openstack-mistral04:24
openstackgerritRenat Akhmerov proposed openstack/mistral master: WIP: improving join  https://review.openstack.org/61046105:03
openstackgerritRenat Akhmerov proposed openstack/mistral master: WIP: improving join  https://review.openstack.org/61046105:14
*** apetrich has joined #openstack-mistral05:47
*** jtomasek has quit IRC05:58
*** jtomasek has joined #openstack-mistral06:05
*** jaosorior has quit IRC06:32
*** jaosorior has joined #openstack-mistral06:34
*** shardy has joined #openstack-mistral07:10
rakhmerovd0ugal: hi, ok to approve https://review.openstack.org/#/c/611771/ ?07:14
vgvoleghello everyone07:32
vgvolegI'm again with cache issue07:33
vgvoleghttps://bugs.launchpad.net/mistral/+bug/179743007:33
openstackLaunchpad bug 1797430 in Mistral "Join task fails after rerun outbound task" [High,Confirmed] - Assigned to Oleg Ovcharuk (vgvoleg)07:33
vgvolegWhat do you think guys about NOT to cache failed tasks?07:34
vgvolegWhat is bad in this approach?07:34
rakhmerovvgvoleg: hi07:39
rakhmerovvgvoleg: I'd recommend not to touch it for now07:39
rakhmerovI'm working on something related and will take that into account07:39
rakhmerovand I also fixed something related with this cache07:40
d0ugalrakhmerov: Yup, seems fine to me07:59
d0ugalrakhmerov: funny that rally passed in the patch to make it non-voting :-D08:00
d0ugal#startmeeting mistral08:03
openstackMeeting started Fri Oct 19 08:03:30 2018 UTC and is due to finish in 60 minutes.  The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot.08:03
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.08:03
*** openstack changes topic to " (Meeting topic: mistral)"08:03
openstackThe meeting name has been set to 'mistral'08:03
d0ugalMorning Everyone. It is the Friday office hour!08:03
* d0ugal is super tired.08:03
d0ugalhttps://etherpad.openstack.org/p/mistral-office-hours08:03
d0ugalPING ^ rakhmerov, apetrich, bobh, mcdoker181818, akovi, hardikjasani08:03
d0ugalWe have 0 untriaged bugs which is nice :)08:04
d0ugalSo, anyone want to chat about anything?08:04
d0ugalCurious to hear what people are up to and what they are planning on working on08:04
d0ugaland if there is anything I can help with!08:04
rakhmerovhere08:05
rakhmerovd0ugal: want to finalize "improving join" and "workflow completion" patches within 1-2 days08:06
apetricho/08:06
rakhmerovd0ugal: it seems like "threading" works the same way as blocking if we set executor_thread_pool_size to 108:08
rakhmerovnot sure on 100% yet though08:08
rakhmerovwe're testing now08:08
d0ugalInteresting08:09
rakhmerovyeah08:09
d0ugalIs having a thread pool size of one just the same as blocking really?08:09
d0ugalfunctionally the same :)08:09
rakhmerovso removal of "blocking" doesn't scare me too much anymore08:09
d0ugalgreat08:09
rakhmerovd0ugal: seems like the same, yes08:09
rakhmerovissues are only with "eventlet"08:10
rakhmeroveven if the thread pool size is 108:10
rakhmerovbtw, I'm again observing deadlocks between green threads that we've seen before08:10
rakhmerovso unfortunately, we can't use "eventlet" for now08:11
d0ugaldamn, where are you seeing that?08:11
rakhmerovand we set it by default so we probably need to change it08:11
d0ugalIn your internal testing?08:11
rakhmerovd0ugal: yeah08:11
d0ugalJust looking at the email thread, it seems like Ben's suggestion could be quite good08:11
rakhmerovI'm also running something locally on my laptop, I have a number of "edge case" examples that help me find issues08:11
rakhmerovd0ugal: yep, agree08:12
d0ugalSo it seems like we have a few options to explore, which is better than before :)08:12
d0ugalrakhmerov: Let me know when I can help reviewing the join and workflow completion patches08:13
rakhmerovyep, I think on Monday08:13
rakhmerovas far as join, there's a big problem with it on some use cases. That's why I'm trying to improve it08:13
rakhmerovbtw, Rally CI job fails also because of the "eventlet" executor08:14
rakhmerovit's part of the same problem08:14
d0ugalaha08:16
rakhmerovyep )08:18
csatariDear cores, can someone please check this https://review.openstack.org/#/c/476499/ ?08:26
rakhmerovcsatari: aah.. ok08:29
d0ugalcsatari: I'll look now!08:29
rakhmerovd0ugal: it's been there for a while really08:29
d0ugalYup08:29
rakhmerovwe need to review it )08:29
d0ugalI am just going to look at the latest change after Vitalii's review08:30
d0ugaloh, wow, I have draft comments from a long time ago I forgot to submit :(08:31
d0ugalbut I think they might have all been resolved.08:31
* d0ugal keeps reading08:31
csatariThanks d0ugal and rakhmerov08:34
d0ugalrakhmerov: I just need to finish reading the "install-ubuntu.rst" page, but I think I'd be happy to merge. What do you think?08:35
d0ugalI should probably help with the install-rdo section :)08:35
d0ugalcsatari: Spotted one minor issue, but otherwise I'd be happy to +208:38
d0ugalI commented on the review.08:38
csatariThanks.08:38
csatariWe are looking into it.08:38
openstackgerritAndrea Visnyei proposed openstack/mistral master: Mistral install guide  https://review.openstack.org/47649908:47
csatarid0ugal: here is the correction ^^^^08:47
d0ugalcsatari: Thanks!08:54
d0ugalrakhmerov: Can you take a look sometime? I added a +208:54
csatarid0ugal: Yes, install rdo part looks for contributors :)09:02
*** zigo has joined #openstack-mistral09:14
d0ugal#endmeeting09:21
*** openstack changes topic to "Mistral the Workflow Service for OpenStack. https://docs.openstack.org/mistral/latest/"09:21
openstackMeeting ended Fri Oct 19 09:21:18 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)09:21
openstackMinutes:        http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-10-19-08.03.html09:21
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-10-19-08.03.txt09:21
openstackLog:            http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-10-19-08.03.log.html09:21
vgvolegrakhmerov: do you have any idea about date when it would be implemented?09:39
rakhmerovd0ugal: had to step out.. Yes, I will, sure09:48
d0ugalrakhmerov: thanks09:50
rakhmerovvgvoleg: soon )09:51
vgvolegrakhmerov: ahah, ok :)09:53
*** s10 has joined #openstack-mistral09:59
openstackgerritMerged openstack/mistral stable/rocky: Make Rally job non-voting till its failures are fixed  https://review.openstack.org/61177110:35
openstackgerritRenat Akhmerov proposed openstack/mistral stable/rocky: Update OnClauseSPec task name criteria  https://review.openstack.org/61155010:45
openstackgerritRenat Akhmerov proposed openstack/mistral stable/rocky: Fix next link in get resource list rest API  https://review.openstack.org/61092610:46
akoviWow, I've just found out that oslo.messaging sends the RPC requests with 4 times escaped JSON content. In a random heartbeat message I found 1650 backslashes in 7332 characters. 22% is just backslashes! What happens when we send heat stacks...10:51
openstackgerritRenat Akhmerov proposed openstack/mistral master: WIP: improving join  https://review.openstack.org/61046111:00
*** lingxian has joined #openstack-mistral11:30
*** lxkong has quit IRC11:39
*** lingxian is now known as lxkong11:40
*** apetrich has quit IRC12:14
*** gkadam has joined #openstack-mistral12:52
*** akovi has quit IRC13:17
*** s10 has quit IRC14:15
*** s10 has joined #openstack-mistral14:21
*** jistr is now known as jistr|call14:29
*** apetrich has joined #openstack-mistral14:36
*** bobh has joined #openstack-mistral14:38
*** apetrich has quit IRC14:43
*** apetrich has joined #openstack-mistral14:44
*** jistr|call is now known as jistr14:47
*** s10 has quit IRC15:01
*** gkadam has quit IRC15:18
*** bobh has quit IRC16:03
*** openstackgerrit has quit IRC16:24
*** shardy has quit IRC16:30
*** openstackgerrit has joined #openstack-mistral16:35
openstackgerritMerged openstack/mistral stable/rocky: Update OnClauseSPec task name criteria  https://review.openstack.org/61155016:35
*** bobh has joined #openstack-mistral16:35
*** bobh has quit IRC17:34
*** apetrich has quit IRC17:51
*** apetrich has joined #openstack-mistral18:04
*** bobh has joined #openstack-mistral18:26
*** bobh has quit IRC19:18
*** bobh has joined #openstack-mistral19:19
*** bobh has quit IRC19:23
*** bobh has joined #openstack-mistral19:23
*** lxkong has quit IRC19:39
*** bobh has quit IRC21:01
*** bobh has joined #openstack-mistral21:01
*** bobh has quit IRC21:05
*** lingxian has joined #openstack-mistral22:14
*** bobh has joined #openstack-mistral23:37
*** bobh has quit IRC23:50

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!