01:59:39 #startmeeting zaqar 01:59:40 Meeting started Tue Jan 10 01:59:39 2017 UTC and is due to finish in 60 minutes. The chair is flwang. Information about MeetBot at http://wiki.debian.org/MeetBot. 01:59:41 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 01:59:43 The meeting name has been set to 'zaqar' 01:59:56 hello 02:00:06 wxy_: hello team 02:00:13 it's our first meeting in 2017 02:00:14 hello everyone 02:00:19 and at the new time 02:00:35 my first time for zaqar IRC meeting. :) 02:00:42 haha 02:01:24 as therve said, we may don't really need a regular weekly meeting 02:01:54 but i think it would be nice to have one to keep us on the same page for the project status 02:02:11 agree 02:02:21 flwang: +1 02:02:42 the meeting won't be too long, basically used to sync the staus, we won't use it to discuss tech details 02:02:51 ok, here is the agenda for today's meeting 02:03:07 #link: https://etherpad.openstack.org/p/zaqar-meeting-agenda 02:03:16 the swift driver 02:03:44 is there any spec for swift driver? 02:04:02 https://review.openstack.org/403762 02:04:32 the spec is not long, i think it just because everybody is familiar with swift 02:04:57 so we don't have to worry about the license and the other tech concerns 02:05:12 especially gnocchi is also using swift as one of the backends recently 02:05:55 ok, will take a look after meeting. 02:06:04 flwang: It will be merged in O, right? 02:06:05 And Thomas 02:06:17 And Thomas's patch works well for me. 02:06:23 wxy_: cool 02:06:33 wanghao: yes, it will be merged in O based on the plan 02:06:40 I'm going to +2 for the spec 02:06:43 flwang: ok 02:06:44 the tempest test passed as well. 02:06:50 but it would be nice if you guys can take a look as well 02:07:07 flwang: yes, sure 02:07:10 the patch and spec do need more work 02:07:21 but i can't see any blocker 02:07:33 If we are both ok with the code frame. I think it can be merged soon. 02:07:40 we can surely merge it and improve it later and tag it as experimental 02:08:15 flwang: emm, agree 02:08:29 the only concern for me is, the storage mode 02:08:45 now swift driver will save all the data into service tenant 02:09:03 for small size deployment, it should be fine 02:09:34 but for large scale, it may have performance issue 02:09:43 but it's hard to evaluate for now 02:10:17 we may need to plan some evaluation work for next cycle about the performance 02:10:53 flwang: yes 02:11:33 flwang: do we consider to use Rally to test the performance? 02:11:48 wanghao: we have a gate job 02:11:56 but it has failed for quite long time 02:12:12 wanghao: it would be nice if you can take a look if you have time 02:12:34 though we have a built-in benchmark tool, but it's not in gate 02:12:59 flwang: ok, 02:13:41 flwang: I noticed the gate job, non-voting now. 02:14:56 wanghao: yes 02:15:08 it's a very simple job, only test queue create IIRC 02:15:16 we should add more test cases for sure 02:15:36 ok, anything we need to discuss for the swift driver? 02:16:19 only ome. 02:16:21 one 02:17:18 just let you know that we still have enough time for O-3. We should be careful with the patch 02:17:45 I suggest everyone can test it locally. :) 02:18:27 wxy_: i have tested it last time, last time the queue metadata doesn't work for me and the TTL 02:18:48 i will test it more for sure, but i'm going to approve the spec 02:18:49 yeah, I saw the review comment 02:19:06 and another thing i would like to test is pooling 02:19:34 cool. That's all for me. 02:19:38 using swift and mongo/redis at the same time 02:20:01 wxy_: thanks for the heads up 02:20:09 #topic subscription confirmation 02:20:24 wanghao: would you mind updating the status of SC? 02:21:42 flwang: don't mind 02:22:43 wanghao: i mean now ;) 02:23:46 I can't say more about the html sample. because I'm not good at js. 02:23:51 flwang: sure, I added two html examples in Zaqar, But I haven't tested it yet. 02:24:37 the python code LGTM. 02:25:25 It's a little hard to test, I'd plan to build a web server in my devstack env to test them. 02:26:03 Do you guys have any idea for them ? 02:28:06 when you say testing you mean manual testing? 02:28:17 flwang: yes. 02:28:25 that should be easy, because you can easy setup a website 02:28:34 flwang: emm, yes 02:28:44 and the email server? 02:29:19 wxy_: you don't have to setup an email server 02:29:29 wxy_: not email server, I want to use gmail to test it. 02:29:37 you can just use your company's email server IIUC 02:29:51 yep, gmail is fine as well 02:29:54 oh, got it. 02:30:07 I'll help with the test as well. 02:30:17 wxy_: thanks. 02:30:17 in the email, user just get a link 02:30:27 and the link can be defined by the deployer 02:30:44 flwang: yes, it's a option configuration. 02:30:50 i will test it 02:31:11 another thing i mentioned that with wxy_ is 02:31:27 flwang: thanks 02:31:35 we're using capital case for those keywords of subscription confirmation feature 02:31:53 it's a little bit inconsistent 02:32:32 yes 02:32:48 wxy_: wanghao: could you guys think about how to work out a backward compatible way to fix it? 02:34:04 flwang: sure, I'd check it with wxy_. 02:34:50 flwang, thanks fro reminding me. 02:35:10 flwang, I forgot it almost. 02:35:27 wxy_: does that make sense for you? 02:36:05 I'll have a check with wanghao later. I think we can find a backward compatible way. Not very hard 02:36:18 cool 02:36:52 ok, any more for this topic? 02:37:25 about sql migration. do you want to say any thing? 02:38:49 ok, so let's move to open discussion 02:38:51 #topic open discussion 02:38:59 as for sql db migration 02:39:15 i found an issue, and i have proposed a fix https://review.openstack.org/416785 02:39:38 I saw that the grenade CI pathc is merged. But can't not see it works. 02:39:51 in zaqar code, we're using table name with capital char, like "Queues' instead of "queues" 02:40:01 any patch has run it? 02:40:07 wxy_: yep, that's what i'm going to investigate 02:40:29 flwang, cool. please go on. 02:41:19 wxy_: maybe i need read more docs about that job 02:41:32 but anyway, we're getting closer 02:43:31 ok 02:43:36 flwang: Do you want to change the table name with 'queues'? 02:44:33 wanghao: nope, because it may break the compatiblity 02:44:46 hence why i changed the migration script 02:46:05 flwang: But I'm thinking what's problem if we changed the name in migration script? 02:48:20 flwang, I suggest you could add the test code as well. 02:48:57 now the migration test is empty. 02:49:45 wxy_: what do yo mean the migration test? 02:49:58 for sql or including the other backend? 02:51:13 flwang: Like other projects, have some UT tests for db migration. 02:52:30 http://logs.openstack.org/85/416785/2/check/gate-zaqar-python27-ubuntu-xenial/4fafabe/testr_results.html.gz 02:52:50 zaqar.tests.unit.storage.sqlalchemy_migration.test_migrations.TestMigrationsMySQL the tests are just skipped. 02:55:13 did I miss something? Bad internet connection. 02:56:38 wxy: no 02:56:59 flwang: still around? 02:57:24 yes 02:57:33 ok, we're running out time 02:57:46 let's close the meeting and back to zaqar channel 02:57:54 sure. 02:57:59 flwang: ok 02:58:20 thank for joining 02:58:28 #endmeeting