Wednesday, 2014-10-08

« Tuesday, 2014-10-07 Index Thursday, 2014-10-09 »
*** bradjones has quit IRC00:22
*** bradjones has joined #openstack-zaqar00:24
*** X019 has joined #openstack-zaqar00:46
*** jeffrey4l has joined #openstack-zaqar01:54
*** jeffrey4l has quit IRC01:59
*** jeffrey4l has joined #openstack-zaqar02:11
*** X019 has quit IRC02:25
*** reed has quit IRC02:46
*** reed has joined #openstack-zaqar02:47
*** prashanthr_ has joined #openstack-zaqar03:31
*** jeffrey4l has quit IRC03:59
*** jeffrey4l has joined #openstack-zaqar04:19
*** sgotliv has quit IRC05:03
*** sgotliv has joined #openstack-zaqar05:10
*** echevemaster has quit IRC06:01
openstackgerritOpenStack Proposal Bot proposed a change to openstack/zaqar: Imported Translations from Transifex  https://review.openstack.org/12680606:05
*** reed has quit IRC06:07
*** jeffrey4l has quit IRC06:54
*** jeffrey4l has joined #openstack-zaqar07:07
openstackgerritJeffrey Zhang proposed a change to openstack/zaqar: Fix no links property in pool list response  https://review.openstack.org/12346208:08
*** prashanthr_ has quit IRC08:18
*** bradjones has quit IRC08:22
*** bradjones has joined #openstack-zaqar08:23
openstackgerritJeffrey Zhang proposed a change to openstack/zaqar: Fix no links property in flavor list response  https://review.openstack.org/12362608:50
exploreshaifalihello, I have a question, in file https://github.com/openstack/zaqar/blob/master/zaqar/queues/storage/mongodb/driver.py09:13
exploreshaifalisomeone around?09:14
*** jeffrey4l has quit IRC09:24
*** prashanthr_ has joined #openstack-zaqar09:24
*** jeffrey4l has joined #openstack-zaqar09:36
openstackgerritJeffrey Zhang proposed a change to openstack/zaqar: Fix the param name typo  https://review.openstack.org/12685009:43
*** jeffrey4l_ has joined #openstack-zaqar09:43
*** jeffrey4l has quit IRC09:47
*** jeffrey4l_ has quit IRC10:01
*** njirap has joined #openstack-zaqar10:12
*** jeffrey4l_ has joined #openstack-zaqar10:18
*** jeffrey4l_ has quit IRC10:54
*** jeffrey4l_ has joined #openstack-zaqar10:54
*** flaper87|afk is now known as flaper8710:55
flaper87exploreshaifali: hey :)11:02
flaper87shoot11:02
exploreshaifalifaper87, hi :)11:03
exploreshaifaliflaper87, in file https://github.com/openstack/zaqar/blob/master/zaqar/queues/storage/mongodb/driver.py11:03
exploreshaifaliat line #195 in def database, it returns **self.connection[name]** but there is no connection attribute inside Class or its parent class.11:03
exploreshaifaliinside ControlDriver class11:05
exploreshaifalior its parent class11:05
*** openstack has joined #openstack-zaqar14:13
*** openstackstatus has joined #openstack-zaqar14:15
*** ChanServ sets mode: +v openstackstatus14:15
flaper87kgriffs: tiny nit in the error message, then +2. Promissed14:16
kgriffskk14:16
*** jeffrey4l_ has joined #openstack-zaqar14:21
flaper87jeffrey4l_: hey, there?14:30
jeffrey4l_flaper87, yep14:30
flaper87jeffrey4l_: did you add tests for links ? https://review.openstack.org/#/c/123626/214:30
*** ametts has joined #openstack-zaqar14:30
jeffrey4l_flaper87, I am trying to do that.14:30
jeffrey4l_I forget to add the test on the latest patch.14:31
flaper87jeffrey4l_: kk, thanks. Any chance you can do that today?14:31
jeffrey4l_It will be commit in few minute.14:31
kgriffsflaper87: re that error message, how about this (at the top of the pad)14:32
kgriffshttps://etherpad.openstack.org/p/zaqar-scratch14:32
flaper87kgriffs: +214:33
kgriffskk, here it comes14:33
flaper87sorry for nitpicking on that but I've seen many people frustrated with error messages14:33
flaper87and we definitely don't want zaqar to be the target for complains on that14:33
* kgriffs really needs to write a sublime macro for splitting long strings14:34
jeffrey4l_flaper87, I will submit it soon.14:34
mpanettahey, that's my job :P (complaining)14:34
kgriffsflaper87: please, nitpick away! Let's make Zaqar have the best error messages of all the projects!14:34
mpanettaMmmm good error messages14:34
mpanettaUs devops folks like that14:35
kgriffsI want people to be all ready for a crappy installation experience, and then be like "WAH? that's all I have to do?"14:35
kgriffsand then when something goes wrong, be all ready to tear my hair out over unhelpful errors14:35
mpanettakgriffs: Did zaqar ever have a crappy installation experience?14:35
kgriffsand then be like "WAH? that actually... makes sense."14:36
kgriffsmpanetta: no, we tried hard to make it easy from the beginning. :D14:36
*** reed has joined #openstack-zaqar14:36
mpanettaThat is what I thought :P14:36
kgriffsmpanetta: I mean, people who are used to installing some other projects that are harder14:36
mpanettaLike devstack? :P14:37
* kgriffs declines to comment14:37
mpanettahaha14:37
flaper87LOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL14:39
*** reed has quit IRC14:39
openstackgerritKurt Griffiths proposed a change to openstack/zaqar: Add support for redis-sentinel  https://review.openstack.org/12147414:39
*** reed has joined #openstack-zaqar14:39
*** njirap has quit IRC14:39
kgriffsflaper87: speaking of error messages we still need to add tracing ID to all of ours14:40
flaper87kgriffs: +214:40
flaper87that's a good task for an OPW mentee14:40
flaper87what do you think?14:40
kgriffsflaper87: now that you are PTL it's your job to keep track of all the things we have to do. :D14:40
flaper87it's not just adding codes but also having a way to track them14:40
* flaper87 feels betrayed14:41
flaper87:P14:41
kgriffsflaper87: yeah, that could be a good job for an intern14:41
kgriffsmaybe as a first task so they get to survey all the code14:41
kgriffshelp them get oriented14:41
*** X019 has joined #openstack-zaqar14:44
flaper87LOL14:45
flaper87somehow I read: "Help *me* get oriented"14:45
flaper87brb14:49
*** flaper87 is now known as flaper87|afk14:49
*** njirap has joined #openstack-zaqar14:53
njirapkgriffs ping15:02
kgriffso/15:02
njirapin the README for running functional tests am supposed to "Change `$ZAQAR_TESTS_CONFIGS_DIR/functional-tests.conf` and15:03
njirap   set `run_tests` to True."15:03
njirapwheres the ZAQAR_TESTS_CONFIGS_DIR/functional-tests.conf15:03
openstackgerritJeffrey Zhang proposed a change to openstack/zaqar: Fix no links property in flavor list response  https://review.openstack.org/12362615:03
*** jchai has joined #openstack-zaqar15:07
*** alcabrera|afk is now known as alcabrera15:29
*** prashanthr_ has joined #openstack-zaqar15:33
kgriffssorry - had to run to a meeting15:38
kgriffsnjirap: the functional tests should run by default, so you don't have to change anything15:39
njirapkgriffs: yeap they've run actually15:40
kgriffskk15:41
*** flaper87|afk is now known as flaper8715:51
*** alcabrera is now known as alcabrera|afk15:52
openstackgerritJeffrey Zhang proposed a change to openstack/zaqar: Fix no links property in pool list response  https://review.openstack.org/12346215:53
jeffrey4l_flaper87, the link property in flavor and pool list response are submitted. unittest test is added. Pls review it when u r free. https://review.openstack.org/123626  https://review.openstack.org/12346215:54
flaper87jeffrey4l_: thanks a lot15:55
flaper87will do15:55
flaper87kgriffs: +2, I'm good with you +2'ing it15:55
*** jchai is now known as jchai_afk15:55
flaper87unless malini can review the patch now :P15:56
kgriffswould be good to have someone at least do a +1 sanity check15:56
kgriffsIMO15:56
maliniwhich patch needs sanity check?15:56
flaper87malini: https://review.openstack.org/#/c/121474/715:57
*** jchai_afk is now known as jchai15:57
flaper87kgriffs: re https://review.openstack.org/#/c/126270/1/zaqar/queues/storage/mongodb/messages.py,cm15:58
flaper87I used many concurrent p_q15:59
kgriffsoic, just wasn't clear from the paste15:59
flaper87In my comment I filtered just /my_queue/ to show how markers are groupped15:59
flaper87grouped15:59
kgriffsanyway, like I said, I think it will be OK15:59
flaper87yeah, TBH, I haven't found a good tool for chunk analysis15:59
flaper87I've been using queries to get that info15:59
flaper87and MMS is just monitoring15:59
kgriffssomeone should create an open-source MMS thing. :D16:00
kgriffsand then we could add things like chunk analysis16:00
kgriffsheatmaps16:00
kgriffsetc.16:00
*** prashanthr_ has quit IRC16:04
*** jeffrey4l_ has quit IRC16:05
*** expshaifali|afk is now known as exploreshaifali16:07
*** prashanthr_ has joined #openstack-zaqar16:08
kgriffsmalini: thanks for the review!16:08
maliniyw :)16:09
flaper87malini: https://review.openstack.org/#/c/126270 I'll thank you too16:11
flaper87:P16:11
openstackgerritKurt Griffiths proposed a change to openstack/zaqar: Reduce duration of some long-running tests  https://review.openstack.org/12694716:12
malininobody gives out pop-tarts these days :-(16:16
*** jchai is now known as jchai_afk16:17
*** jchai_afk is now known as jchai16:19
* flaper87 gives malini a truck full of pop-tarts16:22
* malini starts chomping16:22
maliniyum yum16:22
flaper87malini: kgriffs proposed/juno backports https://review.openstack.org/#/q/status:open+project:openstack/zaqar+branch:proposed/juno,n,z16:23
flaper87no hurries but it would be cool to have them merged by tomorrow16:23
kgriffsok16:23
kgriffsBTW, were you going to try and get some of those Jeffrey Zhang patches as well once they land?16:24
* kgriffs remembers he is supposed to review those16:24
flaper87kgriffs: yup, I'd love to backport them too16:25
kgriffshttps://etherpad.openstack.org/p/zaqar-smoke-testing16:28
kgriffsflaper87: ^^^16:28
openstackgerritA change was merged to openstack/zaqar: Add support for redis-sentinel  https://review.openstack.org/12147416:31
openstackgerritA change was merged to openstack/zaqar: Move marker field to the first position (index)  https://review.openstack.org/12627016:47
*** alcabrera|afk is now known as alcabrera16:51
*** njirap has quit IRC16:59
*** prashanthr_ has quit IRC17:04
*** malini1 has joined #openstack-zaqar17:28
*** malini has quit IRC17:30
*** sriram has quit IRC17:32
*** kgriffs is now known as kgriffs|afk17:46
*** reed has quit IRC17:50
*** anna__ has joined #openstack-zaqar17:59
*** X019 has quit IRC18:01
*** amitgandhinz has joined #openstack-zaqar18:04
*** anna__ is now known as X019_18:05
*** echevemaster has joined #openstack-zaqar18:06
*** shaifali has joined #openstack-zaqar18:19
*** kgriffs|afk is now known as kgriffs18:35
*** kgriffs is now known as kgriffs|afk18:45
*** amitgandhinz has quit IRC19:07
*** kgriffs|afk is now known as kgriffs19:29
*** malini1 has quit IRC19:37
*** amitgandhinz has joined #openstack-zaqar20:15
*** mpanetta has quit IRC21:01
*** earnThis has joined #openstack-zaqar21:06
*** sgotliv has quit IRC21:11
*** alcabrera is now known as alcabrera|afk22:02
*** flaper87 is now known as flaper87|afk22:03
openstackgerritAbhishek Chanda proposed a change to openstack/zaqar: Remove all traces of nose  https://review.openstack.org/12704222:05
*** earnThis has quit IRC22:05
*** echevemaster has quit IRC22:13
*** flwang has quit IRC22:23
*** jchai has quit IRC22:25
*** amitgandhinz has quit IRC22:33
*** flwang has joined #openstack-zaqar22:36
*** kgriffs is now known as kgriffs|afk22:39
*** jeffrey4l_ has joined #openstack-zaqar22:50
*** jeffrey4l_ is now known as jeffrey4l22:50
*** kgriffs|afk is now known as kgriffs22:58
*** kgriffs is now known as kgriffs|afk23:08
*** shaifali has quit IRC23:19
*** jeffrey4l has quit IRC23:56
*** kgriffs|afk is now known as kgriffs23:59
« Tuesday, 2014-10-07 Index Thursday, 2014-10-09 »

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