*** alcabrera|afk has quit IRC | 00:07 | |
*** peopleme1ge has quit IRC | 00:07 | |
*** kgriffs is now known as kgriffs|afk | 00:07 | |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/zaqar: Adds partial field deprecation note in the users guide https://review.openstack.org/117989 | 00:07 |
---|---|---|
*** openstack has joined #openstack-zaqar | 00:09 | |
*** AAzza has quit IRC | 00:11 | |
*** AAzza_afk is now known as AAzza | 00:11 | |
*** dmitryme has quit IRC | 00:11 | |
*** dmitryme2 is now known as dmitryme | 00:11 | |
*** flwang1 has quit IRC | 00:30 | |
openstackgerrit | A change was merged to openstack/zaqar: Enhance docstrings for the config reference https://review.openstack.org/117826 | 00:31 |
*** amitgandhinz has joined #openstack-zaqar | 00:44 | |
*** amitgandhinz has quit IRC | 00:49 | |
*** amitgandhinz has joined #openstack-zaqar | 00:56 | |
*** kgriffs is now known as kgriffs|afk | 00:58 | |
*** echevemaster has joined #openstack-zaqar | 00:58 | |
*** echevemaster has quit IRC | 00:58 | |
*** echevemaster has joined #openstack-zaqar | 00:58 | |
*** amitgandhinz has quit IRC | 01:01 | |
*** pfwang has quit IRC | 01:13 | |
*** wpf has joined #openstack-zaqar | 01:14 | |
*** kgriffs|afk is now known as kgriffs | 01:49 | |
*** amitgandhinz has joined #openstack-zaqar | 01:57 | |
*** kgriffs is now known as kgriffs|afk | 01:58 | |
*** amitgandhinz has quit IRC | 02:02 | |
*** jeffrey4l has joined #openstack-zaqar | 02:25 | |
*** jeffrey4l has quit IRC | 02:30 | |
wpf | flwang: ping | 02:34 |
flwang | wpf: pong | 02:35 |
wpf | flwang: are you familiar with the functional test? I have a question about my patch | 02:35 |
flwang | wpf: I can try | 02:36 |
wpf | all our functional tests are running in the "mock" mode , right? | 02:36 |
flwang | wpf: I don't think so | 02:37 |
flwang | IIRC, some parts of response are mocked, but not the whole functional test | 02:39 |
wpf | ok, I will check it again | 02:39 |
flwang | wpf: you can refer the code under /zaqar/tests/functional | 02:40 |
wpf | thanks | 02:41 |
*** jeffrey4l has joined #openstack-zaqar | 02:43 | |
*** kgriffs|afk is now known as kgriffs | 02:50 | |
*** amitgandhinz has joined #openstack-zaqar | 02:58 | |
*** kgriffs is now known as kgriffs|afk | 02:59 | |
*** amitgandhinz has quit IRC | 03:02 | |
*** kgriffs|afk is now known as kgriffs | 03:50 | |
*** amitgandhinz has joined #openstack-zaqar | 03:59 | |
*** kgriffs is now known as kgriffs|afk | 04:00 | |
*** amitgandhinz has quit IRC | 04:03 | |
*** fifieldt_ has quit IRC | 04:08 | |
*** chandankumar has joined #openstack-zaqar | 04:32 | |
*** kgriffs|afk is now known as kgriffs | 04:51 | |
*** chandankumar has quit IRC | 04:53 | |
*** amitgandhinz has joined #openstack-zaqar | 05:00 | |
*** kgriffs is now known as kgriffs|afk | 05:01 | |
*** chandankumar has joined #openstack-zaqar | 05:01 | |
*** amitgandhinz has quit IRC | 05:04 | |
*** chandankumar has quit IRC | 05:09 | |
openstackgerrit | Peng Fei Wang proposed a change to openstack/zaqar: Not permitted to delete pools which are used by flavor https://review.openstack.org/117706 | 05:09 |
*** chandankumar has joined #openstack-zaqar | 05:14 | |
*** kgriffs|afk is now known as kgriffs | 05:35 | |
*** chandankumar has quit IRC | 05:40 | |
*** kgriffs is now known as kgriffs|afk | 05:45 | |
openstackgerrit | Peng Fei Wang proposed a change to openstack/zaqar: Correct the wrong indent for succeeded_ids https://review.openstack.org/118080 | 05:46 |
*** amitgandhinz has joined #openstack-zaqar | 06:00 | |
*** amitgandhinz has quit IRC | 06:05 | |
*** echevemaster has quit IRC | 06:12 | |
*** flaper87|afk is now known as flaper87 | 06:14 | |
*** chandankumar has joined #openstack-zaqar | 06:19 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/zaqar: Imported Translations from Transifex https://review.openstack.org/118011 | 06:21 |
*** chandankumar has quit IRC | 06:38 | |
*** chandankumar has joined #openstack-zaqar | 06:43 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/zaqar: Raise proper HTTP error when flavor doesn't exist https://review.openstack.org/117760 | 06:46 |
openstackgerrit | Flavio Percoco proposed a change to openstack/zaqar: Ensure `pool` exists before creating a flavor https://review.openstack.org/117773 | 06:46 |
*** chandankumar has quit IRC | 06:54 | |
*** chandankumar has joined #openstack-zaqar | 06:54 | |
openstackgerrit | Peng Fei Wang proposed a change to openstack/zaqar: Fix the flavor tests failing under py33 https://review.openstack.org/118094 | 06:57 |
*** amitgandhinz has joined #openstack-zaqar | 07:01 | |
*** amitgandhinz has quit IRC | 07:06 | |
openstackgerrit | Peng Fei Wang proposed a change to openstack/zaqar: Not permitted to delete pools which are used by flavor https://review.openstack.org/117706 | 07:10 |
wpf | flaper87: ping | 07:12 |
flaper87 | wpf: pong | 07:12 |
wpf | would you please check it again for Not permitted to delete pools which are used by flavor https://review.openstack.org/117706 when you are free ? | 07:12 |
flaper87 | wpf: I already did :P | 07:13 |
flaper87 | wpf: the patch looks good | 07:13 |
flaper87 | I do want to discuss whether we should do those checks in the storage side or the API side | 07:14 |
flaper87 | I've added it as an topic for today's meeting | 07:14 |
flaper87 | I know sqla doesn't implement flavors yet | 07:14 |
wpf | Ok, that's fine | 07:14 |
flaper87 | but there's another patch I wrote that does exactly the same as yours. I mean, it adds a check in the storage side | 07:14 |
flaper87 | and I'd like us to have some rules before this 2 patches land | 07:15 |
flaper87 | wpf: this means I'm blocking my own patch | 07:15 |
flaper87 | What's wrong with me? | 07:15 |
flaper87 | brb | 07:15 |
wpf | yes, I saw that you check the pool existing when creating a flavor | 07:15 |
wpf | -:) | 07:17 |
*** chandankumar has quit IRC | 07:28 | |
openstackgerrit | A change was merged to openstack/zaqar: Imported Translations from Transifex https://review.openstack.org/118011 | 07:39 |
*** chandankumar has joined #openstack-zaqar | 07:41 | |
*** amitgandhinz has joined #openstack-zaqar | 08:02 | |
*** amitgandhinz has quit IRC | 08:06 | |
*** chandankumar has quit IRC | 09:02 | |
*** amitgandhinz has joined #openstack-zaqar | 09:03 | |
*** dguerri`afk is now known as dguerri | 09:04 | |
*** amitgandhinz has quit IRC | 09:07 | |
*** flwang1 has joined #openstack-zaqar | 09:35 | |
*** jeffrey4l has quit IRC | 10:01 | |
*** amitgandhinz has joined #openstack-zaqar | 10:03 | |
*** amitgandhinz has quit IRC | 10:08 | |
*** jeffrey4l has joined #openstack-zaqar | 10:13 | |
*** flaper87 is now known as flaper87|afk | 10:25 | |
*** flwang1 has quit IRC | 10:34 | |
*** flwang1 has joined #openstack-zaqar | 10:35 | |
*** flaper87 has joined #openstack-zaqar | 10:37 | |
*** fesp has joined #openstack-zaqar | 10:41 | |
*** flaper87 has quit IRC | 10:42 | |
*** amitgandhinz has joined #openstack-zaqar | 11:04 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/zaqar: Ensure `pool` exists before creating a flavor https://review.openstack.org/117773 | 11:07 |
fesp | flwang: addressed your comments :) | 11:07 |
fesp | thanks for the review | 11:07 |
*** fesp has quit IRC | 11:08 | |
*** flaper87 has joined #openstack-zaqar | 11:08 | |
*** amitgandhinz has quit IRC | 11:09 | |
flwang1 | who is the fesp? I don't know the guy :) | 11:10 |
flwang1 | flaper87: may I get your comments about this https://review.openstack.org/#/c/117773/2/zaqar/queues/storage/mongodb/flavors.py ? | 11:11 |
flaper87 | ah, I commented and forgot to submit | 11:13 |
flaper87 | sent | 11:14 |
flaper87 | flwang1: TL;DR we don't use project for pools | 11:14 |
flwang1 | flaper87: ah, yep, sorry, I forgot that | 11:15 |
flaper87 | flwang1: could you please review this one too? https://review.openstack.org/#/c/117760/ | 11:16 |
flaper87 | the one you reviewed depends on this | 11:16 |
flwang1 | flaper87: i'm doing that, sir | 11:17 |
flaper87 | flwang1: awesome, thanks a lot | 11:17 |
flwang1 | flaper87: done | 11:25 |
*** chandankumar has joined #openstack-zaqar | 11:26 | |
flwang1 | flaper87: ping | 11:28 |
flwang1 | flaper87: for line 140, https://review.openstack.org/#/c/117773/3/zaqar/queues/transport/wsgi/v1_1/flavors.py should we use _LE ? I'm trying to figure out the difference, but I think it would be more efficient to ask you :) | 11:29 |
flaper87 | flwang1: We should but I'm working on a patch that replaces all those cases | 11:32 |
flaper87 | this patch can land post rc1 | 11:32 |
*** vkmc has joined #openstack-zaqar | 11:36 | |
*** vkmc has quit IRC | 11:36 | |
*** vkmc has joined #openstack-zaqar | 11:36 | |
vkmc | morniiiiiing | 11:37 |
*** chandankumar has quit IRC | 11:39 | |
flwang1 | vkmc: morning :) | 11:40 |
flwang1 | flaper87: got it' | 11:41 |
flaper87 | vkmc: goooooooooooooooooooooooooooooood morning girl | 11:41 |
vkmc | hiii flwang1, flaper87 :) | 11:42 |
flaper87 | vkmc: https://review.openstack.org/#/c/117760/ :D :D :D | 11:48 |
vkmc | flaper87, A+! | 11:50 |
*** flaper87 has quit IRC | 11:52 | |
*** flaper87|afk is now known as flaper87 | 11:53 | |
*** amitgandhinz has joined #openstack-zaqar | 12:05 | |
*** amitgandhinz has quit IRC | 12:09 | |
flaper87 | flwang1: https://review.openstack.org/#/ before you go to sleep | 12:11 |
flaper87 | vkmc: ^ | 12:11 |
flaper87 | vkmc: well, you just woke up so, just click on the link | 12:11 |
flaper87 | :P | 12:11 |
flwang1 | flaper87: https://review.openstack.org/#/c/117773/ | 12:14 |
vkmc | flaper87, did you forget the review number or you want us to review all? haha | 12:14 |
flwang1 | see the comments :) | 12:14 |
flaper87 | flwang1: ah yeah, I meant that one | 12:14 |
flaper87 | :P | 12:14 |
vkmc | loooool | 12:14 |
flaper87 | flwang1: I LOVE THAT ASCIIIIII | 12:14 |
* flaper87 ROFL | 12:15 | |
vkmc | reviews with style! haha | 12:15 |
flaper87 | we should use that more often | 12:15 |
* flaper87 looks up for a good ascii website/tool | 12:15 | |
flwang1 | especially to have more fun when we see a -1 :D | 12:16 |
* flaper87 wants one saying HOLY MOLY | 12:18 | |
flwang1 | flaper87: haha, yep | 12:18 |
flwang1 | I have to drop | 12:18 |
flwang1 | have fun, guys | 12:18 |
flwang1 | have a nice day, team | 12:19 |
vkmc | flwang1, o/ nite! | 12:20 |
flaper87 | flwang1: take care | 12:21 |
flaper87 | vkmc: your vote is missing https://review.openstack.org/#/c/117773/ :P | 12:21 |
flaper87 | just trying to get rid of those so we can *just* focus on docs and redis today | 12:22 |
flaper87 | vkmc: did you see kgriffs|afk comments on your docs patch? | 12:22 |
vkmc | flaper87, about 117773, why you removed the exception handling TODO in storage/pooling.py? | 12:27 |
vkmc | flaper87, I did see them, I'm regenerating the tables and updating those comments :) | 12:27 |
vkmc | the doc-tool takes like 10 minutes to update everything -.- | 12:28 |
flaper87 | vkmc: re 117773, because it's no longer valid. This patch is suppose to fix that | 12:33 |
flaper87 | NotFound will only be raised when the pool is being registered through the pool API | 12:33 |
vkmc | flaper87, true that, it's being handled in transport | 12:34 |
flaper87 | vkmc: right | 12:34 |
vkmc | sooorry | 12:34 |
flaper87 | vkmc: no, you're not!!!! | 12:35 |
flaper87 | :D | 12:35 |
vkmc | haha :) | 12:36 |
vkmc | flaper87, https://launchpad.net/bugs/1348453 should we enable those tests? | 12:41 |
flaper87 | vkmc: YES! I was working on this a month ago or so and when I enabled those tests a whole bunch of other issues raised. That scared me and I ran away screaming like crazy and then jumped into the lake. | 12:42 |
flaper87 | it was horrible | 12:42 |
flaper87 | I should pick this up again | 12:43 |
vkmc | oh noes D: | 12:43 |
flaper87 | this can land post-rc1 so, lets focus on the things we need before Sept 4th | 12:43 |
vkmc | all right | 12:43 |
vkmc | I was asking you because pools functional tests for v1 are in the same place | 12:43 |
vkmc | and I was working on that this weekend | 12:44 |
flaper87 | yeah, TBH, I don't remember why they were disabled | 12:44 |
vkmc | I'll dig a bit more on that today | 12:45 |
flaper87 | vkmc: I'd recommend leaving it for next week or even Kilo. | 12:46 |
vkmc | flaper87, roger that | 12:46 |
flaper87 | fuck tests | 12:46 |
flaper87 | wait, I didn't say that | 12:46 |
flaper87 | who wrote that? | 12:46 |
* flaper87 looks around | 12:46 | |
vkmc | lol | 12:47 |
vkmc | so... should we target this one to k1 too? https://bugs.launchpad.net/zaqar/+bug/1246757 | 12:48 |
flaper87 | vkmc: moved to rc1 for now | 12:50 |
vkmc | flaper87, thanks | 12:51 |
*** amitgandhinz has joined #openstack-zaqar | 13:06 | |
*** kgriffs|afk is now known as kgriffs | 13:07 | |
kgriffs | o/ | 13:09 |
vkmc | kgriffs, \o | 13:10 |
*** amitgandhinz has quit IRC | 13:10 | |
openstackgerrit | A change was merged to openstack/zaqar: Fix the flavor tests failing under py33 https://review.openstack.org/118094 | 13:21 |
flaper87 | kgriffs: o/ | 13:23 |
flaper87 | kgriffs: I left a tiny comment in the redis patch | 13:23 |
* kgriffs looks | 13:23 | |
flaper87 | Was I too paranoid? | 13:23 |
kgriffs | no, comments were helpful | 13:24 |
kgriffs | I'm starting work on expiring all the things now | 13:24 |
flaper87 | kgriffs: ahhh you already updated it | 13:25 |
flaper87 | damn, lemme re-review it | 13:25 |
kgriffs | yessir | 13:25 |
vkmc | kgriffs, I reviewed it during the weekend but I couldn't find anything else besides flaper87 comments | 13:26 |
vkmc | I'll review it again today | 13:26 |
kgriffs | cool, thanks | 13:26 |
kgriffs | btw, my observer patch to benchmark should fix the psutils doc error in vkmc's patch, since it removed psutils | 13:26 |
vkmc | just one question, is the Lua script somewhere? | 13:26 |
vkmc | yup :) | 13:27 |
kgriffs | vkmc: there are no lua scripts for now. I started writing one but decided for the sake of time to go with pip.watch / transactions as prashanth did for posting messages. | 13:27 |
vkmc | kgriffs, cool | 13:27 |
kgriffs | I'd like to revisit later and see if doing things in lua would be better/faster | 13:27 |
kgriffs | see also the note at the bottom of this page: http://redis.io/topics/transactions | 13:28 |
flaper87 | I'll go ahead and +2 this patch. I think it's good enough to begin with | 13:29 |
flaper87 | +2'd | 13:31 |
kgriffs | cool. As long as it doesn't have logic errors, I think it is OK to land. We can do optimizations, DRY cleanup, etc. in other patches as needed | 13:33 |
flaper87 | kgriffs: fully agree | 13:36 |
flaper87 | logic errors and/or weird races | 13:36 |
kgriffs | flaper87: what was the oldest version of RHEL that Rackspace supports? | 13:41 |
kgriffs | and... what version of Redis is supported on that | 13:41 |
flaper87 | kgriffs: good question. I think RAX is on 6.X | 13:42 |
flaper87 | Let me check what version of redis we have there | 13:42 |
* kgriffs is wondering if he's allowed to use SCAN which was introduced in 2.8 | 13:42 | |
flaper87 | kgriffs: I see 2.6, | 13:44 |
flaper87 | I'll double check anyway | 13:44 |
flaper87 | kk, gtg. bbl | 13:45 |
kgriffs | ttfn | 13:47 |
*** openstack has joined #openstack-zaqar | 13:57 | |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/zaqar: Zaqar configuration reference docs https://review.openstack.org/114530 | 13:57 |
*** openstackstatus has joined #openstack-zaqar | 13:58 | |
*** ChanServ sets mode: +v openstackstatus | 13:58 | |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/zaqar: Adds partial field deprecation note in the users guide https://review.openstack.org/117989 | 14:03 |
*** amitgandhinz has joined #openstack-zaqar | 14:06 | |
*** amitgandhinz has quit IRC | 14:11 | |
openstackgerrit | A change was merged to openstack/zaqar: Raise proper HTTP error when flavor doesn't exist https://review.openstack.org/117760 | 14:14 |
openstackgerrit | A change was merged to openstack/zaqar: Ensure `pool` exists before creating a flavor https://review.openstack.org/117773 | 14:17 |
vkmc | wow. much merge! | 14:18 |
kgriffs | seet | 14:31 |
kgriffs | sweet | 14:31 |
kgriffs | vkmc: btw, do you happen to know off the top of your head if the installation/deployment guide recommends installing ntpd ? | 14:31 |
vkmc | right now we don't have a installation/deployment guide for production | 14:37 |
vkmc | some docs on that topic were in the developers guide and we removed them to add to the users guide | 14:38 |
vkmc | precisely, we have to create another guide, the installation guide | 14:39 |
kgriffs | oic | 14:39 |
vkmc | something to work on k-1 I think | 14:40 |
kgriffs | well, once we add those we should recommend installing ntpd on the web boxes and DB servers since we use a lot of timestamps | 14:40 |
kgriffs | vkmc: thanks for the info! | 14:40 |
vkmc | no problem | 14:40 |
vkmc | we could file a bp and add those comments so we don't forget | 14:41 |
vkmc | I'll do that | 14:41 |
kgriffs | good idea, thx | 14:41 |
flaper87 | o/ | 14:41 |
flaper87 | back for 1 more hour | 14:42 |
flaper87 | :D | 14:42 |
kgriffs | quick, everybody hide! | 14:42 |
* flaper87 hides | 14:42 | |
* vkmc hides | 14:42 | |
kgriffs | lol | 14:42 |
flaper87 | I seriously didn't know one could laugh that much just by chatting on IRC | 14:43 |
vkmc | (if we hold our breaths he may not notice we are here) | 14:43 |
* kgriffs holds breath | 14:43 | |
* flaper87 holds his breath | 14:43 | |
flaper87 | Who are we hiding from ? | 14:43 |
kgriffs | I was hiding from flaper87. | 14:43 |
vkmc | me too ^ | 14:44 |
flaper87 | oh me too. Glad we're on the same page | 14:44 |
* kgriffs wasn't doin' nuttin' wrong. nothin' ta see here. move along. | 14:44 | |
vkmc | lol | 14:44 |
* kgriffs has a guilty look on his face. hopes flaper87 doesn't notice. | 14:45 | |
flaper87 | wwwwwwwwaaaaaaaaaaait a minute... Why does kgriffs have that guilty look in his face? | 14:45 |
vkmc | :| | 14:45 |
* kgriffs oh oh | 14:46 | |
* kgriffs carefully puts code back the way it was | 14:46 | |
* vkmc hopes flaper87 don't reads the backlog | 14:50 | |
vkmc | doesn't read* | 14:50 |
* vkmc keeps flaper87 distracted with her grammar disasters | 14:52 | |
flaper87 | mmhh, something smells bad in here | 14:52 |
flaper87 | I think I'll read the backlog | 14:52 |
flaper87 | oh look, there's a typo | 14:52 |
flaper87 | wait, what was I doing? | 14:53 |
flaper87 | mmh, damn! Ok, I'll keep playing with code | 14:53 |
* kgriffs high-fives vkmc and sneaks away while flaper87 is distracted. | 14:54 | |
* vkmc high-fives kgriffs back and sneaks away too | 14:54 | |
*** amitgandhinz has joined #openstack-zaqar | 15:07 | |
*** amitgandhinz has quit IRC | 15:12 | |
flaper87 | brb again | 15:14 |
*** prashanthr_ has joined #openstack-zaqar | 15:27 | |
prashanthr_ | Hello Good morning :) | 15:28 |
kgriffs | o/ | 15:28 |
vkmc | hi prashanthr_! | 15:29 |
prashanthr_ | hi vkmc and kgriffs :) | 15:49 |
prashanthr_ | how are you ? | 15:49 |
vkmc | fine and you prashanthr_? | 15:51 |
vkmc | back home? | 15:51 |
prashanthr_ | Yes me back home now | 15:52 |
prashanthr_ | Doing good as well | 15:52 |
prashanthr_ | just setup a new internet connection as well | 15:52 |
vkmc | \o/ | 15:53 |
*** amitgandhinz has joined #openstack-zaqar | 15:56 | |
*** prashanthr_1 has joined #openstack-zaqar | 16:00 | |
*** prashanthr_ has quit IRC | 16:00 | |
*** amitgandhinz has quit IRC | 16:00 | |
*** prashanthr_1 has quit IRC | 16:03 | |
*** prashanthr_ has joined #openstack-zaqar | 16:03 | |
prashanthr_ | kgriffs: thank you for the help on the patch | 16:07 |
prashanthr_ | Sorry for the delay in me getting back to zaqar | 16:07 |
prashanthr_ | I am free now and can restart with any help | 16:07 |
kgriffs | sure thing. I hope you don't mind. We are just under a time crunch and I was worried it wouldn't get done before j-3 closes. | 16:07 |
prashanthr_ | not a bit :) | 16:07 |
prashanthr_ | Ready to start contributing again :) | 16:08 |
kgriffs | prashanthr_: I think this patch will probably be approved today | 16:08 |
kgriffs | https://review.openstack.org/#/c/97178/ | 16:08 |
kgriffs | I am working on a followup patch to add expiration of claims and messages | 16:08 |
*** prashanthr_ has quit IRC | 16:08 | |
kgriffs | otherwise, there are a few misc. work items that I listed under the "work item" section on the bp | 16:09 |
kgriffs | https://blueprints.launchpad.net/zaqar/+spec/redis-storage-driver | 16:09 |
*** prashanthr_ has joined #openstack-zaqar | 16:09 | |
*** prashanthr_ has quit IRC | 16:10 | |
kgriffs | prashanthr_: I'm trying to find out if we can use Lua for stuff after all... depends on what distros OpenStack officially supports, and what their default Redis versions are | 16:10 |
*** prashanthr_1 has joined #openstack-zaqar | 16:10 | |
prashanthr_1 | kgriffs: When is the basic driver expected to be checked in ? | 16:11 |
kgriffs | also, I have started a list to brainstorm stuff we want to look at in kilo | 16:11 |
kgriffs | https://etherpad.openstack.org/p/zaqar-scratch | 16:11 |
kgriffs | prashanthr_1: as soon as vkmc and/or flwang get around to reviewing it (hint, hint) | 16:11 |
prashanthr_1 | Sure :) | 16:11 |
*** prashanthr_1 has quit IRC | 16:17 | |
*** flaper87 is now known as flaper87|afk | 17:14 | |
*** flaper87 has joined #openstack-zaqar | 17:24 | |
*** fesp has joined #openstack-zaqar | 17:36 | |
*** flaper87 has quit IRC | 17:37 | |
*** amitgandhinz has joined #openstack-zaqar | 17:56 | |
*** amitgandhinz has quit IRC | 18:01 | |
vkmc | flaper87|afk, fesp around? | 18:49 |
*** amitgandhinz has joined #openstack-zaqar | 18:57 | |
*** amitgandhinz has quit IRC | 19:02 | |
vkmc | kgriffs, maybe? | 19:02 |
kgriffs | yeah | 19:03 |
kgriffs | I should be on vacation (holiday today in the US) but... I'm worried about getting redis done in time | 19:03 |
vkmc | ohh, go ahead! | 19:03 |
vkmc | I'm one module away from finishing the review, I haven't find anything odd for now | 19:04 |
kgriffs | cool | 19:04 |
vkmc | still we need other 2 +2's | 19:04 |
kgriffs | I guess I am still allowed to +2 since most of the code was prashanths | 19:05 |
* kgriffs does that | 19:05 | |
vkmc | I think you are yeah :) | 19:05 |
vkmc | on other comment... I not sure why the py33 gate is failing for the docs patch | 19:06 |
vkmc | any idea? | 19:06 |
kgriffs | looking | 19:06 |
kgriffs | hmmm, sphinx errors | 19:08 |
kgriffs | seems like sphinx has to be py33 compatible, though | 19:09 |
kgriffs | something to do with an OpenStack-specific extension | 19:09 |
* kgriffs admits that is a shot in the dark | 19:09 | |
* fesp around now | 19:09 | |
vkmc | there is a python3-sphinx package for sphinx | 19:09 |
vkmc | I wonder if we need that dependency? | 19:10 |
fesp | Anything I should review right now? | 19:13 |
fesp | vkmc: do you like violetta ? | 19:13 |
fesp | I'm about to say something and I don't want to hurt anyone's feelings | 19:13 |
fesp | :P | 19:13 |
vkmc | who is Violetta? | 19:13 |
fesp | vkmc: that's the answer I was looking for | 19:14 |
vkmc | the Argentinian kids singer? from Disney? | 19:14 |
vkmc | lol | 19:14 |
fesp | vkmc: yeah, that one! | 19:14 |
fesp | I just heard her singing "Imagine", I'll never be able to die in peace anymore | 19:15 |
fesp | Damnit! | 19:15 |
fesp | vkmc: btw, if you've some time, could you please review this? https://review.openstack.org/#/c/97178/ | 19:15 |
fesp | I'd like that to be merged before tomorrow's review | 19:15 |
vkmc | aaahaha why did you heard that? | 19:15 |
vkmc | fesp, I'm finishing the review for that one :) | 19:16 |
fesp | vkmc: awesome | 19:16 |
vkmc | fesp, can you review https://review.openstack.org/#/c/116384/? | 19:16 |
fesp | vkmc: I've no idea, it was on the tv, I didn't know she was going to sing that and I didn't have the remote control | 19:16 |
fesp | I know what dying feels like now | 19:16 |
* fesp reviews | 19:16 | |
vkmc | fesp, and https://review.openstack.org/#/c/117894/? | 19:16 |
vkmc | and... | 19:16 |
fesp | The first one is for free, the second one 1k$ the third one | 19:17 |
fesp | lemme think | 19:17 |
fesp | vkmc: were those jenkins/gerrit/launchpad docs copied from other projects? | 19:18 |
* vkmc waits for the price of the third review | 19:18 | |
fesp | I'm asking to know whether there should be some concistency with other projects | 19:18 |
vkmc | fesp, yeah, those are like Nova and Keystone docs | 19:19 |
fesp | cool | 19:20 |
fesp | done, what's the next one? | 19:20 |
fesp | Oh you want the price | 19:20 |
fesp | mmmhhh | 19:20 |
fesp | mhhhh | 19:20 |
fesp | I'm feeling happy today, so it'll be 10k$ | 19:20 |
vkmc | darn you are expensive | 19:21 |
fesp | what? | 19:22 |
vkmc | well it was this one https://review.openstack.org/#/c/114530/ | 19:22 |
fesp | I said that's a special price because I'm happy | 19:22 |
fesp | what's up with jenkins? | 19:22 |
vkmc | but I still have to debug what is going on with the py33 gate | 19:22 |
vkmc | ah well, with more reason... if it's a special price I cannot imagine the real one :p | 19:23 |
fesp | Touché | 19:23 |
vkmc | any idea on why the py33 is failing? | 19:26 |
vkmc | we were chatting with kgriffs that it may be because sphinx is not py33 compatible, but it's odd because all other patches passed | 19:26 |
fesp | plus, this is the user guide | 19:28 |
fesp | it's not actually touching sphinx | 19:28 |
fesp | mmh | 19:29 |
openstackgerrit | A change was merged to openstack/zaqar: Add observer role to benchmark tool https://review.openstack.org/116384 | 19:41 |
openstackgerrit | A change was merged to openstack/zaqar: Adds more resources section to to developers guide https://review.openstack.org/117894 | 19:49 |
vkmc | +2 Redis patch | 19:54 |
vkmc | https://gist.github.com/vkmc/cd2d95626ee2643f509a | 19:54 |
kgriffs | wow. much review. | 19:56 |
kgriffs | so excellent. | 19:56 |
*** amitgandhinz has joined #openstack-zaqar | 19:58 | |
vkmc | such talent. | 19:59 |
*** amitgandhinz has quit IRC | 20:02 | |
*** flwang1 has quit IRC | 20:19 | |
*** kgriffs is now known as kgriffs|afk | 20:44 | |
*** kgriffs|afk is now known as kgriffs | 20:47 | |
fesp | meeting in 10mins | 20:49 |
*** fesp is now known as flaper87 | 20:49 | |
vkmc | aye aye | 20:52 |
*** amitgandhinz has joined #openstack-zaqar | 20:58 | |
flaper87 | kgriffs: meeting | 21:00 |
flaper87 | malini: ^ | 21:00 |
kgriffs | joins | 21:01 |
*** amitgandhinz has quit IRC | 21:03 | |
openstackgerrit | A change was merged to openstack/zaqar: Implements queue, message and claim controllers for Redis https://review.openstack.org/97178 | 21:21 |
vkmc | omg :o ^ | 21:22 |
vkmc | \o/ | 21:22 |
*** kgriffs is now known as kgriffs|afk | 21:39 | |
*** echevemaster has joined #openstack-zaqar | 21:45 | |
flaper87 | w0000000000000000t | 21:47 |
flaper87 | vkmc: minutes :D ? | 22:00 |
* vkmc pikcs the minutes | 22:00 | |
flaper87 | YOU ROCK! | 22:00 |
vkmc | sure thing | 22:00 |
flaper87 | ok, I'm off for the night | 22:00 |
flaper87 | have a great rest of the day everyone | 22:00 |
vkmc | enjoy the rest of your evening :) | 22:00 |
flaper87 | flwang: and you have a great day ;) | 22:00 |
flaper87 | thanks | 22:00 |
* flaper87 will wath The Wire's pilot | 22:01 | |
echevemaster | flaper87: are you busy? | 22:01 |
flaper87 | echevemaster: I was saying bye | 22:01 |
echevemaster | hi vkmc as well | 22:01 |
flaper87 | :P | 22:01 |
vkmc | hi echevemaster! | 22:01 |
flaper87 | echevemaster: what's up ? | 22:01 |
echevemaster | ohhh | 22:01 |
vkmc | echevemaster, news about the package? | 22:01 |
flaper87 | I'll always have time for you | 22:01 |
flaper87 | well, not always | 22:01 |
flaper87 | :P | 22:01 |
echevemaster | hahahahaha | 22:01 |
echevemaster | do you received my message about the python eggs | 22:01 |
echevemaster | ? | 22:01 |
echevemaster | that is the problem I have with the package. | 22:02 |
flaper87 | ah yeah | 22:02 |
vkmc | I didn't | 22:02 |
flaper87 | so, pbr is messing with you | 22:02 |
flaper87 | it's weird that you're having pbr issues with the rpm | 22:02 |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/zaqar: Adds the API reference to the devs guide https://review.openstack.org/116390 | 22:19 |
*** flaper87 has quit IRC | 22:19 | |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/zaqar: Adds python-zaqarclient to the requirements https://review.openstack.org/118223 | 22:34 |
vkmc | bbl! | 22:38 |
*** fifieldt has joined #openstack-zaqar | 23:32 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!