| openstackgerrit | Tin Lam proposed openstack/solar: Use six.moves.urllib.parse to replace urlparse https://review.openstack.org/281588 | 00:09 |
|---|---|---|
| *** tzn has joined #openstack-solar | 00:20 | |
| *** tzn has quit IRC | 00:26 | |
| *** tzn has joined #openstack-solar | 02:24 | |
| *** angdraug has quit IRC | 02:26 | |
| *** tzn has quit IRC | 02:29 | |
| *** tzn has joined #openstack-solar | 03:25 | |
| *** tzn has quit IRC | 03:29 | |
| *** tzn has joined #openstack-solar | 04:26 | |
| *** tzn has quit IRC | 04:30 | |
| *** tzn has joined #openstack-solar | 05:26 | |
| *** tzn has quit IRC | 05:31 | |
| *** tzn has joined #openstack-solar | 06:27 | |
| *** tzn has quit IRC | 06:32 | |
| *** tzn has joined #openstack-solar | 07:28 | |
| *** tzn has quit IRC | 07:32 | |
| *** salmon_ has joined #openstack-solar | 08:18 | |
| *** tzn has joined #openstack-solar | 08:29 | |
| *** tzn has quit IRC | 08:33 | |
| *** tzn has joined #openstack-solar | 09:04 | |
| *** tzn has quit IRC | 09:10 | |
| *** ogelbukh has quit IRC | 09:27 | |
| *** ogelbukh has joined #openstack-solar | 09:41 | |
| pigmej | salmon_: https://review.openstack.org/#/c/281588/ I think we should -1 or even -2 this, because: | 09:46 |
| pigmej | 1) we don't support py3 at all now, chaning this single import changes nothing | 09:46 |
| pigmej | 2) it adds six to solar requirements (which is also not in this patch, but it should be btw) | 09:46 |
| salmon_ | yup | 09:55 |
| *** openstackgerrit has quit IRC | 10:02 | |
| *** openstackgerrit has joined #openstack-solar | 10:03 | |
| *** tzn has joined #openstack-solar | 10:31 | |
| pigmej | salmon_: I really don't know wtf with this bug | 10:32 |
| pigmej | Something is wrong in that environment with orchestration / workers stuff when they are embedded inside testing stuff, it seems that it's somehow related to postgresql backend | 10:33 |
| salmon_ | does it happen with sql? | 10:33 |
| salmon_ | sqlite | 10:33 |
| pigmej | It crashed once, but I'm unable to reproduce | 10:34 |
| pigmej | the tests that are failing are spawning multiple workers, + multiple clients for orchestration, then they wait for graph to complete | 10:34 |
| pigmej | and the graph is always empty there | 10:34 |
| pigmej | so something is definitely wrong but I have no clear idea now, I tested normal workflow and it works | 10:35 |
| salmon_ | :/ | 10:35 |
| pigmej | so it's clearly related to test suite, it may depend on a way that something is used | 10:35 |
| pigmej | I'm stating to think to add it as a bug, skip the tests with pg backend and wait for dima | 10:36 |
| pigmej | because tbh I have no clue what's going on in orchestration module | 10:36 |
| pigmej | all I know is that tasks are saved with PENDING state, no other state is saved, but still test doesn't even see this. So it's clearly related somehow. | 10:37 |
| salmon_ | maybe tests are connecting to a different db? :P | 10:37 |
| pigmej | It may be some issue with PG as a backend | 10:37 |
| pigmej | but it's also possible that sqlite + riak works and pg no, just because of different greenlet scheduling order | 10:38 |
| pigmej | (adding sleeps doesn't help) | 10:38 |
| pigmej | I also don't want to change flow in this test because I'm not sure if problem is in test or in different place | 10:42 |
| salmon_ | are you sure that correct db is used in tests? :) | 10:42 |
| pigmej | I am | 10:42 |
| pigmej | I identified tests that are failing in I would say similar way, but I'm currently off the ideas what's the reason for it, those tests are using the same logic like, create something, spawn gevent rpc workes in other greenlet, execute wait function, wait for finish, check if graph passed to workers is correct | 10:45 |
| pigmej | and there are just 2 tests that behave in this way, and remember that I also discovered new path yesterday taht contained syntax error | 10:46 |
| pigmej | I would skipif these 2 tests, and wait for dima: | 10:47 |
| pigmej | - I don't want to change these tests, but I'm not sure if they are 100% right | 10:47 |
| pigmej | - Other backends are working, but it doesn't mean that backend is not working properly, because all tests that don't use that flow are perfectly fine | 10:47 |
| pigmej | also it's just a PG which is affected, and we're not yet switched... so | 10:51 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Fix TypeError in scheduler https://review.openstack.org/281759 | 10:51 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: Fix dirs sync rsync type for Vagrantfile https://review.openstack.org/281760 | 10:52 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add packer build for docker container https://review.openstack.org/281767 | 11:10 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add packer build for docker container https://review.openstack.org/281767 | 11:12 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add packer build for docker container https://review.openstack.org/281767 | 11:14 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Use psycogreen for PG backend https://review.openstack.org/281769 | 11:16 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Rework conftests + sql tables are cleared on success https://review.openstack.org/281784 | 11:47 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: temporary skip 2 problematic tests on postgresql https://review.openstack.org/281791 | 11:59 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Use psycogreen for PG backend https://review.openstack.org/281769 | 12:02 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Rework conftests + sql tables are cleared on success https://review.openstack.org/281784 | 12:18 |
| openstackgerrit | Merged openstack/solar: Fix TypeError in scheduler https://review.openstack.org/281759 | 13:08 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Rework conftests + sql tables are cleared on success https://review.openstack.org/281784 | 13:21 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Rework conftests + sql tables are cleared on success https://review.openstack.org/281784 | 13:42 |
| pigmej | mkwiek: fixed that comment :) | 13:43 |
| *** tzn has quit IRC | 15:01 | |
| *** tzn has joined #openstack-solar | 15:03 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add packer build for docker container https://review.openstack.org/281767 | 15:04 |
| *** tzn has quit IRC | 15:08 | |
| openstackgerrit | Merged openstack/solar: Fix dirs sync rsync type for Vagrantfile https://review.openstack.org/281760 | 15:13 |
| *** tzn has joined #openstack-solar | 15:31 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add packer build for docker container https://review.openstack.org/281767 | 15:39 |
| *** angdraug has joined #openstack-solar | 15:55 | |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Some speedups after adding PG https://review.openstack.org/281932 | 16:19 |
| *** tzn has quit IRC | 17:05 | |
| *** tzn has joined #openstack-solar | 17:11 | |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Some speedups after adding PG https://review.openstack.org/281932 | 17:14 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add packer build for docker container https://review.openstack.org/281767 | 17:24 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Some speedups after adding PG https://review.openstack.org/281932 | 17:36 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Add more stuff to exclude when syncing tree in tests https://review.openstack.org/281979 | 17:41 |
| openstackgerrit | Merged openstack/solar: Use psycogreen for PG backend https://review.openstack.org/281769 | 18:04 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: temporary skip 2 problematic tests on postgresql https://review.openstack.org/281791 | 18:17 |
| pigmej | salmon_: ^ updated for pep8 :) | 18:17 |
| salmon_ | ;) | 18:29 |
| *** tzn has quit IRC | 19:03 | |
| *** tzn has joined #openstack-solar | 19:14 | |
| *** tzn has quit IRC | 19:18 | |
| *** angdraug has quit IRC | 19:20 | |
| *** pigmej has quit IRC | 20:12 | |
| *** pigmej has joined #openstack-solar | 20:14 | |
| *** tzn has joined #openstack-solar | 21:08 | |
| *** tzn has quit IRC | 21:12 | |
| openstackgerrit | Merged openstack/solar: Some speedups after adding PG https://review.openstack.org/281932 | 21:42 |
| salmon_ | jenkins is so fast :) | 21:43 |
| pigmej | yeah.... | 22:47 |
| *** tzn has joined #openstack-solar | 23:24 | |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Add more stuff to exclude when syncing tree in tests https://review.openstack.org/281979 | 23:24 |
| pigmej | I clicked rebase because jenkins died itself :D | 23:25 |
| *** tzn has quit IRC | 23:39 | |
| *** openstackgerrit has quit IRC | 23:47 | |
| *** openstackgerrit_ is now known as openstackgerrit | 23:47 | |
| *** openstackgerrit_ has joined #openstack-solar | 23:47 | |
| *** openstackgerrit_ is now known as openstackgerrit | 23:48 | |
| *** openstackgerrit_ has joined #openstack-solar | 23:48 | |
| *** salmon_ has quit IRC | 23:54 | |
| *** openstackgerrit_ has quit IRC | 23:55 | |
| *** openstackgerrit_ has joined #openstack-solar | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!