| *** malini_afk is now known as malini | 00:09 | |
| *** nosnos has joined #openstack-marconi | 01:04 | |
| *** oz_akan_ has joined #openstack-marconi | 01:27 | |
| *** kgriffs_afk is now known as kgriffs | 02:04 | |
| *** malini is now known as malini_afk | 02:22 | |
| *** kgriffs is now known as kgriffs_afk | 03:08 | |
| *** vkmc has quit IRC | 03:22 | |
| *** ayoung has quit IRC | 03:48 | |
| *** oz_akan_ has quit IRC | 04:07 | |
| *** oz_akan_ has joined #openstack-marconi | 05:18 | |
| *** oz_akan_ has quit IRC | 05:23 | |
| *** tvb has joined #openstack-marconi | 06:14 | |
| *** tvb has quit IRC | 06:14 | |
| *** tvb has joined #openstack-marconi | 06:14 | |
| *** ykaplan has joined #openstack-marconi | 06:28 | |
| *** flaper87|afk has quit IRC | 07:17 | |
| *** flaper87|afk has joined #openstack-marconi | 07:18 | |
| *** flaper87|afk is now known as flaper87 | 07:18 | |
| *** flaper87 has quit IRC | 07:30 | |
| *** flaper87 has joined #openstack-marconi | 07:30 | |
| *** yassine has joined #openstack-marconi | 08:23 | |
| *** ykaplan has quit IRC | 09:15 | |
| *** ykaplan has joined #openstack-marconi | 09:20 | |
| *** tvb has quit IRC | 09:36 | |
| *** tvb has joined #openstack-marconi | 09:40 | |
| *** ykaplan has quit IRC | 09:50 | |
| *** ykaplan has joined #openstack-marconi | 09:55 | |
| *** tvb has quit IRC | 10:02 | |
| *** tvb has joined #openstack-marconi | 10:33 | |
| *** flaper87 is now known as flaper87|afk | 10:40 | |
| *** tvb has quit IRC | 10:41 | |
| *** tvb has joined #openstack-marconi | 10:45 | |
| *** ykaplan has quit IRC | 10:50 | |
| *** nosnos has quit IRC | 10:56 | |
| *** tvb has quit IRC | 11:19 | |
| *** ykaplan has joined #openstack-marconi | 11:21 | |
| *** tedross has joined #openstack-marconi | 11:30 | |
| *** flaper87|afk is now known as flaper87 | 12:04 | |
| *** tvb has joined #openstack-marconi | 12:39 | |
| *** flaper87 is now known as flaper87|afk | 12:52 | |
| *** oz_akan_ has joined #openstack-marconi | 12:55 | |
| *** oz_akan_ has quit IRC | 12:56 | |
| *** oz_akan_ has joined #openstack-marconi | 12:56 | |
| *** flaper87|afk is now known as flaper87 | 13:06 | |
| *** mpanetta has joined #openstack-marconi | 13:07 | |
| mpanetta | mornin | 13:07 |
|---|---|---|
| flaper87 | mpanetta: morning | 13:08 |
| mpanetta | How are things going flaper87? | 13:14 |
| flaper87 | mpanetta: alll good, struggling with rebase hell now. You? | 13:15 |
| mpanetta | Trying a new test idea | 13:16 |
| mpanetta | The proxy is not behaving quite as well as I had hoped, trying to nail down a good baseline to see at what point things break even. | 13:17 |
| fvollero | mpanetta: great way to keep him quiet :) | 13:20 |
| flaper87 | fvollero: LOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL | 13:21 |
| fvollero | flaper87: <3 <3 you know that bro | 13:22 |
| flaper87 | you better do, you BETTER do! :P | 13:22 |
| *** mpanetta has quit IRC | 13:22 | |
| *** mpanetta has joined #openstack-marconi | 13:23 | |
| fvollero | flaper87: r u going to some conf in those weeks ? | 13:23 |
| mpanetta | haha | 13:23 |
| flaper87 | fvollero: PyConIE next week, MongoTorino Oct 26th and OS Summit | 13:25 |
| fvollero | flaper87: cool OS Summit where it is ? | 13:25 |
| *** malini_afk is now known as malini | 13:28 | |
| flaper87 | fvollero: HKG | 13:29 |
| fvollero | flaper87: Awwww wanna comeee! | 13:29 |
| flaper87 | fvollero: I'll think about you when I'm there. Does that count? | 13:30 |
| flaper87 | :D | 13:30 |
| flaper87 | fvollero: you can insult me now | 13:30 |
| flaper87 | fvollero: not that you couldn't do it before, though | 13:31 |
| *** ykaplan has quit IRC | 13:32 | |
| *** vkmc has joined #openstack-marconi | 13:34 | |
| *** vkmc has joined #openstack-marconi | 13:34 | |
| *** amitgandhi has joined #openstack-marconi | 13:35 | |
| *** ykaplan has joined #openstack-marconi | 13:35 | |
| *** mpanetta has quit IRC | 13:48 | |
| *** mpanetta has joined #openstack-marconi | 13:48 | |
| *** mpanetta_ has joined #openstack-marconi | 13:56 | |
| *** mpanetta has quit IRC | 13:56 | |
| fvollero | flaper87: ok... say good bye to the Cocosette :P | 13:56 |
| flaper87 | :'( now you broke my heart! | 13:57 |
| flaper87 | not cool man, not cool | 13:57 |
| *** alcabrera has joined #openstack-marconi | 13:59 | |
| flaper87 | al-maisan: YOUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUU | 13:59 |
| flaper87 | alcabrera: ^ | 13:59 |
| flaper87 | that was meant for you | 13:59 |
| flaper87 | alcabrera: goood morning!! | 13:59 |
| alcabrera | flaper87: Good morning! :D | 13:59 |
| flaper87 | alcabrera: Mr. Lets break flaper87's config refactor | 13:59 |
| flaper87 | :D | 13:59 |
| *** jcru has joined #openstack-marconi | 13:59 | |
| alcabrera | ;) | 13:59 |
| alcabrera | I'll break all the things - and I often do. :D | 14:00 |
| flaper87 | alcabrera: just kidding, I had to go through rebase hell but I'm out now! | 14:00 |
| flaper87 | 30mins clean | 14:00 |
| alcabrera | woot | 14:00 |
| alcabrera | I had to rebase so much while working on those 4 proxy patches. :P | 14:01 |
| alcabrera | I had it down to a science. | 14:01 |
| alcabrera | 15 minutes to rebase 4. | 14:01 |
| alcabrera | I wonder what's in the review queue this morning...? | 14:02 |
| flaper87 | I meant that I finished rebasing that stuff 30mins ago, I'm clean now :D | 14:02 |
| flaper87 | alcabrera: all your patches got merged | 14:02 |
| alcabrera | flaper87: oooohhh | 14:03 |
| alcabrera | flaper87: also, hurray! I noticed last night while doing *one last* email check. :P | 14:03 |
| alcabrera | brb | 14:03 |
| fvollero | alcabrera: morning :) | 14:04 |
| *** ayoung has joined #openstack-marconi | 14:05 | |
| * flaper87 just git rm common.config | 14:05 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Use oslo.config directly instead of common.config https://review.openstack.org/49550 | 14:13 |
| flaper87 | alcabrera: https://review.openstack.org/#/c/49550/ | 14:13 |
| flaper87 | :) | 14:13 |
| *** mpanetta has joined #openstack-marconi | 14:17 | |
| *** mpanetta_ has quit IRC | 14:17 | |
| *** mpanetta_ has joined #openstack-marconi | 14:18 | |
| *** mpanetta has quit IRC | 14:18 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Use oslo.config directly instead of common.config https://review.openstack.org/49550 | 14:21 |
| *** tvb has quit IRC | 14:22 | |
| *** jdaggett has joined #openstack-marconi | 14:23 | |
| alcabrera | back | 14:24 |
| alcabrera | fvollero: good morning! | 14:24 |
| alcabrera | flaper87: ooohh, migrating to oslo.config - awesome! | 14:24 |
| alcabrera | flaper87: also, jenkins is pleased with your patch - sweet | 14:25 |
| malini | o///// | 14:26 |
| flaper87 | malini: hellooooooooooooooooooooooooooooooooooooooo | 14:27 |
| * alcabrera notices malini has 5 arms - amazing! | 14:27 | |
| * alcabrera or maybe she's waving so fast that it creates after effects | 14:27 | |
| malini | alcabrera: I usually just bring two..today was an exception ;) | 14:27 |
| alcabrera | :D | 14:29 |
| alcabrera | flaper87: I'm considering adding asynchronous behavior to marconi at the transport layer using eventlet. When I say marconi, I mean both at the proxy level and at the queues level. Do you have any thoughts on this? | 14:32 |
| mpanetta_ | haha | 14:32 |
| *** mpanetta_ is now known as mpanetta | 14:32 | |
| alcabrera | At the moment, I've got a working experimental version where I just introduce async before forwarding requests from the proxy. | 14:33 |
| alcabrera | (needs to benchmark a little) | 14:33 |
| malini | if anybody is getting bored https://bugs.launchpad.net/marconi/+bug/1234481 | 14:35 |
| alcabrera | malini: triaged and confirmed | 14:37 |
| flaper87 | alcabrera: mmh, not sure about that, Is there something wrong about letting gunicorn (for example) take care of that | 14:38 |
| flaper87 | ? | 14:38 |
| malini | thanks alcabrera! | 14:41 |
| flaper87 | alcabrera: I'll wait for that first partch to merge before going forward with the complete removal of global instances | 14:42 |
| alcabrera | flaper87: I need to investigate using alternate workers, too. I've benchmarked on localhost, but that leads to (eventlet workers) great slow downs. I'm curious to see how using alternative workers works across a network. | 14:43 |
| flaper87 | alcabrera: TBH, I'm a bit hesitant to depend on either eventlet and gevent, I'd definitely prefer leaving that to others | 14:44 |
| flaper87 | which is the main reason why we provide a WSGI app and not a full prod-ready server | 14:44 |
| alcabrera | hmmm | 14:46 |
| zyuan | the functional tests is not broken; there is always some issue with tox and testtools. i got the leasted version test passed | 14:52 |
| zyuan | malini: ^^ | 14:52 |
| malini | zyuan: do you mean to say that list queues with detailed=True works with the master ? | 14:54 |
| zyuan | all tests work with master... | 14:55 |
| malini | zyuan: do you have the proxy running ? | 14:56 |
| zyuan | no, i just run testtools | 14:56 |
| zyuan | you mean you started funtional tests with proxy in front? | 14:57 |
| malini | I am running the tests against an env that has proxy set up | 14:57 |
| zyuan | ok... | 14:57 |
| openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Create v1 package and move messages there https://review.openstack.org/48664 | 15:02 |
| flaper87 | alcabrera: ^ | 15:03 |
| alcabrera | flaper87: +2'd | 15:05 |
| alcabrera | :) | 15:05 |
| alcabrera | I still need to review the config patch. | 15:05 |
| flaper87 | alcabrera: take your time, it should be fairly straightforward | 15:06 |
| *** flaper87 is now known as flaper87|afk | 15:11 | |
| *** flaper87|afk is now known as flaper87 | 15:11 | |
| alcabrera | zyuan, malini: bug report updated: https://bugs.launchpad.net/marconi/+bug/1234481 | 15:12 |
| alcabrera | flaper87: hahaha, I realize now I should've been using "Partially-implements" all this time. | 15:12 |
| alcabrera | flaper87: Every single one of my patches *fully implemented* the placement service. That's kind of silly. :P | 15:13 |
| flaper87 | alcabrera: LOOL :D | 15:19 |
| flaper87 | well, in your defense, that's pretty new stuff | 15:19 |
| alcabrera | flaper87: I'm loving the config patch, and I'll have a lot of questions by the end of it. | 15:20 |
| *** kgriffs_afk is now known as kgriffs | 15:20 | |
| flaper87 | alcabrera: sounds good, lemme know when you finish reviewing it! | 15:20 |
| alcabrera | flaper87: -1'd - full of questions, full of promise. :) | 15:25 |
| kgriffs | flaper87: you like hot sauce? | 15:27 |
| kgriffs | </random-question> | 15:27 |
| flaper87 | kgriffs: I do | 15:27 |
| flaper87 | A LOT! | 15:27 |
| kgriffs | excellent | 15:27 |
| alcabrera | oohh, hot sauce... | 15:27 |
| alcabrera | I like the stuff. Jalapeño is my favorite. :) | 15:28 |
| * flaper87 used to eat Jalapeños at his "Merienda" when he was in Venezuela | 15:29 | |
| flaper87 | I seriously think my tongue is completely bruned | 15:29 |
| alcabrera | flaper87: lol, that's a bit much for me, but *that* jalapeño flavor in moderation is amazing. | 15:30 |
| kgriffs | Heh. I'm growing Moruga Scorpion peppers at the moment in my back yard. If all goes well, I'm going to have some pretty awesome hot sauce. I'll have to send you a batch. | 15:30 |
| flaper87 | kgriffs: wooooow, sounds goood!!! Looking forward to have some of that homemade hot sauce | 15:31 |
| alcabrera | kgriffs: spicy! | 15:31 |
| *** tvb has joined #openstack-marconi | 15:32 | |
| *** tvb has quit IRC | 15:35 | |
| *** tvb has joined #openstack-marconi | 15:35 | |
| *** tvb has quit IRC | 15:35 | |
| *** tvb has joined #openstack-marconi | 15:35 | |
| alcabrera | kgriffs: A little microbenchmark for you: http://paste.openstack.org/show/47915/ | 15:41 |
| alcabrera | The results are especially interesting on pypy-2.1 | 15:41 |
| alcabrera | kgriffs: perf bug report filed for string concatenation on hot code paths: https://bugs.launchpad.net/marconi/+bug/1234789 | 15:47 |
| alcabrera | flaper87: allow `update` to update multiple fields: https://bugs.launchpad.net/marconi/+bug/1234790 | 15:49 |
| * alcabrera is capturing bugs from proxy patch reviews and noting them for later | 15:49 | |
| flaper87 | alcabrera: awesome, thanks for that! I've a pretty bad memory :P | 15:50 |
| kgriffs | this review should only take a moment: https://review.openstack.org/#/c/49451/ | 15:50 |
| flaper87 | that also helps splitting work | 15:50 |
| alcabrera | there - I captured all the bugs from the merged proxy patches. | 15:51 |
| kgriffs | alcabrera: ah, cool. pypy rocks | 15:51 |
| alcabrera | There's the matter of a docstring missing a type and a typo (middlewarez), but those are minor enough that they can be handled eventually. :) | 15:52 |
| kgriffs | anyone who thinks otherwise is wrong. | 15:52 |
| kgriffs | :D | 15:52 |
| alcabrera | kgriffs: lol, yeah, pypy is pretty sweet. | 15:52 |
| alcabrera | Someday, given enough time, I'd love to contribute to pypy. I really need to get deep into that to understand how it runs. | 15:52 |
| kgriffs | flaper87: fixed that last Py3 bug: https://review.openstack.org/#/c/48776/ | 15:53 |
| kgriffs | oops | 15:53 |
| kgriffs | i meant | 15:53 |
| kgriffs | alcabrera: fixed that last Py3 bug: | 15:53 |
| *** yassine has quit IRC | 15:53 | |
| kgriffs | #copy-paste-fail | 15:53 |
| flaper87 | kgriffs: small question in that review | 15:54 |
| kgriffs | sure | 15:54 |
| flaper87 | not big deal but it may be worth considering | 15:54 |
| flaper87 | feel free to ignore that or to postpone it | 15:54 |
| kgriffs | hmm | 15:55 |
| alcabrera | lessee... | 15:57 |
| alcabrera | kgriffs: +2 approved! | 15:58 |
| alcabrera | for the non-ascii queues deal | 15:58 |
| kgriffs | sweet | 15:59 |
| kgriffs | flaper87: were you thinking to just to use the skip or skipIf decorator? | 16:00 |
| alcabrera | kgriffs: +2 on the other patch, as well, with flaper87's suggestion noted. | 16:00 |
| kgriffs | or perhaps the requires_mongodb could apply the decorator itself | 16:01 |
| flaper87 | kgriffs: yeah, either that or make the decorator you created smart enough to skip the whole suite | 16:01 |
| kgriffs | let me play around with that idea | 16:01 |
| flaper87 | kgriffs: exactly | 16:01 |
| alcabrera | `(marconi)$ git pull upstream master` => 87 files changed, 1238 insertions(+), 472 deletions(-) | 16:02 |
| alcabrera | Such a busy week. | 16:02 |
| *** oz_akan_ has quit IRC | 16:04 | |
| alcabrera | kgriffs: https://review.openstack.org/#/c/48776/ (has merge conflicts, jenkins is displeased) | 16:07 |
| *** jdaggett has quit IRC | 16:10 | |
| *** jdaggett has joined #openstack-marconi | 16:10 | |
| *** jdaggett has joined #openstack-marconi | 16:12 | |
| kgriffs | alcabrera: allow me to cheer him up | 16:13 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: add logging to proxy https://review.openstack.org/49585 | 16:14 |
| alcabrera | kgriffs, flaper87: I'm adding logging to -proxy. Whenever you can, the patch above could use some love. :D | 16:14 |
| kgriffs | kk | 16:15 |
| *** flaper87 is now known as flaper87|afk | 16:16 | |
| *** flaper87|afk is now known as flaper87 | 16:17 | |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: add logging to proxy https://review.openstack.org/49585 | 16:19 |
| alcabrera | ^^ fixed pep8 error | 16:21 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(wsgi): Non-ASCII characters in the queue name trigger 500 status https://review.openstack.org/48776 | 16:24 |
| kgriffs | If that doesn't make Jenkins happy, I've got $50 somewhere that should do the trick. | 16:25 |
| alcabrera | kgriffs: hahaha | 16:26 |
| alcabrera | kgriffs: (https://review.openstack.org/#/c/48776/) -1 - could you move the before hook to marconi.common.transport.wsgi.helpers? | 16:48 |
| alcabrera | I'd forgotten that I started this refactoring with the proxy patches. | 16:48 |
| alcabrera | (jenkins is happy - now I'm complaining!) :D | 16:48 |
| *** alcabrera is now known as alcabrera|afk | 16:51 | |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: chore: Rename MONGODB_TEST_LIVE to MARCONI_TEST_MONGODB https://review.openstack.org/49451 | 16:54 |
| *** alcabrera|afk is now known as alcabrera | 16:57 | |
| alcabrera | kgriffs: -1: name change request ^^. (almost there!) | 16:58 |
| kgriffs | ah, good catch | 17:00 |
| kgriffs | name cruft | 17:00 |
| kgriffs | :p | 17:00 |
| alcabrera | yup! The little things. :D | 17:00 |
| kgriffs | stevedore is cool, but I really hate how it obfuscates syntax errors and missing imports and stuff. | 17:02 |
| kgriffs | thank goodness for pyflakes | 17:02 |
| flaper87 | kgriffs: agreed, I've struggled with that as well | 17:05 |
| flaper87 | it's really annoying | 17:05 |
| flaper87 | I used to first check entry_points and then errors | 17:06 |
| flaper87 | I do it the other way around | 17:06 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(wsgi): Non-ASCII characters in the queue name trigger 500 status https://review.openstack.org/48776 | 17:06 |
| flaper87 | which just means I am way too dumb and the probability to have an error in my code instead of the entry_points definition is higher | 17:07 |
| alcabrera | stevedore is awesome for creating an extensible, evolvable architecture. I agree entirely, though - parsing those error/exception logs is an art. | 17:09 |
| alcabrera | It's like parsing through c++ template error logs, but slightly easier. | 17:09 |
| alcabrera | kgriffs: +2 | 17:11 |
| kgriffs | flaper87: last patchset just brought things in line with the refactoring the proxy patch did on the global hooks | 17:12 |
| kgriffs | https://review.openstack.org/#/c/48776/ | 17:12 |
| flaper87 | kgriffs: already approved | 17:12 |
| flaper87 | :) | 17:12 |
| kgriffs | wow | 17:12 |
| kgriffs | you were so fast, i didn't even see you move! | 17:12 |
| * flaper87 is trying to beat pypy | 17:12 | |
| alcabrera | "Those reviews were fast as lightning..." | 17:12 |
| mpanetta | He is one with the matrix | 17:12 |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: chore: Rename MONGODB_TEST_LIVE to MARCONI_TEST_MONGODB https://review.openstack.org/49451 | 17:13 |
| kgriffs | alcabrera: I've got $50 with your name on it. Just sayin'. | 17:14 |
| kgriffs | ;) | 17:14 |
| openstackgerrit | A change was merged to openstack/marconi: fix(wsgi): Non-ASCII characters in the queue name trigger 500 status https://review.openstack.org/48776 | 17:14 |
| alcabrera | kgriffs: +2 ;) | 17:15 |
| * kgriffs slides bills under table | 17:15 | |
| Alex_Gaynor | alcabrera: you should complain to dhellman if there are bad error messages, I'm sure he'd like to improve them if possible | 17:15 |
| alcabrera | Alex_Gaynor: Good suggestion! I'll keep a paste around in the future. | 17:16 |
| alcabrera | They get pretty mighty. | 17:16 |
| kgriffs | malini: ping | 17:18 |
| *** flaper87 is now known as flaper87|afk | 17:21 | |
| kgriffs | alcabrera: https://review.openstack.org/#/c/48012/4/tests/functional/wsgi/v1/test_queues.py | 17:22 |
| kgriffs | do you know if malini meant to say she would submit the DRY stuff in another *patch*, not *patchset* | 17:22 |
| alcabrera | kgriffs: I think she meant patchset. It's already present in patch set 5 (DRYness) | 17:23 |
| alcabrera | 'assertMessageCount' captures part of the duplicated logic. | 17:24 |
| *** ykaplan has quit IRC | 17:26 | |
| kgriffs | I think she forgot to use it | 17:26 |
| kgriffs | nevermind | 17:27 |
| kgriffs | assertQueueStats is in there | 17:27 |
| alcabrera | kgriffs, malini: There's a bug in my on_put logic for proxy queues. It will duplicate project/queue on each registered partition because it doesn't reference the catalogue before making a decision. | 17:39 |
| * alcabrera discovered while tinkering with a personal marconi environment | 17:40 | |
| kgriffs | oops | 17:40 |
| kgriffs | you owe me $50 | 17:40 |
| alcabrera | I'll write a bug report and fix it up very soon. | 17:40 |
| alcabrera | kgriffs: dang... DL | 17:40 |
| alcabrera | D: | 17:40 |
| kgriffs | LOL | 17:40 |
| jdaggett | Hi, who can tell me what the service catalog entry will be for Marconi? | 17:40 |
| jdaggett | "messaging"? | 17:40 |
| zyuan | alcabrera: ...the "bug" i found is not the cause; _catalogue.list() is storage API, which gives you iterator... | 17:42 |
| malini | kgriffs: I;ll update https://review.openstack.org/#/c/48012/5/tests/functional/wsgi/v1/test_queues.py to use ddt. IS there anything else you wanted ? | 17:43 |
| alcabrera | zyuan: ah, okay. | 17:43 |
| openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: feat(proxy): listing opts returns lists https://review.openstack.org/49595 | 17:44 |
| alcabrera | jdaggett: I'm not sure that's been decided yet. kgriffs, any thoughts? | 17:44 |
| kgriffs | jdaggett: yeah, TBH we haven't discussed it | 17:46 |
| jdaggett | kgriffs: OK, it can wait, thanks! | 17:46 |
| kgriffs | I'll bring it up with the gang with flaper87's around | 17:47 |
| alcabrera | bug filed, initiating fix: https://bugs.launchpad.net/marconi/+bug/1234849 | 17:48 |
| alcabrera | ^^ duplicate queues | 17:48 |
| *** jergerber has joined #openstack-marconi | 17:49 | |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/marconi: Add Tests for Queue Stats https://review.openstack.org/48012 | 18:14 |
| malini | kgriffs: The above has the ddt fix..Thanks for the feeback, wonder why I didn't do it the first time around :) | 18:15 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: fix: do not duplicate queues across partitions https://review.openstack.org/49599 | 18:22 |
| alcabrera | kgriffs, flaper87|afk, malini: bug fix - duplicate queues ^^ | 18:22 |
| alcabrera | malini: +2 on your queue stats. Awesome! | 18:24 |
| zyuan | alcabrera: saw your email. thanks! | 18:24 |
| alcabrera | zyuan: np. :) | 18:24 |
| zyuan | i'm playing with your env with requests... | 18:36 |
| alcabrera | zyuan: sweet | 18:45 |
| alcabrera | zyuan: the /clear command is now implemented | 18:45 |
| alcabrera | It's available via port :8001 on each env | 18:45 |
| zyuan | admin command? | 18:46 |
| alcabrera | nope - separate wsgi app, made special for the test env. :P | 18:46 |
| openstackgerrit | A change was merged to openstack/marconi: Add Tests for Queue Stats https://review.openstack.org/48012 | 18:47 |
| alcabrera | malini: congrats! Thanks for the test patch. :) | 18:48 |
| malini | alcabrera: :D | 18:48 |
| kgriffs | alcabrera: self._catalogue.exists | 18:58 |
| kgriffs | that utilizes the cache, right? | 18:58 |
| alcabrera | kgriffs: no, not yet. :/ | 18:58 |
| alcabrera | That pings primary storage. | 18:59 |
| alcabrera | (primary proxy storage) | 18:59 |
| kgriffs | baaaaaad llama! | 18:59 |
| alcabrera | lol | 18:59 |
| alcabrera | That's an awesome caching point, though, Good catch. | 18:59 |
| malini | alcabrera: what is the external endpoint I can use to hit your env ? | 19:16 |
| alcabrera | malini: I set it all in an email. Did you get it? :x | 19:18 |
| alcabrera | *sent | 19:20 |
| malini | alcabrera: Sorry I missed it | 19:20 |
| malini | checking | 19:20 |
| alcabrera | malini: no worries (lots of email!) | 19:22 |
| *** ykaplan has joined #openstack-marconi | 19:44 | |
| *** malini has left #openstack-marconi | 19:49 | |
| *** vkmc has quit IRC | 20:09 | |
| *** tvb has quit IRC | 20:41 | |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: fix: do not duplicate queues across partitions https://review.openstack.org/49599 | 20:46 |
| alcabrera | kgriffs: This patch above is *awesome*. Caching added, as requested, as well as the first set of unit tests to address the proxy queues resources. | 20:46 |
| kgriffs | excellent | 20:47 |
| alcabrera | kgriffs: oh, I also addressed some of the '{0}/{1}'.format stuff in the hot path. | 20:47 |
| alcabrera | I now have helper functions that perform 'p.' + project + '/' + queue | 20:48 |
| alcabrera | kgriffs: -1'd my own patch for lack of cache validation, heh. :P | 20:50 |
| alcabrera | brb | 20:50 |
| openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: fix: do not duplicate queues across partitions https://review.openstack.org/49599 | 20:58 |
| alcabrera | kgriffs: ready for review now | 20:58 |
| alcabrera | flaper87|afk: ^^ | 20:59 |
| alcabrera | zyuan: check out the unit tests in the patch above - they handle the /v1/queues/* endpoints for the proxy | 20:59 |
| alcabrera | I'm out for the night, guys. | 20:59 |
| alcabrera | Take care, fix bugs, have a great night! :D | 20:59 |
| *** alcabrera has left #openstack-marconi | 21:01 | |
| *** jdaggett has quit IRC | 21:13 | |
| *** ykaplan has quit IRC | 21:33 | |
| *** jdaggett has joined #openstack-marconi | 21:37 | |
| *** jdaggett has quit IRC | 21:39 | |
| *** flaper87|afk is now known as flaper87 | 21:51 | |
| *** jcru has quit IRC | 21:53 | |
| * flaper87 is back | 21:56 | |
| flaper87 | :D | 21:56 |
| *** tedross has quit IRC | 22:02 | |
| *** vkmc has joined #openstack-marconi | 22:05 | |
| *** jdaggett has joined #openstack-marconi | 22:09 | |
| *** jdaggett has quit IRC | 22:18 | |
| *** amitgandhi has quit IRC | 22:31 | |
| *** jdaggett has joined #openstack-marconi | 22:33 | |
| *** mpanetta has quit IRC | 22:39 | |
| *** flaper87 is now known as flaper87|afk | 22:39 | |
| *** kgriffs is now known as kgriffs_afk | 23:06 | |
| *** vkmc has quit IRC | 23:06 | |
| *** jdaggett has quit IRC | 23:15 | |
| *** jdaggett has joined #openstack-marconi | 23:18 | |
| *** jdaggett has quit IRC | 23:44 | |
| *** ayoung has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!