Thursday, 2016-02-25

*** amitgandhinz has joined #openstack-zaqar00:35
*** amitgandhinz has quit IRC00:48
*** amitgandhinz has joined #openstack-zaqar01:45
*** amitgandhinz has quit IRC01:58
*** malini has joined #openstack-zaqar02:10
*** malini has left #openstack-zaqar02:11
*** csoukup has joined #openstack-zaqar02:12
*** csoukup has quit IRC02:16
Eva-iflwang: I left super review to your patch: https://review.openstack.org/#/c/265723/. Reviews like this usually take few hours for me. =)02:51
*** amitgandhinz has joined #openstack-zaqar02:55
flwangEva-i: so the -1 is mainly because this comment https://review.openstack.org/#/c/265723/5/zaqar/transport/wsgi/v2_0/messages.py right?02:59
Eva-iflwang: yes03:00
flwangEva-i: cool, it's a really nice catch03:01
flwangEva-i: thank you03:01
Eva-iflwang: no problem =)03:01
*** boris-42 has quit IRC03:04
Eva-iI'm a very strict reviewer according to stackalytics.com03:04
flwangEva-i: haha, it's good, i like strict reviewer always if i'm not rush :D03:05
*** amitgandhinz has quit IRC03:08
openstackgerritwangxiyuan proposed openstack/zaqar: Add tempest test for v1.1  https://review.openstack.org/28390603:11
openstackgerritwangxiyuan proposed openstack/zaqar: Fix the tempest test error  https://review.openstack.org/28346603:11
openstackgerritwangxiyuan proposed openstack/zaqar: Add tempest test for wsgi v2  https://review.openstack.org/28401603:11
Eva-iflwang: here are few bugs in zaqar-ui http://paste.openstack.org/show/VpAGJbI3IlAbK5ogqwwe/03:15
Eva-igood bye03:15
*** Eva-i has quit IRC03:16
openstackgerritwangxiyuan proposed openstack/zaqar: Add tempest test for wsgi v2  https://review.openstack.org/28401604:03
*** amitgandhinz has joined #openstack-zaqar04:05
*** amitgandhinz has quit IRC04:18
*** achanda has quit IRC04:35
*** achanda has joined #openstack-zaqar04:55
*** amitgandhinz has joined #openstack-zaqar05:14
*** amitgandhinz has quit IRC05:28
*** amitgandhinz has joined #openstack-zaqar06:24
*** amitgandhinz has quit IRC06:37
openstackgerritwanghao proposed openstack/zaqar-specs: Confirm subscriber befor send messages  https://review.openstack.org/28455506:44
*** achanda_ has joined #openstack-zaqar06:45
*** achanda has quit IRC06:46
openstackgerritwanghao proposed openstack/zaqar-specs: Confirm subscriber befor send messages  https://review.openstack.org/28455506:57
openstackgerritwanghao proposed openstack/zaqar-specs: Confirm subscriber befor send messages  https://review.openstack.org/28455507:04
openstackgerritwanghao proposed openstack/zaqar-specs: Confirm subscriber befor send messages  https://review.openstack.org/28455507:13
*** amitgandhinz has joined #openstack-zaqar07:34
*** jtomasek has joined #openstack-zaqar07:46
*** amitgandhinz has quit IRC07:48
*** dynarro has joined #openstack-zaqar08:36
*** amitgandhinz has joined #openstack-zaqar08:44
*** boris-42 has joined #openstack-zaqar08:52
*** amitgandhinz has quit IRC08:58
*** achanda_ has quit IRC09:00
*** achanda has joined #openstack-zaqar09:04
*** achanda has quit IRC09:19
*** achanda has joined #openstack-zaqar09:24
*** achanda has quit IRC09:29
openstackgerritwanghao proposed openstack/zaqar-specs: Confirm subscriber befor send messages  https://review.openstack.org/28455509:40
*** amitgandhinz has joined #openstack-zaqar09:54
*** amitgandhinz has quit IRC10:08
*** Eva-i has joined #openstack-zaqar10:23
*** shu-mutou is now known as shu-mutou-AWAY10:29
*** achanda has joined #openstack-zaqar10:29
*** achanda has quit IRC10:35
*** pcaruana has joined #openstack-zaqar10:42
*** flwang1 has joined #openstack-zaqar10:56
*** dynarro has quit IRC10:58
*** amitgandhinz has joined #openstack-zaqar11:04
*** amitgandhinz has quit IRC11:17
*** flwang1 has quit IRC11:20
*** flwang1 has joined #openstack-zaqar11:31
*** wanghao has quit IRC11:33
*** wanghao has joined #openstack-zaqar11:35
*** mdnadeem has quit IRC11:50
*** mdnadeem has joined #openstack-zaqar11:51
*** amitgandhinz has joined #openstack-zaqar12:14
*** amitgandhinz has quit IRC12:27
*** achanda has joined #openstack-zaqar12:33
*** dynarro has joined #openstack-zaqar12:36
*** achanda has quit IRC12:40
*** pcaruana has quit IRC12:55
*** pcaruana has joined #openstack-zaqar13:09
*** AAzza_ has joined #openstack-zaqar13:09
*** AAzza has quit IRC13:10
*** AAzza has joined #openstack-zaqar13:13
*** AAzza_ has quit IRC13:14
*** amitgandhinz has joined #openstack-zaqar13:24
*** achanda has joined #openstack-zaqar13:37
*** amitgandhinz has quit IRC13:37
*** amitgandhinz has joined #openstack-zaqar13:39
*** amitgandhinz has quit IRC13:39
*** amitgandhinz has joined #openstack-zaqar13:40
*** achanda has quit IRC13:41
*** sriram has joined #openstack-zaqar14:08
*** sriram has quit IRC14:09
*** sriram has joined #openstack-zaqar14:09
*** amitgandhinz has quit IRC15:26
*** csoukup has joined #openstack-zaqar15:27
*** amitgandhinz has joined #openstack-zaqar15:28
*** achanda has joined #openstack-zaqar15:39
*** achanda has quit IRC15:45
*** ksheedlo has quit IRC15:48
*** ksheedlo-rax has joined #openstack-zaqar15:49
Eva-iHello. What does this paragraph means: https://wiki.openstack.org/wiki/Zaqar/specs/api/v1.1#Resource_Media_Types? I can send request to Zaqar with body packed in msgpack and Zaqar will respond with body in msgpack?16:09
Eva-i*mean16:10
vkmcEva-i, that is outdated16:14
vkmcthere was an ongoing effort to support msgpack16:14
vkmcbut that didn't happen16:14
Eva-ivkmc: thanks! As I thought. This will help me to write a patch.16:15
vkmcyes :)16:15
*** malini1 has joined #openstack-zaqar17:07
*** dynarro has quit IRC17:34
*** achanda has joined #openstack-zaqar17:36
*** sebasmagri has joined #openstack-zaqar17:37
*** wanghao has quit IRC18:17
*** wanghao has joined #openstack-zaqar18:17
*** jtomasek has quit IRC18:49
Eva-iryansb: wow, thanks for reviews18:51
* ryansb does review festival18:52
Eva-iryansb: you love festivals18:52
ryansbYes - it helps stay positive about my review backlog18:53
ryansbunfun: time to do all my reviews in the backlog18:53
ryansbfun: review festival time!18:53
Eva-i=)18:53
*** sebasmagri has quit IRC19:05
*** flwang1 has quit IRC19:15
openstackgerritMerged openstack/python-zaqarclient: Fix client not working on Python 3  https://review.openstack.org/27189019:22
openstackgerritEva Balycheva proposed openstack/zaqar: (WIP) Fix freeze on some requests without content-type  https://review.openstack.org/28489719:34
openstackgerritEva Balycheva proposed openstack/zaqar: (WIP) Fix freeze on some requests without content-type  https://review.openstack.org/28489719:58
openstackgerritEva Balycheva proposed openstack/zaqar: (WIP) Fix freeze on some requests without content-type  https://review.openstack.org/28489720:06
openstackgerritDavid Cusatis proposed openstack/zaqar-ui: WIP: add subscriptions panel  https://review.openstack.org/27725720:29
openstackgerritDavid Cusatis proposed openstack/zaqar-ui: add pycharm files to gitignore  https://review.openstack.org/28491420:34
*** flwang1 has joined #openstack-zaqar20:35
flwangryansb: i haven't worked out a client patch for queue metadata update20:36
flwangif you want to take it would be awesome20:37
ryansbgotcha20:37
openstackgerritDavid Cusatis proposed openstack/zaqar-ui: WIP: add subscriptions panel  https://review.openstack.org/28492120:42
*** malini has joined #openstack-zaqar20:55
*** malini1 has quit IRC20:56
flwangryansb: vkmc: flaper87: i have a little bit concern about this patch https://review.openstack.org/28094121:00
flwangi'm hesitate to change the v1.1 api21:00
flwangthoughts?21:00
ryansbIt's an additive change, I think, since the new behavior lives in the PATCH action21:00
flwangryansb: yep, i can see your point21:18
ryansbI'm ambivalent, personally - I'd be fine with a v2-only change, but also adding to 1.1 doesn't bug me21:19
flwangryansb: cool, thanks for the feedback21:19
flwangryansb: vkmc: flaper87: Eva-i: we're asked the summit session slots21:20
flwangat Tokyo summit, we have 1 3 021:20
flwangFishbowl/Work rooms/Meetup21:20
flwangdo you think we need more slots?21:21
vkmcflwang, how many topics you consider we have in our prio list?21:21
flwangvkmc: good question21:21
vkmcthe refactoring eva suggested could be targeted for a work room21:21
flwang1. subscription confirm21:21
vkmcI like work rooms better than fishbowls21:21
flwang2. zaqar UI21:22
flwang3. redis support21:23
*** achanda has quit IRC21:23
flwang4. delayed queue21:23
flwang5. dead letter queue21:24
flwang6. subscription enhancement21:24
flwang7. pool group and flavor21:24
flwangthose the topics in my mind now21:25
flwangryansb: vkmc: flaper87: Eva-i: anything else?21:27
vkmchm I don't think so, it sounds we are covered21:27
flwangwe can definitely combine topics in 1 slot21:29
*** malini has quit IRC21:29
flwangso basically, we need at least 4, basically same slots number as Tokyo21:29
flwangbut i would like to apply for a meet up slot21:29
flwangsince we have some new contributors21:29
*** achanda has joined #openstack-zaqar21:31
*** achanda has quit IRC21:33
*** achanda has joined #openstack-zaqar21:37
flwangcomments?21:39
vkmcwe can split a few of those topics21:43
vkmcas we did with last summit21:43
vkmcand... it would be nice to apply for a meet up slot yes21:43
vkmc:D21:43
flwangok, so 1 3 1, sounds good?21:46
vkmcyeah21:47
vkmcit does21:48
flwangvkmc: ryansb: flaper87: thank you21:49
*** sriram has quit IRC22:00
flwangEva-i: would you mind revisiting https://review.openstack.org/270464 ?22:06
Eva-iflwang: I'll think later about topics for session. And will google what is dead letter queue.22:07
Eva-iflwang: do we need a comment about permanent subscriptions here https://review.openstack.org/#/c/270464/9/zaqar/storage/mongodb/subscriptions.py?22:09
Eva-iflwang: so you decided to not implement permanent subscriptions?22:09
*** achanda has quit IRC22:09
flwangEva-i: oh, i will delete the comment line22:09
flwangEva-i: yep, now we don't have a permanent sub :)22:10
Eva-iflwang: oki, that was the only concern I had.22:11
openstackgerritFei Long Wang proposed openstack/zaqar: Fix the TTL issue of subscriptions for MongoDB  https://review.openstack.org/27046422:11
flwangEva-i: cool22:11
flwangFeb 29-Mar 4 is m-322:12
flwangwe need to get those patches done asap22:12
Eva-iOkay22:12
*** mpanetta has joined #openstack-zaqar22:12
*** mpanetta has quit IRC22:12
*** mpanetta has joined #openstack-zaqar22:13
Eva-iflwang: about this patch: https://review.openstack.org/280941. If API v2 was released long time ago and we still will change it, changing it will not differ much from changing our older APIs. And because of it I would personally prefer making addittive changes also to our older APIs for consistency. The only problem from additive changes is this: the client will get problems if he bind his code to API v1.1 of newer version of Zaqar, but then f22:25
flwangbut then f?22:33
Eva-iflwang: what?22:38
*** malini has joined #openstack-zaqar22:41
flwangEva-i: can you reword your above msg?22:43
openstackgerritMerged openstack/zaqar: Fix arguments order in error_response (Websocket)  https://review.openstack.org/28377322:43
*** malini has quit IRC22:44
openstackgerritMerged openstack/zaqar: Catch NoPoolFound on resource listing in transport  https://review.openstack.org/28318422:45
Eva-iflwang: sure. I think it's okay to change also API v1.1 in this case.22:50
flwangEva-i: cool, thank you22:53
flwangEva-i: can you rebase https://review.openstack.org/#/c/256978/ ?22:56
flwangi'm going to review it today22:56
flwanghopefully we can merge it next monday22:57
Eva-iflwang: yes22:57
openstackgerritEva Balycheva proposed openstack/zaqar: Support binary messages over websocket  https://review.openstack.org/25697822:59
Eva-iflwang: thank you22:59
*** achanda has joined #openstack-zaqar23:04
*** csoukup has quit IRC23:09
*** achanda has quit IRC23:38
*** amitgandhinz has quit IRC23:44
*** achanda has joined #openstack-zaqar23:49
*** mpanetta has quit IRC23:51

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