| *** mpanetta has joined #openstack-zaqar | 00:12 | |
| *** mpanetta has quit IRC | 00:21 | |
| *** vkmc has quit IRC | 01:08 | |
| *** mpanetta has joined #openstack-zaqar | 01:17 | |
| *** alcabrera|afk is now known as alcabrera | 01:47 | |
| *** alcabrera is now known as alcabrera|afk | 02:17 | |
| *** jeffrey4l has quit IRC | 02:27 | |
| *** jeffrey4l has joined #openstack-zaqar | 02:39 | |
| *** flwang has quit IRC | 03:41 | |
| *** mpanetta has quit IRC | 03:46 | |
| *** flwang has joined #openstack-zaqar | 03:55 | |
| *** jeffrey4l has quit IRC | 04:07 | |
| *** jeffrey4l has joined #openstack-zaqar | 04:28 | |
| *** jeffrey4l has quit IRC | 04:53 | |
| *** jeffrey4l has joined #openstack-zaqar | 05:05 | |
| *** flwang has quit IRC | 05:34 | |
| *** flaper87 has quit IRC | 06:05 | |
| *** flaper87 has joined #openstack-zaqar | 06:05 | |
| *** jeffrey4l has quit IRC | 06:06 | |
| *** jeffrey4l has joined #openstack-zaqar | 06:06 | |
| jeffrey4l | flaper87, ping | 06:06 |
|---|---|---|
| flaper87 | jeffrey4l: pong | 06:06 |
| jeffrey4l | Pool and flavor exist on apiv 1.1. But on the python-zaqarclient, It just support api v1. | 06:07 |
| jeffrey4l | how can we add the support for pool and flavor u asked me? | 06:08 |
| jeffrey4l | flaper87, Pool and flavor exist on apiv 1.1. But on the python-zaqarclient, It just support api v1. | 06:09 |
| flaper87 | jeffrey4l: zaqarclient supports v1.1 too | 06:09 |
| flaper87 | jeffrey4l: and good morning/afternoon | 06:09 |
| flaper87 | jeffrey4l: this is the v1.1 spec https://github.com/openstack/python-zaqarclient/blob/master/zaqarclient/queues/v1/api.py#L243 | 06:09 |
| flaper87 | jeffrey4l: also, pools exist since v1 | 06:10 |
| flaper87 | jeffrey4l: AFAICT, we're just missing support for flavors | 06:10 |
| jeffrey4l | Ok. I didn't read the code carefully. I just read the v1 spec on wiki page: https://wiki.openstack.org/wiki/Zaqar/specs/api/v1 It seems they are out of date. | 06:11 |
| flaper87 | jeffrey4l: :D don't worry, that's what we are here | 06:11 |
| flaper87 | :) | 06:11 |
| jeffrey4l | flaper87, anyway, i will add flavor and pool support ASAP. | 06:12 |
| jeffrey4l | btw, good afternoon ;p | 06:13 |
| flaper87 | jeffrey4l: there's already support for pool https://github.com/openstack/python-zaqarclient/blob/master/zaqarclient/queues/v1/pool.py | 06:13 |
| flaper87 | jeffrey4l: wait, actually https://github.com/openstack/python-zaqarclient/blob/master/zaqarclient/queues/v1/flavor.py | 06:13 |
| flaper87 | I added flavors too | 06:13 |
| flaper87 | mmmh, I can't remember what the exact request was now | 06:13 |
| flaper87 | oh lord | 06:13 |
| flaper87 | jeffrey4l: aahhh wait | 06:13 |
| flaper87 | jeffrey4l:the *CLI* support is missing | 06:14 |
| flaper87 | that's what we need | 06:14 |
| flaper87 | (sorry, I just woke up) | 06:14 |
| jeffrey4l | Yep, I notice that. | 06:14 |
| jeffrey4l | another question: Should we separate the v1 and v1.1 | 06:16 |
| jeffrey4l | i.e. add openstack.queuing.v1.1 in the setup.cfg. | 06:16 |
| jeffrey4l | we need that, right? | 06:16 |
| flaper87 | jeffrey4l: mmh, nope, we don't need that this time. | 06:22 |
| flaper87 | As far as I can tell, we should be able to re-use the ones we have now | 06:22 |
| *** flwang has joined #openstack-zaqar | 06:26 | |
| jeffrey4l | flaper87, yep, we re-use the most current implement in v1/cli.py. But how can we handle the difference between api v1 and v2? For example, on the deleting message, v1.1 api has a extra param `pop`. | 06:34 |
| flaper87 | jeffrey4l: pop is a separate action | 06:34 |
| flaper87 | there's a method for pop | 06:34 |
| flaper87 | I mean, although it is part of the DELETE in the API, we consider it a separate action | 06:34 |
| jeffrey4l | flaper87, but v1 has no this kind of method. It is good to mix this up? It is confused when using the client. | 06:47 |
| jeffrey4l | After thinking for a while, maybe u r right. In api/client design, minor api version(like zaqar v1/v1.1) update is different with major api version( like keystone v2.0 and v3) | 06:47 |
| flaper87 | exactly my point :) | 06:49 |
| flaper87 | the changes are small | 06:49 |
| flaper87 | users will get a `UnsupportedVersion` error if they try to call `pop` on a `v1` endpoint | 06:49 |
| jeffrey4l | flaper87, got. Thanks for explanation. I will add this to CLI ASAP. cheers. | 06:51 |
| *** flwang has quit IRC | 06:55 | |
| jeffrey4l | flaper87, There are still lack of some method in pool/flavor. For example list pool/flavor, update pool/flavor. A u going to add it? | 07:39 |
| flaper87 | jeffrey4l: not now, feel free to add them | 07:40 |
| jeffrey4l | ok. | 07:40 |
| *** sgotliv_ has joined #openstack-zaqar | 08:09 | |
| *** flwang has joined #openstack-zaqar | 08:21 | |
| *** sgotliv_ has quit IRC | 08:45 | |
| *** flwang has quit IRC | 09:00 | |
| *** flwang has joined #openstack-zaqar | 09:00 | |
| *** sgotliv_ has joined #openstack-zaqar | 09:01 | |
| *** sgotliv_ has quit IRC | 09:08 | |
| *** sgotliv_ has joined #openstack-zaqar | 09:13 | |
| *** sgotliv_ has quit IRC | 09:28 | |
| *** jeffrey4l has quit IRC | 10:04 | |
| *** jeffrey4l has joined #openstack-zaqar | 10:21 | |
| *** mpanetta has joined #openstack-zaqar | 10:45 | |
| *** mpanetta has quit IRC | 10:49 | |
| *** sgotliv_ has joined #openstack-zaqar | 10:56 | |
| *** sphoorti has joined #openstack-zaqar | 11:04 | |
| sphoorti | Hello All! | 11:09 |
| *** vkmc has joined #openstack-zaqar | 11:09 | |
| vkmc | morniiing | 11:10 |
| sphoorti | hello | 11:11 |
| vkmc | hi sphoorti! :) | 11:13 |
| sphoorti | :) | 11:14 |
| sphoorti | vkmc: in reference to this bug https://bugs.launchpad.net/zaqar/+bug/1369536 for delete message with expired claim id : https://github.com/openstack/zaqar/blob/master/zaqar/queues/transport/wsgi/v1_1/messages.py#L330 is this correct ? | 11:16 |
| vkmc | sphoorti, you need a subset of those, the ones that have been claimed | 11:22 |
| sphoorti | so a unit test to check only the messages which have expired ? vkmc ? | 11:26 |
| vkmc | to make sure that the expired claims has been removed | 11:28 |
| sphoorti | Oh okay! got i! | 11:31 |
| vkmc | cool :) | 11:31 |
| flaper87 | o/ | 11:31 |
| flaper87 | gooooooooooood morning world | 11:31 |
| vkmc | flaper87, hiiiiiiiiiiiiiii \o | 11:31 |
| flaper87 | vkmc: had a great weekend? how are you doing? | 11:32 |
| sphoorti | well then vkmc I have to write a test for the function in which message has expired right ? | 11:35 |
| vkmc | flaper87, good too! I had enough resting and did some stuff I had in my to do list heh :) | 11:37 |
| sphoorti | flaper87: hello :) | 11:37 |
| flaper87 | sphoorti: hey hey :) | 11:37 |
| flaper87 | how's it going? | 11:37 |
| sphoorti | trying to work on bug https://bugs.launchpad.net/zaqar/+bug/1369536 | 11:40 |
| sphoorti | as my opw contribution | 11:40 |
| vkmc | sphoorti, yeah... remember that we want those which claim has expired, not the message expiration itself (message ttl) | 11:41 |
| sphoorti | understanding the code | 11:41 |
| vkmc | check out this post if you need an starting point about some of the concepts and main operations http://developer.rackspace.com/blog/openstack-marconi-api.html | 11:42 |
| sphoorti | thanks a lot! | 11:55 |
| *** cpallares has joined #openstack-zaqar | 11:56 | |
| *** openstackgerrit has joined #openstack-zaqar | 12:14 | |
| *** jchai has joined #openstack-zaqar | 12:55 | |
| *** sriram1 has joined #openstack-zaqar | 13:04 | |
| *** sriram1 has left #openstack-zaqar | 13:04 | |
| *** sriram1 has joined #openstack-zaqar | 13:05 | |
| *** sriram1 has quit IRC | 13:06 | |
| openstackgerrit | Victoria MartÃnez de la Cruz proposed a change to openstack/python-zaqarclient: Updates API name https://review.openstack.org/122856 | 13:20 |
| *** mpanetta has joined #openstack-zaqar | 13:22 | |
| *** sphoorti has quit IRC | 13:38 | |
| *** sriram has joined #openstack-zaqar | 13:41 | |
| *** alcabrera|afk is now known as alcabrera | 13:54 | |
| *** malini|afk is now known as malini | 13:54 | |
| *** echevemaster has joined #openstack-zaqar | 13:56 | |
| flaper87 | vkmc: re https://bugs.launchpad.net/zaqar/+bug/1372488 | 14:28 |
| flaper87 | my concern is | 14:28 |
| flaper87 | Say we have a freaking-fast flavor and we want to create a queue that is stored there | 14:29 |
| flaper87 | we have 2 issues right now | 14:29 |
| flaper87 | 1. The map we have is 1:1 between flavors and pools | 14:29 |
| flaper87 | which means, *all* the queues created in the freaking-fast flavor will go to the same pool | 14:29 |
| flaper87 | and we have no way to balance that | 14:29 |
| flaper87 | Instead, I think the mapping should be flavor:pool_type_or_scheme_probably_scheme_and_I_dunno_why_I_am_using_so_much_underscores | 14:30 |
| vkmc | lol | 14:30 |
| flaper87 | s/much/many | 14:31 |
| flaper87 | now | 14:31 |
| vkmc | wow much underscore | 14:31 |
| vkmc | 2.? | 14:31 |
| flaper87 | lets assume we fix that, we make flavor's pool field to store the type/scheme | 14:32 |
| flaper87 | with that, we can filter pools by type and balance pools there | 14:32 |
| vkmc | y | 14:32 |
| vkmc | sure | 14:33 |
| flaper87 | generally speaking, balancing queues between a mongodb node and a redis node it's not fair | 14:33 |
| flaper87 | they don't share the same capabilities nor they scale the same way | 14:33 |
| flaper87 | it is possible, yes. | 14:33 |
| flaper87 | but I'm unsure on whether that's a sane thing to do | 14:33 |
| vkmc | yeah.. but how sane it's to put a constrain on that? | 14:34 |
| vkmc | again, we are sacrificing interoperability | 14:34 |
| flaper87 | are we? if any, we're adding consistency | 14:34 |
| flaper87 | for example | 14:34 |
| flaper87 | What happens if I (the operator) add a redis pool because I want to have a freaking-fast flavor | 14:35 |
| flaper87 | ? | 14:35 |
| flaper87 | that doesn't mean I want zaqar to balance on it | 14:35 |
| flaper87 | oh well, wait | 14:35 |
| flaper87 | I think we actually have a fix for that | 14:35 |
| vkmc | I get what you are thinking | 14:35 |
| flaper87 | https://github.com/openstack/zaqar/blob/master/zaqar/common/storage/select.py#L39 | 14:36 |
| flaper87 | but again | 14:36 |
| flaper87 | that would prevent the algorithm to balance on it | 14:36 |
| flaper87 | ... in a per flavor basis | 14:36 |
| flaper87 | I think we can get around this without adding a new config option, TBH. | 14:36 |
| flaper87 | (which I'd personally prefer) | 14:36 |
| flaper87 | holy crap, 16:40 already :( | 14:37 |
| vkmc | yeah if there is a way of doing so, we should go for it | 14:37 |
| vkmc | something I'm not getting... you are mixing (or I'm doing so) flavors with pools type | 14:38 |
| vkmc | <flaper87> lets assume we fix that, we make flavor's pool field to store the type/scheme | 14:38 |
| vkmc | <flaper87> with that, we can filter pools by type and balance pools there | 14:38 |
| flaper87 | yeah | 14:39 |
| flaper87 | because right now our mapping is flavor:pool_name | 14:39 |
| vkmc | yeah | 14:39 |
| flaper87 | 2s, let me re-read pool's model | 14:39 |
| vkmc | so your idea is to have the mapping pool_name:pool_type | 14:39 |
| vkmc | so you end up having a 1 to many relationship with flavor and pool_type | 14:40 |
| vkmc | and that way do the balance? | 14:40 |
| flaper87 | no, the idea would be to have flavor:pool_type | 14:40 |
| flaper87 | yes | 14:40 |
| vkmc | transitive property ;) | 14:40 |
| flaper87 | there's another way to do it | 14:41 |
| flaper87 | without adding a type | 14:41 |
| vkmc | so directly, you want to change the mapping flavor:pool_name to flavor:pool_type | 14:41 |
| flaper87 | https://github.com/openstack/zaqar/blob/master/zaqar/queues/storage/mongodb/pools.py#L51-L54 | 14:41 |
| flaper87 | remove that index | 14:41 |
| flaper87 | I'm not sure why we made pool_name unique | 14:41 |
| vkmc | because its our id | 14:42 |
| flaper87 | wah no wayt | 14:42 |
| flaper87 | that's stupid | 14:42 |
| flaper87 | nevermind, we need pool_type | 14:42 |
| flaper87 | or pool_group | 14:42 |
| vkmc | you cannot remove it | 14:42 |
| vkmc | pool group sounds good | 14:42 |
| flaper87 | yeah, that'd make things like `dc-cluster` not possible | 14:42 |
| vkmc | ffh | 14:42 |
| flaper87 | ok, so | 14:43 |
| flaper87 | lets summarize | 14:43 |
| vkmc | k | 14:43 |
| flaper87 | we need a group field for pools | 14:43 |
| flaper87 | if no group is specified, the pool will be used globally | 14:44 |
| flaper87 | then flavors documentation must be updated to reflect that pool is the pool group and not the pool type | 14:45 |
| flaper87 | we need to make all this before the release | 14:45 |
| flaper87 | :) | 14:45 |
| vkmc | what is we allivate the 1:1 relationship between flavors and pools? | 14:45 |
| vkmc | you could have more than 1 pool with the same flavor | 14:46 |
| *** mpanetta has quit IRC | 14:46 | |
| vkmc | you probably went through that already | 14:46 |
| *** mpanetta has joined #openstack-zaqar | 14:46 | |
| flaper87 | yeah, the reason I don't like that is because you likely want your flavors balanced across all the pools of the same type | 14:46 |
| vkmc | that way we don't need to add new parameters | 14:47 |
| flaper87 | if we alleviate the 1:1 mapping, operators would have to add all the mongo pool names to the flavor | 14:47 |
| vkmc | but that is done when you create the pool | 14:48 |
| flaper87 | nope | 14:48 |
| flaper87 | pools must exist before flavors | 14:48 |
| vkmc | I know yeah | 14:48 |
| vkmc | if we do that change I mean | 14:49 |
| vkmc | hmm that would be so many changes | 14:50 |
| vkmc | adding an the pool_group param is an easier fix | 14:50 |
| vkmc | when is the release? | 14:50 |
| flaper87 | we still have a couple of weeks and this bug is critical | 14:51 |
| vkmc | ok, let's do it then | 14:52 |
| flaper87 | is this something you want to work on? otherwise I can stab it | 14:52 |
| flaper87 | btw, I don't see any items in the agenda | 14:55 |
| vkmc | I have some stuff to do this week, but I can organize myself and try to finish it soon if you are too busy | 14:55 |
| flaper87 | I'll try to get to it tomorrow morning, if I don't, I'll let you know | 14:56 |
| vkmc | sure | 14:56 |
| vkmc | please let me know | 14:56 |
| vkmc | I wanted to finish the functional tests for pools before the release too | 14:56 |
| flaper87 | if there are no things to discuss, I think we can skip the meeting today | 14:57 |
| kgriffs | flaper87: I have something to talk about wrt redis driver, but I can do that in here if we skip the mtg | 14:57 |
| * kgriffs believes rumors of his death have been greatly exaggerated | 14:58 | |
| flaper87 | kgriffs: yeah, lets do that here if you don't mind | 14:58 |
| flaper87 | kgriffs: LOL | 14:58 |
| kgriffs | sure thing | 14:58 |
| kgriffs | so, how is everybody? I had to go off the radar for a bit to get some other stuff done | 14:58 |
| mpanetta | Great, I've been playing with autobahn.ws | 14:59 |
| kgriffs | mpanetta: cool | 15:00 |
| kgriffs | looks like an interesting projecg! | 15:00 |
| mpanetta | It is quite interesting. | 15:01 |
| * flaper87 wishes there was a js driver for zaqar | 15:03 | |
| flaper87 | :( | 15:03 |
| kgriffs | somebody was talking about a js driver a couple weeks back... | 15:03 |
| * kgriffs can't remember their name | 15:03 | |
| *** alcabrera is now known as alcabrera|afk | 15:03 | |
| mpanetta | Ooo that would be nice | 15:03 |
| kgriffs | i.e., they were going to implement one | 15:03 |
| *** flwang has quit IRC | 15:04 | |
| flaper87 | kgriffs: yeah, it was yograterol | 15:05 |
| flaper87 | and myself | 15:05 |
| flaper87 | :P | 15:05 |
| kgriffs | oic | 15:05 |
| flaper87 | but I don't think he has started it yet | 15:05 |
| kgriffs | gtk | 15:05 |
| kgriffs | flaper87: so, rc1 is this thursday right? | 15:08 |
| flaper87 | right | 15:09 |
| kgriffs | flaper87: ok, so I was doing some exploratory perf testing on claims on friday before doing all the assorted graphs | 15:14 |
| vkmc | flaper87, kgriffs please review https://review.openstack.org/#/c/122856/ | 15:15 |
| kgriffs | and when I hit it with 1000 concurrent requests, started timing out on claiming messages - i mean, some requests started giving up in that transaction loop in claims.py | 15:15 |
| vkmc | hi kgriffs :) | 15:15 |
| flaper87 | kgriffs: that's specific to the redis driver, right? | 15:16 |
| kgriffs | correct | 15:16 |
| kgriffs | so, I hacked in a backoff sleep similar to what is in the mongo driver and it helped a lot | 15:17 |
| kgriffs | but, I want to try rewriting the claim stuff in lua today to see if it is even better | 15:17 |
| kgriffs | if I do that, it will require 2.6 as we've discussed previously | 15:17 |
| flaper87 | I think >=2.6 is fine | 15:18 |
| kgriffs | kk | 15:18 |
| kgriffs | in any case, I think we better get this done for rc | 15:18 |
| kgriffs | because if we don't, claiming under high concurrency is going to really be ugly | 15:18 |
| flaper87 | we've other's RCs comming. If this is a critical bug, lets file one and we can getting in. | 15:19 |
| kgriffs | ok | 15:19 |
| flaper87 | s/getting/get it/ | 15:19 |
| flaper87 | WTF? | 15:19 |
| flaper87 | :P | 15:19 |
| * flaper87 feels naked without his email | 15:19 | |
| * kgriffs goes off to do that | 15:19 | |
| * kgriffs tries not imagine flaper87 naked | 15:19 | |
| * flaper87 is experimenting a whole bunch of new emotions related to email | 15:19 | |
| kgriffs | https://bugs.launchpad.net/zaqar/+bug/1372510 | 15:23 |
| kgriffs | flaper87, vkmc: needs triaging ^^^ | 15:23 |
| vkmc | kgriffs, checking it | 15:24 |
| flaper87 | kgriffs: done | 15:24 |
| flaper87 | assign it to yourself | 15:24 |
| flaper87 | :D | 15:24 |
| kgriffs | oh, I should also say that I also tried adding the random sleep jitter in addition to the backoff and it made it a little better - so that random jitter in the mongo driver is actually helpful, turns out. :p | 15:24 |
| kgriffs | done | 15:25 |
| kgriffs | I'll keep working on it | 15:25 |
| flaper87 | kgriffs: cool | 15:25 |
| flaper87 | kgriffs: I added a comment in the bug | 15:25 |
| flaper87 | re versioning | 15:25 |
| flaper87 | btw, I've been using tags lately | 15:25 |
| flaper87 | `redis`, `mongodb`, `pools` etc | 15:26 |
| flaper87 | we should probably document them | 15:26 |
| flaper87 | if we haven't yet | 15:26 |
| flaper87 | I'll do that | 15:26 |
| flaper87 | OMFG, it's almost 17:30 | 15:26 |
| vkmc | flaper87, do you have an scheduled reboot? | 15:27 |
| flaper87 | kindof :P | 15:27 |
| kgriffs | oh | 15:34 |
| kgriffs | just remembered one other thing | 15:34 |
| kgriffs | when running those pub-sub benchmarks I forgot to setup the gc as a cron job | 15:34 |
| kgriffs | so... that may explain why snapshotting was taking so long and spiking some of those graphs | 15:34 |
| * kgriffs blushes | 15:34 | |
| flaper87 | :D | 15:40 |
| flaper87 | don't worry, just don't forget to do it in production | 15:41 |
| flaper87 | :P | 15:41 |
| *** sgotliv_ has quit IRC | 15:52 | |
| *** AAzza is now known as AAzza_afk | 15:58 | |
| *** AAzza_afk is now known as AAzza | 16:15 | |
| *** reed has joined #openstack-zaqar | 16:23 | |
| *** sphoorti has joined #openstack-zaqar | 16:42 | |
| sphoorti | vkmc: are you around ? | 17:17 |
| sphoorti | kgriffs: is this bug taken ? https://bugs.launchpad.net/zaqar/+bug/1357397 ? Can I start working on it ? | 17:21 |
| vkmc | sphoorti, hey! sup? :) | 17:38 |
| vkmc | sphoorti, I think you can grab it, nobody was assigned to it | 17:38 |
| sphoorti | yeah I just did that vkmc | 17:39 |
| sphoorti | :) | 17:39 |
| vkmc | cool | 17:39 |
| *** kgriffs is now known as kgriffs|afk | 17:43 | |
| * vkmc lurks | 18:23 | |
| *** malini is now known as malini|afk | 18:26 | |
| *** kgriffs|afk is now known as kgriffs | 18:34 | |
| kgriffs | sphoorti: thanks for taking that; i don't think anyone was working on it afaik | 18:35 |
| sphoorti | kgriffs: I am done with my changes. looking out for how to send for review | 18:36 |
| sphoorti | kgriffs: so far I made the changes and then ran tox -e py27 and that worked | 18:37 |
| sphoorti | am I on right track ? | 18:37 |
| kgriffs | yep | 18:37 |
| kgriffs | I also run pep8 too - it is easy to have a line that is too long or something | 18:37 |
| kgriffs | tox -e pep8,py27 | 18:37 |
| sphoorti | okay I will run pep8 too | 18:38 |
| kgriffs | then you will use git review to submit - have you set that up? | 18:38 |
| sphoorti | I have that setup | 18:38 |
| kgriffs | kewl | 18:38 |
| sphoorti | however I am slightly confused about the commit message | 18:38 |
| sphoorti | does git commit -a open the commit script ? | 18:39 |
| sphoorti | or we have to manually enter the contents in the right format ? | 18:39 |
| sphoorti | kgriffs: | 18:39 |
| kgriffs | oh | 18:40 |
| kgriffs | git commit -a will "add" the changes or "stage" them for the commit | 18:40 |
| kgriffs | wait | 18:40 |
| kgriffs | sorry | 18:40 |
| kgriffs | git commit -a will add and create the commit at the same time | 18:40 |
| kgriffs | then it will prompt you for your commit message | 18:41 |
| kgriffs | it will use whatever editor you specified in your config | 18:41 |
| kgriffs | (git config) | 18:41 |
| kgriffs | then you have to type it in according to the OpenStack guidelines | 18:41 |
| kgriffs | (it won't give you a template or anything) | 18:41 |
| kgriffs | does that make sense? | 18:41 |
| sphoorti | okay. I will do it | 18:41 |
| sphoorti | thanks a lot | 18:41 |
| kgriffs | rock on | 18:42 |
| sphoorti | pep8 too succeeded ! | 18:42 |
| sphoorti | I am ready to commit my changes now! | 18:43 |
| sphoorti | hope it goes well | 18:43 |
| vkmc | :) | 18:43 |
| *** sgotliv_ has joined #openstack-zaqar | 18:47 | |
| *** openstack has joined #openstack-zaqar | 18:55 | |
| -sendak.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 18:55 | |
| *** wpf has quit IRC | 19:01 | |
| *** wpf has joined #openstack-zaqar | 19:03 | |
| *** jchai is now known as jchai_afk | 19:03 | |
| *** jchai_afk is now known as jchai | 19:03 | |
| sphoorti | I am getting this error on git review http://paste.openstack.org/show/114275. I did use ssh-add and also checked on https://help.github.com/articles/generating-ssh-keys and https://help.github.com/articles/error-permission-denied-publickey | 19:05 |
| sphoorti | vkmc: kgriffs | 19:05 |
| sphoorti | what must be going wrong ? | 19:05 |
| boris-42 | vkmc hey there | 19:06 |
| boris-42 | vkmc how are you? | 19:06 |
| sphoorti | .whois boris-42 | 19:06 |
| boris-42 | sphoorti lol=) | 19:06 |
| sphoorti | hahahaha | 19:06 |
| sphoorti | :p | 19:06 |
| * boris-42 boris-42 is just dude | 19:06 | |
| vkmc | boris-42 is a cool dud that works on Rally | 19:06 |
| sphoorti | yayyieee | 19:06 |
| vkmc | hi boris-42 :) | 19:06 |
| boris-42 | vkmc so I saw your patch thank you | 19:07 |
| vkmc | sphoorti, did you upload your ssh keys to both Launchpad and Gerrit? | 19:07 |
| vkmc | boris-42, thank you for looking into it | 19:07 |
| boris-42 | vkmc some crap with requiremtns | 19:07 |
| sphoorti | is that boris pavlovic ? | 19:07 |
| boris-42 | sphoorti ya that one | 19:07 |
| sphoorti | yayy! | 19:07 |
| vkmc | is boris-42 famous? | 19:07 |
| vkmc | :D | 19:07 |
| boris-42 | vkmc lol | 19:07 |
| sphoorti | my friend k4no had once told me about you! | 19:07 |
| sphoorti | atleast here | 19:07 |
| boris-42 | sphoorti ya he is core in rally btw | 19:08 |
| sphoorti | he is famous | 19:08 |
| sphoorti | hello boris-42 | 19:08 |
| boris-42 | sphoorti hi there | 19:08 |
| boris-42 | sphoorti so you are working on zaqar?) | 19:08 |
| sphoorti | I just started working on zaqar. I am trying to submit my first patch to zaqar | 19:09 |
| vkmc | boris-42, looking at the logs now... hmm | 19:10 |
| boris-42 | vkmc ya seems like recheck requried | 19:10 |
| boris-42 | vkmc something went wrong during check of requreimtns | 19:10 |
| vkmc | boris-42, oh ok, calling it then | 19:10 |
| vkmc | oh you already did | 19:10 |
| vkmc | thanks :) | 19:10 |
| openstackgerrit | Sphoorti proposed a change to openstack/zaqar: Replace misleading max_message_size option name https://review.openstack.org/123223 | 19:12 |
| vkmc | ah well, it looks like you managed to fix it sphoorti | 19:12 |
| sphoorti | vkmc: all thanks to you! I had forgotten to replace the gerrit ssh key | 19:13 |
| vkmc | kewl | 19:14 |
| sphoorti | vkmc: would you review my patch ? | 19:16 |
| vkmc | sure, I have to finish some stuff first and then I'll review it | 19:17 |
| sphoorti | yayy :) | 19:18 |
| *** earnThis has joined #openstack-zaqar | 19:19 | |
| vkmc | earnThis, guess what :) | 19:22 |
| earnThis | hey vkmc, what? | 19:23 |
| *** flwang has joined #openstack-zaqar | 19:23 | |
| vkmc | earnThis, https://review.openstack.org/#/c/117574/ merged | 19:24 |
| earnThis | haha, thats awesome | 19:25 |
| vkmc | kudos :) | 19:25 |
| earnThis | thank you, now i just need to find time to get to that wsgi error standardization and ill really be on a roll | 19:27 |
| vkmc | haha sure thing | 19:27 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/zaqar: Move Redis driver's claim transaction to Lua https://review.openstack.org/123230 | 19:30 |
| *** earnThis has quit IRC | 19:33 | |
| *** AAzza is now known as AAzza_afk | 19:35 | |
| *** echevemaster has quit IRC | 19:37 | |
| *** earnThis has joined #openstack-zaqar | 19:39 | |
| sphoorti | kgriffs: could you please explain me a bit more about max_message_size ? | 19:48 |
| kgriffs | sure | 19:49 |
| kgriffs | so, this has a bit of history | 19:49 |
| kgriffs | originally we did check the sizes of the individual bodies | 19:50 |
| kgriffs | but that was changed and now we just check the entire size of the messages list | 19:50 |
| kgriffs | but the param name was never updated | 19:50 |
| sphoorti | so it could be one message or many together ? | 19:50 |
| kgriffs | the client can post a list of messages to enqueue | 19:50 |
| kgriffs | sphoorti: correct | 19:50 |
| kgriffs | so they can do one big message or a bunch of small messages | 19:51 |
| sphoorti | so how about max_size_message_post_list ? | 19:52 |
| kgriffs | hmm, sounds bit wordy | 19:52 |
| sphoorti | or max_size_message_list ? | 19:52 |
| *** sgotliv_ has quit IRC | 19:53 | |
| *** AAzza_afk is now known as AAzza | 19:54 | |
| * kgriffs thinking | 19:54 | |
| sphoorti | and maybe describe a little in the zaqar.conf.sample ? | 19:54 |
| kgriffs | max_message_list_size | 19:55 |
| *** sgotliv_ has joined #openstack-zaqar | 19:55 | |
| *** flwang has quit IRC | 19:55 | |
| sphoorti | yayy! that sounds better than max_size_message_list | 19:56 |
| kgriffs | only thing with that is it may be confused with listing messages, whereas max_messages_per_page is used for that | 19:56 |
| kgriffs | but, it may be the best we can do without getting too wordy | 19:56 |
| *** flwang has joined #openstack-zaqar | 19:56 | |
| kgriffs | flaper87: what do you think? | 19:56 |
| sphoorti | wouldnt max_size_message_list do ? | 19:56 |
| * flaper87 reads backlog | 19:56 | |
| kgriffs | sphoorti: re the help text, we should definitely be very clear there | 19:56 |
| kgriffs | and btw, the sample is auto-generated so once you update validation.py | 19:57 |
| kgriffs | you can re-run the script to generate a new sample | 19:57 |
| boris-42 | vkmc oh this requires some kind of rebase https://review.openstack.org/#/c/122876/ | 19:57 |
| kgriffs | sphoorti: look under tools/config | 19:57 |
| flaper87 | max_message_post_size ? | 19:57 |
| flaper87 | it refers to the specifc HTTP method | 19:57 |
| flaper87 | and message action | 19:58 |
| kgriffs | what if we add an s | 19:58 |
| kgriffs | max_messages_post_size | 19:58 |
| vkmc | boris-42, done :) | 19:58 |
| kgriffs | I'm thinking the plural form would make people think more along the lines of the list of messages | 19:59 |
| flaper87 | kgriffs: +1 | 19:59 |
| flaper87 | sold | 19:59 |
| sphoorti | so max_messages_post_size it is?! | 20:00 |
| flaper87 | sphoorti: +1 from me | 20:00 |
| boris-42 | vkmc ouch you specified not proper version for zaqar client | 20:01 |
| vkmc | boris-42, let me check | 20:01 |
| boris-42 | vkmc https://review.openstack.org/#/c/122876/2/requirements.txt | 20:01 |
| boris-42 | vkmc otherwise lgtm | 20:01 |
| sphoorti | yay flaper87! | 20:01 |
| vkmc | flaper87, so nice to see you around :) | 20:02 |
| flaper87 | vkmc: :) | 20:02 |
| * flaper87 had a writers day | 20:02 | |
| vkmc | (questions incoming...!) | 20:03 |
| openstackgerrit | Flavio Percoco proposed a change to openstack/zaqar: Enable ZAQAR_TEST_SLOW for py27 https://review.openstack.org/122732 | 20:03 |
| flaper87 | my brain was sweatting | 20:03 |
| flaper87 | oh dear lord | 20:03 |
| vkmc | LOL | 20:03 |
| flaper87 | just if you review that ^ | 20:03 |
| vkmc | you thought I was being nice | 20:03 |
| flaper87 | kgriffs: ^ | 20:03 |
| flaper87 | very very easy review | 20:03 |
| flaper87 | you are nice, your questions aren't | 20:03 |
| flaper87 | I'M JOOOOOOOOOOOOOOKING | 20:03 |
| flaper87 | sphoorti: don't freak out, I just like trolling vkmc | 20:03 |
| flaper87 | :D | 20:03 |
| sphoorti | hahaha | 20:03 |
| sphoorti | you all are so much fun to be with | 20:03 |
| sphoorti | :D | 20:04 |
| flaper87 | aren't we? | 20:04 |
| sphoorti | totally | 20:04 |
| flaper87 | told you this is the funniest channel E.V.E.R | 20:04 |
| vkmc | flaper87, ok, now that I'm blessed with your wonderful presence | 20:04 |
| flaper87 | wait 'til you see me dancing macarena | 20:04 |
| vkmc | flaper87, which version of the client should we stick to? pypi is 0.1.0, we use 0.0.2 for the bench tool and boris-42 pointed out that we are using 0.0.3 in the global reqs | 20:05 |
| flaper87 | vkmc: the right one would be 0.1.0 but it hasn't hit global requirements yet so 0.0.3 | 20:05 |
| boris-42 | vkmc so you have to use what is specified in global requreiments | 20:05 |
| vkmc | thanks guys | 20:05 |
| boris-42 | vkmc otherwise I won't be able to merge your patch | 20:05 |
| flaper87 | https://review.openstack.org/#/c/122706/ once this lands, you can use 0.1.0 | 20:05 |
| boris-42 | vkmc that's how global requriemtns works=) | 20:05 |
| * vkmc sad panda | 20:05 | |
| * flaper87 wonders what's wrong with pandas | 20:05 | |
| flaper87 | they're always sad | 20:06 |
| flaper87 | Ain't they eat enough bamboos ? | 20:06 |
| flaper87 | gosh | 20:06 |
| boris-42 | flaper87 =) | 20:06 |
| *** flwang has quit IRC | 20:06 | |
| boris-42 | vkmc so when this land https://review.openstack.org/#/c/122706/ we will get OpenStack update requirements patch in rally | 20:06 |
| * kgriffs has no idea what flaper87 is talking about. this channel is sooooo boring. Thank goodness for Solitaire or I would DIE from boredom. http://goo.gl/OJUB4f | 20:06 | |
| boris-42 | vkmc so let's from now add 0.0.3 | 20:06 |
| boris-42 | from/for* | 20:07 |
| vkmc | boris-42, added | 20:07 |
| boris-42 | vkmc 2ed | 20:07 |
| vkmc | boris-42, refresh :) | 20:07 |
| *** echevemaster has joined #openstack-zaqar | 20:07 | |
| vkmc | boris-42, thanks for the review | 20:08 |
| vkmc | we are so darn efficient | 20:08 |
| *** alcabrera|afk is now known as alcabrera | 20:08 | |
| sphoorti | kgriffs: this channel is awesome! you should this newcomers word for it :D | 20:09 |
| kgriffs | woot | 20:09 |
| vkmc | flaper87, I exchange you a review for https://review.openstack.org/#/c/122732/ for a review for https://review.openstack.org/#/c/122856/ | 20:09 |
| * kgriffs wonders what is going on here | 20:09 | |
| * kgriffs would not be surprised to see bags of gummy bears exchanged next | 20:10 | |
| flaper87 | kgriffs: http://en.wikipedia.org/wiki/Taenia_%28genus%29 <- this solitaire ? | 20:10 |
| flaper87 | vkmc: :O | 20:11 |
| flaper87 | vkmc: lets do this, I'll review your patch but you'll have to review mine and 2 more | 20:11 |
| flaper87 | mine is a 1 line thingy | 20:11 |
| flaper87 | yours is +10,-10 | 20:11 |
| vkmc | flaper87, 0 lines then | 20:11 |
| vkmc | you can't ask for an easier review | 20:12 |
| flaper87 | nope, but I can provide a more difficult one for you | 20:12 |
| vkmc | why I have the feeling that I end losing always? | 20:12 |
| flaper87 | at least 1 that makes this exchange fair | 20:12 |
| flaper87 | vkmc: yoooooooooooooooou woooooooooooooont, everything is going to be fiiiiiiiiiiiiiiiiiiiiiiiine | 20:13 |
| * flaper87 smiles and walks away | 20:13 | |
| vkmc | flaper87, to make this fair you should give me some of your gummybears | 20:13 |
| vkmc | flaper87, now that you probably have a new brand bag | 20:13 |
| flaper87 | vkmc: I've 2 :) | 20:13 |
| vkmc | last week you ate them all by Thursday | 20:13 |
| flaper87 | vkmc: ok, I'll give you 1 gummy bear | 20:13 |
| vkmc | (yeah I don't forget about that) | 20:14 |
| * kgriffs chuckles | 20:14 | |
| flaper87 | red, green, yellow, white ? | 20:14 |
| vkmc | who wants white gummy bears? | 20:14 |
| flaper87 | someone that likes pineapple | 20:14 |
| flaper87 | or is that lime ? | 20:14 |
| flaper87 | I've no idea | 20:14 |
| flaper87 | They're sweet, gummy and also bears | 20:14 |
| vkmc | red... violet :)~ | 20:15 |
| flaper87 | I just eat them | 20:15 |
| sphoorti | hahahhahaa | 20:15 |
| vkmc | and a green one too | 20:15 |
| flaper87 | vkmc: ok, let me explain this once and straight | 20:15 |
| flaper87 | 1 + 1 = 2 | 20:15 |
| flaper87 | I said, 1 | 20:15 |
| * vkmc covers her ears | 20:15 | |
| * flaper87 puts gummy bears in vkmc ears... >.> | 20:16 | |
| sphoorti | vkmc: should cover her eyes. She can see on irc :p | 20:16 |
| vkmc | yummm | 20:16 |
| flaper87 | there you have your 2 gummy bears | 20:16 |
| flaper87 | now go and review my patch | 20:16 |
| vkmc | sphoorti, I didn't because gummybears hurt on eyes | 20:16 |
| vkmc | in my ears... I don't mind | 20:16 |
| sphoorti | hahhahaha | 20:16 |
| kgriffs | brb | 20:16 |
| flaper87 | kgriffs: you stay here, we might need someone to chime in in this fight | 20:17 |
| flaper87 | vkmc: reviewed | 20:17 |
| kgriffs | lol | 20:17 |
| vkmc | flaper87, thanks Fla | 20:18 |
| flaper87 | my pleasure :) | 20:18 |
| vkmc | oh and you approved it :) good flaper | 20:18 |
| kgriffs | did a quick test to see how the lua script compares to doing a backoff sleep and retry with pipe.watch | 20:19 |
| kgriffs | guess which is which | 20:20 |
| kgriffs | https://gist.github.com/anonymous/92979c03e934b07e3210 | 20:20 |
| kgriffs | https://gist.github.com/anonymous/9a6cda2b7b1c127a124a | 20:20 |
| flaper87 | OMFG | 20:20 |
| flaper87 | :0 | 20:20 |
| flaper87 | 10x | 20:20 |
| vkmc | w00t | 20:21 |
| kgriffs | not bad for a first try | 20:21 |
| kgriffs | :) | 20:21 |
| flaper87 | kgriffs: you are having fun, aren't ya' ? | 20:22 |
| kgriffs | who, me? | 20:22 |
| * kgriffs looks innocent | 20:22 | |
| openstackgerrit | Sphoorti proposed a change to openstack/zaqar: Replace misleading max_message_size option name https://review.openstack.org/123223 | 20:23 |
| * flaper87 pictures kgriffs: http://photos1.blogger.com/blogger/5284/1129/400/Gummy%20Bear%20Persecution%20by%20Fire.jpg | 20:23 | |
| vkmc | nooooo hahaha | 20:23 |
| kgriffs | now that's fun | 20:24 |
| kgriffs | LOL | 20:24 |
| * flaper87 has done that | 20:24 | |
| kgriffs | reminds me of that time in fourth grade when I somehow got a magnifying glass | 20:24 |
| flaper87 | it stinks | 20:24 |
| kgriffs | recess was suddenly a lot more fun | 20:24 |
| flaper87 | kgriffs: lol | 20:24 |
| * kgriffs looks innocent again | 20:25 | |
| vkmc | kgriffs, did you use it with ants? | 20:25 |
| vkmc | lol | 20:25 |
| kgriffs | who, me? ants? | 20:25 |
| * kgriffs continues looking innocent | 20:25 | |
| vkmc | aahahaha | 20:25 |
| kgriffs | ok, brb for realz | 20:25 |
| flaper87 | kgriffs: http://i.ytimg.com/vi/xVB2NDPxKqA/hqdefault.jpg | 20:26 |
| flaper87 | that's you in fourth grade | 20:26 |
| flaper87 | seriously, that kid looks a lot like me | 20:26 |
| * flaper87 admits he is pyromaniac | 20:26 | |
| vkmc | lol | 20:27 |
| openstackgerrit | A change was merged to openstack/python-zaqarclient: Updates API name https://review.openstack.org/122856 | 20:29 |
| flaper87 | vkmc: 'sup with my patch? | 20:29 |
| kgriffs | ROFL | 20:29 |
| flaper87 | I didn't have a flamethrower but hell I wanted one | 20:30 |
| flaper87 | I had to build my own | 20:30 |
| vkmc | flaper87, I'm locally running it | 20:30 |
| sriram | ha, this channel is always fun, just popped in for a bit ;) | 20:32 |
| vkmc | flaper87, did you build your own flamethrower? good thing we are friends :o | 20:32 |
| vkmc | hey sriram! | 20:32 |
| sriram | hey there vkmc! :) | 20:32 |
| sriram | kgriffs: whoa those numbers! :D thats awesome. | 20:34 |
| flaper87 | vkmc: I did | 20:34 |
| sriram | flaper87: a flamethrower, wow | 20:34 |
| flaper87 | I was always careful, I used to play inside the house to avoid burning our garden | 20:34 |
| flaper87 | :D | 20:34 |
| vkmc | flaper87, that is common sense :p | 20:34 |
| * sriram runs away really really fast | 20:34 | |
| flaper87 | jokes apart, I did build my on flamethrower | 20:35 |
| flaper87 | ahh, goooood old times | 20:35 |
| vkmc | haha I used to cook a lot | 20:36 |
| vkmc | now it has been a while | 20:36 |
| *** sriram has quit IRC | 20:38 | |
| vkmc | flaper87, reviewed | 20:38 |
| flaper87 | vkmc: after having seen breaking bad, cooking stopped being what it was | 20:38 |
| flaper87 | so be more specific | 20:39 |
| vkmc | flaper87, lol | 20:39 |
| vkmc | you are right | 20:40 |
| vkmc | so... bakery fits in this case | 20:40 |
| *** sgotliv_ has quit IRC | 20:42 | |
| openstackgerrit | Sphoorti proposed a change to openstack/zaqar: Replace misleading max_message_size option name https://review.openstack.org/123223 | 20:44 |
| openstackgerrit | Sphoorti proposed a change to openstack/zaqar: Replace misleading max_message_size option name https://review.openstack.org/123223 | 20:46 |
| sphoorti | vkmc: flaper87 could you review this ? | 20:48 |
| vkmc | I don't know sphoorti | 20:50 |
| sphoorti | okay :) | 20:51 |
| vkmc | haha | 20:51 |
| vkmc | I'll check it out soon | 20:51 |
| sphoorti | sure | 20:52 |
| sphoorti | I ll go to bed now! | 20:52 |
| vkmc | ok :) sweet dreams! o/ | 20:52 |
| sphoorti | Adios :) | 20:52 |
| sphoorti | yayyiiee :D | 20:53 |
| * vkmc dunno to say bye in marathi | 20:53 | |
| vkmc | :( | 20:53 |
| sphoorti | tata is bye in marathi | 20:53 |
| vkmc | :D tata | 20:53 |
| sphoorti | hehehehe | 20:53 |
| *** sphoorti has quit IRC | 20:54 | |
| *** mpanetta has quit IRC | 20:55 | |
| *** jogo has joined #openstack-zaqar | 21:02 | |
| jogo | what is Zaqar's HA story? | 21:02 |
| *** jchai has quit IRC | 21:02 | |
| boris-42 | jogo heh does other projects have HA story? | 21:03 |
| boris-42 | do* | 21:03 |
| boris-42 | vkmc so are you ready to write first benchmark?) | 21:03 |
| vkmc | boris-42, of course :) | 21:06 |
| boris-42 | vkmc btw how to get zaqar installed via devastack? | 21:06 |
| *** kgriffs is now known as kgriffs|afk | 21:06 | |
| boris-42 | vkmc just enable+=zaqar? | 21:06 |
| vkmc | boris-42, exactly yeah... you have to add the line 'enable zaqar-server' in local.conf | 21:07 |
| boris-42 | vkmc ok will do | 21:08 |
| vkmc | 'enable-service zaqar-server' | 21:08 |
| boris-42 | vkmc trying | 21:09 |
| vkmc | boris-42, kewl | 21:09 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/zaqar: Move Redis driver's claim transaction to Lua https://review.openstack.org/123230 | 21:12 |
| *** flaper87 is now known as flaper87|afk | 21:13 | |
| boris-42 | vkmc hm no new records in keystone catalog | 21:17 |
| boris-42 | and service-list | 21:17 |
| vkmc | boris-42, hm a new patchset merged earlier today | 21:23 |
| vkmc | boris-42, it may be because of that | 21:23 |
| boris-42 | vkmc maybe, I'll try on fresh VM | 21:23 |
| boris-42 | vkmc to repeat a bit later | 21:23 |
| *** AAzza has left #openstack-zaqar | 21:25 | |
| vkmc | boris-42, sure thing | 21:25 |
| vkmc | boris-42, let me know | 21:25 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/zaqar: Move Redis driver's claim transaction to Lua https://review.openstack.org/123230 | 21:32 |
| vkmc | sorry guys, the change in the client broke Tempest :/ | 21:46 |
| vkmc | https://review.openstack.org/#/c/123252/2 | 21:46 |
| vkmc | hope they review it soon | 21:46 |
| *** earnThis has quit IRC | 22:15 | |
| jogo | vkmc: you broke tempest? | 22:24 |
| vkmc | :x | 22:25 |
| vkmc | I guess it's a way of saying so | 22:26 |
| jogo | vkmc: can you expand on that, you didn't break it for everyone did you? | 22:26 |
| vkmc | jogo, of course not | 22:27 |
| jogo | vkmc: you startled me for a second | 22:27 |
| vkmc | jogo, a change in Zaqar client is making the integration testing for Zaqar fail | 22:27 |
| vkmc | haha | 22:27 |
| vkmc | jogo, you wouldn't believe I'm that powerful :p | 22:28 |
| jogo | vkmc: just the zaqar testing not the standard tempest devstack gate | 22:29 |
| jogo | good | 22:29 |
| vkmc | yeah... not good, less worse I guess | 22:30 |
| jogo | heh yeah | 22:31 |
| *** reed has quit IRC | 22:56 | |
| *** reed has joined #openstack-zaqar | 22:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
