Tuesday, 2013-10-01

*** reed_ has quit IRC00:00
*** kgriffs is now known as kgriffs_afk00:10
openstackgerritA change was merged to openstack/marconi: chore: Expose commit hash in version module  https://review.openstack.org/4745900:19
*** malini_afk is now known as malini00:26
*** nosnos has joined #openstack-marconi00:50
*** kgriffs_afk is now known as kgriffs01:11
openstackgerritKurt Griffiths proposed a change to openstack/marconi: feat(mongodb): Partition data across multiple databases  https://review.openstack.org/4595201:32
*** kgriffs is now known as kgriffs_afk01:42
*** malini is now known as malini_afk02:05
*** ayoung has quit IRC03:59
*** tvb has joined #openstack-marconi04:46
*** tvb has quit IRC04:58
*** tvb|afk has joined #openstack-marconi05:54
*** tvb|afk has quit IRC05:54
*** tvb|afk has joined #openstack-marconi05:54
*** flaper87|afk is now known as flaper8706:43
*** ykaplan has joined #openstack-marconi07:59
*** yassine has joined #openstack-marconi08:11
*** tvb|afk has quit IRC08:14
*** tvb|afk has joined #openstack-marconi08:14
*** tvb|afk has quit IRC08:14
*** tvb|afk has joined #openstack-marconi08:14
*** flaper87 is now known as flaper87|afk08:55
*** flaper87|afk is now known as flaper8708:56
*** ykaplan has quit IRC10:13
*** flaper87 is now known as flaper87|afk10:17
*** flaper87|afk is now known as flaper8710:29
*** ykaplan has joined #openstack-marconi10:39
*** flaper87 is now known as flaper87|afk11:16
*** flaper87|afk is now known as flaper8711:21
*** nosnos has quit IRC11:33
*** tedross has joined #openstack-marconi11:45
*** yassine_ has joined #openstack-marconi11:52
*** yassine has quit IRC11:56
*** ykaplan has quit IRC12:16
*** flaper87 is now known as flaper87|afk12:20
*** flaper87|afk is now known as flaper8712:33
*** kgriffs_afk is now known as kgriffs12:44
*** mpanetta has joined #openstack-marconi12:59
openstackgerritKurt Griffiths proposed a change to openstack/marconi: feat(mongodb): Partition data across multiple databases  https://review.openstack.org/4595213:04
*** ykaplan has joined #openstack-marconi13:08
mpanettaMornin all.13:11
flaper87mpanetta: good morning13:11
mpanettaflaper87: How goes?13:12
flaper87mpanetta: good, what about you?13:12
mpanettaNot too bad, stayed up way too late last night heh.13:12
*** jcru has joined #openstack-marconi13:14
*** yassine has joined #openstack-marconi13:15
flaper87mpanetta: hahahaha, \_/? here, take some coffee13:15
mpanettaflaper87: Thanks hehe13:16
*** yassine_ has quit IRC13:19
* flaper87 is used to stay up too late13:20
flaper87dunno if it's a good thing but I'm not used to sleep much13:21
mpanettaI used to be.13:21
* flaper87 gets bored while sleeping13:21
mpanettaWas working on PCB design, and listening to music.13:21
*** oz_akan_ has joined #openstack-marconi13:22
mpanettaflaper87: Do you know if all the patches for proxy have been merged yet?13:24
flaper87mpanetta: not all of them13:24
flaper87there are some pending13:25
flaper87I'll review them in a bit13:25
mpanettaOk, cool.13:25
*** acabrera has joined #openstack-marconi13:33
*** acabrera is now known as alcabrera13:34
flaper87alcabrera: good morning13:34
alcabreraflaper87: :)13:34
kgriffsanybody know the trick to passing multiple params to a test function with ddt?13:38
kgriffsor do you just pass in a tuple to a single param and explode it yourself?13:39
alcabrerakgriffs: I've seen the technique in malini's tests before, hmm...13:40
flaper87mmh, not me13:40
alcabreraI'd need to look it up.13:41
alcabrerakgriffs: found it!13:41
flaper87kgriffs: when you get a chance, could you take a look here: https://review.openstack.org/#/q/status:open+project:openstack/python-marconiclient,n,z13:42
flaper87those patches need some love :(13:42
alcabreraCheck out unit/queues/transport/wsgi/test_claims:7613:42
kgriffsI got a dentist appt soon, but will be back and can review stuff after that13:42
flaper87kgriffs: thank you!13:42
flaper87and good luck with that!13:42
alcabreraflaper87: I'm looking - wait, I've already reviewed these. :P13:42
kgriffsalcabrera: ah, I see. must be args, kwargs13:42
alcabrerakgriffs: dentist, yikes! Hope that goes well. :)13:42
flaper87alcabrera: yup :P13:43
kgriffsme too!13:43
flaper87I was very direct!13:43
*** malini_afk is now known as malini13:44
alcabreraflaper87, kgriffs: Whenever you can, I'm hoping to have these three proxy patches get a sanity check and a merge (https://review.openstack.org/#/q/status:open+marconi+branch:master+topic:bp/placement-service,n,z). :D13:48
alcabreraI'm still ironing out wrinkles on the latest proxy patch, but those three have been through the review gauntlet. :)13:49
kgriffsalcabrera: replied to your question on https://review.openstack.org/#/c/45952/13:50
kgriffsnow i *really* gotta run13:50
alcabrerakgriffs: take care!13:50
*** kgriffs is now known as kgriffs_afk13:50
mpanettaalcabrera: Is there a patchset ready to test yet?  Or am I just too excited to test?13:51
*** amitgandhi has joined #openstack-marconi13:52
alcabrerampanetta: this one (https://review.openstack.org/#/c/48737/) has been ready since yesterday. :D13:54
alcabrerampanetta: It will get a few modifications API-wise (moving to PATCH from PUT for changing hosts and weight on an existing partition), but the core of it is solid.13:54
mpanettaHmm, we do not have a test for the admin interface yet AFAIK.13:54
mpanettaI think we are only testing passthrough.13:55
mpanettaI assume we will manually be creating the partitions for test.13:55
alcabrerampanetta: That's correct on both counts: no tests for admin interface (except unit tests), and partitions will be manually created.13:55
alcabreraI'd be interested in the results of passthrough testing, in any case.13:56
mpanettaI can't wait until I get a chance to start working on automating these tests.13:56
mpanettaWhich reminds me...  Is there a document that describes how to hook in to the system, like the garritbot does here so I can get commit notfications programmatically?13:57
alcabreraNot as far as I know.13:58
alcabreraStarring the github repo for patches merged to master?13:58
mpanettaThat just sends emails tho doesn't it?13:58
alcabreraThat doesn't give you news on patches submitted for review, though.13:58
mpanettaThat is all done through launchpad?13:59
alcabrerampanetta: Github exposes an RSS feed for repos you're watching. That's independent of launchpad.13:59
flaper87you can also watch projects on gerrit14:00
flaper87it'll send you emails14:00
flaper87but it's a bit spammy14:00
alcabrerahmmm, I've yet to try that.14:00
alcabreraI don't need more spam, haha. >.>14:00
flaper87damn, #@!$#@%! connection14:02
mpanettaI didn't see ya drop14:04
mpanettaActually I am surprsed I have not dropped yet today...14:04
flaper87mpanetta: I'm just like that! :P14:04
mpanettaflaper87: Or you use an IRC proxy, or have it in a screen somewhere :P14:05
flaper87mpanetta: damn, you figured it out! :P14:05
flaper87I think my former ISP is taking revenge on me in these last days of contract14:05
flaper87I can picture a guy unplugging my wire every 5 mins14:06
* mpanetta loves tuesdays at rackspace14:06
mpanettaMmm frut tuesdays!14:06
*** Alex_Gaynor has quit IRC14:15
*** gleicon___ has quit IRC14:17
*** tvb|afk has quit IRC14:18
openstackgerritZhihao Yuan proposed a change to openstack/marconi: fix(proxy): forward marconi headers  https://review.openstack.org/4762814:24
zyuankgriffs_afk: i failed to find a parameter name better than `d` :(14:24
malinimpanetta, alcabrera: Sorry missed the discussion on proxy tests14:26
maliniI have somewhat of a skeleton for the proxy functional tests14:27
malini+ ideas on enhancement, make them runnable with tox14:27
malinithe second part of it is going to take some time & wont be ready in the next week or so14:27
maliniBut the basic API tests (tht assume we have a partition to play around with on the tests) are almost there14:28
malini& what do you call a pass through testing ?14:29
alcabrera malini: When I hear "pass thorugh testing", I imagine the functional tests for the queues API being run so that they're pointed at a proxy.14:32
alcabreraThen, it just checks that the proxy is able to behave like a marconi-server (given that the proxy has marconi-server partitions registered prior to the test runnning).14:33
malinialcabrera: assuming the proxy is already set up (or) the tests spin up the proxy ?14:33
malinialcabrera: in tht case, I have most of pass through testing already in the works14:34
maliniI am almost 75% through it14:34
alcabreramalini: assuming the proxy is already set up would probably be easiest in the short term, but having the tests launch two marconi-servers, an admin proxy, a proxy forwarder, registering the partitions, and then executing would be better for automation in the long run.14:34
alcabreraawesome, malini!14:34
* flaper87 is working on the config thing14:34
flaper87wish me luck14:34
malinialcabrera: exactly..you read my mind :)14:34
flaper87if you don't hear from me every 5mins, CALL 91114:35
flaper87no wait, it'll take them way to much to get here14:35
maliniflaper87: do I need the area code ? ;)14:35
flaper87CALL 11814:35
malinigood luck!14:35
alcabreraflaper87: lol, good luck. :)14:37
flaper87btw, do you guys follow OSReactions on twitter?14:37
flaper87we're not publishing all our posts on the planet14:37
flaper87you know, we don't want to spam OS Planet14:37
flaper87but twitter seems like a spamable stream14:38
alcabreraflaper87: not following yet. :P14:38
alcabreraflaper87: now I'm following, haha.14:39
flaper87alcabrera: not that you should, just wanted to let you know14:39
flaper87alcabrera: awesome14:39
alcabreraflaper87: haha, of course. A little openstack amusement won't hurt my twitter stream. :D14:41
*** Alex_Gaynor has joined #openstack-marconi14:41
*** ayoung has joined #openstack-marconi14:44
zyuanhttps://review.openstack.org/#/c/45952/ ready to merge, afaics14:45
*** mpanetta_ has joined #openstack-marconi14:58
*** mpanetta has quit IRC14:58
*** mpanetta_ is now known as mpanetta15:24
*** kgriffs_afk is now known as kgriffs15:29
alcabrerakgriffs: o/15:30
* flaper87 just saw something15:31
alcabreraflaper87: ?15:31
maliniit better be UFO15:32
maliniThe aliens got him :(15:33
alcabreramalini: we15:34
alcabreramalini: no more flaper87. With aliens, he's in a whole new world now. :P15:34
maliniwas a good guy..15:35
mpanettaAh crap, how am I gonna test now? :P15:35
flaper87soooooo: What's wrong here? https://github.com/openstack/marconi/blob/master/marconi/queues/bootstrap.py#L4815:35
flaper87and line 6015:35
alcabreraI see.15:35
* flaper87 fixes that NOW!15:36
alcabrerathat things that get missed when porting projects to new directory structures~15:36
flaper87we were lucky because we didn't fix that here neither https://github.com/openstack/marconi/blob/master/setup.cfg#L2915:36
alcabreralol I just checked15:36
alcabreraThings continued to work silently, deviously.15:36
kgriffsmpanetta: you see the new marconi.version.verify_sha function?15:37
mpanettakgriffs: Yes, TYV!15:37
mpanettaer TYVM!15:37
kgriffshope it helps. :D15:38
mpanettaOh crap, I just checked out and installed master on test env heh15:38
flaper87malini: btw, LOL @ UFO15:38
maliniglad to have you back flaper87 ;)15:39
mpanettaHe could have at least brought a UFO back with him...15:41
flaper87you realize you wiped your laptop when git review gives you "Command not found"15:42
openstackgerritFlavio Percoco proposed a change to openstack/marconi: Fix queue's packages namespaces  https://review.openstack.org/4917415:42
flaper87very quick review ^15:42
flaper87alcabrera: kgriffs ^15:42
zyuankgriffs: ping15:43
kgriffsmpanetta: I'd love to get a perf test on this sucker - https://review.openstack.org/#/c/45952/15:43
kgriffs(before it is merged)15:43
zyuankgriffs: i said in comments, the name 'capitalized' is like 'sorted'15:43
mpanettaEr, um.  Will I have time before y'all are done mergng for main test?15:44
zyuanit can't be a verb15:44
zyuankgriffs: and i can't find a name better than 'd'; can you give me a suggestion?15:44
kgriffsno, you can leave as-is15:44
kgriffsI forgot that python uses that convention15:45
zyuan(dict -- name collision; 'dic'? oh no....)15:45
alcabreraflaper87: +2'd15:45
alcabrerazyuan: how about headers?15:46
zyuanyou had headers_dict somewhere15:46
zyuanit's ok... i mean 'headers'15:46
kgriffsflaper87: https://review.openstack.org/#/c/48287/7/marconiclient/errors.py15:47
kgriffswhat is the advantage of that iml vs. just calling super and passing in the msg15:47
zyuanerr, not quite...15:47
zyuanthis function is generic, it does not know whether the input is headers...15:48
zyuanmaybe i just say 'dictionary'15:48
flaper87kgriffs: I guess message formatting for subclasses and more specific errors15:48
flaper87notice the *maybe*15:49
alcabrerazyuan: given that it is generic, it does make assumptions about the format of the input strings. They are of the format "a-b".15:49
flaper87I'm not very happy with that, I think I pulled that in when porting some things from oslo.messaging and then never used it.15:49
kgriffsif you like, you could make it generic by allowing the caller to pass a sep string15:49
kgriffsand it could default to "-"15:49
zyuanalcabrera: ah, yea15:50
kgriffsbut, not sure if it's worth it (YAGNI)15:50
flaper87kgriffs: I prefer the caller to format the message itself15:50
zyuanstop... lets just call it headers...15:50
flaper87kgriffs: I don't think it is15:50
alcabrerazyuan: +115:50
flaper87it's a left over from a very nasty copy / paste15:50
kgriffsflaper87: heh. paste happens.15:50
flaper87kgriffs: yeaaah :P15:51
flaper87kgriffs: let me do that now that you're reviewing15:51
kgriffsflaper87: how accurate is this? http://bhavin.directi.com/rabbitmq-vs-apache-activemq-vs-apache-qpid/15:51
kgriffsit's a few years old, so I am just wondering if RabbitMQ has fixed some of those warts in the meantime15:52
*** yassine has quit IRC15:52
flaper87kgriffs: Haven't read that one, I'll take a look and let you know15:53
kgriffskewl, thanks15:53
openstackgerritZhihao Yuan proposed a change to openstack/marconi: fix(proxy): forward marconi headers  https://review.openstack.org/4762815:54
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Add a base transport class  https://review.openstack.org/4828715:54
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Add prepare_request function to requests.py  https://review.openstack.org/4866915:54
flaper87kgriffs: ^15:55
flaper87alcabrera: ^15:55
flaper87alcabrera: you just need to look at this one: https://review.openstack.org/#/c/48287/815:55
alcabrerawoot, reviews15:55
flaper87no big changes, just applied kgriffs suggestion re MarconiError15:56
alcabreraflaper87: +2'd :)15:56
flaper87alcabrera: thank you. sir.15:56
alcabrerazyuan: +2'd15:57
*** malini has left #openstack-marconi16:00
*** malini has joined #openstack-marconi16:03
alcabreraflaper87, kgriffs: I take back what I said about python-jsonschema being slow. I learned how to use it better and ran some new tests with the increased knowledge.16:12
flaper87alcabrera: awesome, so glad to hear that! Any plans to use it instead of our current impl ?16:12
alcabreraif you use jsonschema.validate(data, schema), it's thoroughly slow (1.46ms per check). If you first instantiate a validator instance (jsonschema.Draft4Validator(schema), then do instance.validate(data) => 46.2 us per loop16:13
alcabreraflaper87: I'm tinkering with it for the PATCH implementation of proxy/partitions16:13
*** gleicon___ has joined #openstack-marconi16:14
*** flaper87 is now known as flaper87|afk16:21
kgriffsalcabrera: perf on pypy?16:24
kgriffsalso, if it looks like something we want to use, we should have a hackday and make it scream16:25
alcabrerakgriffs: lemme check that out on pypy :D16:26
*** mpanetta has quit IRC16:27
*** mpanetta has joined #openstack-marconi16:27
*** mpanetta is now known as mpanetta_lunch16:29
*** ykaplan has quit IRC16:36
alcabrerakgriffs: the results are in: http://paste.openstack.org/show/47779/16:37
alcabreraA proper microbenchmarking across pypy-2.1, python-2.7.5, and python-3.3.216:37
kgriffsu should make that a gist16:38
kgriffsyou know, for posterity16:38
kgriffsand for reference when we get around to making an I <3 microbenchmarking website16:38
kgriffshmm, 8 us is still pretty slow16:39
kgriffsthere must be a way to optimize jsonschema16:39
* kgriffs adds that to hackday brainstorming list16:39
alcabrerakgriffs: done: https://gist.github.com/cabrera/6781406 :)16:40
*** ykaplan has joined #openstack-marconi16:41
kgriffsbtw… https://code.google.com/p/google-mobwrite/source/browse/trunk/lib/json_validator.py16:41
kgriffsfor simple validation16:41
kgriffswould you mind benching that as well?16:41
kgriffsthat is what I used in RSE16:41
alcabreraI'm checking it out now.16:43
kgriffszyuan: https://review.openstack.org/#/c/47628/6/marconi/proxy/resources/queues.py16:43
kgriffsI was just thinking, it may be nice to add a # NOTE(zyuan) comment16:43
kgriffsexplaining why capitalized is necessary16:43
kgriffsand also maybe a TODO to remove it once normalization is added to falcon16:43
alcabrerakgriffs: I'm running into an interesting result on benching that already. json.dumps is faster than json_validator.is_valid on all interpreters I have.16:47
alcabrera~10x faster on pythons, and about .25 us faster on pypy.16:47
alcabrerapasting now16:48
kgriffsreally? seems like it was fast back in RSE days when i tested.16:48
kgriffsare you using simplejson?16:48
alcabrerano, I'm using native json16:48
alcabreraimport json16:48
kgriffsthat is curious. even faster on 2.6 ?16:48
alcabreraI haven't checked 2.6. I wonder if I have that installed...? >>>16:48
kgriffsoh, and results may differ with complex docs16:49
kgriffsyou should try something with nested objects and stuff16:49
kgriffsjust be sure16:49
alcabrerasure thing16:49
alcabrerakgriffs: found an error in my benchmark - retrying. This invalidates all previous data, heh.16:51
alcabreraI was doing something along the lines of doc = json.dumps(...); %timeit json.dumps(doc) [double dumping]16:51
kgriffsobjectid.ObjectId() is slow16:59
alcabrerakgriffs: http://paste.openstack.org/show/47780/ (json vs. json_validator)17:00
kgriffssetting manipulate=False and letting mongod generate the id is 10 us faster17:00
kgriffs(slow is a relative term)17:00
kgriffs(one man's fast is another man's slow, esp. when that other man is kgriffs)17:00
alcabreralol, indeed17:01
alcabrerajson_validator looks wicked slow compared to json17:01
openstackgerritA change was merged to openstack/marconi: Fix queue's packages namespaces  https://review.openstack.org/4917417:01
alcabrerastill in the us range, got about 30-40x slower than json.dumps17:01
kgriffsnot sure why I thought it was faster. unless it is a py26 thing17:02
alcabrerakgriffs: seems to be a python 2.6 thing: http://stackoverflow.com/questions/5158773/how-to-speed-up-json-decode-and-encode-operations-in-python17:03
alcabreraI imagine they ported the improvements in simplejson to the stdlib from 2.6 -> 2.717:03
*** reed has joined #openstack-marconi17:08
*** mpanetta_lunch is now known as mpanetta17:12
torgomaticyeah, simplejson on 2.6 is way faster than stdlib, but in 2.7, they're fairly equivalent17:19
torgomaticI did some benchmarking at one point; results are here: https://gist.github.com/smerritt/608382817:19
*** ykaplan has quit IRC17:21
alcabreratorgomatic: thanks for sharing! Via reading your gist, I also learned about pypi:benchmark and pypi:benchmarker17:26
alcabreracool, cool. :D17:26
mpanettakgriffs: TypeError: verify_sha() takes exactly 1 argument (0 given)17:27
mpanettaUm, is that a class method or an instance method? heh17:27
mpanettaI need to pass in the expected commit sha?17:29
kgriffsit is a string17:29
kgriffsyou can see the dosctring either from a REPL (help(marconi.version.verify_sha)) or by browsing teh source.17:30
kgriffsprotip: ^^^17:31
mpanettahaha, yeah that is how I figured it out17:31
mpanettaMmmm source!17:31
mpanettaCrap... Quote interpolation strikes again...17:34
kgriffsLuuuuuke. Use the source, Luke! You must learn the ways of the source, if you're to come with me to OpenStack.17:37
*** tvb|afk has joined #openstack-marconi17:57
*** JRow has joined #openstack-marconi18:10
*** JRow has left #openstack-marconi18:10
kgriffsflaper87|afk - pls review https://review.openstack.org/#/c/45952/18:25
openstackgerritZhihao Yuan proposed a change to openstack/marconi: fix(proxy): forward marconi headers  https://review.openstack.org/4762818:27
*** ykaplan has joined #openstack-marconi18:46
zyuankgriffs: what is the format of project ID?19:08
zyuankgriffs: i think it make sense to validate client ID if we know it's UUID19:08
kgriffsit is either an integer or a uuid19:08
zyuanbut project ID is deployment-dependent; it can be anything19:08
zyuanfor example, if marconi is deployed outside rackspace, it really can be anything19:09
kgriffsi asked on the dev list, and they said it used to be a number (auto-inc primary key, I suspect), but in recent releases it is now a uuid19:09
kgriffsmeaning, the openstack dev list19:09
zyuankgriffs: ^^ which ID?19:09
kgriffsproject ID19:10
zyuanthen it's different from rackspace19:10
kgriffslet me forward the email19:10
kgriffsso, looks like the best we can do is check for a sane length19:11
kgriffsi suspect that only allowing ASCII would also be fine - can't image why people would generate an ID with non-ascii chars19:12
zyuanUUID is fairly easy; just uuid.UUID(hex='...') you done19:12
kgriffszyuan: forwarded the email19:12
zyuanthen... what's the desized client ID?19:13
zyuanUUID, which version? another thing is, if we know it's UUID, we can store the bytes form instead of the hex form in DB19:13
zyuan(i like binary as impl :)19:14
zyuankgriffs: ^^19:14
*** malini is now known as malini_afk19:21
*** EmilienM has quit IRC19:27
alcabrerazyuan, kgriffs: project-ID is deployment-dependent - it could theoretically be any string, and this may be moreso the case based on somethings I've heard about keystone.19:27
*** EmilienM has joined #openstack-marconi19:30
alcabrerampanetta: proxy notes sent. :)19:31
mpanettaalcabrera: Danke :)19:33
mpanettaalcabrera: so I should run marconi as user marconi now, correct?  Currently we are running as root.19:35
mpanettaer s/marconi/proxy/19:35
alcabrerampanetta: I've always run an a non-root user. :)19:36
mpanettaalcabrera: It is a good practice to have19:36
mpanettaI don't even allow root logins on my systems that have internet access...  Only sudo.19:37
alcabreraso there should be no restrictions on running as non-root, as long as you ensure that the logging directory in the confs points to a directory the user can write to.19:37
mpanettaprobably will set it to ~marconi/log/19:37
mpanettaActually if it is looking in ~/.marconi I can still run as root for test...19:38
mpanettaso I can keep it as /usr/local19:38
mpanettaas /var/log19:39
mpanettaevery time I see pbr I think 'Pabst Blue Ribbon'... And I don't even drink beer...19:43
alcabrerampanetta: lol19:55
mpanettaalcabrera: Oh, I just saw your email, feel free to answer my latest question *after* you are done fixing install ;)19:57
*** vkmc has joined #openstack-marconi20:01
*** vkmc has quit IRC20:01
*** vkmc has joined #openstack-marconi20:01
zyuancan i get this merged btw? https://review.openstack.org/#/c/47628/20:04
alcabrerampanetta: I think I figured it out. Doing 'pip install .' seems to work reliably, though it seemed like I needed to log off to get the environment to register that I installed pbr. >.>20:04
alcabrerazyuan: +2'd again20:06
mpanettaOh ok, that was what we are doing already then...20:06
kgriffszyuan: thanks for updating the commit message20:31
*** tvb|afk has quit IRC20:31
*** flaper87|afk is now known as flaper8720:32
kgriffshowever, i am not seeing a mention in there as to why the normalization is necessary. I'd like to see a mention of the fact that resp.headers are lowercase and falcon expects them to be captalized - maybe even a todo to remove the transform once falcon is updated to support normalization itself20:32
kgriffsthe commit message is fine20:32
kgriffsi'd like to see that extra info added to the NOTE20:33
kgriffssorry to nitpick on this, but I can see someone some day getting tripped up by this code without having a proper hint20:33
openstackgerritZhihao Yuan proposed a change to openstack/marconi: fix(proxy): forward marconi headers  https://review.openstack.org/4762820:35
flaper87kgriffs: +2'd thanks for the hard work there20:38
flaper87oz_akan_: thank you as well!20:38
flaper87oz_akan_: great idea!20:38
oz_akan_what did I do?20:39
kgriffsoz_akan gets kudos for the algorithm. His idea to perform the partition lookup via an  array index was rather elegant.20:40
flaper87kgriffs: LOOOL20:40
zyuanaha, mongo can store UUID directlyt20:41
oz_akan_oh, yes, that is very great, pure awesomeness, as if I had 10 ideas to choose from :D20:41
kgriffsoz_akan_: what can you say, you're a natural!20:42
oz_akan_thanks to kgriffs for getting it done. Ideas don't really matter unless they see sun light20:43
oz_akan_zyuan: as well for the idea to have only one db for queues20:44
alcabrerakgriffs, oz_akan_: awesome work, guys! multidb is pretty sweet for the mongodb storage layer. I'm sure future storage drivers will be able to utilize a similar technique.20:44
oz_akan_..and multiple for messages20:44
kgriffsalso thanks to zyuan20:46
alcabrerazyuan: +120:46
kgriffsyou guys are almost as awesome as this guy. almost. :D20:46
kgriffsbtw, he is doing all of that in real time - no video tricks20:47
flaper87zyuan: +120:47
openstackgerritA change was merged to openstack/marconi: feat(mongodb): Partition data across multiple databases  https://review.openstack.org/4595220:47
kgriffsjust like oz_akan_ and zyuan: no tricks. they are 100% for realz20:48
kgriffsthat's the last schema-changing patch, hoooray!20:49
kgriffsthanks again for everyone who helped git-r-done20:49
zyuantoo many sudo rm /var/lib/db....20:49
zyuanit's not the last20:50
zyuanclient-id as uuid20:50
kgriffsalcabrera: https://review.openstack.org/#/c/47628/20:52
kgriffsready to rock on that20:52
alcabrerakgriffs: flaper87 beat me to it. ;)20:54
*** ykaplan has quit IRC20:55
* flaper87 sheath his sword and bows...20:56
alcabreraflaper87: +10 XP20:56
alcabreraflaper87: +2 GP20:56
openstackgerritA change was merged to openstack/marconi: fix(proxy): forward marconi headers  https://review.openstack.org/4762820:56
*** tedross has quit IRC21:06
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: proxy: mirror structure of marconi queues + bootstrap  https://review.openstack.org/4764221:11
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: separate config for queues and proxy  https://review.openstack.org/4833321:12
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: split proxy API into admin and public apps  https://review.openstack.org/4849521:15
*** jcru has quit IRC21:15
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat (proxy/admin): allow partition modifications  https://review.openstack.org/4873721:22
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat (proxy/admin): allow partition modifications  https://review.openstack.org/4926221:23
*** ayoung has quit IRC21:25
*** vkmc has quit IRC21:30
*** flaper87 is now known as flaper87|afk21:30
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: split proxy API into admin and public apps  https://review.openstack.org/4849521:31
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat (proxy/admin): allow partition modifications  https://review.openstack.org/4926221:33
alcabreraflaper87|afk, kgriffs, zyuan: The rebasing frenzy is over. Whenever you guys get the chance, I'd love a sanity check on the first three patches and a review on the last one (PATCH /v1/partitions/{partition}). :D21:35
alcabreraI'm out for the night. Take care, guys.21:35
*** alcabrera has quit IRC21:36
*** oz_akan_ has quit IRC22:19
*** oz_akan_ has joined #openstack-marconi22:19
*** oz_akan_ has quit IRC22:24
*** flaper87|afk is now known as flaper8722:27
*** mpanetta_ has joined #openstack-marconi22:28
*** amitgandhi has quit IRC22:29
*** mpanetta has quit IRC22:32
flaper87kgriffs: ping22:32
flaper87kgriffs: could you take another look at client's patches ? https://review.openstack.org/#/q/status:open+project:openstack/python-marconiclient,n,z22:32
flaper87I'd like to get those merge22:35
flaper87and keep working on the client tomorrow22:36
kgriffsflaper87: sure22:40
flaper87kgriffs: thank youuuuuuuu!!!! :D22:41
kgriffsmy pleasure22:42
*** mpanetta_ is now known as mpanetta22:43
*** mpanetta has quit IRC22:55
openstackgerritA change was merged to openstack/python-marconiclient: Restructure client code  https://review.openstack.org/4822322:55
openstackgerritKurt Griffiths proposed a change to openstack/marconi: fix(wsgi): Non-ASCII characters in the queue name trigger 500 status  https://review.openstack.org/4877622:56
*** kgriffs is now known as kgriffs_afk23:11
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Add a base transport class  https://review.openstack.org/4828723:13
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Add prepare_request function to requests.py  https://review.openstack.org/4866923:13
*** reed has quit IRC23:13
*** oz_akan_ has joined #openstack-marconi23:30
*** flaper87 is now known as flaper87|afk23:30
*** oz_akan_ has quit IRC23:34
openstackgerritA change was merged to openstack/python-marconiclient: Add common http class  https://review.openstack.org/4823623:40

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