*** mpanetta has joined #openstack-zaqar | 01:32 | |
*** yanyanhu has joined #openstack-zaqar | 01:34 | |
*** mpanetta has quit IRC | 01:37 | |
*** wxy__ has quit IRC | 05:50 | |
*** wxy__ has joined #openstack-zaqar | 05:51 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/zaqar-ui: Imported Translations from Zanata https://review.openstack.org/392431 | 06:44 |
---|---|---|
*** rcernin has joined #openstack-zaqar | 06:50 | |
*** tesseract has joined #openstack-zaqar | 07:17 | |
*** tesseract is now known as Guest48712 | 07:17 | |
*** openstackgerrit has quit IRC | 07:48 | |
*** openstackgerrit has joined #openstack-zaqar | 07:48 | |
*** pcaruana has joined #openstack-zaqar | 08:09 | |
*** yanyanhu has quit IRC | 08:13 | |
*** GB21 has joined #openstack-zaqar | 08:55 | |
*** GB21 has quit IRC | 09:15 | |
*** exploreshaifali has joined #openstack-zaqar | 09:52 | |
*** exploreshaifali has quit IRC | 10:22 | |
*** exploreshaifali has joined #openstack-zaqar | 10:25 | |
*** exploreshaifali has quit IRC | 10:43 | |
*** exploreshaifali has joined #openstack-zaqar | 10:57 | |
*** GB21 has joined #openstack-zaqar | 11:15 | |
*** GB21 has quit IRC | 11:33 | |
*** jasond has joined #openstack-zaqar | 14:00 | |
*** GB21 has joined #openstack-zaqar | 14:01 | |
*** mpanetta has joined #openstack-zaqar | 14:20 | |
*** dynarro has joined #openstack-zaqar | 14:52 | |
*** dynarro has quit IRC | 15:07 | |
*** dynarro has joined #openstack-zaqar | 15:08 | |
*** exploreshaifali has quit IRC | 15:22 | |
*** dynarro has quit IRC | 15:22 | |
*** dynarro has joined #openstack-zaqar | 15:28 | |
*** pcaruana has quit IRC | 16:06 | |
*** rcernin has quit IRC | 16:09 | |
*** Guest48712 has quit IRC | 16:34 | |
*** dynarro has quit IRC | 16:45 | |
*** harlowja has quit IRC | 17:21 | |
*** harlowja has joined #openstack-zaqar | 17:44 | |
*** mpanetta has quit IRC | 17:51 | |
*** mpanetta has joined #openstack-zaqar | 17:51 | |
*** dynarro has joined #openstack-zaqar | 18:05 | |
*** openstackgerrit has quit IRC | 18:18 | |
*** openstackgerrit has joined #openstack-zaqar | 18:18 | |
*** exploreshaifali has joined #openstack-zaqar | 18:19 | |
*** GB21 has quit IRC | 18:24 | |
*** exploreshaifali has quit IRC | 19:28 | |
*** rcernin has joined #openstack-zaqar | 19:45 | |
*** openstack has joined #openstack-zaqar | 19:58 | |
*** jtomasek has quit IRC | 20:17 | |
*** openstackgerrit has quit IRC | 20:18 | |
*** openstackgerrit has joined #openstack-zaqar | 20:18 | |
*** dynarro has quit IRC | 21:11 | |
*** exploreshaifali has joined #openstack-zaqar | 21:25 | |
flwang | jasond: pong. still around? | 21:26 |
jasond | flwang: hey | 21:26 |
flwang | jasond: how can i help? | 21:26 |
flwang | i missed your last ping, since i just back to NZ | 21:26 |
jasond | flwang: trying to remember.. i think it was about https://bugs.launchpad.net/heat/+bug/1637304 | 21:27 |
openstack | Launchpad bug 1637304 in heat "Zaqar queue can be inadvertently deleted" [Low,Triaged] - Assigned to Jason Dunsmore (jasondunsmore) | 21:27 |
jasond | flwang: but it's pretty much settled, we can handle it on the Heat side | 21:27 |
jasond | flwang: oh BTW, if subscriptions have unique IDs, then why is it that Subscriptions are unique by project + queue/topic + subscriber? | 21:28 |
jasond | flwang: seems like we might have the same issue with lazily-created subscriptions | 21:29 |
flwang | since if we just use subscriber to distinguish a subscription, it's not easy to manage | 21:30 |
flwang | can you see my point? | 21:30 |
flwang | firstly, a subscription must be associated with a queue/topic | 21:30 |
jasond | flwang: sure | 21:30 |
flwang | though we can use queue+subscriber to distinguish a subscription, but | 21:31 |
flwang | it could be something like: myqueye+https://myverylong_webhook:8080/aaaa/bbb/ccc/queuue.aspx?xx=113 | 21:32 |
flwang | or something like that | 21:32 |
jasond | flwang: on second thought, if two stacks never share the same queue, then i don't think we'll have a problem with subscriptions | 21:32 |
flwang | the subscription id is mainly used to do the update/delete/get a particular subscription | 21:32 |
flwang | yes | 21:32 |
jasond | flwang: ok, got it. thanks | 21:33 |
flwang | hence why i asked in the bug, why does heat need to share a same queue for different stack creations? | 21:33 |
flwang | i think that's the root cause | 21:33 |
jasond | flwang: actually, we're trying to avoid that, but it might happen accidentally | 21:33 |
flwang | because for any other similar scenarios, if you want to share same object between two cases, and try to delete the object separately, you will run into problem for sure, right? | 21:34 |
jasond | flwang: eg. if someone chooses a very simple queue name | 21:34 |
flwang | i see | 21:34 |
flwang | then why don't you just give the queue with a default name -- using stack's name | 21:34 |
flwang | unless user want to specify a customized name | 21:35 |
jasond | flwang: yeah, i think that will work | 21:35 |
flwang | and IIRC, the main purpose of the queue, is to saving the status change event/notification, right? | 21:35 |
jasond | flwang: the part i'm working on is the template resource, so the end user could create it for any number of purposes | 21:36 |
flwang | i see | 21:36 |
flwang | anyway, "sharing" the same queue is not a best practice :) | 21:38 |
jasond | flwang: ok, thanks, good tip :) | 21:38 |
flwang | feel free let me know if there is anything i can do from our side | 21:39 |
jasond | flwang: and deleting a shared queue is even worse practice | 21:39 |
flwang | hah, true | 21:39 |
jasond | flwang: ok, will do | 21:39 |
flwang | actually, like therve said in the comments, it's not really necessary to delete the queue for most of the cases because it's lazy | 21:39 |
flwang | all the messages have TTl | 21:40 |
flwang | so technically, there is nothing left | 21:40 |
flwang | only the queue/topic, which is a logic concept and it won't occupy your much resource | 21:40 |
jasond | flwang: ah, so maybe we shouldn't delete the queue, in case the user decides to specify the optional subscription queue name (which defaults to stack name) | 21:41 |
jasond | flwang: nevermind the last half of that sentence :) | 21:42 |
flwang | hah | 21:43 |
flwang | yep | 21:43 |
jasond | flwang: one more thing. is there a way to make a subscription never expire? | 21:46 |
flwang | technically yes | 21:46 |
flwang | wait a sec | 21:46 |
flwang | see https://github.com/openstack/zaqar/blob/master/zaqar/transport/validation.py#L529 | 21:48 |
flwang | if the TTL + current time is a valid time in Python, then it's ok | 21:48 |
flwang | the max year in python is 9999 | 21:49 |
jasond | flwang: ok, great | 21:49 |
flwang | so, you got the point | 21:51 |
flwang | i tried to support permanent subscription, but it's objected by the other core developers | 21:51 |
flwang | so that's the middle point | 21:52 |
jasond | flwang: yeah, that will work fine | 21:53 |
flwang | cool | 21:53 |
*** harlowja has quit IRC | 22:02 | |
*** jasond has quit IRC | 22:17 | |
*** flaper87 has quit IRC | 22:29 | |
*** exploreshaifali has quit IRC | 22:31 | |
openstackgerrit | Merged openstack/zaqar-ui: Change "Openstack" to "OpenStack" https://review.openstack.org/390567 | 22:34 |
openstackgerrit | Merged openstack/python-zaqarclient: Files with no code must be left completely empty https://review.openstack.org/378452 | 22:48 |
*** mpanetta has quit IRC | 23:02 | |
*** david-lyle has quit IRC | 23:04 | |
*** harlowja has joined #openstack-zaqar | 23:08 | |
*** david-lyle has joined #openstack-zaqar | 23:30 | |
*** rcernin has quit IRC | 23:31 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!