Wednesday, 2015-03-18

« Tuesday, 2015-03-17 Index Thursday, 2015-03-19 »
*** achanda has quit IRC00:15
*** achanda has joined #openstack-zaqar00:16
*** achanda has quit IRC00:18
*** achanda has joined #openstack-zaqar00:18
*** achanda has quit IRC00:30
*** chandankumar has joined #openstack-zaqar02:44
*** exploreshaifali has joined #openstack-zaqar02:55
*** exploreshaifali has quit IRC03:00
*** chandankumar has quit IRC03:05
*** cpallares has quit IRC03:21
vkmcflwang, so, that change is missing something, not sure what :(03:30
flwangvkmc: yep, I will take a look later, let's work it out03:32
flwangbtw, I would like to apply for the FFE for notification03:32
flwangbut I can't catch flaper87, pls forward my request to him if you meet him03:32
vkmcsure03:33
vkmcI'll let him know03:33
*** csoukup has joined #openstack-zaqar03:33
*** exploreshaifali has joined #openstack-zaqar03:39
*** chandankumar has joined #openstack-zaqar03:39
*** csoukup has quit IRC03:48
*** flwang1 has quit IRC03:50
*** csoukup has joined #openstack-zaqar04:04
*** csoukup has quit IRC04:09
*** boris-42 has joined #openstack-zaqar04:47
*** achanda has joined #openstack-zaqar05:06
*** exploreshaifali has quit IRC05:23
*** achanda has quit IRC06:25
*** achanda has joined #openstack-zaqar06:27
openstackgerritZhi Yan Liu proposed openstack/zaqar: Integrate OSprofiler with Zaqar  https://review.openstack.org/14135607:20
*** achanda has quit IRC07:35
*** achanda has joined #openstack-zaqar07:39
*** achanda has quit IRC08:10
*** exploreshaifali has joined #openstack-zaqar08:12
openstackgerritZhi Yan Liu proposed openstack/zaqar: Integrate OSprofiler with Zaqar  https://review.openstack.org/14135608:24
*** dynarro has joined #openstack-zaqar08:40
*** boris-42 has quit IRC09:12
*** dynarro has quit IRC09:40
*** amalagon has quit IRC10:02
*** amalagon has joined #openstack-zaqar10:03
*** exploreshaifali has quit IRC10:21
*** exploreshaifali has joined #openstack-zaqar10:45
*** boris-42 has joined #openstack-zaqar10:51
*** achanda has joined #openstack-zaqar11:45
*** achanda has quit IRC11:49
vkmcmorning11:51
*** bradjones has joined #openstack-zaqar12:17
*** bradjones has joined #openstack-zaqar12:17
flaper87vkmc: gooooooooooood morning12:28
flaper87vkmc: could you review dynarro's patch ?12:30
vkmcflaper87, buen dia señor Fla12:30
vkmcflaper87, yeah, I rebased exploreshaifali patch on it12:31
vkmcI'll approve12:31
vkmcnot sure what is missing from exploreshaifali patch12:32
vkmcbut its blocking flwang12:32
vkmcbtw, he asked for FFE12:32
exploreshaifaligooooooooood morning!12:33
exploreshaifaliI am working for patch only12:34
vkmchttps://review.openstack.org/#/c/165153/ <- flaper87, where is my torroncini?12:34
exploreshaifaliwill notify in few minutes :)12:34
vkmcexploreshaifali, how is it going?12:34
exploreshaifalivkmc, looking what else is needed, though there will be many test cases that will fail12:35
exploreshaifalias now I will remove queuecontroller and queuedatabase methods from DataDriver12:35
vkmccool12:35
vkmcmakes sense12:35
vkmcflaper87, https://review.openstack.org/#/q/status:open+project:openstack/zaqar+branch:master+topic:bp/persistent-transport,n,z12:43
exploreshaifaliflaper87, vkmc now we are getting obvious error - Stage <zaqar.storage.mongodb.messages.MessageQueueHandler instance at 0x7fb1fc4644d0> does not implement create12:59
exploreshaifalido we need to add a create method inside messageQueueHandler?12:59
exploreshaifaliWhat we have decided is only to keep those methods inside MessageQueueHandler that need message_controller()13:00
exploreshaifalione more thing13:01
*** boris-42 has quit IRC13:02
exploreshaifaliredis queues._create method https://github.com/openstack/zaqar/blob/master/zaqar/storage/redis/queues.py#L136 needs message_controller() so I am shifting this method also into redis's MessageQueueHandler13:02
vkmcexploreshaifali, those are warnings13:03
vkmcexploreshaifali, I think we would require changing the pipeline implementation, but I dunno what flaper87 has in mind13:03
vkmcexploreshaifali, leave them as is right now and focus on debugging the tests13:03
exploreshaifalivkmc, okay :D13:04
openstackgerritMerged openstack/zaqar: Make the Pipeline inmutable  https://review.openstack.org/16515313:05
* flaper87 back13:09
flaper87vkmc: I ate your torroncini13:09
vkmcflaper87, you wouldn't dare13:11
*** sriram has joined #openstack-zaqar13:11
*** dynarro has joined #openstack-zaqar13:12
vkmcdynarro, Merged openstack/zaqar: Make the Pipeline inmutable  https://review.openstack.org/16515313:13
vkmcdynarro, thx :)13:13
dynarrovkmc: np ;)13:13
vkmcflaper87, btw, I managed to check out the tests yesterday13:16
vkmcflaper87, websocket13:16
vkmcit doesn't seem we are using the transport at all, we are simply testing the api13:17
vkmcis that right?13:17
*** boris-42 has joined #openstack-zaqar13:40
*** amitgandhinz has joined #openstack-zaqar13:47
*** csoukup has joined #openstack-zaqar13:48
*** fifieldt has quit IRC13:51
openstackgerritMerged openstack/zaqar: This change adds missing help strings for claims  https://review.openstack.org/16316813:54
flaper87vkmc: that sounds correct13:56
flaper87for the unittests, we don't need to go through the wire13:57
*** cpallares has joined #openstack-zaqar13:57
vkmcflaper87, all right13:59
vkmcno, I lied13:59
flaper87lol14:00
flaper87LIAR!14:00
vkmcif we don't go through the wire we are not testing the transport at all14:00
vkmcwe are testing the api14:00
vkmconly14:00
vkmcin the case of WSGI, Falcon provides an structure to create the environment for the request being issued14:00
vkmcbut in this case, we don't have that14:00
*** cpallares has quit IRC14:03
flaper87vkmc: well, the wire is just one part of the whole workflow that we're skipping in the unittests but we won't skip in the functional tests14:03
flaper87The most importan parts are precisely testing the API, the response objects, the request objects, etc14:03
vkmcflaper87, ok, so... maybe it would be clearer if we put those unit tests under an api module14:04
flaper87vkmc: where are they right now ?14:04
flaper87I guess yeah, that'd make sense, but we need to have that documented somewhere14:05
vkmcflaper87, transport/websocket14:05
exploreshaifalivkmc, this is  some I am playing with, could not figure out what to do http://paste.openstack.org/show/193118/14:14
*** amalagon has quit IRC14:21
*** amalagon has joined #openstack-zaqar14:28
*** exploreshaifali has quit IRC14:31
*** mpanetta has joined #openstack-zaqar14:35
*** achanda has joined #openstack-zaqar14:47
*** achanda has quit IRC14:52
*** reed has joined #openstack-zaqar14:59
*** chandankumar has quit IRC14:59
*** chandankumar has joined #openstack-zaqar14:59
*** amalagon has quit IRC15:00
*** amalagon has joined #openstack-zaqar15:00
*** cpallares has joined #openstack-zaqar15:18
*** pcaruana has joined #openstack-zaqar15:19
*** amalagon has quit IRC15:38
openstackgerritVictor Sergeyev proposed openstack/zaqar: Use generate_uuid() from oslo.utils  https://review.openstack.org/16546615:41
*** chandankumar has quit IRC16:03
*** exploreshaifali has joined #openstack-zaqar16:06
*** amalagon has joined #openstack-zaqar16:10
*** dynarro has quit IRC16:34
*** amalagon has quit IRC16:34
*** akanksha has joined #openstack-zaqar16:44
akankshavkmc: ping?16:44
vkmcakanksha, pong16:44
akankshavkmc, hello. I am sorry I was out of action for a while. My university exams were going on. Just two more left now. I worked on two bugs but I am having issues submitting patches and running tests.16:46
vkmcakanksha, I saw you submitted one already, the UUID one16:47
akankshaI didnt submit it on gerrit yet. I just discussed it with shaifali on irc. I cant seem to submit a patch. I get an error message saying I havent updated contact information and when I try to do that I get an error from the openstack review server.  Also I need to work on my Outreachy Application for which I wanted to discuss the project under you.16:49
vkmcakanksha, isn't this yours? https://review.openstack.org/#/c/164671/16:51
akankshaVkmc, No,16:52
vkmcakanksha, you were working on that, right?16:52
akankshaAnd also I looked through this patch. It isnt what kgriffs wanted as far as I know. He discussed some of it with me.16:52
akankshaYeah I have.16:52
vkmcI remember seeing that discussion16:53
vkmcand considering that the patch was submitted one day after it, I thought it was yours16:53
akankshaI worked on it and tried to submit but ran into issues and then I had my exams this week so I couldnt work further.16:54
vkmcits ok16:54
vkmcok, submit it anyways16:54
vkmcwhat blockers did you find?16:55
akankshahttps://usercontent.irccloud-cdn.com/file/kUM1LjJR/Screen+Shot+2015-03-18+at+10.31.19+pm.png16:58
*** openstackgerrit has quit IRC16:59
vkmcakanksha, did you add your affiliation?16:59
*** openstackgerrit has joined #openstack-zaqar16:59
akankshaNo16:59
akankshavkmc, how do i do that?17:00
vkmcakanksha, you have to do so when you register here https://www.openstack.org/join/17:00
akankshavkmc, on it!17:01
*** achanda has joined #openstack-zaqar17:04
akankshavkmc, I got it done. Thanks a lot. I'll submit my patch asap.17:08
akankshavkmc, meanwhile can we discuss the project?17:08
vkmcakanksha, sure17:09
vkmcwhich one you were interested in?17:09
akankshaSo my primary interest is in the adding v1.1 feature support in zaqar-python client17:09
vkmcok17:10
akankshaWhile working on the redis bug I happened to look around the code a bit and I also looked up pythonclient code on github,17:10
vkmcgreat17:10
vkmcwell, you may have noticed that we have three versions of the server currently17:11
vkmcv1, already frozen17:11
vkmcv1.1, current version17:11
vkmcv2, next version, under development17:11
vkmcthere are some features missing from the client, and we would like to add support from them17:11
vkmcthose features are flavors (flaper87 add some support for it, but it requires some extension), capabilities, notifications and websockets17:12
vkmcthe timeline should be as follows17:13
akankshaOkay. So I need to add a timeline in my project proposal. We can break all the tasks down (perhaps feature wise) and can you tell me exactly how much time and effort do you estimate will be required for each of the task.17:13
vkmcsure17:14
*** bradjones has quit IRC17:14
vkmcweeks 1,2: time for you to get more involved with the code base and with the zaqar team17:14
vkmcweeks 3,4: extend support for flavors, add the tests for it17:14
vkmcweeks 5,6: add support for notifications, add the tests for it17:14
vkmcweeks 7,8: add support for capabilities, add the tests for it17:15
vkmcweeks 9, 10: add support for websockets, add the tests for it17:15
*** exploreshaifali has quit IRC17:15
vkmcweek 11: polishing and documentation17:15
vkmcweek 12: backup week in case some task takes more time than expected17:16
vkmcwe hope you can also help us with bug fixing, bug triaging and reviewing tasks during the internship17:16
vkmcits not mandatory but it generally is good if you have some other stuff to do if you find a blocker in your main task17:16
vkmchow does that sound?17:16
akankshaThis sounds perfect. Thanks so much!  :)I already worked on my proposal a bit lemme add this information. I'll send you the link to my propsal in a minute. Can you please review it.17:18
vkmcsure thing17:19
vkmclooking forward for it17:19
akankshavkmc, heres the rough draft of my proposal.17:53
akankshahttps://docs.google.com/document/d/1s4_FN5vhaQ_S8Nx8BH3PAZIAwVb_iCyuo_LMrT9nzRM/edit17:53
vkmcakanksha, thx17:55
vkmcakanksha, it looks really good17:58
*** exploreshaifali has joined #openstack-zaqar18:00
akankshavkmc, Thanks :D I am going to update my github repository soon with my college classwork and projects soon and put in a link to that under Projects section. Apart from that do you think I need to work on the timeline or read up more about the project?18:05
vkmchmm you seem to have a good understanding of the project :) and the timeline looks fine as well18:11
vkmcfocus on submit your contribution18:11
vkmcso you can upload the application18:11
akankshaYeah I will get the submission done by today hopefully.18:13
vkmccool18:13
exploreshaifalivkmc, some thoughts on http://paste.openstack.org/show/193118/ please18:21
vkmcexploreshaifali, sure :)18:23
exploreshaifali:D18:23
openstackgerritAkanksha Srivastava proposed openstack/zaqar: Correcting Instruction For Mongodb URI in README.rst  https://review.openstack.org/16554718:26
akankshaThis was a test commit on one of the easier bugs just to test out whether I could submit patches. Uploading the next one soon.18:28
vkmcakanksha, its really good, that could count as your contribution18:33
vkmcakanksha, what we look in the first contributions is that you gain practice with the process18:33
vkmcso a fix like that is good for us :)18:33
akankshavkmc, :)18:45
vkmcakanksha, also, good catch!18:45
*** achanda has quit IRC18:48
*** achanda has joined #openstack-zaqar19:01
*** exploreshaifali has quit IRC19:13
*** boris-42 has quit IRC19:37
*** boris-42 has joined #openstack-zaqar19:41
*** kgriffs|afk has quit IRC19:58
*** kgriffs|afk has joined #openstack-zaqar19:59
*** kgriffs|afk is now known as kgriffs19:59
*** flwang has quit IRC19:59
*** achanda has quit IRC20:00
*** flwang has joined #openstack-zaqar20:01
flwangflaper87: ping20:02
*** openstackgerrit has quit IRC20:14
*** openstackgerrit has joined #openstack-zaqar20:15
*** exploreshaifali has joined #openstack-zaqar20:16
*** achanda has joined #openstack-zaqar20:48
*** exploreshaifali has quit IRC20:56
openstackgerritMerged openstack/zaqar: Adds a status code field to the Response  https://review.openstack.org/16400721:22
*** flaper87 has quit IRC21:50
*** flaper87 has joined #openstack-zaqar21:53
*** csoukup has quit IRC21:59
*** boris-42 has quit IRC22:02
*** sriram has quit IRC22:10
openstackgerritAkanksha Srivastava proposed openstack/zaqar: Decorator fuction to Validate UUIDs in Redis Driver  https://review.openstack.org/16563222:25
akankshavkmc, ping22:26
akanksha?22:26
vkmcakanksha, hey22:26
akankshaI submitted a patch but I am not sure if I ran all the tests properly.22:26
*** amitgandhinz has quit IRC22:27
*** cpallares has quit IRC22:30
*** echevemaster has joined #openstack-zaqar22:54
*** boris-42 has joined #openstack-zaqar23:30
vkmcflwang, hi, around?23:45
vkmcor kragniz maybe?23:45
« Tuesday, 2015-03-17 Index Thursday, 2015-03-19 »

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