Wednesday, 2014-07-16

« Tuesday, 2014-07-15 Index Thursday, 2014-07-17 »
openstackgerritOpenStack Proposal Bot proposed a change to openstack/marconi: Updated from global requirements  https://review.openstack.org/10650600:07
*** wirehead_ has joined #openstack-marconi00:12
*** jimhoagland has quit IRC00:13
*** oz_akan has joined #openstack-marconi01:12
*** prashanthr_ has joined #openstack-marconi01:20
*** prashanthr_ has left #openstack-marconi01:20
*** prashanthr_ has joined #openstack-marconi01:20
prashanthr_flwang: Good morning :)01:21
flwangprashanthr_: hi01:29
prashanthr_flwang: I just wanted to ask you if you were free to review this https://review.openstack.org/#/c/97178/ sometime today, because i was thinking of having the redis basic driver before j-2.01:30
openstackgerritPrashanth Raghu proposed a change to openstack/marconi: Adding dependencies for redis support in marconi  https://review.openstack.org/9601701:31
flwangprashanthr_: it's on my todo list, thanks for the reminder, I will review it today :D01:31
prashanthr_flwang: thanks a lot :) sry for bugging01:31
flwangno worries01:32
flwangprashanthr_: ping01:38
prashanthr_flwang: pong :)01:38
openstackgerritFei Long Wang proposed a change to openstack/marconi: (WIP)feat(v1.1): Implement new /health endpoint for v1.1  https://review.openstack.org/7655501:39
flwangprashanthr_: as for this, https://review.openstack.org/#/c/96017/01:39
flwangI think it would be nice if you can add a comment to explain it01:39
prashanthr_flwang: Sure makes sense as redis would be the first backend to make a shift away from requirements.txt01:40
flwangso that the other reviewers can understand the background better and it will speed up the review, how do you think?01:40
flwangcool01:40
prashanthr_flwang: The comment should be the commit text. Am i right ?01:41
flwangyep, that's ok.01:41
prashanthr_flwang: Sure. Doing it now.01:42
openstackgerritPrashanth Raghu proposed a change to openstack/marconi: Adding dependencies for redis support in marconi  https://review.openstack.org/9601701:43
*** nosnos has joined #openstack-marconi01:58
*** haomaiw__ has quit IRC02:43
*** haomaiwang has joined #openstack-marconi02:44
*** haomai___ has joined #openstack-marconi03:00
*** haomaiwang has quit IRC03:02
*** vkmc has quit IRC03:04
*** chandankumar has joined #openstack-marconi03:35
*** nosnos has quit IRC03:43
*** prashanthr_ has quit IRC03:43
*** nosnos has joined #openstack-marconi04:32
*** prashanthr_ has joined #openstack-marconi05:06
*** chandankumar has quit IRC05:16
*** rossk has joined #openstack-marconi05:21
*** rossk has quit IRC05:33
*** oz_akan has quit IRC05:50
*** reed has quit IRC05:58
*** chandankumar has joined #openstack-marconi05:58
openstackgerritOpenStack Proposal Bot proposed a change to openstack/marconi: Imported Translations from Transifex  https://review.openstack.org/10694406:06
*** oz_akan has joined #openstack-marconi06:21
*** oz_akan has quit IRC06:26
*** flaper87 has quit IRC06:34
*** flaper87 has joined #openstack-marconi06:34
*** ChanServ sets mode: +o flaper8706:34
*** oz_akan has joined #openstack-marconi07:22
*** oz_akan has quit IRC07:26
*** ChanServ changes topic to "OpenStack Queuing and Notification Service || Smile :D || Meetings every Tuesday @ 15:00 UTC || Wiki: https://wiki.openstack.org/wiki/Marconi || Paste: http://paste.openstack.org/ || Send messages and make some noise :D"07:54
*** oz_akan has joined #openstack-marconi07:56
*** oz_akan has quit IRC08:01
*** oz_akan has joined #openstack-marconi08:57
*** oz_akan has quit IRC09:01
*** prashanthr_ has quit IRC09:04
*** prashanthr_ has joined #openstack-marconi09:05
*** mkoderer has joined #openstack-marconi09:11
*** haomai___ has quit IRC09:40
*** haomaiwang has joined #openstack-marconi09:41
*** prashanthr_ has quit IRC09:43
*** oz_akan has joined #openstack-marconi09:58
*** oz_akan has quit IRC10:02
*** oz_akan has joined #openstack-marconi10:58
*** ykaplan has joined #openstack-marconi11:00
*** oz_akan has quit IRC11:03
*** stannie has joined #openstack-marconi11:55
*** vkmc has joined #openstack-marconi12:02
vkmcmorning!12:02
flaper87vkmc: gooooooooooooood morning12:10
vkmcflaper87, hey fla :)12:13
flaper87vkmc: how are you doing?12:17
vkmcflaper87, all good and you?12:18
flaper87vkmc: all gooooooooooooooood :D12:18
vkmcgood to know... more after you warned us there was a slight chance you lost your mind moving queues to control12:19
flaper87fun thing, I'm about to get back to that. Everything's broken12:20
flaper87but hey, I was hired to break things12:20
flaper87.... I think/hope12:21
vkmcof course!12:21
*** oz_akan has joined #openstack-marconi12:21
vkmc:)12:21
vkmcI'm breaking things on transport12:21
vkmcI cannot figure out what's going on -.-12:21
vkmcabout that12:22
vkmcI'm a bit worried about the GSoC deadline12:23
vkmcI'm supposed to hand a code sample and still there is nothing good enough to hit the code base12:23
vkmcflaper87, ^12:23
flaper87vkmc: mmh, that's a weird and unfortunate request12:24
flaper87vkmc: lets move this convo somewhere else12:25
vkmcflaper87, k12:25
*** mwagner_lap has quit IRC12:30
*** sriram has joined #openstack-marconi12:45
*** nosnos has quit IRC12:49
*** prashanthr_ has joined #openstack-marconi12:49
*** ametts has joined #openstack-marconi12:56
*** Obulpathi has joined #openstack-marconi13:08
*** cath6184 has joined #openstack-marconi13:10
*** mpanetta has joined #openstack-marconi13:14
*** amitgandhi has joined #openstack-marconi13:23
*** oz_akan has quit IRC13:38
*** oz_akan has joined #openstack-marconi13:38
*** mwagner_lap has joined #openstack-marconi13:42
prashanthr_Hello good morning :)13:48
prashanthr_I need some +1's on this https://review.openstack.org/#/c/96017/13:48
*** amalagon has quit IRC13:49
vkmchi prashanthr_!13:49
*** prashanthr_ has quit IRC13:51
*** prashanthr_ has joined #openstack-marconi13:51
prashanthr_vkmc: Hi :)13:51
prashanthr_sriram: thanks for the +1 :)13:52
sriramsure, and good morning all :)13:52
*** AAzza has joined #openstack-marconi14:05
AAzzahi guys14:05
prashanthr_AAzza: Hello!14:09
*** dmakogon_ is now known as denis_makogon14:10
AAzzaprashanthr_: how is things doing there?) i'm missing now everything14:11
prashanthr_AAzza: Things are going good. Redis is getting reviewed now. We had a interesting meeting yest where some API's was being reshaped.14:12
prashanthr_You getting busy with preparations ?14:12
AAzzaprashanthr_: oh, need to view log of the meeting. I'm just get married and now have several days of rest in spain)14:14
*** cpallares has joined #openstack-marconi14:14
*** cath6184 has left #openstack-marconi14:15
prashanthr_AAzza: That's great :). Spain's a wonderful place)14:15
vkmcAAzza, heeeeeeeeey :) congratz!!14:15
* prashanthr_ Passes AAzza a virtual gift :D 14:15
* vkmc throws rice to AAzza 14:16
prashanthr_vkmc: thanks for the review :)14:16
vkmcprashanthr_, np!14:16
AAzzaprashanthr_: vkmc:  thanks)14:16
prashanthr_throwing rice is a tradition ?14:16
vkmcyeah... it's an american tradition14:17
prashanthr_nice)14:17
vkmcit's symbolizes prosperity14:18
vkmcit's a great waste of rice though :o14:18
*** amalagon has joined #openstack-marconi14:20
AAzzaprashanthr_: any progress/problems with python 3 for redis?14:20
prashanthr_vkmc: Hmm hope the birds clear it up)14:21
prashanthr_AAzza: Not yet I want the queue and message controllers into the code14:21
*** Obulpathi has quit IRC14:22
*** Obulpathi has joined #openstack-marconi14:22
AAzzaprashanthr_: by the way. why do you not upload tests with the code for the review? or i'm missing something?14:22
AAzzaprashanthr_: i suppose it might be better if the gate run the tests14:23
prashanthr_AAzza: Yeah I could do that. Just that we had planned the tests to go at the last stage14:24
prashanthr_Anyway it's ready14:24
prashanthr_I could start up with it in the reviews14:24
*** abettadapur has joined #openstack-marconi14:25
AAzzaprashanthr_: aha I see, tests will be the last commit) and when you upload them, we definetely will need to do py3 support)14:26
prashanthr_yes14:26
prashanthr_definitely14:26
prashanthr_meanwhile14:26
prashanthr_I can start up with my local branch14:26
prashanthr_that can help with some debugging right14:26
prashanthr_?14:26
*** tongli has joined #openstack-marconi14:26
prashanthr_I could use your help14:26
*** tonytan4ever has joined #openstack-marconi14:27
sriramflaper87: thanks for the review, left some comments :) will have a patch up shortly.14:27
AAzzayeah, i want to help) maybe I can download your github repo (i suppose it contains the last version with tests?) and run the tests, and if I will see smth that can be fixed right away, i will point you in the review?14:29
*** malini has joined #openstack-marconi14:31
*** cath6184 has joined #openstack-marconi14:38
prashanthr_AAzza: Sure. That would be great.14:39
openstackgerritSriram Madapusi Vasudevan proposed a change to openstack/marconi: feat(benchmarking) : Producer-Consumer scenario  https://review.openstack.org/9887514:39
prashanthr_AAzza: It does not contain the latest version14:39
prashanthr_will do that in 10 minutes14:39
sriramflaper87: new patch is out.14:40
*** tonytan4ever has quit IRC14:41
*** cath6184 has quit IRC14:42
*** cath6184_ has joined #openstack-marconi14:42
*** ametts has quit IRC14:46
*** pquerna has quit IRC14:46
*** pquerna has joined #openstack-marconi14:47
prashanthr_vkmc: the private repo is now synced14:52
prashanthr_https://github.com/PrashanthRaghu/marconi-redis14:52
*** AAzza has quit IRC14:52
*** cath6184_ has quit IRC14:54
*** cath6184 has joined #openstack-marconi14:55
prashanthr_malini: Thanks for the workflow approval. Good morning :)14:55
maliniGood Evening prashanthr_!14:55
maliniyw14:56
*** tonytan4ever has joined #openstack-marconi14:57
maliniflaper87: ping14:58
*** abettadapur has quit IRC15:03
vkmcbrb15:08
prashanthr_flaper87, alcabrera|afk, malini, sriram: I have updated the changes for queues and messages controllers, can you have a look when you are free: https://review.openstack.org/#/c/97178/15:09
prashanthr_vkmc: It will be nice if you can also have a look.15:10
flaper87malini: pong15:10
*** AAzza has joined #openstack-marconi15:10
flaper87sriram: +1 on all your comments15:11
*** jimhoagland has joined #openstack-marconi15:11
maliniflaper87: git a question on https://review.openstack.org/#/c/98793/ - doesn't it also need to be updated in pooling.py?15:11
*** alcabrera|afk is now known as alcabrera15:13
flaper87malini: what do you mean?15:14
maliniflaper87: How does the pools know which flavor a queue belongs to?15:15
*** sriram has left #openstack-marconi15:16
*** prashanthr_ has quit IRC15:16
*** sriram has joined #openstack-marconi15:16
openstackgerritA change was merged to openstack/marconi: Adding dependencies for redis support in marconi  https://review.openstack.org/9601715:16
flaper87malini: ah, so, the way it works is:15:16
flaper87A queue is created w/ a flavor. That info gets stored in queue's metadata15:16
flaper87since the flavor has a pool assigned to it, when a message is posted to the queue, it'll be sent to that specific pool15:17
flaper87which means the flavor know's what pool it belongs to, not the other way around15:17
maliniaah..ok15:18
maliniThanks flaper87!15:18
flaper87malini: np15:20
flaper87:)15:20
*** denis_makogon has quit IRC15:31
*** cath6184 has quit IRC15:37
*** cath6184 has joined #openstack-marconi15:39
*** amitgandhi has quit IRC15:46
*** kgriffs|afk is now known as kgriffs15:47
sriramflaper87: the new patch is up, please have a look, when you have the time. https://review.openstack.org/#/c/98875/15:48
* flaper87 clicks15:48
*** sriram has left #openstack-marconi15:48
flaper87sriram I'll take my A+ back because you left15:49
flaper87>.>15:49
*** sriram has joined #openstack-marconi15:49
* sriram accidentally closed the chat window, and was looking for it :P15:50
*** reed has joined #openstack-marconi15:51
*** ykaplan has quit IRC16:03
srirambbl lunch16:07
*** stannie has quit IRC16:10
*** Obulpathi has quit IRC16:11
*** chandankumar has quit IRC16:13
openstackgerritA change was merged to openstack/marconi: feat(benchmarking) : Producer-Consumer scenario  https://review.openstack.org/9887516:14
*** jimhoagland_ has joined #openstack-marconi16:19
*** jimhoagland has quit IRC16:19
*** jimhoagland_ is now known as jimhoagland16:19
*** cath6184 is now known as Catherine_16:30
*** Catherine_ is now known as catherine_16:32
*** rossk has joined #openstack-marconi16:53
*** tonytan4ever has quit IRC16:56
*** AAzza has quit IRC16:58
vkmcbrb17:01
*** Obulpathi has joined #openstack-marconi17:01
*** amitgandhi has joined #openstack-marconi17:12
*** openstackgerrit has quit IRC17:16
*** openstackgerrit has joined #openstack-marconi17:18
kgriffsbtw folks, while doing reviews this week, please keep in mind: https://wiki.openstack.org/wiki/Reviewer_Guide_(Marconi)17:28
*** shakamunyi has joined #openstack-marconi17:39
*** shakamunyi has quit IRC17:39
*** shakamunyi has joined #openstack-marconi17:40
*** shakamunyi has quit IRC17:42
*** tongli has quit IRC17:46
*** ametts has joined #openstack-marconi17:50
*** ametts has quit IRC18:05
*** cpallares has quit IRC18:06
*** flaper87 is now known as flaper87|afk18:10
*** mkoderer has quit IRC18:12
kgriffspleeeeeeese review: https://review.openstack.org/#/c/106888/18:16
*** reed has quit IRC18:23
*** catherine_ has quit IRC18:23
*** flaper87 has joined #openstack-marconi18:25
*** ChanServ sets mode: +o flaper8718:25
*** catherin_ has joined #openstack-marconi18:29
*** abettadapur has joined #openstack-marconi18:32
vkmckgriffs, why in https://review.openstack.org/#/c/106888/ you use global constants for v1_0 and class variables for v1_1 for post_specs?18:46
*** tonytan4ever has joined #openstack-marconi18:49
peoplemergekgriffs: I'm still swamped at work but I'm hoping to clear my plate today and be back to coding+reviews tomorrow.  If you have time for reviews today, I'd like to get your feedback on https://review.openstack.org/#/c/105830/4/marconi/tests/queues/transport/wsgi/v1_1/test_messages.py around line 48318:52
*** mwagner_lap has quit IRC18:53
*** jimhoagland has quit IRC18:59
*** amitgandhi has quit IRC19:00
peoplemergekgriffs: If not, I can do some more research to figure out what test separation strategy makes sense.19:04
*** jmckind has joined #openstack-marconi19:09
vkmchi peoplemerge!19:10
vkmcI'm not an expert19:10
vkmcbut maybe that test could make more sense in https://github.com/openstack/marconi/blob/master/tests/unit/queues/transport/wsgi/test_utils.py19:11
vkmcor in some file like test_utils19:11
peoplemergevkmc: interesting.19:13
peoplemergevkmc: this doesn't seem to be a json test19:13
kgriffsvkmc: re constants vs class variables19:14
peoplemergeI had some questions about if the test design for msgpack should cover MongoDB as well19:14
kgriffsvkmc: couple reasons. First, I am trying not to touch v1_0 if I don't have to (if it ain't broken, don't fix it, heh)19:15
kgriffssecond is that in v1_0 the spec never changes19:15
*** alcabrera is now known as alcabrera|afk19:15
kgriffsbut in v1_1 it is dynamic - depends on the TTL configuration options19:15
vkmcpeoplemerge, yeah that's why I mentioned 'some file like test_utils'... we should try to keep it independent from other tests :)19:16
peoplemergevkmc: thinking about  test_utils.py there will probably be tests coming I'll need to write there, IMHO different ones tho19:16
peoplemergevkmc: :D19:16
vkmckgriffs, that's clear :)19:16
*** ametts has joined #openstack-marconi19:17
kgriffsok, thanks19:18
kgriffsTBH, I saw a lot of things I was tempted to clean up while making this patch, but I decided it would be best to just do the minimal work, and worry about fixing "broken windows" later19:19
vkmckgriffs, thanks you!19:19
vkmcyup19:19
kgriffsfor example, I think the filter_streams function should be broken up into 2-3 "before" falcon hooks19:19
kgriffsalso, I'm not totally happy with the config settings living inside the validation module19:20
peoplemergekgriffs: "broken windows" love it :D I'm not the same Dave Thomas as who wrote this19:21
vkmcyeah it may not be semantically correct19:22
* kgriffs hands peoplemerge a Pop-Tart™ for picking up the reference19:22
vkmcoh19:23
peoplemergekgriffs: :) I have a slide deck on this I give at work, meetups19:23
* vkmc googles19:23
vkmclol19:23
malinikgriffs: regarding https://review.openstack.org/#/c/106888/19:24
maliniDo you want to update the v1.0 API ?19:24
* kgriffs thinks peoplemerge is teh awesome19:29
*** malini has quit IRC19:30
kgriffsmalini|afk: not sure I follow... what would I do to update the v1.0 API?19:30
kgriffsand yeah, needs functional test. :D19:30
*** saikrishna has joined #openstack-marconi19:42
*** kgriffs is now known as kgriffs|afk19:43
*** alcabrera|afk is now known as alcabrera19:44
*** Obulpathi has quit IRC19:46
*** kgriffs|afk is now known as kgriffs20:01
*** flaper87 has quit IRC20:01
peoplemergekgriffs: #D20:14
peoplemergekgriffs: https://docs.google.com/presentation/d/1aBzGmCdKtEavfjILNgblarZZkIKZ4-qaKsUbANZOya8/edit#slide=id.i020:16
openstackgerritKurt Griffiths proposed a change to openstack/marconi: Add flavors support to mongodb  https://review.openstack.org/9879320:16
kgriffsrebased flavio's patch ^^^20:16
peoplemerge^-- everything I believe about the practice of programming ... or used to 3 yrs ago before I learned functional programming20:17
kgriffspeoplemerge: nice, thanks for sharing!20:17
kgriffslol re Big Ball of Mud as the standard enterprise pattern20:18
peoplemergehehe20:18
kgriffsare your peoplemerge on twitter?20:21
peoplemergeyes20:21
kgriffscool20:22
*** Obulpathi has joined #openstack-marconi20:26
*** alcabrera is now known as alcabrera|afk20:28
*** amitgandhi has joined #openstack-marconi20:29
peoplemergekgriffs: I'm also http://www.peoplemerge.com20:30
peoplemergeresume here (hint hint :)  ----^20:30
kgriffs:)20:30
kgriffsthanks, gtk20:30
*** rossk has quit IRC20:31
*** saikrishna has quit IRC20:31
kgriffsabettadapur: reminder that there are some comments to address here - https://review.openstack.org/#/c/96159/320:31
*** sriram has quit IRC20:31
*** abettadapur has quit IRC20:34
*** Obulpathi has quit IRC20:34
*** Obulpathi has joined #openstack-marconi20:35
*** oz_akan has quit IRC20:44
*** mwagner_lap has joined #openstack-marconi20:58
*** Obulpathi has quit IRC21:08
*** jay-atl has quit IRC21:09
*** mpanetta has quit IRC21:17
*** jmckind has quit IRC21:21
*** tonytan4ever has quit IRC21:22
*** tonytan4ever has joined #openstack-marconi21:25
*** ametts has quit IRC21:34
*** catherin_ has quit IRC21:34
*** amitgandhi has quit IRC21:58
*** oz_akan has joined #openstack-marconi22:12
*** tonytan4ever has quit IRC22:16
*** oz_akan has quit IRC22:17
*** kgriffs is now known as kgriffs|afk22:29
*** jimhoagland has joined #openstack-marconi23:25
*** jimhoagland has quit IRC23:33
openstackgerritFei Long Wang proposed a change to openstack/marconi: (WIP)feat(v1.1): Implement new /health endpoint for v1.1  https://review.openstack.org/7655523:56
« Tuesday, 2014-07-15 Index Thursday, 2014-07-17 »

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