| *** kgriffs is now known as kgriffs_afk | 00:08 | |
| *** oz_akan_ has joined #openstack-marconi | 00:09 | |
| *** malini is now known as malini_afk | 00:25 | |
| *** amitgandhi1 has joined #openstack-marconi | 00:36 | |
| *** amitgandhi has quit IRC | 00:40 | |
| *** malini_afk is now known as malini | 00:44 | |
| *** oz_akan_ has quit IRC | 00:48 | |
| *** oz_akan_ has joined #openstack-marconi | 00:49 | |
| *** oz_akan_ has quit IRC | 00:53 | |
| *** oz_akan_ has joined #openstack-marconi | 00:53 | |
| *** kgriffs_afk is now known as kgriffs | 00:59 | |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Fix Functional Tests https://review.openstack.org/46851 | 01:00 |
|---|---|---|
| *** kgriffs is now known as kgriffs_afk | 01:13 | |
| *** vkmc has quit IRC | 01:20 | |
| *** jergerber has quit IRC | 01:23 | |
| *** jergerber has joined #openstack-marconi | 01:23 | |
| *** jergerber has quit IRC | 01:43 | |
| *** amitgandhi1 has quit IRC | 01:56 | |
| *** malini has left #openstack-marconi | 01:57 | |
| *** oz_akan_ has quit IRC | 01:57 | |
| *** kgriffs_afk is now known as kgriffs | 02:04 | |
| *** kgriffs is now known as kgriffs_afk | 02:14 | |
| *** ayoung is now known as ayoung-ZzZzZzZ | 02:57 | |
| *** ayoung-ZzZzZzZ has quit IRC | 03:00 | |
| *** kgriffs_afk is now known as kgriffs | 03:05 | |
| *** kgriffs is now known as kgriffs_afk | 03:14 | |
| *** kgriffs_afk is now known as kgriffs | 04:05 | |
| *** rektide has quit IRC | 04:09 | |
| *** kgriffs is now known as kgriffs_afk | 04:14 | |
| *** kgriffs_afk is now known as kgriffs | 05:05 | |
| *** kgriffs is now known as kgriffs_afk | 05:15 | |
| *** kgriffs_afk is now known as kgriffs | 06:06 | |
| *** russell_h has quit IRC | 06:09 | |
| *** al-maisan has quit IRC | 06:09 | |
| *** al-maisan has joined #openstack-marconi | 06:09 | |
| *** openstack has joined #openstack-marconi | 06:18 | |
| *** gleicon___ has joined #openstack-marconi | 06:20 | |
| *** gleicon__ has quit IRC | 06:23 | |
| *** gleicon___ is now known as gleicon__ | 06:24 | |
| *** oz_akan_ has joined #openstack-marconi | 06:29 | |
| *** oz_akan_ has quit IRC | 06:45 | |
| *** gleicon__ has quit IRC | 06:53 | |
| *** gleicon__ has joined #openstack-marconi | 06:53 | |
| *** kgriffs_afk is now known as kgriffs | 07:06 | |
| *** kgriffs is now known as kgriffs_afk | 07:16 | |
| *** ykaplan has joined #openstack-marconi | 07:27 | |
| *** oz_akan_ has joined #openstack-marconi | 07:42 | |
| *** oz_akan_ has quit IRC | 07:46 | |
| *** gordonsim has joined #openstack-marconi | 07:50 | |
| *** kgriffs_afk is now known as kgriffs | 08:07 | |
| *** cthulhup has joined #openstack-marconi | 08:11 | |
| cthulhup | greetings all | 08:12 |
| *** kgriffs is now known as kgriffs_afk | 08:16 | |
| *** ykaplan has quit IRC | 08:49 | |
| *** flaper87|afk is now known as flaper87 | 09:03 | |
| *** kgriffs_afk is now known as kgriffs | 09:07 | |
| *** kgriffs is now known as kgriffs_afk | 09:17 | |
| openstackgerrit | A change was merged to openstack/marconi: chore: Update Falcon dep to allow version 0.1.7 https://review.openstack.org/46775 | 09:33 |
| *** kgriffs_afk is now known as kgriffs | 10:08 | |
| *** ykaplan has joined #openstack-marconi | 10:11 | |
| *** kgriffs is now known as kgriffs_afk | 10:17 | |
| *** oz_akan_ has joined #openstack-marconi | 10:54 | |
| *** oz_akan_ has quit IRC | 10:58 | |
| *** kgriffs_afk is now known as kgriffs | 11:08 | |
| *** kgriffs is now known as kgriffs_afk | 11:18 | |
| *** whenry has quit IRC | 11:25 | |
| *** tedross has joined #openstack-marconi | 11:35 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Update pbr dependency to latest https://review.openstack.org/46260 | 11:36 |
| *** kgriffs_afk is now known as kgriffs | 11:38 | |
| *** kgriffs is now known as kgriffs_afk | 11:48 | |
| *** ykaplan has quit IRC | 12:05 | |
| *** cppcabrera has joined #openstack-marconi | 12:15 | |
| cppcabrera | Good morning. :) | 12:19 |
| *** ykaplan has joined #openstack-marconi | 12:20 | |
| flaper87 | cppcabrera: GOOOD MORNING | 12:21 |
| flaper87 | :D | 12:21 |
| cppcabrera | flaper87: o/ | 12:21 |
| cppcabrera | :) | 12:21 |
| flaper87 | how are you doing? | 12:21 |
| cppcabrera | A little groggy, but otherwise pretty good. How about you? | 12:22 |
| * flaper87 just learnt a new word: groggy | 12:24 | |
| flaper87 | doing well, at the office. We'll have our monthly PythonMilan meetup today | 12:24 |
| cppcabrera | hehe | 12:25 |
| cppcabrera | sweet! What kinds of topics will be covered at the meetup? Do they plan out talks and such before hand at this meet up? | 12:25 |
| flaper87 | cppcabrera: yeah, today we'll talk about OpenStack | 12:27 |
| flaper87 | guess who's talking | 12:27 |
| * cppcabrera points at flaper87 | 12:27 | |
| flaper87 | Actually, I'm the one organizing these meetups, we use Red Hat's Milan office | 12:27 |
| cppcabrera | :P | 12:27 |
| flaper87 | :D | 12:27 |
| cppcabrera | haha | 12:28 |
| flaper87 | http://www.meetup.com/Python-Milano/events/133133302/ | 12:28 |
| flaper87 | cppcabrera: talks poll: http://www.meetup.com/Python-Milano/polls/867752/ | 12:28 |
| cppcabrera | I can almost make out the meaning of most of the words on that page. Approximation by Spanish helps there. :) | 12:29 |
| cppcabrera | Openstack sure got a lot of votes, | 12:30 |
| flaper87 | :D | 12:31 |
| flaper87 | YEAHHH!!! | 12:31 |
| cppcabrera | so you're going to be talking about all of openstack at the meetup? That's a pretty big topic, flaper87! | 12:34 |
| cppcabrera | Fitting that into an hour sounds like a challenge. | 12:34 |
| flaper87 | yeah, it is. I'll try to cover the main modules and then say something about openstack's community | 12:36 |
| cppcabrera | -infra needs a shout out. It's a pretty unique "feature" of the openstack community. | 12:37 |
| * flaper87 is restructuring marconis source code | 12:38 | |
| flaper87 | what a painful process | 12:39 |
| *** kgriffs_afk is now known as kgriffs | 12:39 | |
| flaper87 | cppcabrera: +1 | 12:40 |
| cppcabrera | restructuring as per: marconi/{api, proxy, notifications} ? | 12:40 |
| flaper87 | cppcabrera: yeah | 12:40 |
| flaper87 | mmh, wait | 12:40 |
| flaper87 | wasn't it queues ? | 12:40 |
| cppcabrera | It was. :P | 12:40 |
| cppcabrera | I'd forgotten the name. | 12:40 |
| flaper87 | ah ok | 12:40 |
| flaper87 | phew | 12:40 |
| cppcabrera | lol | 12:40 |
| flaper87 | cppcabrera: re merging proxy patches | 12:41 |
| cppcabrera | I was just about to ask! :D | 12:41 |
| flaper87 | I agree with you, I'd like to see some of the comments in the first patch addressed before merging it, though | 12:41 |
| flaper87 | they seem to be very simple | 12:41 |
| flaper87 | and very easy to forget too | 12:42 |
| flaper87 | :D | 12:42 |
| flaper87 | at least for me | 12:42 |
| flaper87 | thoughts? | 12:42 |
| flaper87 | by first patch, I mean "marconi proxy" the one I already reviewed | 12:42 |
| cppcabrera | So that's - API version in routes, body_encoded, and... ensure_ascii | 12:42 |
| flaper87 | cppcabrera: I think so, yes | 12:43 |
| cppcabrera | quick question on body_encoded - how is that meant to be used? AFAICT, I've had to do ```response.body = <thing>, response.body = response.body_encoded```, but it seems that it might be implicitly performed. | 12:44 |
| flaper87 | you just have to set response.body with you're non encoded body | 12:46 |
| flaper87 | falcon will do the rest | 12:46 |
| flaper87 | right before sending the request | 12:46 |
| flaper87 | and if you need your encoded body | 12:47 |
| flaper87 | you can use response.body_encoded | 12:47 |
| * flaper87 is using Enter as a separator | 12:47 | |
| flaper87 | :P | 12:47 |
| *** kgriffs is now known as kgriffs_afk | 12:48 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Change Marconi's codebase structure https://review.openstack.org/46939 | 12:49 |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Change Marconi's codebase structure https://review.openstack.org/46939 | 12:49 |
| flaper87 | there it goes | 12:49 |
| flaper87 | lets hope for the best | 12:49 |
| flaper87 | :D | 12:49 |
| flaper87 | that change shouldn't break marconi.proxy | 12:50 |
| flaper87 | actually, it ******SHOULD****** all work | 12:50 |
| flaper87 | but, you know... | 12:50 |
| *** ykaplan has quit IRC | 12:51 | |
| cppcabrera | brb | 12:51 |
| flaper87 | sure | 12:51 |
| flaper87 | gordonsim: ping | 12:54 |
| gordonsim | flaper87: pong! | 12:56 |
| flaper87 | gordonsim: yo! | 12:57 |
| flaper87 | gordonsim: do you have some minutes to discuss Marconi's client API? | 12:57 |
| flaper87 | tedross: ^ | 12:57 |
| gordonsim | flaper87: sure | 12:57 |
| flaper87 | awesome | 12:57 |
| flaper87 | gordonsim: have you looked at https://wiki.openstack.org/wiki/Marconi/PythonClient | 12:58 |
| flaper87 | ? | 12:58 |
| gordonsim | flaper87: very briefly only | 12:59 |
| flaper87 | gordonsim: https://etherpad.openstack.org/marconi-client-api | 12:59 |
| flaper87 | lets mock something up there | 12:59 |
| flaper87 | gordonsim: feel free to change things in that pad and write your comments there | 13:03 |
| *** ykaplan has joined #openstack-marconi | 13:04 | |
| *** amitgandhi has joined #openstack-marconi | 13:09 | |
| *** amitgandhi has quit IRC | 13:10 | |
| *** amitgandhi has joined #openstack-marconi | 13:10 | |
| *** jergerber has joined #openstack-marconi | 13:13 | |
| *** oz_akan_ has joined #openstack-marconi | 13:13 | |
| cppcabrera | flaper87: Ah, so it is mostly implicit. Cool. (body_encoded) :) | 13:13 |
| *** oz_akan_ has quit IRC | 13:15 | |
| *** oz_akan_ has joined #openstack-marconi | 13:16 | |
| flaper87 | cppcabrera: gordonsim thanks for your inputs guys. I think that's enough info to start a POC of the API and see how it goes | 13:23 |
| cppcabrera | +1 - thanks gordonsim! | 13:24 |
| flaper87 | gordonsim: are you signed up in review.openstack.org ? | 13:24 |
| flaper87 | if you give me your username, I'll make sure to keep you in the loop | 13:24 |
| flaper87 | and add you to the reviews | 13:24 |
| cppcabrera | I added the etherpad session link to the client bindings wiki page. | 13:26 |
| flaper87 | cppcabrera: thanks, big +1 | 13:27 |
| cppcabrera | np~ | 13:28 |
| gordonsim | flaper87: no, I should sign up now | 13:29 |
| gordonsim | flaper87: all: thanks for the chat! | 13:29 |
| flaper87 | cppcabrera: https://review.openstack.org/#/c/46939/ | 13:35 |
| flaper87 | thoughts? | 13:35 |
| cppcabrera | Checking it out. :) | 13:35 |
| flaper87 | awesome | 13:36 |
| flaper87 | thanks | 13:36 |
| cppcabrera | First thought - Jenkins is happy about it, That's usually good news. :D | 13:36 |
| flaper87 | I tried to make sure it doesn't conflict with your proxy patches | 13:36 |
| flaper87 | it shouldn't | 13:36 |
| flaper87 | please, double check :D | 13:36 |
| cppcabrera | If it does, I'll live. I'm happy about the restructuring, overall. | 13:36 |
| cppcabrera | :D | 13:36 |
| flaper87 | :D | 13:36 |
| flaper87 | glad you said that | 13:37 |
| flaper87 | :D | 13:37 |
| openstackgerrit | A change was merged to openstack/python-marconiclient: Update .gitreview for incubation org move https://review.openstack.org/46629 | 13:39 |
| flaper87 | guys, client patches need some care https://review.openstack.org/#/q/status:open+project:openstack/python-marconiclient,n,z :D | 13:39 |
| flaper87 | kgriffs_afk: ^ | 13:39 |
| *** kgriffs_afk is now known as kgriffs | 13:39 | |
| cppcabrera | flaper87: +2 for restructuring | 13:41 |
| cppcabrera | After fighting with oslo.config yesterday, all of the configuration details suddenly make sense to me. | 13:43 |
| cppcabrera | I see cfg.Stropt, and I **know** what's being talked about. | 13:43 |
| amitgandhi | amazing what a good nights sleep can do | 13:43 |
| cppcabrera | amitgandhi: Ha, I have been sleeping slightly better this week than last. :P | 13:44 |
| * cppcabrera looks at client auth patch | 13:45 | |
| cppcabrera | Interesting - authenticate returns a Client instance | 13:46 |
| cppcabrera | I wasn't expecting that. | 13:46 |
| cppcabrera | Works for me. | 13:46 |
| cppcabrera | authenticate :: version -> request -> client | 13:47 |
| flaper87 | cppcabrera: wait, auth is in WIP | 13:47 |
| flaper87 | it's actually a draft | 13:47 |
| flaper87 | and you are the only one that can see it :D | 13:47 |
| flaper87 | but yes, that's the overall idea | 13:48 |
| flaper87 | I guess | 13:48 |
| * cppcabrera can see the special WIP secrets | 13:48 | |
| *** kgriffs is now known as kgriffs_afk | 13:49 | |
| cppcabrera | As far as the auth patch, I'm happy with all of it so far, though I'm skeptical about utils.env and how that brings ceilometerclient in as a dep. | 13:49 |
| cppcabrera | I'm currently reviewing that patch of code for obvious py3k breaks. | 13:49 |
| zyuan | flaper87: is absolute import allowed? | 13:50 |
| flaper87 | cppcabrera: I removed the ceilo thing, I just didn't see it before submitting the patch | 13:51 |
| flaper87 | :D | 13:51 |
| flaper87 | zyuan: yup | 13:51 |
| flaper87 | relative's aren't though | 13:51 |
| flaper87 | relatives | 13:51 |
| zyuan | oh... i used wrong word | 13:52 |
| zyuan | i mean import .base | 13:52 |
| zyuan | import ..unit.xxxx | 13:52 |
| flaper87 | no, relative import are not allowed | 13:52 |
| zyuan | because tests is not a public module | 13:52 |
| zyuan | then how can i import ..? | 13:52 |
| zyuan | even C support local only import, why not python | 13:53 |
| zyuan | why not here | 13:53 |
| flaper87 | why do you need to import things from tests ? | 13:53 |
| zyuan | because im doing forwarding tests | 13:53 |
| zyuan | proxy -> wsgi | 13:53 |
| * cppcabrera catches up after intense WIP reviewing | 13:55 | |
| flaper87 | mmh, not sure I follow. I mean, I know what you're trying to test but I don't know how are you trying to test it | 13:55 |
| *** ayoung has joined #openstack-marconi | 13:57 | |
| zyuan | i launched multiple marconi server as nodes in test, then test proxy with mock. the proxy->wsgi forwarding part is suppose to be testable by pulling wsgi test | 13:58 |
| zyuan | or i can do from tests.unit.transport import wsgi as wsgi_test | 13:59 |
| zyuan | :( | 13:59 |
| flaper87 | TBH, I would try to keep them separate. Wsgi tests are for wsgi v1, if / when we'll add a new version for the API this can become an issue since proxy tests will be tight to the wsgi v1 tests (assuming I understood exactly what you're doing) | 14:01 |
| zyuan | flaper87: this is another issue. no matter how you separate, it still make sense to pull backends' test for proxy | 14:01 |
| flaper87 | mmh, I don't fully agree with that, however, I think from tests.unit. etc is the rigth import path to use | 14:03 |
| *** malini has joined #openstack-marconi | 14:06 | |
| *** kgriffs_afk is now known as kgriffs | 14:06 | |
| kgriffs | o/ | 14:08 |
| kgriffs | good morning | 14:09 |
| kgriffs | good afternoon | 14:09 |
| kgriffs | flaper87: nice job breaking everyone's patches. ;) | 14:10 |
| openstackgerrit | A change was merged to openstack/marconi: Change Marconi's codebase structure https://review.openstack.org/46939 | 14:14 |
| flaper87 | kgriffs: :D | 14:15 |
| *** kgriffs is now known as kgriffs_afk | 14:15 | |
| flaper87 | kgriffs_afk: sorry about that :( | 14:15 |
| cppcabrera | brb demos~ | 14:18 |
| cppcabrera | kgriffs_afk: morning. :) | 14:24 |
| *** kgriffs_afk is now known as kgriffs | 14:27 | |
| *** kgriffs is now known as kgriffs_afk | 14:30 | |
| *** kgriffs-test has joined #openstack-marconi | 14:31 | |
| kgriffs-test | testing 1,2,3... | 14:31 |
| *** kgriffs_afk is now known as kgriffs | 14:31 | |
| *** kgriffs-test has quit IRC | 14:31 | |
| * flaper87 just discovered kgriffs has unit-tests integrated | 14:32 | |
| flaper87 | at least they all pass | 14:32 |
| flaper87 | :D | 14:32 |
| *** ykaplan has quit IRC | 14:41 | |
| cppcabrera | lol | 14:43 |
| zyuan | cppcabrera: found another issue | 14:45 |
| *** ykaplan has joined #openstack-marconi | 14:45 | |
| zyuan | before sending requests to marconi, you sometimes need to decode json | 14:46 |
| cppcabrera | oh? | 14:46 |
| zyuan | there is no defending code for malformed json | 14:46 |
| zyuan | in proxy | 14:46 |
| kgriffs | cppcabrera: if all you need to do is verify the JSON, then you can use the code from falcon - it is much faster than actually parsing into a dict | 14:46 |
| kgriffs | sorry, not falcon | 14:47 |
| kgriffs | rse | 14:47 |
| zyuan | kgriffs: different cases | 14:47 |
| kgriffs | zyuan: what is the argument for not just passing through the raw body and letting the backend deal with it? | 14:47 |
| zyuan | if doing forwarding, there is no need to verify; just send the raw request as is | 14:47 |
| zyuan | if doing proxy processing, just need to catch an exception | 14:48 |
| kgriffs | oic | 14:48 |
| zyuan | so called "defending" | 14:48 |
| zyuan | re verifying api input | 14:48 |
| zyuan | err, i don't know which is faster or whether it's really needed | 14:49 |
| zyuan | if we need to prevent ourself from crashing the server, there are 1 or 2 cases missing (i commented in test, hopefully) | 14:49 |
| *** kgriffs is now known as kgriffs_afk | 14:50 | |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: WIP: add mongo driver for marconi proxy storage https://review.openstack.org/46375 | 14:54 |
| cppcabrera | There, now I can go fix the minor marconi proxy issues. | 14:55 |
| cppcabrera | zyuan: hmmm, so re: JSON handling, what route/resource are you talking about? | 14:56 |
| zyuan | queue metadata | 14:56 |
| cppcabrera | gotcha | 14:57 |
| zyuan | but wait... | 14:57 |
| zyuan | anyway, i'm not very sure how metadata is handled in proxy | 14:57 |
| zyuan | but some "bad" metadata tests failed | 14:57 |
| zyuan | let me post the WIP patch on gerrit | 14:58 |
| cppcabrera | client -> proxy -> (proxy stores queue name/metadata, generates location) -> marconi | 14:58 |
| *** kgriffs_afk is now known as kgriffs | 14:58 | |
| zyuan | cppcabrera: yea, i know the rough steps, but i'm not sure which step caused the problem | 14:59 |
| cppcabrera | Ahh, I see. | 14:59 |
| zyuan | ah!! | 15:00 |
| zyuan | dependency | 15:00 |
| zyuan | hope it works.... | 15:00 |
| *** kgriffs is now known as kgriffs_afk | 15:01 | |
| *** ametts has quit IRC | 15:01 | |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: test(proxy): forwarded endpoints tested https://review.openstack.org/46963 | 15:01 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): partition related endpoints tested https://review.openstack.org/46362 | 15:01 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): catalog tested https://review.openstack.org/46769 | 15:01 |
| zyuan | ah ha, ah ha, ah ha ha ha.... | 15:01 |
| zyuan | cppcabrera: https://review.openstack.org/#/c/46963/ | 15:02 |
| cppcabrera | lots of reviews | 15:02 |
| *** kgriffs_afk is now known as kgriffs | 15:02 | |
| zyuan | not... | 15:02 |
| zyuan | run dependencies | 15:02 |
| zyuan | wrong* | 15:02 |
| zyuan | T_T | 15:02 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: marconi proxy https://review.openstack.org/43909 | 15:03 |
| cppcabrera | I'm pretty sure I broke my own dependency chain with ^^ | 15:06 |
| cppcabrera | Well, marconi-proxy patch 1 is ready to merge. :P | 15:06 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: marconi proxy (v1, health) https://review.openstack.org/44356 | 15:07 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: marconi-proxy forwarding https://review.openstack.org/44364 | 15:07 |
| zyuan | let me run test | 15:07 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: define interfaces for proxy storage https://review.openstack.org/46328 | 15:07 |
| cppcabrera | rebasing ^^ | 15:07 |
| flaper87 | cppcabrera: LGTM | 15:08 |
| flaper87 | thanks a lot! | 15:09 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: WIP: add mongo driver for marconi proxy storage https://review.openstack.org/46375 | 15:12 |
| cppcabrera | np, flaper87. Thank you! | 15:13 |
| cppcabrera | So let's see, what I'd like to have merged soon includes... (fetches list) | 15:13 |
| cppcabrera | This: https://etherpad.openstack.org/marconi-proxy-issues | 15:13 |
| cppcabrera | all the way up to the proxy storage interface in the long chain of patches. | 15:14 |
| cppcabrera | including proxy storage interface, but not the mongo driver. | 15:14 |
| cppcabrera | All this patch juggling has made me have to get pretty good @ 'git review'. :P | 15:15 |
| cppcabrera | ```git review -d``` became a good friend of mine over the past week, heh. | 15:15 |
| cppcabrera | kgriffs, flaper87: ^ (re: merging proxy patches) | 15:15 |
| flaper87 | cppcabrera: hehehe, you could also use rebase + edit mode :) | 15:20 |
| flaper87 | cppcabrera: I'll focus on reviewing those patches in the next few days. Please, consider that I'll be with limited internet access tomorrow and the day after | 15:21 |
| flaper87 | but I'll do my best | 15:21 |
| cppcabrera | thanks! | 15:22 |
| cppcabrera | the mongo driver patch is fixing most of the design pain points. | 15:22 |
| cppcabrera | It's where I realized most of what won't work in the long run, e.g., Redis in code, no interface to storage, round robin in storage... | 15:23 |
| zyuan | now functional test went back to non-working states on both of my freebsd and fedora machies... | 15:23 |
| zyuan | with nosetests... | 15:24 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): partition related endpoints tested https://review.openstack.org/46362 | 15:25 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): catalog tested https://review.openstack.org/46769 | 15:28 |
| flaper87 | http://www.sandywalsh.com/2013/09/notification-usage-in-openstack-report.html | 15:29 |
| kgriffs | interesting note about swift | 15:31 |
| cppcabrera | I like the note on oslo adoption, and why that's currently pretty low. | 15:32 |
| *** ykaplan has quit IRC | 15:32 | |
| cppcabrera | I agree with the sentiment - if I can't pip install a particular oslo component, it makes me pretty uncomfortable using thye copy-paste approach. | 15:32 |
| cppcabrera | thanks for sharing, flaper87 | 15:33 |
| cppcabrera | we're likely to end up in the swift boat, since we expect a very high transactional volume. | 15:34 |
| al-maisan | flaper87: that looks interesting indeed | 15:34 |
| cppcabrera | "Which Notifications Should Services Emit?" is my favorite section as a developer of openstack services. It makes it clear what's expected. | 15:34 |
| flaper87 | agreed, I can see Marconi emitting notifications, however, I'd prefer to wait 'til ceilo's support for statsd | 15:35 |
| flaper87 | or something like that | 15:35 |
| al-maisan | cppcabrera: yeah .. identifying what is "interesting" will be interesting ;) | 15:35 |
| zyuan | can someone here? | 15:37 |
| zyuan | help | 15:37 |
| zyuan | http://logs.openstack.org/69/46769/6/check/gate-marconi-python26/d18b147/console.html | 15:37 |
| zyuan | why i can't import bootstrap? | 15:37 |
| zyuan | it works on my machine, with tox | 15:37 |
| zyuan | what's wrong with jenkins? | 15:38 |
| flaper87 | zyuan: from marconi.queues import bootstrap | 15:38 |
| flaper87 | I guess you've some .pyc around | 15:38 |
| zyuan | moved? | 15:38 |
| zyuan | ... | 15:38 |
| flaper87 | duuuude, you have to read git logs before rebasing :D | 15:39 |
| zyuan | i'm rebsing upon cppcabrera's change -_ | 15:39 |
| cppcabrera | Did I forget to change something... ? >.> | 15:40 |
| zyuan | nothing... | 15:40 |
| cppcabrera | Ah, k. I knew I had to handle the project restructure in the proxy-mongo driver since I use marconi.queues.storage.mongodb.utils. | 15:41 |
| flaper87 | I just found 2 imports pointing to the old path in cmd | 15:42 |
| flaper87 | submitting a patch now | 15:42 |
| cppcabrera | which makes me think it might not hurt to at some point move common storage tools into marconi.common.storage.* | 15:42 |
| cppcabrera | +1 flaper87 | 15:42 |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Fix old import paths https://review.openstack.org/46973 | 15:42 |
| cppcabrera | Also, am I the only one that frequently typos marconi as macaroni? My fingers just want to type out the food. :P | 15:42 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): catalog tested https://review.openstack.org/46769 | 15:42 |
| flaper87 | kgriffs: could you set yourself as approver here? https://blueprints.launchpad.net/marconi/+spec/change-marconi-structure | 15:42 |
| flaper87 | danke :) | 15:43 |
| * flaper87 moves his fingers as a jedi | 15:43 | |
| zyuan | after reviewed the 1st proxy patch | 15:44 |
| zyuan | pls take a look at the twotest patches depending on it | 15:44 |
| cppcabrera | flaper87: Could you handle the errors in HACKING.rst, as well? | 15:44 |
| cppcabrera | +107 (import marconi.transport), +115 (import marconi.transport.wsgi as wsgi) | 15:44 |
| flaper87 | zyuan: kk | 15:45 |
| flaper87 | cppcabrera: will fix those as well | 15:45 |
| cppcabrera | woot | 15:45 |
| flaper87 | cppcabrera: thanks for noticing | 15:45 |
| cppcabrera | I figured I'd do a 'grin' session since we're on the topic now. :D | 15:45 |
| zyuan | i stop pushing 3rd test patch before the first 2 being merged | 15:45 |
| * flaper87 heading to a meeting | 15:45 | |
| flaper87 | brb | 15:45 |
| kgriffs | falper87: done; i need to get in the habit of doing that. :p | 15:47 |
| *** flaper87 is now known as flaper87|afk | 15:47 | |
| *** flaper87|afk is now known as flaper87 | 15:55 | |
| *** ykaplan has joined #openstack-marconi | 15:57 | |
| zyuan | another breaking change is 'nodes' -> 'hosts' | 15:58 |
| *** ykaplan has quit IRC | 16:00 | |
| cppcabrera | good point, zyuan | 16:02 |
| cppcabrera | I'm changing the entire world with the mongo patch. :P | 16:02 |
| zyuan | 87 'nodes': data['hosts'], | 16:03 |
| zyuan | -_-||| | 16:03 |
| *** amitgandhi has quit IRC | 16:03 | |
| malini | flaper87: do you know which openstack project uses testscenarios ? I checked tempest & nova so far. | 16:16 |
| cppcabrera | malini: It seems neutron might use testscenarios at this point: http://lists.openstack.org/pipermail/openstack-dev/2013-February/005457.html | 16:24 |
| cppcabrera | I found that while googling about. | 16:24 |
| cppcabrera | There's also mention of another module, testresources | 16:24 |
| malini | thanks cppcabrera..I am doing some more reading on testscenarios. | 16:26 |
| malini | Looks like it doesnt support Py3 yet.. | 16:27 |
| malini | its on a 'wishlist' | 16:27 |
| cppcabrera | +1 malini - I'd love to hear if you find out some really cool things about either. | 16:27 |
| cppcabrera | python 3 support... :( | 16:27 |
| *** flaper87 is now known as flaper87|afk | 16:49 | |
| cppcabrera | I noticed that full screen emacs looked pretty cool, and that I had a good reason to use >2 windows for once, so this happened: https://twitter.com/cppcabrera/status/380010252734693376 | 16:49 |
| *** amitgandhi has joined #openstack-marconi | 16:54 | |
| zyuan | cppcabrera: there is nothing run with find_one and find.. limit(1) does not help. | 17:27 |
| zyuan | the reason why benchmark showed that find_one is slow is not find_one itself is slow; the slow thing is the usage. | 17:28 |
| zyuan | just use find_one instead of catching the exception by yourself | 17:28 |
| kgriffs | guys, what do you think about allowing a message to live up to 60 seconds after it expires (meaning, it could also be returned in listings)? | 17:35 |
| kgriffs | i would like to use mongo's TTL in my patch that is rendering the external gc unecessary | 17:35 |
| kgriffs | that means I need to add a Date type field to the messages again | 17:36 |
| cppcabrera | I think that as long as we state that messages are garbage collected using a best effort algorithm, and we explicitly note that they may appear in listings, it';s cool. | 17:36 |
| kgriffs | and I was thinking to just stop filtering by 'e' so we don't have to deal with Date at all in our code | 17:36 |
| kgriffs | (other than setting on new messages) | 17:36 |
| kgriffs | flaper87: ^^^ | 17:37 |
| kgriffs | megan_w: ^^^ | 17:37 |
| zyuan | i +1 for live longer time than TTL | 17:37 |
| cppcabrera | +1 from me, btw | 17:37 |
| cppcabrera | find vs find_one with an empty queue: http://paste.openstack.org/show/47173/ | 17:38 |
| kgriffs | looks like both are doing find() | 17:39 |
| kgriffs | ? | 17:39 |
| cppcabrera | oops | 17:39 |
| cppcabrera | I had the proper data in my terminal and I botched the paste. :P | 17:39 |
| cppcabrera | Trying again, with more data points, too. | 17:40 |
| cppcabrera | kgriffs: http://paste.openstack.org/show/47174/ | 17:43 |
| zyuan | cppcabrera: but after you catch the exception, they are the same | 17:43 |
| cppcabrera | The exception is being caught above. ^^ | 17:43 |
| cppcabrera | That's included in the benchmark time. | 17:44 |
| cppcabrera | Unless I'm missing something? | 17:44 |
| zyuan | return? | 17:44 |
| cppcabrera | Ahh, I'll try that. | 17:45 |
| zyuan | not needed now | 17:46 |
| zyuan | find_one's source code | 17:46 |
| zyuan | if spec_or_id is not None and not isinstance(spec_or_id, dict): | 17:46 |
| zyuan | spec_or_id = {"_id": spec_or_id} | 17:46 |
| zyuan | one more branch | 17:46 |
| cppcabrera | Interesting... changing 'pass' -> 'None' sped this up by a lot. >.> | 17:46 |
| zyuan | but... serious, not worth | 17:46 |
| cppcabrera | lol | 17:46 |
| cppcabrera | Yeah, find_one() is good enough, IMO. | 17:46 |
| cppcabrera | Especially if I plan to use the result. | 17:47 |
| cppcabrera | I wanted to dispel rumors that there's a massive difference between find and find_one that I'd heard while reading year old articles about mongo. | 17:47 |
| cppcabrera | +1 for the nitpick, btw, zyuan. | 17:47 |
| cppcabrera | a::b vs a :: b | 17:48 |
| cppcabrera | I like a :: b, too. | 17:48 |
| cppcabrera | haha | 17:48 |
| zyuan | how i install pbr? | 17:49 |
| zyuan | pip install -U pbr expands egg into marconi/ | 17:50 |
| zyuan | but i can't import?! | 17:50 |
| zyuan | same as d2to1 | 17:51 |
| zyuan | ok, need to quite venv and install again... | 17:52 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: Update pbr dependency to latest https://review.openstack.org/46260 | 18:03 |
| *** tedross has quit IRC | 18:06 | |
| *** ametts has joined #openstack-marconi | 18:09 | |
| kgriffs | cppcabrera: re those patches you want merged | 18:21 |
| kgriffs | cppcabrera: ping | 18:21 |
| cppcabrera | kgriffs: pong | 18:21 |
| cppcabrera | they're a messy bunch, and I've been working to clean up the poor design decisions in the mongo driver patch. | 18:21 |
| kgriffs | hey, so I am thinking I will just ninjapprove these, with the caveat that we will do a full review/audit of the entire proxy in future patches | 18:22 |
| cppcabrera | +1 kgriffs - that was my hope and my promise | 18:22 |
| zyuan | wow, i love this: | 18:22 |
| kgriffs | we can do it this time since all the code is segregated into a new dir | 18:22 |
| zyuan | '{:.2%}'.format(0.132) | 18:22 |
| *** tedross has joined #openstack-marconi | 18:22 | |
| zyuan | kgriffs: we did a lot | 18:23 |
| cppcabrera | kgriffs: agreed - if this touched main marconi logic, I would've worked around it all somehow. | 18:23 |
| zyuan | and we collect things need to improve in a etherpad | 18:23 |
| cppcabrera | or I should, marconi-queues logic. | 18:23 |
| zyuan | https://etherpad.openstack.org/marconi-proxy-issues | 18:23 |
| zyuan | we need to resolve dependency first | 18:23 |
| zyuan | hopefully: | 18:24 |
| cppcabrera | *should -> should say | 18:24 |
| zyuan | proxy1 <- test1 <- test2 <- proxy 2 <- proxy 3 <- test3 <- proxy storage1 <- proxy mongo <- new test | 18:24 |
| zyuan | currently the dependency is stucked | 18:25 |
| kgriffs | cppcabrera: as I come across issues, let me just ping you | 18:26 |
| kgriffs | https://review.openstack.org/#/c/44356/8/marconi/proxy/app.py | 18:26 |
| kgriffs | proxy should return it's own health response (don't send to backend) | 18:26 |
| cppcabrera | noted: etherpad'd | 18:26 |
| zyuan | kgriffs: we can test it :) | 18:27 |
| openstackgerrit | A change was merged to openstack/marconi: feat: marconi proxy https://review.openstack.org/43909 | 18:27 |
| zyuan | kgriffs: https://review.openstack.org/#/c/46362/14 | 18:28 |
| kgriffs | cppcabrera: obviously we can't make this a habit. how will you avoid this situation in the future? | 18:28 |
| cppcabrera | Two ways, kgriffs: | 18:28 |
| cppcabrera | 1. I've learned about interface driven development this round, and how powerful it can be towards avoiding encoding hard requirements too early. I'll be using this more later. | 18:29 |
| kgriffs | +1 | 18:29 |
| cppcabrera | 2. Going of (1), I'll prototype using that interface to see if that's enough. | 18:29 |
| cppcabrera | (2) is most of what I'm doing now - it's an amazing way to test my assumptions. | 18:29 |
| kgriffs | ok, sounds good. prototyping can be really helpful when there are a lot of unknowns | 18:29 |
| zyuan | kgriffs: why this can't be a "habit"? | 18:30 |
| zyuan | why we can't use marconi repo for prototyping? | 18:30 |
| zyuan | proto - test refinement, that looks agile | 18:30 |
| cppcabrera | Because others will eventually come to depend on openstack/marconi to be as stable as possible. | 18:31 |
| zyuan | interface -> impl, that's not | 18:31 |
| zyuan | cppcabrera: we have release plan | 18:31 |
| kgriffs | cppcabrera: https://review.openstack.org/#/c/44364/6/marconi/proxy/resources/forward.py | 18:31 |
| openstackgerrit | A change was merged to openstack/marconi: feat: marconi proxy (v1, health) https://review.openstack.org/44356 | 18:31 |
| kgriffs | I was wondering if there's something we could add to Falcon to DRY having to do all these responders that only forward stuff | 18:32 |
| zyuan | kgriffs: we added, i suppose | 18:32 |
| cppcabrera | kgriffs: flaper87|afk mentioned using a ForwardingMixin class. | 18:32 |
| cppcabrera | That could DRY up those responders. | 18:32 |
| zyuan | don't forget https://review.openstack.org/#/c/46362/ and https://review.openstack.org/#/c/46769/ | 18:33 |
| zyuan | (smile | 18:34 |
| kgriffs | zyuan: yep, working my way down the list | 18:34 |
| kgriffs | cppcabrera: so, the only thing there is that falcon now has to make http client requests itself | 18:40 |
| cppcabrera | you mean regarding the Mixin class, kgriffs? | 18:43 |
| kgriffs | no | 18:43 |
| kgriffs | I think this is better solved in Falcon | 18:43 |
| kgriffs | https://etherpad.openstack.org/marconi-proxy-issues | 18:43 |
| kgriffs | make sense? | 18:43 |
| cppcabrera | ah | 18:44 |
| cppcabrera | I see now. | 18:44 |
| cppcabrera | I was missing the context. | 18:44 |
| kgriffs | so, the single proxy func handles all forwards, and Falcon takes care of policing the http methods (get, post, put) per route | 18:45 |
| cppcabrera | Hmm... | 18:46 |
| zyuan | proxy & pre-post processing | 18:46 |
| kgriffs | brb | 18:48 |
| cppcabrera | k | 18:50 |
| cppcabrera | zyuan: +1 | 18:50 |
| cppcabrera | I see what you're saying. | 18:50 |
| cppcabrera | Given both optional pre/post hooks + core proxy functionality. | 18:51 |
| * cppcabrera goes to etherpad | 18:51 | |
| cppcabrera | kgriffs, zyuan: I'm going to let the forwarding API thought sit on the back burner for a bit and return to the storage implementation. | 19:00 |
| cppcabrera | I'm fond of api.forward and api.route as names, and I'm on the wall about accepting pre/post closure arguments. | 19:00 |
| *** cthulhup has quit IRC | 19:05 | |
| kgriffs | cppcabrera: how is the cached metadata to be used? | 19:11 |
| kgriffs | https://review.openstack.org/#/c/44364/6/marconi/proxy/resources/metadata.py | 19:11 |
| zyuan | well, at least i see where the bug (bad json not handled) is... | 19:14 |
| *** gordonsim has quit IRC | 19:14 | |
| zyuan | kgriffs: i suggest to merge this first, so that i can change it in my forward test patch | 19:16 |
| cppcabrera | kgriffs: cached metadata is required for GET /v1/queues | 19:17 |
| *** ayoung has quit IRC | 19:17 | |
| cppcabrera | Specifically, GET /v1/queues?detailed=true | 19:17 |
| zyuan | but not /v1/queues/q/metadata? | 19:17 |
| cppcabrera | That, too. :P | 19:18 |
| cppcabrera | I knew I was missing a route. | 19:18 |
| cppcabrera | Well, actually... | 19:18 |
| cppcabrera | not the latter case. | 19:18 |
| cppcabrera | Only for listings. | 19:18 |
| cppcabrera | The latter case can be forwarded. | 19:18 |
| cppcabrera | But only the proxy knows about all the queues registered under a single project. | 19:18 |
| cppcabrera | Therefore, detailed listings require caching metadata. | 19:19 |
| zyuan | noted on etherpad | 19:19 |
| kgriffs | oh, so you are trying to avoid querying all backends and merging the listings? | 19:19 |
| kgriffs | the proxy knows which backends are associated with which projects, so it could just query those backends, then merge the results | 19:22 |
| cppcabrera | yup, kgriffs. | 19:22 |
| cppcabrera | It sounded very expensive compared to maintaining that data in the catalogue. | 19:22 |
| *** tedross has quit IRC | 19:23 | |
| kgriffs | relatively, yes | 19:23 |
| kgriffs | i suppose metadata shouldn't take up much space in the catalog/cache | 19:23 |
| cppcabrera | generally, it shouldn't. I feel the real cost is one of maintenance - keeping the marconi-queues listing consistent with the marconi-proxy implementation. | 19:24 |
| *** malini is now known as malini_afk | 19:24 | |
| kgriffs | yeah | 19:24 |
| kgriffs | I have a feeling people won't be listing queues very often | 19:25 |
| kgriffs | so it might be YAGNI | 19:25 |
| kgriffs | latency is less important there too | 19:25 |
| zyuan | control panel will | 19:25 |
| zyuan | oh, yea | 19:25 |
| zyuan | then discard metadata in proxy storage? | 19:26 |
| kgriffs | well, i guess so, but nothing like the volume we expect to see though clients listing messages | 19:26 |
| zyuan | i don;t know how front-end guys going to implement this | 19:26 |
| zyuan | but if i can form an "attack" with refreshing control panel.... | 19:27 |
| zyuan | that's not good | 19:27 |
| kgriffs | my preference would be to make n requests - i think the proxy can probably do them in parallel if we want using eventlet or gevent | 19:27 |
| cppcabrera | +1 for eventlet | 19:27 |
| kgriffs | just ask for a queue listing for the project from each partition | 19:28 |
| kgriffs | then merge sort | 19:28 |
| cppcabrera | Hmmm... | 19:28 |
| kgriffs | make an educated guess on how many to ask from each partition | 19:28 |
| kgriffs | so you get the page limit | 19:29 |
| kgriffs | if you get a few less, no big deal, as long as the "next" href is correct | 19:29 |
| cppcabrera | I like the idea for the sake of DRYness. | 19:30 |
| kgriffs | DRY data | 19:30 |
| cppcabrera | However, I'd like to revisit that in a later patch. How about after getting Hcaching through? Thoughts? | 19:31 |
| kgriffs | i suppose you could cache metadata for a short period of time, but whatever you do, you want to avoid having 2 sources of truth | 19:31 |
| kgriffs | no problem, I am just making sure we get these things on etherpad as I go along | 19:31 |
| kgriffs | speaking of which… :D | 19:31 |
| cppcabrera | Noted. :) | 19:31 |
| kgriffs | btw, some srlsy good hacking music: http://www.youtube.com/watch?v=LadRcupWDuI | 19:32 |
| cppcabrera | +1 kgriffs | 19:32 |
| cppcabrera | That Tron Legacy OST is sweeeett | 19:33 |
| kgriffs | anything you want to point out on this one? | 19:33 |
| kgriffs | https://review.openstack.org/#/c/46328 | 19:33 |
| kgriffs | can we rename this to "errors" | 19:34 |
| kgriffs | https://review.openstack.org/#/c/46328/3/marconi/proxy/storage/exceptions.py | 19:34 |
| kgriffs | ? | 19:34 |
| kgriffs | (in a future patch) | 19:34 |
| kgriffs | I have it on my todo list to rename similar modules in queues | 19:34 |
| cppcabrera | a lot is changing in that interface. :x | 19:35 |
| cppcabrera | I've done some renaming, some shifting of arguments. | 19:35 |
| kgriffs | kk | 19:35 |
| cppcabrera | It's all included in the mongo implementation patch. | 19:36 |
| kgriffs | I added a note about "errors" to the etherpad | 19:36 |
| cppcabrera | +1 | 19:36 |
| cppcabrera | exceptions -> errors | 19:36 |
| cppcabrera | I did a git oops earlier today. I had implemented a simple RoundRobin class and accidentally nuked it before commiting it to any of my trees. :P | 19:36 |
| cppcabrera | Reimplementing now. | 19:36 |
| cppcabrera | "Plan to throw one away", haha | 19:37 |
| zyuan | git stash is your friend | 19:37 |
| zyuan | kgriffs: we don't need to merge the interface patch for now; that one is innocent... | 19:38 |
| cppcabrera | thanks for all your help with the proxy, zyuan, kgriffs, flaper87|afk. | 19:39 |
| kgriffs | no problem. That's be $250,000 | 19:40 |
| zyuan | ............ | 19:40 |
| cppcabrera | well, there goes my everything. you broke the bank, kgriffs. :P | 19:40 |
| zyuan | add_proxy -> add_agent | 19:40 |
| zyuan | it reminds me MTA -- mail transfer agent, which does forwarding | 19:41 |
| cppcabrera | Hmm, considering round-robin, it'd be cool if I could implement that as a functional closure that takes advantage of itertools.cycle... hmm... | 19:41 |
| kgriffs | we could just overload add_route, i guess | 19:42 |
| zyuan | that one is COMPLEX | 19:42 |
| kgriffs | it would check whether it was given a class or a function | 19:43 |
| zyuan | give it another name pls. CS teacher tells me, one function do one thing | 19:43 |
| kgriffs | heh | 19:44 |
| kgriffs | that is usually best, yes | 19:44 |
| openstackgerrit | A change was merged to openstack/marconi: feat: marconi-proxy forwarding https://review.openstack.org/44364 | 19:46 |
| openstackgerrit | A change was merged to openstack/marconi: feat: define interfaces for proxy storage https://review.openstack.org/46328 | 19:46 |
| openstackgerrit | A change was merged to openstack/marconi: test(proxy): partition related endpoints tested https://review.openstack.org/46362 | 19:47 |
| openstackgerrit | A change was merged to openstack/marconi: test(proxy): catalog tested https://review.openstack.org/46769 | 19:47 |
| cppcabrera | look at all the merging. | 19:50 |
| cppcabrera | $250,000 worth of merging | 19:50 |
| zyuan | pls allow me to make a YaoMing face... | 19:50 |
| zyuan | ok i'm going to fix bugs in forwarding | 19:55 |
| cppcabrera | lol @ YaoMing face | 19:56 |
| cppcabrera | Looking at the weighted_select code, that's totally a scanl + a reduce. | 19:57 |
| zyuan | wuu | 19:57 |
| cppcabrera | because of that acc, I can't think of a way to use a listcomp. :/ | 19:57 |
| *** tedross has joined #openstack-marconi | 20:10 | |
| *** ayoung has joined #openstack-marconi | 20:22 | |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: test(proxy): forwarded endpoints tested https://review.openstack.org/46963 | 20:38 |
| zyuan | ^^ fixed 1 bug; awaiting some structural changes/// | 20:38 |
| zyuan | kgriffs: the term 'responder' has been used in Resource code, which refers to on_get, on_put, and even na_responder, etc. | 20:40 |
| zyuan | need a new term to name "a resource with an all-in-one responder" | 20:40 |
| kgriffs | that thing actually is a responder, just a standalone one - although, yes, it could be confusing because responders are currently defined as 1:1 (responder:verb) | 20:45 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: test(proxy): forwarded endpoints tested https://review.openstack.org/46963 | 20:46 |
| zyuan | dominator | 20:46 |
| zyuan | ha ha ha ha | 20:46 |
| cppcabrera | lol dominator... :P | 20:46 |
| zyuan | a | 20:47 |
| zyuan | http://psychopass.wikia.com/wiki/The_Dominator | 20:47 |
| *** oz_akan_ has quit IRC | 20:56 | |
| *** oz_akan_ has joined #openstack-marconi | 20:56 | |
| *** oz_akan_ has quit IRC | 21:01 | |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: Add mongo driver for marconi proxy storage https://review.openstack.org/46375 | 21:16 |
| cppcabrera | Heading home, guys. | 21:18 |
| kgriffs | k | 21:19 |
| zyuan | bye | 21:19 |
| cppcabrera | I left a not-so-terrible patch for review above. | 21:19 |
| kgriffs | watch for a monster patch from yours truly | 21:19 |
| cppcabrera | More removed than added. :D | 21:19 |
| cppcabrera | awesome, kgriffs. I'll check that out tomorrow. | 21:19 |
| kgriffs | (unrelated to proxy) | 21:19 |
| cppcabrera | (theme: monster patches) | 21:19 |
| kgriffs | …with big hairy feet | 21:19 |
| cppcabrera | I striked out the design flaws handled by patch #46375 | 21:19 |
| kgriffs | cool beans | 21:20 |
| cppcabrera | Good night~ :D | 21:20 |
| *** cppcabrera has left #openstack-marconi | 21:20 | |
| kgriffs | ciao! | 21:20 |
| kgriffs | o/ | 21:20 |
| zyuan | need a new create_http_method_map | 21:25 |
| zyuan | hehe | 21:25 |
| zyuan | see you kurt | 21:29 |
| kgriffs | ttfn | 21:36 |
| *** oz_akan_ has joined #openstack-marconi | 22:07 | |
| *** oz_akan_ has quit IRC | 22:12 | |
| *** amitgandhi has quit IRC | 22:27 | |
| *** tedross has quit IRC | 23:06 | |
| *** kgriffs is now known as kgriffs_afk | 23:11 | |
| *** tedross has joined #openstack-marconi | 23:35 | |
| *** tedross has quit IRC | 23:44 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!