| *** achanda has joined #openstack-zaqar | 00:28 | |
| *** achanda has quit IRC | 00:51 | |
| *** achanda has joined #openstack-zaqar | 00:52 | |
| *** achanda has quit IRC | 00:52 | |
| *** amitgandhinz has joined #openstack-zaqar | 01:30 | |
| *** amitgandhinz has quit IRC | 01:30 | |
| *** amitgandhinz has joined #openstack-zaqar | 01:31 | |
| *** achanda has joined #openstack-zaqar | 01:44 | |
| *** boris-42 has quit IRC | 01:50 | |
| *** csoukup has joined #openstack-zaqar | 02:24 | |
| *** Qiming_ has quit IRC | 02:53 | |
| *** Qiming_ has joined #openstack-zaqar | 02:54 | |
| *** Qiming_ has quit IRC | 02:56 | |
| *** Qiming_ has joined #openstack-zaqar | 02:57 | |
| openstackgerrit | Merged openstack/zaqar: Updated from global requirements https://review.openstack.org/219506 | 03:02 |
|---|---|---|
| openstackgerrit | Merged openstack/zaqar: Support various paths for signed URLs https://review.openstack.org/215638 | 03:05 |
| openstackgerrit | Merged openstack/zaqar: Restore sqlalchemy unit tests https://review.openstack.org/218777 | 03:05 |
| *** amitgandhinz has quit IRC | 03:06 | |
| openstackgerrit | Merged openstack/zaqar: Add unittests for Flavors in sqlalchemy https://review.openstack.org/218784 | 03:08 |
| *** csoukup has quit IRC | 03:16 | |
| *** achanda has quit IRC | 03:28 | |
| *** dguerri` has quit IRC | 03:30 | |
| *** AAzza has quit IRC | 03:31 | |
| *** AAzza has joined #openstack-zaqar | 03:32 | |
| *** dguerri` has joined #openstack-zaqar | 03:32 | |
| *** dguerri` is now known as dguerri | 03:32 | |
| *** dguerri has joined #openstack-zaqar | 03:32 | |
| *** Qiming_ has quit IRC | 03:51 | |
| *** AAzza has quit IRC | 03:55 | |
| *** AAzza has joined #openstack-zaqar | 03:57 | |
| *** boris-42 has joined #openstack-zaqar | 03:59 | |
| *** Qiming_ has joined #openstack-zaqar | 05:12 | |
| *** achanda has joined #openstack-zaqar | 05:34 | |
| *** achanda has quit IRC | 05:40 | |
| *** achanda has joined #openstack-zaqar | 06:00 | |
| *** achanda has quit IRC | 06:19 | |
| flaper87 | therve: hey, when you're around. Could you expand on how you forsee this being used? https://review.openstack.org/#/c/215638/ | 06:25 |
| openstackgerrit | Flavio Percoco proposed openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 06:37 |
| *** achanda has joined #openstack-zaqar | 06:48 | |
| *** stanchan has joined #openstack-zaqar | 07:01 | |
| therve | flaper87, 'morning! | 07:23 |
| therve | flaper87, The idea is to pass an extra paths parameter to the signed url creation, so that users of that signature can used different URLs | 07:25 |
| therve | flaper87, Ie, subscribe to the queue they got access to | 07:25 |
| flaper87 | ah, nvm, I see where the previous implementation was limited. I'll read your patch a bit better | 07:27 |
| flaper87 | Does it require getting a signed url for all sub-paths of /queues/my-queue/ ? | 07:27 |
| flaper87 | or just getting /queues/my-queue/ will also allow for subscribing and other ops ? | 07:27 |
| therve | The latter | 07:31 |
| therve | Well, the way I did it is yu create the signature for my-queue with several paths underneath it | 07:32 |
| flaper87 | yeah, gotcha | 07:34 |
| flaper87 | thanks :) | 07:34 |
| therve | flaper87, I know why the test is failing in https://review.openstack.org/#/c/202374/ FWIW | 07:51 |
| therve | Can submit a fix if you want | 07:52 |
| flaper87 | therve: I thought your executor patch would have fixed it | 07:52 |
| flaper87 | therve: please, do! | 07:52 |
| therve | flaper87, Still need to shutdown the executor in the test | 07:52 |
| flaper87 | I don't think flwang1 minds if you fix the issue | 07:52 |
| flaper87 | therve: ah, there you go | 07:53 |
| openstackgerrit | Thomas Herve proposed openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 07:55 |
| therve | ^^^ | 07:56 |
| flaper87 | therve: danke | 07:57 |
| *** achanda has quit IRC | 08:36 | |
| openstackgerrit | Thomas Herve proposed openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 08:46 |
| therve | Okkkk. Maybe this time. | 08:46 |
| flaper87 | :P | 08:47 |
| *** diga has joined #openstack-zaqar | 08:52 | |
| therve | flaper87, Looks happier | 09:22 |
| flaper87 | therve: yeah, approving now | 09:22 |
| openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix duplicated notification https://review.openstack.org/218072 | 09:27 |
| *** flwang has joined #openstack-zaqar | 09:31 | |
| therve | flwang1 flwang , The linked bug on https://review.openstack.org/#/c/218072 is incorrect | 09:31 |
| flwang | therve: let me check :) | 09:31 |
| therve | Trying to understand the issue :) | 09:32 |
| flwang | therve: oh, the issue is we're leveraging the pipeline to trigger the notifier | 09:33 |
| flwang | but if the pooling is enabled, then the notifier will be triggered 2 times | 09:33 |
| therve | flwang, But the notifier is in the message_pipeline, right? Not the subscription_pipeline | 09:34 |
| flwang | one is at the pooling's subscription controller level, another one is at the storage backend level | 09:34 |
| therve | I guess I don't know about those well enough to understand the subtlety | 09:35 |
| flwang | yep | 09:36 |
| flwang | you can add a breakpoint in notifier.py | 09:36 |
| flwang | then you check the type of subscription_controller | 09:36 |
| flwang | therve: btw, i'd like to take this chance to say thank you for your effort on my patches and for zaqar | 09:36 |
| therve | flwang, You're welcome! | 09:36 |
| flwang | really awesome | 09:36 |
| flaper87 | flwang: <3 it's my pleasure. I hope to see more of those patches coming! | 09:37 |
| flaper87 | flwang: your work was superb and you bared with us | 09:38 |
| flaper87 | :P | 09:38 |
| flaper87 | which is not a simple thing to do | 09:38 |
| flwang | flaper87: i have been working under such a boss more than 1 years, so i have been used to keep working hard :) | 09:39 |
| flwang | and I know there no another way but make the code more solid :) | 09:40 |
| * flaper87 bows and says some words in Chinese | 09:41 | |
| flaper87 | to bad I've no idea what I said | 09:41 |
| flwang | flaper87: you need google translate :) | 09:42 |
| therve | Arg, ordering issue | 09:43 |
| openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix duplicated notification https://review.openstack.org/218072 | 09:44 |
| flwang | therve: just updated the bug number | 09:44 |
| openstackgerrit | Thomas Herve proposed openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 09:46 |
| therve | flwang, Thanks | 09:46 |
| therve | flaper87, ^^^ | 09:46 |
| flwang | therve: thanks for the patch set | 09:48 |
| flwang | the mock for subprocess.popen is really weird | 09:49 |
| therve | Yeah that one was a pain | 09:49 |
| flwang | that means i can use the same way like webhook | 09:49 |
| flwang | so i have to that like current way, really painful | 09:49 |
| openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix duplicated notification https://review.openstack.org/218072 | 11:05 |
| therve | flaper87, flwang: I tried the following approach instead for removing dups: http://paste.openstack.org/show/444140/ | 11:31 |
| flwang | clicking... | 11:34 |
| flwang | therve: my network is very slow... | 11:40 |
| flwang | therve: sorry, i can't open paste.openstack.org now :( | 11:53 |
| vkmc | morning | 11:54 |
| flaper87 | morning | 11:57 |
| openstackgerrit | Merged openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 12:15 |
| flaper87 | w0000h00000 | 12:15 |
| flaper87 | FINALLY | 12:15 |
| flaper87 | yeah | 12:15 |
| * flaper87 jumps | 12:15 | |
| flwang | flaper87: yep, thank you, fla | 12:18 |
| therve | Hi vkmc! | 12:33 |
| vkmc | hi therve! | 12:37 |
| *** malini has joined #openstack-zaqar | 12:42 | |
| *** malini has quit IRC | 12:46 | |
| *** malini has joined #openstack-zaqar | 12:46 | |
| *** diga has quit IRC | 12:48 | |
| flwang | therve: i saw your change and i think it works, but the reason I don't want to do that is i want to limit the impact at this stage | 12:52 |
| *** ryansb has quit IRC | 12:52 | |
| flwang | therve: you know, release is coming | 12:52 |
| * therve shrugs | 12:53 | |
| flwang | so how about we fix it in a managed scope (in notifier.py) and revisit it in M? | 12:53 |
| therve | Yeah I don't mind | 12:53 |
| therve | I'm not against a refactoring of stages in M | 12:54 |
| flwang | therve: TBH, i'm not sure if there is any other stuff depends current mechanism | 12:54 |
| therve | It looks like a nuclear bomb to crush an insect | 12:54 |
| flwang | therve: thanks for the understanding | 12:54 |
| *** ryansb has joined #openstack-zaqar | 12:54 | |
| *** ryansb has quit IRC | 12:54 | |
| *** ryansb has joined #openstack-zaqar | 12:54 | |
| flwang | therve: ok, another option is merged current patch and i will continually follow it in Liberty to figure out it | 12:55 |
| flwang | therve: i mean try to adopt your fix and make sure no regression | 12:56 |
| therve | flwang, There is no hurry | 12:56 |
| flwang | therve: cool | 12:56 |
| flwang | thanks | 12:57 |
| flwang | have fun, guys, i'm going to sleep | 12:57 |
| * flaper87 is almost done with the policy patch | 13:02 | |
| flaper87 | flwang: sleep well | 13:02 |
| *** sriram has joined #openstack-zaqar | 13:07 | |
| *** amitgandhinz has joined #openstack-zaqar | 13:24 | |
| *** theonewolf has joined #openstack-zaqar | 13:45 | |
| theonewolf | hey is Zaqar going to be in the next OpenStack release? | 13:46 |
| flaper87 | theonewolf: it's been in the OpenStack release since Icehouse :D | 13:46 |
| flaper87 | theonewolf: short answer, Yes! | 13:46 |
| flaper87 | also, welcome to openstack-zaqar | 13:46 |
| flaper87 | I hope you'll stay long and have fun with us | 13:46 |
| theonewolf | flaper87: oh I see, I don't read about it like nova, cinder, keystone, etc. | 13:46 |
| flaper87 | theonewolf: yeah, we're getting there. Small community, small steps but concrete path | 13:47 |
| flaper87 | theonewolf: The community is getting stronger and growing, though! | 13:47 |
| flaper87 | we added a new core in Liberty and a bunch of cool features | 13:47 |
| flaper87 | theonewolf: may ask why you're interested? I mean, are you planning to install it and offer a service? or are you looking at integrate with it? | 13:48 |
| flaper87 | or just pure curiosity | 13:48 |
| theonewolf | flaper87: i need distributed messaging for applications running inside many VMs | 13:48 |
| flaper87 | theonewolf: bam, another happy customer! | 13:49 |
| flaper87 | :D | 13:49 |
| theonewolf | flaper87: looking at lots of Python queues, ZeroMQ, other things | 13:49 |
| theonewolf | flaper87: wondering what's the best, maybe something integrated into OpenStack is | 13:49 |
| flaper87 | theonewolf: interesting, do you know what scales/loads you'll have? | 13:49 |
| theonewolf | flaper87: does zaqar appear inside Horizon on the dashboard? trying to find the quick/easy guide | 13:49 |
| theonewolf | flaper87: as far as I know currently, _fairly_ lightly loaded | 13:50 |
| theonewolf | flaper87: although we also assume Consul is deployed in our setup, so we might use that as a poor-man's message service (tweaking KV pairs with long-polling) | 13:51 |
| flaper87 | theonewolf: A dashboard on horizon is under discussion. For now, we've been focusing on CLI and the service. | 13:53 |
| theonewolf | flaper87: that makes sense | 13:53 |
| flaper87 | theonewolf: I'm curious to know what you'd like to see on that dashboard | 13:53 |
| theonewolf | flaper87: I've also deployed and managed several private OpenStack clusters, and am wondering if we should custom orchestrate our software | 13:54 |
| flaper87 | we've had those discussions and just showing messages kinda doesn't make sense. We've been talking more about monitoring, stats and health | 13:54 |
| theonewolf | flaper87: what would I want? the ability to see something about my messaging graph and topology | 13:54 |
| theonewolf | flaper87: are messages flowing, where are they failing, etc. | 13:55 |
| *** ametts has joined #openstack-zaqar | 13:55 | |
| flaper87 | theonewolf: exactly, I fully agree with that | 13:55 |
| flaper87 | that's what our latest discussions has flown around | 13:55 |
| flaper87 | have* | 13:55 |
| flaper87 | theonewolf: thanks for the feedback | 13:56 |
| theonewolf | flaper87: no problem | 13:56 |
| theonewolf | flaper87: I'm trying to figure out if OpenStack is needed for my deployable cloudlet boxes designed to live on-premise | 13:58 |
| *** Qiming_ has quit IRC | 14:43 | |
| openstackgerrit | Thomas Herve proposed openstack/zaqar: Don't register the default pool https://review.openstack.org/220171 | 14:43 |
| therve | That's a fun one | 14:44 |
| flaper87 | therve: anyway to test it? :D | 14:45 |
| therve | flaper87, Manually, yeah. Didn't manage to do it in unittest | 14:45 |
| therve | flaper87, You started it: https://github.com/openstack/zaqar/commit/bfcc60c170369b3a3e3ab116e477d7a6e706e1be :p | 14:46 |
| flaper87 | therve: I know, that's why I'm asking | 14:47 |
| flaper87 | hahahaha | 14:47 |
| therve | Ah | 14:47 |
| flaper87 | I saw the patch and then: | 14:47 |
| * flaper87 whistles | 14:47 | |
| *** dynarro has joined #openstack-zaqar | 14:47 | |
| *** kgriffs|afk is now known as kgriffs | 14:51 | |
| *** mpanetta has joined #openstack-zaqar | 14:59 | |
| *** mpanetta has quit IRC | 15:00 | |
| *** mpanetta has joined #openstack-zaqar | 15:00 | |
| *** mpanetta has quit IRC | 15:02 | |
| *** mpanetta has joined #openstack-zaqar | 15:03 | |
| *** akanksha has joined #openstack-zaqar | 15:04 | |
| *** csoukup has joined #openstack-zaqar | 15:07 | |
| *** amitgandhinz has quit IRC | 15:10 | |
| openstackgerrit | Doraly Navarro proposed openstack/python-zaqarclient: Added support to v2 https://review.openstack.org/209238 | 15:16 |
| *** amitgandhinz has joined #openstack-zaqar | 15:24 | |
| *** dynarro has quit IRC | 15:33 | |
| *** flwang has quit IRC | 15:41 | |
| theonewolf | ~\quit | 16:02 |
| theonewolf | \quit | 16:02 |
| *** theonewolf has quit IRC | 16:02 | |
| *** sriram has quit IRC | 16:07 | |
| openstackgerrit | Flavio Percoco proposed openstack/zaqar: Policy support https://review.openstack.org/209910 | 16:20 |
| flaper87 | that's hopefully green | 16:23 |
| flaper87 | that'll hopefully be green | 16:23 |
| flaper87 | that might be green | 16:23 |
| flaper87 | that may be green | 16:23 |
| flaper87 | that'll never be green, right? | 16:24 |
| *** stanchan has quit IRC | 16:25 | |
| ryansb | the power of positive gate-thinking | 16:27 |
| *** malini has quit IRC | 16:28 | |
| *** malini has joined #openstack-zaqar | 16:31 | |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Adds oslo_cache dependency https://review.openstack.org/219374 | 16:46 |
| *** malini has quit IRC | 16:49 | |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Adds oslo_cache dependency https://review.openstack.org/219374 | 17:03 |
| *** achanda has joined #openstack-zaqar | 17:14 | |
| *** achanda has quit IRC | 17:21 | |
| *** achanda has joined #openstack-zaqar | 17:22 | |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Adds oslo_cache dependency https://review.openstack.org/219374 | 17:26 |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Remove unused zaqar.openstack.common modules https://review.openstack.org/220250 | 17:26 |
| *** stanchan has joined #openstack-zaqar | 17:32 | |
| *** malini has joined #openstack-zaqar | 17:41 | |
| *** malini has quit IRC | 17:42 | |
| *** malini has joined #openstack-zaqar | 17:42 | |
| *** bradjones has joined #openstack-zaqar | 17:53 | |
| *** bradjones has joined #openstack-zaqar | 17:53 | |
| *** stanchan has quit IRC | 19:15 | |
| *** achanda has quit IRC | 19:18 | |
| *** dguerri is now known as dguerri` | 19:29 | |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Remove unused zaqar.openstack.common modules https://review.openstack.org/220250 | 19:53 |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Adds oslo_cache dependency https://review.openstack.org/219374 | 19:53 |
| *** akanksha has quit IRC | 20:08 | |
| *** malini has quit IRC | 20:15 | |
| *** dguerri` is now known as dguerri | 20:18 | |
| *** achanda has joined #openstack-zaqar | 20:18 | |
| *** dguerri is now known as dguerri` | 20:20 | |
| *** achanda has quit IRC | 20:24 | |
| *** malini has joined #openstack-zaqar | 20:26 | |
| *** achanda has joined #openstack-zaqar | 20:29 | |
| *** malini has quit IRC | 20:37 | |
| *** achanda has quit IRC | 20:38 | |
| openstackgerrit | Claudiu Belu proposed openstack/zaqar: Adds oslo_cache dependency https://review.openstack.org/219374 | 20:39 |
| *** malini has joined #openstack-zaqar | 20:40 | |
| *** achanda has joined #openstack-zaqar | 20:47 | |
| *** malini has quit IRC | 20:47 | |
| *** malini has joined #openstack-zaqar | 20:47 | |
| *** ametts has quit IRC | 21:06 | |
| *** amitgandhinz has quit IRC | 21:46 | |
| *** csoukup has quit IRC | 23:17 | |
| *** bradjones has quit IRC | 23:31 | |
| *** achanda has quit IRC | 23:31 | |
| *** kgriffs is now known as kgriffs|afk | 23:33 | |
| *** Qiming_ has joined #openstack-zaqar | 23:37 | |
| *** mpanetta has quit IRC | 23:44 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!