Friday, 2014-10-10

« Thursday, 2014-10-09 Index Saturday, 2014-10-11 »
*** jeffrey4l_ has quit IRC00:49
*** jeffrey4l_ has joined #openstack-zaqar01:38
*** jeffrey4l_ has quit IRC01:49
*** jeffrey4l_ has joined #openstack-zaqar02:01
*** openstackgerrit has quit IRC02:58
*** jeffrey4l_ has quit IRC03:49
*** jeffrey4l_ has joined #openstack-zaqar04:06
*** echevemaster has quit IRC04:14
*** jeffrey4l_ has quit IRC04:22
*** jeffrey4l_ has joined #openstack-zaqar04:35
*** sgotliv has joined #openstack-zaqar05:45
*** flaper87|afk is now known as flaper8706:46
*** reed has joined #openstack-zaqar06:50
jeffrey4l_flaper87, Good morning~07:11
flaper87jeffrey4l_: good morning/afternoon, Sir!07:12
jeffrey4l_flaper87, is there any difference between v1 and v1.1 in list queue?07:12
flaper87yes07:12
flaper87v1.1 did return 204. We decided to change that behavior for v1.107:12
flaper87erm, sorry. v1 returns 204, that is.07:12
jeffrey4l_yes.07:13
jeffrey4l_I just notice that.07:13
jeffrey4l_My patch about the link pro in list flavor/queue change both v1 and v1.107:14
flaper87jeffrey4l_: that's fine. We were discussing that yday. I think your patch is fine. We need to fix v107:15
flaper87Pools shouldn't have been released in v107:15
flaper87anyway, I'm glad you fixed that07:15
jeffrey4l_flaper87, I am a little confused by the wiki page which is a little out of date.07:16
flaper87jeffrey4l_: the v1.1 one?07:16
flaper87please, if you spot something, let us know07:16
jeffrey4l_You means that the wiki page is the latest version?07:18
flaper87jeffrey4l_: it should contain the spec of how we expect the API to be07:19
flaper87if there're mismatches we need to address them asap :P07:19
flaper87or at least discuss them07:19
jeffrey4l_check these two link https://git.openstack.org/cgit/openstack/zaqar/tree/zaqar/queues/transport/wsgi/v1_0/messages.py#n107   https://wiki.openstack.org/wiki/Zaqar/specs/api/v1#List_Messages07:19
flaper87jeffrey4l_: ah wait, forget about v1, we just care about v1.1 now.07:20
jeffrey4l_based on the wiki page, v1 list messages should return 204 when the message is empty.07:20
flaper87as in, we're not going to make breaking changes on v1 unless there're really aweful bugs (like the pools one)07:20
jeffrey4l_OK.07:21
jeffrey4l_Ah, it seems that I make a mistake. The one mentioned above is right. I missed some lines.07:23
*** sgotliv has quit IRC07:24
jeffrey4l_flaper87, Check this http://paste.openstack.org/show/120080/ . About "Capture the stdout and stderr based on environment" https://review.openstack.org/#/c/124273/  . I blocked when using ipdb.set_trace()07:32
flaper87jeffrey4l_: yeah,I know what happens. As I mentioned in the review. I'm fine with the patch, I guess I haven't been beaten by that because I just use `nosetests` for debug07:34
jeffrey4l_flaper87, I using the nosetests too(check the comments in the paste link). I suppose that we may using difference version of ipdb/ipython/nose?07:36
flaper87jeffrey4l_: mmh, that's weird07:38
flaper87jeffrey4l_: can you try with just *pdb* ?07:38
flaper87instead of ipdb07:38
jeffrey4l_flaper87, Yes. I tried. pdb works.07:38
flaper87oh ok, so, it's something about ipdb07:39
flaper87interesting07:39
jeffrey4l_What's the version of ipdb/ipython are u using?07:39
flaper87mmh, right now I'm just using pdb07:40
*** sgotliv has joined #openstack-zaqar07:40
jeffrey4l_Found it. it is caused by ipython07:43
jeffrey4l_/home/jeffrey/.dotfiles/virtualenvs/zaqar/local/lib/python2.7/site-packages/IPython/utils/io.py07:43
jeffrey4l_stderr = IOStream(sys.stderr, fallback=devnull)07:43
jeffrey4l_flaper87, in that file. ipython wrap the sys.stderr which raised exception.07:43
flaper87mmh, bad ipython, bad!07:44
flaper87:D07:44
jeffrey4l_;p07:44
*** sgotliv has quit IRC08:37
*** sphoorti has joined #openstack-zaqar09:02
sphoortiflaper87: Hello :D09:02
*** exploreshaifali has joined #openstack-zaqar09:04
*** exploreshaifali has left #openstack-zaqar09:05
*** jeffrey4l_ is now known as jeffrey4l09:06
*** exploreshaifali has joined #openstack-zaqar09:07
*** exploreshaifali has quit IRC09:11
*** exploreshaifali has joined #openstack-zaqar09:12
*** exploreshaifali has quit IRC09:27
*** jeffrey4l has quit IRC09:35
*** jeffrey4l has joined #openstack-zaqar09:49
*** sgotliv has joined #openstack-zaqar09:59
*** sgotliv_ has joined #openstack-zaqar10:35
*** sgotliv has quit IRC10:38
*** X019 has quit IRC10:41
*** X019 has joined #openstack-zaqar10:54
*** malini has joined #openstack-zaqar11:00
*** malini has quit IRC11:01
*** malini has joined #openstack-zaqar11:01
*** sphoorti has quit IRC11:33
*** X019 has quit IRC11:41
*** X019 has joined #openstack-zaqar11:44
*** jeffrey4l has quit IRC12:08
*** jchai has joined #openstack-zaqar13:00
*** jeffrey4l has joined #openstack-zaqar13:02
*** malini has quit IRC13:07
*** sriram has joined #openstack-zaqar13:09
*** amitgandhinz has joined #openstack-zaqar13:14
*** mpanetta has joined #openstack-zaqar13:16
*** mpanetta has quit IRC13:18
*** mpanetta has joined #openstack-zaqar13:18
*** mpanetta has quit IRC13:19
*** mpanetta has joined #openstack-zaqar13:19
*** amitgandhinz has quit IRC13:37
*** malini has joined #openstack-zaqar13:47
flaper87malini: kgriffs|afk thoughts? https://review.openstack.org/#/c/127518/14:14
flaper87I think this one is rc2 worth it too14:14
* malini checking14:14
*** openstackgerrit has joined #openstack-zaqar14:15
maliniCan you also update https://wiki.openstack.org/wiki/Zaqar/specs/api/v1.1#Renew_Claim ?14:16
flaper87malini: yup, will do14:18
*** jchai has quit IRC14:33
*** jchai has joined #openstack-zaqar14:34
*** sgotliv_ has quit IRC14:39
flaper87malini: I'll also wait for kgriffs|afk to review that patch before updating the wiki. Just to make sure he also agrees with the default ttl thing14:39
malinisure14:40
maliniflaper87: https://review.openstack.org/#/c/127518/1/zaqar/queues/transport/wsgi/v1_0/claims.py14:40
flaper87malini: 'sup ?14:41
flaper87:P14:41
malinishould it be ('grace', int, 0) or ('grace', int, None) ?14:41
flaper870 so it's not a required param in the API and it does not fail in the metadata['ttl'] + metadata['grace'] sum in the claim controller14:42
maliniaah..ok14:42
*** kgriffs|afk is now known as kgriffs14:46
*** jeffrey4l has quit IRC14:48
kgriffsflavio, re https://review.openstack.org/#/c/127518/14:49
flaper87kgriffs: shoot14:51
kgriffsI think there has been a misunderstanding about the purpose of the grace period. It applies to the message expiration, not the claim expiration.14:52
kgriffsthe idea is that once a claim expires, you want messages to outlive it by some "grace" period so another worker has a chance to claim the message before it self-destructs14:52
flaper87kgriffs: oh mmh, I thought I did that.14:54
flaper87aaaaaaaaah wait14:54
flaper87I applied it to everything14:54
flaper87claim AND message14:54
flaper87(facepalm)14:54
kgriffsyeah14:54
kgriffs:)14:54
flaper87damn, now I feel dumb14:54
flaper87good thing we have you, kgriffs14:54
flaper87:D14:54
kgriffsit's OK. I feel dumb at least 10 times a day14:54
kgriffs;)14:54
flaper87kgriffs: does the rest of the patch look fine ?14:55
maliniand somebody +2s flaper87 10 times a day14:55
*** amitgandhinz has joined #openstack-zaqar14:56
kgriffslol14:56
kgriffsflaper87: other thing I noticed at first glance was that the redis driver claim controller also needs it's defaults changed from 60 seconds14:58
kgriffss/it's/its14:58
flaper87kgriffs: yeah, I was revisiting that now14:58
* kgriffs stupid english14:58
*** jeffrey4l has joined #openstack-zaqar15:01
openstackgerritFlavio Percoco proposed a change to openstack/zaqar: Fix ttl and grace story for claims updates  https://review.openstack.org/12751815:06
flaper87kgriffs: ^15:06
kgriffskk, I'll take a look15:07
*** alcabrera|afk is now known as alcabrera15:20
*** alcabrera is now known as alcabrera|afk15:21
kgriffsoh cool, this is going to be handy. http://redis.io/commands/slowlog15:45
kgriffs</random>15:45
*** malini has left #openstack-zaqar15:48
*** reed has quit IRC15:55
*** jchai is now known as jchai_afk16:06
*** alcabrera|afk is now known as alcabrera16:14
*** jeffrey4l has quit IRC16:46
*** sgotliv_ has joined #openstack-zaqar16:49
*** sgotliv_ has quit IRC16:55
*** sgotliv_ has joined #openstack-zaqar17:38
*** exploreshaifali has joined #openstack-zaqar17:53
*** jchai_afk is now known as jchai17:57
*** kgriffs is now known as kgriffs|afk18:31
exploreshaifaliflaper87, around?18:35
exploreshaifaliflaper87, ping18:39
*** openstackgerrit has quit IRC18:48
*** jchai is now known as jchai_afk18:52
*** openstackgerrit has joined #openstack-zaqar18:54
exploreshaifaliflaper87, ting19:05
exploreshaifali:D19:05
*** mpanetta has quit IRC19:24
*** jchai_afk is now known as jchai19:32
exploreshaifalihi jchai19:34
jchaiexploreshaifali: hello19:34
exploreshaifalijchai, are u a developer of zaqar, I have few questions relatead to it19:35
jchaiexploreshaifali: nope. I'd try kgriffs|afk, flaper87, alcabrera, sriram19:41
exploreshaifalijchai, ok, Thanks!19:42
alcabreraoh, hi all.19:42
alcabrerajchai, exploreshaifali: o/19:42
exploreshaifalialcabrera, hi19:43
alcabreraI'm an ex-developer of zaqar, but I may be able to help. :)19:44
exploreshaifaliI just want to know in code https://github.com/openstack/zaqar/tree/master/zaqar/queues/storage19:44
exploreshaifalimongo/driver19:44
exploreshaifaliwhere what conf object represent19:45
exploreshaifaliconf is an object of oslo.config.ConfOPts19:45
exploreshaifaliright?19:45
alcabreraexploreshaifali: yup!19:45
alcabrerait's... loaded at some point to contain the values specified here: https://github.com/openstack/zaqar/blob/master/etc/zaqar.conf.sample19:45
exploreshaifalialcabrera, sorry but I am unablt to connect how it works19:48
exploreshaifalijust a min19:48
exploreshaifalialcabrera, see https://github.com/openstack/zaqar/blob/master/zaqar/queues/storage/base.py#L4319:48
exploreshaifaliI am searching for class oslo.conf.ConfOpts19:49
exploreshaifaliso that I can understand what exactly conf holds19:50
alcabreraah19:50
alcabreraoslo.config lives in a separate project, and the link to it is -19:50
alcabrerahttps://github.com/openstack/oslo.config/blob/master/oslo/config/cfg.py#L157419:50
exploreshaifaliyes19:50
exploreshaifaliI have gone through it a bit19:51
exploreshaifalialcabrera, probaly it is relatead to https://github.com/openstack/zaqar/blob/master/zaqar/queues/storage/mongodb/options.py19:51
alcabreraah, yes! :)19:51
exploreshaifalibut how?19:52
exploreshaifaliI mean as per my knowledge of Python says19:52
exploreshaifalithere should be oslo.conf.ConfOpts class19:53
exploreshaifalialcabrera, ^19:53
alcabrerahmmm19:54
alcabrerathere is such a class, though. o.o19:54
alcabreraoslo.config.ConfigOpts19:54
alcabrerawhich is brought into scope by 'import oslo.config'19:54
alcabreraerrm19:54
alcabrerawhich part is missing? :x19:54
exploreshaifalialcabrera, yes19:56
exploreshaifalicorrect, got it19:56
exploreshaifaliand one more question19:56
exploreshaifalialcabrera, https://github.com/openstack/zaqar/blob/master/zaqar/queues/storage/mongodb/options.py19:57
exploreshaifaliabove file also import oslo.config.conf19:57
exploreshaifalithis file specify configs for storage driver19:58
exploreshaifaliright?19:58
alcabrerayup. :)19:58
exploreshaifalithis is for storage layer of zaqar19:58
exploreshaifali?19:58
alcabreraexploreshaifali: yup.19:58
exploreshaifaliand storage layer have 2 parts19:59
exploreshaifalidata plane and control plane19:59
exploreshaifalialcabrera, both planes are managed by this single file19:59
alcabreraexploreshaifali: that's correct. data: DataDriver, control: ControlDriver20:00
exploreshaifaliat present, database configs for both planes are same as there is same databse for both20:01
exploreshaifalialcabrera, okay, first things first20:01
exploreshaifaliI am trying to understand if I want to divide data plane and control plane20:02
exploreshaifaliin 2 separate parts20:02
exploreshaifaliwht do I need to do with config fils20:02
exploreshaifali*files20:02
exploreshaifalialcabrera, ^20:02
alcabreraI don't think I can help there20:02
alcabreraIt's been too long since I last looked at zaqar20:03
exploreshaifalialcabrera, np20:03
alcabreragood luck, exploreshaifali. :)20:03
exploreshaifaliThanks alot20:03
exploreshaifalialcabrera, I ll still keep catching u for my more questions :P20:04
alcabrerahehe. :P20:09
*** jchai is now known as jchai_afk20:11
*** jchai_afk is now known as jchai20:11
*** sgotliv_ has quit IRC20:30
*** jchai has quit IRC20:43
*** alcabrera is now known as alcabrera|afk20:59
*** sriram has quit IRC21:22
*** amitgandhinz has quit IRC22:13
*** ametts has quit IRC22:27
*** AAzza has quit IRC22:29
*** AAzza_afk has joined #openstack-zaqar22:32
*** AAzza_afk is now known as AAzza22:32
*** sgotliv_ has joined #openstack-zaqar22:38
*** flaper87 is now known as flaper87|afk22:46
*** echevemaster has joined #openstack-zaqar22:52
*** openstackgerrit has quit IRC22:54
« Thursday, 2014-10-09 Index Saturday, 2014-10-11 »

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