| *** reed has quit IRC | 00:05 | |
| *** reed has joined #openstack-marconi | 00:06 | |
| *** malini1 has joined #openstack-marconi | 00:10 | |
| *** malini1 has quit IRC | 00:16 | |
| *** malini1 has joined #openstack-marconi | 00:16 | |
| *** amitgandhi has quit IRC | 00:22 | |
| *** nosnos has joined #openstack-marconi | 00:23 | |
| *** oz_akan_ has joined #openstack-marconi | 01:00 | |
| *** kgriffs_afk is now known as kgriffs | 01:13 | |
| *** cthulhup has joined #openstack-marconi | 01:26 | |
| *** cthulhup has quit IRC | 01:30 | |
| *** kgriffs is now known as kgriffs_afk | 01:36 | |
| *** reed has quit IRC | 01:42 | |
| *** malini1 has quit IRC | 02:13 | |
| *** kgriffs_afk is now known as kgriffs | 02:14 | |
| *** kgriffs is now known as kgriffs_afk | 02:14 | |
| *** kgriffs_afk is now known as kgriffs | 02:15 | |
| *** oz_akan_ has quit IRC | 02:32 | |
| *** oz_akan_ has joined #openstack-marconi | 02:32 | |
| *** oz_akan_ has quit IRC | 02:37 | |
| *** kgriffs is now known as kgriffs_afk | 02:47 | |
| *** reed has joined #openstack-marconi | 02:57 | |
| *** AyoungTheRed is now known as ayoung-ZzZzZz | 03:05 | |
| *** oz_akan_ has joined #openstack-marconi | 03:33 | |
| *** oz_akan_ has quit IRC | 03:41 | |
| *** reed has quit IRC | 05:11 | |
| al-maisan | Looking at https://wiki.openstack.org/wiki/Marconi/specs/api/v1#Endpoints_Synopsis -- are the parameters prefixed by '?' optional? | 05:26 |
|---|---|---|
| *** rektide has joined #openstack-marconi | 05:54 | |
| al-maisan | hmm .. by looking at the request examples I am getting the impression that all GET URL params are optional .. | 06:41 |
| *** cthulhup has joined #openstack-marconi | 06:51 | |
| *** cthulhup has quit IRC | 06:55 | |
| *** gordonsim has joined #openstack-marconi | 07:22 | |
| *** flaper87|afk is now known as flaper87 | 07:34 | |
| flaper87 | al-maisan: ping | 07:56 |
| al-maisan | flaper87: pong | 08:11 |
| flaper87 | al-maisan: yo, got your email | 08:11 |
| al-maisan | OK | 08:11 |
| flaper87 | al-maisan: it's very weird that Jenkins +1 that review | 08:11 |
| flaper87 | keystoneclient is indeed missing | 08:12 |
| flaper87 | :/ | 08:12 |
| al-maisan | I believe I figured out what params are mandatory/optional in the mean time | 08:12 |
| flaper87 | stupid jenkins | 08:12 |
| al-maisan | flaper87: yeah .. I was just puzzled | 08:12 |
| * flaper87 kicks jenkins legs | 08:12 | |
| al-maisan | :-) | 08:12 |
| flaper87 | al-maisan: did you put those params in an etherpad or something? | 08:12 |
| * flaper87 is curious | 08:12 | |
| al-maisan | I will be updating the branch in a minute | 08:13 |
| al-maisan | flaper87: http://paste.openstack.org/show/47300/ | 08:13 |
| openstackgerrit | Muharem Hrnjadovic proposed a change to openstack/python-marconiclient: This adds a transport layer request spec class. The idea here is to be declarative i.e. specify *what* is desired. It's up to the respective transport to turn a request spec into an appropriate request. https://review.openstack.org/47443 | 08:26 |
| flaper87 | al-maisan: here's a wiki page w.r.t commit message and some guidelines we follow in OpenStack. https://wiki.openstack.org/wiki/GitCommitMessages | 08:27 |
| al-maisan | flaper87: thanks! I am violating all kinds of rules here :-P | 08:28 |
| al-maisan | biab | 08:29 |
| flaper87 | al-maisan: :D, kk | 08:30 |
| *** oz_akan_ has joined #openstack-marconi | 08:50 | |
| *** oz_akan_ has joined #openstack-marconi | 08:50 | |
| *** torgomatic has quit IRC | 08:55 | |
| *** torgomatic has joined #openstack-marconi | 08:55 | |
| al-maisan | flaper87: do we care about python26 test failures (due to 'TestRequest' object has no attribute 'assertIs') | 08:58 |
| al-maisan | ? | 08:58 |
| flaper87 | al-maisan: yup, we should be using testtools for that. This aptch here introduces a TestBase class | 09:00 |
| flaper87 | al-maisan: https://review.openstack.org/#/c/46288/ | 09:00 |
| flaper87 | I'd suggest you to make your current work depend on this patch here: https://review.openstack.org/#/c/47169/ | 09:00 |
| flaper87 | which also depends on the TestBase one | 09:00 |
| flaper87 | the last patch cahnges Marconiclient's structure to reflect Marconi's | 09:00 |
| al-maisan | how do I do that? " make your current work depend on this patch here"? | 09:01 |
| flaper87 | to make your patch depend on that patch, you just need to rebase your local branch with the changes in that review | 09:01 |
| flaper87 | hehehe | 09:01 |
| al-maisan | branch from your branch? | 09:01 |
| al-maisan | abandon the patch set, branch from your branch and re-upload to reviews...? | 09:02 |
| flaper87 | al-maisan: you can do git review -d PATCH_ID, that will create a new branch with the changes in the review you want to depend on | 09:02 |
| al-maisan | OK | 09:02 |
| flaper87 | al-maisan: then you can cherry-pick your commit into that new branch | 09:02 |
| flaper87 | and then git review | 09:02 |
| flaper87 | git review will ask you if the dependency chain is correct before submitting it | 09:03 |
| al-maisan | flaper87: do I need to pass any special params to "git review" | 09:26 |
| al-maisan | also, the rebase prior to "git review" should be with respect to master or your "change-marconi-structure" branch? | 09:27 |
| al-maisan | flaper87: this is what I see when trying a "git review" : http://paste.openstack.org/show/47301/ | 09:28 |
| * flaper87 looking | 09:31 | |
| flaper87 | no params needed | 09:31 |
| flaper87 | al-maisan: that looks correct to me | 09:31 |
| flaper87 | just write yes and you're done | 09:31 |
| al-maisan | so, I should just say 'yes' then? | 09:31 |
| *** leizhang has joined #openstack-marconi | 09:32 | |
| flaper87 | btw, no '.' needed at the end of the commit 'title' | 09:32 |
| al-maisan | OK | 09:32 |
| al-maisan | I was afraid I'd create an avalanche of reviews again :-P | 09:32 |
| flaper87 | al-maisan: yeah, also, if you use git review -y it'll submit everything without asking you | 09:32 |
| flaper87 | al-maisan: LOL :D | 09:32 |
| al-maisan | :) | 09:32 |
| al-maisan | OK | 09:32 |
| al-maisan | then | 09:32 |
| flaper87 | al-maisan: it checks if there's a review for each commit's change-id | 09:32 |
| al-maisan | cool | 09:33 |
| * al-maisan pulls the trigger | 09:33 | |
| openstackgerrit | Muharem Hrnjadovic proposed a change to openstack/python-marconiclient: This adds a transport layer request spec class https://review.openstack.org/47538 | 09:33 |
| al-maisan | pheewwww .. only *one* review :-P | 09:33 |
| flaper87 | al-maisan: COOOOOOL, thanks for contributing :D I'll take a look in a bit | 09:33 |
| * flaper87 creates a new OS acronym | 09:34 | |
| al-maisan | thanks! | 09:34 |
| flaper87 | tlab | 09:34 |
| flaper87 | take a look in a bit | 09:34 |
| flaper87 | w000t | 09:34 |
| al-maisan | nice one :) | 09:34 |
| flaper87 | I admit it, I first wrote itlab but then I thought that apple may sue me if I dare to use an 'i' in front of anything, so.... | 09:35 |
| al-maisan | he-he :) | 09:35 |
| al-maisan | jenkins is happy :) good! | 09:36 |
| * al-maisan takes a break | 09:38 | |
| al-maisan | see ya later in the afternoon flaper87 ! | 09:38 |
| flaper87 | al-maisan: kk, thanks! ttyl | 09:38 |
| openstackgerrit | Muharem Hrnjadovic proposed a change to openstack/python-marconiclient: This adds a transport layer request spec class https://review.openstack.org/47538 | 09:43 |
| *** leizhang has quit IRC | 09:56 | |
| *** leizhang has joined #openstack-marconi | 09:56 | |
| *** nosnos has quit IRC | 10:17 | |
| *** oz_akan_ has quit IRC | 12:11 | |
| *** oz_akan_ has joined #openstack-marconi | 12:12 | |
| *** cthulhup has joined #openstack-marconi | 12:16 | |
| *** oz_akan_ has quit IRC | 12:16 | |
| *** cthulhup has quit IRC | 12:20 | |
| *** tedross has joined #openstack-marconi | 12:38 | |
| *** cppcabrera has joined #openstack-marconi | 12:40 | |
| cppcabrera | Morning~ :D | 12:42 |
| *** kgriffs_afk is now known as kgriffs | 12:44 | |
| flaper87 | cppcabrera: goood morning | 12:52 |
| flaper87 | :D | 12:52 |
| flaper87 | cppcabrera: how are you doing? | 12:52 |
| flaper87 | cppcabrera: btw, didn't mean to be a PITA with the request of breaking that patch into several | 12:52 |
| *** oz_akan_ has joined #openstack-marconi | 12:54 | |
| cppcabrera | no worries, flaper87. I'm happy to help with making the review process easier. :) | 12:54 |
| cppcabrera | I'm doing pretty awesome this morning, though a little on the hungry side. How about you? | 12:55 |
| flaper87 | I'm doing well, already caught up with email, bugs and some reviews. I'm going through the proxy patches now | 12:56 |
| cppcabrera | woot! I've got some fixes to apply to the second one, namely, it parses the config file twice when running the test suites. | 12:58 |
| flaper87 | oh, ok. I'll keep that in mind. I'm reviewing gozilla now | 12:58 |
| cppcabrera | lol | 12:59 |
| openstackgerrit | Muharem Hrnjadovic proposed a change to openstack/python-marconiclient: This adds a transport layer request spec class https://review.openstack.org/47538 | 13:00 |
| al-maisan | flaper87: this is the fix I mentioned ^ | 13:00 |
| *** amitgandhi has joined #openstack-marconi | 13:04 | |
| flaper87 | al-maisan: awesome, thanks! tlib :D | 13:05 |
| flaper87 | btw guys, al-maisan is helping us out with marconiclient, we had a great brainstorm session yday | 13:05 |
| flaper87 | w00000t | 13:05 |
| kgriffs | awesome sauce | 13:05 |
| kgriffs | flaper87: I'm reimplementing my changes to messages.py | 13:05 |
| kgriffs | not sure where they went | 13:06 |
| kgriffs | :p | 13:06 |
| al-maisan | :) | 13:06 |
| flaper87 | kgriffs: LOOOOOOOOL | 13:06 |
| al-maisan | uh-oh | 13:06 |
| kgriffs | #git-fail | 13:06 |
| kgriffs | #user-error | 13:06 |
| kgriffs | flaper87: gozilla? I thought we were writing the proxy in Python. ;) | 13:07 |
| * cppcabrera almost caught up with morning email | 13:07 | |
| flaper87 | kgriffs: btw, not sure if you noticed the comment I left on the left side of the diff (in messages.py) | 13:07 |
| al-maisan | kgriffs: take it easy -- git is a bit of a beast .. it's very easy to make errors :-P | 13:07 |
| flaper87 | kgriffs: we are, it just ate Godzilla | 13:08 |
| *** jergerber has joined #openstack-marconi | 13:09 | |
| al-maisan | flaper87: I thought gozilla is a thinly veiled reference to rust ;-) | 13:09 |
| flaper87 | not sure if it is godzilla that has a Python skin or it is python with godzilla's shape | 13:09 |
| flaper87 | mmhh | 13:09 |
| kgriffs | go-zilla | 13:09 |
| flaper87 | LOOOOL | 13:09 |
| kgriffs | pyzilla | 13:09 |
| al-maisan | yeah ;) | 13:09 |
| kgriffs | :[ | 13:09 |
| flaper87 | rustzilla | 13:09 |
| al-maisan | hmm .. | 13:10 |
| kgriffs | now ur talkin! | 13:10 |
| kgriffs | flaper87: re that other comment, let me explain | 13:10 |
| kgriffs | while bulk inserts are not atomic, they are inserted in order, and as soon as one fails, the rest are abandoned | 13:11 |
| * flaper87 reads carefully O.O | 13:11 | |
| kgriffs | in the case of a collision, it will always happen on the first message document in the list | 13:11 |
| * kgriffs thinks about that to make sure it is correct | 13:11 | |
| kgriffs | i'm pretty sure that is correct | 13:12 |
| flaper87 | mmh, I think that's correct as well | 13:12 |
| kgriffs | since the counter is not incremented until batch A is in | 13:12 |
| flaper87 | very well thought | 13:12 |
| flaper87 | have you had some sleep lately? | 13:13 |
| flaper87 | :D | 13:13 |
| kgriffs | batch B cannot proceed until request A increments the counter | 13:13 |
| kgriffs | flaper87: heh, not enough! | 13:13 |
| flaper87 | i can tell ^^ | 13:13 |
| flaper87 | aaaaaaanyway, I agree with you | 13:13 |
| kgriffs | FWIW, I tested this with mongo and indeed, thats the behavior | 13:13 |
| flaper87 | it wasn't obvious to me when I reviewed the patch | 13:14 |
| flaper87 | cool | 13:14 |
| * flaper87 so happy | 13:14 | |
| kgriffs | but, I am glad that you are considering the patch carefully; I'm still worried that there's a race condition or something I missed. | 13:14 |
| flaper87 | we can remove that regex-based error interpretation utility | 13:14 |
| kgriffs | +1 zillion | 13:14 |
| kgriffs | let me do that | 13:14 |
| kgriffs | flaper87: btw - https://review.openstack.org/#/c/46757/ | 13:16 |
| kgriffs | flaper87: also - https://review.openstack.org/#/c/47482/ | 13:17 |
| flaper87 | kgriffs: +1 | 13:17 |
| cppcabrera | falcon 0.1.7~ | 13:17 |
| kgriffs | that last one was driving me crazy yesterday | 13:17 |
| * kgriffs had to fix it | 13:17 | |
| flaper87 | kgriffs: approved | 13:18 |
| cppcabrera | #python_gotchas | 13:18 |
| kgriffs | w00t | 13:18 |
| kgriffs | at first i thought i was experiencing a time-space anomaly. then I realized it was just Python. | 13:19 |
| cppcabrera | Python produces space time anomalies - this is now known. | 13:19 |
| flaper87 | ROFL | 13:19 |
| al-maisan | :) | 13:20 |
| *** malini1 has joined #openstack-marconi | 13:22 | |
| kgriffs | "Our continuing mission: To boldly go where no developer has gone before!" | 13:23 |
| *** malini1 has quit IRC | 13:23 | |
| kgriffs | cppcabrera: be sure to make an entry of this incident in the logs so future explorers know to steer clear of this part of the galaxy. | 13:23 |
| * kgriffs jk | 13:24 | |
| * kgriffs shuts up and gets back to work | 13:24 | |
| *** malini has joined #openstack-marconi | 13:24 | |
| cppcabrera | lol | 13:24 |
| cppcabrera | Which reminds me that I need to finish watching Star Trek: NG at some point. :P | 13:25 |
| * kgriffs after that last joke, I think people are going to steer clear of this IRC channel | 13:25 | |
| cppcabrera | They | 13:25 |
| cppcabrera | 'll return for the space-time anomalies. | 13:25 |
| * cppcabrera goes to have breakfast | 13:26 | |
| cppcabrera | brb | 13:26 |
| kgriffs | well, if our project gets canceled, we can always look forward to Marconi: The Next Generation | 13:26 |
| * kgriffs *really* needs to get more sleep | 13:27 | |
| cppcabrera | lol | 13:28 |
| cppcabrera | relevant: http://goo.gl/ux78UI | 13:29 |
| kgriffs | double-relevant: http://pictr.io/pic/5768226071379683792_0_20768900.jpg | 13:31 |
| kgriffs | ROFL | 13:32 |
| cppcabrera | hahaha | 13:33 |
| cppcabrera | such an interesting service, pictr.io. Love == incr_TTL, Hate == decr_TTL. | 13:34 |
| * kgriffs loves this irc channel | 13:34 | |
| al-maisan | where do you guys find all these pictures? | 13:39 |
| al-maisan | :) | 13:39 |
| kgriffs | we work for the NSA | 13:39 |
| kgriffs | you wouldn't *believe* what we have access to | 13:39 |
| al-maisan | aaaaaaaaaaaaaah .. I understand now ;) | 13:39 |
| al-maisan | so, Marconi is NSA-scale then? | 13:40 |
| kgriffs | I | 13:42 |
| kgriffs | I | 13:42 |
| kgriffs | I can neither confirm nor deny that. | 13:42 |
| flaper87 | LOOOOL at double-relevant | 13:42 |
| flaper87 | lol, lol, lol, lol | 13:42 |
| flaper87 | btw, isn't it hackday at RAX ? | 13:43 |
| flaper87 | what are you guys hacking on? | 13:43 |
| amitgandhi | thats next week | 13:43 |
| amitgandhi | last friday of every month | 13:43 |
| flaper87 | ah, I thought it was every week | 13:43 |
| flaper87 | ah ok | 13:43 |
| flaper87 | ok | 13:43 |
| * flaper87 STFU | 13:43 | |
| amitgandhi | you should join us | 13:43 |
| amitgandhi | guest racker haha | 13:43 |
| * flaper87 stops prettending he knows RAX that well | 13:43 | |
| flaper87 | pretending* | 13:43 |
| flaper87 | amitgandhi: I will work on a rust-marconiclient | 13:44 |
| flaper87 | done, I said it! | 13:44 |
| kgriffs | w00t | 13:44 |
| al-maisan | kgriffs: deny it the Linus way: http://linux.slashdot.org/story/13/09/19/0227238/linus-torvalds-admits-hes-been-asked-to-insert-backdoor-into-linux | 13:44 |
| flaper87 | but please, make sure to remind that to me next friday, otherwise I'll start working on a brain-control algorithm | 13:45 |
| * al-maisan starts an elixir-marconiclient :) | 13:45 | |
| cppcabrera | all the marconiclients | 13:46 |
| amitgandhi | rust looks interesting | 13:46 |
| flaper87 | al-maisan: +1 | 13:46 |
| cppcabrera | there will be no shortage | 13:46 |
| cppcabrera | of marconi flavords | 13:46 |
| cppcabrera | *flavors | 13:46 |
| al-maisan | yeah :) | 13:46 |
| flaper87 | amitgandhi: rust is not just interesting, it is damn sexy | 13:46 |
| cppcabrera | +1 flaper87 | 13:47 |
| kgriffs | al-maisan: Linus +1 | 13:47 |
| al-maisan | flaper87: never say that in front of your girl friend ;) | 13:47 |
| cppcabrera | which reminds me, I want to start hanging out in the rust IRC channel. :P | 13:47 |
| flaper87 | al-maisan: I did, she just told me: As long as you're not talking about another girl, that's fine with me | 13:48 |
| flaper87 | cppcabrera: irc.mozilla.something | 13:48 |
| cppcabrera | .org | 13:48 |
| cppcabrera | I just looked it up. :D | 13:48 |
| flaper87 | cppcabrera: that :D | 13:48 |
| *** russellb is now known as rustlebee | 13:48 | |
| cppcabrera | I get a daily dose of haskell on freenode#haskell (it's so active!), might as well get a dose of rustlang. :) | 13:48 |
| flaper87 | cppcabrera: rust's channel is very active | 13:49 |
| al-maisan | flaper87: pheww .. you were just plain lucky :) | 13:50 |
| * al-maisan googles six.add_metaclass(abc.ABCMeta) | 13:51 | |
| cppcabrera | +1 al-maisan | 13:54 |
| al-maisan | cppcabrera: thank you very much !! | 13:54 |
| *** oz_akan_ has quit IRC | 13:55 | |
| *** oz_akan_ has joined #openstack-marconi | 13:55 | |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: proxy: adds oslo.cache, config support, & stevedore https://review.openstack.org/47475 | 14:07 |
| cppcabrera | that should fix the unit test config loading bug. ^^ | 14:09 |
| flaper87 | kgriffs: when you get a chance, could you pls take a look at marconiclient patches? | 14:20 |
| *** ayoung-ZzZzZz is now known as ayoung | 14:21 | |
| *** ayoung has quit IRC | 14:21 | |
| al-maisan | flaper87: in marconiclient/auth/keystone.py, line 91 you describe 'kwargs' as a param but it's a local dict introduced later in the method body | 14:23 |
| flaper87 | al-maisan: erm, most likely left over from ceilometers code | 14:24 |
| flaper87 | o.0 did I say that? | 14:24 |
| flaper87 | :D | 14:24 |
| flaper87 | al-maisan: I'll fix that | 14:24 |
| flaper87 | al-maisan: pls, -1 the patch with a comment | 14:24 |
| al-maisan | let me look through the rest | 14:24 |
| al-maisan | flaper87: can you please explain why authenticate() needs to be an abstract method? | 14:25 |
| al-maisan | because there is no meaningful default implementation? | 14:25 |
| zyuan | pls review https://review.openstack.org/#/c/45952/ | 14:25 |
| zyuan | this is done | 14:26 |
| flaper87 | al-maisan: yes but, I was thinking about that this morning. I would like to have a noop implementation that just returns the transport obj we talked about | 14:26 |
| zyuan | flaper87: how to add dependecy to openstack? | 14:26 |
| zyuan | we need pylru and mmh3 | 14:27 |
| flaper87 | al-maisan: so, I guess we could either let it land like that and fix implement the noop implementation later or do it in that same patch | 14:27 |
| flaper87 | zyuan: add it to global-requirements.txt in openstack/requirements | 14:27 |
| flaper87 | github.com/openstack/requirements | 14:27 |
| al-maisan | flaper87: is `request` in marconiclient/auth/keystone.py, line 86 meant to be an instance of `transport.request.Spec` ? | 14:27 |
| al-maisan | I am just trying to understand how all these pieces will fit together.. | 14:28 |
| zyuan | flaper87: where is gerrit of this? | 14:28 |
| flaper87 | zyuan: https://review.openstack.org/#/q/status:open+project:openstack/requirements,n,z | 14:29 |
| zyuan | thanks | 14:29 |
| flaper87 | al-maisan: yeah, me too :P | 14:29 |
| flaper87 | al-maisan: so, I think I'll make that patch depend on yours | 14:30 |
| flaper87 | I think we shouldn't pass a request there and instead, let the auth implementation return one | 14:31 |
| flaper87 | I'm thinking about making that work pretty much as a middleware chain would | 14:32 |
| flaper87 | al-maisan: ^ | 14:32 |
| flaper87 | auth[returns req-spec]->lower_api->transport | 14:32 |
| flaper87 | or something like that | 14:32 |
| flaper87 | al-maisan: mmh, actually, I think it might be worth passing the request spec | 14:34 |
| flaper87 | to the auth method | 14:34 |
| al-maisan | yeah .. it's a pass through .. auth just adds the auth data | 14:34 |
| al-maisan | and then it goes to the transport | 14:35 |
| flaper87 | al-maisan: plus, I think keystone's v3 will have RBAC as well, so it'll need to know what ops the user is trying to do | 14:35 |
| flaper87 | ok | 14:35 |
| al-maisan | yup .. so passing the request spec to the auth layer make sense IMHO | 14:36 |
| flaper87 | al-maisan: btw, how are you thinking to differentiate headers from query-string params? | 14:36 |
| flaper87 | I think we should have 2 separate dicts for those | 14:36 |
| al-maisan | isn't that something the transport needs to sort out? What goes where? | 14:36 |
| al-maisan | the request spec is general and does not know about headers | 14:37 |
| flaper87 | al-maisan: mmh, don't think so, for example: | 14:37 |
| flaper87 | if we have 2 different auth providers, they'll most likely use different headers parameters. If we let the transport to sort which param is a header and which is a query param, we'll have to map them somewhere | 14:38 |
| flaper87 | does that make sense? | 14:38 |
| flaper87 | I agree that the impl needs to be generic | 14:38 |
| flaper87 | I mean, it should just care about params, content and endpoints | 14:39 |
| flaper87 | but, I think having them in separate structures will make transport's implementation cleaner and easier | 14:39 |
| al-maisan | flaper87: is there a situation where we have more than one auth provider per transport? e.g. hhtp? | 14:39 |
| kgriffs | flaper87: question - http://paste.openstack.org/show/47318/ | 14:39 |
| al-maisan | *http even | 14:39 |
| kgriffs | do you think we can remove the $or there since c.e is always set/initialized on post message? | 14:40 |
| flaper87 | al-maisan: there could be, I guess. But I'm thinking about the client having to talk to 2 different endpoints that use different auth providers. | 14:40 |
| flaper87 | al-maisan: let me try to explain this better | 14:40 |
| al-maisan | would the client talk to 2 different endpoints simultaneously? | 14:41 |
| al-maisan | flaper87: we can also do a hangout session if that helps.. | 14:41 |
| flaper87 | yeah, lets do that | 14:42 |
| al-maisan | give me 2 minutes | 14:42 |
| flaper87 | kgriffs: looking | 14:43 |
| zyuan | https://review.openstack.org/#/c/47580/ | 14:44 |
| flaper87 | kgriffs: ah yes, I was looking at that yday while reviewing your patch | 14:44 |
| flaper87 | I think we can get rid of that or | 14:44 |
| flaper87 | kgriffs: is c.e set to now on post message? | 14:44 |
| flaper87 | 'now' | 14:45 |
| al-maisan | flaper87: https://plus.google.com/hangouts/_/ca7fe30c69c2c9cf70e7b193dc0d91d375bec5cb?hl=en | 14:45 |
| flaper87 | zyuan: I think you'll need 2 separte patches for that | 14:45 |
| al-maisan | ready whenever you are | 14:45 |
| flaper87 | al-maisan: can you invite me through G+? I'd have to send the link to my ipad otherwise | 14:46 |
| al-maisan | OK | 14:46 |
| * flaper87 can't G+ on his laptop | 14:46 | |
| kgriffs | flaper87: c': {'id': None, 'e': now}, | 14:46 |
| kgriffs | yes | 14:46 |
| flaper87 | kgriffs: cool, I guess we can get rid of that $or | 14:46 |
| kgriffs | excellent | 14:46 |
| kgriffs | I will do that in another patch | 14:47 |
| * flaper87 smiles while imagining those queries going faster | 14:47 | |
| al-maisan | Thank God for G+ hangout :) | 14:56 |
| flaper87 | yeaaahhh | 14:58 |
| zyuan | Google's religion? | 14:58 |
| flaper87 | we figured the whole thing in 5 seconds | 14:58 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: perf(mongodb): Combine project and queue message fields https://review.openstack.org/47259 | 14:58 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load https://review.openstack.org/47272 | 14:58 |
| al-maisan | flaper87: yeah :) | 14:59 |
| openstackgerrit | Muharem Hrnjadovic proposed a change to openstack/python-marconiclient: This adds a transport layer request spec class https://review.openstack.org/47538 | 15:01 |
| al-maisan | flaper87: done ^ | 15:01 |
| * al-maisan signs off | 15:02 | |
| flaper87 | al-maisan: danke! | 15:02 |
| al-maisan | flaper87: di nulla :) | 15:03 |
| cppcabrera | Time to do some reviews. :) | 15:04 |
| *** ayoung has joined #openstack-marconi | 15:04 | |
| flaper87 | cppcabrera: ignore my auth patch for the client, I'll update it in a bit based on al-maisan changes | 15:04 |
| cppcabrera | kk, flaper87. | 15:05 |
| cppcabrera | I'll check out kgriff's patches first. | 15:05 |
| flaper87 | cppcabrera: +1 | 15:05 |
| flaper87 | http://www.pixelbeat.org/patches/coreutils/inbox_aug_2013.html#1379298622 | 15:07 |
| *** reed has joined #openstack-marconi | 15:18 | |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Remove $or clause used to claim messages https://review.openstack.org/47590 | 15:21 |
| * flaper87 can see the smoke comming out kgriffs's keyboard | 15:25 | |
| kgriffs | :D | 15:26 |
| kgriffs | flaper87: I am looking at http://goo.gl/aVGVtg | 15:27 |
| kgriffs | grep for "if isinstance(value, datetime.datetime):" | 15:28 |
| flaper87 | line 429 ? | 15:28 |
| kgriffs | at one point there used to be a bson.datetime | 15:28 |
| kgriffs | right | 15:28 |
| cppcabrera | kgriffs: all patches reviewed. | 15:28 |
| cppcabrera | The marker simplification patch was -1'd because of a potential 'TypeError'. | 15:29 |
| cppcabrera | The rest are +2s, with one awaiting benchmarking. | 15:29 |
| kgriffs | i was thinking, why not have bson.date class which can be constructed from a unix timestamp or default to current time, or constructed from a datetime.datetime | 15:29 |
| kgriffs | internally, it would just store millis | 15:30 |
| kgriffs | using calendar is lame and slow | 15:30 |
| zyuan | cppcabrera: http://paste.openstack.org/show/47322/ | 15:30 |
| kgriffs | imo | 15:30 |
| kgriffs | flaper87: thoughts? | 15:30 |
| zyuan | cppcabrera: with that diff, tests pass (so far) | 15:30 |
| kgriffs | cppcabrera: looking | 15:31 |
| flaper87 | kgriffs: I agree with both the idea and calendar being lame and slow | 15:31 |
| cppcabrera | thanks, zyuan. | 15:32 |
| cppcabrera | I'll go fix up the patch. :) | 15:32 |
| kgriffs | cppcabrera: nice catch there | 15:32 |
| * kgriffs fixes that | 15:32 | |
| kgriffs | btw, i found a nice way to deal with dependent patches | 15:32 |
| *** openstackgerrit has quit IRC | 15:33 | |
| kgriffs | make a change, commit | 15:33 |
| *** openstackgerrit has joined #openstack-marconi | 15:33 | |
| kgriffs | then rebase, and reorder+squash into the correct commit | 15:33 |
| kgriffs | as long as you preserve the change ID, ur good | 15:33 |
| cppcabrera | I've also had good luck preserving the change ID. I go a step further and usually 'git review -d' the dependency and rebase off of that, | 15:34 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: perf(mongodb): Combine project and queue message fields https://review.openstack.org/47259 | 15:38 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load https://review.openstack.org/47272 | 15:38 |
| kgriffs | cppcabrera: fixed those issues you noted ^^^ | 15:39 |
| cppcabrera | +2'd | 15:40 |
| * kgriffs cheers | 15:40 | |
| zyuan | kgriffs: irrc, '-' is allowed in queue name | 15:41 |
| zyuan | use '/' instead | 15:41 |
| kgriffs | hmm, good point | 15:42 |
| kgriffs | how about ':' | 15:42 |
| flaper87 | kgriffs: +1 for : | 15:42 |
| zyuan | '/' is never be used | 15:42 |
| zyuan | allowed* | 15:42 |
| flaper87 | lets use :: as Rust | 15:42 |
| flaper87 | hahahah | 15:42 |
| kgriffs | :: | 15:42 |
| zyuan | ... | 15:42 |
| flaper87 | just kiding | 15:42 |
| zyuan | project/queue | 15:43 |
| flaper87 | zyuan: mmh '...' is a bit long, though | 15:43 |
| zyuan | and just queue when project is None | 15:43 |
| flaper87 | :D | 15:43 |
| zyuan | project/queue and queue | 15:43 |
| cppcabrera | lol | 15:43 |
| cppcabrera | :: is tempting because I already think of projects as namespaces. | 15:44 |
| flaper87 | either work for me, as long as you don't pick 'win-php' namespace style, I'm good | 15:44 |
| cppcabrera | But it's not worth the extra character, heh. | 15:44 |
| zyuan | i jsut pick the char will *never* be used | 15:44 |
| flaper87 | I guess / is fine | 15:44 |
| zyuan | no matter how we change API in the future | 15:44 |
| zyuan | and prject/queue and queue will never clush since neither can contains '/' | 15:46 |
| cppcabrera | marconi proxy is introducing a few config file additions, namely: [drivers:proxy], [drivers:proxy:storage:{type}], and [drivers:proxy:transport:{type}]. How do you guys feel about this? | 15:46 |
| kgriffs | hmm | 15:46 |
| flaper87 | zyuan: ah no? What if we start using unix-paths as queues names | 15:46 |
| * flaper87 runs runs away | 15:46 | |
| zyuan | flaper87: you can't | 15:46 |
| flaper87 | zyuan: duuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuude, it's a joooooooooooooooooooooooooooooooooookeeee | 15:46 |
| kgriffs | i'm ok with just 'queue' when no project, but only if we think that folks won't mix single-tenant and multitenant modes | 15:46 |
| zyuan | we have no way to add this "feature" | 15:46 |
| flaper87 | :D | 15:47 |
| kgriffs | as in, all or nothing | 15:47 |
| cppcabrera | I'm sure we'll find all the edge cases when people *can't* stop using marconi. ;) | 15:47 |
| flaper87 | mmh, but, wouldn't that mess up the "all queues for this project" query ? | 15:47 |
| flaper87 | I'd use None/queue | 15:48 |
| zyuan | cppcabrera: lengthy... you mentioned that you want a separated config file right? | 15:48 |
| flaper87 | plus, what if a queue in tenant None has the same name of an existing project? | 15:48 |
| zyuan | None/queue.............................. | 15:48 |
| kgriffs | flaper87: good point, we want to be able to do a prefix query some day | 15:48 |
| flaper87 | but also, what if a project is called None | 15:48 |
| flaper87 | :P | 15:48 |
| zyuan | i can create a project named "None" | 15:48 |
| flaper87 | zyuan: I won :D | 15:48 |
| zyuan | ... | 15:48 |
| zyuan | queue along works | 15:49 |
| kgriffs | hmm | 15:49 |
| flaper87 | project/queue and /queue I guess | 15:49 |
| cppcabrera | zyuan: I'm considering a separate config file, but I'm not entirely sure how to trigger that. | 15:49 |
| kgriffs | if there is only ONE project (single tenant) then it is fine | 15:49 |
| flaper87 | that would allow us to say "everything after the /" | 15:49 |
| zyuan | what if i creae a project named ""? | 15:49 |
| zyuan | then "/queue" clush with "/queue" | 15:50 |
| cppcabrera | My best guess is that it's determined when you do config.project('marconi_proxy') | 15:50 |
| flaper87 | zyuan: project is not something we create | 15:50 |
| zyuan | flaper87: it can be anothing in test | 15:50 |
| kgriffs | but if you allow project=None as well as project=1234 in the same DB, things aren't as nice | 15:50 |
| flaper87 | but anyway, project "" == no project to me | 15:50 |
| zyuan | so i don't want our code rely on some external facts | 15:50 |
| flaper87 | zyuan: agreed about not relying on external things. | 15:51 |
| kgriffs | _/my-queue | 15:51 |
| zyuan | a project alled "_' | 15:51 |
| kgriffs | 104854/bobs_queue | 15:51 |
| kgriffs | in practice, won't happen | 15:51 |
| kgriffs | project IDs are numbers, uuids, stuff like that | 15:51 |
| zyuan | "$queue" does not depends on practice to work | 15:51 |
| zyuan | it just works and always works... | 15:52 |
| flaper87 | to me no project is like project without name | 15:52 |
| kgriffs | hmm | 15:52 |
| flaper87 | and we can use projects id, btw | 15:53 |
| kgriffs | so to ask the question "what are all the global queues" we would have to say "give me everything that does not contain '/' | 15:53 |
| kgriffs | the trouble with that in Mongo is you can't use an index | 15:53 |
| flaper87 | kgriffs: nope | 15:53 |
| flaper87 | prj/queue or /queue | 15:53 |
| zyuan | i'm ok with "/queue" | 15:53 |
| flaper87 | so, give me ^/ | 15:53 |
| kgriffs | flaper87: right, we would need the prefix there | 15:53 |
| zyuan | there is actualy a bug in the current code which does not allow a project of "" | 15:54 |
| zyuan | but it's arguable | 15:54 |
| zyuan | i'm fine with "project/queue" and "/queue" | 15:54 |
| kgriffs | sure, i just don't want to make schema decision that reduces our options later on | 15:54 |
| kgriffs | I kinda like ^/my-queue | 15:54 |
| kgriffs | to tell you the truth | 15:54 |
| kgriffs | we are picking nits now | 15:55 |
| kgriffs | main thing is to use / | 15:55 |
| kgriffs | stand by | 15:55 |
| zyuan | another possibilty is to use | 15:55 |
| zyuan | /project/queue | 15:55 |
| zyuan | and /queue | 15:55 |
| flaper87 | but then again, if there's a project called /, that query will fail, we need something like ^/\w | 15:55 |
| zyuan | that looks more consistent | 15:55 |
| kgriffs | btw, should i treat the empty string as the same as None? | 15:55 |
| flaper87 | kgriffs: yes | 15:55 |
| flaper87 | IMHO | 15:56 |
| zyuan | otherwise, project/queue and queue | 15:56 |
| zyuan | kgriffs: yes and no | 15:56 |
| flaper87 | I'm fine with either /prj/queue and /queue or prj/queue and /queue | 15:56 |
| flaper87 | I agree with zyuan, /prj/queue is more consistent | 15:56 |
| zyuan | kgriffs: i'm ok with that, but there might be some problem in our code | 15:56 |
| zyuan | kgriffs: i'm None and "" stuff | 15:57 |
| flaper87 | but, dunno if that makes the query more complex, lets make sure we still use the index with the regex | 15:57 |
| zyuan | i mean... | 15:57 |
| zyuan | prj/queue is little bit better than /prj/queue; less similarity, more fair distribution | 15:57 |
| zyuan | (i'm going to use the name directly in multidb) | 15:58 |
| zyuan | for same reason, 'queue' may works better than '/queue', but we might not care about single tenant case that much | 15:59 |
| zyuan | it's a relatively more "private | 15:59 |
| zyuan | " deploment. | 15:59 |
| cppcabrera | ultimately, we're trying to (idx :: String -> String -> String; idx project queue = project ++ '/' ++ queue), so as long as the chosen separator character is in the set of disallowed charcaters, I think it's enough to just use that as a separator. | 15:59 |
| cppcabrera | '/' is one of the disallowed charcaters, IIRC, so that's cool with me. | 16:00 |
| zyuan | idx None queue = queue | 16:00 |
| zyuan | idx project queue = project ++ "/" ++ queue | 16:01 |
| cppcabrera | I've been using '.' while prototyping, fwiw. | 16:01 |
| zyuan | anyway, we can vote | 16:01 |
| flaper87 | '.' is forbidden by mongo | 16:01 |
| cppcabrera | gtk, flaper87 | 16:01 |
| zyuan | i prefer to distinguish None and "" as much as I can | 16:02 |
| zyuan | :) | 16:02 |
| kgriffs | https://etherpad.openstack.org/queuing-scratch | 16:05 |
| kgriffs | I still like ^ | 16:05 |
| cppcabrera | /queues | 16:12 |
| flaper87 | kgriffs: so, just to be clear, I'll create a bug to forbid empty-names for projects | 16:14 |
| flaper87 | and you'll update your patch with what we discussed in the etherpad | 16:15 |
| flaper87 | right? | 16:15 |
| kgriffs | flaper87: right | 16:16 |
| flaper87 | kk | 16:16 |
| flaper87 | danke | 16:16 |
| flaper87 | doing it now | 16:16 |
| kgriffs | also, maybe register a bp for single-tenant mode? | 16:17 |
| flaper87 | not sure, I don't think we care much about that. it's a matter of whether project is being passed or not | 16:17 |
| kgriffs | hmm | 16:17 |
| flaper87 | do we need to distinguish that ? | 16:17 |
| kgriffs | idk what the WSGI driver does if Project-Id is missing | 16:18 |
| kgriffs | I think it complains | 16:18 |
| zyuan | kgriffs: it does not | 16:18 |
| flaper87 | kgriffs: nope, https://github.com/openstack/marconi/blob/master/marconi/queues/transport/wsgi/driver.py#L54 | 16:19 |
| kgriffs | ah, so it would pass through None | 16:19 |
| zyuan | yes | 16:19 |
| zyuan | and! | 16:19 |
| kgriffs | do we have tests covering that? | 16:19 |
| zyuan | mongo driver allows "" | 16:19 |
| zyuan | sqlite driver allows "" but clush with None | 16:20 |
| zyuan | that's what i said "our impl does not have an consensus on this" | 16:20 |
| flaper87 | https://bugs.launchpad.net/marconi/+bug/1228249 | 16:22 |
| flaper87 | brb | 16:23 |
| zyuan | requirements freze | 16:23 |
| zyuan | hehe | 16:23 |
| malini | hmm..what is a project name? | 16:25 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: proxy: memory storage driver + tests + fixes https://review.openstack.org/47399 | 16:26 |
| kgriffs | malini: project ID | 16:26 |
| cppcabrera | ^^ removes normalization at transport layer for proxy | 16:27 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: proxy: mongodb storage driver https://review.openstack.org/47402 | 16:27 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: proxy: adds oslo.cache, config support, & stevedore https://review.openstack.org/47475 | 16:27 |
| cppcabrera | rebase ^^ | 16:28 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: perf(mongodb): Combine project and queue message fields https://review.openstack.org/47259 | 16:36 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load https://review.openstack.org/47272 | 16:36 |
| cppcabrera | reviewed, kgriffs: +2'd on both. | 16:40 |
| cppcabrera | Finding the small changes was made possible by two browser tabs and gerrit math. :P | 16:40 |
| kgriffs | flaper87: ping | 16:52 |
| flaper87 | kgriffs: pong | 17:02 |
| cppcabrera | seeing enough pings and pongs, I start to think that we've got more or less an ad-hoc heartbeat system going in this channel. | 17:05 |
| kgriffs | commented: https://review.openstack.org/#/c/46757/ | 17:09 |
| kgriffs | d00d, whenever i ask keystone questions, I just get crickets | 17:09 |
| kgriffs | i was redirected to #openstack-dev from #openstack-keystone | 17:10 |
| kgriffs | still now luck | 17:10 |
| kgriffs | booh | 17:10 |
| kgriffs | I'll try the mailing list | 17:11 |
| cppcabrera | I've almost got the proxy transport restructured to mirror marconi.queues | 17:13 |
| cppcabrera | I just finished duking it out with stevedore over my tiny errors. :D | 17:13 |
| cppcabrera | woooo, I got it obeying the config file! | 17:14 |
| cppcabrera | Let's see what tox says... | 17:14 |
| kgriffs | flaper87, cppcabrera: thanks for reviewing my comment on that reqs patch | 17:16 |
| cppcabrera | np, kgriffs. | 17:16 |
| cppcabrera | whenever you have time, proxy needs some love. :) | 17:17 |
| cppcabrera | I'm currently finalizing the last patch needed before the proxy is featureful enough to work with at scale. | 17:17 |
| flaper87 | kgriffs: thank you for those lovely patches. | 17:18 |
| zyuan | kgriffs: openstack requirements is frozen | 17:19 |
| zyuan | does not mean we can't get the multidb patch merged? | 17:19 |
| * flaper87 should stop talking about code as it was a human being | 17:19 | |
| zyuan | if not, can we deploy it? | 17:19 |
| kgriffs | flaper87: :D | 17:19 |
| flaper87 | zyuan: no req, no land :( | 17:20 |
| kgriffs | Alex_Gaynor is on the reqs team - maybe he can give you some direction on that? | 17:20 |
| flaper87 | jenkins will fail if the requirements don't match | 17:20 |
| flaper87 | so the patch can't land | 17:20 |
| zyuan | Patch Set 3: I would prefer that you didn't merge this | 17:20 |
| zyuan | We're in a requirements freeze until development on icehouse opens up. | 17:20 |
| zyuan | http://lists.openstack.org/pipermail/openstack-dev/2013-September/015131.html | 17:20 |
| zyuan | https://review.openstack.org/#/c/47580/ | 17:21 |
| flaper87 | yeah, i know! I guess we'll have to wait until havana is released and Ith developemnt starts | 17:22 |
| flaper87 | at least for that patch to land | 17:22 |
| flaper87 | man, what's wrong with jenkins | 17:22 |
| flaper87 | ? | 17:22 |
| flaper87 | T_T | 17:22 |
| cppcabrera | is jenkins grumping again? | 17:23 |
| * cppcabrera checks emails | 17:23 | |
| flaper87 | yeah | 17:23 |
| flaper87 | at least it's really slow | 17:23 |
| cppcabrera | Ahh, I see. | 17:23 |
| * cppcabrera checks -infra | 17:23 | |
| flaper87 | erm, and grumping, I guess | 17:24 |
| flaper87 | :D | 17:24 |
| cppcabrera | zuul was restarted. | 17:24 |
| cppcabrera | Also, I happened upon this... | 17:24 |
| cppcabrera | Add Python 3.3 as Experimental for Oslo-Incubator https://review.openstack.org/47170 | 17:24 |
| cppcabrera | if oslo.incubator gets 3-ized, that's amazing news for the rest of OS. | 17:25 |
| flaper87 | cppcabrera: there's been a lot of work in that area | 17:25 |
| flaper87 | so, I wouldn't be surprissed if the gate is made voting soon. | 17:25 |
| flaper87 | kgriffs: jenkins didn't like your patch :( | 17:26 |
| flaper87 | brb | 17:26 |
| cppcabrera | flaper87: It looks like it's going to be made voting, but experimental. | 17:26 |
| cppcabrera | So you'd have to leave a comment of 'check experimental' to trigger it. | 17:26 |
| cppcabrera | kgriffs: message_scope isn't defined yet in your markers patch, so jenkins is upset about this. | 17:29 |
| zyuan | flaper87: does that mean falcon>0.1.7 can't go as well | 17:31 |
| zyuan | ? | 17:31 |
| cppcabrera | I suspect that is the case, zyuan. We may have to wait on falcon>=0.1.7 'til after Icehouse begins. | 17:36 |
| zyuan | cppcabrera: hehe | 17:38 |
| zyuan | cppcabrera: you saw this right? https://github.com/racker/falcon/issues/185 | 17:38 |
| cppcabrera | yup, zyuan. :P | 17:39 |
| zyuan | then what do you want to do with it? | 17:40 |
| cppcabrera | My thoughts are to work with it as it is, atm. | 17:41 |
| cppcabrera | middleware can be used to normalize headers, if necessary. | 17:41 |
| zyuan | then proxy gives wrong results | 17:41 |
| zyuan | it can't, because they can conflict | 17:41 |
| zyuan | they have different values | 17:41 |
| zyuan | cppcabrera: try this | 17:42 |
| zyuan | '-'.join([s.capitalize() for s in a.split('-')]) | 17:42 |
| zyuan | on requests.headers | 17:43 |
| kgriffs | cppcabrera: crappies, my rebase went awry | 17:49 |
| *** cthulhup has joined #openstack-marconi | 17:51 | |
| *** cthulhup has quit IRC | 17:51 | |
| *** cthulhup has joined #openstack-marconi | 17:52 | |
| *** reed has quit IRC | 17:56 | |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): fix proxy-transport tests https://review.openstack.org/47612 | 17:57 |
| *** cthulhup has quit IRC | 17:58 | |
| amitgandhi | kgriffs: meeting | 18:09 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: perf(mongodb): Combine project and queue message fields https://review.openstack.org/47259 | 18:30 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load https://review.openstack.org/47272 | 18:30 |
| cppcabrera | re-reviewing, kgriffs | 18:31 |
| kgriffs | oz_akan_: can you take a look at this version patch? | 18:45 |
| kgriffs | https://review.openstack.org/#/c/47459/ | 18:45 |
| kgriffs | +1 if it does what you need | 18:45 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: test(proxy): fix proxy-transport tests https://review.openstack.org/47612 | 18:46 |
| oz_akan_ | kgriffs: on the phone, will check after | 18:48 |
| kgriffs | kk, thanks | 18:48 |
| *** leizhang has quit IRC | 18:55 | |
| *** mpanetta has joined #openstack-marconi | 18:57 | |
| mpanetta | Quiet... | 18:58 |
| kgriffs | Too quiet... | 19:01 |
| cppcabrera | :D | 19:01 |
| ekarlso | how's progress marconi folks ? | 19:05 |
| kgriffs | yoyo | 19:06 |
| cppcabrera | it's fun times, ekarlso | 19:06 |
| cppcabrera | busy as ever | 19:06 |
| kgriffs | we are finishing up the last major patches for a "1.0" release. There will still be minor bugs and stuff, but all the big stuff will be good to go. | 19:07 |
| ekarlso | will there be other drivers then mongo btw ? | 19:07 |
| * ekarlso stares wit hevil eyes at GPL | 19:07 | |
| kgriffs | yes | 19:07 |
| kgriffs | looking like SQLAlchemy and Redis | 19:08 |
| kgriffs | won't use the SQLAlchemy ORM | 19:08 |
| kgriffs | (probably) | 19:08 |
| kgriffs | (fwiw) | 19:08 |
| ekarlso | what's the bad with the orm ? :p | 19:08 |
| kgriffs | performance | 19:09 |
| ekarlso | ah | 19:09 |
| kgriffs | and honestly, there are only a handful of docs, so the extra boilerplate of defining objects doesn't seem justified | 19:10 |
| kgriffs | but I can be persuaded otherwise | 19:10 |
| * kgriffs likes brownies | 19:10 | |
| ekarlso | what's the bad thing with sqlalchemy docs ? | 19:10 |
| * kgriffs and pop-tarts | 19:10 | |
| kgriffs | sorry, not docs as in "documentation" | 19:10 |
| kgriffs | doc as in dict or resource or document | 19:11 |
| ekarlso | k | 19:11 |
| * kgriffs has document DBs on the brain | 19:11 | |
| ekarlso | cassandra ? | 19:12 |
| kgriffs | if someone would like to write a 3rd-party driver for it, be my quest, but TBH it isn't well-suited to storing lots and lots of short-lived messages | 19:13 |
| kgriffs | Cassandra slows down quite a bit when you keep deleting things | 19:13 |
| zyuan | btw, ykaplan is going to write sqlalchemy storage right? progress? | 19:13 |
| kgriffs | ekarlso: at least, that was Comcast's experience | 19:14 |
| kgriffs | they wrote an SQS/SNS clone on top of Cassandra and ran into all kinds of problems | 19:14 |
| kgriffs | …but maybe there's a Cassandra hacker who could make it work | 19:15 |
| * kgriffs isn't one | 19:15 | |
| ekarlso | kgriffs: just that GPL is evil though | 19:16 |
| kgriffs | You mean AGPL | 19:16 |
| kgriffs | ? | 19:16 |
| kgriffs | guys: I gotta run for bit | 19:17 |
| cppcabrera | see ya, kgriffs | 19:18 |
| ekarlso | kgriffs: yes :p | 19:18 |
| kgriffs | ekarlso: i totally understand. definitely some angst there | 19:18 |
| kgriffs | ttfn | 19:18 |
| *** kgriffs is now known as kgriffs_afk | 19:19 | |
| *** malini is now known as malini_afk | 19:33 | |
| *** gordonsim has quit IRC | 19:45 | |
| *** kgriffs_afk is now known as kgriffs | 19:49 | |
| *** kgriffs is now known as kgriffs_afk | 19:58 | |
| zyuan | cppcabrera: http://paste.openstack.org/show/47340/ | 19:59 |
| *** kgriffs_afk is now known as kgriffs | 19:59 | |
| zyuan | you also need to set headers v1, queues, whenever you uses requests lib. | 20:00 |
| zyuan | you can wrap the Captitalize code into a function somewhere | 20:00 |
| *** kgriffs is now known as kgriffs_afk | 20:00 | |
| cppcabrera | I'll give that a try in the next patch. | 20:03 |
| cppcabrera | for now, I will save that patch in the marconi-proxy-issues etherpad. | 20:03 |
| cppcabrera | Thanks, zyuan! | 20:03 |
| *** amitgandhi has quit IRC | 20:07 | |
| zyuan | cppcabrera: if you are busy, i can make the patch (on gerrit) | 20:11 |
| cppcabrera | Go ahead, zyuan. I won't get to it today. | 20:20 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: fix(proxy): forward marconi headers https://review.openstack.org/47628 | 20:34 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: test(proxy): forwarded endpoints tested https://review.openstack.org/46963 | 20:55 |
| *** malini_afk is now known as malini | 21:04 | |
| *** oz_akan_ has quit IRC | 21:09 | |
| *** EmilienM has quit IRC | 21:22 | |
| *** EmilienM has joined #openstack-marconi | 21:23 | |
| *** tedross has quit IRC | 21:30 | |
| *** ametts has quit IRC | 21:43 | |
| *** flaper87 is now known as flaper87|afk | 21:44 | |
| *** cppcabrera has quit IRC | 21:57 | |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: proxy: mirror structure of marconi queues + bootstrap https://review.openstack.org/47642 | 22:01 |
| *** mpanetta has quit IRC | 22:06 | |
| *** openstack has quit IRC | 22:35 | |
| *** openstack has joined #openstack-marconi | 22:35 | |
| *** malini is now known as malini_afk | 22:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
