Tuesday, 2015-03-24

*** achanda has quit IRC00:03
*** JAHoagie has quit IRC00:11
*** kgriffs is now known as kgriffs|afk01:18
*** amalagon has joined #openstack-zaqar02:02
*** achanda has joined #openstack-zaqar02:37
*** achanda has quit IRC02:53
*** achanda has joined #openstack-zaqar02:56
*** chandankumar has joined #openstack-zaqar03:15
*** achanda has quit IRC03:24
*** chandankumar has quit IRC03:25
*** chandankumar has joined #openstack-zaqar03:25
*** achanda has joined #openstack-zaqar03:32
*** achanda has quit IRC03:41
*** achanda has joined #openstack-zaqar03:43
*** achanda has quit IRC03:53
*** achanda has joined #openstack-zaqar03:59
*** chandankumar has quit IRC04:52
*** chandankumar has joined #openstack-zaqar04:54
*** JAHoagie has joined #openstack-zaqar04:58
*** chandankumar has quit IRC05:04
*** reed has quit IRC05:26
*** boris-42 has quit IRC05:28
*** chandankumar has joined #openstack-zaqar05:38
*** JAHoagie has quit IRC06:03
*** achanda has quit IRC06:15
*** pcaruana has quit IRC06:41
*** fesp has quit IRC06:54
*** akanksha has joined #openstack-zaqar07:07
*** fesp has joined #openstack-zaqar07:24
*** fesp has quit IRC07:35
akankshavkmc, ping?08:58
*** flwang1 has joined #openstack-zaqar09:14
flwang1vkmc: ping09:14
flaper87flwang1: ping09:19
flwang1flaper87: yes sir09:19
flaper87flwang1: how are you doing ?09:19
flwang1flaper87: busy to die09:19
flaper87flwang1: that's good and bad :D09:20
flaper87flwang1: have you been able to work on the notification thing for redis ?09:20
flwang1my parents in law have back to China09:20
flwang1flaper87: i'm stilling working on that, will upload a new patch set today or tomorrow09:20
flwang1https://review.openstack.org/16489309:21
flwang1btw, TBH, i'm not familiar with redis, so it's little bit frustrating to work on that09:21
flaper87flwang1: do you want me to help?09:21
akankshaHello everyone. I want to work on some easy bugs that can help me get familiarised with zaqar in general. I am an applicant for Outreachy. Can someone suggest some tasks that I can manage?09:22
flwang1flaper87: it would be lovely, if you can start review it, though it's ugly now09:23
flwang1i have implemented the 'create' method, and will implement the 'list' and 'get' soon09:23
flwang1btw, should I send an official FFE in mail list?09:24
flwang1akanksha: welcome to zaqar?09:24
akankshaflwang: Thanks. :) I have been around for a while and I submitted fixes for two low hanging fruit bugs. I have been in contact with vkmc. She reviewed my proposal and all. She doesnt seem to be around for a while.09:26
flaper87flwang1: I'll do that right away so you'll have material to work on09:26
flaper87akanksha: welcome :D09:26
akankshaSo in the meanwhile I was hoping to work a bit more.09:26
flwang1flaper87: it would be awesome09:27
flwang1flaper87: as for the webhook driver, do you think we need to improve the performance for now?09:27
flwang1or you think it's "OK"?09:27
flwang1flaper87: i just wanna confirm if there is any other big change except the redis driver?09:28
flwang1I think we need to complete all the work before 19 Apr, right?09:28
flaper87flwang1: The more we can do the better. However, I think performance issues are electable for backports so, whenever we find one, we can simply backport it09:28
flaper87flwang1: yes, we should've cut K-3 last week but I was traveling09:29
flaper87I've to talk with ttx09:29
flwang1flaper87: so should I send out a FFE request?09:29
flaper87flwang1: let me cut K-3 first and I'll let you know09:30
flwang1flaper87: ok, cool09:30
flaper87akanksha: ping09:31
flaper87akanksha: you worked on this patch, right? https://review.openstack.org/#/c/165632/09:31
akankshaflaper87: Yeah09:32
flaper87akanksha: ok, first and foremost, thanks!09:39
flaper87now, I wanted to ask you if you could re-write that decorator in a way it does not have to introspect function arguments09:39
flaper87while that works, it adds some magic in the code that I'd honestly rather not have09:39
flaper87The reasons are that if we ever change the position or name of one of those arguments, this code will break09:39
flaper87It'd be probably better to just have an utility function that is call at the beginning of every functions09:40
flaper87Also, there are other places outside the redis driver that need this kind of `uuid` checks, would you mind extending your patch for those cases too?09:40
*** achanda has joined #openstack-zaqar09:51
akankshaflaper87, my decorator can be extended to any sort of function. What it essentially does is that it first makes a dictionary of all the function parameter names and  the values passed to it in a particular call. This way all we need to do is update the name of the parameter in the list of parameters that i made which is of type uuid and it will check it.09:56
akankshaTherefore the position or name does not matter according to me.09:56
*** achanda has quit IRC09:56
akankshaI'll try and find some other way to do it nonetheless.09:56
openstackgerritMerged openstack/zaqar: Fix common misspellings  https://review.openstack.org/16684809:57
flaper87akanksha: it matters because you need to keep that list of parameters updated which means you're mixing arguments introspection with hard-coded names09:58
akankshaflaper87: Hmmm. Yeah that is dirty way of doing it perhaps. I had a discussion with kgriffs prior to implementing the decorator and he'd asked me to model it around pyvalid accept decorator http://pydoc.net/Python/pyvalid/0.0.2/setup/09:59
akankshaflaper87: I'll try and come up with a solution to the hardcoding.09:59
akankshaperhaps something that will identify that the argument given in the call is meant to be a uuid.10:00
akankshaUmm. I think we can do that utility function thing.10:01
flaper87akanksha: what about just having a `validate_uuid` function that we call in the method's body for the arguments that need to be validated?10:01
akankshaYeah. That would be easy to do.10:01
akankshaLet me implement that.10:01
openstackgerritMerged openstack/zaqar: Update json schema for API v1_1  https://review.openstack.org/14841410:32
*** chandankumar_ has joined #openstack-zaqar10:44
*** chandankumar has quit IRC10:46
*** exploreshaifali has joined #openstack-zaqar11:20
*** exploreshaifali has quit IRC11:23
*** flwang1 has quit IRC11:49
*** boris-42 has joined #openstack-zaqar12:07
*** chandankumar_ has quit IRC12:18
*** exploreshaifali has joined #openstack-zaqar12:51
*** chandankumar has joined #openstack-zaqar12:57
vkmchi guys!13:29
*** chandankumar has quit IRC13:35
*** dynarro has joined #openstack-zaqar13:39
*** chandankumar has joined #openstack-zaqar13:41
*** sriram1 has joined #openstack-zaqar13:41
openstackgerritMerged openstack/zaqar: Updated from global requirements  https://review.openstack.org/16648213:48
*** amitgandhinz has joined #openstack-zaqar13:48
flaper87vkmc: HELLOOOOOOOOOOOOOOOOOOOOOOO!13:49
*** mpanetta has joined #openstack-zaqar13:49
vkmcflaper87, HIIIIIIIIII13:49
flaper87vkmc: shhhhh, there's people sleeping here13:49
flaper87vkmc: my oneplus one got home13:49
flaper87w000t13:49
* flaper87 happy13:49
flaper87fucking phone is bigger than the nexus but oh well13:50
vkmclol13:50
vkmcwow, I didn't know you were getting a phone13:50
vkmcthe one plus is the sexiest thing ever in the smartphone land13:50
*** achanda has joined #openstack-zaqar13:55
flaper87vkmc: I wasn't until someone sent me an invite. I just couldn't resist13:56
*** cpallares has joined #openstack-zaqar14:00
*** achanda has quit IRC14:00
*** chandankumar has quit IRC14:09
*** kgriffs|afk is now known as kgriffs14:30
*** achanda has joined #openstack-zaqar14:32
*** mpanetta_ has joined #openstack-zaqar14:34
*** mpanetta has quit IRC14:34
*** akanksha has quit IRC14:37
*** reed has joined #openstack-zaqar14:41
*** achanda has quit IRC14:42
*** achanda has joined #openstack-zaqar15:06
*** achanda has quit IRC15:06
*** mpanetta has joined #openstack-zaqar15:12
*** mpanetta_ has quit IRC15:12
*** cpallares has quit IRC15:23
*** cpallares has joined #openstack-zaqar15:24
*** exploreshaifali has quit IRC15:30
*** chandankumar has joined #openstack-zaqar15:36
*** bradjones has quit IRC15:51
*** bradjones has joined #openstack-zaqar15:51
*** bradjones has quit IRC15:51
*** bradjones has joined #openstack-zaqar15:51
*** wpf has quit IRC16:21
*** dynarro has quit IRC16:39
*** wpf has joined #openstack-zaqar16:42
*** exploreshaifali has joined #openstack-zaqar16:54
*** chandankumar has quit IRC16:58
*** chandankumar has joined #openstack-zaqar17:10
*** chandankumar has quit IRC17:27
*** achanda has joined #openstack-zaqar17:28
*** exploreshaifali has quit IRC17:53
*** exploreshaifali has joined #openstack-zaqar18:09
*** reed has quit IRC18:24
*** achanda has quit IRC18:28
*** reed has joined #openstack-zaqar18:34
*** achanda has joined #openstack-zaqar18:57
*** achanda has quit IRC19:00
*** achanda has joined #openstack-zaqar19:00
*** achanda has quit IRC19:42
*** exploreshaifali has quit IRC19:44
*** akanksha has joined #openstack-zaqar20:12
*** achanda has joined #openstack-zaqar20:35
*** reed has quit IRC20:42
*** openstackgerrit has quit IRC21:07
*** openstackgerrit has joined #openstack-zaqar21:07
*** amitgandhinz has quit IRC21:14
*** reed has joined #openstack-zaqar21:20
*** sriram1 has quit IRC21:34
openstackgerritMerged openstack/zaqar: Use generate_uuid() from oslo.utils  https://review.openstack.org/16546621:52
*** mpanetta has quit IRC21:59
*** achanda_ has joined #openstack-zaqar22:04
*** achanda has quit IRC22:08
*** AAzza_afk has joined #openstack-zaqar22:23
*** AAzzal has joined #openstack-zaqar22:24
*** AAzza has quit IRC22:25
*** AAzzal is now known as AAzza22:25
*** AAzza_afk has quit IRC22:28

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