| *** boris-42 has joined #openstack-zaqar | 00:04 | |
| *** shu-mutou has joined #openstack-zaqar | 00:08 | |
| *** achanda has quit IRC | 00:36 | |
| *** achanda has joined #openstack-zaqar | 01:37 | |
| *** achanda has quit IRC | 01:42 | |
| *** achanda has joined #openstack-zaqar | 01:52 | |
| *** kgriffs|afk is now known as kgriffs | 02:59 | |
| *** kgriffs is now known as kgriffs|afk | 03:10 | |
| openstackgerrit | Chaozhe Chen(ccz) proposed openstack/zaqar: Update requirements https://review.openstack.org/281884 | 03:57 |
|---|---|---|
| *** flwang1 has quit IRC | 03:58 | |
| *** GB21 has joined #openstack-zaqar | 05:10 | |
| *** ksheedlo-rax is now known as ksheedlo | 07:31 | |
| *** jtomasek has joined #openstack-zaqar | 07:46 | |
| *** pcaruana has joined #openstack-zaqar | 08:05 | |
| *** achanda has quit IRC | 08:42 | |
| *** achanda has joined #openstack-zaqar | 08:46 | |
| *** achanda has quit IRC | 08:58 | |
| *** flwang1 has joined #openstack-zaqar | 09:22 | |
| *** dynarro has joined #openstack-zaqar | 09:33 | |
| wanghao | hi guys, happy Lantern Festival! | 10:04 |
| Eva-i | wanghao: hello | 10:05 |
| wanghao | hi~ | 10:05 |
| wanghao | flwang: hi, I have changed this patch:https://review.openstack.org/#/c/273931/, as we discussed it in here, plz have a look if you're free. | 10:08 |
| *** dynarro has quit IRC | 10:19 | |
| *** flwang1 has quit IRC | 11:06 | |
| *** wxy has quit IRC | 11:39 | |
| *** zigo has quit IRC | 11:40 | |
| *** ksheedlo has quit IRC | 11:40 | |
| *** zigo has joined #openstack-zaqar | 11:41 | |
| *** ameade has quit IRC | 11:41 | |
| *** ameade has joined #openstack-zaqar | 11:42 | |
| *** wxy has joined #openstack-zaqar | 11:43 | |
| *** ksheedlo-rax has joined #openstack-zaqar | 11:45 | |
| *** dynarro has joined #openstack-zaqar | 14:08 | |
| *** openstackgerrit has quit IRC | 14:17 | |
| *** openstackgerrit has joined #openstack-zaqar | 14:17 | |
| *** amitgandhinz has joined #openstack-zaqar | 14:33 | |
| *** sriram has joined #openstack-zaqar | 14:58 | |
| *** achanda has joined #openstack-zaqar | 15:34 | |
| *** achanda has quit IRC | 15:45 | |
| *** mpanetta has joined #openstack-zaqar | 15:55 | |
| *** mpanetta has quit IRC | 15:56 | |
| *** mpanetta has joined #openstack-zaqar | 15:57 | |
| *** mpanetta has quit IRC | 15:59 | |
| *** mpanetta has joined #openstack-zaqar | 15:59 | |
| *** mpanetta has quit IRC | 15:59 | |
| *** mpanetta has joined #openstack-zaqar | 16:02 | |
| *** kgriffs|afk is now known as kgriffs | 16:04 | |
| *** mpanetta has quit IRC | 16:21 | |
| vkmc | flwang, won't be able to make it for today's meeting | 16:22 |
| vkmc | just FYI | 16:22 |
| *** pcaruana has quit IRC | 16:27 | |
| ryansb | ok | 16:31 |
| ryansb | noted | 16:31 |
| *** kgriffs is now known as kgriffs|afk | 16:33 | |
| *** kgriffs|afk is now known as kgriffs | 16:41 | |
| *** akanksha_ has joined #openstack-zaqar | 16:41 | |
| Eva-i | ryansb: hello. About your comment here https://review.openstack.org/#/c/282338. What do you think about name "responds_with_500_on_error"? | 16:46 |
| ryansb | that works, or just "can_send_500" | 16:47 |
| ryansb | but yeah, responds_with_500 is great | 16:47 |
| ryansb | thanks :) | 16:48 |
| Eva-i | ryansb: or maybe "sends_500_on_error"? | 16:48 |
| ryansb | "exception_sends_500" | 16:48 |
| Eva-i | ryansb: "on_exception_sends_500"? =) | 16:49 |
| ryansb | works for me | 16:49 |
| Eva-i | ryansb: good | 16:50 |
| openstackgerrit | Eva Balycheva proposed openstack/zaqar: Fix cross-transport error handler https://review.openstack.org/282338 | 16:54 |
| *** kgriffs is now known as kgriffs|afk | 16:54 | |
| openstackgerrit | Eva Balycheva proposed openstack/zaqar: Fix cross-transport error handler https://review.openstack.org/282338 | 16:55 |
| ryansb | thanks Eva-i !! | 16:56 |
| Eva-i | ryansb: yes, no problem | 16:57 |
| *** malini1 has joined #openstack-zaqar | 17:22 | |
| *** kgriffs|afk is now known as kgriffs | 17:33 | |
| openstackgerrit | Eva Balycheva proposed openstack/zaqar: Catch NoPoolsFound on queue list on websocket API https://review.openstack.org/283184 | 17:35 |
| *** mpanetta has joined #openstack-zaqar | 17:42 | |
| openstackgerrit | Eva Balycheva proposed openstack/zaqar: Fix subscriptions_post resource in wsgi 2 homedoc https://review.openstack.org/283189 | 17:47 |
| *** mpanetta has quit IRC | 18:00 | |
| *** mpanetta has joined #openstack-zaqar | 18:03 | |
| *** kgriffs is now known as kgriffs|afk | 18:03 | |
| *** dynarro has quit IRC | 18:08 | |
| *** davideagnello has joined #openstack-zaqar | 18:31 | |
| *** david-lyle_ has quit IRC | 18:34 | |
| *** fribeiro has joined #openstack-zaqar | 18:48 | |
| *** mpanetta has quit IRC | 18:52 | |
| *** mpanetta has joined #openstack-zaqar | 19:05 | |
| *** mpanetta has quit IRC | 19:09 | |
| *** mpanetta has joined #openstack-zaqar | 19:53 | |
| *** mpanetta has quit IRC | 20:01 | |
| *** mpanetta has joined #openstack-zaqar | 20:02 | |
| flwang | vkmc: ack | 20:05 |
| EmilienM | flwang: can you look https://review.openstack.org/#/c/282074/ when possible? thanks a ton | 20:06 |
| Eva-i | flwang: hello | 20:06 |
| flwang | EmilienM: sure thing | 20:06 |
| *** flwang1 has joined #openstack-zaqar | 20:06 | |
| flwang | mtreinish: ping re https://review.openstack.org/#/c/282074/ | 20:09 |
| flwang | mtreinish: EmilienM: just double confirm, with patch https://review.openstack.org/#/c/282074/, we just move the test from tempest into zaqar tree, but no change for the code | 20:10 |
| flwang | so the failure on test_messages and test_claims are still there, right? | 20:10 |
| flwang | Eva-i: hey :) | 20:10 |
| EmilienM | right | 20:10 |
| EmilienM | mtreinish did not fix tests for you | 20:10 |
| EmilienM | he just moved the code to your repo | 20:10 |
| EmilienM | so you can fix it and merge it quickly | 20:10 |
| EmilienM | iiuc | 20:10 |
| flwang | EmilienM: i see. does the plugin depends on Tempest to run? | 20:11 |
| EmilienM | yes | 20:11 |
| EmilienM | it's a tempest plugin | 20:11 |
| EmilienM | so if you don't run tempest in your gate, you might want to consider changing that | 20:12 |
| EmilienM | since your tempest tests seem broken (at least for me) you might want to set them non voting to start | 20:12 |
| mtreinish | flwang: I just copied the code as is | 20:12 |
| mtreinish | well I updated a couple of imports and things for living out of tree | 20:12 |
| mtreinish | but that's it | 20:12 |
| flwang | mtreinish: so my question is if i want to test it locally, do i have to setup Tempest first? | 20:13 |
| mtreinish | flwang: you clone tempest install zaqar into the same python (either system or venv) and tempest will detect the zaqar plugin like it was in tree | 20:14 |
| mtreinish | http://docs.openstack.org/developer/tempest/plugin.html#using-plugins | 20:14 |
| flwang | ah, i see | 20:14 |
| flwang | so I still need tempest, got it | 20:14 |
| flwang | i will merge it and fix the test in a separate patch | 20:16 |
| flwang | EmilienM: mtreinish: thank you guys | 20:16 |
| *** cpallares has quit IRC | 20:20 | |
| *** cpallares has joined #openstack-zaqar | 20:21 | |
| *** openstackgerrit has quit IRC | 20:47 | |
| *** openstackgerrit has joined #openstack-zaqar | 20:47 | |
| *** fribeiro has quit IRC | 20:48 | |
| *** fribeiro1 has joined #openstack-zaqar | 20:48 | |
| openstackgerrit | Merged openstack/zaqar: Update operation should return updated result https://review.openstack.org/280563 | 20:49 |
| openstackgerrit | Merged openstack/zaqar: Fix subscriptions_post resource in wsgi 2 homedoc https://review.openstack.org/283189 | 20:49 |
| *** malini1 has quit IRC | 20:49 | |
| openstackgerrit | Merged openstack/zaqar: Updated from global requirements https://review.openstack.org/282790 | 20:57 |
| Eva-i | flwang: ryansb: isn't it meeting time? =) | 21:01 |
| ryansb | yup | 21:02 |
| ryansb | vkmc won't be here today | 21:02 |
| *** sriram has quit IRC | 21:02 | |
| *** sriram has joined #openstack-zaqar | 21:04 | |
| *** mpanetta has quit IRC | 21:04 | |
| *** mpanetta has joined #openstack-zaqar | 21:04 | |
| *** flwang1 has quit IRC | 21:05 | |
| *** flwang1 has joined #openstack-zaqar | 21:07 | |
| *** sriram has quit IRC | 21:11 | |
| *** mpanetta has quit IRC | 21:12 | |
| *** mpanetta has joined #openstack-zaqar | 21:13 | |
| *** kgriffs|afk is now known as kgriffs | 21:33 | |
| *** itisha has joined #openstack-zaqar | 21:51 | |
| *** kgriffs is now known as kgriffs|afk | 21:55 | |
| Eva-i | ryansb: flwang1: think sometime about the case I described. I hope it's just my imagination gone too far or I don't understand something. | 22:02 |
| *** jtomasek has quit IRC | 22:03 | |
| ryansb | so that situation is exactly why I'm against permanent subscriptions | 22:03 |
| ryansb | and why there's no such thing in SQS, workers must check in to get new messages | 22:03 |
| flwang1 | ryansb: it's valid point | 22:04 |
| *** sriram has joined #openstack-zaqar | 22:04 | |
| *** sriram has quit IRC | 22:04 | |
| *** sriram has joined #openstack-zaqar | 22:04 | |
| *** sriram has quit IRC | 22:06 | |
| *** amitgandhinz has quit IRC | 22:07 | |
| *** fribeiro1 has quit IRC | 22:08 | |
| Eva-i | Are users (clients) of Zaqar server completely trusted entities? | 22:13 |
| ryansb | are users of Rackspace completely trusted? | 22:13 |
| ryansb | I'd have to say probably not | 22:13 |
| Eva-i | There can be no situation one of the users can try to abuse the system? | 22:13 |
| ryansb | they just have a valid credit card and a pulse | 22:13 |
| ryansb | and nobody checks the pulse. | 22:13 |
| Eva-i | ok | 22:13 |
| *** mpanetta has quit IRC | 22:17 | |
| Eva-i | I noticed also that SNS also asks confirmation from subscriber endpoint before starting to notify it. I wonder what is main reason behind this. | 22:20 |
| *** mpanetta has joined #openstack-zaqar | 22:23 | |
| ryansb | yeah, it may well be | 22:24 |
| flwang | Eva-i: that's one good feature we need to add for email subscriber | 22:25 |
| flwang | Eva-i: we had discussed this, but don't have much bandwidth to do it | 22:26 |
| flwang | i will create a bp to track it in case we miss it again :) | 22:26 |
| Eva-i | flwang: oki | 22:26 |
| *** akanksha_ has quit IRC | 22:27 | |
| Eva-i | flwang: and what about adding this feature to webhook? | 22:27 |
| *** kgriffs|afk is now known as kgriffs | 22:29 | |
| Eva-i | funny (maybe not real) case: zaqar-server is running on Ubuntu. The bad user subscribes "ubuntu.com" (or "security.ubuntu.com") to high load queue. Zaqar starts to send many requests to ubuntu.com. Eventually ubuntu's servers automatically ban zaqar-server's IP. Then the system, on which zaqar-server is running, is unable to get package updates. | 22:38 |
| flwang | Eva-i: how to get confirm from webhook? | 22:39 |
| ryansb | flwang: in SNS, it sends a message with a "subscription confirmation token" | 22:42 |
| flwang | ryansb: for each message? | 22:43 |
| ryansb | and the subscriber has to use that to sign a request to SNS saying "yes, that was me, I want to subscribe" | 22:43 |
| Eva-i | flwang: this is how I imagine: webhook server is listening for requests. When special payload from Zaqar comes to webhook server, the webhook server need to respond properly. After webhook server has responded, the subscription can be considered as confirmed. | 22:43 |
| ryansb | no, for the subscription | 22:43 |
| ryansb | so step 1, api call to subscribe an endpoint | 22:43 |
| Eva-i | flwang: not for each message, but for new subscription | 22:43 |
| ryansb | step 2, sns sends token to say "do you definitely want to subscribe?" | 22:43 |
| ryansb | step 3, subscriber sends message using token saying yes | 22:44 |
| flwang | ryansb: i see, for email, it's easy, but for webhook, i'm thinking how to do it elegantly | 22:45 |
| ryansb | recommendation: see SNS' docs on the topic | 22:47 |
| ryansb | notification to the topic and Amazon SNS sends an HTTP POST request delivering the contents of the notification t | 22:47 |
| ryansb | err http://docs.aws.amazon.com/sns/latest/dg/SendMessageToHttp.html | 22:48 |
| ryansb | I have to run - dinner time in NY | 22:48 |
| ryansb | If you want to open this as a bug that would be awesome, I think we need more discussion on this | 22:49 |
| flwang | ryansb: cool | 22:49 |
| flwang | i would like to make it as a feature | 22:49 |
| flwang | since it's a big change | 22:49 |
| flwang | and i will open a bug as well to track this :) | 22:51 |
| *** mpanetta has quit IRC | 22:58 | |
| *** malini has joined #openstack-zaqar | 23:03 | |
| *** kgriffs is now known as kgriffs|afk | 23:08 | |
| *** malini has quit IRC | 23:13 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!