Monday, 2015-01-19

« Sunday, 2015-01-18 Index Tuesday, 2015-01-20 »
*** yograterol has joined #openstack-zaqar00:17
*** yograterol has quit IRC00:29
*** yograterol has joined #openstack-zaqar00:30
*** achanda has quit IRC01:17
*** achanda has joined #openstack-zaqar01:23
*** achanda has quit IRC01:34
*** achanda has joined #openstack-zaqar01:37
*** achanda has quit IRC02:23
*** achanda has joined #openstack-zaqar02:23
*** achanda has quit IRC02:24
*** achanda has joined #openstack-zaqar02:56
*** diga has joined #openstack-zaqar03:40
*** achanda has quit IRC03:46
*** achanda has joined #openstack-zaqar04:02
*** achanda has quit IRC05:31
*** pcaruana has quit IRC05:32
*** achanda has joined #openstack-zaqar05:37
*** achanda has quit IRC05:41
*** achanda has joined #openstack-zaqar06:08
*** achanda has quit IRC06:18
*** yograterol has quit IRC07:12
*** achanda has joined #openstack-zaqar07:12
*** exploreshaifali has joined #openstack-zaqar07:38
exploreshaifaliflaper87: morning!!  :)07:39
*** pcaruana has joined #openstack-zaqar07:42
*** achanda has quit IRC08:15
exploreshaifaliflaper87: around?08:15
flaper87exploreshaifali: morning :)08:17
flaper87exploreshaifali: good post :)08:17
exploreshaifaliflaper87: :D thanks!!08:17
exploreshaifaliflaper87: I need serious help :P08:18
exploreshaifaliflaper87: its 3-4 weeks over and I am unable to complete queue_controller task08:18
exploreshaifaliI am unable to think if we need to remove queue_controller() method totally, what should be done08:18
flaper87exploreshaifali: what do you mean by removing it totally?08:19
exploreshaifalifor places where queue_controller is used08:19
flaper87we need to remove it from the *data* controller, not the management controller08:19
exploreshaifaliflaper87: removing it from DataDriverBase and shifting it to COntrolDriver08:19
flaper87and we don't need to remove it from all the places where it's being used but provide another way to access i08:19
flaper87it*08:19
exploreshaifaliflaper87: yes, I think that is what we have done up till now....changing the defination of queue_controller08:20
flaper87exploreshaifali: did you push your latest patch?08:22
exploreshaifaliflaper87: yes it is there08:22
flaper87exploreshaifali: https://review.openstack.org/#/c/144340/4/zaqar/storage/base.py,cm here u still have it in the base data driver08:28
flaper87exploreshaifali: https://review.openstack.org/#/c/144340/4/zaqar/storage/mongodb/driver.py,cm we now probably need to pass an instance of control driver to the data driver08:29
flaper87exploreshaifali: ^ also, don't move the message controller08:30
exploreshaifaliflaper87: regarding base.py, yes queue_controller() is there in database, but it is not directly getting QueueController object, now it is getting through controlDrive08:31
exploreshaifaliflaper87: this is what needed.....right?08:31
exploreshaifalis/database/DataDriveBase08:32
flaper87exploreshaifali: yes but instead of creating an instance of ControlDriver every time, you should probably pass the control drive instance to the DataDriver when it's created08:32
*** dynarro has joined #openstack-zaqar08:34
exploreshaifaliflaper87: so you are saying that every new created DataDrive should have an instance of ControlDrive?08:34
exploreshaifali*new created DataDrive instance08:34
flaper87yes08:34
flaper87otherwise you'd be creating a new ControlDriver instance every time you call `queue_controller`08:34
flaper87and you don't want that :P08:35
exploreshaifaliflaper87: yes, but if so, I need to add ControlDrive instance inside __init__() of DataDrive?08:35
flaper87exploreshaifali: yes08:37
flaper87:)08:37
exploreshaifaliflaper87: okay, let me try that :)08:37
*** flwang1 has joined #openstack-zaqar08:48
*** amalagon has quit IRC09:04
*** amalagon has joined #openstack-zaqar09:05
*** exploreshaifali has quit IRC09:05
*** amalagon has quit IRC09:09
*** exploreshaifali has joined #openstack-zaqar09:43
*** dynarro has quit IRC09:43
exploreshaifaliflaper87: even after puting ControlDrive instance inside __init__() of DataDrive, getting same set of errors, with and withoud message_controller in controlDrive10:04
flaper87exploreshaifali: push the latest patch, lets review10:04
exploreshaifaliflaper87: I am in my college, there due to firewalls cannot push it10:05
exploreshaifaliflaper87: okay, let me move to my home then I will ping you again :)10:05
flaper87exploreshaifali: hehe, ok ok :D10:06
flaper87whatever works best for you :)10:06
* flaper87 will re-review the existing patch in the meantime10:06
exploreshaifaliya.... I will be back in half an hour :)10:06
exploreshaifalithanks!10:06
*** exploreshaifali has quit IRC10:11
openstackgerritMerged openstack/zaqar: Implement mongodb driver for notifications  https://review.openstack.org/13979210:33
*** flwang1 has quit IRC11:16
*** dynarro has joined #openstack-zaqar11:17
*** vkmc_ has joined #openstack-zaqar11:26
vkmc_morning...!11:27
*** dynarro has quit IRC11:29
*** exploreshaifali has joined #openstack-zaqar11:30
kragnizvkmc_: morning!11:31
kragnizoh god: https://www.youtube.com/watch?v=zwH7p80bOdw11:33
vkmc_omg kragniz11:35
openstackgerritShaifali Agrawal proposed openstack/zaqar: Removing QueueController from data to control plane  https://review.openstack.org/14434012:10
exploreshaifaliflaper87, latest patch ^12:12
exploreshaifalivkmc_, kragniz morning!! :)12:14
vkmc_morning exploreshaifali!12:14
exploreshaifali:)12:16
exploreshaifalivkmc_, remember my current patch giving mannnnnny assertion error12:21
exploreshaifaliand out of them first one is claim_test12:21
*** dynarro has joined #openstack-zaqar12:21
vkmc_I do12:21
exploreshaifalivkmc_, so we decided to try to claim messages12:21
exploreshaifaliI tried and it worked well12:21
vkmc_aaaand?12:22
vkmc_huh12:22
vkmc_but the tests fails12:22
exploreshaifaliyes12:23
exploreshaifalivkmc_, that could be because, there were not all resources available12:23
exploreshaifalivkmc_, test failed at line https://github.com/openstack/zaqar/blob/master/tests/functional/wsgi/v1/test_claims.py#L5212:24
flaper87o/12:24
exploreshaifaliwhich means it does not have all the resources needed12:25
exploreshaifaliflaper87, \o12:25
exploreshaifaliflaper87, can we start reviewing the current patch for removing queue_controller()12:26
flaper87exploreshaifali: I'm on it12:26
exploreshaifali:P12:27
exploreshaifaliokay12:27
flaper87exploreshaifali: wrote some comments, I'll submit them as soon as I can check jenkins' results12:30
exploreshaifaliflaper87, till than, I just realized few flaws, can I update the patch?12:32
flaper87exploreshaifali: lemme submit my comments then12:37
flaper87so you can address them right away12:37
flaper87exploreshaifali: done12:37
exploreshaifaliflaper87, ok :) thanks!!12:37
openstackgerritFlavio Percoco proposed openstack/zaqar: Forbid adding stores with mismatching capabilities  https://review.openstack.org/14578312:53
*** vkmc_ has quit IRC13:14
*** jchai has joined #openstack-zaqar13:35
*** exploreshaifali has quit IRC13:39
*** dynarro has quit IRC14:01
*** dynarro has joined #openstack-zaqar14:01
*** sriram has joined #openstack-zaqar14:12
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: Refactor Request and Response classes to enhance consistency  https://review.openstack.org/14826514:19
flaper87vkmc:14:24
flaper87ops14:24
vkmcflaper87:14:24
flaper87vkmc:14:25
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: Refactor Request and Response classes to enhance consistency  https://review.openstack.org/14826514:32
vkmcflaper87, https://review.openstack.org/#/c/141280/15 let me know when you have a moment to chat :)14:33
* flaper87 ignores vkmc14:35
flaper87:P14:35
flaper87vkmc: jokes apart, gimme at least 1h to finish this patch14:36
flaper87and I'll ping you14:36
vkmchow daaaaaaaare you?14:38
vkmcsure haha, no rush14:38
vkmcI'm working on something else14:38
vkmcs/working on/breaking14:38
*** csoukup has joined #openstack-zaqar14:40
flaper87vkmc: yeah, sure... now you're going to play the: "I'm a busy chick" card now.....14:41
flaper87>.>14:41
vkmcflaper87, I am >.>14:42
* vkmc chases butterflies14:43
* flaper87 sets some worms on fire14:43
flaper87(because that's what kids with nothing to do do)14:43
* kragniz calls the rspca14:43
* flaper87 has no idea what rspca is14:44
flaper87probably for the better14:44
flaper87ohhhh14:44
flaper87is that actually a thing?14:45
flaper87Do organizations like that exist?14:45
kragnizmy bad, it's a UK thing14:45
flaper87mmhhh... good thing I'm not cruel14:45
flaper87ah nice, I take my last message back14:45
kragnizRoyal Society for the Prevention of Cruelty to Animals14:45
* flaper87 keeps setting worms on fire14:45
flaper87so, why is setting worms on fire something cruel and chasing butterflies to keep them trapped forever in some random kid's hands isn't?14:46
* vkmc now kills ants with a magnifying glass14:46
flaper87ah? ah? ah?14:46
kragnizflaper87: you have the ENPA, though14:46
flaper87vkmc: boooooooooooooooooooooooooooooooooooooooooooooooring14:46
kragnizEnte Nazionale Protezione Animali14:46
flaper87kragniz: yeah, but this is italy14:46
* flaper87 keeps setting worms on fire14:46
flaper87vkmc: use the lighter (or a match)14:47
kragnizpoor little wormies :(14:47
kragnizwhat did they do to flaper8714:47
vkmcflaper87, you are the pyromaniac one haha14:47
flaper87kragniz: if they ain't gummy, they ain't good14:48
*** amitgandhinz has joined #openstack-zaqar14:48
flaper87vkmc: my mom used to say the same thing14:48
kragniznow I see it all14:48
*** ametts has joined #openstack-zaqar14:48
flaper87until I left14:48
flaper87now my gf says the same thing14:48
flaper87but I can asure you, they are wrong14:48
vkmcflaper87, they are the ones that know you the best... I would say I agree with them :P14:49
flaper87but but but.... mmmhhh not fair14:49
* flaper87 plays with his anti-stress lighter14:50
*** kgriffs|afk is now known as kgriffs14:50
kragnizflaper87: prove them wrong by setting them on fire14:50
kragnizit can't go wrong14:50
flaper87kragniz: LOOOOOOL, I'm sure they'll agree with me afterwards14:51
flaper87mmhhh14:51
vkmcaaaahahahaha14:51
vkmc'no, I'm not a pyromaniac' while holding a flamethrower14:52
* flaper87 did want one of those for a long time14:53
kragnizwho doesn't?14:53
*** kgriffs is now known as kgriffs|afk14:53
kragnizwait, you don't want one now?14:53
flaper87I act like I don't14:54
flaper87but sssh14:54
flaper87really, I asked for one14:54
flaper87no kiddings14:54
kragnizheh14:54
flaper87I ended up getting a slingshot and using it to throw fireworks to my neighbors' place14:55
* flaper87 had this one http://assets.academy.com/mgen/73/10218673.jpg?is=500,50014:55
kragnizI had one similar to that!14:55
flaper87kragniz: one of the coolest things ever, isn't it?14:55
kragnizflaper87: I had this one: http://www.pastimesgunsandtackle.com/media/catalog/product/cache/1/image/1140x900/9df78eab33525d08d6e5fb8d27136e95/p/r/prod_2211.jpg14:56
flaper87how do you call those small fireworks that just make noise but you still have to light them?14:56
* flaper87 googles14:56
kragnizflaper87: firecrackers?14:56
flaper87kragniz: ah no wait, I had exactly that one. you could fold yours, right?14:57
kragnizyeah14:57
kragnizblackwidow14:57
flaper87then yeah, it's exactly that one14:57
kragnizit was cool14:57
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: API handler and API v1.1 Queue endpoints  https://review.openstack.org/14128014:58
*** kgriffs|afk is now known as kgriffs14:58
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: API handler and API v1.1 Queue endpoints  https://review.openstack.org/14128015:02
*** dynarro has quit IRC15:04
*** dynarro has joined #openstack-zaqar15:25
flaper87vkmc: there ?15:49
openstackgerritFlavio Percoco proposed openstack/zaqar: Split MongoDB's default implementation from FIFO's  https://review.openstack.org/14829715:49
flaper87315:49
flaper87215:49
flaper87115:49
flaper87oh well15:49
vkmcflaper87, not so fast15:50
*** exploreshaifali has joined #openstack-zaqar15:56
vkmcflaper87, ?15:56
vkmcok, brb16:14
*** diga_ has joined #openstack-zaqar16:28
*** diga_ has quit IRC16:32
*** dynarro has quit IRC16:33
*** amalagon has joined #openstack-zaqar16:38
*** diga_ has joined #openstack-zaqar16:39
*** cpallares has joined #openstack-zaqar16:40
cpallareso/16:45
cpallareshello zaqarians16:45
cpallaresvkmc: o/16:46
cpallaresexploreshaifali: o/16:46
cpallaresflaper87: o/16:46
cpallareskgriffs: o/16:46
exploreshaifalicpallares, \o :D 'sup?16:46
cpallareshowdy exploreshaifali. How are you doing?16:46
exploreshaifaliall good :)16:47
kragnizcpallares: \o16:47
cpallareshi kragniz16:47
cpallareshow are you kragniz?16:47
kragnizcpallares: slowly recovering16:48
cpallareskragniz: From?16:48
kragnizcpallares: this https://www.youtube.com/watch?v=zwH7p80bOdw16:48
cpallareslol!16:49
kgriffscpallares: o/16:50
cpallareshey kgriffs :)16:51
vkmccpallares, heeeeeeeeey16:52
vkmcwelcome back16:52
*** sriram has quit IRC16:55
cpallaresthanks vkmc :)16:56
*** sriram has joined #openstack-zaqar16:56
*** sriram has quit IRC16:56
*** kgriffs is now known as kgriffs|afk17:15
*** kgriffs|afk is now known as kgriffs17:19
*** sriram has joined #openstack-zaqar17:32
*** diga_ has quit IRC17:34
*** amalagon has quit IRC17:36
exploreshaifalivkmc, around?17:44
vkmcexploreshaifali, here17:48
exploreshaifalivkmc, I am searching where does DataDriver is instantiated https://github.com/openstack/zaqar/blob/master/zaqar/storage/mongodb/driver.py#L6617:48
exploreshaifalis/DataDriver/DataDrive17:48
exploreshaifalican you help please?17:49
vkmcexploreshaifali,17:49
vkmcsure17:49
exploreshaifalivkmc, I have seen in bootstrap.py but there is only storage.pooling.DataDrive17:50
exploreshaifaliand storage.pipeling.DataDrive are used17:50
vkmcexploreshaifali, https://github.com/openstack/zaqar/blob/master/zaqar/bootstrap.py#L95-L10817:51
vkmcthat's fine17:51
vkmcit uses the pipeline17:51
exploreshaifalivkmc, but pipeline and pooling have their own DataDrive https://github.com/openstack/zaqar/blob/master/zaqar/storage/pipeline.py#L8417:52
exploreshaifaliI want to know from where storage/mongodb.driver.DataDrive is used17:53
vkmcso... bootstrap is where everything is instantiated17:56
vkmcthe datadriver and controldriver are initializated when storage and control methods are called17:57
vkmcthose methods call https://github.com/openstack/zaqar/blob/master/zaqar/storage/utils.py#L6817:58
vkmcbasically we delegate the loading part to stevedore17:58
vkmchttps://pypi.python.org/pypi/stevedore17:59
exploreshaifalik17:59
vkmcstevedore then loads the required driver17:59
vkmcif you put 'mongodb' in the config17:59
vkmcthen https://github.com/openstack/zaqar/blob/master/zaqar/storage/mongodb/driver.py#L66 is called18:00
vkmcdoes that makes sense?18:00
exploreshaifalivkmc, yes yes..... just need to look at above links you mentioned. Thanks!!18:01
vkmccool18:02
vkmc:)18:02
*** pcaruana has quit IRC18:08
*** exploreshaifali has quit IRC18:15
*** openstackgerrit has quit IRC18:37
*** openstackgerrit has joined #openstack-zaqar18:37
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: Adds websockets driver to Zaqar server  https://review.openstack.org/14015518:38
*** achanda has joined #openstack-zaqar18:46
*** ametts has quit IRC18:54
*** dynarro has joined #openstack-zaqar18:55
*** ametts has joined #openstack-zaqar18:59
*** achanda has quit IRC19:05
*** achanda has joined #openstack-zaqar19:10
flaper87vkmc: what's up girl?19:22
flaper87sorry for dissapearing earlier today19:23
flaper87vkmc: so, I wanted to get an update from you on the websocket thingy19:23
vkmcflaper87, heeeey yo :)19:23
vkmcall right, I'm implementing the tests for the queues endpoint19:23
vkmcand implementing other endpoints19:23
flaper87awesome, awesome, awesome. Any problems so far?19:25
flaper87I mean, any problems that need discussion?19:25
flaper87actually, you know what? keep your problems for yourself19:25
* flaper87 ducks19:25
flaper87:P19:25
flaper87or throw them at kgriffs19:26
* flaper87 ducks, again19:26
kgriffslol.19:26
vkmcits his fault19:26
flaper87it's always kgriffs fault19:26
* kgriffs is pretty sure everything is his fault19:26
vkmcfor now its going ok19:27
vkmconly one problem19:27
vkmc:D because it wouldn't be so fun otherwise19:27
vkmcthe schemas we defined for validation are tightly coupled to wsgi19:27
flaper87mmh, could you elaborate?19:28
vkmcsure19:29
vkmcwhen the websocket transport receives the request and process them, we decided to validate it using https://github.com/openstack/zaqar/blob/master/zaqar/common/api/api.py#L5219:30
vkmcand this is the schema we have defined for v1_1 https://github.com/openstack/zaqar/blob/master/zaqar/api/v1_1/request.py19:30
flaper87ah you mean it's tightly coupled because of the `method` thing?19:31
flaper87feel free to change it, that was based on a previous analysis, which I think we agreed was not good enough19:32
vkmcyeah... in fact, not only the method param19:33
*** exploreshaifali has joined #openstack-zaqar19:33
vkmcwe would required to add the action param19:33
vkmcrequire*19:33
vkmcand some other changes19:33
vkmcdo we use that schema somewhere else? I was about to check that19:34
flaper87nope, I know we use the response one for tests19:37
flaper87but not the request one19:37
flaper87it was actually added for this specific thing19:37
flaper87so, feel free to adapt it19:37
vkmcawesome19:37
vkmc:D19:37
*** dynarro has quit IRC19:42
*** vkmc has left #openstack-zaqar19:51
*** vkmc has joined #openstack-zaqar19:51
*** jchai is now known as jchai_afk19:59
*** jchai_afk is now known as jchai20:03
*** exploreshaifali has quit IRC20:10
*** flwang1 has joined #openstack-zaqar20:18
*** amalagon has joined #openstack-zaqar20:20
vkmcflaper87, meeting in 30?20:22
*** amalagon has quit IRC20:29
* vkmc puts a flamethrower and waits20:30
vkmco.o20:30
*** achanda has quit IRC20:31
flwangvkmc: hi there20:40
flwangi have a meeting in 5 min20:40
flwangso maybe late for our weekly meeting20:40
vkmcflwang, no worries :)20:40
*** achanda has joined #openstack-zaqar21:02
*** achanda_ has joined #openstack-zaqar21:03
flwangvkmc: meeting?21:04
vkmcflwang, no flaper87 :(21:05
vkmcflwang, is there something you want to discuss?21:05
*** achanda has quit IRC21:07
*** amalagon has joined #openstack-zaqar21:08
flwangvkmc: yep21:08
flwangthe notification sending implement21:08
flwangand a potential bug21:08
flwangwhen enabled pool, GET /queues will run into 500 error if there is no queue created21:09
vkmchmm...21:09
vkmcthat is... pools enabled, pools created and no queue?21:10
flwangyep21:10
vkmcI think its a bug21:12
vkmcit should list queues... no matter its in v1 or v1.121:12
flwangvkmc: right21:13
flwangok, I will file a bug and submit a patch for that21:13
vkmcthat sounds good!21:13
vkmcyou already discovered what is causing that?21:13
flwangvkmc: yep21:13
vkmcflwang, cool!21:13
flwangjust wanna confirm if it's a bug or known issue21:14
*** ametts has quit IRC21:19
*** ametts has joined #openstack-zaqar21:41
*** jchai has quit IRC22:18
*** sriram has quit IRC22:21
*** echevemaster has joined #openstack-zaqar22:40
*** achanda_ has quit IRC22:42
*** amitgandhinz has quit IRC22:54
*** achanda has joined #openstack-zaqar22:54
*** echevemaster has quit IRC22:54
*** achanda has quit IRC22:59
*** achanda has joined #openstack-zaqar23:09
*** ametts has quit IRC23:25
*** csoukup has quit IRC23:26
*** achanda has quit IRC23:35
*** achanda has joined #openstack-zaqar23:40
« Sunday, 2015-01-18 Index Tuesday, 2015-01-20 »

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