| *** chlong has quit IRC | 00:01 | |
| openstackgerrit | Brad P. Crochet proposed openstack/mistral: Put useFixture in the proper place https://review.openstack.org/419680 | 00:41 |
|---|---|---|
| *** bobh has joined #openstack-mistral | 00:57 | |
| *** bobh has quit IRC | 01:03 | |
| *** gongysh has joined #openstack-mistral | 01:40 | |
| *** weshay has quit IRC | 01:47 | |
| *** weshay has joined #openstack-mistral | 01:49 | |
| *** thrash is now known as thrash|g0ne | 01:54 | |
| *** gongysh has quit IRC | 03:29 | |
| rakhmerov | kong: here | 04:17 |
| rakhmerov | kong: why do you think the other one will fail? | 04:26 |
| rakhmerov | in mysql yes, but only if 1st TX will rollback | 04:26 |
| rakhmerov | and why do you use 'repeatable read', Mistral was designed based on 'read committed' properties | 04:27 |
| rakhmerov | 'repeatable read' in mysql is not even a real 'repeatable read' | 04:27 |
| rakhmerov | kong: on mysql yes, there may be situations when it detects a deadlock between these threads, not because we do something wrong but just because of mysql itself, they clearly write in their docs that we should always design assuming that deadlocks can happen | 04:28 |
| rakhmerov | in order to handle this I recently added a decorator for method engine.on_action_complete() called "retry_on_deadlock" | 04:29 |
| rakhmerov | you can take a look | 04:29 |
| rakhmerov | I did a lot of tests for this, with multiple engines, under load, it seems to be working fine | 04:29 |
| rakhmerov | kong: I just rechecked your statement about an error in either 2nd or 3rd TX. You're right. It happens for both commit and rollback. That's mysql problem. On Postgres it works w/o issues. That's why I added this decorator | 04:40 |
| *** sharatss has quit IRC | 05:21 | |
| *** sharatss has joined #openstack-mistral | 05:21 | |
| *** gongysh has joined #openstack-mistral | 05:30 | |
| *** sharatss has quit IRC | 06:30 | |
| *** sharatss has joined #openstack-mistral | 06:31 | |
| *** ist has joined #openstack-mistral | 06:39 | |
| *** gongysh has quit IRC | 06:57 | |
| *** sharatss has quit IRC | 07:23 | |
| *** doriftoshoes has quit IRC | 07:28 | |
| *** lkannan has quit IRC | 07:30 | |
| *** lkannan has joined #openstack-mistral | 07:32 | |
| *** sharatss has joined #openstack-mistral | 07:33 | |
| *** doriftoshoes has joined #openstack-mistral | 07:37 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 07:39 | |
| *** shardy has joined #openstack-mistral | 08:12 | |
| *** sharatss has quit IRC | 08:16 | |
| *** jpich has joined #openstack-mistral | 08:53 | |
| *** sharatss has joined #openstack-mistral | 09:04 | |
| *** sharatss has quit IRC | 09:40 | |
| *** sharatss has joined #openstack-mistral | 09:40 | |
| *** jpich has quit IRC | 09:55 | |
| *** jpich has joined #openstack-mistral | 09:56 | |
| *** sharatss has quit IRC | 10:08 | |
| *** tuan_ has joined #openstack-mistral | 10:08 | |
| kong | rakhmerov: thanks for the answer, just want to make sure i totally understand the mechanism | 10:10 |
| openstackgerrit | Merged openstack/python-mistralclient: Add user and project domain name parameters for target cloud https://review.openstack.org/419332 | 10:18 |
| *** openstackgerrit has quit IRC | 10:18 | |
| *** sharatss has joined #openstack-mistral | 10:24 | |
| rakhmerov | kong: ok, yes | 10:40 |
| rakhmerov | ist: hi Istvan, can you please rebase https://review.openstack.org/#/c/418305/ ? | 10:44 |
| rakhmerov | we'll approve it quickly | 10:44 |
| *** openstackgerrit has joined #openstack-mistral | 10:55 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Test for inline comment styles - DO NOT MERGE https://review.openstack.org/419883 | 10:55 |
| ddeja | rakhmerov: did you saw this: https://www.openstack.org/assets/telecoms-and-nfv/OpenStack-Foundation-NFV-Report.pdf | 10:56 |
| ddeja | mistral is mentioned as 'Other Key OpenStack Projects' | 10:56 |
| rakhmerov | ddeja: nope, I didn't | 11:00 |
| ddeja | rakhmerov: I thought you may be interested ;) | 11:01 |
| rakhmerov | yes, I am :) | 11:01 |
| rakhmerov | thank you! | 11:02 |
| ddeja | no problem | 11:02 |
| ddeja | this is just an advantage of sitting in the open-space with other openstack developers ;) | 11:02 |
| rakhmerov | yeah, true | 11:04 |
| rakhmerov | btw, please share things like that | 11:04 |
| rakhmerov | I'm trying to be in the loop too but pretty often miss important things | 11:04 |
| ddeja | rakhmerov: sure | 11:10 |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Make kombu driver work in multi-thread manner https://review.openstack.org/419888 | 11:24 |
| ddeja | rakhmerov: if you have time look on this https://review.openstack.org/419888 | 11:26 |
| ddeja | suprisingly, it was fairly easy | 11:26 |
| tuan_ | ddeja: your patch looks like the implementation in others projects | 11:29 |
| tuan_ | e.g. Nova | 11:29 |
| tuan_ | but it looks good for me | 11:30 |
| ddeja | it's similar to what is inside oslo | 11:30 |
| ddeja | tuan_ ^ | 11:31 |
| *** openstackgerrit has quit IRC | 11:47 | |
| rakhmerov | ddeja: ok, I'll do | 12:01 |
| *** shardy is now known as shardy_lunch | 12:06 | |
| *** catintheroof has joined #openstack-mistral | 12:30 | |
| *** catintheroof has quit IRC | 12:31 | |
| *** catintheroof has joined #openstack-mistral | 12:32 | |
| *** openstackgerrit has joined #openstack-mistral | 12:46 | |
| openstackgerrit | Merged openstack/mistral: Fix for failing kombu dsvm gate https://review.openstack.org/414533 | 12:46 |
| openstackgerrit | Merged openstack/mistral: Updated from global requirements https://review.openstack.org/419344 | 12:47 |
| openstackgerrit | Merged openstack/mistral: Insecure flag added to openstack context https://review.openstack.org/417314 | 12:47 |
| *** dprince has joined #openstack-mistral | 12:49 | |
| *** thrash|g0ne is now known as thrash | 12:59 | |
| *** shardy_lunch is now known as shardy | 13:04 | |
| *** ist has quit IRC | 13:34 | |
| *** AlexeyAbashkin has quit IRC | 13:37 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 13:40 | |
| *** rbrady-afk is now known as rbrady | 14:02 | |
| *** Ephur has joined #openstack-mistral | 14:36 | |
| *** chlong has joined #openstack-mistral | 14:45 | |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Fix unit test that rely on order of return from DB https://review.openstack.org/420006 | 14:59 |
| *** jaosorior has joined #openstack-mistral | 15:05 | |
| *** jaosorior has quit IRC | 15:07 | |
| openstackgerrit | Merged openstack/python-mistralclient: Update .gitignore https://review.openstack.org/419375 | 15:13 |
| *** jaosorior has joined #openstack-mistral | 15:16 | |
| *** jaosorior has quit IRC | 15:19 | |
| *** rbrady has quit IRC | 15:20 | |
| *** jaosorior has joined #openstack-mistral | 15:22 | |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Fix unit test that rely on order of return from DB https://review.openstack.org/420006 | 15:28 |
| *** rbrady has joined #openstack-mistral | 15:44 | |
| tuan_ | ddeja: Hope you are still here | 15:54 |
| tuan_ | i have checked your patch | 15:54 |
| tuan_ | it is ok to me | 15:54 |
| tuan_ | just only this one: _process_message | 15:54 |
| tuan_ | it do not see how it is used in the patch | 15:54 |
| tuan_ | ? | 15:54 |
| ddeja | tuan_: oh, that's my mistake, I'll upload a fix in a minute | 15:55 |
| tuan_ | okay | 15:56 |
| tuan_ | :) | 15:56 |
| tuan_ | i would put +2 if i was core viewer | 15:56 |
| tuan_ | :) | 15:56 |
| tuan_ | of course with the new patch | 15:56 |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Fix unit test that rely on order of return from DB https://review.openstack.org/420006 | 15:59 |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Make kombu driver work in multi-thread manner https://review.openstack.org/419888 | 16:01 |
| ddeja | tuan_: here you are ^^ | 16:01 |
| ddeja | thanks for pointing this out | 16:01 |
| tuan_ | ok, now it is good | 16:09 |
| tuan_ | have a nice weekend | 16:09 |
| tuan_ | :) | 16:09 |
| ddeja | tuan_: you too, thanks | 16:10 |
| *** dprince has quit IRC | 16:19 | |
| *** tuan_ has quit IRC | 16:19 | |
| *** dprince has joined #openstack-mistral | 16:21 | |
| openstackgerrit | Merged openstack/python-mistralclient: Initial commit for mistral-i18n support https://review.openstack.org/417833 | 16:25 |
| *** bobh has joined #openstack-mistral | 16:33 | |
| *** chlong has quit IRC | 16:38 | |
| *** chlong has joined #openstack-mistral | 16:43 | |
| *** chlong has quit IRC | 16:48 | |
| *** AlexeyAbashkin has quit IRC | 16:49 | |
| *** chlong has joined #openstack-mistral | 16:59 | |
| *** jpich has quit IRC | 17:02 | |
| *** chlong has quit IRC | 17:09 | |
| *** dprince_ has joined #openstack-mistral | 17:20 | |
| *** dprince has quit IRC | 17:22 | |
| *** shardy has quit IRC | 17:46 | |
| *** sharatss has quit IRC | 17:55 | |
| *** sharatss has joined #openstack-mistral | 17:56 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-mistralclient: Updated from global requirements https://review.openstack.org/420131 | 18:51 |
| *** openstack has joined #openstack-mistral | 18:59 | |
| *** jamielennox is now known as jamielennox|away | 19:12 | |
| *** chlong has joined #openstack-mistral | 19:14 | |
| *** openstack has joined #openstack-mistral | 19:59 | |
| *** openstack has joined #openstack-mistral | 20:29 | |
| *** jaosorior has quit IRC | 20:45 | |
| *** jaosorior has joined #openstack-mistral | 20:45 | |
| *** rbrady is now known as rbrady-afk | 21:15 | |
| *** weshay is now known as weshay_afk | 21:44 | |
| *** jamielennox|away is now known as jamielennox | 21:59 | |
| *** dprince_ has quit IRC | 22:26 | |
| *** catintheroof has quit IRC | 22:38 | |
| *** bobh has quit IRC | 22:46 | |
| *** bobh has joined #openstack-mistral | 22:47 | |
| *** thrash is now known as thrash|g0ne | 22:58 | |
| *** bobh has quit IRC | 23:03 | |
| *** jaosorior has quit IRC | 23:31 | |
| *** chlong has quit IRC | 23:38 | |
| *** harlowja has quit IRC | 23:56 | |
| *** Ephur has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!