| *** openstackgerrit has quit IRC | 01:37 | |
| *** d0ugal has quit IRC | 01:37 | |
| *** mcdoker18 has quit IRC | 01:37 | |
| *** vgvoleg has quit IRC | 01:37 | |
| *** csatari has quit IRC | 01:37 | |
| *** ruhe has quit IRC | 01:37 | |
| *** aspiers has quit IRC | 01:37 | |
| *** lxkong has quit IRC | 01:37 | |
| *** szaher has quit IRC | 01:37 | |
| *** jistr has quit IRC | 01:37 | |
| *** thrash has quit IRC | 01:37 | |
| *** pgaxatte has quit IRC | 01:37 | |
| *** jlejeune has quit IRC | 01:37 | |
| *** therve has quit IRC | 01:37 | |
| *** jaosorior has quit IRC | 01:37 | |
| *** jrist has quit IRC | 01:37 | |
| *** mmethot has quit IRC | 01:37 | |
| *** zigo has quit IRC | 01:37 | |
| *** icatchnullptrs has quit IRC | 01:37 | |
| *** toure has quit IRC | 01:37 | |
| *** rakhmerov has quit IRC | 01:37 | |
| *** rakhmerov has joined #openstack-mistral | 01:43 | |
| *** d0ugal has joined #openstack-mistral | 01:43 | |
| *** toure has joined #openstack-mistral | 01:43 | |
| *** icatchnullptrs has joined #openstack-mistral | 01:43 | |
| *** zigo has joined #openstack-mistral | 01:43 | |
| *** mmethot has joined #openstack-mistral | 01:43 | |
| *** jrist has joined #openstack-mistral | 01:43 | |
| *** szaher has joined #openstack-mistral | 01:44 | |
| *** vgvoleg has joined #openstack-mistral | 01:44 | |
| *** jlejeune has joined #openstack-mistral | 01:44 | |
| *** openstackgerrit has joined #openstack-mistral | 01:44 | |
| *** jistr has joined #openstack-mistral | 01:45 | |
| *** thrash has joined #openstack-mistral | 01:45 | |
| *** csatari has joined #openstack-mistral | 01:45 | |
| *** therve has joined #openstack-mistral | 01:46 | |
| *** pgaxatte has joined #openstack-mistral | 01:46 | |
| *** jaosorior has joined #openstack-mistral | 01:46 | |
| *** aspiers has joined #openstack-mistral | 01:58 | |
| openstackgerrit | Merged openstack/mistral master: Don't quote {posargs} in tox.ini https://review.openstack.org/608827 | 03:43 |
|---|---|---|
| *** akovi has joined #openstack-mistral | 04:19 | |
| openstackgerrit | Merged openstack/mistral-lib master: Don't quote {posargs} in tox.ini https://review.openstack.org/609137 | 04:26 |
| openstackgerrit | Merged openstack/mistral-extra master: Don't quote {posargs} in tox.ini https://review.openstack.org/609136 | 04:26 |
| *** hardikjasani has joined #openstack-mistral | 04:34 | |
| *** gkadam has joined #openstack-mistral | 04:42 | |
| *** apetrich has joined #openstack-mistral | 05:09 | |
| *** jrist has quit IRC | 05:11 | |
| *** jtomasek has joined #openstack-mistral | 05:32 | |
| rakhmerov | vgvoleg: hi Oleg | 05:53 |
| rakhmerov | reviewed your patch | 05:53 |
| *** gkadam has quit IRC | 06:18 | |
| vgvoleg | ty!!! btw I forgot to handle cases in task_handler, when any task function throws an exception. Gonna update this today | 06:21 |
| vgvoleg | rakhmerov: hi :) | 06:22 |
| rakhmerov | ok | 06:40 |
| *** apetrich has quit IRC | 06:45 | |
| *** gkadam has joined #openstack-mistral | 07:55 | |
| *** apetrich has joined #openstack-mistral | 07:56 | |
| *** shardy has joined #openstack-mistral | 08:19 | |
| vgvoleg | rakhmerov: I don't want to have misunderstanding here https://review.openstack.org/#/c/603208/. Update is ready, i'll push if you sure it is necessarily | 08:49 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Make the rally job non-voting https://review.openstack.org/608906 | 08:50 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Change the rally job to use a lighter workflow https://review.openstack.org/608910 | 08:51 |
| rakhmerov | vgvoleg: you can easily do that optimization within _cut_wf_def_from_all() | 08:55 |
| rakhmerov | but the code will be more concise | 08:55 |
| rakhmerov | w/o repeating pieces | 08:56 |
| rakhmerov | please do | 08:56 |
| vgvoleg | rakhmerov: I can't get how to make this simple and readable :D | 09:14 |
| vgvoleg | The last way that I got is to pass either the len and the original definition | 09:15 |
| vgvoleg | If this is what you want to see - ok | 09:15 |
| vgvoleg | in this case we will still have one useless convertation in case of len==1 | 09:23 |
| openstackgerrit | Oleg Ovcharuk proposed openstack/mistral master: Divide yaml input to save it into definitions separately in case of create/update multiple workflows from one yaml https://review.openstack.org/603208 | 09:44 |
| *** apetrich has quit IRC | 10:11 | |
| *** apetrich has joined #openstack-mistral | 10:18 | |
| rakhmerov | d0ugal: did you see these issues with rpc_backend? | 10:30 |
| rakhmerov | know how to fix it? | 10:30 |
| d0ugal | rakhmerov: Which issues? | 10:31 |
| rakhmerov | http://logs.openstack.org/10/608910/2/check/openstack-tox-py36/9bb5009/testr_results.html.gz | 10:31 |
| rakhmerov | I guess the option was removed from o.m | 10:31 |
| d0ugal | oh, damn | 10:32 |
| rakhmerov | yeah | 10:32 |
| d0ugal | rakhmerov: Yup. https://docs.openstack.org/releasenotes/oslo.messaging/unreleased.html#upgrade-notes | 10:33 |
| *** apetrich has quit IRC | 10:33 | |
| rakhmerov | I tried to set transport_url but got other issues instead | 10:33 |
| d0ugal | It is a shame those upgrade notes don't tell us what we should use instead. | 10:33 |
| rakhmerov | yep | 10:35 |
| rakhmerov | I need to leave now | 10:35 |
| rakhmerov | can continue with that tomorrow if it's still not fixed ) | 10:35 |
| d0ugal | Sure | 10:35 |
| d0ugal | I'll keep looking | 10:35 |
| *** jrist has joined #openstack-mistral | 11:14 | |
| akovi | d0ugal: cfg.CONF.set_default("transport_url", 'fake:/') | 11:30 |
| akovi | seems to work | 11:31 |
| *** bobh has joined #openstack-mistral | 12:06 | |
| *** bobh has quit IRC | 12:11 | |
| *** akovi has quit IRC | 12:15 | |
| *** apetrich has joined #openstack-mistral | 13:46 | |
| *** bobh has joined #openstack-mistral | 14:11 | |
| vgvoleg | Hey guys, found this bug https://bugs.launchpad.net/mistral/+bug/1797136 during writing tests for this review https://review.openstack.org/#/c/608469/. Can I fix this bug in this review, or should create another? | 14:12 |
| openstack | Launchpad bug 1797136 in Mistral "Incorrect size calculation in length validator" [Undecided,New] | 14:12 |
| d0ugal | vgvoleg: when it is something small like that I think it is okay to fix it in the same review | 14:17 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Remove remaining references to the rpc_backend https://review.openstack.org/609426 | 14:38 |
| *** hardikjasani has quit IRC | 14:43 | |
| openstackgerrit | Hardik Jasani proposed openstack/mistral master: Fix next link in get resource list rest API https://review.openstack.org/608677 | 15:04 |
| *** jrist has quit IRC | 15:23 | |
| *** shardy has quit IRC | 16:44 | |
| *** apetrich has quit IRC | 18:03 | |
| *** apetrich has joined #openstack-mistral | 18:04 | |
| *** bobh has quit IRC | 21:05 | |
| *** bobh has joined #openstack-mistral | 21:07 | |
| *** bobh has quit IRC | 21:12 | |
| *** bobh has joined #openstack-mistral | 21:44 | |
| *** toure is now known as toure|gone | 22:00 | |
| *** apetrich has quit IRC | 22:12 | |
| *** bobh has quit IRC | 22:21 | |
| *** apetrich has joined #openstack-mistral | 22:31 | |
| *** jtomasek has quit IRC | 22:36 | |
| *** bobh has joined #openstack-mistral | 22:56 | |
| *** bobh has quit IRC | 23:01 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!