*** shu-mutou-AFK is now known as shu-mutou | 00:25 | |
*** wxy has joined #openstack-zaqar | 01:19 | |
*** Kevin_Zheng has joined #openstack-zaqar | 01:43 | |
Kevin_Zheng | ping flwang1 | 01:43 |
---|---|---|
openstackgerrit | wangxiyuan proposed openstack/zaqar: Avoid multi-threading problem with sql backend https://review.openstack.org/324157 | 01:47 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Support Tempest test as gate job https://review.openstack.org/302479 | 01:48 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Add reserved metadata check https://review.openstack.org/305560 | 02:00 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Add reserved metadata check https://review.openstack.org/305560 | 02:05 |
Kevin_Zheng | I think we have some problem with api-ref | 02:10 |
Kevin_Zheng | http://docs-draft.openstack.org/37/325637/2/check/gate-zaqar-api-ref/5e3a699//api-ref/build/html/ | 02:10 |
Eva-i | Kevin_Zheng: hello | 02:11 |
Kevin_Zheng | for the current implementation, everything will be listed twice | 02:11 |
Eva-i | Kevin_Zheng: you're right, we must fix it :o | 02:12 |
Kevin_Zheng | I think the idea is to put things like messages, queues into v2 folder? | 02:12 |
Kevin_Zheng | yes, because we indexed also v2/index in the outer index | 02:13 |
* Eva-i checking... | 02:14 | |
*** tqtran has joined #openstack-zaqar | 02:16 | |
wxy | I guess flwang fix it in this patch: https://review.openstack.org/#/c/323059/ | 02:17 |
Kevin_Zheng | Hm...yes | 02:19 |
*** tqtran has quit IRC | 02:21 | |
openstackgerrit | wanghao proposed openstack/zaqar-specs: Subscription confirmation support https://review.openstack.org/284555 | 02:24 |
Eva-i | Kevin_Zheng: perhaps you can do the same thing in your patch. Or it would be better if I do a tiny patch solving this specific problem, so you can rebase? | 02:25 |
Eva-i | wxy: thanks for the link =) | 02:25 |
*** mpanetta has joined #openstack-zaqar | 03:01 | |
*** mpanetta has quit IRC | 03:06 | |
Kevin_Zheng | I think wait until the above patch merged is OK | 04:24 |
*** pcaruana has quit IRC | 05:13 | |
*** GB21 has joined #openstack-zaqar | 05:49 | |
*** openstackgerrit has quit IRC | 06:02 | |
*** openstackgerrit has joined #openstack-zaqar | 06:03 | |
*** tqtran has joined #openstack-zaqar | 06:18 | |
*** tqtran has quit IRC | 06:22 | |
*** GB21 has quit IRC | 06:25 | |
*** tesseract has joined #openstack-zaqar | 06:53 | |
*** pcaruana has joined #openstack-zaqar | 07:14 | |
*** tesseract has quit IRC | 07:32 | |
*** tesseract has joined #openstack-zaqar | 07:33 | |
*** tesseract has quit IRC | 07:46 | |
*** tesseract has joined #openstack-zaqar | 07:46 | |
openstackgerrit | Merged openstack/zaqar: Avoid multi-threading problem with sql backend https://review.openstack.org/324157 | 09:08 |
openstackgerrit | Merged openstack/zaqar: Add reserved metadata check https://review.openstack.org/305560 | 09:08 |
openstackgerrit | Merged openstack/zaqar: Fix fail of claim after some message expire(Redis) https://review.openstack.org/314273 | 09:18 |
openstackgerrit | Merged openstack/zaqar: Support Tempest test as gate job https://review.openstack.org/302479 | 09:59 |
*** shu-mutou is now known as shu-mutou-AFK | 10:02 | |
*** venkat has joined #openstack-zaqar | 11:12 | |
*** venkat has quit IRC | 11:31 | |
openstackgerrit | Flavio Percoco proposed openstack/zaqar: DO NOT MERGE THIS CHANGE https://review.openstack.org/325849 | 12:09 |
*** venkat has joined #openstack-zaqar | 12:20 | |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Adding Negative tests cases to Zaqar v2 https://review.openstack.org/319825 | 12:26 |
*** rcernin has joined #openstack-zaqar | 12:57 | |
*** rcernin has quit IRC | 12:59 | |
*** venkat has quit IRC | 13:06 | |
*** ametts has joined #openstack-zaqar | 13:57 | |
*** mpanetta has joined #openstack-zaqar | 14:04 | |
*** mpanetta has joined #openstack-zaqar | 14:04 | |
*** GB21 has joined #openstack-zaqar | 14:41 | |
*** wanghao has quit IRC | 14:59 | |
*** wanghao has joined #openstack-zaqar | 14:59 | |
*** tqtran has joined #openstack-zaqar | 16:21 | |
*** tqtran has quit IRC | 16:25 | |
jtomasek | Hi, I have a question. I am connecting to a zaqar queue with web client using websockets. After connecting to socket, I authenticate, create queue and subscribe to it. Then when there is more than 10 subscriptions, the messages suddenly stop coming to the client. I can see them by calling zaqar api, but client won't receive them. Anone seen this? | 16:30 |
vkmc | hi jtomasek | 16:31 |
vkmc | haven't seen that before | 16:31 |
vkmc | jtomasek, can you share some logs? | 16:31 |
*** tesseract has quit IRC | 16:33 | |
jtomasek | vkmc: which log should I look for errors in? | 16:33 |
jtomasek | vkmc: I'll have look at zaqar.log if there is anything | 16:33 |
*** pcaruana has quit IRC | 16:34 | |
vkmc | jtomasek, yeah the server log works for me | 16:34 |
vkmc | under /var/log/zaqar | 16:34 |
jtomasek | vkmc: only thing constantly repeating there is http://paste.openstack.org/show/508415/ | 16:37 |
jtomasek | vkmc: nothing else there | 16:37 |
*** Ravikiran_K has joined #openstack-zaqar | 16:47 | |
vkmc | jtomasek, I haven't seen this before | 16:54 |
vkmc | I'll try to reproduce later today | 16:54 |
vkmc | flwang, ^ | 16:54 |
*** GB21 has quit IRC | 17:03 | |
jtomasek | vkmc: thanks | 17:07 |
*** tqtran has joined #openstack-zaqar | 17:43 | |
*** tqtran has quit IRC | 17:52 | |
*** pcaruana has joined #openstack-zaqar | 18:29 | |
*** jtomasek has quit IRC | 18:39 | |
*** mpanetta has quit IRC | 19:15 | |
*** mpanetta has joined #openstack-zaqar | 19:16 | |
*** mpanetta has quit IRC | 19:16 | |
*** mpanetta has joined #openstack-zaqar | 19:17 | |
*** tqtran has joined #openstack-zaqar | 19:23 | |
therve | vkmc, https://github.com/openstack/zaqar/blob/master/zaqar/notification/notifier.py#L36 | 19:28 |
therve | That may be problematic | 19:28 |
vkmc | ouch | 19:29 |
vkmc | well spotted | 19:29 |
*** Ravikiran_K has quit IRC | 19:30 | |
therve | I mean it shouldn't be an issue, but the number 10 doesn't sound like a coincidence :) | 19:31 |
*** mpanetta_ has joined #openstack-zaqar | 19:46 | |
*** mpanetta has quit IRC | 19:49 | |
*** mpanetta_ has quit IRC | 19:50 | |
*** mpanetta has joined #openstack-zaqar | 19:50 | |
*** openstackstatus has quit IRC | 19:55 | |
*** openstackstatus has joined #openstack-zaqar | 19:58 | |
*** ChanServ sets mode: +v openstackstatus | 19:58 | |
*** flwang1 has quit IRC | 20:13 | |
*** pt_15 has joined #openstack-zaqar | 20:21 | |
*** itisha has joined #openstack-zaqar | 20:36 | |
*** mpanetta_ has joined #openstack-zaqar | 20:56 | |
*** mpanett__ has joined #openstack-zaqar | 20:58 | |
*** mpanetta has quit IRC | 21:00 | |
*** mpanetta_ has quit IRC | 21:01 | |
*** mpanett__ is now known as mpanetta | 21:01 | |
*** pcaruana has quit IRC | 21:08 | |
flwang | Eva-i: ping | 21:20 |
flwang | vkmc: got, i will take a look today | 21:22 |
*** flwang1 has joined #openstack-zaqar | 21:23 | |
Eva-i | Pong | 21:35 |
Eva-i | flwang: pong | 21:36 |
flwang | Eva-i: sorry, my home's network doesn't work, so i missed the meeting | 21:37 |
flwang | did you guys have a meeting? | 21:37 |
flwang | i assume no? | 21:37 |
vkmc | no flwang | 21:38 |
flwang | vkmc: ok, we can have a quick catch-up in zaqar channel if you want | 21:39 |
Eva-i | flwang: okay, let's do catch-up | 21:39 |
flwang | 1. we have merged the tempest gate job, yeah! | 21:40 |
flwang | and i'm going to make it as voting and meanwhile make the functional test of client as non-voting, any concern? | 21:40 |
flwang | no comments? | 21:43 |
flwang | ok, then i'm going to propose a change in project-config to change the voting | 21:44 |
Eva-i | no concern | 21:46 |
Eva-i | flwang: thank you | 21:46 |
*** tqtran has quit IRC | 21:47 | |
Eva-i | flwang: also Fei Long. I feel so releived you took api-ref task. | 21:47 |
Eva-i | flwang: you are great | 21:47 |
flwang | Eva-i: ah, yep, thanks for the reminding | 21:53 |
flwang | we have also merged the init commit for api ref | 21:53 |
flwang | so the next work, just adding each api endpoints into the framework | 21:54 |
Eva-i | flwang: as you may know, Kevin_Zheng is working on "messages" api-ref part. | 21:54 |
flwang | and we just release newton miletone 1 :) | 21:54 |
flwang | Eva-i: oh, yes | 21:55 |
Eva-i | flwang: he wants our "queues" part to be merged | 21:55 |
Eva-i | flwang: merged first | 21:55 |
flwang | here is the assignment https://etherpad.openstack.org/p/newton-zaqar-assignment | 21:55 |
*** mpanetta has quit IRC | 21:55 | |
flwang | Eva-i: oh, yes, i will upstream a new patchset today and hopefully, we can get it in quickly | 21:56 |
Eva-i | flwang: oki =) | 21:56 |
flwang | but IMHO, even there is a conflict, it's easy to fix | 21:56 |
Eva-i | yes, maybe | 21:56 |
flwang | yes for sure, since it's just an independent .inc file | 21:57 |
* Eva-i will propose today lazy queues in subscriptions patch | 21:57 | |
flwang | ok, no updates from me :) | 21:57 |
flwang | Eva-i: you turn | 21:57 |
Eva-i | flwang: nothing else from me :o | 21:57 |
*** mpanetta has joined #openstack-zaqar | 22:00 | |
flwang | Eva-i: what's the status of install guide? do you need any help? | 22:00 |
Eva-i | flwang: No, I don't need any help. Status is the same as at the start of previous week. | 22:01 |
Eva-i | flwang: I haven't worked on it past week | 22:01 |
flwang | Eva-i: so generally, we will keep the install guide in our tree,right? | 22:03 |
Eva-i | flwang: yes | 22:03 |
flwang | Eva-i: and like api-ref, there will be a tox job and a publish job? | 22:03 |
Eva-i | flwang: I think so | 22:04 |
*** mpanetta has quit IRC | 22:05 | |
flwang | Eva-i: ok, sounds good | 22:05 |
flwang | Eva-i: thanks for working on that. | 22:06 |
flwang | Eva-i: and i'm looking for the lazy queue in subscription | 22:06 |
Eva-i | flwang: ;) | 22:06 |
flwang | and the message id in notification | 22:06 |
flwang | i haven't heard any status about that ;) | 22:06 |
Eva-i | flwang: I still haven't implemented the second way of putting "message id" in notification. | 22:07 |
Eva-i | flwang: will do it tomorrow | 22:07 |
flwang | Eva-i: cool | 22:08 |
flwang | vkmc: any updates? | 22:08 |
vkmc | anything for me, sorry | 22:09 |
flwang | vkmc: ok, no problem | 22:10 |
flwang | Eva-i: what will your focus in next week? | 22:14 |
Eva-i | flwang: on next week I don't know. On this week on install guide. | 22:15 |
* flwang is going to get the queues api-ref in and start to investigate the mistral support in notification and review the subscription confirmation to merge it asap | 22:16 | |
flwang | i mean this week, then :) | 22:16 |
flwang | Eva-i: so you just mentioned you will submit a new patch set for lazy queue and will work on install guide | 22:20 |
Eva-i | flwang: rite | 22:20 |
flwang | and if you have time, write a CV :D | 22:20 |
Eva-i | flwang: I'm not sure if I will have time this week, I will host a foreign guest. I think before writing CV and applying for a job I should read some book about advanced python programming. I think I also need to understand networks more. | 22:24 |
Eva-i | flwang: *time for CV | 22:24 |
flwang | Eva-i: no problem, my point is you need to reserve some time for your personal stuff and | 22:25 |
*** ametts has quit IRC | 22:25 | |
flwang | again, let me know if you need any help | 22:25 |
Eva-i | flwang: oki | 22:25 |
flwang | like a recommendation letter if that helps | 22:25 |
Eva-i | flwang: yes | 22:25 |
flwang | Eva-i: can you rebase your 'age' patches so that we can see the tempest result? | 22:29 |
Eva-i | flwang: ah, yes, sure | 22:29 |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Let v2 tempest tests base on the base tests. https://review.openstack.org/324161 | 22:30 |
flwang | Eva-i: and can you help review this https://review.openstack.org/#/c/321350/ ? | 22:30 |
flwang | vkmc: ^ | 22:31 |
flwang | and https://review.openstack.org/#/c/322693/ | 22:31 |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Show 'age' field in subscriptions (mongodb) https://review.openstack.org/288907 | 22:32 |
flwang | and https://review.openstack.org/#/c/308443/ | 22:32 |
flwang | those patches have got my bless, need another care from you guys :) | 22:32 |
flwang | and feel free -1, hah | 22:33 |
Eva-i | flwang: yes | 22:37 |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Show 'age' field in subscriptions (redis) https://review.openstack.org/288908 | 22:40 |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Updated from global requirements https://review.openstack.org/322589 | 22:42 |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Disable DEPRECATED API versions by default https://review.openstack.org/294611 | 22:43 |
openstackgerrit | Merged openstack/zaqar: Define context.roles with base class https://review.openstack.org/302370 | 22:49 |
flwang | Eva-i: just FYI, i have figure out why our release notes are empty | 23:12 |
Eva-i | flwang: yes, why? | 23:12 |
flwang | since i'm failing to require the releasenotes change for important patches | 23:13 |
flwang | Eva-i: in the future, we need to require the release note change for each important fix | 23:15 |
flwang | ('features', 'New Features'), ('issues', 'Known Issues'), ('upgrade', 'Upgrade Notes'), ('deprecations', 'Deprecation Notes'), ('critical', 'Critical Issues'), ('security', 'Security Issues'), ('fixes', 'Bug Fixes'), | 23:15 |
Eva-i | Oh, oki | 23:15 |
flwang | and i noticed the other projects' release notes also have problems, i will talk with release team to figure out the issue | 23:17 |
Eva-i | flwang: it would be nice of you | 23:17 |
*** tqtran has joined #openstack-zaqar | 23:39 | |
openstackgerrit | Merged openstack/zaqar: Rename notification/task to notification/tasks https://review.openstack.org/321350 | 23:40 |
Eva-i | flwang: should we close https://bugs.launchpad.net/zaqar/+bug/1553834 ? | 23:41 |
openstack | Eva-i: Error: malone bug 1553834 not found | 23:41 |
flwang | Eva-i: have you verified that? | 23:45 |
Eva-i | flwang: yes, see my last comment there. I'm not sure 100% though. | 23:48 |
flwang | Eva-i: ok, i see. | 23:50 |
flwang | i will close it then | 23:50 |
Eva-i | flwang: I used to believe there might be a more elegant way to solve the problem than in the https://review.openstack.org/#/c/317134/ patch, like stopping using global conf. But it's better than nothing. And it seems to work. | 23:50 |
Eva-i | flwang: now we have fixed all our gate issues. =) I hope we will forget "recheck" command for some time. | 23:51 |
Eva-i | flwang: oh, rally one is still not fixed | 23:51 |
flwang | Eva-i: yep, we still have a failed rally :D | 23:53 |
*** shu-mutou-AFK is now known as shu-mutou | 23:55 | |
shu-mutou | flwang: Hi | 23:57 |
flwang | shu-mutou: hi :) | 23:57 |
shu-mutou | flwang: could you review this? https://review.openstack.org/#/c/323753/ | 23:57 |
flwang | shu-mutou: done :) | 23:58 |
shu-mutou | flwang: thanks (/・ω・)/ | 23:59 |
flwang | shu-mutou: thank you for all your work :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!