flwang | flaper87: thanks for sharing the thoughts | 00:11 |
---|---|---|
*** Qiming has quit IRC | 00:15 | |
flwang | ryansb: flaper87: can you help take a look https://review.openstack.org/#/c/267906/ ? | 00:21 |
*** david-lyle has joined #openstack-zaqar | 00:42 | |
flwang | Eva-i: ping | 00:43 |
Eva-i | flwang: I'm here, testing signed urls patch for the client | 00:43 |
flwang | Eva-i: ok, cool | 00:44 |
flwang | Eva-i: i'm looking at the comments you left on https://review.openstack.org/#/c/267923/ | 00:44 |
Eva-i | flwang: yes, so what do you think? | 00:45 |
flwang | Eva-i: let's clear the problem, did you mean current fix doesn't work? | 00:46 |
Eva-i | flwang: it works, but one part of code of it never executes | 00:46 |
flwang | Eva-i: what do you mean? | 00:47 |
flwang | never executes? | 00:47 |
Eva-i | flwang: 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|afk | 00:51 | |
flwang | Eva-i: really? | 00:57 |
flwang | Eva-i: no matter if the queue exists or not? | 00:58 |
Eva-i | flwang: yes | 00:59 |
flwang | Eva-i: let me try again, what's the api version you're testing? | 01:01 |
*** Qiming has joined #openstack-zaqar | 01:02 | |
openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Add subscriptions for queue object https://review.openstack.org/269398 | 01:04 |
*** kgriffs|afk is now known as kgriffs | 01:06 | |
openstackgerrit | Merged openstack/python-zaqarclient: Fixing the deprecated library function https://review.openstack.org/266198 | 01:15 |
Eva-i | flwang: 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 |
flwang | Eva-i: yep, you're right | 01:20 |
flwang | Eva-i: i don't really want to check the api version at here, so i prefer to catch the ResouceNotFound exception | 01:21 |
flwang | the 2nd comment you left | 01:21 |
Eva-i | flwang: okay | 01:22 |
Eva-i | flwang: 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 |
flwang | Eva-i: basically yep, since after 1.1 the queue is lazy | 01:25 |
Eva-i | flwang: oki, thank you | 01:27 |
Eva-i | flwang: 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 |
flwang | Eva-i: are you sure? see https://github.com/openstack/zaqar/blob/master/zaqar/common/urls.py#L68 | 01:46 |
Eva-i | flwang: oh, I see. I made a mistake | 01:47 |
flwang | Eva-i: :) | 01:48 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Ensure the format of input be JSON https://review.openstack.org/270748 | 01:49 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Ensure JSON sent over websocket is a dictionary https://review.openstack.org/270748 | 01:52 |
openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Fix queue stats for v1.1 and above versions https://review.openstack.org/267923 | 01:55 |
*** amitgandhinz has joined #openstack-zaqar | 02:02 | |
*** david-lyle has quit IRC | 02:05 | |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Ensure JSON sent over websocket is a dictionary https://review.openstack.org/270748 | 02:09 |
openstackgerrit | Fei Long Wang proposed openstack/python-zaqarclient: Fix queue stats for v1.1 and above versions https://review.openstack.org/267923 | 02:12 |
*** kgriffs is now known as kgriffs|afk | 02:23 | |
*** amitgandhinz has quit IRC | 02:28 | |
openstackgerrit | Merged openstack/zaqar-ui: Fix installation instruction in README.rst https://review.openstack.org/272133 | 02:31 |
Eva-i | I have some questions about Zaqar that make me feel uneasy: https://etherpad.openstack.org/p/eva-asks | 02:40 |
Eva-i | flwang: if you'll have some free time, perhaps you could answer some of these questions ^^^ | 02:40 |
*** superflyy has joined #openstack-zaqar | 02:50 | |
*** pradk has quit IRC | 03:02 | |
*** achanda has joined #openstack-zaqar | 03:09 | |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Ensure JSON sent over websocket is a dictionary https://review.openstack.org/270748 | 03:16 |
*** pradk has joined #openstack-zaqar | 03:16 | |
*** david-lyle has joined #openstack-zaqar | 03:25 | |
openstackgerrit | Merged openstack/zaqar: Make enforce_type=True in CONF.set_override https://review.openstack.org/266333 | 03:27 |
Eva-i | flwang: 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/1534427 | 03:43 |
openstack | Launchpad 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 |
flwang | Eva-i: oh, thanks for the heads up | 03:45 |
flwang | i will upload another patch | 03:46 |
Eva-i | flwang: not patch set? Then another bug report should be opened | 03:46 |
flwang | Eva-i: no worries, patch set could work either | 03:47 |
flwang | we can use Partial-Bug tag | 03:47 |
Eva-i | flwang: oki, you decide | 03:48 |
flwang | :) | 03:49 |
Eva-i | flwang: 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 kgriffs | 03:55 | |
flwang | Eva-i: but it works for me | 03:55 |
flwang | Eva-i: i will wait for the comments from the horizon guy | 03:57 |
flwang | for now, you can try the comment from Shu Muto to see if it works for you | 03:57 |
Eva-i | flwang: yes =) | 03:57 |
*** superflyy has quit IRC | 04:01 | |
*** flwang1 has quit IRC | 04:04 | |
*** achanda has quit IRC | 04:23 | |
shu-mutou | Hi flwang, for blank panel issue, could you pull latest horizon and check again. | 04:23 |
*** achanda has joined #openstack-zaqar | 04:23 | |
openstackgerrit | ting wang proposed openstack/zaqar: Replace six.iteritems(dict) with dict.items() https://review.openstack.org/268410 | 04:31 |
*** achanda has quit IRC | 04:36 | |
*** pradk has quit IRC | 04:56 | |
Eva-i | flwang: thank you for today, I'm going to sleep =) | 05:00 |
*** pradk has joined #openstack-zaqar | 05:08 | |
openstackgerrit | Shu Muto proposed openstack/python-zaqarclient: [typo] queue list doesn't work https://review.openstack.org/272394 | 05:52 |
*** achanda has joined #openstack-zaqar | 06:28 | |
*** kgriffs is now known as kgriffs|afk | 06:37 | |
*** rcernin has joined #openstack-zaqar | 06:53 | |
*** GB21 has joined #openstack-zaqar | 07:05 | |
*** GB21 has quit IRC | 07:59 | |
openstackgerrit | ting wang proposed openstack/zaqar-ui: remove python 3.3 and 2.6 classifier https://review.openstack.org/272426 | 08:19 |
*** achanda has quit IRC | 08:23 | |
*** wanghao has joined #openstack-zaqar | 08:25 | |
*** GB21 has joined #openstack-zaqar | 08:27 | |
*** dynarro has joined #openstack-zaqar | 08:48 | |
openstackgerrit | ting wang proposed openstack/zaqar-ui: Trival: Remove 'MANIFEST.in' https://review.openstack.org/272431 | 08:57 |
*** achanda has joined #openstack-zaqar | 09:24 | |
*** achanda has quit IRC | 09:30 | |
*** dynarro has quit IRC | 10:04 | |
*** Qiming has quit IRC | 10:05 | |
*** achanda has joined #openstack-zaqar | 10:27 | |
*** achanda has quit IRC | 10:33 | |
*** eliqiao has quit IRC | 10:55 | |
*** Qiming has joined #openstack-zaqar | 10:56 | |
*** eliqiao has joined #openstack-zaqar | 10:57 | |
*** Qiming has quit IRC | 11:09 | |
*** Qiming has joined #openstack-zaqar | 11:09 | |
*** achanda has joined #openstack-zaqar | 11:30 | |
*** achanda has quit IRC | 11:35 | |
*** pradk has quit IRC | 12:52 | |
vkmc | flwang, hey hey, can we merge this one? https://review.openstack.org/#/c/271780/ | 13:30 |
*** achanda has joined #openstack-zaqar | 13:32 | |
*** achanda has quit IRC | 13:38 | |
ryansb | vkmc: done | 13:40 |
vkmc | thanks ryansb | 13:40 |
ryansb | ah, looks like flaper beat me to it | 13:40 |
ryansb | Mr. Quick-Draw over there | 13:41 |
vkmc | hahaha | 13:41 |
vkmc | flaper87 is a ninja reviewer | 13:42 |
ryansb | truth | 13:42 |
*** jtomasek has joined #openstack-zaqar | 13:52 | |
openstackgerrit | Merged openstack/python-zaqarclient: Fix typo preventing queue list for v1 https://review.openstack.org/271780 | 13:55 |
openstackgerrit | Ryan Brown proposed openstack/python-zaqarclient: Add CLI support for signed URLs https://review.openstack.org/253731 | 13:57 |
*** sriram has joined #openstack-zaqar | 14:04 | |
*** amitgandhinz has joined #openstack-zaqar | 14:09 | |
*** dynarro has joined #openstack-zaqar | 14:34 | |
*** pradk has joined #openstack-zaqar | 14:34 | |
*** boris-42 has quit IRC | 14:34 | |
*** boris-42 has joined #openstack-zaqar | 14:35 | |
*** Qiming has quit IRC | 14:53 | |
*** kgriffs|afk is now known as kgriffs | 14:54 | |
*** malini has joined #openstack-zaqar | 15:14 | |
openstackgerrit | Merged openstack/zaqar: Ensure JSON sent over websocket is a dictionary https://review.openstack.org/270748 | 15:33 |
*** achanda has joined #openstack-zaqar | 15:36 | |
*** achanda has quit IRC | 15:41 | |
*** pradk has left #openstack-zaqar | 15:43 | |
*** GB21 has quit IRC | 15:54 | |
*** GB21 has joined #openstack-zaqar | 16:01 | |
*** GB21 has quit IRC | 16:23 | |
*** mpanetta has joined #openstack-zaqar | 16:26 | |
*** achanda has joined #openstack-zaqar | 16:49 | |
*** ametts has joined #openstack-zaqar | 17:01 | |
*** jtomasek has quit IRC | 17:07 | |
*** jtomasek has joined #openstack-zaqar | 17:08 | |
*** dynarro has quit IRC | 17:17 | |
*** openstackgerrit has quit IRC | 17:17 | |
*** openstackgerrit has joined #openstack-zaqar | 17:17 | |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Support binary messages over websocket https://review.openstack.org/256978 | 17:29 |
*** EmilienM has quit IRC | 17:30 | |
*** EmilienM has joined #openstack-zaqar | 17:34 | |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Support binary messages over websocket https://review.openstack.org/256978 | 17:36 |
*** achanda has quit IRC | 17:40 | |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Fix typo in test_on_mesage_with_invalid_input https://review.openstack.org/272675 | 17:53 |
*** achanda has joined #openstack-zaqar | 18:10 | |
*** malini has quit IRC | 18:20 | |
*** achanda has quit IRC | 18:29 | |
*** malini has joined #openstack-zaqar | 18:47 | |
*** achanda has joined #openstack-zaqar | 18:48 | |
*** kgriffs is now known as kgriffs|afk | 18:49 | |
*** sriram has quit IRC | 19:03 | |
*** sriram has joined #openstack-zaqar | 19:03 | |
*** amitgandhinz has quit IRC | 19:04 | |
*** amitgandhinz has joined #openstack-zaqar | 19:05 | |
*** achanda has quit IRC | 19:21 | |
*** achanda has joined #openstack-zaqar | 19:23 | |
*** achanda has quit IRC | 19:25 | |
openstackgerrit | Merged openstack/zaqar: Replace six.iteritems(dict) with dict.items() https://review.openstack.org/268410 | 19:46 |
openstackgerrit | Merged openstack/zaqar: Fix typo in test_on_mesage_with_invalid_input https://review.openstack.org/272675 | 19:49 |
*** rcernin has quit IRC | 19:58 | |
*** pradk_ has joined #openstack-zaqar | 20:14 | |
*** pradk_ is now known as pradk | 20:14 | |
*** kgriffs|afk is now known as kgriffs | 20:18 | |
*** flwang1 has joined #openstack-zaqar | 20:20 | |
flwang | morning, guys | 20:21 |
*** rcernin has joined #openstack-zaqar | 20:32 | |
*** flwang has left #openstack-zaqar | 20:33 | |
*** flwang has joined #openstack-zaqar | 20:34 | |
openstackgerrit | Merged openstack/python-zaqarclient: Clean up flavor test classes https://review.openstack.org/271812 | 20:49 |
*** AJaeger has joined #openstack-zaqar | 20:50 | |
AJaeger | hi 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 anymore | 20:50 |
AJaeger | , please? | 20:50 |
vkmc | AJaeger, yup | 20:51 |
AJaeger | thanks | 20:51 |
*** malini has quit IRC | 20:52 | |
*** pradk has quit IRC | 20:53 | |
*** pradk has joined #openstack-zaqar | 20:54 | |
flwang | YEAH! just realized i have been removed from glance core | 20:54 |
flwang | now i can only focus on zaqar | 20:55 |
flwang | no applause ? | 20:56 |
vkmc | lol | 20:57 |
vkmc | \o/ | 20:57 |
flwang | vkmc: haha | 21:00 |
*** AJaeger has left #openstack-zaqar | 21:02 | |
ryansb | \o/ | 21:03 |
openstackgerrit | Merged openstack/zaqar: Remove argparse from requirements https://review.openstack.org/270401 | 21:09 |
Eva-i | flwang: yay | 21:09 |
*** boris-42 has quit IRC | 21:23 | |
*** achanda has joined #openstack-zaqar | 21:27 | |
*** rcernin has quit IRC | 21:28 | |
*** achanda has quit IRC | 21:30 | |
flwang | ryansb: did you see the discussion flaper87 and me about the summit pres? | 21:30 |
ryansb | no, when was it? Might be in my logs | 21:31 |
flwang | ryansb: flaper87 suggested we should focus on a special user case | 21:35 |
ryansb | special as in just one specific one? | 21:38 |
ryansb | e.g. people with worker farms, or something? | 21:38 |
flwang | ryansb: i think he is talking about a specific function, like pre-signed, or notification | 21:39 |
ryansb | gotcha | 21:40 |
flwang | ryansb: but i think we can demo most of our functions in a user case | 21:40 |
flwang | flaper87: ^ | 21:40 |
ryansb | I 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 |
flwang | ryansb: i'm on your side :D | 21:42 |
flwang | maybe i missed something | 21:42 |
flwang | for flaper87's suggestion | 21:42 |
flwang | ryansb: personally, i would like to work out a demo how zaqar can work with a worker farm(micro services cluster) | 21:44 |
flwang | and how the application/micro service can be benefited from zaqar | 21:44 |
ryansb | ++ | 21:44 |
flwang | classic image process sample https://aws.amazon.com/articles/PHP/1602 | 21:45 |
flwang | https://aws.amazon.com/search?searchQuery=SQS&searchPath=articles&x=0&y=0 | 21:46 |
flwang | we really need to have those kind of docs | 21:46 |
flwang | even demo/sample code | 21:46 |
ryansb | yeah, those would be really nice guides | 21:52 |
*** ametts has quit IRC | 21:55 | |
flwang | ryansb: you're the user of SQS/SNS, right? do you have any idea about this user case? | 21:59 |
flwang | can show the power of zaqar? | 21:59 |
ryansb | I don't know if there's specific use case that is zaqar-specific | 22:03 |
ryansb | the same SQS case (image resizer or whatever) will do fine with zaqar | 22:03 |
flwang | ryansb: yep | 22:03 |
ryansb | and workers can be given signed URLs to grab their jobs from a central server | 22:03 |
ryansb | so that's neat | 22:04 |
ryansb | I'll see if I can put together a little worker script that just pops messages and pretends to work a bit | 22:05 |
ryansb | (in python, of course) | 22:05 |
flwang | yep, python should be fine | 22:06 |
flwang | ryansb: oh, and sorry, i didn't send email | 22:07 |
ryansb | no worries :) | 22:07 |
flwang | yesterday about this topic | 22:07 |
ryansb | I have to run again - dinner time in my tz. I'll let you know when I have the demo worker | 22:08 |
flwang | ryansb: awesome, thanks a lot | 22:09 |
openstackgerrit | Merged openstack/zaqar: Add tutorial for writing storage pipeline stages https://review.openstack.org/268720 | 22:32 |
*** amitgandhinz has quit IRC | 22:50 | |
flwang | jasondotstar: ping re the puppet work | 22:52 |
*** sriram has quit IRC | 22:54 | |
*** boris-42 has joined #openstack-zaqar | 22:54 | |
*** mpanetta has quit IRC | 22:58 | |
*** achanda has joined #openstack-zaqar | 22:58 | |
*** kgriffs is now known as kgriffs|afk | 23:29 | |
*** Qiming has joined #openstack-zaqar | 23:50 | |
*** achanda has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!