| *** hparekh has quit IRC | 03:05 | |
| *** hparekh has joined #openstack-zaqar | 03:19 | |
| *** davideag_ has quit IRC | 04:00 | |
| *** davideagnello has joined #openstack-zaqar | 05:00 | |
| *** diga has joined #openstack-zaqar | 05:03 | |
| *** davideagnello has quit IRC | 05:05 | |
| *** Eva-i has quit IRC | 05:26 | |
| *** GB21 has joined #openstack-zaqar | 05:33 | |
| *** davideagnello has joined #openstack-zaqar | 06:01 | |
| *** davideagnello has quit IRC | 06:06 | |
| *** davideagnello has joined #openstack-zaqar | 07:02 | |
| *** davideagnello has quit IRC | 07:07 | |
| *** davideagnello has joined #openstack-zaqar | 08:03 | |
| *** davideagnello has quit IRC | 08:08 | |
| openstackgerrit | Yatin Kumbhare proposed openstack/zaqar: Changing LOG.warn to LOG.warning https://review.openstack.org/263169 | 09:04 |
|---|---|---|
| *** davideagnello has joined #openstack-zaqar | 09:04 | |
| *** Eva-i has joined #openstack-zaqar | 09:07 | |
| *** davideagnello has quit IRC | 09:08 | |
| *** diga_ has joined #openstack-zaqar | 09:45 | |
| *** diga has quit IRC | 09:46 | |
| *** vipuls has quit IRC | 09:55 | |
| *** vipuls has joined #openstack-zaqar | 09:58 | |
| *** openstackgerrit has quit IRC | 10:02 | |
| *** openstackgerrit has joined #openstack-zaqar | 10:02 | |
| *** davideagnello has joined #openstack-zaqar | 10:05 | |
| *** davideagnello has quit IRC | 10:11 | |
| *** GB21 has quit IRC | 11:06 | |
| *** davideagnello has joined #openstack-zaqar | 11:07 | |
| *** davideagnello has quit IRC | 11:12 | |
| *** GB21 has joined #openstack-zaqar | 11:18 | |
| *** GB21 has quit IRC | 11:51 | |
| *** Guest52219 is now known as VeggieMeat | 12:12 | |
| *** diga_ has quit IRC | 12:46 | |
| *** enthurohini has joined #openstack-zaqar | 12:56 | |
| *** davideagnello has joined #openstack-zaqar | 13:09 | |
| *** enthurohini has left #openstack-zaqar | 13:09 | |
| *** davideagnello has quit IRC | 13:13 | |
| openstackgerrit | Merged openstack/zaqar: Put py34 first in the env order of tox https://review.openstack.org/254121 | 13:31 |
| *** dynarro has joined #openstack-zaqar | 13:46 | |
| *** mpanetta has joined #openstack-zaqar | 13:56 | |
| *** davideagnello has joined #openstack-zaqar | 14:10 | |
| *** davideagnello has quit IRC | 14:15 | |
| *** sriram has joined #openstack-zaqar | 14:40 | |
| openstackgerrit | Yatin Kumbhare proposed openstack/zaqar: Fix for the deprecated library function https://review.openstack.org/263288 | 14:40 |
| *** ametts has joined #openstack-zaqar | 14:52 | |
| *** kgriffs|afk is now known as kgriffs | 14:53 | |
| *** malini has joined #openstack-zaqar | 14:57 | |
| *** csoukup has joined #openstack-zaqar | 14:57 | |
| *** amitgandhinz has joined #openstack-zaqar | 15:07 | |
| *** davideagnello has joined #openstack-zaqar | 15:10 | |
| *** shakamunyi has quit IRC | 15:15 | |
| *** barra204 has quit IRC | 15:15 | |
| *** davideagnello has quit IRC | 15:15 | |
| *** pradk_ has joined #openstack-zaqar | 15:19 | |
| *** pradk_ has quit IRC | 15:19 | |
| *** pradk has joined #openstack-zaqar | 15:22 | |
| *** davideagnello has joined #openstack-zaqar | 15:24 | |
| *** shakamunyi has joined #openstack-zaqar | 15:29 | |
| *** kgriffs is now known as kgriffs|afk | 15:44 | |
| *** malini has quit IRC | 16:14 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/zaqar: Replace deprecated LOG.warn with warning https://review.openstack.org/263323 | 16:17 |
| *** dmowrer has joined #openstack-zaqar | 16:19 | |
| *** malini has joined #openstack-zaqar | 16:24 | |
| *** kgriffs|afk is now known as kgriffs | 16:36 | |
| *** ametts has quit IRC | 16:37 | |
| *** ametts has joined #openstack-zaqar | 16:41 | |
| *** barra204 has joined #openstack-zaqar | 16:51 | |
| *** shakamunyi has quit IRC | 16:51 | |
| Eva-i | flwang: ping | 17:20 |
| *** dynarro has quit IRC | 17:33 | |
| openstackgerrit | LiuNanke proposed openstack/zaqar: Test: make enforce_type=True in CONF.set_override https://review.openstack.org/263359 | 17:34 |
| *** dmowrer has quit IRC | 17:36 | |
| *** openstackgerrit has quit IRC | 18:32 | |
| *** openstackgerrit has joined #openstack-zaqar | 18:32 | |
| *** malini has quit IRC | 18:46 | |
| *** dmowrer has joined #openstack-zaqar | 18:47 | |
| *** boris-42 has quit IRC | 18:53 | |
| *** malini has joined #openstack-zaqar | 19:13 | |
| *** openstackstatus has quit IRC | 19:20 | |
| *** openstackstatus has joined #openstack-zaqar | 19:23 | |
| *** ChanServ sets mode: +v openstackstatus | 19:23 | |
| flwang | pong | 19:28 |
| flwang | Eva-i: pong | 19:28 |
| Eva-i | flwang: how are you? Are your holidays ended? | 19:30 |
| flwang | Eva-i: very good, thank you | 19:30 |
| flwang | yes, today is my first working day after the holiday | 19:30 |
| Eva-i | flwang: oki, me too. Today I catched Anne Gentle and asked her about who will be maintainer of Zaqar config-ref in openstack-manuals repository and she said this: | 19:32 |
| Eva-i | flwang: "Zaqar team for the most part, the doc team handles any translation or build work necessary but the project teams should care for the content updates as they'll know when those are needed" | 19:32 |
| Eva-i | flwang: I think I should include info about inability to generate configuration tables automatically in my config-ref patch commit message. | 19:33 |
| flwang | define 'include'? :D | 19:38 |
| Eva-i | flwang: kind of like this: http://pastebin.com/uWtT8Gfh | 19:40 |
| flwang | i see, it's ok for me | 19:43 |
| Eva-i | flwang: okay, I'm submitting the patch then | 19:44 |
| Eva-i | flwang: https://review.openstack.org/263405 here | 19:45 |
| Eva-i | flwang: also I fixed Zaqar dsvm gates recently! ;) | 19:47 |
| flwang | Eva-i: yep, i saw that, i would like to give you a big hug | 19:49 |
| flwang | Eva-i: thank you very much | 19:49 |
| vkmc | o/ | 19:51 |
| vkmc | hey guys | 19:51 |
| vkmc | Eva-i, flwang | 19:51 |
| vkmc | happy new year and welcome back :) | 19:51 |
| flwang | vkmc: oh, hey | 19:51 |
| Eva-i | vkmc: hello | 19:51 |
| flwang | i miss you | 19:52 |
| vkmc | flwang, miss you too! | 19:52 |
| vkmc | has been quite long holidays heh | 19:52 |
| vkmc | do we have the meeting in one hour? | 19:52 |
| flwang | vkmc: i would like to have a quick meeting | 19:53 |
| vkmc | cool :) | 19:53 |
| vkmc | sounds good to me too | 19:53 |
| *** amitgandhinz has quit IRC | 20:05 | |
| *** amitgandhinz has joined #openstack-zaqar | 20:06 | |
| flwang | Eva-i: ping | 20:13 |
| Eva-i | flwang: pong | 20:13 |
| flwang | as for the subscription create patch | 20:13 |
| flwang | did you test it? | 20:13 |
| Eva-i | flwang: hm, I tested it before patch set 10. The next patch sets I haven't tested. | 20:16 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/zaqar: Updated from global requirements https://review.openstack.org/258076 | 20:16 |
| flwang | Eva-i: ok, no problem. | 20:18 |
| flwang | i'm going to submit a new patchset | 20:18 |
| flwang | as for the your TTL comments | 20:18 |
| flwang | i noticed the bug | 20:18 |
| flwang | but i'm thinking if we should fix it or just totally remove the TTL | 20:18 |
| Eva-i | flwang: personally I think TTL is needed. If Zaqar server will remember subscriptions forever and send notifications, it will be a bad thing. | 20:21 |
| flwang | why it's bad? | 20:22 |
| flwang | it should be handled by end user manually | 20:22 |
| Eva-i | flwang: the users might forget to delete subscriptions manually | 20:22 |
| flwang | for me, the hard part is in mongo db, there is no way to have a non expired TTL index | 20:23 |
| flwang | i know TTL is useful in some cases, maybe | 20:23 |
| flwang | but now the problem is there is no way to have a permanent subscription | 20:24 |
| Eva-i | flwang: okay, I'll think about this today | 20:27 |
| Eva-i | flwang: by "there is no way to have a non expired TTL index" you mean to make mongodb automatically clean up subscriptions, when TTL index is expired? | 20:27 |
| ryansb | I think TTLs should exist for almost everything | 20:29 |
| ryansb | esp. subscriptions, since they require resources to maintain | 20:29 |
| ryansb | and in cloud-y messaging scenarios there's no guarantee about how long a subscriber will stick around (or that it will know ahead of time it's getting killed) | 20:30 |
| ryansb | (also, happy new year folks!) | 20:30 |
| Eva-i | ryansb: hello, happy new year | 20:30 |
| flwang | ryansb: hey man | 20:31 |
| flwang | ryansb: good comments | 20:31 |
| flwang | ryansb: but does that mean we just need to let the end user input a very big number for the TTL if they want a very long even permanent subscription? | 20:32 |
| ryansb | no, it means users should *renew* subscriptions | 20:32 |
| ryansb | either by creating new ones or by extending the TTL of the one they're using | 20:32 |
| ryansb | we could event do a use-based TTL, like "if this subscriber isn't seen in X minutes/hours/days" | 20:33 |
| flwang | create new ones? i think we don't allow create duplicated subscriptions for same subscriber | 20:33 |
| ryansb | gotcha, so scratch that one | 20:34 |
| flwang | and what do you mean 'extending the TTL'? | 20:34 |
| Eva-i | flwang: claims can be renewed(TTL can be extended) https://wiki.openstack.org/wiki/Zaqar/specs/api/v1.1#Renew_Claim, maybe ryansb means something like this | 20:39 |
| ryansb | The subscription TTL, I mean | 20:39 |
| ryansb | Eva-i: exactly, actually | 20:39 |
| ryansb | that way we don't have a zillion subscriptions that will never get used | 20:39 |
| ryansb | but users don't have to make new client IDs or other hassles | 20:39 |
| flwang | ryansb: i see | 20:40 |
| Eva-i | "subscription update" | 20:41 |
| ryansb | yeah, basically | 20:41 |
| flwang | ok, i can see your point just like why i keep the TTL there when i coded it | 20:41 |
| flwang | but i still would like to figure out a way to let end user can create a permanent subscription | 20:41 |
| flwang | anyway | 20:41 |
| flwang | i will fix the bug | 20:41 |
| Eva-i | flwang: also I think now that the new client patches related to subscriptions are missing ttl and age properties in zaqarclient/queues/v2/subscription.py like in zaqarclient/queues/v1/claim.py | 20:45 |
| flwang | age? | 20:50 |
| *** malini has quit IRC | 20:50 | |
| flwang | Eva-i: i don't think we have an 'age' property for subscription | 20:51 |
| Eva-i | flwang: the user might want to know subscription's age to know if time update the subscription has come, I'm not sure | 20:51 |
| flwang | Eva-i: hmm... i see. but i need some thinking :) | 20:53 |
| *** amitgandhinz has quit IRC | 20:57 | |
| flwang | vkmc: ryansb: Eva-i: flaper87: meeting | 20:59 |
| *** ametts has quit IRC | 20:59 | |
| Eva-i | flwang: yes, sure. Maybe I wasn't right when I said in the bug report that automatic ttl expiration is easy to make in mongo db. For some reason claim controller zaqar/storage/mongodb/claims.py don't use expireAfterSeconds parameter | 20:59 |
| vkmc | \o/ | 20:59 |
| Eva-i | *doesn't | 21:00 |
| *** amitgandhinz has joined #openstack-zaqar | 21:16 | |
| *** ametts has joined #openstack-zaqar | 21:26 | |
| *** mdnadeem has joined #openstack-zaqar | 21:29 | |
| *** mdnadeem has quit IRC | 21:42 | |
| openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Add support for subscription create v2 https://review.openstack.org/249395 | 21:58 |
| flwang | ryansb: vkmc: could you pls review https://review.openstack.org/249395 ? | 22:03 |
| ryansb | yeah | 22:04 |
| flwang | ryansb: thanks :) | 22:09 |
| openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Add support for subscription create v2 https://review.openstack.org/249395 | 22:13 |
| flwang | ryansb: comments addressed :) | 22:13 |
| Eva-i | flwang: with this simple workaround diff, Zaqar is able to be used with the table autogeneration tools: http://pastebin.com/tfp2GP6H | 22:14 |
| Eva-i | I can explain this in the config reference patch and provide the link to this diff. | 22:14 |
| *** dmowrer has quit IRC | 22:20 | |
| *** dmowrer has joined #openstack-zaqar | 22:21 | |
| *** dmowrer has quit IRC | 22:25 | |
| *** mpanetta has quit IRC | 22:27 | |
| flwang | Eva-i: cool, and i will talk with AJ to see what's his concern | 22:31 |
| flwang | thanks for the work | 22:31 |
| Eva-i | flwang: oki, commit message updated: https://review.openstack.org/#/c/263405/ | 22:36 |
| * Eva-i going to bed | 22:36 | |
| Eva-i | good night | 22:36 |
| openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Add support for subscription show v2 https://review.openstack.org/261282 | 22:38 |
| flwang | Eva-i: thank you, good night | 22:40 |
| openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Add support for subscription update v2 https://review.openstack.org/261303 | 22:40 |
| *** kgriffs is now known as kgriffs|afk | 22:46 | |
| *** amitgandhinz has quit IRC | 22:46 | |
| *** kgriffs|afk is now known as kgriffs | 23:02 | |
| *** ametts has quit IRC | 23:11 | |
| openstackgerrit | Merged openstack/zaqar: remove python 2.6 trove classifier https://review.openstack.org/260852 | 23:11 |
| openstackgerrit | Merged openstack/zaqar: Fix for the deprecated library function https://review.openstack.org/263288 | 23:13 |
| *** dmowrer has joined #openstack-zaqar | 23:21 | |
| *** dmowrer has quit IRC | 23:27 | |
| *** csoukup has quit IRC | 23:27 | |
| *** boris-42 has joined #openstack-zaqar | 23:33 | |
| *** pradk has quit IRC | 23:35 | |
| *** stanchan has joined #openstack-zaqar | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!