| *** oz_akan_ has quit IRC | 00:04 | |
| *** kgriffs_afk is now known as kgriffs | 00:05 | |
| *** vkmc has quit IRC | 00:05 | |
| *** amitgandhi has joined #openstack-marconi | 00:09 | |
| *** amitgandhi has quit IRC | 00:14 | |
| *** oz_akan_ has joined #openstack-marconi | 00:14 | |
| *** oz_akan_ has quit IRC | 00:18 | |
| *** oz_akan_ has joined #openstack-marconi | 00:18 | |
| *** kgriffs is now known as kgriffs_afk | 00:20 | |
| *** oz_akan_ has quit IRC | 00:22 | |
| *** kgriffs_afk is now known as kgriffs | 00:30 | |
| *** amitgandhi has joined #openstack-marconi | 01:00 | |
| *** reed has quit IRC | 01:02 | |
| *** amitgandhi has quit IRC | 01:04 | |
| *** haomaiwang has joined #openstack-marconi | 01:08 | |
| *** nosnos has joined #openstack-marconi | 01:09 | |
| *** amitgandhi has joined #openstack-marconi | 01:18 | |
| *** oz_akan_ has joined #openstack-marconi | 01:29 | |
| *** kgriffs is now known as kgriffs_afk | 01:29 | |
| *** oz_akan_ has quit IRC | 01:33 | |
| *** kgriffs_afk is now known as kgriffs | 01:34 | |
| *** VeggieMeat has joined #openstack-marconi | 01:39 | |
| *** kgriffs is now known as kgriffs_afk | 01:49 | |
| *** kgriffs_afk is now known as kgriffs | 02:10 | |
| *** oz_akan_ has joined #openstack-marconi | 02:15 | |
| *** malini_afk is now known as malini | 02:16 | |
| *** amitgandhi has quit IRC | 02:21 | |
| *** amitgandhi has joined #openstack-marconi | 02:22 | |
| *** amitgandhi has quit IRC | 02:26 | |
| *** kgriffs is now known as kgriffs_afk | 02:26 | |
| *** malini is now known as malini_afk | 02:29 | |
| *** malini_afk is now known as malini | 02:33 | |
| *** cpallares has quit IRC | 02:36 | |
| *** malini is now known as malini_afk | 02:36 | |
| *** ayoung has joined #openstack-marconi | 02:40 | |
| *** oz_akan__ has joined #openstack-marconi | 03:17 | |
| *** oz_akan_ has quit IRC | 03:17 | |
| *** haomaiwang has quit IRC | 03:18 | |
| *** cpallares has joined #openstack-marconi | 03:21 | |
| *** amitgandhi has joined #openstack-marconi | 03:22 | |
| *** amitgandhi has quit IRC | 03:26 | |
| *** fifieldt has joined #openstack-marconi | 03:36 | |
| *** oz_akan__ has quit IRC | 04:00 | |
| *** oz_akan_ has joined #openstack-marconi | 04:01 | |
| *** oz_akan_ has quit IRC | 04:06 | |
| *** amitgandhi has joined #openstack-marconi | 04:23 | |
| *** cpallares has joined #openstack-marconi | 04:27 | |
| *** amitgandhi has quit IRC | 04:28 | |
| *** amitgandhi has joined #openstack-marconi | 05:23 | |
| *** amitgandhi has quit IRC | 05:28 | |
| *** flwang has joined #openstack-marconi | 05:37 | |
| *** amitgandhi has joined #openstack-marconi | 06:24 | |
| *** nosnos_ has joined #openstack-marconi | 06:25 | |
| *** nosnos has quit IRC | 06:29 | |
| *** amitgandhi has quit IRC | 06:31 | |
| *** nosnos_ has quit IRC | 07:05 | |
| *** nosnos has joined #openstack-marconi | 07:05 | |
| *** amitgandhi has joined #openstack-marconi | 07:27 | |
| *** amitgandhi has quit IRC | 07:31 | |
| *** flaper87|afk is now known as flaper87 | 07:36 | |
| *** amitgandhi has joined #openstack-marconi | 08:28 | |
| *** amitgandhi has quit IRC | 08:33 | |
| *** haomaiwang has joined #openstack-marconi | 08:45 | |
| *** haomaiwang has quit IRC | 08:52 | |
| *** yassine has joined #openstack-marconi | 09:06 | |
| *** haomaiwang has joined #openstack-marconi | 09:14 | |
| *** amitgandhi has joined #openstack-marconi | 09:28 | |
| *** amitgandhi has quit IRC | 09:32 | |
| *** flwang has quit IRC | 09:44 | |
| *** flwang has joined #openstack-marconi | 10:14 | |
| *** amitgandhi has joined #openstack-marconi | 10:29 | |
| *** ykaplan has joined #openstack-marconi | 10:31 | |
| *** amitgandhi has quit IRC | 10:34 | |
| *** aniuskad has joined #openstack-marconi | 10:41 | |
| *** aniuskad has left #openstack-marconi | 10:42 | |
| *** ykaplan has quit IRC | 10:50 | |
| *** ykaplan has joined #openstack-marconi | 10:54 | |
| *** ykaplan has quit IRC | 11:00 | |
| *** ykaplan has joined #openstack-marconi | 11:12 | |
| *** ykaplan has quit IRC | 11:18 | |
| *** amitgandhi has joined #openstack-marconi | 11:30 | |
| *** amitgandhi has quit IRC | 11:34 | |
| *** malini_afk is now known as malini | 11:58 | |
| *** nosnos_ has joined #openstack-marconi | 12:09 | |
| *** nosnos has quit IRC | 12:12 | |
| *** nosnos_ has quit IRC | 12:14 | |
| *** amitgandhi has joined #openstack-marconi | 12:30 | |
| *** amitgandhi has quit IRC | 12:35 | |
| *** malini is now known as malini_afk | 12:49 | |
| *** tedross has joined #openstack-marconi | 12:56 | |
| *** fifieldt has quit IRC | 13:07 | |
| *** alcabrera has joined #openstack-marconi | 13:16 | |
| alcabrera | Good morning! :D | 13:16 |
|---|---|---|
| flaper87 | alcabrera: gooooooood morning | 13:28 |
| flwang | morning or evening :) | 13:30 |
| *** amitgandhi has joined #openstack-marconi | 13:31 | |
| *** amitgandhi has quit IRC | 13:35 | |
| flwang | guys, seems there is no statistics for Marconi in Russell Bryant's website http://russellbryant.net/openstack-stats/ would you mind me adding that? | 13:42 |
| *** vds has quit IRC | 13:46 | |
| *** jcru has joined #openstack-marconi | 13:47 | |
| *** vds has joined #openstack-marconi | 13:56 | |
| alcabrera | flaper87: yo! :) | 14:04 |
| alcabrera | flwang: I don't mind at all. | 14:04 |
| flaper87 | flwang: not at all, I didn't add them because I thought about waiting for graduation | 14:04 |
| flaper87 | but feel free to submit it | 14:04 |
| flwang | just talked with flaper87 and i'm going to contribute some reviews, so i'd like to see my statistic at there :) | 14:04 |
| flwang | flaper87: when will we graduation? | 14:05 |
| flwang | after Icehouse? | 14:05 |
| flaper87 | erm, that's a good question. We're hoping to do that before Icehouse but we'll see | 14:05 |
| alcabrera | yup, yup. We still have quite a few items to address. | 14:06 |
| alcabrera | Lemme see if I can find that link... | 14:06 |
| alcabrera | flwang: https://wiki.openstack.org/wiki/Marconi/Incubation/Graduation | 14:06 |
| flwang | seems we need to meet some criteria for jenkins gate, is it? | 14:06 |
| flwang | alcabrera: cool, thanks | 14:06 |
| alcabrera | so, sqlalchemy is near the top of the list for things to be done (agpl == mongodb). | 14:07 |
| alcabrera | also, the client lib is almost done (thanks, flaper87!) | 14:07 |
| alcabrera | I'm slowly looking into Heat integration, and malini_afk is looking into Tempest. | 14:08 |
| flaper87 | ykaplan is working on the sqlalchemy thing | 14:08 |
| flwang | alcabrera: should we complete all the items before graduation? | 14:08 |
| flaper87 | flwang: yup | 14:08 |
| flwang | btw, why Pecan is a base line for graduation? it's confusing me | 14:09 |
| *** mpanetta has joined #openstack-marconi | 14:09 | |
| alcabrera | flwang: this was given as a requirement when Marconi was reviewed by the TC, because there's a movement to unify... | 14:10 |
| alcabrera | all projects under a common WSGI framework. | 14:11 |
| alcabrera | so~ | 14:11 |
| alcabrera | That's the why of Pecan. | 14:11 |
| alcabrera | Though... | 14:11 |
| alcabrera | I wonder what the state of that requirement is, given the conversations that kgriffs_afk and flaper87 have had with the maintainer of Pecan? | 14:11 |
| flwang | alcabrera: got it, it's coded by dhellmann | 14:11 |
| alcabrera | flwang: yup | 14:12 |
| flwang | seems until now only Ceilometer is using it | 14:12 |
| flaper87 | alcabrera: AFAIK, it's not a *strong* requirement | 14:12 |
| flaper87 | we're definitely going to work on that | 14:12 |
| *** amitgandhi has joined #openstack-marconi | 14:17 | |
| flwang | flaper87: any guide for marconi client? | 14:17 |
| flaper87 | flwang: not yet :( | 14:17 |
| flaper87 | flwang: it's still under heavy development | 14:17 |
| flaper87 | I'll prepare some examples today | 14:17 |
| flwang | flaper87: so what's the command name? :) | 14:18 |
| flwang | I have installed it | 14:18 |
| flaper87 | flwang: there's no command :D | 14:18 |
| flaper87 | we'd like to keep it as a library | 14:18 |
| flwang | pls kill me :) | 14:18 |
| *** jergerber has joined #openstack-marconi | 14:18 | |
| flaper87 | and use openstack-cli for the CLI | 14:18 |
| flwang | oh, sounds good | 14:18 |
| flaper87 | https://wiki.openstack.org/wiki/OpenStackClient | 14:19 |
| flwang | so for now, i have to use curl to test something, is it? | 14:19 |
| flaper87 | flwang: so far, there's support for queues and messages - hopefully will land today. | 14:19 |
| *** ykaplan has joined #openstack-marconi | 14:19 | |
| flaper87 | Guys, look who's here!!! ykaplan :D | 14:20 |
| ykaplan | flaper87 :) | 14:20 |
| alcabrera | ykaplan: hey! :D | 14:21 |
| alcabrera | flwang: I love using httpie for interacting with marconi. | 14:21 |
| mpanetta | mmmm httpie | 14:22 |
| flwang | httpie? | 14:22 |
| flwang | so seems I can use any http client plugin of browser, right? | 14:22 |
| alcabrera | pip install httpie: https://github.com/jkbr/httpie | 14:22 |
| alcabrera | it's like a *really* friendly curl. | 14:22 |
| alcabrera | flwang: that's right. :) | 14:23 |
| mpanetta | Makes things so much easier | 14:23 |
| alcabrera | marconi's API is pretty simple | 14:23 |
| flwang | alcabrera: got it, cool | 14:23 |
| flwang | hope I can contribute a little bit :) | 14:23 |
| alcabrera | flwang: we're happy to help, and there's a lot of places where marconi can use a boost! :D | 14:26 |
| *** vds has quit IRC | 14:26 | |
| flwang | alcabrera: yep, believe so :) | 14:27 |
| *** vds` has joined #openstack-marconi | 14:33 | |
| flwang | btw, any body can help me clear the mission of Marconi? is it only used by Openstack as a queue? Or it can be leverage by the cloud user to publish a queue service, I mean for the tenant user | 14:35 |
| *** vds` is now known as vds | 14:39 | |
| alcabrera | flwang: I can clarify on this. | 14:39 |
| flwang | alcabrera: appreciate :) | 14:39 |
| alcabrera | Marconi is a queuing service that can: 1) be used by openstack, 2) can also be used by cloud providers to make available a queuing service to users, and 3) provide an open alternative to existing queuing services, so in short... | 14:40 |
| alcabrera | opensource, openstack-integrated, queuing for the cloud | 14:40 |
| alcabrera | I wonder if I reached my buzzword limit for the day? :P | 14:41 |
| flwang | alcabrera: thank you so much. so any overlap with oslo.messaging? | 14:42 |
| alcabrera | flwang: I'm sure we'll find some eventually! IMO, oslo.mesasging is more general-purpose. The goal of that project (AFAIK) is to expose a common abstraction usable for RPC over different transports. The point of overlap with Marconi might be with regards to alternative transport implementations, e.g., how the users of marconi interact with the service (zmq, websockets, etc.). | 14:44 |
| flwang | gotcha, thanks :) | 14:46 |
| flaper87 | alcabrera: that's the goal of oslo.messaging | 14:47 |
| flaper87 | :) | 14:47 |
| *** malini_afk is now known as malini | 14:48 | |
| alcabrera | flaper87: thanks for verifying! | 14:49 |
| flwang | thanks for you guys answer my idiot questions :) | 14:50 |
| alcabrera | no idiot questions here - ask whatever you want. I won't judge! ;) | 14:51 |
| *** ekarlso has quit IRC | 14:51 | |
| *** ekarlso has joined #openstack-marconi | 14:52 | |
| *** ekarlso has quit IRC | 14:52 | |
| *** ekarlso has joined #openstack-marconi | 14:53 | |
| flwang | alcabrera: haha, thanks | 15:08 |
| *** alcabrera is now known as alcabrera|afk | 15:09 | |
| *** kgriffs_afk is now known as kgriffs | 15:14 | |
| *** aniuskad has joined #openstack-marconi | 15:15 | |
| *** ykaplan has quit IRC | 15:37 | |
| flwang | seems there is a weekly meeting after 10 mins, right? | 15:48 |
| *** aniuskad has left #openstack-marconi | 15:48 | |
| flaper87 | flwang: every Monday at 16:00 UTC | 15:49 |
| flwang | flaper87: got it | 15:49 |
| flwang | it's means 00:00am for me :( | 15:50 |
| *** ykaplan has joined #openstack-marconi | 15:50 | |
| flaper87 | flwang: damnit! | 15:51 |
| flaper87 | I hate TZs | 15:51 |
| flwang | yep, please let me know if there is any company is hiring a Chinese OpenStacker :) | 15:51 |
| flwang | I'd like to live in the west sphere :) | 15:52 |
| flaper87 | flwang: man, we have to make the west want to live in the East | 15:53 |
| flaper87 | :D | 15:53 |
| *** kgriffs is now known as kgriffs_afk | 15:53 | |
| *** kgriffs_afk is now known as kgriffs | 15:58 | |
| *** kgriffs is now known as kgriffs_afk | 16:12 | |
| *** amitgandhi has quit IRC | 16:18 | |
| *** alcabrera|afk is now known as alcabrera | 16:21 | |
| flwang | alcabrera: ping | 16:31 |
| alcabrera | flwang: o/ | 16:31 |
| flwang | seems Marconi is using PUT to create a new queue, right? | 16:31 |
| alcabrera | that's correct. | 16:32 |
| alcabrera | PUT /v1/queues/{name} | 16:32 |
| flwang | hmm... why we don't use POST? | 16:33 |
| flwang | based on my test, if user run PUT for an existed queue name, the response is 204. for a new name, it's 201 | 16:35 |
| flwang | but you know, PUT is idempotent | 16:35 |
| *** amitgandhi has joined #openstack-marconi | 16:37 | |
| alcabrera | lessee... | 16:37 |
| alcabrera | POST isn't used because we want the user to select the name of the queue. | 16:37 |
| alcabrera | flwang: as for the idempotency - I forgot the rationale behind the choice to return 201 vs. 204. kgriffs_afk would know best in this area. | 16:39 |
| flwang | i see. so we just want the user to use the format /v1/queues/{name} without content to create new queue, right? | 16:39 |
| alcabrera | yup! | 16:40 |
| flwang | alcabrera: sure, you know, i'm new, so I just look around and wanna find some bugs if there is :) | 16:40 |
| alcabrera | ooohh, we do maintain a list of existing bugs! :D | 16:41 |
| alcabrera | https://bugs.launchpad.net/marconi/ | 16:41 |
| flwang | haha, i see. I'm always working on Glance actually :) | 16:41 |
| flwang | working with flaper87 most of time as glance core | 16:41 |
| flwang | flaper87 suggest me take a look at Marconi, so i'm here to join you guys to get some fun :) | 16:42 |
| alcabrera | cool. I've only worked on Marconi so far. :) | 16:42 |
| alcabrera | It is pretty cool, and there's always room to try new ideas. | 16:43 |
| * flaper87 back | 16:43 | |
| flwang | yep | 16:43 |
| alcabrera | The coolest part, IMO, is the pluggable storage/transport drivers. | 16:43 |
| flaper87 | Marconi is so freaking, super, dupper awesome | 16:43 |
| flaper87 | :D | 16:43 |
| flwang | flaper87: see my above question? | 16:44 |
| flaper87 | flwang: POST one ? | 16:44 |
| flwang | yep, any comments? | 16:44 |
| flaper87 | flwang: so, part of the rational there is that queues are lazy resources - not as lazy as I'd like - and we don't want to treat them as required. | 16:45 |
| flaper87 | for example | 16:45 |
| flaper87 | you can query an un-existing queue for messages and it won't fail with 404 but return an empty list of message. (Hopefully, I didn't forget how our API works) | 16:46 |
| flwang | hmm... but do you think it's the desire action user want | 16:48 |
| flaper87 | and the answer to the 204 status code is that we didn't change the content nor got anything back | 16:48 |
| flwang | i mean user may just want to query 'helllllo', but he input 'hellllo', then a new queue will be created | 16:48 |
| flaper87 | the fewer stesp you have to take to post a message, the better and easier it'll be to do that | 16:48 |
| flaper87 | flwang: we don't create queues on queries | 16:49 |
| flaper87 | just using PUT | 16:49 |
| flaper87 | but if you query an un-existing queue, Marconi doesn't raise an exception | 16:49 |
| flwang | sorry, forget previous | 16:49 |
| flwang | ok, the only reason we use PUT to create a queue is we just wanna use the consistent format /v1/queues/{name}, right? | 16:50 |
| alcabrera | flwang: the reason for using PUT is to allow the user to name their queues. | 16:54 |
| flwang | ok, i see. | 16:54 |
| flwang | thanks for the clarification, guys | 16:54 |
| alcabrera | np. :) | 16:55 |
| flaper87 | alcabrera: news from kgriffs ? | 16:58 |
| alcabrera | flaper87: none so far. I think he's still rocking the San Francisco Solum meetup. | 17:00 |
| flaper87 | alcabrera: kk, let me know if we need some mafia intervention | 17:01 |
| flaper87 | did I just say that? :D | 17:01 |
| * flaper87 is not Italian | 17:02 | |
| alcabrera | lol | 17:02 |
| mpanetta | There are other mafias that may work as well :P | 17:02 |
| mpanetta | I hear if you need some hacking done, the russian mafia is a good choice :P | 17:03 |
| flaper87 | are there other mafias ? | 17:05 |
| flaper87 | this is war | 17:05 |
| * flaper87 gets his thompson M1928 | 17:07 | |
| * alcabrera watches the explosions while eating a salad | 17:08 | |
| *** cpallares has joined #openstack-marconi | 17:17 | |
| flaper87 | cpallares: ping | 17:25 |
| cpallares | hi flaper87! | 17:25 |
| flaper87 | cpallares: helloooooo | 17:26 |
| flaper87 | :) | 17:26 |
| flaper87 | any news ? | 17:26 |
| flaper87 | :) | 17:26 |
| flaper87 | :) | 17:26 |
| flaper87 | :) | 17:26 |
| flaper87 | :) | 17:26 |
| cpallares | hahaha, wellllllllll... i broke my local marconi :s i'm trying to fix it | 17:27 |
| flaper87 | whaaaaaaaaaaaaaaattttttttttttT????? | 17:27 |
| flaper87 | Marconi never breaks | 17:27 |
| flaper87 | O.O | 17:27 |
| cpallares | flaper87: I broke it u_u | 17:28 |
| cpallares | flaper87: ah nvm, it works now :D | 17:29 |
| cpallares | flaper87: idk what i did but it kept telling me it couldn't find wsgi | 17:30 |
| flaper87 | mmh, mhh, mhhh | 17:30 |
| flaper87 | it's all alcabrera's fault | 17:30 |
| flaper87 | don't worry | 17:30 |
| cpallares | flaper87: yep, it sure looks that way tsk tsk alcabrera | 17:31 |
| cpallares | breaking my marconi | 17:31 |
| flaper87 | so, have you had time to look / work on the API jsonschema schema ? | 17:31 |
| * alcabrera checks to see what he broke this time | 17:32 | |
| cpallares | flaper87: you mean besides look at it and thinking 'what is this'? | 17:32 |
| * cpallares can't read chinese | 17:32 | |
| alcabrera | yup, I took all the wsgi's in the world roughly 5 minutes ago. | 17:33 |
| alcabrera | I brought them back. Everything's better now. | 17:33 |
| cpallares | hahaha just in time | 17:34 |
| alcabrera | :D | 17:34 |
| cpallares | flaper87: I've been looking into jsonschema and trying to figure out the differences | 17:36 |
| openstackgerrit | A change was merged to openstack/marconi: Sync with global requirements https://review.openstack.org/54415 | 17:37 |
| *** kgriffs_afk is now known as kgriffs | 17:37 | |
| flaper87 | o.0 | 17:39 |
| flaper87 | stupid jenkins | 17:39 |
| flaper87 | cpallares: ok, lets do this. I gotta step out for dinner and I'll be back later. We can then go through the migration from HTTP defined API's to jsonschema | 17:39 |
| cpallares | flaper87: yeeees, thank you :D | 17:40 |
| *** ykaplan has quit IRC | 17:42 | |
| *** amitgandhi has quit IRC | 17:45 | |
| *** reed_ has joined #openstack-marconi | 17:45 | |
| *** whenry has joined #openstack-marconi | 17:51 | |
| *** ykaplan has joined #openstack-marconi | 17:52 | |
| *** amitgandhi has joined #openstack-marconi | 17:57 | |
| *** whenry has quit IRC | 18:08 | |
| cpallares | alcabrera, how do i quit marconi locally? | 18:09 |
| alcabrera | cpallares: hmm... | 18:12 |
| alcabrera | I usually just Ctrl-C. | 18:12 |
| cpallares | alcabrera: ah yeah I had a terminal running in the background which i couldn't find | 18:14 |
| * cpallares feels silly | 18:14 | |
| alcabrera | I've had to start putting titles on my terminals since I started working on marconi. Sometimes, I'll have 4 or 5 open, with a few tabs each. It's easy to get mixed up. :) | 18:15 |
| cpallares | alcabrera: how long have you been working on marconi? i saw your video on youtube when it wasn't part of openstack yet | 18:18 |
| alcabrera | I joined the project around April or May of this year, so about ~6 months. | 18:20 |
| alcabrera | It took awhile to put all the pieces together. I made that video to help me understand how everything fit together. | 18:21 |
| cpallares | alcabrera: yes, it helped me too :) thank you for making it | 18:21 |
| *** kgriffs is now known as kgriffs_afk | 18:21 | |
| alcabrera | cpallares: glad to hear it. :D | 18:22 |
| alcabrera | I'm thinking of making another one around the start of next year that explains how to write a new storage backend for marconi. | 18:22 |
| *** kgriffs_afk is now known as kgriffs | 18:24 | |
| cpallares | alcabrera: you should do it! :) i bet there's developers who'd like to customize marconi with a different backend. | 18:27 |
| cpallares | alcabrera: I like how marconi is modular, pretty neat design | 18:27 |
| alcabrera | cpallares: it's mny favorite part of marconi - the modularity! :D | 18:30 |
| alcabrera | I'm curious to see what marconi with Redis runs like, and marconi with postgresql, and marconi with (insert DB here). | 18:31 |
| alcabrera | What is faster? What is easier? What works better with little messages? What works better with big messages? There's so many possibilities! | 18:31 |
| alcabrera | And marconi makes it easy to play. :) | 18:31 |
| kgriffs | alcabrera: http://goo.gl/N7er6j | 18:38 |
| cpallares | alcabreara: that is pretty cool :) | 18:39 |
| kgriffs | wait for it... | 18:39 |
| kgriffs | wait for it... | 18:39 |
| alcabrera | kgriffs: :P | 18:39 |
| *** jcru has quit IRC | 18:40 | |
| kgriffs | d00000d, zuul is taking it's time today | 18:40 |
| alcabrera | very much so! | 18:47 |
| alcabrera | Tiny patch gets approved a@ 8am | 18:47 |
| alcabrera | It gets merged 4 hours later. :P | 18:47 |
| alcabrera | Same thing happened yesterday. | 18:47 |
| alcabrera | I haven't kept up with -infra, but I suspect something is going on. | 18:48 |
| alcabrera | kgriffs: w000t, sharding patch 1 - merg... queue'd! | 18:50 |
| *** ykaplan has quit IRC | 18:55 | |
| alcabrera | brb | 19:04 |
| *** alcabrera is now known as alcabrera|afk | 19:04 | |
| *** jcru has joined #openstack-marconi | 19:19 | |
| *** alcabrera|afk is now known as alcabrera | 19:19 | |
| * flaper87 half back | 19:23 | |
| flaper87 | uuuu | 19:23 |
| flaper87 | kgriffs: is around | 19:23 |
| flaper87 | or kinda | 19:23 |
| flaper87 | kgriffs: https://review.openstack.org/#/c/52389/ | 19:23 |
| flaper87 | did I drop that in here? | 19:23 |
| flaper87 | :D | 19:23 |
| flaper87 | kgriffs: how are you? How's SFO and the Solum hackathon ? | 19:24 |
| flaper87 | awesome awesome awesome, kgriffs +2 the sharding patch | 19:24 |
| flaper87 | w0000000000000000000000000000t | 19:24 |
| alcabrera | :) | 19:25 |
| flaper87 | ok, with that! I've to step out again! | 19:25 |
| flaper87 | :D | 19:25 |
| flaper87 | brb | 19:25 |
| kgriffs | flaper87: solum talks are going well. We are converging on a shared vision and general understanding of the architecture and process the team will follow. | 19:27 |
| kgriffs | folks should be able to hit the ground running after this | 19:27 |
| *** ametts has joined #openstack-marconi | 19:47 | |
| *** kgriffs is now known as kgriffs_afk | 19:48 | |
| *** reed_ is now known as reed | 19:54 | |
| *** reed has quit IRC | 19:54 | |
| *** reed has joined #openstack-marconi | 19:54 | |
| *** kgriffs_afk is now known as kgriffs | 20:07 | |
| cpallares | hi flaper87! | 20:12 |
| *** malini has left #openstack-marconi | 20:18 | |
| flaper87 | cpallares: yo! | 20:19 |
| flaper87 | back | 20:20 |
| *** asalkeld is now known as are_we_all_chang | 20:20 | |
| cpallares | flaper87: how was your dinner? | 20:20 |
| *** are_we_all_chang is now known as angusss | 20:20 | |
| cpallares | flaper87: if it's too late in your time zone right now we can just meet tomorrow. I can try to make it extra early | 20:21 |
| kgriffs | cpallares: what is your timezone? | 20:21 |
| cpallares | us central | 20:22 |
| flaper87 | cpallares: no worries | 20:24 |
| flaper87 | we can talk | 20:24 |
| flaper87 | :) | 20:24 |
| flaper87 | cpallares: soooooooooooooo, API | 20:24 |
| cpallares | flaper87: okay awesome | 20:25 |
| *** angusss is now known as asalkeld | 20:26 | |
| flaper87 | k, sorry, got distracted with email | 20:27 |
| flaper87 | cpallares: so, what we have to do right now is create a representation of our API v1 using jsonschema | 20:27 |
| * flaper87 tries to get the etherpad link | 20:28 | |
| flaper87 | and by try I mean, 'try to remember where the heck that link is' | 20:28 |
| flaper87 | cpallares: https://etherpad.openstack.org/p/cross-transport-api-spec | 20:29 |
| flaper87 | cpallares: there? | 20:29 |
| cpallares | flaper87: yes | 20:29 |
| flaper87 | cpallares: cool | 20:29 |
| flaper87 | cpallares: this is what our API looks like: https://wiki.openstack.org/wiki/Marconi/specs/api/v1 | 20:30 |
| cpallares | yes | 20:30 |
| cpallares | it's just specifying which endpoints and getting those endpoints, right? | 20:31 |
| flaper87 | cpallares: correct | 20:31 |
| flaper87 | cpallares: this is the json-schema URL, http://json-schema.org/ | 20:32 |
| flaper87 | for reference | 20:32 |
| flaper87 | cpallares: and this is an example of what I did for the client: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/queues/v1/api.py | 20:32 |
| flaper87 | TBH, I think you could copy that as is | 20:32 |
| cpallares | i don't need to change the end points to fit the marconi api? | 20:33 |
| cpallares | i thought they weren't the same | 20:35 |
| flaper87 | They are the same, the only difference is that it's in the client but it should be in the server :P | 20:35 |
| flaper87 | so, the client shouldn't have any part of the API defined there, instead, it should 'download' the schema and use that | 20:36 |
| flaper87 | (I know that sounds a lot like SOAP but no) | 20:36 |
| flaper87 | :D | 20:36 |
| cpallares | flaper87: mmm... download the schema? from where? Marconi | 20:37 |
| flaper87 | cpallares: yup | 20:38 |
| flaper87 | but we'll get there later | 20:38 |
| cpallares | flaper87: okay, so I'm confused as to what I need to do right now. Where do I copy it to? | 20:38 |
| openstackgerrit | A change was merged to openstack/marconi: feat: connect sharding manager to control drivers https://review.openstack.org/54605 | 20:39 |
| flaper87 | cpallares: ok, that's something we need to decide | 20:39 |
| flaper87 | :) | 20:39 |
| flaper87 | cpallares: together | 20:39 |
| cpallares | flaper87: ah okay :) | 20:39 |
| flaper87 | cpallares: so | 20:40 |
| flaper87 | cpallares: right now we have queues/{transport,storage} | 20:41 |
| flaper87 | I was thinking that maybe adding a new package under queues called api, makes sense | 20:41 |
| flaper87 | https://github.com/openstack/marconi/tree/master/marconi/queues | 20:41 |
| flaper87 | cpallares: ^ | 20:41 |
| flaper87 | at some point - I'm not sure about this - we may be able to pull transports out of queues and reuse that code for the notification stuff | 20:41 |
| flaper87 | but we'll get there | 20:42 |
| flaper87 | This is what I was thinking: queues/api/{v1,v2,v3,....} | 20:42 |
| cpallares | flaper87: okay, that makes sense | 20:42 |
| flaper87 | cpallares: feel free to say: "Shut up you silly, that doesn't make sense at all" | 20:43 |
| flaper87 | ok? | 20:43 |
| openstackgerrit | A change was merged to openstack/python-marconiclient: Bootstrap Messages support https://review.openstack.org/52389 | 20:43 |
| cpallares | flaper87: haha okay | 20:43 |
| flaper87 | I CANNOT BELIEVE IT! | 20:43 |
| flaper87 | the messages patch was merged | 20:43 |
| * flaper87 cries | 20:43 | |
| flaper87 | w00000t | 20:43 |
| flaper87 | we officially have support for messages in the client | 20:43 |
| cpallares | yay! | 20:43 |
| * cpallares high-fives flaper87 | 20:44 | |
| flaper87 | kgriffs: thanks for reviewing it! :) | 20:44 |
| flaper87 | I know you're busy there | 20:44 |
| kgriffs | :D | 20:44 |
| flaper87 | this one failed to merge: https://review.openstack.org/#/c/53669/ :( I'll rebase it tomorrow | 20:45 |
| kgriffs | https://gist.github.com/anonymous/ec7f1a1e61ae80837147 | 20:45 |
| kgriffs | flaper87: kk | 20:45 |
| flaper87 | kgriffs: https://review.openstack.org/#/c/57265/ this one should be straightforward to review | 20:45 |
| flaper87 | LOOOOL at that gist | 20:45 |
| flaper87 | cpallares: so, if that makes sense we could start adding those packages | 20:48 |
| cpallares | flaper87: you mean the client api? | 20:48 |
| flaper87 | cpallares: nope, I mean: https://github.com/openstack/marconi/tree/master/marconi/queues | 20:49 |
| flaper87 | Lets add the marconi/queues/api/v1 package | 20:49 |
| flaper87 | just to prepare the ground for the API work | 20:49 |
| flaper87 | :) | 20:49 |
| kgriffs | flaper87: The Solum design summit ends today, so I should have more time for reviews tomorrow | 20:50 |
| flaper87 | then we'll add the api | 20:50 |
| cpallares | okay so all those should be inside of api? | 20:50 |
| kgriffs | (fwi) | 20:50 |
| kgriffs | s/fwi/fyi | 20:50 |
| kgriffs | cpallares: where are you located? | 20:50 |
| kgriffs | (out of curiosity) | 20:50 |
| cpallares | flaper87: queues/api/storage | 20:50 |
| cpallares | kgriffs: I'm in Dallas, Texas :) fresh out of Austin, Texas | 20:51 |
| kgriffs | cool | 20:51 |
| cpallares | kgriffs, where are you? | 20:51 |
| kgriffs | I'm in the Austin Rackspace office | 20:51 |
| kgriffs | feel free to ping me next time you are in town | 20:51 |
| cpallares | kgriffs: ah no way :) | 20:51 |
| kgriffs | we can meet up | 20:51 |
| cpallares | kgriffs: I will. I love visiting Austin. | 20:51 |
| * flaper87 is in Como, Texas | 20:51 | |
| cpallares | hahaha | 20:52 |
| kgriffs | heh | 20:52 |
| cpallares | flaper87: sooo queues/api/storage and queues/api/transport | 20:53 |
| flaper87 | cpallares: nope, queues/api/v1 | 20:54 |
| flaper87 | cpallares: sorry If I sometimes fail at explaining things :) | 20:54 |
| cpallares | flaper87: okay so we're adding an /api/v1 to queues and adding what packages? | 20:55 |
| flaper87 | cpallares: The idea behind queues/api/ is to have a package that will contain sub-packages for each API version we support | 20:56 |
| cpallares | so for http, zmq, etc? | 20:56 |
| *** jergerber has quit IRC | 20:57 | |
| flaper87 | cpallares: yeah, http and zmq will then introspect the API defined under queues/api/v1 and expose that through the transport | 20:57 |
| cpallares | flaper87: ok i get you | 20:57 |
| flaper87 | cpallares: cool | 20:58 |
| cpallares | so the marconi client api could just be placed under api/v1/ ? | 20:58 |
| cpallares | flaper87: that's where we're copying it to? | 20:58 |
| alcabrera | I'm out, guys. Have a good night! :) | 20:59 |
| flaper87 | alcabrera: take care | 20:59 |
| *** alcabrera has quit IRC | 20:59 | |
| flaper87 | cpallares: lets ignore the client for now | 20:59 |
| cpallares | bye alcabrera have a good night | 20:59 |
| cpallares | flaper87: okay | 21:00 |
| flaper87 | cpallares: first step is to create those 2 empty packages | 21:00 |
| cpallares | flaper87: okay | 21:01 |
| cpallares | flaper87: ...and then? | 21:03 |
| flaper87 | cpallares: then we need to pull into Marconi, a base class for the API: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/transport/api.py | 21:04 |
| *** kgriffs is now known as kgriffs_afk | 21:04 | |
| flaper87 | IMHO, that should go under marconi/common | 21:04 |
| flaper87 | until we find a better place for it, thoughts? | 21:04 |
| cpallares | flaper87: why common? | 21:05 |
| flaper87 | because that class will be used by the notification API as well | 21:06 |
| flaper87 | cpallares: notice that's the base class that all API definitions will inherit from | 21:06 |
| cpallares | flaper87: oh okay, it's a non-specific api class | 21:07 |
| flaper87 | cpallares: yup | 21:07 |
| cpallares | flaper87: makes sense to place it there | 21:08 |
| flaper87 | cool! | 21:08 |
| flaper87 | cpallares: then you can copy this under queues/api/v1/schema.py https://github.com/openstack/python-marconiclient/blob/master/marconiclient/queues/v1/api.py | 21:09 |
| flaper87 | or should we call it api.py instead of schema ? | 21:09 |
| cpallares | flaper87: api makes sense | 21:09 |
| flaper87 | I don't like having api twice in the module path: marconi.queues.api.v1.api | 21:10 |
| cpallares | cpallares: good point | 21:10 |
| cpallares | flaper87: schema it is | 21:10 |
| flaper87 | we can rename it later | 21:11 |
| flaper87 | cpallares: also, pull this under tests/unit/common https://github.com/openstack/python-marconiclient/blob/master/tests/unit/transport/test_api.py | 21:12 |
| cpallares | flaper87: okay | 21:13 |
| *** ayoung has quit IRC | 21:14 | |
| flaper87 | cpallares: I suggest you to do the package creation first and then pulling the base api class + the test_api | 21:16 |
| cpallares | flaper87: I did it as you were talking about it :) | 21:16 |
| flaper87 | cpallares: awesome :D | 21:16 |
| flaper87 | cpallares: we'll need to complete the api definition so that should definitely land in a separate commit | 21:17 |
| cpallares | flaper87: I don't understand, a separate commit? | 21:18 |
| flaper87 | cpallares: separate review | 21:19 |
| cpallares | ummmmmmm.... so the api_test and the api are separate commits? | 21:19 |
| * cpallares is confused | 21:19 | |
| *** ayoung has joined #openstack-marconi | 21:19 | |
| flaper87 | nope | 21:20 |
| flaper87 | you want to 2 this in 2 commits: 1 that creates the package swe talked about and pulls the base api class and the test | 21:20 |
| flaper87 | to do this* | 21:20 |
| cpallares | flaper87: okay yes | 21:20 |
| flaper87 | the second review will contain https://github.com/openstack/python-marconiclient/blob/master/marconiclient/queues/v1/api.py | 21:21 |
| flaper87 | that we need to complete | 21:21 |
| flaper87 | following the spec we have in the wiki | 21:21 |
| cpallares | flaper87: ahh we need to complete it. That's where I was confused. I thought we needed to change it. | 21:22 |
| flaper87 | cpallares: sorry about that! :D | 21:22 |
| cpallares | flaper87: no worries, I got it :D | 21:23 |
| cpallares | flaper87: the first commit should not contain the schema.py at all? | 21:24 |
| flaper87 | cpallares: correct | 21:25 |
| cpallares | flaper87: git won't let me add an empty directory | 21:27 |
| cpallares | flaper87: wait, do i commit this right now or both when I finish the schema.py? | 21:28 |
| flaper87 | cpallares: ah, you need to create a __init__.py under each package | 21:29 |
| flaper87 | :D | 21:29 |
| cpallares | flaper87: ah i didn't know that | 21:31 |
| cpallares | flaper87: I was wondering why al the __init__.py's were empty. | 21:31 |
| flaper87 | cpallares: http://docs.python.org/2/tutorial/modules.html#packages | 21:32 |
| flaper87 | you may want to take a look there | 21:32 |
| flaper87 | init files can have code, it's not recommemded, though. | 21:32 |
| flaper87 | recommended* | 21:32 |
| *** kgriffs_afk is now known as kgriffs | 21:33 | |
| cpallares | flaper87: thanks | 21:34 |
| openstackgerrit | Cindy Pallares proposed a change to openstack/marconi: Added empty packages and a test for the API v1 https://review.openstack.org/57536 | 21:45 |
| flaper87 | cpallares: the test should go here: https://github.com/openstack/marconi/tree/master/tests/unit/common | 21:48 |
| *** kgriffs is now known as kgriffs_afk | 21:48 | |
| flaper87 | cpallares: and the test is for https://github.com/openstack/python-marconiclient/blob/master/marconiclient/transport/api.py | 21:48 |
| flaper87 | so you may want to copy that file under marconi/common/api.py | 21:48 |
| *** kgriffs_afk is now known as kgriffs | 21:57 | |
| cpallares | flaper87: mmmm... the api_test is importing errors module from marconi, but there's no errors module there... | 22:01 |
| flaper87 | cpallares: yeah, you'll have to tweak that code a bit and adapt it to marconi | 22:05 |
| flaper87 | there are some imports from marconiclient that you'll need to replace | 22:05 |
| *** amitgandhi has quit IRC | 22:06 | |
| *** amitgandhi has joined #openstack-marconi | 22:08 | |
| *** amitgandhi has quit IRC | 22:09 | |
| *** amitgandhi has joined #openstack-marconi | 22:09 | |
| *** kgriffs is now known as kgriffs_afk | 22:12 | |
| *** kgriffs_afk is now known as kgriffs | 22:14 | |
| *** ekarlso has quit IRC | 22:15 | |
| *** ekarlso has joined #openstack-marconi | 22:17 | |
| cpallares | flaper87: the test_api uses a method called fakeApi() which doesn't exist in the api.py, is that in another file imported by api.py? | 22:23 |
| cpallares | flaper87: or why is that fakeapi() there? | 22:23 |
| *** tedross has quit IRC | 22:23 | |
| flaper87 | cpallares: you mean, https://github.com/openstack/python-marconiclient/blob/master/tests/unit/transport/test_api.py#L25 | 22:24 |
| cpallares | yeah I get a 'module' object has no attribute 'FakeApi' | 22:25 |
| *** kgriffs is now known as kgriffs_afk | 22:26 | |
| flaper87 | cpallares: you may need to port that fakeapi module as well | 22:28 |
| flaper87 | it implements a FakeApi to use just for tests | 22:28 |
| *** kgriffs_afk is now known as kgriffs | 22:34 | |
| *** mpanetta has quit IRC | 22:42 | |
| *** yassine has quit IRC | 22:42 | |
| *** kgriffs is now known as kgriffs_afk | 22:43 | |
| cpallares | flaper87: I need to run but I'll continue this later on tonight (for me) thanks so much for taking the time to explain it to me! | 23:01 |
| flaper87 | cpallares: my pleasure, thanks for contributing | 23:02 |
| flaper87 | don't hesitate to ask | 23:02 |
| *** asalkeld has left #openstack-marconi | 23:07 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Add `simple` example for the client library https://review.openstack.org/57554 | 23:11 |
| *** flaper87 is now known as flaper87|afk | 23:13 | |
| *** fifieldt has joined #openstack-marconi | 23:24 | |
| *** amitgandhi has quit IRC | 23:37 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!