Tuesday, 2015-02-17

*** kgriffs is now known as kgriffs|afk01:03
*** zhenq has joined #openstack-zaqar02:02
*** sriram has joined #openstack-zaqar02:30
*** zhenq has quit IRC03:01
*** sriram has quit IRC03:20
*** reed has quit IRC03:56
*** achanda has joined #openstack-zaqar03:56
*** exploreshaifali has joined #openstack-zaqar05:18
*** exploreshaifali has quit IRC05:29
*** achanda has quit IRC05:43
*** achanda has joined #openstack-zaqar05:46
*** pcaruana has quit IRC05:47
*** davideagnello has quit IRC05:53
*** cpallares has quit IRC06:13
*** achanda has quit IRC07:47
*** achanda has joined #openstack-zaqar07:54
*** fifieldt has joined #openstack-zaqar08:09
*** achanda has quit IRC08:20
*** achanda has joined #openstack-zaqar08:24
*** achanda has quit IRC08:30
flaper87flwang: ping08:37
*** achanda has joined #openstack-zaqar08:37
*** achanda has quit IRC08:47
*** exploreshaifali has joined #openstack-zaqar08:54
*** fifieldt has quit IRC08:56
*** exploreshaifali has quit IRC11:03
*** exploreshaifali has joined #openstack-zaqar11:32
*** exploreshaifali has quit IRC11:50
vkmcheeeeeeello11:57
kragnizello11:58
*** exploreshaifali has joined #openstack-zaqar12:09
*** zhenq has joined #openstack-zaqar12:34
*** exploreshaifali has quit IRC13:43
flaper87LOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL https://twitter.com/kelseyhightower/status/56748821089735065614:04
* flaper87 rofl14:04
flaper87goooooood morning PEOPLE!14:05
kragnizflaper87: lol14:05
*** csoukup has joined #openstack-zaqar14:06
vkmclol14:07
*** sriram has joined #openstack-zaqar14:07
*** JAHoagie has joined #openstack-zaqar14:31
*** kgriffs|afk is now known as kgriffs14:52
*** exploreshaifali has joined #openstack-zaqar14:57
*** malini has joined #openstack-zaqar15:09
*** amitgandhinz has joined #openstack-zaqar15:14
*** JAHoagie has quit IRC15:16
*** achanda has joined #openstack-zaqar15:32
flaper87vkmc: is the websocket patch ready for review?15:35
vkmcflaper87, yes15:39
* flaper87 does that15:40
vkmcwait15:40
vkmccross api transport or websocket?15:40
flaper87>.>15:40
vkmcI'm rebasing according to the dependency chain15:41
vkmcthere is one to review15:41
flaper87I'll start from the bottom15:41
*** achanda has quit IRC15:41
exploreshaifaliflaper87, morning :)15:42
exploreshaifalivkmc, morning :)15:42
flaper87exploreshaifali: hey hey hey!15:42
flaper87how are you doing?15:42
flaper87I was reviewing your patch yday, btw15:42
flaper87looked for you but you weren't around15:42
flaper87I'll keep reviewing it today15:43
exploreshaifaliflaper87, Thanks :D15:43
flaper87What issues have you been hitting?15:43
exploreshaifaliflaper87, I have added MessageQueueHandler class inside mongodb/messages.py15:43
exploreshaifaliflaper87, now we need to add this to entry point15:43
exploreshaifaliflaper87, I am unable to figure out how to do that15:44
exploreshaifaliyou gave hint - to hack inside setup.cfg15:44
exploreshaifaliso I have modified it a bit...... but what after that ?15:44
exploreshaifaliflaper87, and also i am not sure the changes in setup.cfg are correct or not15:45
exploreshaifaliflaper87, main files to review are these two - setup.cfg and mongodb/messages.py15:45
flaper87vkmc: am I crazy or you simply didn't use the `raise_conn_error` decorator anywhere https://review.openstack.org/#/c/141280/23/zaqar/api/v1_1/endpoints.py,cm ?15:47
flaper87I'm pretty sure I'm just crazy15:47
vkmclol15:47
flaper87exploreshaifali: ok, thanks for all the info, I'll review now15:47
exploreshaifaliflaper87, Thanks :)15:47
flaper87vkmc: you know what that means, right? TEST IT!15:48
flaper87:P15:48
vkmcflaper87, sure... sorry15:48
flaper87vkmc: LOL, just kidding girl :D15:48
* flaper87 has done worse things15:49
flaper87like I once submitted the same error in 2 different patch-sets15:49
flaper87T_T15:49
flaper87I swore I had fixed it by just moving the code from one place to another15:49
vkmchahaha I should add a test suite for it... I was waiting for it to merge and then add the tests for the websocket driver15:49
vkmcflaper87, btw, you approve the generic generic exception?16:01
vkmcI almost put the oops I did it again chorus... but I'm afraid we would get too popular and we would have to work harder16:01
flaper87yeah, I think that's fine16:03
*** achanda has joined #openstack-zaqar16:05
*** achanda has quit IRC16:06
*** reed has joined #openstack-zaqar16:11
*** JAHoagie has joined #openstack-zaqar16:15
*** amalagon has quit IRC16:20
*** exploreshaifali has quit IRC16:32
*** amalagon has joined #openstack-zaqar17:30
*** miqui_ has quit IRC17:34
*** amalagon has quit IRC17:35
*** exploreshaifali has joined #openstack-zaqar17:36
*** miqui has quit IRC17:37
*** davideagnello has joined #openstack-zaqar18:09
*** achanda has joined #openstack-zaqar18:18
*** exploreshaifali has quit IRC18:34
*** jasondotstar has joined #openstack-zaqar18:45
*** malini has quit IRC19:05
*** openstackgerrit has quit IRC19:20
*** openstackgerrit has joined #openstack-zaqar19:20
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: API handler and API v1.1 Queue endpoints  https://review.openstack.org/14128019:26
*** exploreshaifali has joined #openstack-zaqar19:39
*** exploreshaifali has quit IRC20:23
flwangflaper87: ping20:29
*** achanda has quit IRC20:36
flwangvkmc: around?20:38
vkmcflwang, here :)20:38
vkmcflwang, how are you?20:38
*** csoukup has quit IRC20:42
flwangvkmc: good good20:45
flwangvkmc: brb, sorry20:45
*** csoukup has joined #openstack-zaqar20:49
*** malini has joined #openstack-zaqar20:54
vkmcnp20:55
flaper87flwang: pong ?21:09
vkmcflaper87, ping21:09
flaper87vkmc: pong ?21:10
flaper87:D21:10
vkmc:)21:11
*** achanda has joined #openstack-zaqar21:11
*** achanda has quit IRC21:11
*** achanda has joined #openstack-zaqar21:12
*** malini has left #openstack-zaqar21:14
flwangflaper87: still around?21:26
flaper87flwang: yes21:26
flwangflaper87: as for the notification service status, the code is almost done, but I don't know why the redis gate is not happy for both my patches21:27
flaper87flwang: ok, I'll dig into that and check21:27
flaper87We need to get these patches merged21:27
flaper87all blueprints, asap. Like yday21:27
flaper87:)21:27
flwangflaper87: agree21:28
flwangI was very busy in last week21:28
flaper87flwang: I figured, np at all.21:28
flwangI will focus on it this week21:28
flwang2015-02-11 02:07:47.278 | [ERROR] /opt/stack/new/devstack/lib/zaqar:204 Zaqar did not start21:28
flaper87I'm in a deep dive mode on my pending things for Zaqar21:29
flaper87flwang: I'll give that a go on a VM21:29
flwangfrom the devstack log, I saw the zaqar server failed to start, but I haven't figure out why21:29
flaper87lets see21:29
vkmcits the pooling thing with redis again IIRC21:29
flwangvkmc: what's that mean?21:29
flaper87mmh, u sure?21:30
flaper87it sounds weird21:30
flaper87could you elaborate on why you think it's the pool thing?21:30
vkmclet me go through that again21:30
vkmcI reviewed it a while ago21:31
flwangvkmc: did you mean there are something I need to enhance in my patches for pooling?21:31
vkmcflwang, no, I was thinking in a different patch21:36
vkmcthere are not traces of pooling in the webhook one21:36
flwangvkmc: yep, that's why it's weird21:41
flwangit's hard to debug and most of the other gates work21:42
vkmcyeah.. and it works for other patches21:42
vkmclet's recheck21:44
*** flwang1 has joined #openstack-zaqar21:56
*** achanda has quit IRC22:10
*** achanda has joined #openstack-zaqar22:29
*** amitgandhinz has quit IRC22:31
flaper87flwang: vkmc http://paste.openstack.org/show/176334/22:41
flaper87that's the exception22:41
vkmcflaper87, you got that locally?22:43
*** amitgandhinz has joined #openstack-zaqar22:44
flaper87vkmc: yes22:45
*** zhenq has quit IRC22:49
flaper87it's the fucking ubuntu trusty redis version22:53
flaper87apparently it doesn't have sentinel anymore ?/22:53
*** sriram has quit IRC22:55
*** JAHoagie has quit IRC22:56
flaper87so no, ubuntu python-redis package doesn't have sentinel. Question is: Why didn't this fail before and why patches like "storage capabilities" passed22:59
*** zhenq has joined #openstack-zaqar23:04
*** csoukup has quit IRC23:11
flwang1flaper87: yep23:16
flwang1i have the same question, why the other patches can pass23:16
flwang1flaper87: and are your sure that's the root cause why zaqar can't start?23:17
flaper87flwang1: I just chekced ubuntu's python-redis package23:18
flaper87it ships python-redis 2.7.223:18
flaper87according to git's tag, sentinel is definitely missing23:18
flaper87really no idea why it's failing for your patch, though.23:18
flwang1i see23:20
flwang1given all the other gates can  pass, the only reason i can imagine is it must be related to redis23:21
flaper87yeah, but all the other patches had this gate enabled23:24
flaper87this is so weird23:24
flwang1flaper87: yep23:26
*** zhenq has quit IRC23:36
*** amalagon has joined #openstack-zaqar23:53
*** zhenq has joined #openstack-zaqar23:53
*** amitgandhinz has quit IRC23:54
*** flwang1 has quit IRC23:55

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