Tuesday, 2016-01-26

flwangflaper87: thanks for sharing the thoughts00:11
*** Qiming has quit IRC00:15
flwangryansb: flaper87: can you help take a look https://review.openstack.org/#/c/267906/ ?00:21
*** david-lyle has joined #openstack-zaqar00:42
flwangEva-i: ping00:43
Eva-iflwang: I'm here, testing signed urls patch for the client00:43
flwangEva-i: ok, cool00:44
flwangEva-i: i'm looking at the comments you left on https://review.openstack.org/#/c/267923/00:44
Eva-iflwang: yes, so what do you think?00:45
flwangEva-i: let's clear the problem, did you mean current fix doesn't work?00:46
Eva-iflwang: it works, but one part of code of it never executes00:46
flwangEva-i: what do you mean?00:47
flwangnever executes?00:47
Eva-iflwang: I commented the "if queue.metadata() is None:" condition. The code under this condition never executes, because the condition always evaluates to False.00:49
*** kgriffs is now known as kgriffs|afk00:51
flwangEva-i: really?00:57
flwangEva-i: no matter if the queue exists or not?00:58
Eva-iflwang: yes00:59
flwangEva-i: let me try again, what's the api version you're testing?01:01
*** Qiming has joined #openstack-zaqar01:02
openstackgerritFei Long Wang proposed openstack/python-zaqarclient: Add subscriptions for queue object  https://review.openstack.org/26939801:04
*** kgriffs|afk is now known as kgriffs01:06
openstackgerritMerged openstack/python-zaqarclient: Fixing the deprecated library function  https://review.openstack.org/26619801:15
Eva-iflwang: I was a bit wrong about condition, but still the code under condition does not execute. In case of API v1 another exception is thrown before the condition evaluates: http://paste.openstack.org/show/aFOdLZgfenTiZYxUvnF5/. In case of API v1.1 and API v2 the empty dict is not None, here I'm playing with pdb: http://paste.openstack.org/show/E9jNFlux5coYmMQn3Yio/01:20
flwangEva-i: yep, you're right01:20
flwangEva-i: i don't really want to check the api version at here, so i prefer to catch the ResouceNotFound exception01:21
flwangthe 2nd comment you left01:21
Eva-iflwang: okay01:22
Eva-iflwang: am I right that in Zaqar API versions >=1.1, there's no way for the client to know, the queue exists or not? If yes, is it bad?01:24
flwangEva-i: basically yep, since after 1.1 the queue is lazy01:25
Eva-iflwang: oki, thank you01:27
Eva-iflwang: ryansb: is it okay that when I create a signed url without passing "expires" parameter, Zaqar returns a signed url with current time as "expires" parameter, resulting in signed url not valid right after creation.01:33
Eva-i?01:38
flwangEva-i: are you sure? see https://github.com/openstack/zaqar/blob/master/zaqar/common/urls.py#L6801:46
Eva-iflwang: oh, I see. I made a mistake01:47
flwangEva-i: :)01:48
openstackgerritwangxiyuan proposed openstack/zaqar: Ensure the format of input be JSON  https://review.openstack.org/27074801:49
openstackgerritwangxiyuan proposed openstack/zaqar: Ensure JSON sent over websocket is a dictionary  https://review.openstack.org/27074801:52
openstackgerritFei Long Wang proposed openstack/python-zaqarclient: Fix queue stats for v1.1 and above versions  https://review.openstack.org/26792301:55
*** amitgandhinz has joined #openstack-zaqar02:02
*** david-lyle has quit IRC02:05
openstackgerritwangxiyuan proposed openstack/zaqar: Ensure JSON sent over websocket is a dictionary  https://review.openstack.org/27074802:09
openstackgerritFei Long Wang proposed openstack/python-zaqarclient: Fix queue stats for v1.1 and above versions  https://review.openstack.org/26792302:12
*** kgriffs is now known as kgriffs|afk02:23
*** amitgandhinz has quit IRC02:28
openstackgerritMerged openstack/zaqar-ui: Fix installation instruction in README.rst  https://review.openstack.org/27213302:31
Eva-iI have some questions about Zaqar that make me feel uneasy: https://etherpad.openstack.org/p/eva-asks02:40
Eva-iflwang: if you'll have some free time, perhaps you could answer some of these questions ^^^02:40
*** superflyy has joined #openstack-zaqar02:50
*** pradk has quit IRC03:02
*** achanda has joined #openstack-zaqar03:09
openstackgerritwangxiyuan proposed openstack/zaqar: Ensure JSON sent over websocket is a dictionary  https://review.openstack.org/27074803:16
*** pradk has joined #openstack-zaqar03:16
*** david-lyle has joined #openstack-zaqar03:25
openstackgerritMerged openstack/zaqar: Make enforce_type=True in CONF.set_override  https://review.openstack.org/26633303:27
Eva-iflwang: I see you uploaded a new patch set for queue stats in the client. Would you like to fix the same problem in patch for "queue set metadata" and "queue get metadata"? If not, I should roll back changes to your bug report: https://bugs.launchpad.net/python-zaqarclient/+bug/153442703:43
openstackLaunchpad bug 1534427 in Python client library for Zaqar "Some queue CLI operations don't work for v1.1 and above" [Medium,In progress] - Assigned to Fei Long Wang (flwang)03:43
flwangEva-i: oh, thanks for the heads up03:45
flwangi will upload another patch03:46
Eva-iflwang: not patch set? Then another bug report should be opened03:46
flwangEva-i: no worries, patch set could work either03:47
flwangwe can use Partial-Bug tag03:47
Eva-iflwang: oki, you decide03:48
flwang:)03:49
Eva-iflwang: hm, this guy says that the blank panel will appear in zaqar-ui, if some tag is not written: https://review.openstack.org/#/c/258474/6/zaqar_ui/content/queues/templates/queues/index.html@7. Maybe this is why I see a blank panel.03:52
*** kgriffs|afk is now known as kgriffs03:55
flwangEva-i: but it works for me03:55
flwangEva-i:  i will wait for the comments from the horizon guy03:57
flwangfor now, you can try the comment from Shu Muto to see if it works for you03:57
Eva-iflwang: yes =)03:57
*** superflyy has quit IRC04:01
*** flwang1 has quit IRC04:04
*** achanda has quit IRC04:23
shu-mutouHi flwang, for blank panel issue, could you pull latest horizon and check again.04:23
*** achanda has joined #openstack-zaqar04:23
openstackgerritting wang proposed openstack/zaqar: Replace six.iteritems(dict) with dict.items()  https://review.openstack.org/26841004:31
*** achanda has quit IRC04:36
*** pradk has quit IRC04:56
Eva-iflwang: thank you for today, I'm going to sleep =)05:00
*** pradk has joined #openstack-zaqar05:08
openstackgerritShu Muto proposed openstack/python-zaqarclient: [typo] queue list doesn't work  https://review.openstack.org/27239405:52
*** achanda has joined #openstack-zaqar06:28
*** kgriffs is now known as kgriffs|afk06:37
*** rcernin has joined #openstack-zaqar06:53
*** GB21 has joined #openstack-zaqar07:05
*** GB21 has quit IRC07:59
openstackgerritting wang proposed openstack/zaqar-ui: remove python 3.3 and 2.6 classifier  https://review.openstack.org/27242608:19
*** achanda has quit IRC08:23
*** wanghao has joined #openstack-zaqar08:25
*** GB21 has joined #openstack-zaqar08:27
*** dynarro has joined #openstack-zaqar08:48
openstackgerritting wang proposed openstack/zaqar-ui: Trival: Remove 'MANIFEST.in'  https://review.openstack.org/27243108:57
*** achanda has joined #openstack-zaqar09:24
*** achanda has quit IRC09:30
*** dynarro has quit IRC10:04
*** Qiming has quit IRC10:05
*** achanda has joined #openstack-zaqar10:27
*** achanda has quit IRC10:33
*** eliqiao has quit IRC10:55
*** Qiming has joined #openstack-zaqar10:56
*** eliqiao has joined #openstack-zaqar10:57
*** Qiming has quit IRC11:09
*** Qiming has joined #openstack-zaqar11:09
*** achanda has joined #openstack-zaqar11:30
*** achanda has quit IRC11:35
*** pradk has quit IRC12:52
vkmcflwang, hey hey, can we merge this one? https://review.openstack.org/#/c/271780/13:30
*** achanda has joined #openstack-zaqar13:32
*** achanda has quit IRC13:38
ryansbvkmc: done13:40
vkmcthanks ryansb13:40
ryansbah, looks like flaper beat me to it13:40
ryansbMr. Quick-Draw over there13:41
vkmchahaha13:41
vkmcflaper87 is a ninja reviewer13:42
ryansbtruth13:42
*** jtomasek has joined #openstack-zaqar13:52
openstackgerritMerged openstack/python-zaqarclient: Fix typo preventing queue list for v1  https://review.openstack.org/27178013:55
openstackgerritRyan Brown proposed openstack/python-zaqarclient: Add CLI support for signed URLs  https://review.openstack.org/25373113:57
*** sriram has joined #openstack-zaqar14:04
*** amitgandhinz has joined #openstack-zaqar14:09
*** dynarro has joined #openstack-zaqar14:34
*** pradk has joined #openstack-zaqar14:34
*** boris-42 has quit IRC14:34
*** boris-42 has joined #openstack-zaqar14:35
*** Qiming has quit IRC14:53
*** kgriffs|afk is now known as kgriffs14:54
*** malini has joined #openstack-zaqar15:14
openstackgerritMerged openstack/zaqar: Ensure JSON sent over websocket is a dictionary  https://review.openstack.org/27074815:33
*** achanda has joined #openstack-zaqar15:36
*** achanda has quit IRC15:41
*** pradk has left #openstack-zaqar15:43
*** GB21 has quit IRC15:54
*** GB21 has joined #openstack-zaqar16:01
*** GB21 has quit IRC16:23
*** mpanetta has joined #openstack-zaqar16:26
*** achanda has joined #openstack-zaqar16:49
*** ametts has joined #openstack-zaqar17:01
*** jtomasek has quit IRC17:07
*** jtomasek has joined #openstack-zaqar17:08
*** dynarro has quit IRC17:17
*** openstackgerrit has quit IRC17:17
*** openstackgerrit has joined #openstack-zaqar17:17
openstackgerritEva Balycheva proposed openstack/zaqar: Support binary messages over websocket  https://review.openstack.org/25697817:29
*** EmilienM has quit IRC17:30
*** EmilienM has joined #openstack-zaqar17:34
openstackgerritEva Balycheva proposed openstack/zaqar: Support binary messages over websocket  https://review.openstack.org/25697817:36
*** achanda has quit IRC17:40
openstackgerritEva Balycheva proposed openstack/zaqar: Fix typo in test_on_mesage_with_invalid_input  https://review.openstack.org/27267517:53
*** achanda has joined #openstack-zaqar18:10
*** malini has quit IRC18:20
*** achanda has quit IRC18:29
*** malini has joined #openstack-zaqar18:47
*** achanda has joined #openstack-zaqar18:48
*** kgriffs is now known as kgriffs|afk18:49
*** sriram has quit IRC19:03
*** sriram has joined #openstack-zaqar19:03
*** amitgandhinz has quit IRC19:04
*** amitgandhinz has joined #openstack-zaqar19:05
*** achanda has quit IRC19:21
*** achanda has joined #openstack-zaqar19:23
*** achanda has quit IRC19:25
openstackgerritMerged openstack/zaqar: Replace six.iteritems(dict) with dict.items()  https://review.openstack.org/26841019:46
openstackgerritMerged openstack/zaqar: Fix typo in test_on_mesage_with_invalid_input  https://review.openstack.org/27267519:49
*** rcernin has quit IRC19:58
*** pradk_ has joined #openstack-zaqar20:14
*** pradk_ is now known as pradk20:14
*** kgriffs|afk is now known as kgriffs20:18
*** flwang1 has joined #openstack-zaqar20:20
flwangmorning, guys20:21
*** rcernin has joined #openstack-zaqar20:32
*** flwang has left #openstack-zaqar20:33
*** flwang has joined #openstack-zaqar20:34
openstackgerritMerged openstack/python-zaqarclient: Clean up flavor test classes  https://review.openstack.org/27181220:49
*** AJaeger has joined #openstack-zaqar20:50
AJaegerhi zaqar cores, could I get another +2 on https://review.openstack.org/#/c/270401/ - removes argparse which is a python 2.6 relict and zaqar does not support 2.6 anymore20:50
AJaeger, please?20:50
vkmcAJaeger, yup20:51
AJaegerthanks20:51
*** malini has quit IRC20:52
*** pradk has quit IRC20:53
*** pradk has joined #openstack-zaqar20:54
flwangYEAH!   just realized i have been removed from glance core20:54
flwangnow i can only focus on zaqar20:55
flwangno applause ?20:56
vkmclol20:57
vkmc\o/20:57
flwangvkmc: haha21:00
*** AJaeger has left #openstack-zaqar21:02
ryansb\o/21:03
openstackgerritMerged openstack/zaqar: Remove argparse from requirements  https://review.openstack.org/27040121:09
Eva-iflwang: yay21:09
*** boris-42 has quit IRC21:23
*** achanda has joined #openstack-zaqar21:27
*** rcernin has quit IRC21:28
*** achanda has quit IRC21:30
flwangryansb: did you see the discussion flaper87 and me about the summit pres?21:30
ryansbno, when was it? Might be in my logs21:31
flwangryansb: flaper87 suggested we should focus on a special user case21:35
ryansbspecial as in just one specific one?21:38
ryansbe.g. people with worker farms, or something?21:38
flwangryansb: i think he is talking about a specific function, like pre-signed,  or notification21:39
ryansbgotcha21:40
flwangryansb: but i think we can demo most of our functions in a user case21:40
flwangflaper87: ^21:40
ryansbI think "hey check out signed URLs" is less interesting to your average summit attendee than "if you've got a worker farm, here's how zaqar makes your life easy"21:41
flwangryansb: i'm on your side :D21:42
flwangmaybe i missed something21:42
flwangfor flaper87's suggestion21:42
flwangryansb: personally, i would like to work out a demo how zaqar can work with a worker farm(micro services cluster)21:44
flwangand how the application/micro service can be benefited from zaqar21:44
ryansb++21:44
flwangclassic image process sample https://aws.amazon.com/articles/PHP/160221:45
flwanghttps://aws.amazon.com/search?searchQuery=SQS&searchPath=articles&x=0&y=021:46
flwangwe really need to have those kind of docs21:46
flwangeven demo/sample code21:46
ryansbyeah, those would be really nice guides21:52
*** ametts has quit IRC21:55
flwangryansb: you're the user of SQS/SNS, right? do you have any idea about this user case?21:59
flwangcan show the power of zaqar?21:59
ryansbI don't know if there's specific use case that is zaqar-specific22:03
ryansbthe same SQS case (image resizer or whatever) will do fine with zaqar22:03
flwangryansb: yep22:03
ryansband workers can be given signed URLs to grab their jobs from a central server22:03
ryansbso that's neat22:04
ryansbI'll see if I can put together a little worker script that just pops messages and pretends to work a bit22:05
ryansb(in python, of course)22:05
flwangyep, python should be fine22:06
flwangryansb: oh, and sorry, i didn't send email22:07
ryansbno worries :)22:07
flwangyesterday about this topic22:07
ryansbI have to run again - dinner time in my tz. I'll let you know when I have the demo worker22:08
flwangryansb: awesome, thanks a lot22:09
openstackgerritMerged openstack/zaqar: Add tutorial for writing storage pipeline stages  https://review.openstack.org/26872022:32
*** amitgandhinz has quit IRC22:50
flwangjasondotstar: ping re the puppet work22:52
*** sriram has quit IRC22:54
*** boris-42 has joined #openstack-zaqar22:54
*** mpanetta has quit IRC22:58
*** achanda has joined #openstack-zaqar22:58
*** kgriffs is now known as kgriffs|afk23:29
*** Qiming has joined #openstack-zaqar23:50
*** achanda has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!