Friday, 2013-10-25

*** amitgandhi has quit IRC00:10
*** amitgandhi has joined #openstack-marconi00:11
*** amitgandhi has quit IRC00:15
*** vkmc has quit IRC00:23
*** nosnos has joined #openstack-marconi00:49
*** amitgandhi has joined #openstack-marconi01:09
*** oz_akan_ has joined #openstack-marconi01:15
*** oz_akan_ has quit IRC01:19
*** oz_akan_ has joined #openstack-marconi01:20
*** etoews1 has joined #openstack-marconi01:20
*** kgriffs_afk is now known as kgriffs02:37
*** amitgandhi has quit IRC03:00
*** amitgandhi has joined #openstack-marconi03:00
*** amitgandhi has quit IRC03:04
openstackgerritA change was merged to openstack/python-marconiclient: Implement HTTP Transport  https://review.openstack.org/4978803:15
*** oz_akan_ has quit IRC03:17
*** kgriffs is now known as kgriffs_afk03:23
*** amitgandhi has joined #openstack-marconi03:30
*** amitgandhi has quit IRC03:48
*** amitgandhi has joined #openstack-marconi04:13
*** amit has joined #openstack-marconi04:15
*** amitgandhi has quit IRC04:15
*** amit is now known as Guest6250804:16
*** cpallares has quit IRC04:23
*** Guest62508 has quit IRC04:32
*** amitgandhi has joined #openstack-marconi04:57
*** amitgandhi has quit IRC05:15
*** amitgandhi has joined #openstack-marconi05:41
*** amitgandhi has quit IRC06:00
*** amitgandhi has joined #openstack-marconi06:26
*** amit has joined #openstack-marconi06:28
*** amitgandhi has quit IRC06:28
*** amit is now known as Guest5757306:29
*** flaper87|afk is now known as flaper8706:37
*** Guest57573 has quit IRC06:42
*** amitgandhi has joined #openstack-marconi07:08
*** amitgandhi has quit IRC07:10
*** amitgandhi has joined #openstack-marconi07:11
*** flaper87 is now known as flaper87|afk07:27
*** amitgandhi has quit IRC07:28
*** amitgandhi has joined #openstack-marconi07:53
*** yassine has joined #openstack-marconi07:57
*** amit has joined #openstack-marconi08:25
*** amitgandhi has quit IRC08:25
*** amit is now known as Guest5510008:25
*** Guest55100 has quit IRC08:34
*** amitgandhi has joined #openstack-marconi09:00
*** amitgandhi has quit IRC09:15
*** amitgandhi has joined #openstack-marconi09:40
*** amit has joined #openstack-marconi09:43
*** amitgandhi has quit IRC09:43
*** amit is now known as Guest9138409:43
*** flaper87|afk is now known as flaper8710:13
*** Guest91384 has quit IRC10:14
*** ekarlso has quit IRC10:17
*** ekarlso has joined #openstack-marconi10:17
*** nosnos has quit IRC10:31
*** amitgandhi has joined #openstack-marconi10:38
*** amit has joined #openstack-marconi10:41
*** amitgandhi has quit IRC10:41
*** amit is now known as Guest8351410:41
*** flaper87 is now known as flaper87|afk10:48
*** Guest83514 has quit IRC10:54
*** tedross has joined #openstack-marconi11:19
*** amitgandhi has joined #openstack-marconi11:19
*** amitgandhi has quit IRC11:36
*** malini_afk is now known as malini11:39
*** amitgandhi has joined #openstack-marconi12:00
*** malini is now known as malini_afk12:02
*** amitgandhi has quit IRC12:05
*** flaper87|afk is now known as flaper8712:09
*** vkmc has joined #openstack-marconi12:17
*** amitgandhi has joined #openstack-marconi12:37
*** amitgandhi has quit IRC12:42
*** oz_akan_ has joined #openstack-marconi13:04
*** oz_akan_ has quit IRC13:06
*** oz_akan_ has joined #openstack-marconi13:06
*** mpanetta has joined #openstack-marconi13:07
*** mpanetta has quit IRC13:09
*** mpanetta has joined #openstack-marconi13:09
*** mpanetta has joined #openstack-marconi13:10
*** russellb is now known as rustlebee13:12
*** amitgandhi has joined #openstack-marconi13:22
*** alcabrera has joined #openstack-marconi13:30
alcabreraMorning! :D13:31
flaper87alcabrera: goood morning!13:31
flaper87alcabrera: did you read the tstream news?13:32
alcabreraYep, just now!13:32
alcabrera"I'll get that implemented soon." :D13:32
*** amitgandhi has quit IRC13:32
alcabrerajtauber is awesome. :)13:33
*** amitgandhi has joined #openstack-marconi13:33
alcabrerawait wait wait...13:33
alcabreraIt's already implemented!13:33
alcabreraFast!13:33
* alcabrera is catching up on email13:34
*** jcru has joined #openstack-marconi13:43
flaper87alcabrera: :D13:45
flaper87it is and it works13:45
flaper87now we've markdown and atom there13:45
alcabrerasweeeet13:46
alcabreraI'll be flaper87 - I've been called for office-type duties. :P13:46
alcabreras/be/brb13:46
*** alcabrera is now known as alcabrera|afk13:46
*** malini_afk is now known as malini13:47
*** kgriffs_afk is now known as kgriffs13:58
kgriffsmorning/afternoon!13:59
flaper87kgriffs: hellooooo14:02
openstackgerritA change was merged to openstack/python-marconiclient: change assertEquals to assertEqual  https://review.openstack.org/5351214:04
kgriffsflaper87: so...14:05
kgriffsI was doing some thinking about the client design14:05
kgriffsnot sure yet if this makes sense, but here is my idea14:06
flaper87kgriffs: go ahead14:06
kgriffsthe goal is to reduce churn in the client when the api changes14:06
kgriffsso, what if the api schema was loaded from the api itself?14:07
kgriffsinstead of it being hard-coded into the client14:07
kgriffsthe client *would* have to hard-code operation names14:07
flaper87that's the plan, I wanted to use the /home endpoint, but I don't think the current format is very helpful14:08
kgriffsso it can look them up still in the schema14:08
flaper87and that's where a server side schema will come handy14:08
flaper87:D14:08
kgriffsflaper87: cool, so that was my other point14:08
kgriffsJSON home is geared toward HTTP14:08
kgriffsi mean, that is the bias14:09
*** zyuan_ has joined #openstack-marconi14:10
kgriffssecond thing is that HATEOAS isn't limited to HTTP14:10
kgriffsyou could have a schema of entry points and then subsequent requests would return refs or whatever to follow for other operations14:10
flaper87indeed, TBH, the fact that it's geared toward HTTP is not a huge issue here. If you check the API schema I'm currently using for the client, it's basically related to HTTP14:11
flaper87since it's the most partitioned protocol we're supporting so far14:11
flaper87and it's easier to translate from HTTP to zmq than the other way around14:11
flaper87kgriffs: re refs returned, totally agreed14:12
kgriffsyeah. I have a feeling we will have to get another protocol in place before we will be able to really work out the best design14:12
kgriffsso, back to the home doc thing14:12
flaper87I was thinking about adding support for that after queues + messages + claims land14:12
flaper87since we're already returning refs14:12
kgriffsexcellent14:12
flaper87the missing part would be "Getting the base API schema" from the server14:13
flaper87which we already agreed on14:13
flaper87:D14:13
flaper87kk, back to home document14:13
kgriffsre home doc, let's start thinking about what a more *useful* document format would look like14:13
kgriffsit would be cool to define a rich schema behind the scenes, that could then be exposed as various media types, home doc being one if it ends up being useful to developers14:14
kgriffss/home doc/JSON home14:14
zyuan_useful to machine or to user14:15
kgriffsi think this plays nicely into what you were talking about the other day re server-side schema14:15
zyuan_?14:15
kgriffszyuan_: either, but it seems like home doc is mostly for humans or for simple documentation "click through my API" tools.14:15
kgriffssorry, by home doc I mean json-home14:16
kgriffswhat i am proposing is we have some other kind of x-home format that works better for flaper87's needs14:16
flaper87kgriffs: indeed, that's exactly what I was talking about the other day. A server side schema will allow us to easily support discovering and extensions14:16
zyuan_hmm, i don't think a human will find json home document is readable to "click through"...14:16
kgriffszyuan_: i mean, you would have a web app or something that consumes it and turns it into HTML14:17
zyuan_ok14:17
kgriffsflaper87: ok, looks like once again we have proved our brains are quantum-entangled.14:18
flaper87kgriffs: btw, I checked that 'API extensions' talk you shared the other day14:18
flaper87kgriffs: hahahahaha14:18
flaper87kgriffs: Programmatically speaking, I was thinking about supporting API schema inheritance, so, newer API's would be able to inherit from older if needed14:19
flaper87kgriffs: we would play with mixins for extensions14:20
flaper87and transport will register everything found in an API spec dynamically14:20
flaper87I'm still putting thoughts on that, I created a blueprint for it (you should've an email for that) and I'll start writing some stuff on a epad first14:21
flaper87I don't think I'll have some code ready for our next meeting14:21
flaper87but definitelly I'll have some specs written down and a more detailed idea14:21
kgriffscool beans14:23
kgriffsyou definitely want to be able to inherit for minor API version bumps14:23
kgriffsbtw, re sharding additions to the API14:24
* kgriffs wonders where alcabrera|afk got off to14:24
flaper87kgriffs: he said he had some office duties to attend14:28
flaper87I'll get to sharding patches very soon14:28
kgriffssounds fishy to me. ;)14:30
kgriffsanyway14:30
kgriffsflaper87: I was thinking that sharding could be implemented as a v1.0 extension14:30
kgriffsthen promoted into v1.1 when we start work on that14:30
kgriffsthat being said14:31
kgriffsit is a backwards-compatible update to the api, so we could sneak it into v114:31
kgriffsnot sure which way to go14:31
kgriffstechnically, v1.0 is frozen14:32
kgriffsso, maybe we let the patches land as-is, then circle back and move them into extension land14:33
kgriffsor14:33
kgriffswe just start work on v1.1 now14:33
kgriffswhat do you think?14:34
* flaper87 thinking14:35
flaper87I'm leaning towards sneaking it into v1, TBH.14:35
zyuan_what are the changes you want now?14:36
flaper87We should see how far we get with the API spec blueprint and perhaps decide based on that14:36
kgriffsok14:37
kgriffsso, for now we will sneak it in, but not document it until v1.114:37
kgriffsand revisit as the API spec stuff evolves14:37
kgriffsman, I wish wiki.openstack.org had an option to submit markdown14:39
zyuan_me too...14:39
*** ametts has joined #openstack-marconi14:40
flaper87kgriffs: zyuan_ tell me about it! :/14:40
kgriffsinteresting article here, has lessons applicable beyond SolidFire14:45
kgriffshttp://blog.mongodb.org/post/64962828969/performance-tuning-mongodb-on-solidfire14:45
kgriffsbtw, the guy you started SolidFire is a former boss of mine14:45
kgriffss/you/who14:45
*** ayoung has joined #openstack-marconi14:45
flaper87ametts: it now has atom and markdown: https://thoughtstreams.io/combined/marconi-progress-and-updates/14:47
flaper87:)14:47
flaper87our combined stream14:47
amettsWow.  I should send a letter to the customer service department of that software company... :)14:48
flaper87ametts: you can just tweet them :D14:48
flaper87ametts: this is how that feature got requested and implemented: https://twitter.com/cppcabrera/status/39373292700920627214:49
flaper87ametts: I mean, this: https://twitter.com/flaper87/status/39338393241191628814:49
amettsYeah, I saw that -- nice that they did it so fast.  So I've added it to my newsreader, and now I'll know everything you guys are thinking....14:50
kgriffs:o14:51
zyuan_json is pypy: http://morepypy.blogspot.com/2011/10/speeding-up-json-encoding-in-pypy.html14:51
zyuan_in*14:51
*** alcabrera|afk is now known as alcabrera14:51
alcabreraback~14:51
alcabreraI missed all the things, didn't I?14:52
flaper87alcabrera: you did, I'm not goint to lie14:53
flaper87:D14:53
alcabreralol14:54
alcabreraI've caughtup at this point.14:54
alcabrerax-home and such14:54
flaper87LOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL: https://thoughtstreams.io/combined/marconi-progress-and-updates/ (read the last 2 posts)14:54
flaper87kgriffs: man, this is scaring me now. ^14:54
alcabrerawow14:54
kgriffs:D14:54
alcabrerayou guys are so sync'd. :P14:55
flaper87I think the "We're now thinking about" is the scariest part of both posts14:55
kgriffsalcabrera: hurry up and find your own doppelgänger so we can have 4 core team members instead of just 3.14:56
alcabrerakgriffs: I'll have one ready by the end of the day. :P14:56
flaper87kgriffs: btw, I love the sharding diagram14:57
flaper87seriously :D14:57
flaper87also, I added this: https://wiki.openstack.org/wiki/OutreachProgramForWomen/Ideas#Marconi_API_Spec14:57
flaper87I'm not expecting the intern to do it all but to work together on that14:57
kgriffscool14:58
alcabreraawesome!14:58
kgriffsI wasn't sure about the idea of putting Pecan POC on there, since it would be cool to give them something that would get them more into the guts of the project14:59
flaper87kgriffs: agreed!15:00
kgriffsnobody likes to get that "here is this thing that we are procrastinating and may or may not turn out to be useful" project15:00
kgriffsbtw, thanks a ton for working on the OPW15:01
alcabreraI'm favorable towards having people working on alternative storage drivers. It's quite at the heart of the project and gives volunteers a chance to work on/with cool technologies. :D15:03
kgriffsi mean, it will definitely be useful in one way or another, but may not be as satisfying as some other types of work15:03
kgriffsflaper87: ready to talk about the talk?15:04
flaper87alcabrera: I thought about adding a storage driver there (like redis one) but I then remembered that we want to keep the number of storage drivers in the code base small15:04
alcabrerathat's true15:04
flaper87so, I'd like us all to discuss what drivers it makes sense to have in the codebase first15:04
flaper87kgriffs: yup, gimme 2 secs. I'll find a quite place.15:05
flaper87read ?15:08
kgriffshttps://plus.google.com/hangouts/_/72cpjvbuiq4ajgfi9ri5m3ublk?hl=en15:10
kgriffshttps://etherpad.openstack.org/p/icehouse-marconi-talk15:12
*** jergerber has joined #openstack-marconi15:14
kgriffsflaper87: can't hear you...15:16
kgriffsflaper87: let's just chat here15:22
kgriffssilly internets15:22
*** flaper87 is now known as flaper87|afk15:24
* ametts is back and having a google hangout with nobody but himself15:29
kgriffsheh15:30
kgriffsflaper87|afk: is still MIA15:30
kgriffsametts: do you think 10 minutes would be better for demo, or just 5?15:31
amettsDepends on what we demo, I guess.  Let's think first about what we want the content/messaging to be, and what we set aside as backup material for extra time and Q/A.15:33
amettsThen we can figure out the time allocations, I guess.15:33
alcabreraalright, now with all my office duties complete, let's see how things are looking in marconi-land~15:34
malinithe bagel were good, werent they alcabrera ?15:35
amettsWe have bagels?15:35
kgriffsametts: https://plus.google.com/hangouts/_/72cpjvbuiq4ajgfi9ri5m3ublk?hl=en15:35
alcabreraI had no idea we had bagels. :P15:36
maliniametts: I was just referring to alej's office duty ;)15:36
alcabreralol15:36
* ametts is hungry15:36
malinime too...15:37
alcabreraI am, too.15:42
alcabreraI'll probably step out of the office for lunch today. Rare things happen sometimes. ;)15:42
openstackgerritChad Lung proposed a change to openstack/marconi: Update the Marconi readme with more detailed installation instructions  https://review.openstack.org/5367315:45
mpanettaWe have bagles?15:45
mpanettabagels too?15:46
openstackgerritChad Lung proposed a change to openstack/marconi: Update the Marconi readme with more detailed installation instructions  https://review.openstack.org/5367315:47
alcabrerampanetta: maybe? If we did, they'd be in the break room.15:52
alcabreraI haven't checked, heh.15:52
maliniI am sorry everybody..There are NO bagels :(15:54
maliniI am so hungry, my mind is playing tricks on everybody15:54
mpanettamalini: Damn you!  Stop being hungry! :P15:54
*** cpallares has joined #openstack-marconi15:55
*** alcabrera is now known as alcabrera|lunch16:02
*** flaper87|afk is now known as flaper8716:06
cpallareshi flaper8716:08
flaper87damn it, got kicked out again16:08
*** etoews1 has quit IRC16:08
flaper87cpallares: heeey :d16:09
flaper87:D16:09
flaper87cpallares: how are you doing?16:09
cpallaresflaper87: Good I got marconi installed, but I'm a bit confused about the bug. Is it to rename exceptions.py?16:09
cpallarescpallares: Also I'm good and how are you? :)16:10
flaper87cpallares: doing well, thanks :D16:10
flaper87cpallares: that bug is about renaming exceptions.py into errors.py, replacing Exception with Error in classes' names and fixing the imports all around16:12
*** etoews1 has joined #openstack-marconi16:12
flaper87cpallares: btw, I added a new project to the wiki page16:12
flaper87in case you want to check that out16:12
cpallaresflaper87: I will, there's two exceptions.py, but this one is just for the proxy dir, right?16:13
*** etoews1 has quit IRC16:13
flaper87cpallares: feel fre to change queues/* as well, just to keep consistency16:14
flaper87cpallares: also, proxy will probably go away16:14
cpallaresSo, all the exceptions.py that I find?16:15
cpallaresThere's one in /common16:16
cpallaresI mean change them to keep consistency16:17
flaper87cpallares: yup!16:18
cpallaresok!16:18
flaper87cpallares: but not the ones under openstack/common16:19
flaper87ok?16:19
cpallaresok16:19
flaper87cpallares: quick intro to Oslo. All files under openstack/common come from oslo-incubator and we never modify them directly in Marconi, instead we do it in oslo-incubator and then backport the patches16:19
flaper87cpallares: https://wiki.openstack.org/wiki/Oslo16:19
cpallaresok16:21
cpallareswhat about marconi/common?16:23
flaper87cpallares: that's under our control :D16:25
flaper87so, go ahead :D16:25
*** alcabrera|lunch is now known as alcabrera16:29
alcabreracpallares, flaper87: o/16:32
alcabreraI made it back with 100% more lunch available. I skipped breakfast so I was rather hungry. :x16:32
* flaper87 will have this for dinner today: http://southamericanfood.about.com/od/holidayrecipes/r/Pan-De-Jamon-Venezuelan-Ham-And-Olive-Bread.htm16:34
* flaper87 is starving, so so so much16:34
alcabrerammmm... :x16:35
alcabrerasounds yummy!16:35
alcabreraI love olives.16:35
flaper87alcabrera: this looks even better: http://i1.trekearth.com/photos/32025/pan_de_jamon__191203mod.jpg16:36
alcabreraflaper87: ohhh yes. :D16:37
alcabreraI am having sausages for lunch: these (http://www.johnsonville.com/products/beer-brats.html) and these (http://www.jimmydean.com/products/fresh-sausage/premium-original-pork-sausage-patties)16:38
*** whenry has joined #openstack-marconi16:38
kgriffsflaper87: you should totally come to montreal next april16:46
kgriffshttps://us.pycon.org/2014/16:46
* flaper87 proposed a talk, fingers crossed16:47
flaper87:D16:47
flaper87alcabrera: that looks GOOOOOOD16:47
kgriffslet me know if you end up going - we can meet up16:47
flaper87now I'm even more hungry16:47
kgriffsthe pizza truck is calling my name16:48
alcabreralol16:48
flaper87kgriffs: totally, If I do, I'm planning to spend some time overseas but, shhhhhhhhhh16:48
alcabreraWe're all aiming to go to pycon Montreal, afaict. :)16:48
zyuan_forgot to say, simplejson only .read() stream then do parsing; no json parser i found works on stream level.16:48
flaper87I hope to be there even if I'm not talking but, you know!16:48
zyuan_wow16:48
kgriffszyuan_: gtk16:49
*** cpallares has quit IRC16:50
*** jergerber has quit IRC16:50
*** cpallares has joined #openstack-marconi16:50
*** amitgandhi has quit IRC16:50
*** amitgandhi has joined #openstack-marconi16:50
*** jergerber has joined #openstack-marconi16:51
kgriffsalcabrera: so, about sharding16:52
kgriffsany updates on your patches? I am getting ready to review them again16:54
*** amitgandhi has quit IRC16:55
alcabrerakgriffs: none yet. It's been one of those chaotic mornings. I'm almost done updating the catalogue patch and I need to work on removing the admin concept from the storage layer for the other patches.16:57
flaper87kgriffs: in the meantime you can keep approving client patches :D16:57
kgriffsheh16:57
alcabreraI'm all for that. :D16:57
kgriffsalcabrera: re admin concept16:58
kgriffscan you elaborate?16:58
*** oz_akan_ has quit IRC16:58
alcabrerakgriffs: I mean, to stop using AdminDriver(catalogue_controller, shards_controller) and Driver(messages_, claims_, queues_) for storage, and just use Driver.16:59
*** yassine has quit IRC17:01
kgriffsso, in that case sharding.catalog will reference mongodb.Driver ?17:01
alcabrerayup17:01
kgriffsi mean, get the shard controller from mongodb.Driver17:01
kgriffsok17:01
kgriffshmm17:02
alcabreraadding in the AdminDriver thing for storage made things weird, which stood out to me all the more yesterday when flaper87 was talking about it.17:02
kgriffsconceptually, I still feel like it would be better to separate control plane and data plane drivers17:02
*** cpallares has quit IRC17:02
kgriffshttp://grab.by/rsoI17:02
*** cpallares has joined #openstack-marconi17:02
kgriffswhat is the weirdness you mentioned?17:04
alcabreralemme look it up...17:04
kgriffsif we combine the two17:05
kgriffswhen sharding is disabled, you use one subset of the driver17:05
kgriffswhen it is enabled you use a different, non-overlapping set17:05
alcabreraso there's this: https://review.openstack.org/#/c/50998/7/marconi/queues/storage/sharding.py (how is a sharding AdminDriver handled?) and https://review.openstack.org/#/c/50998/7/marconi/queues/storage/pipeline.py (what does AdminDriver mean in a pipeline?)17:06
* kgriffs reads that back and tries to determine whether it makes sense17:06
*** oz_akan_ has joined #openstack-marconi17:06
alcabrerawhether or not sharding is enabled, messages_, claims_, and queues_ come into the picture. shards_ and catalogue_ only come into play when sharding is enabled.17:07
kgriffshmmm17:07
alcabrerawhether or not they are in an admin context or not (since they're both needed for making performing lookups: (project, queue) => shard).17:07
alcabreras/making//17:08
kgriffsThe control API endpoints don't go through the pipeline17:08
kgriffsthe get a reference to the control plane AKA admin driver's controllers directly17:08
kgriffssame with sharding.Catalog17:08
alcabrerahmm17:10
kgriffsand the admin driver doesn't have multiple backends, only one, so it is never sharded17:10
kgriffs(so no need for that meta-driver in storage.sharding)17:10
alcabreraalright, so I can remove the AdminDriver concept from pipeline and sharding17:11
kgriffsthat's correct17:11
alcabreraI wonder if it helps to remove the admin notion from the storage layer entirely...17:11
alcabreraSince the transport layer determines that.17:12
alcabreraThis is orthogonal thinking to the notion of control plane vs. data plane17:12
alcabreraI feel like having the admin notion duplicated in the storage layer is redundant.17:12
kgriffsa couple points17:13
kgriffsfirst, exposing shardcontroller through the data drivers just feels like we are violating the principle of cohesion in object-oriented design.17:14
kgriffsmainly, because we are putting N of those drivers behind sharding.Driver, and shardcontroller isn't sharded itself17:15
kgriffsmakes things weird17:15
kgriffsalso, there is the control vs. data17:15
kgriffsif that point is valid, then the second point holds17:15
kgriffsthat we need to load a separate driver17:15
alcabreraI see what you're saying with regards - I see.17:16
kgriffsand, assuming that, you probably don't want to make people configure that control plane driver unless they are going to use it17:16
kgriffswhich brings back the idea of an admin mode17:17
alcabreraYeah, it's a bit silly to require sharding.Driver to also provide a sharding and a catalogue controller.17:17
*** amitgandhi has joined #openstack-marconi17:17
alcabreraSo that takes care of my redundancy concern - it's not redundant.17:17
kgriffsso, if we follow this design: http://grab.by/rsoI17:18
kgriffsthen the bootstrap would load the admin driver and pass it to the admin transport which grafts on to the data API17:18
kgriffsalso, bootstrap would pass the same driver through so that eventually sharding.Catalog can reference it as well17:19
kgriffs(so it can dynamically create/delete mappings, as well as lookup mappings)17:19
kgriffsanyway, that is my $0.0217:20
alcabreraMakes sense.17:20
kgriffsI think that if we do it that way, it ends up being less weird. It would be nice to get Flavio's blessing, but I have a hard time seeing how putting everything in a single driver is going to be a good design17:21
alcabreraThat's pretty much what convinced me.17:21
alcabreraI think the last bit of terminology is the problem.17:21
alcabreraDriver -> DataDriver, AdminDriver -> ControlDriver17:22
kgriffsi suspect that he pushed back on the idea originally because it *was* looking weird17:22
alcabreraBecause it's not about admin or public - it's about control and data.17:22
kgriffsyeah17:22
alcabreraI can update the names if you like the idea above.17:22
kgriffskudos to amitgandhi for giving me that idea17:22
kgriffs(terminology)17:22
alcabreraIt makes it more clear we're talking about data/control rather than permissions (admin/public).17:22
alcabreraamitgandhi: +117:22
kgriffsyeah17:23
kgriffspermissions are there too17:23
kgriffsbut permissions and planes aren't the same thing17:23
alcabrerayup17:23
kgriffshelps to think about them as complimentary instead17:23
kgriffsaaaanyway17:23
kgriffsi know you are sick of refactoring...17:24
kgriffswhat is the quickest way to get something in malini's hands to kick the tires?17:24
kgriffsand then what is the plan for getting to the new design we just discussed above?17:25
alcabreraget the admin API merged in. :/17:25
alcabrerathat's what'll give malini the ability to at least test the control side of things.17:25
kgriffsdo we just merge everything in as-is and then refactor in subsequent patches, or do we just have her deploy from gerrit?17:26
kgriffsmalini: ^^^17:26
alcabrerahmmm, she's not at her desk/computer.17:26
kgriffshackday?17:27
alcabreranah - Glenn's visiting. :)17:27
alcabreraHackday's next week.17:27
malinikgriffs: ping17:27
kgriffsah17:28
maliniSorry missed you17:28
kgriffsno worries17:28
kgriffsso, let's come up with a plan to get you testing sharding ASAP17:28
malinitht wud be awesome!!17:28
kgriffsat this point, we know what the functionality will be and the data schema17:28
* flaper87 lost a great convo17:28
flaper87damn it!17:28
flaper87malini: quick question: Are you planning to work on tempest support for Marconi?17:29
maliniI have a personal server tht runs marconi..I can start playing on tht, if we are not ready to deploy to an actual env17:29
maliniflaper87: I am17:29
kgriffsmalini: my idea was to get something in your hands right away and then we can circle back and refine the implementation17:29
flaper87malini: cool beans!17:29
maliniflaper87: I havent started yet, But I am starting to plan poking around it soon17:29
kgriffsflaper87: What do you think about this design? http://grab.by/rsoI17:30
maliniflaper87: are you working on the devstack part ?17:30
kgriffsthe original approach to splitting into an admin and non-admin storage drivers *was* weird17:30
kgriffsbut I think we have a plan for making it *not*17:30
flaper87kgriffs: ha, you missed my happy comments and many congrats about that design earlier today :D17:31
kgriffsand indeed, preserving cohesion17:31
flaper87kgriffs: it looks really good!17:31
kgriffsok, lets run with it and see how the implementation turns out17:31
kgriffsflaper87, alcabrera: thoughts on refactoring patches in-place, or letting them land and following up17:32
flaper87malini: I am, I finally got some feedback about it and I need to re-work it a bit17:33
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: add catalogue storage driver for queues  https://review.openstack.org/5216017:33
flaper87kgriffs: I'd prefer not letting those patches land and work on our final thoughts right away17:33
* kgriffs just heard that the pizza truck is not coming by today and is very sad17:33
*** amit has joined #openstack-marconi17:33
alcabrerakgriffs: pizza. :(17:33
kgriffsok17:33
alcabreraI'm cool with refactoring in place.17:33
kgriffsmalini: Can you just fetch from gerrit directly then and start kicking the tires?17:33
*** amit is now known as Guest3498317:34
alcabreraI just finished taking care of the catalogue storage driver updates above. ^^17:34
malinikgriffs: sure..17:34
kgriffsalcabrera: can you assist malini in getting started?17:34
kgriffsok guys17:34
kgriffsI need some food and then i've got some client patches screaming for attention. ;)17:35
flaper87w0000000t17:35
flaper87I've to step away for a bit! I'll hopefully be back later and review patches, otherwise I'll do it tomorrow17:36
alcabreraflaper87: thanks!17:36
alcabrerakgriffs: get the noms!17:36
flaper87(yes tomorrow, Got to go to MongoTorino)17:36
flaper87an talk about marconi17:36
flaper87HOLY SHIT!17:36
flaper87My slides!17:36
flaper87O.O17:36
*** amitgandhi has quit IRC17:36
kgriffsflaper87: rock on17:37
*** flaper87 is now known as flaper87|afk17:39
alcabreramalini: https://wiki.openstack.org/wiki/Marconi/specs/sharding/v117:42
*** oz_akan_ has quit IRC17:44
*** oz_akan_ has joined #openstack-marconi17:45
*** Guest34983 has quit IRC18:08
*** amitgandhi has joined #openstack-marconi18:08
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: shards storage controller interface  https://review.openstack.org/5072118:11
alcabrerakgriffs: refactored for Data/Control drivers - part I. ^^18:11
alcabreraI'll be refactoring child patches of that now.18:11
alcabrerahmmm... I seem to have forgotten to fix some super().__init__'s. Going back for those.18:13
alcabreranvm18:14
alcabreragit rebase does some crazy things sometimes.18:14
alcabreraPatch above is ready for review. ^^18:14
*** kgriffs is now known as kgriffs_afk18:17
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: shards storage controller interface  https://review.openstack.org/5072118:30
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: shards mongodb driver + tests  https://review.openstack.org/5081518:31
*** ayoung has quit IRC18:36
*** etoews1 has joined #openstack-marconi18:37
cpallaresif I'm ready to commit a fix for a bug, do I have to put anything as a commit message, like a bug number or something?18:42
*** kgriffs_afk is now known as kgriffs18:42
malinicpallares: https://wiki.openstack.org/wiki/GerritWorkflow#Committing_Changes18:43
cpallaresthanks malini!18:44
alcabrerakgriffs: thought - what do you think of changing the name of the bootstrap storage attribute to data and exposing a control attribute?18:47
*** amitgandhi has quit IRC18:47
alcabrerarationale: make that data/control distinction more prominent and make it possible to initialize a control driver if needed.18:47
*** amitgandhi has joined #openstack-marconi18:47
alcabreraamitgandhi: hey, hey. :)18:48
*** relaxdiego has joined #openstack-marconi18:49
*** ayoung has joined #openstack-marconi18:50
amitgandhialcabrera: yo18:51
alcabreraamitgandhi: you gonna join in on our other IRC channel? :P18:51
*** amitgandhi has quit IRC18:52
*** amitgandhi has joined #openstack-marconi18:52
amitgandhikgriffs: ping19:02
*** relaxdiego has quit IRC19:18
*** relaxdiego has joined #openstack-marconi19:22
*** alcabrera is now known as alcabrera|afk19:30
*** cpallares1 has joined #openstack-marconi19:31
*** cpallares has quit IRC19:31
kgriffsw00t: http://icehousedesignsummit.sched.org/type/marconi19:33
kgriffsamitgandhi: pong19:33
*** cpallares1 has quit IRC19:33
*** cpallares has joined #openstack-marconi19:33
*** malini is now known as malini_afk19:36
*** kgriffs has left #openstack-marconi19:38
*** kgriffs has joined #openstack-marconi19:38
*** ChanServ sets mode: +o kgriffs19:38
*** cpallares has quit IRC19:44
*** cpallares has joined #openstack-marconi19:44
*** cpallares has quit IRC19:47
*** cpallares has joined #openstack-marconi19:47
*** cpallares has quit IRC19:49
*** cpallares has joined #openstack-marconi19:50
*** amitgandhi has quit IRC19:58
*** amitgandhi has joined #openstack-marconi19:59
*** amitgandhi has quit IRC20:03
openstackgerritA change was merged to openstack/python-marconiclient: Pass keywords to the request initialization  https://review.openstack.org/5056720:04
*** amitgandhi has joined #openstack-marconi20:04
*** amitgandhi has quit IRC20:07
*** alcabrera|afk is now known as alcabrera20:08
*** amitgandhi has joined #openstack-marconi20:08
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: integrate shard storage with transport  https://review.openstack.org/5099820:11
kgriffsalcabrera: there you are!20:11
alcabrerayup. :D20:12
alcabreraAll patches rebased.20:12
alcabreraThe refactoring was epic, but not so bad.20:12
alcabreraDataDriver + ControlDriver - and everyone was happy.20:12
alcabrerakgriffs: ^20:12
kgriffsah, ok20:13
kgriffslet me finish reviewing this client library patch20:13
alcabreraI kept the name of the storage property the same, and added a control property.20:13
alcabrerathe transport takes that control and uses it as needed.20:13
alcabrerakgriffs: sure thing20:13
kgriffsah20:15
kgriffsI think renaming storage property could actually be good, but we can look at that in another patch20:15
alcabrera+120:16
alcabrerawe can talk about it some more with flaper87|afk come Monday morning.20:16
kgriffscrazy idea20:27
kgriffssupport msgpack media type20:27
kgriffsas in20:27
kgriffsapplication/msgpack20:27
kgriffsand make it the default in client libs20:27
alcabrerahmmm20:27
kgriffsfaster, more compact, more predictable (size)20:27
kgriffsand20:28
kgriffssupports binary message bodies!20:28
alcabreravery true.20:28
alcabreraRefactoring towards supporting different protocols should be fun. :P20:28
alcabreracontent-type -> handler -> handler.load() -> process -> respond20:29
alcabreraalso: content-type -> handler -> handler_not_found -> respond20:29
alcabrerakgriffs: as time allows, can I get you to review these: https://review.openstack.org/#/c/50721/, https://review.openstack.org/#/c/50815/, https://review.openstack.org/#/c/50998/ ?20:30
alcabreraThat's the rest of the admin API.20:30
alcabreraI'm going to rebase the catalogue to depend on the storage interface in the first of those,20:30
kgriffshttps://blueprints.launchpad.net/marconi/+spec/msgpack20:33
kgriffsback to reviewing20:33
alcabrerakgriffs: nice!20:33
alcabrerakgriffs: when sharding is done, I'm going to triage those BPs in the style of 'sharding-merge-strategy'.20:34
alcabreraWe have *a lot* of BPs that need some love.20:34
kgriffsyes we do!20:34
alcabreraback to coding thee things~20:35
alcabrera*the20:35
alcabreraI'm heading out for the night, guys. :)20:43
alcabreraemail me if I can help speed up the review process. I'm willing to help out during the weekend.20:44
kgriffskk20:44
kgriffsthanks!20:44
alcabrerao/20:44
*** alcabrera has quit IRC20:44
*** tedross has quit IRC20:50
*** ayoung has quit IRC21:07
*** relaxdiego has quit IRC21:23
*** oz_akan_ has quit IRC21:52
*** oz_akan_ has joined #openstack-marconi21:52
*** oz_akan_ has quit IRC21:57
openstackgerritCindy Pallares proposed a change to openstack/marconi: Closes-Bug: 1232074  https://review.openstack.org/5396221:59
*** amitgandhi has quit IRC22:07
*** amitgandhi has joined #openstack-marconi22:07
*** amitgandhi has quit IRC22:12
*** kgriffs is now known as kgriffs_afk22:16
*** mpanetta has quit IRC22:19
*** ametts has quit IRC22:24
openstackgerritCindy Pallares proposed a change to openstack/marconi: Closes-Bug: 1232074  https://review.openstack.org/5396722:28
*** cpallares has quit IRC22:43
*** etoews1 has quit IRC22:46
*** jergerber has quit IRC22:51
*** amitgandhi has joined #openstack-marconi23:08
*** amitgandhi has quit IRC23:14
*** jcru has quit IRC23:17
*** amitgandhi has joined #openstack-marconi23:18
*** amitgandhi has quit IRC23:22

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