| *** shu-mutou-AFK is now known as shu-mutou | 00:05 | |
| Eva-i | flwang: many patches I made recently didn't required much patch sets and reviewing, so maybe they're good. Maybe this one will be good too. I don't know. | 00:07 |
|---|---|---|
| Eva-i | flwang: if I understand correctly, when pooling is enabled in Zaqar, the configuration options section [drivers:message_store:mongodb] isn't used, right? | 00:14 |
| *** tqtran has quit IRC | 00:45 | |
| Eva-i | ryansb: hello. Maybe you know ^? | 00:49 |
| flwang | Eva-i: yes | 00:51 |
| flwang | Eva-i: if the pooling is enabled, then messages will go into pools | 00:51 |
| Eva-i | flwang: oki, thank you | 00:52 |
| flwang | Eva-i: can you give this a quick review https://review.openstack.org/#/c/317116/ ? | 00:52 |
| flwang | it fixes the rally job | 00:52 |
| Eva-i | flwang: no, I don't like this fix | 00:53 |
| flwang | Eva-i: why? | 00:53 |
| Eva-i | flwang: it doesn't solve the real cause. The real problem will still exist in keystone client after merge of this patch. What if real users will have such load on zaqar and it will fail their expectations? I find it good that this test loads zaqar enough for it fail. As long as it fails, there's a reminder that something has to be fixed in keystone client. | 00:59 |
| flwang | Eva-i: you're point does make sense, but that also means if we leave it as failed, we also can't test any performance of zaqar, see my point? | 01:02 |
| flwang | since you won't open it the link anymore | 01:02 |
| Eva-i | flwang: I don't find rally job critical, because it only tests queues now. I find the problem in keystone/rally/apache/devstack critical, and we better not close our eyes on it. | 01:09 |
| Eva-i | flwang: my bug report should be updated, it's invalid now: https://bugs.launchpad.net/python-keystoneclient/+bug/1544839. I don't have time to find where's the real cause of such behavior. | 01:10 |
| openstack | Launchpad bug 1544839 in python-keystoneclient "Job gate-rally-dsvm-zaqar-zaqar fails since the recent Rally patch" [Undecided,Incomplete] | 01:10 |
| Eva-i | Maybe it's not in keystone client, as commenters say | 01:12 |
| flwang | Eva-i: so it's a apache config issue of devstack? | 01:14 |
| Eva-i | flwang: maybe | 01:14 |
| Eva-i | flwang: one of the commenters says this | 01:15 |
| Eva-i | flwang: but someone else has to check | 01:15 |
| -openstackstatus- NOTICE: Gerrit is about to be restarted to help with page timeouts | 01:47 | |
| flwang | Eva-i: ok, i will leave it a while | 01:50 |
| *** mpanetta_ has joined #openstack-zaqar | 01:57 | |
| *** mpanetta_ has quit IRC | 02:01 | |
| openstackgerrit | Eva Balycheva proposed openstack/zaqar: DNM: Zaqar production installation guide - old https://review.openstack.org/268851 | 02:11 |
| openstackgerrit | Eva Balycheva proposed openstack/zaqar: Add Zaqar production installation guide https://review.openstack.org/268851 | 02:13 |
| *** lei-zh has joined #openstack-zaqar | 02:16 | |
| Eva-i | flwang: finally uploaded install guide. =) It is up-to-date and working, Though doc team will probably find many style problems if I try to upload it to the official place. | 02:25 |
| flwang | Eva-i: we may keep it in our tree based on the new way | 02:30 |
| Eva-i | flwang: install guides will be hosted in project repos, right? | 02:32 |
| flwang | i think so | 02:34 |
| *** mdnadeem has joined #openstack-zaqar | 02:51 | |
| *** njohnston-- is now known as njohnston__ | 03:54 | |
| *** njohnston__ has quit IRC | 03:56 | |
| *** njohnsto_ has joined #openstack-zaqar | 03:57 | |
| *** njohnsto_ is now known as njohnston__ | 03:57 | |
| *** flwang1 has quit IRC | 04:58 | |
| *** njohnston__ has quit IRC | 05:01 | |
| *** njohnsto_ has joined #openstack-zaqar | 05:08 | |
| *** njohnsto_ has quit IRC | 05:17 | |
| *** rcernin has joined #openstack-zaqar | 05:47 | |
| *** david-lyle has quit IRC | 06:05 | |
| *** njohnsto_ has joined #openstack-zaqar | 06:14 | |
| *** njohnsto_ has quit IRC | 06:19 | |
| *** jtomasek has joined #openstack-zaqar | 06:45 | |
| *** rcernin has quit IRC | 06:47 | |
| *** prashanthraghu has joined #openstack-zaqar | 06:47 | |
| *** rcernin has joined #openstack-zaqar | 06:51 | |
| *** dynarro has joined #openstack-zaqar | 07:18 | |
| *** AAzza has quit IRC | 07:35 | |
| *** david-lyle has joined #openstack-zaqar | 07:57 | |
| *** mpanetta_ has joined #openstack-zaqar | 07:59 | |
| *** mpanetta_ has quit IRC | 08:04 | |
| *** AAzza has joined #openstack-zaqar | 08:10 | |
| *** njohnsto_ has joined #openstack-zaqar | 08:16 | |
| *** njohnsto_ has quit IRC | 08:20 | |
| *** dynarro has quit IRC | 08:32 | |
| *** prashanthraghu has quit IRC | 08:38 | |
| *** vipuls has joined #openstack-zaqar | 08:42 | |
| *** Kevin_Zheng_ has joined #openstack-zaqar | 08:42 | |
| *** kgriffs_alt has joined #openstack-zaqar | 08:45 | |
| *** ameade has quit IRC | 08:45 | |
| *** vipul- has quit IRC | 08:45 | |
| *** kgriffs has quit IRC | 08:45 | |
| *** Kevin_Zheng has quit IRC | 08:45 | |
| *** odyssey4me has quit IRC | 08:45 | |
| *** david_cu has quit IRC | 08:45 | |
| *** vkmc has quit IRC | 08:45 | |
| *** vkmc has joined #openstack-zaqar | 08:45 | |
| *** vkmc has quit IRC | 08:45 | |
| *** vkmc has joined #openstack-zaqar | 08:45 | |
| *** kgriffs_alt is now known as kgriffs | 08:46 | |
| *** Kevin_Zheng_ is now known as Kevin_Zheng | 08:46 | |
| *** odyssey4me has joined #openstack-zaqar | 08:46 | |
| *** kong has quit IRC | 08:48 | |
| *** wxy has quit IRC | 08:49 | |
| *** ameade has joined #openstack-zaqar | 08:55 | |
| *** kong has joined #openstack-zaqar | 08:56 | |
| *** wxy has joined #openstack-zaqar | 09:03 | |
| openstackgerrit | wangxiyuan proposed openstack/zaqar: Make the notifier max_workers configurable https://review.openstack.org/308443 | 09:10 |
| *** prashanthraghu has joined #openstack-zaqar | 09:15 | |
| *** Kevin_Zheng has quit IRC | 09:16 | |
| *** Kevin_Zheng has joined #openstack-zaqar | 09:21 | |
| *** shu-mutou is now known as shu-mutou-AFK | 10:17 | |
| *** lei-zh has quit IRC | 10:21 | |
| *** AAzza has quit IRC | 10:50 | |
| *** AAzza has joined #openstack-zaqar | 10:50 | |
| *** GB21 has joined #openstack-zaqar | 11:02 | |
| *** Kevin_Zheng has quit IRC | 11:11 | |
| *** njohnsto_ has joined #openstack-zaqar | 11:13 | |
| *** njohnsto_ has quit IRC | 11:18 | |
| *** njohnsto_ has joined #openstack-zaqar | 11:27 | |
| *** GB21 has quit IRC | 11:32 | |
| *** GB21 has joined #openstack-zaqar | 11:33 | |
| *** prashanthraghu has quit IRC | 11:49 | |
| *** mpanetta_ has joined #openstack-zaqar | 12:00 | |
| *** mpanetta_ has quit IRC | 12:05 | |
| *** prashanthraghu has joined #openstack-zaqar | 12:18 | |
| *** GB21 has quit IRC | 12:19 | |
| *** njohnsto_ has quit IRC | 12:29 | |
| *** Kevin_Zheng has joined #openstack-zaqar | 12:46 | |
| *** njohnsto- has joined #openstack-zaqar | 13:05 | |
| *** njohnsto- is now known as njohnston-- | 13:06 | |
| *** dynarro has joined #openstack-zaqar | 13:30 | |
| *** ametts has joined #openstack-zaqar | 13:49 | |
| *** prashanthraghu has quit IRC | 13:50 | |
| *** njohnston-- is now known as njohnston__ | 14:00 | |
| *** mpanetta_ has joined #openstack-zaqar | 14:01 | |
| *** mpanetta_ has quit IRC | 14:01 | |
| *** mpanetta_ has joined #openstack-zaqar | 14:02 | |
| *** dynarro has quit IRC | 14:26 | |
| *** Kevin_Zheng has quit IRC | 15:01 | |
| *** rcernin has quit IRC | 15:24 | |
| *** njohnston__ is now known as njohnston | 15:25 | |
| *** itisha has joined #openstack-zaqar | 15:26 | |
| *** njohnston has quit IRC | 16:37 | |
| Eva-i | flwang: Hello. Please take my "lazy queues in subscriptions" task, if you think this task needs to be done in N-1. Otherwise I'd like to make it myself in N-2. | 16:39 |
| *** AAzza has quit IRC | 17:41 | |
| *** tqtran has joined #openstack-zaqar | 17:43 | |
| *** AAzza has joined #openstack-zaqar | 17:43 | |
| *** flwang1 has joined #openstack-zaqar | 18:54 | |
| *** pt_15 has joined #openstack-zaqar | 19:03 | |
| *** flwang1 has quit IRC | 19:18 | |
| *** flwang1 has joined #openstack-zaqar | 19:26 | |
| *** njohnston has joined #openstack-zaqar | 19:27 | |
| *** mpanetta_ has quit IRC | 19:46 | |
| openstackgerrit | Merged openstack/python-zaqarclient: Updated from global requirements https://review.openstack.org/303164 | 19:53 |
| *** flwang1 has quit IRC | 19:58 | |
| *** ametts has quit IRC | 20:13 | |
| *** ametts has joined #openstack-zaqar | 20:30 | |
| *** ametts has quit IRC | 21:07 | |
| EmilienM | flwang: I found new bugs in Zaqar | 21:29 |
| EmilienM | a patch in zaqar between 1 week ago and today broke our CI | 21:29 |
| EmilienM | see failure: http://logs.openstack.org/20/316520/3/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/52af87a/console.html#_2016-05-18_20_28_10_265 | 21:30 |
| EmilienM | vkmc: ^ | 21:31 |
| flwang | EmilienM: clicking... | 21:33 |
| EmilienM | zaqar logs are in http://logs.openstack.org/20/316520/3/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/52af87a/logs/zaqar | 21:34 |
| flwang | EmilienM: so it's breaking the tempest cases | 21:35 |
| *** flwang1 has joined #openstack-zaqar | 21:35 | |
| flwang | it's weird since it can pass the tempest gate, am i missing something? | 21:35 |
| EmilienM | are you running tempest in your gate? | 21:35 |
| flwang | EmilienM: ah, i see. I have proposed the patch, but not yet. That makes much more sense. | 21:37 |
| flwang | EmilienM: i will fix it, thanks for the heads up | 21:37 |
| flwang | and that's a good example, we should enable tempest in our gate asap | 21:38 |
| EmilienM | yeah.. please :-) | 21:39 |
| EmilienM | flwang: please let me know (email or IRC) when the fix is up | 21:39 |
| EmilienM | we're going to disable zaqar testing in the meantime :( | 21:39 |
| flwang | EmilienM: sorry? do you mean you're going to disable zaqar testing no matter if we fix the bug? | 21:40 |
| flwang | or just for now? | 21:40 |
| EmilienM | just tonight :-) | 21:40 |
| flwang | EmilienM: awesome, thanks | 21:40 |
| EmilienM | no worries, we want to keep testing zaqar :P | 21:41 |
| flwang | EmilienM: cooooooooool | 21:41 |
| EmilienM | flwang: do you have a launchpad about this problem? | 21:42 |
| EmilienM | or any review in gerrit already? | 21:43 |
| EmilienM | so I can monitor it | 21:43 |
| flwang | EmilienM: for this, no, i'm going to file one | 21:43 |
| EmilienM | ok I'll wait before posting the patch to disable tests, please let me know URL | 21:43 |
| flwang | EmilienM: sure | 21:44 |
| therve | flwang, You shouldn't add a new gate with tempest. You may want to move in-tree integration tests to tempest, though | 21:46 |
| EmilienM | therve: right | 21:47 |
| EmilienM | puppet CI is using in-tree tests | 21:47 |
| flwang | therve: that's another option right? | 21:53 |
| therve | flwang, Sorry? | 21:53 |
| flwang | therve: so you mean just using the tempest test replace current integration test? | 21:54 |
| flwang | now our tox-integration is calling the tests under zaqar.tests.functional | 21:55 |
| therve | flwang, Kinda. I mean reusing test infrastructure from tempest, so that when you run zaqar integration tests with tempest runner | 21:56 |
| flwang | so it it possible executing the tempest test as well? are you talking about this? | 21:56 |
| therve | Basically all the tests should be in-tree is my point | 21:56 |
| flwang | therve: yep, it's in tree. see https://review.openstack.org/302479 | 21:56 |
| EmilienM | flwang: I'm going to bed in 2 min | 21:56 |
| EmilienM | I'll push without launchpad URL, let me know when you did it | 21:57 |
| flwang | EmilienM: here is link https://bugs.launchpad.net/zaqar/+bug/1583358 | 21:57 |
| openstack | Launchpad bug 1583358 in zaqar "Zaqar tempest test failed" [Undecided,New] | 21:57 |
| EmilienM | ah :) | 21:57 |
| flwang | EmilienM: sorry, was in a standup meeting | 21:57 |
| EmilienM | thx, good night | 21:57 |
| flwang | good night | 21:57 |
| therve | I guess I misunderstood the problem then :) | 21:58 |
| flwang | therve: :) i think the problem is | 21:59 |
| flwang | we merged your patch, but it broke our tempest test | 21:59 |
| flwang | however, we haven't enable tempest in our gate | 21:59 |
| flwang | so we're not aware of that issue | 21:59 |
| flwang | but Puppet CI is using zaqar's tempest, so Puppet CI failed :) | 22:00 |
| EmilienM | we're running tempest in our gate | 22:00 |
| EmilienM | so we test what you're doing | 22:01 |
| flwang | EmilienM: yep, that's cool | 22:01 |
| flwang | at least, we have a gate guard :D | 22:01 |
| therve | OK that makes a bit more sense | 22:03 |
| therve | Though I don't understand why they fail :) | 22:03 |
| flwang | therve: me too, but i'm going to figure it out :D | 22:05 |
| flwang | therve: btw, do you like Zane's idea about zaqar + mistral? | 22:05 |
| therve | I haven't digested it yet | 22:05 |
| flwang | therve: http://lists.openstack.org/pipermail/openstack-dev/2016-May/095376.html | 22:06 |
| therve | Yeah I saw I just need some time :) | 22:09 |
| therve | Totally agree with his point about webhooks being tricky though | 22:09 |
| therve | (And zaqar implementation is suboptimal at best) | 22:10 |
| flwang | therve: cool, glad to know you like the idea, so did i | 22:10 |
| therve | I didn't say that, did I? | 22:11 |
| flwang | haha | 22:13 |
| flwang | therve: anyway, pls drop your comments no matter you like or not :) thanks | 22:14 |
| flwang | therve: since your opinion are invaluable for us | 22:14 |
| therve | That's very nice of you | 22:14 |
| therve | My opinion is not even invaluable to me :) | 22:15 |
| flwang | therve: btw, is it true that heat will drop out the autoscaling stuff? | 22:15 |
| flwang | and which will be done in Senlin? | 22:15 |
| therve | Not that I know | 22:16 |
| therve | We may not improve autoscaling much more, though | 22:16 |
| flwang | therve: btw, do you think it's a good idea to let aodh talk with zaqar and then let zaqar to trigger heat's autoscaling? | 22:18 |
| flwang | the benefit is you got a queue and the trigger can be tried(will do it in Newton) | 22:18 |
| flwang | i just wondering if we(zaqar team) should pay effort on this case to improve current autoscaling | 22:19 |
| therve | It's somewhat interesting, but not really important IMHO | 22:22 |
| flwang | ok, got, thanks | 22:23 |
| therve | flwang, BTW the tempest failure is because they're using falcon 0.3 | 22:27 |
| therve | So we wouldn't have caught it the gate anway | 22:27 |
| therve | anywa | 22:27 |
| flwang | therve: oh, really, why? | 22:27 |
| therve | Because we use 1.0 | 22:27 |
| flwang | i assume when they installed zaqar, the new falcon version will be installed | 22:28 |
| *** itisha has quit IRC | 22:29 | |
| therve | It's possible they're using packages and the new version is not package yet | 22:31 |
| therve | packaged | 22:31 |
| flwang | therve: possibly, but i think we still should figure out a good way so that the code can work with 0.3 | 22:33 |
| flwang | btw | 22:33 |
| flwang | do you think we should backport the patch or just bump the falcon version for those stable releases? | 22:33 |
| therve | You mean restrict? | 22:33 |
| flwang | mitaka and the old ones | 22:33 |
| flwang | yep | 22:34 |
| therve | Your call :) | 22:34 |
| therve | Mayb we need to fix working with 0.3 before backporting | 22:34 |
| therve | Or not backport and enforce 1.0 in master | 22:34 |
| flwang | therve: yep, that's my plan, if we can figure out a way to let the code work with 0.3, then all good | 22:34 |
| *** Eva-i has quit IRC | 22:35 | |
| therve | It may be tricky, given that extra parameter | 22:35 |
| flwang | therve: yep :( | 22:35 |
| therve | Enforcing 1.0 in master sounds fine to me | 22:35 |
| flwang | but if we don't backport, we have to restrict the falcon version for all the old releases | 22:35 |
| therve | We need to anyway | 22:36 |
| therve | Because the code won't work for < 0.3 | 22:36 |
| flwang | i see. so we need restrict it like falcon>=0.1.6 and <1.0.0 | 22:38 |
| flwang | then bump master with >=1.0.0 | 22:38 |
| flwang | if we can't figure out a way to let the code work with the old verions :( | 22:39 |
| therve | Yeah I think that would work | 22:39 |
| *** pt_15 has quit IRC | 22:46 | |
| flwang | therve: cool, thanks | 22:55 |
| *** mpanetta_ has joined #openstack-zaqar | 23:20 | |
| *** mpanetta_ has quit IRC | 23:20 | |
| *** mpanetta_ has joined #openstack-zaqar | 23:21 | |
| openstackgerrit | Fei Long Wang proposed openstack/zaqar: Restrict falcon version https://review.openstack.org/318371 | 23:26 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!